generic-adc-battery: Fix checking if none of the channels are supported
authorAxel Lin <axel.lin@ingics.com>
Fri, 24 May 2013 17:41:44 +0000 (01:41 +0800)
committerAnton Vorontsov <anton@enomsg.org>
Fri, 7 Jun 2013 00:33:40 +0000 (17:33 -0700)
If none of the channels are supported, index is 0. Also ensure to return
error code instead of 0 in goto second_mem_fail path.

Signed-off-by: Axel Lin <axel.lin@ingics.com>
Acked-by: anish kumar <anish198519851985@gmail.com>
Signed-off-by: Anton Vorontsov <anton@enomsg.org>
drivers/power/generic-adc-battery.c

index 8cb5d7f67acebac1e60cf5006ae857835e8161cb..59a1421f92885ea8fa98f0468b9a8b5660010c65 100644 (file)
@@ -299,8 +299,10 @@ static int gab_probe(struct platform_device *pdev)
        }
 
        /* none of the channels are supported so let's bail out */
-       if (index == ARRAY_SIZE(gab_chan_name))
+       if (index == 0) {
+               ret = -ENODEV;
                goto second_mem_fail;
+       }
 
        /*
         * Total number of properties is equal to static properties