RE: [PATCH] staging:iio: gpio build dependency fixing

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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


[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux