lkml.org 
[lkml]   [2015]   [Oct]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH v3 0/4] iio: bmc150 regmap and SPI
From
Date
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
>>
>>
>



\
 
 \ /
  Last update: 2015-10-03 13:21    [W:0.106 / U:0.112 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site