diff options
| author | Jeroen Hofstee <jeroen@myspectrum.nl> | 2014-06-11 23:12:28 +0200 |
|---|---|---|
| committer | Tom Warren <twarren@nvidia.com> | 2014-06-19 09:18:05 -0700 |
| commit | c42ff090e41d4b9e50faa727bac490ee790f82f4 (patch) | |
| tree | 3635d1f0f333229ba6fbaa87281f7e2e86777a66 /arch/arm/cpu/armv7/tegra20 | |
| parent | 0a26e1d6c394aacbf1153977b7348d1dff85db3f (diff) | |
| download | blackbird-obmc-uboot-c42ff090e41d4b9e50faa727bac490ee790f82f4.tar.gz blackbird-obmc-uboot-c42ff090e41d4b9e50faa727bac490ee790f82f4.zip | |
tegra20: display: fix checking of return value
The calling code seems a bit in doubt about the return
value of fdtdec_lookup_phandle. Since it returns a negative
value on error (and fdt_node_offset_by_phandle as well),
check for that.
cc: Wei Ni <wni@nvidia.com>
Signed-off-by: Jeroen Hofstee <jeroen@myspectrum.nl>
Tested-by: Stephen Warren <swarren@nvidia.com>
Signed-off-by: Tom Warren <twarren@nvidia.com>
Diffstat (limited to 'arch/arm/cpu/armv7/tegra20')
| -rw-r--r-- | arch/arm/cpu/armv7/tegra20/display.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/arm/cpu/armv7/tegra20/display.c b/arch/arm/cpu/armv7/tegra20/display.c index 488f0c639d..fd77f3f0ef 100644 --- a/arch/arm/cpu/armv7/tegra20/display.c +++ b/arch/arm/cpu/armv7/tegra20/display.c @@ -328,7 +328,7 @@ static int tegra_display_decode_config(const void *blob, rgb = fdt_subnode_offset(blob, node, "rgb"); config->panel_node = fdtdec_lookup_phandle(blob, rgb, "nvidia,panel"); - if (!config->panel_node < 0) { + if (config->panel_node < 0) { debug("%s: Cannot find panel information\n", __func__); return -1; } |
