All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jonathan Cameron <jic23@kernel.org>
To: Anthony Olech <anthony.olech.opensource@diasemi.com>
Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>,
	Mauro Carvalho Chehab <mchehab@redhat.com>,
	Ashish Jangam <ashish.jangam@kpitcummins.com>,
	linux-iio@vger.kernel.org,
	Grant Likely <grant.likely@secretlab.ca>,
	Randy Dunlap <rdunlap@xenotime.net>,
	Jonathan Cameron <jic23@cam.ac.uk>,
	rtc-linux@googlegroups.com, lm-sensors@lm-sensors.org,
	Eric Andersson <eric.andersson@unixphere.com>,
	linux-input@vger.kernel.org, Axel Lin <axel.lin@gmail.com>,
	Liam Girdwood <lrg@ti.com>,
	Paul Gortmaker <p_gortmaker@yahoo.com>,
	Andrew Jones <drjones@redhat.com>,
	Michael Krufky <mkrufky@kernellabs.com>,
	Arnd Bergmann <arnd@arndb.de>,
	Samuel Ortiz <sameo@linux.intel.com>,
	Alessandro Zummo <a.zummo@towertech.it>,
	Linus Walleij <linus.walleij@stericsson.com>,
	Mark Brown <broonie@opensource.wolfsonmicro.com>,
	LKML <linux-kernel@vger.kernel.org>,
	Steven Toth <stoth@kernellabs.com>, Anton Vorontov <cb>
Subject: Re: [NEW DRIVER V3 0/8] DA9058 PMIC - please comment on this new driver
Date: Thu, 16 Aug 2012 20:10:15 +0100	[thread overview]
Message-ID: <502D4597.6080006@kernel.org> (raw)
In-Reply-To: <201208151518.q7FFI98a011599@latitude.olech.com>

You've obviously gone to a lot of effort to get the various parts of this patch
to go to the right people / lists.

There are however dependencies. I'd like to see the drivers that are dependent
on the iio one and obviously everyone may be interested in the core parts.

Hence I'd suggest just sending the whole series to everyone concerned.
We are all well practiced in the art of pressing delete if we don't want
to read something :)

(Given Lars-Peter has highlighted some bits and bobs in the adc driver
- and he always does a good job - I'll probably wait for the next version
before taking a look).

Good to see this driver making progress!

