diff options
author | Mallikarjun Kasoju <mkasoju@nvidia.com> | 2013-11-22 15:40:12 +0530 |
---|---|---|
committer | Laxman Dewangan <ldewangan@nvidia.com> | 2013-11-25 01:36:06 -0800 |
commit | e08c5b75b14b81ddacfd423727a87e7a020934ca (patch) | |
tree | 5f29cd8fdbc2394170d5482febc43a644129c8c4 /arch/arm/mach-tegra/panel-s-edp-uhdtv-15-6.c | |
parent | 2bd68779e8a2c04ea2f6aaf34884765988d69fb0 (diff) |
ARM: tegra: fix regulator_get() return value check
regulator_get() returns either error pointer or valid pointers.
It never returns NULL. Hence it is not a correct to check
the return pointer as IS_ERR_OR_NULL(). It should be only IS_ERR().
Bug 1410234
Change-Id: I8cf668ca0bced49ee4f47ccf9b02d70fd0263cd6
Signed-off-by: Mallikarjun Kasoju <mkasoju@nvidia.com>
Reviewed-on: http://git-master/r/334471
GVS: Gerrit_Virtual_Submit
Reviewed-by: Laxman Dewangan <ldewangan@nvidia.com>
Diffstat (limited to 'arch/arm/mach-tegra/panel-s-edp-uhdtv-15-6.c')
-rw-r--r-- | arch/arm/mach-tegra/panel-s-edp-uhdtv-15-6.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/arch/arm/mach-tegra/panel-s-edp-uhdtv-15-6.c b/arch/arm/mach-tegra/panel-s-edp-uhdtv-15-6.c index bf2d8ea96382..7639b03b8e71 100644 --- a/arch/arm/mach-tegra/panel-s-edp-uhdtv-15-6.c +++ b/arch/arm/mach-tegra/panel-s-edp-uhdtv-15-6.c @@ -51,7 +51,7 @@ static int shield_edp_regulator_get(struct device *dev) return 0; vdd_ds_1v8 = regulator_get(dev, "vdd_ds_1v8"); - if (IS_ERR_OR_NULL(vdd_ds_1v8)) { + if (IS_ERR(vdd_ds_1v8)) { pr_err("vdd_ds_1v8 regulator get failed\n"); err = PTR_ERR(vdd_ds_1v8); vdd_ds_1v8 = NULL; @@ -60,7 +60,7 @@ static int shield_edp_regulator_get(struct device *dev) /* backlight */ vdd_lcd_bl_en = regulator_get(dev, "vdd_lcd_bl_en"); - if (IS_ERR_OR_NULL(vdd_lcd_bl_en)) { + if (IS_ERR(vdd_lcd_bl_en)) { pr_err("vdd_lcd_bl_en regulator get failed\n"); err = PTR_ERR(vdd_lcd_bl_en); vdd_lcd_bl_en = NULL; @@ -69,7 +69,7 @@ static int shield_edp_regulator_get(struct device *dev) /* lcd */ avdd_lcd = regulator_get(dev, "avdd_lcd"); - if (IS_ERR_OR_NULL(avdd_lcd)) { + if (IS_ERR(avdd_lcd)) { pr_err("avdd_lcd regulator get failed\n"); err = PTR_ERR(avdd_lcd); avdd_lcd = NULL; @@ -78,7 +78,7 @@ static int shield_edp_regulator_get(struct device *dev) /* LCD_RST */ avdd_3v3_dp = regulator_get(dev, "avdd_3v3_dp"); - if (IS_ERR_OR_NULL(avdd_3v3_dp)) { + if (IS_ERR(avdd_3v3_dp)) { pr_err("avdd_3v3_dp regulator get failed\n"); err = PTR_ERR(avdd_3v3_dp); avdd_3v3_dp = NULL; |