From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S964821AbbHIGp6 (ORCPT ); Sun, 9 Aug 2015 02:45:58 -0400 Received: from metis.ext.pengutronix.de ([92.198.50.35]:37572 "EHLO metis.ext.pengutronix.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932882AbbHIGp5 (ORCPT ); Sun, 9 Aug 2015 02:45:57 -0400 Date: Sun, 9 Aug 2015 08:45:46 +0200 From: Markus Pargmann To: Jonathan Cameron Cc: Srinivas Pandruvada , Vlad Dogaru , Paul Bolle , linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org, kernel@pengutronix.de Subject: Re: [PATCH v2 5/6] iio: bmg160: Separate i2c and core driver Message-ID: <20150809064546.GU19600@pengutronix.de> References: <1438686179-18635-1-git-send-email-mpa@pengutronix.de> <1438686179-18635-6-git-send-email-mpa@pengutronix.de> <55C62A16.3030004@kernel.org> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="FCPLy5NpE1Kdjj9y" Content-Disposition: inline In-Reply-To: <55C62A16.3030004@kernel.org> X-Sent-From: Pengutronix Hildesheim X-URL: http://www.pengutronix.de/ X-IRC: #ptxdist @freenode X-Accept-Language: de,en X-Accept-Content-Type: text/plain X-Uptime: 08:44:07 up 7 days, 10:18, 54 users, load average: 0.42, 0.46, 0.39 User-Agent: Mutt/1.5.23 (2014-03-12) X-SA-Exim-Connect-IP: 2001:67c:670:100:1d::7 X-SA-Exim-Mail-From: mpa@pengutronix.de X-SA-Exim-Scanned: No (on metis.ext.pengutronix.de); SAEximRunCond expanded to false X-PTX-Original-Recipient: linux-kernel@vger.kernel.org Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --FCPLy5NpE1Kdjj9y Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Sat, Aug 08, 2015 at 05:11:02PM +0100, Jonathan Cameron wrote: > On 04/08/15 12:02, Markus Pargmann wrote: > > This patch separates the core driver using regmap and the i2c driver > > which creates the i2c regmap. Also in the Kconfig file BMG160 and > > BMG160_I2C are separate now. > >=20 > > Signed-off-by: Markus Pargmann > Don't you need to EXPORT the various functions to be used > across the modules? Yes, thanks. Will fix it. Best Regards, Markus > > --- > > drivers/iio/gyro/Kconfig | 18 +++++-- > > drivers/iio/gyro/Makefile | 3 +- > > drivers/iio/gyro/bmg160.h | 10 ++++ > > drivers/iio/gyro/{bmg160.c =3D> bmg160_core.c} | 77 ++++--------------= ---------- > > drivers/iio/gyro/bmg160_i2c.c | 73 ++++++++++++++++++++= ++++++ > > 5 files changed, 110 insertions(+), 71 deletions(-) > > create mode 100644 drivers/iio/gyro/bmg160.h > > rename drivers/iio/gyro/{bmg160.c =3D> bmg160_core.c} (94%) > > create mode 100644 drivers/iio/gyro/bmg160_i2c.c > >=20 > > diff --git a/drivers/iio/gyro/Kconfig b/drivers/iio/gyro/Kconfig > > index b44984b3bb24..1e6cfff5546f 100644 > > --- a/drivers/iio/gyro/Kconfig > > +++ b/drivers/iio/gyro/Kconfig > > @@ -51,18 +51,28 @@ config ADXRS450 > > will be called adxrs450. > > =20 > > config BMG160 > > - tristate "BOSCH BMG160 Gyro Sensor" > > - depends on I2C > > + bool "BOSCH BMG160 Gyro Sensor" > > select IIO_BUFFER > > select IIO_TRIGGERED_BUFFER > > select REGMAP > > - select REGMAP_I2C > > help > > Say yes here to build support for Bosch BMG160 Tri-axis Gyro Sensor > > driver. This driver also supports BMI055 gyroscope. > > =20 > > +if BMG160 > > + > > +config BMG160_I2C > > + tristate "Driver for connection via I2C" > > + depends on I2C > > + select REGMAP_I2C > > + help > > + Say yes here to build support for Bosch BMG160 Tri-axis Gyro Sensor > > + driver connected via I2C. This driver also supports BMI055 gyroscop= e. > > + > > This driver can also be built as a module. If so, the module > > - will be called bmg160. > > + will be called bmg160_i2c. > > + > > +endif > > =20 > > config HID_SENSOR_GYRO_3D > > depends on HID_SENSOR_HUB > > diff --git a/drivers/iio/gyro/Makefile b/drivers/iio/gyro/Makefile > > index f46341b39139..608401041550 100644 > > --- a/drivers/iio/gyro/Makefile > > +++ b/drivers/iio/gyro/Makefile > > @@ -8,7 +8,8 @@ obj-$(CONFIG_ADIS16130) +=3D adis16130.o > > obj-$(CONFIG_ADIS16136) +=3D adis16136.o > > obj-$(CONFIG_ADIS16260) +=3D adis16260.o > > obj-$(CONFIG_ADXRS450) +=3D adxrs450.o > > -obj-$(CONFIG_BMG160) +=3D bmg160.o > > +obj-$(CONFIG_BMG160) +=3D bmg160_core.o > > +obj-$(CONFIG_BMG160_I2C) +=3D bmg160_i2c.o > > =20 > > obj-$(CONFIG_HID_SENSOR_GYRO_3D) +=3D hid-sensor-gyro-3d.o > > =20 > > diff --git a/drivers/iio/gyro/bmg160.h b/drivers/iio/gyro/bmg160.h > > new file mode 100644 > > index 000000000000..72db723c8fb6 > > --- /dev/null > > +++ b/drivers/iio/gyro/bmg160.h > > @@ -0,0 +1,10 @@ > > +#ifndef BMG160_H_ > > +#define BMG160_H_ > > + > > +extern const struct dev_pm_ops bmg160_pm_ops; > > + > > +int bmg160_core_probe(struct device *dev, struct regmap *regmap, int i= rq, > > + const char *name); > > +void bmg160_core_remove(struct device *dev); > > + > > +#endif /* BMG160_H_ */ > > diff --git a/drivers/iio/gyro/bmg160.c b/drivers/iio/gyro/bmg160_core.c > > similarity index 94% > > rename from drivers/iio/gyro/bmg160.c > > rename to drivers/iio/gyro/bmg160_core.c > > index 865ef4c1c43f..ba37db5d491a 100644 > > --- a/drivers/iio/gyro/bmg160.c > > +++ b/drivers/iio/gyro/bmg160_core.c > > @@ -13,7 +13,6 @@ > > */ > > =20 > > #include > > -#include > > #include > > #include > > #include > > @@ -30,7 +29,6 @@ > > #include > > #include > > =20 > > -#define BMG160_DRV_NAME "bmg160" > > #define BMG160_IRQ_NAME "bmg160_event" > > #define BMG160_GPIO_NAME "gpio_int" > > =20 > > @@ -137,14 +135,6 @@ static const struct { > > { 133, BMG160_RANGE_250DPS}, > > { 66, BMG160_RANGE_125DPS} }; > > =20 > > -struct regmap_config bmg160_regmap_i2c_conf =3D { > > - .reg_bits =3D 8, > > - .val_bits =3D 8, > > - > > - .use_single_rw =3D true, > > - .cache_type =3D REGCACHE_NONE, > > -}; > > - > > static int bmg160_set_mode(struct bmg160_data *data, u8 mode) > > { > > int ret; > > @@ -998,31 +988,22 @@ static const char *bmg160_match_acpi_device(struc= t device *dev) > > return dev_name(dev); > > } > > =20 > > -static int bmg160_probe(struct i2c_client *client, > > - const struct i2c_device_id *id) > > +int bmg160_core_probe(struct device *dev, struct regmap *regmap, int i= rq, > > + const char *name) > > { > > struct bmg160_data *data; > > struct iio_dev *indio_dev; > > int ret; > > - const char *name =3D NULL; > > - struct regmap *regmap; > > - struct device *dev =3D &client->dev; > > - > > - regmap =3D devm_regmap_init_i2c(client, &bmg160_regmap_i2c_conf); > > - if (IS_ERR(regmap)) { > > - dev_err(&client->dev, "Failed to register i2c regmap %d\n", > > - (int)PTR_ERR(regmap)); > > - return PTR_ERR(regmap); > > - } > > =20 > > - indio_dev =3D devm_iio_device_alloc(&client->dev, sizeof(*data)); > > + indio_dev =3D devm_iio_device_alloc(dev, sizeof(*data)); > > if (!indio_dev) > > return -ENOMEM; > > =20 > > data =3D iio_priv(indio_dev); > > dev_set_drvdata(dev, indio_dev); > > data->dev =3D dev; > > - data->irq =3D client->irq; > > + data->irq =3D irq; > > + data->regmap =3D regmap; > > =20 > > ret =3D bmg160_chip_init(data); > > if (ret < 0) > > @@ -1030,9 +1011,6 @@ static int bmg160_probe(struct i2c_client *client, > > =20 > > mutex_init(&data->mutex); > > =20 > > - if (id) > > - name =3D id->name; > > - > > if (ACPI_HANDLE(dev)) > > name =3D bmg160_match_acpi_device(dev); > > =20 > > @@ -1128,14 +1106,14 @@ err_trigger_unregister: > > return ret; > > } > > =20 > > -static int bmg160_remove(struct i2c_client *client) > > +void bmg160_core_remove(struct device *dev) > > { > > - struct iio_dev *indio_dev =3D i2c_get_clientdata(client); > > + struct iio_dev *indio_dev =3D dev_get_drvdata(dev); > > struct bmg160_data *data =3D iio_priv(indio_dev); > > =20 > > - pm_runtime_disable(&client->dev); > > - pm_runtime_set_suspended(&client->dev); > > - pm_runtime_put_noidle(&client->dev); > > + pm_runtime_disable(dev); > > + pm_runtime_set_suspended(dev); > > + pm_runtime_put_noidle(dev); > > =20 > > iio_device_unregister(indio_dev); > > iio_triggered_buffer_cleanup(indio_dev); > > @@ -1148,8 +1126,6 @@ static int bmg160_remove(struct i2c_client *clien= t) > > mutex_lock(&data->mutex); > > bmg160_set_mode(data, BMG160_MODE_DEEP_SUSPEND); > > mutex_unlock(&data->mutex); > > - > > - return 0; > > } > GIven use across a module boundary I'd have thought an EXPORT was needed. > > =20 > > #ifdef CONFIG_PM_SLEEP > > @@ -1212,40 +1188,9 @@ static int bmg160_runtime_resume(struct device *= dev) > > } > > #endif > > =20 > > -static const struct dev_pm_ops bmg160_pm_ops =3D { > > +const struct dev_pm_ops bmg160_pm_ops =3D { > > SET_SYSTEM_SLEEP_PM_OPS(bmg160_suspend, bmg160_resume) > > SET_RUNTIME_PM_OPS(bmg160_runtime_suspend, > > bmg160_runtime_resume, NULL) > > }; > > =20 > > -static const struct acpi_device_id bmg160_acpi_match[] =3D { > > - {"BMG0160", 0}, > > - {"BMI055B", 0}, > > - {}, > > -}; > > - > > -MODULE_DEVICE_TABLE(acpi, bmg160_acpi_match); > > - > > -static const struct i2c_device_id bmg160_id[] =3D { > > - {"bmg160", 0}, > > - {"bmi055_gyro", 0}, > > - {} > > -}; > > - > > -MODULE_DEVICE_TABLE(i2c, bmg160_id); > > - > > -static struct i2c_driver bmg160_driver =3D { > > - .driver =3D { > > - .name =3D BMG160_DRV_NAME, > > - .acpi_match_table =3D ACPI_PTR(bmg160_acpi_match), > > - .pm =3D &bmg160_pm_ops, > > - }, > > - .probe =3D bmg160_probe, > > - .remove =3D bmg160_remove, > > - .id_table =3D bmg160_id, > > -}; > > -module_i2c_driver(bmg160_driver); > > - > > -MODULE_AUTHOR("Srinivas Pandruvada "); > > -MODULE_LICENSE("GPL v2"); > > -MODULE_DESCRIPTION("BMG160 Gyro driver"); > > diff --git a/drivers/iio/gyro/bmg160_i2c.c b/drivers/iio/gyro/bmg160_i2= c.c > > new file mode 100644 > > index 000000000000..4a39dff13cf2 > > --- /dev/null > > +++ b/drivers/iio/gyro/bmg160_i2c.c > > @@ -0,0 +1,73 @@ > > +#include > > +#include > > +#include > > +#include > > +#include > > + > > +#include "bmg160.h" > > + > > +static const struct regmap_config bmg160_regmap_i2c_conf =3D { > > + .reg_bits =3D 8, > > + .val_bits =3D 8, > > + > > + .use_single_rw =3D true, > > + .cache_type =3D REGCACHE_NONE, > > +}; > > + > > +static int bmg160_i2c_probe(struct i2c_client *client, > > + const struct i2c_device_id *id) > > +{ > > + struct regmap *regmap; > > + const char *name =3D NULL; > > + > > + regmap =3D devm_regmap_init_i2c(client, &bmg160_regmap_i2c_conf); > > + if (IS_ERR(regmap)) { > > + dev_err(&client->dev, "Failed to register i2c regmap %d\n", > > + (int)PTR_ERR(regmap)); > > + return PTR_ERR(regmap); > > + } > > + > > + if (id) > > + name =3D id->name; > > + > > + return bmg160_core_probe(&client->dev, regmap, client->irq, name); > > +} > > + > > +static int bmg160_i2c_remove(struct i2c_client *client) > > +{ > > + bmg160_core_remove(&client->dev); > > + > > + return 0; > > +} > > + > > +static const struct acpi_device_id bmg160_acpi_match[] =3D { > > + {"BMG0160", 0}, > > + {"BMI055B", 0}, > > + {}, > > +}; > > + > > +MODULE_DEVICE_TABLE(acpi, bmg160_acpi_match); > > + > > +static const struct i2c_device_id bmg160_i2c_id[] =3D { > > + {"bmg160", 0}, > > + {"bmi055_gyro", 0}, > > + {} > > +}; > > + > > +MODULE_DEVICE_TABLE(i2c, bmg160_i2c_id); > > + > > +static struct i2c_driver bmg160_i2c_driver =3D { > > + .driver =3D { > > + .name =3D "bmg160_i2c", > > + .acpi_match_table =3D ACPI_PTR(bmg160_acpi_match), > > + .pm =3D &bmg160_pm_ops, > > + }, > > + .probe =3D bmg160_i2c_probe, > > + .remove =3D bmg160_i2c_remove, > > + .id_table =3D bmg160_i2c_id, > > +}; > > +module_i2c_driver(bmg160_i2c_driver); > > + > > +MODULE_AUTHOR("Srinivas Pandruvada "); > > +MODULE_LICENSE("GPL v2"); > > +MODULE_DESCRIPTION("BMG160 I2C Gyro driver"); > >=20 >=20 >=20 --=20 Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | --FCPLy5NpE1Kdjj9y Content-Type: application/pgp-signature; name="signature.asc" Content-Description: Digital signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQIcBAEBAgAGBQJVxvcaAAoJEEpcgKtcEGQQWb4QALNRgUBspMMkGy3KwqkN9VLH PEQvrWa2kXScRF0ktD4EP+QMBN68deQWHqKuJHrCJz51tqu4CjKrIlI/1AHIwlnV 7lUmkCiudhTFr8urTSZFD31twHgS0957DCEtPROQC8ng4ZGD7nTweZUXYPVlxuAw ovnDC5xH82M9bKu7x/oBCHsthnOf+hBI2dxaSeTLB/MqzDxNEmpG3B6VplXOQqkQ TuddsqqFe2DIM39f4tx1Sg4t2K3u5R6pJHBlId0ZVepXQfj6yPZBXR0zkxTHxCY/ oqA89ivupRjA4dMJ7tk22TM6X5juujOUpIRL16FMnBubE4gQL+9BUt55dTgtsFj1 Syi4cLdG/szUKZHiT8i1WmET/caYmEtUqYmfmK1MAODiQMQ3V5BEQOZfDUM+2Cv1 5BcJZfsITgy7xMe8GKK1IIYZ6T0bhnrE7exelT3WOJPEmG0BYD1qh4H1mwAw3yom 1cXmJh9JedMvnFT7ki1naxChODCII7dvLnKpIEU8qz7dRR3DllnNvQZu0GxLi5yy Dz6rOtkPHUCwkXRG5Ihg09BqTVrvZXZj0oHPMeIbqE6gAchydEexU6PW9D223LG7 5laZaAb48+V/0RDS7Xio3ubC3SLMzTci1bQfBK6VqzMcawadw2owHUFf/MTVkNwA 22vZo1oPXgsI9JTkgKpz =go4Z -----END PGP SIGNATURE----- --FCPLy5NpE1Kdjj9y--