Re: [PATCH v3 0/4] iio: bmc150 regmap and SPI

[Jonathan Cameron <jic23@kernel.org>]

On 24/09/15 08:11, Markus Pargmann wrote:
> Hi Irina,
> 
> On Wed, Sep 23, 2015 at 12:46:04PM +0000, Tirdea, Irina wrote:
>>
>>
>>> -----Original Message-----
>>> From: Markus Pargmann [mailto:mpa@pengutronix.de]
>>> Sent: 21 September, 2015 13:55
>>> To: Jonathan Cameron
>>> Cc: Srinivas Pandruvada; Tirdea, Irina; Lars-Peter Clausen; linux-iio@vger.kernel.org; linux-kernel@vger.kernel.org;
>>> kernel@pengutronix.de; Markus Pargmann
>>> Subject: [PATCH v3 0/4] iio: bmc150 regmap and SPI
>>>
>>> Hi,
>>>
>>
>> Hi Markus,
>>
>> I tested the new version of you patches and everything works fine.
>>
>> I used a BMA250E chip connected on the i2c bus.
>> The tests included the iio buffer code path and the i2c code path
>> (including using the fifo and forcing the i2c bus to use
>> the regmap_i2c_smbus_i2c_block calls you added to regmap).
>>
>>> this series converts the bmc150 driver to use regmap and adds an SPI interface.
>>>
>>> Thanks for testing and review so far. I rebased the series onto v4.3-rc2 now
>>> (the togreg branch seems to be on v4.2).
>>> It still works for me but there were some differences regarding the chip id.
>>>
>>
>> I actually used the togreg branch (to get the latest bmc150 driver changes) and
>> cherry-picked the regmap patches. Everything applied without any conflicts.
> 
> Thank you.
> 
> It is probably best if I rebase this onto togreg then as soon as it is
> based on v4.3 to have the necessary regmap dependencies.
As Irina said it went on fairly clean I applied it from these
(before making my  v4.3 tree available publicly - it is now).

Anyhow, was fairly straight forward (I think) as such things
go.

Thanks Irina for testing and Markus for your hard work getting
this in.

Jonathan
> 
> Best Regards,
> 
> Markus
> 
>>
>> Thanks,
>> Irina
>>
>>> Changes in v3:
>>> - Fixed type of variable 'step' which lead to compile warnings. Type is now
>>>   size_t.
>>> - Fixed patch that moved irq variable without reason
>>> - Readded MODULE_* to the core driver
>>> - Reintroduced check id NULL check
>>>
>>> Changes in v2:
>>> - Removed default values for regmap_config fields.
>>> - Redesigned the fifo_transfer function to avoid running in errors first.
>>> - Dropped irq checks patch as it is already mainline
>>> - Core can now be built as module with autoselection of i2c and spi parts
>>>
>>> As my hardware is missing an interrupt line from the SPI connected bmc150 I am
>>> not able to test the iio buffer code path and the i2c code path. Tests would be
>>> appreciated.
>>>
>>> @Srinivas:
>>> As there were some rebase conflicts on the first patch, I removed your
>>> reviewed-by tag again for the moment.
>>>
>>> Best regards,
>>>
>>> Markus
>>>
>>>
>>> Markus Pargmann (4):
>>>   iio: bmc150: Use i2c regmap
>>>   iio: bcm150: Remove i2c_client from private data
>>>   iio: bmc150: Split the driver into core and i2c
>>>   iio: bmc150: Add SPI driver
>>>
>>>  drivers/iio/accel/Kconfig                          |  14 +-
>>>  drivers/iio/accel/Makefile                         |   4 +-
>>>  .../accel/{bmc150-accel.c => bmc150-accel-core.c}  | 388 ++++++++-------------
>>>  drivers/iio/accel/bmc150-accel-i2c.c               | 102 ++++++
>>>  drivers/iio/accel/bmc150-accel-spi.c               |  80 +++++
>>>  drivers/iio/accel/bmc150-accel.h                   |  20 ++
>>>  6 files changed, 366 insertions(+), 242 deletions(-)
>>>  rename drivers/iio/accel/{bmc150-accel.c => bmc150-accel-core.c} (82%)
>>>  create mode 100644 drivers/iio/accel/bmc150-accel-i2c.c
>>>  create mode 100644 drivers/iio/accel/bmc150-accel-spi.c
>>>  create mode 100644 drivers/iio/accel/bmc150-accel.h
>>>
>>> --
>>> 2.5.1
>>
>>
>