All of lore.kernel.org
 help / color / mirror / Atom feed
From: Lars-Peter Clausen <lars@metafoo.de>
To: Markus Pargmann <mpa@pengutronix.de>,
	Jonathan Cameron <jic23@kernel.org>
Cc: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>,
	Vlad Dogaru <vlad.dogaru@intel.com>,
	Paul Bolle <pebolle@tiscali.nl>,
	linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org,
	kernel@pengutronix.de, Irina Tirdea <irina.tirdea@intel.com>
Subject: Re: [PATCH v3 0/6] iio: bmg160: Add SPI connection
Date: Wed, 12 Aug 2015 17:04:05 +0200	[thread overview]
Message-ID: <55CB6065.8000901@metafoo.de> (raw)
In-Reply-To: <1439391009-6051-1-git-send-email-mpa@pengutronix.de>

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).

- 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
> 


  parent reply	other threads:[~2015-08-12 15:04 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-08-12 14:50 [PATCH v3 0/6] iio: bmg160: Add SPI connection Markus Pargmann
2015-08-12 14:50 ` [PATCH v3 1/6] iio: bmg160: Use i2c regmap instead of direct i2c access Markus Pargmann
2015-08-12 16:01   ` Srinivas Pandruvada
2015-08-15 14:41     ` Jonathan Cameron
2015-08-15 14:43       ` Jonathan Cameron
2015-08-12 14:50 ` [PATCH v3 2/6] iio: bmg160: Remove i2c_client from data struct Markus Pargmann
2015-08-12 16:03   ` Srinivas Pandruvada
2015-08-15 14:50     ` Jonathan Cameron
2015-08-12 14:50 ` [PATCH v3 3/6] iio: bmg160: Use generic dev_drvdata Markus Pargmann
2015-08-12 16:05   ` Srinivas Pandruvada
2015-08-15 14:50     ` Jonathan Cameron
2015-08-12 14:50 ` [PATCH v3 4/6] iio: bmg160: Remove remaining uses of i2c_client Markus Pargmann
2015-08-12 16:08   ` Srinivas Pandruvada
2015-08-15 14:53     ` Jonathan Cameron
2015-08-12 14:50 ` [PATCH v3 5/6] iio: bmg160: Separate i2c and core driver Markus Pargmann
2015-08-12 16:16   ` Srinivas Pandruvada
2015-08-15 14:59     ` Jonathan Cameron
2015-08-12 14:50 ` [PATCH v3 6/6] iio: bmg160: Add SPI driver Markus Pargmann
2015-08-12 15:19   ` Peter Meerwald
2015-08-15 15:03     ` Jonathan Cameron
2015-08-17  6:41       ` Markus Pargmann
2015-08-17  6:39     ` Markus Pargmann
2015-08-12 15:04 ` Lars-Peter Clausen [this message]
2015-08-15 14:49   ` [PATCH v3 0/6] iio: bmg160: Add SPI connection Jonathan Cameron
2015-08-17  9:27   ` Tirdea, Irina

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=55CB6065.8000901@metafoo.de \
    --to=lars@metafoo.de \
    --cc=irina.tirdea@intel.com \
    --cc=jic23@kernel.org \
    --cc=kernel@pengutronix.de \
    --cc=linux-iio@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mpa@pengutronix.de \
    --cc=pebolle@tiscali.nl \
    --cc=srinivas.pandruvada@linux.intel.com \
    --cc=vlad.dogaru@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.