Re: [patch] generic-adc-battery: forever loop in gab_remove()

2013-02-16 Thread Anton Vorontsov
On Thu, Feb 14, 2013 at 10:26:43AM +0300, Dan Carpenter wrote: > There is a forever loop calling iio_channel_release() because the > "chan < " part of the "chan < ARRAY_SIZE()" is missing. This is in both > the error handling on probe and also in the remove function. > > The other thing is that i

Re: [patch] generic-adc-battery: forever loop in gab_remove()

2013-02-14 Thread anish singh
On Thu, Feb 14, 2013 at 2:06 PM, anish singh wrote: > On Thu, Feb 14, 2013 at 12:56 PM, Dan Carpenter > wrote: >> There is a forever loop calling iio_channel_release() because the >> "chan < " part of the "chan < ARRAY_SIZE()" is missing. This is in both >> the error handling on probe and also i

Re: [patch] generic-adc-battery: forever loop in gab_remove()

2013-02-14 Thread anish singh
On Thu, Feb 14, 2013 at 12:56 PM, Dan Carpenter wrote: > There is a forever loop calling iio_channel_release() because the > "chan < " part of the "chan < ARRAY_SIZE()" is missing. This is in both > the error handling on probe and also in the remove function. > > The other thing is that it's poss

[patch] generic-adc-battery: forever loop in gab_remove()

2013-02-13 Thread Dan Carpenter
There is a forever loop calling iio_channel_release() because the "chan < " part of the "chan < ARRAY_SIZE()" is missing. This is in both the error handling on probe and also in the remove function. The other thing is that it's possible for some of the elements of the adc_bat->channel[chan] array