On 16/02/16 16:14, Andrew F. Davis wrote: > On 02/16/2016 09:52 AM, Arnd Bergmann wrote: >> The newly added afe4403 driver uses the regmap facility to abstract >> the I2C and SPI access. However, it fails to ensure that regmap_spi >> is actually present: >> >> drivers/iio/built-in.o: In function `afe4403_probe': >> :(.text+0x9bf8): undefined reference to `__devm_regmap_init_spi' >> >> This adds a Kconfig select statement like the afe4404 I2C driver >> has. >> >> Signed-off-by: Arnd Bergmann <arnd@xxxxxxxx> >> Fixes: eec96d1e2d31 ("iio: health: Add driver for the TI AFE4403 heart monitor") >> --- >> drivers/iio/health/Kconfig | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/drivers/iio/health/Kconfig b/drivers/iio/health/Kconfig >> index f0c19779ac27..c5f004a8e447 100644 >> --- a/drivers/iio/health/Kconfig >> +++ b/drivers/iio/health/Kconfig >> @@ -10,6 +10,7 @@ menu "Heart Rate Monitors" >> config AFE4403 >> tristate "TI AFE4403 Heart Rate Monitor" >> depends on SPI_MASTER >> + select REGMAP_SPI >> select IIO_BUFFER >> select IIO_TRIGGERED_BUFFER >> help >> > > Oversight on my part, thanks for fix. > > Acked-by: Andrew F. Davis <afd@xxxxxx> Applied to the togreg branch of iio.git - initially pushed out as testing for the autobuilders to fail to trigger similar issues... (ah well, they can't hammer everything!) Thanks Arnd! Jonathan -- 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