> This is submission attempt number 3 to have this driver included in
> the linux kernel source tree. This is the driver for the Dialog DA9058.
> 
> The DA9058 is a low power Power Management Integrated Circuit with extra
> functionality. It is a Multi Function Device controlled only from an I2C
> bus whose components can raise an interrupt request on a single IRQ line.
> 
> The driver for the DA9058 consists of a core (i2c) device driver that
> instantiates the individual component device drivers for:
> 
> adc - 5 ADC channels
> gpio - 2 available pins
> onkey - 1 device
> regulator - 4 BUCKS, 19 LDO and 3 fixed
> rtc - low power clock
> hwmon - 5 monitored voltages/temperatures
> power - battery information
> 
> All the above seven component device drivers depend on the 'core'
> component driver, which is number one in the patch series.
> 
> This driver has been tested on a Samsung SMDK6410 connected to a Dialog
> DA9058 Evaluation Board via one GPIO and a 3-wire I2C connection.
> 
> All the components can be builtin to the kernel or compiled as modules.
> As far as I can tell, all the latest APIs both for the core driver and
> all the component drivers have been adhered to, but if I have missed
> something please let me know.
> 
> Many thanks,
> Anthony Olech, Dialog Semiconductor Ltd.
> 
> Tony Olech (at Home) (8):
>   DA9058 MFD core driver
>   DA9058 ADC driver
>   DA9058 ONKEY driver
>   DA9058 POWER driver
>   DA9058 RTC driver
>   DA9058 GPIO driver
>   DA9058 HWMON driver
>   DA9058 REGULATOR driver
> 
>  Documentation/hwmon/da9058           |   30 ++
>  drivers/gpio/Kconfig                 |   12 +
>  drivers/gpio/Makefile                |    1 +
>  drivers/gpio/gpio-da9058.c           |  367 +++++++++++++
>  drivers/hwmon/Kconfig                |   10 +
>  drivers/hwmon/Makefile               |    1 +
>  drivers/hwmon/da9058-hwmon.c         |  425 +++++++++++++++
>  drivers/iio/adc/Kconfig              |   10 +
>  drivers/iio/adc/Makefile             |    1 +
>  drivers/iio/adc/da9058-adc.c         |  397 ++++++++++++++
>  drivers/input/misc/Kconfig           |   10 +
>  drivers/input/misc/Makefile          |    1 +
>  drivers/input/misc/da9058_onkey.c    |  177 +++++++
>  drivers/mfd/Kconfig                  |   18 +
>  drivers/mfd/Makefile                 |    3 +
>  drivers/mfd/da9058-core.c            |   83 +++
>  drivers/mfd/da9058-i2c.c             |   97 ++++
>  drivers/mfd/da9058-info.c            |  935 ++++++++++++++++++++++++++++++++++
>  drivers/mfd/da9058-irq.c             |   56 ++
>  drivers/power/Kconfig                |   10 +
>  drivers/power/Makefile               |    1 +
>  drivers/power/da9058_power.c         |  397 ++++++++++++++
>  drivers/regulator/Kconfig            |   11 +
>  drivers/regulator/Makefile           |    1 +
>  drivers/regulator/da9058-regulator.c |  251 +++++++++
>  drivers/rtc/Kconfig                  |   10 +
>  drivers/rtc/Makefile                 |    1 +
>  drivers/rtc/rtc-da9058.c             |  458 +++++++++++++++++
>  include/linux/mfd/da9058/bat.h       |   33 ++
>  include/linux/mfd/da9058/codec.h     |   22 +
>  include/linux/mfd/da9058/conf.h      |   22 +
>  include/linux/mfd/da9058/core.h      |   66 +++
>  include/linux/mfd/da9058/gpio.h      |   19 +
>  include/linux/mfd/da9058/hwmon.h     |   36 ++
>  include/linux/mfd/da9058/i2c.h       |   22 +
>  include/linux/mfd/da9058/irq.h       |   50 ++
>  include/linux/mfd/da9058/onkey.h     |   17 +
>  include/linux/mfd/da9058/pdata.h     |   32 ++
>  include/linux/mfd/da9058/registers.h |  480 +++++++++++++++++
>  include/linux/mfd/da9058/regulator.h |   33 ++
>  include/linux/mfd/da9058/rtc.h       |   17 +
>  41 files changed, 4623 insertions(+), 0 deletions(-)
>  create mode 100644 Documentation/hwmon/da9058
>  create mode 100644 drivers/gpio/gpio-da9058.c
>  create mode 100644 drivers/hwmon/da9058-hwmon.c
>  create mode 100644 drivers/iio/adc/da9058-adc.c
>  create mode 100644 drivers/input/misc/da9058_onkey.c
>  create mode 100644 drivers/mfd/da9058-core.c
>  create mode 100644 drivers/mfd/da9058-i2c.c
>  create mode 100644 drivers/mfd/da9058-info.c
>  create mode 100644 drivers/mfd/da9058-irq.c
>  create mode 100644 drivers/power/da9058_power.c
>  create mode 100644 drivers/regulator/da9058-regulator.c
>  create mode 100644 drivers/rtc/rtc-da9058.c
>  create mode 100644 include/linux/mfd/da9058/bat.h
>  create mode 100644 include/linux/mfd/da9058/codec.h
>  create mode 100644 include/linux/mfd/da9058/conf.h
>  create mode 100644 include/linux/mfd/da9058/core.h
>  create mode 100644 include/linux/mfd/da9058/gpio.h
>  create mode 100644 include/linux/mfd/da9058/hwmon.h
>  create mode 100644 include/linux/mfd/da9058/i2c.h
>  create mode 100644 include/linux/mfd/da9058/irq.h
>  create mode 100644 include/linux/mfd/da9058/onkey.h
>  create mode 100644 include/linux/mfd/da9058/pdata.h
>  create mode 100644 include/linux/mfd/da9058/registers.h
>  create mode 100644 include/linux/mfd/da9058/regulator.h
>  create mode 100644 include/linux/mfd/da9058/rtc.h
> 

_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors

WARNING: multiple messages have this Message-ID (diff)
From: Jonathan Cameron <jic23@kernel.org>
To: Anthony Olech <anthony.olech.opensource@diasemi.com>
Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>,
	Mauro Carvalho Chehab <mchehab@redhat.com>,
	Ashish Jangam <ashish.jangam@kpitcummins.com>,
	linux-iio@vger.kernel.org,
	Grant Likely <grant.likely@secretlab.ca>,
	Randy Dunlap <rdunlap@xenotime.net>,
	Jonathan Cameron <jic23@cam.ac.uk>,
	rtc-linux@googlegroups.com, lm-sensors@lm-sensors.org,
	Eric Andersson <eric.andersson@unixphere.com>,
	linux-input@vger.kernel.org, Axel Lin <axel.lin@gmail.com>,
	Liam Girdwood <lrg@ti.com>,
	Paul Gortmaker <p_gortmaker@yahoo.com>,
	Andrew Jones <drjones@redhat.com>,
	Michael Krufky <mkrufky@kernellabs.com>,
	Arnd Bergmann <arnd@arndb.de>,
	Samuel Ortiz <sameo@linux.intel.com>,
	Alessandro Zummo <a.zummo@towertech.it>,
	Linus Walleij <linus.walleij@stericsson.com>,
	Mark Brown <broonie@opensource.wolfsonmicro.com>,
	LKML <linux-kernel@vger.kernel.org>,
	Steven Toth <stoth@kernellabs.com>, Anton Vorontov <cb>
