Jonathan Cameron wrote on 2011-10-12: > Numerous drivers either had pointless includes of gpio.h > or should have been dependent on GENERIC_GPIO and were not. > Conversion of ads1210 to use array registration triggered > build failures that highlighted all was not well. > > Signed-off-by: Jonathan Cameron <jic23@xxxxxxxxx> > Reported-by: Randy Dunlap <rdunlap@xxxxxxxxxxxx> Acked-by: Michael Hennerich <michael.hennerich@xxxxxxxxxx> > --- > > Randy pointed out the issue with ad2s1210 triggered by > building without GENERIC_GPIO. This motivated a quick > use of git grep on all the drivers and this patch fixes > all cases where there should have been a dependency, > or where they was and shouldn't have been (be it implied > by gpio.h includes). > > One has to wonder what the point in having stubs for the > array calls is. I'll post on this point separately shortly. > > I have no idea what the sca3000 was checking those gpios > for. It is probably lost somewhere in history but clearly > doesn't make sense. > > Jonathan > drivers/staging/iio/accel/Kconfig | 1 + > drivers/staging/iio/accel/adis16204_core.c | 1 - > drivers/staging/iio/accel/sca3000_core.c | 7 +++---- > drivers/staging/iio/adc/Kconfig | 1 + > drivers/staging/iio/addac/Kconfig | 1 + > drivers/staging/iio/dac/ad5504.c | 1 - > drivers/staging/iio/dac/ad5624r_spi.c | 1 - > drivers/staging/iio/dac/ad5686.c | 1 - > drivers/staging/iio/dac/ad5791.c | 1 - > drivers/staging/iio/gyro/adis16080_core.c | 1 - > drivers/staging/iio/gyro/adis16260_core.c | 1 - > drivers/staging/iio/gyro/adxrs450_core.c | 1 - > drivers/staging/iio/magnetometer/Kconfig | 1 + > drivers/staging/iio/meter/ade7753.c | 1 - > drivers/staging/iio/meter/ade7754.c | 1 - > drivers/staging/iio/meter/ade7758_core.c | 1 - > drivers/staging/iio/meter/ade7759.c | 1 - > drivers/staging/iio/meter/ade7854.c | 1 - > drivers/staging/iio/resolver/Kconfig | 2 ++ > 19 files changed, 9 insertions(+), 17 deletions(-) Greetings, Michael -- Analog Devices GmbH Wilhelm-Wagenfeld-Str. 6 80807 Muenchen Sitz der Gesellschaft: Muenchen; Registergericht: Muenchen HRB 40368; Geschaeftsfuehrer:Dr.Carsten Suckrow, Thomas Wessel, William A. Martin, Margaret Seif -- To unsubscribe from this list: send the line "unsubscribe linux-iio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html