regulator: aat2870: Fix the logic of checking if no id is matched in aat2870_get_regu...
authorAxel Lin <axel.lin@gmail.com>
Mon, 28 Nov 2011 06:06:31 +0000 (14:06 +0800)
committerMark Brown <broonie@opensource.wolfsonmicro.com>
Mon, 28 Nov 2011 11:48:15 +0000 (11:48 +0000)
In current implementation, the pointer ri is not NULL if no id is matched.
Fix it by checking i == ARRAY_SIZE(aat2870_regulators) if no id is matched.

Signed-off-by: Axel Lin <axel.lin@gmail.com>
Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
Cc: stable@kernel.org
drivers/regulator/aat2870-regulator.c

index 5abeb3ac3e8da43df3d58bf677c3ac0e3d17116e..298c6c6a279574561eb24f329b115fe134d274a1 100644 (file)
@@ -160,7 +160,7 @@ static struct aat2870_regulator *aat2870_get_regulator(int id)
                        break;
        }
 
-       if (!ri)
+       if (i == ARRAY_SIZE(aat2870_regulators))
                return NULL;
 
        ri->enable_addr = AAT2870_LDO_EN;