Subject: Re: [lm-sensors] [NEW DRIVER V3 0/8] DA9058 PMIC - please comment on this new driver
Date: Thu, 16 Aug 2012 19:10:15 +0000	[thread overview]
Message-ID: <502D4597.6080006@kernel.org> (raw)
In-Reply-To: <201208151518.q7FFI98a011599@latitude.olech.com>

You've obviously gone to a lot of effort to get the various parts of this patch
to go to the right people / lists.

There are however dependencies. I'd like to see the drivers that are dependent
on the iio one and obviously everyone may be interested in the core parts.

Hence I'd suggest just sending the whole series to everyone concerned.
We are all well practiced in the art of pressing delete if we don't want
to read something :)

(Given Lars-Peter has highlighted some bits and bobs in the adc driver
- and he always does a good job - I'll probably wait for the next version
before taking a look).

Good to see this driver making progress!

> This is submission attempt number 3 to have this driver included in
> the linux kernel source tree. This is the driver for the Dialog DA9058.
> 
> The DA9058 is a low power Power Management Integrated Circuit with extra
> functionality. It is a Multi Function Device controlled only from an I2C
> bus whose components can raise an interrupt request on a single IRQ line.
> 
> The driver for the DA9058 consists of a core (i2c) device driver that
> instantiates the individual component device drivers for:
> 
> adc - 5 ADC channels
> gpio - 2 available pins
> onkey - 1 device
> regulator - 4 BUCKS, 19 LDO and 3 fixed
> rtc - low power clock
> hwmon - 5 monitored voltages/temperatures
> power - battery information
> 
> All the above seven component device drivers depend on the 'core'
> component driver, which is number one in the patch series.
> 
> This driver has been tested on a Samsung SMDK6410 connected to a Dialog
> DA9058 Evaluation Board via one GPIO and a 3-wire I2C connection.
> 
> All the components can be builtin to the kernel or compiled as modules.
> As far as I can tell, all the latest APIs both for the core driver and
> all the component drivers have been adhered to, but if I have missed
> something please let me know.
> 
> Many thanks,
> Anthony Olech, Dialog Semiconductor Ltd.
> 
> Tony Olech (at Home) (8):
>   DA9058 MFD core driver
>   DA9058 ADC driver
>   DA9058 ONKEY driver
>   DA9058 POWER driver
>   DA9058 RTC driver
>   DA9058 GPIO driver
>   DA9058 HWMON driver
>   DA9058 REGULATOR driver
> 
>  Documentation/hwmon/da9058           |   30 ++
>  drivers/gpio/Kconfig                 |   12 +
>  drivers/gpio/Makefile                |    1 +
>  drivers/gpio/gpio-da9058.c           |  367 +++++++++++++
>  drivers/hwmon/Kconfig                |   10 +
>  drivers/hwmon/Makefile               |    1 +
>  drivers/hwmon/da9058-hwmon.c         |  425 +++++++++++++++
>  drivers/iio/adc/Kconfig              |   10 +
>  drivers/iio/adc/Makefile             |    1 +
>  drivers/iio/adc/da9058-adc.c         |  397 ++++++++++++++
>  drivers/input/misc/Kconfig           |   10 +
>  drivers/input/misc/Makefile          |    1 +
>  drivers/input/misc/da9058_onkey.c    |  177 +++++++
>  drivers/mfd/Kconfig                  |   18 +
>  drivers/mfd/Makefile                 |    3 +
>  drivers/mfd/da9058-core.c            |   83 +++
>  drivers/mfd/da9058-i2c.c             |   97 ++++
>  drivers/mfd/da9058-info.c            |  935 ++++++++++++++++++++++++++++++++++
>  drivers/mfd/da9058-irq.c             |   56 ++
>  drivers/power/Kconfig                |   10 +
>  drivers/power/Makefile               |    1 +
>  drivers/power/da9058_power.c         |  397 ++++++++++++++
>  drivers/regulator/Kconfig            |   11 +
>  drivers/regulator/Makefile           |    1 +
>  drivers/regulator/da9058-regulator.c |  251 +++++++++
>  drivers/rtc/Kconfig                  |   10 +
>  drivers/rtc/Makefile                 |    1 +
>  drivers/rtc/rtc-da9058.c             |  458 +++++++++++++++++
>  include/linux/mfd/da9058/bat.h       |   33 ++
>  include/linux/mfd/da9058/codec.h     |   22 +
>  include/linux/mfd/da9058/conf.h      |   22 +
>  include/linux/mfd/da9058/core.h      |   66 +++
>  include/linux/mfd/da9058/gpio.h      |   19 +
>  include/linux/mfd/da9058/hwmon.h     |   36 ++
>  include/linux/mfd/da9058/i2c.h       |   22 +
>  include/linux/mfd/da9058/irq.h       |   50 ++
>  include/linux/mfd/da9058/onkey.h     |   17 +
>  include/linux/mfd/da9058/pdata.h     |   32 ++
>  include/linux/mfd/da9058/registers.h |  480 +++++++++++++++++
>  include/linux/mfd/da9058/regulator.h |   33 ++
>  include/linux/mfd/da9058/rtc.h       |   17 +
>  41 files changed, 4623 insertions(+), 0 deletions(-)
>  create mode 100644 Documentation/hwmon/da9058
>  create mode 100644 drivers/gpio/gpio-da9058.c
>  create mode 100644 drivers/hwmon/da9058-hwmon.c
>  create mode 100644 drivers/iio/adc/da9058-adc.c
>  create mode 100644 drivers/input/misc/da9058_onkey.c
>  create mode 100644 drivers/mfd/da9058-core.c
>  create mode 100644 drivers/mfd/da9058-i2c.c
>  create mode 100644 drivers/mfd/da9058-info.c
>  create mode 100644 drivers/mfd/da9058-irq.c
>  create mode 100644 drivers/power/da9058_power.c
>  create mode 100644 drivers/regulator/da9058-regulator.c
>  create mode 100644 drivers/rtc/rtc-da9058.c
>  create mode 100644 include/linux/mfd/da9058/bat.h
>  create mode 100644 include/linux/mfd/da9058/codec.h
>  create mode 100644 include/linux/mfd/da9058/conf.h
>  create mode 100644 include/linux/mfd/da9058/core.h
>  create mode 100644 include/linux/mfd/da9058/gpio.h
>  create mode 100644 include/linux/mfd/da9058/hwmon.h
>  create mode 100644 include/linux/mfd/da9058/i2c.h
>  create mode 100644 include/linux/mfd/da9058/irq.h
>  create mode 100644 include/linux/mfd/da9058/onkey.h
>  create mode 100644 include/linux/mfd/da9058/pdata.h
>  create mode 100644 include/linux/mfd/da9058/registers.h
>  create mode 100644 include/linux/mfd/da9058/regulator.h
>  create mode 100644 include/linux/mfd/da9058/rtc.h
> 

