linux-kernel - Re: [PATCH v3 0/6] iio: bmg160: Add SPI connection

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Sat, 15 Aug 2015 15:49:15 +0100
From:	Jonathan Cameron <jic23@...nel.org>
To:	Lars-Peter Clausen <lars@...afoo.de>,
	Markus Pargmann <mpa@...gutronix.de>
Cc:	Srinivas Pandruvada <srinivas.pandruvada@...ux.intel.com>,
	Vlad Dogaru <vlad.dogaru@...el.com>,
	Paul Bolle <pebolle@...cali.nl>, linux-iio@...r.kernel.org,
	linux-kernel@...r.kernel.org, kernel@...gutronix.de,
	Irina Tirdea <irina.tirdea@...el.com>
Subject: Re: [PATCH v3 0/6] iio: bmg160: Add SPI connection

On 12/08/15 16:04, Lars-Peter Clausen wrote:
> Hi,
> 
> Markus and Irina can you try to synchronize your efforts. This series will
> conflict with http://marc.info/?l=linux-iio&m=143938994602171&w=2
> 
> I think it would be best if the multi-read emulation support is added to
> regmap (which I think Markus' earlier series did).
Good spot Lars, I'd managed to miss this clash entirely (and had both sets
marked for applying - though I'd marked them for such on totally different
days!)

Anyhow, I'm going to take this series for now and we'll have to deal with
the emulation stuff on top of it.

Jonathan
> 
> - Lars
> 
> On 08/12/2015 04:50 PM, Markus Pargmann wrote:
>> Hi,
>>
>> bmg160 and bmi055 can be used via I2C and SPI. This series converts the driver
>> to regmap and splits core driver and I2C/SPI.
>>
>> Changes in v3:
>>  - removed 'select REGMAP' as it is selected by REGMAP_I2C
>>  - added EXPORT_SYMBOL_GPL for the core functions
>>  - removed default values from regmap_config
>>  - Added max_register and unset use_single_rw in regmap_config
>>  - Changed Makefile to always compile bmg160-core with either spi or i2c. It is
>>    not possible now to compile the core alone.
>>
>> Changes in v2:
>>  - Added the id->name from the SPI driver to be used as iio device name
>>  - Fixed Kconfig in patch 2 to add selects for REGMAP_I2C
>>  - Fixed regmap configs to be static const
>>
>>
>> Best regards,
>>
>> Markus
>>
>>
>> Markus Pargmann (6):
>>   iio: bmg160: Use i2c regmap instead of direct i2c access
>>   iio: bmg160: Remove i2c_client from data struct
>>   iio: bmg160: Use generic dev_drvdata
>>   iio: bmg160: Remove remaining uses of i2c_client
>>   iio: bmg160: Separate i2c and core driver
>>   iio: bmg160: Add SPI driver
>>
>>  drivers/iio/gyro/Kconfig                     |  28 ++-
>>  drivers/iio/gyro/Makefile                    |   3 +-
>>  drivers/iio/gyro/bmg160.h                    |  10 +
>>  drivers/iio/gyro/{bmg160.c => bmg160_core.c} | 358 +++++++++++----------------
>>  drivers/iio/gyro/bmg160_i2c.c                |  71 ++++++
>>  drivers/iio/gyro/bmg160_spi.c                |  57 +++++
>>  6 files changed, 306 insertions(+), 221 deletions(-)
>>  create mode 100644 drivers/iio/gyro/bmg160.h
>>  rename drivers/iio/gyro/{bmg160.c => bmg160_core.c} (74%)
>>  create mode 100644 drivers/iio/gyro/bmg160_i2c.c
>>  create mode 100644 drivers/iio/gyro/bmg160_spi.c
>>
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-iio" in
> the body of a message to majordomo@...r.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/