Re: [PATCH 2/2] iio: magnetometer: Add Infineon TLV493D-A1B6

2017-12-10 Thread Jonathan Cameron
On Sun, 10 Dec 2017 18:14:44 +0100 Andreas Färber wrote: > Hi, Some quick notes inline. Jonathan > > Am 10.12.2017 um 10:08 schrieb Peter Meerwald-Stadler: > >> The Infineon TLV493D is a 3D magnetic sensor, A1B6 is I2C based. > > > > comments below > > > >> It is found among others on t

Re: [PATCH 2/2] iio: magnetometer: Add Infineon TLV493D-A1B6

2017-12-10 Thread Andreas Färber
Hi, Am 10.12.2017 um 10:08 schrieb Peter Meerwald-Stadler: >> The Infineon TLV493D is a 3D magnetic sensor, A1B6 is I2C based. > > comments below > >> It is found among others on the Infineon 3D Magnetic Sensor 2Go Kit, >> which allows to detach the sensor as a breakout board. >> >> Cc: Marius

Re: [PATCH 2/2] iio: magnetometer: Add Infineon TLV493D-A1B6

2017-12-10 Thread Jonathan Cameron
On Sun, 10 Dec 2017 10:08:04 +0100 (CET) Peter Meerwald-Stadler wrote: > > The Infineon TLV493D is a 3D magnetic sensor, A1B6 is I2C based. > > comments below A few additions from me, but Peter got most stuff. You could make the driver safe for i2c DMA but it's borderline on whether any of th

Re: [PATCH 2/2] iio: magnetometer: Add Infineon TLV493D-A1B6

2017-12-10 Thread Peter Meerwald-Stadler
> The Infineon TLV493D is a 3D magnetic sensor, A1B6 is I2C based. comments below > It is found among others on the Infineon 3D Magnetic Sensor 2Go Kit, > which allows to detach the sensor as a breakout board. > > Cc: Marius Tarcatu > Signed-off-by: Andreas Färber > --- > drivers/iio/magnet

Re: [PATCH 2/2] iio: magnetometer: Add Infineon TLV493D-A1B6

2017-12-10 Thread Philippe Ombredanne
Andreas, On Sun, Dec 10, 2017 at 12:25 AM, Andreas Färber wrote: > The Infineon TLV493D is a 3D magnetic sensor, A1B6 is I2C based. > > It is found among others on the Infineon 3D Magnetic Sensor 2Go Kit, > which allows to detach the sensor as a breakout board. > > Cc: Marius Tarcatu > Signed-of

[PATCH 2/2] iio: magnetometer: Add Infineon TLV493D-A1B6

2017-12-09 Thread Andreas Färber
The Infineon TLV493D is a 3D magnetic sensor, A1B6 is I2C based. It is found among others on the Infineon 3D Magnetic Sensor 2Go Kit, which allows to detach the sensor as a breakout board. Cc: Marius Tarcatu Signed-off-by: Andreas Färber --- drivers/iio/magnetometer/Kconfig | 9 + drivers/