_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors

  reply	other threads:[~2012-08-16 19:10 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-08-15 15:05 [NEW DRIVER V3 0/8] DA9058 PMIC - please comment on this new driver Anthony Olech
2012-08-15 15:05 ` Anthony Olech
2012-08-15 15:05 ` [lm-sensors] " Anthony Olech
2012-08-16 19:10 ` Jonathan Cameron [this message]
2012-08-16 19:10   ` Jonathan Cameron

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=502D4597.6080006@kernel.org \
    --to=jic23@kernel.org \
    --cc=a.zummo@towertech.it \
    --cc=anthony.olech.opensource@diasemi.com \
    --cc=arnd@arndb.de \
    --cc=ashish.jangam@kpitcummins.com \
    --cc=axel.lin@gmail.com \
    --cc=broonie@opensource.wolfsonmicro.com \
    --cc=dmitry.torokhov@gmail.com \
    --cc=drjones@redhat.com \
    --cc=eric.andersson@unixphere.com \
    --cc=grant.likely@secretlab.ca \
    --cc=jic23@cam.ac.uk \
    --cc=linus.walleij@stericsson.com \
    --cc=linux-iio@vger.kernel.org \
    --cc=linux-input@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lm-sensors@lm-sensors.org \
    --cc=lrg@ti.com \
    --cc=mchehab@redhat.com \
    --cc=mkrufky@kernellabs.com \
    --cc=p_gortmaker@yahoo.com \
    --cc=rdunlap@xenotime.net \
    --cc=rtc-linux@googlegroups.com \
    --cc=sameo@linux.intel.com \
    --cc=stoth@kernellabs.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.