Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1449412
| From | Maxime Ripard <maxime.ripard@free-electrons.com> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [PATCH v2 3/4] mfd: add support for Allwinner SoCs ADC |
| Date | 2016-07-25 12:00 +0200 |
| Message-ID | <rYKQ2-2ZK-29@gated-at.bofh.it> (permalink) |
| References | (3 earlier) <rWxNh-18Q-35@gated-at.bofh.it> <rX1ii-33s-25@gated-at.bofh.it> <rXl7j-7jq-13@gated-at.bofh.it> <rXss9-3T5-7@gated-at.bofh.it> <rXJ9D-6An-5@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
[Multipart message — attachments visible in raw view] - view raw
On Fri, Jul 22, 2016 at 02:55:56PM +0100, Lee Jones wrote:
> On Thu, 21 Jul 2016, Maxime Ripard wrote:
>
> > 1;4205;0c
> > On Thu, Jul 21, 2016 at 01:12:53PM +0100, Lee Jones wrote:
> > > On Wed, 20 Jul 2016, Jonathan Cameron wrote:
> > >
> > > > On 19/07/16 08:31, Lee Jones wrote:
> > > > > On Mon, 18 Jul 2016, Jonathan Cameron wrote:
> > > > >
> > > > >> On 15/07/16 10:59, Quentin Schulz wrote:
> > > > >>> The Allwinner SoCs all have an ADC that can also act as a touchscreen
> > > > >>> controller and a thermal sensor. For now, only the ADC and the thermal
> > > > >>> sensor drivers are probed by the MFD, the touchscreen controller support
> > > > >>> will be added later.
> > > > >>>
> > > > >>> Signed-off-by: Quentin Schulz <quentin.schulz@free-electrons.com>
> > > > >> Hmm. Previous patch includes the header this one creates. Ordering issue?
> > > > >> The depends kind of prevents build failures by ensuring that can't be built
> > > > >> until this one is in place, but it is certainly an ugly way to do it.
> > > > >>
> > > > >> Few little bits innline.
> > > > >>> ---
> > > > >>>
> > > > >>> v2:
> > > > >>> - add license headers,
> > > > >>> - reorder alphabetically includes,
> > > > >>> - add SUNXI_GPADC_ prefixes for defines,
> > > > >>>
> > > > >>> drivers/mfd/Kconfig | 14 +++
> > > > >>> drivers/mfd/Makefile | 2 +
> > > > >>> drivers/mfd/sunxi-gpadc-mfd.c | 197 ++++++++++++++++++++++++++++++++++++
> > > > >>> include/linux/mfd/sunxi-gpadc-mfd.h | 23 +++++
> > > > >>> 4 files changed, 236 insertions(+)
> > > > >>> create mode 100644 drivers/mfd/sunxi-gpadc-mfd.c
> > > > >>> create mode 100644 include/linux/mfd/sunxi-gpadc-mfd.h
> > > > >
> > > > > [...]
> > > > >
> > > > >>> +static struct mfd_cell sun6i_gpadc_mfd_cells[] = {
> > > > >>> + {
> > > > >>> + .name = "sun6i-a31-gpadc-iio",
> > > > >>> + .resources = adc_resources,
> > > > >>> + .num_resources = ARRAY_SIZE(adc_resources),
> > > > >>> + }, {
> > > > >>> + .name = "iio_hwmon",
> > > > >> I still really dislike using this to force the probe of that driver but
> > > > >> kind of up to the hwmon / mfd guys on this.
> > > > >
> > > > > Can you at least say *why* you don't like it?
> > > > It just feels odd to have an mfd child that isn't really dependent
> > > > on the mfd hardware itself.
> > > >
> > > > Still if you are happy, mfd is your domain and my objections were
> > > > as you probably noticed not that strong - or well described!)
> > > >
> > > > So I'm fine with this.
> > >
> > > I see. So it's not actually part of the same IP/chip?
> >
> > The chip has a temperature sensor, and we want to expose that
> > temperature through IIO.
> >
> > But we don't really have the choice on how we probe iio-hwmon
> > here. The binding was already there, and we have to keep it.
>
> What binding?
The binding introduced for the original driver this serie aims to
replace:
https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/Documentation/devicetree/bindings/input/touchscreen/sun4i.txt
Maxime
--
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread
[PATCH v2 0/4] add support for Allwinner SoCs ADC Quentin Schulz <quentin.schulz@free-electrons.com> - 2016-07-15 12:10 +0200
[PATCH v2 3/4] mfd: add support for Allwinner SoCs ADC Quentin Schulz <quentin.schulz@free-electrons.com> - 2016-07-15 12:10 +0200
Re: [PATCH v2 3/4] mfd: add support for Allwinner SoCs ADC Maxime Ripard <maxime.ripard@free-electrons.com> - 2016-07-18 15:10 +0200
Re: [PATCH v2 3/4] mfd: add support for Allwinner SoCs ADC Quentin Schulz <quentin.schulz@free-electrons.com> - 2016-07-19 14:10 +0200
Re: [PATCH v2 3/4] mfd: add support for Allwinner SoCs ADC Jonathan Cameron <jic23@kernel.org> - 2016-07-19 07:40 +0200
Re: [PATCH v2 3/4] mfd: add support for Allwinner SoCs ADC Lee Jones <lee.jones@linaro.org> - 2016-07-19 09:40 +0200
Re: [PATCH v2 3/4] mfd: add support for Allwinner SoCs ADC Jonathan Cameron <jic23@kernel.org> - 2016-07-20 17:10 +0200
Re: [PATCH v2 3/4] mfd: add support for Allwinner SoCs ADC Lee Jones <lee.jones@linaro.org> - 2016-07-21 14:20 +0200
Re: [PATCH v2 3/4] mfd: add support for Allwinner SoCs ADC Maxime Ripard <maxime.ripard@free-electrons.com> - 2016-07-21 22:10 +0200
Re: [PATCH v2 3/4] mfd: add support for Allwinner SoCs ADC Lee Jones <lee.jones@linaro.org> - 2016-07-22 16:00 +0200
Re: [PATCH v2 3/4] mfd: add support for Allwinner SoCs ADC Jonathan Cameron <jic23@kernel.org> - 2016-07-23 13:00 +0200
Re: [PATCH v2 3/4] mfd: add support for Allwinner SoCs ADC Maxime Ripard <maxime.ripard@free-electrons.com> - 2016-07-25 12:00 +0200
Re: [PATCH v2 3/4] mfd: add support for Allwinner SoCs ADC Quentin Schulz <quentin.schulz@free-electrons.com> - 2016-07-19 10:40 +0200
[PATCH v2 1/4] hwmon: iio_hwmon: defer probe when no channel is found Quentin Schulz <quentin.schulz@free-electrons.com> - 2016-07-15 12:10 +0200
Re: [v2,1/4] hwmon: iio_hwmon: defer probe when no channel is found Guenter Roeck <linux@roeck-us.net> - 2016-07-16 19:10 +0200
Re: [v2,1/4] hwmon: iio_hwmon: defer probe when no channel is found Maxime Ripard <maxime.ripard@free-electrons.com> - 2016-07-18 12:10 +0200
Re: [v2,1/4] hwmon: iio_hwmon: defer probe when no channel is found Guenter Roeck <linux@roeck-us.net> - 2016-07-18 15:30 +0200
[PATCH v2 4/4] hwmon: iio: add label for channels read by iio_hwmon Quentin Schulz <quentin.schulz@free-electrons.com> - 2016-07-15 12:10 +0200
Re: [PATCH v2 4/4] hwmon: iio: add label for channels read by iio_hwmon Guenter Roeck <linux@roeck-us.net> - 2016-07-15 16:10 +0200
Re: [PATCH v2 4/4] hwmon: iio: add label for channels read by iio_hwmon Quentin Schulz <quentin.schulz@free-electrons.com> - 2016-07-15 16:40 +0200
Re: [PATCH v2 4/4] hwmon: iio: add label for channels read by iio_hwmon Guenter Roeck <linux@roeck-us.net> - 2016-07-16 05:00 +0200
Re: [PATCH v2 4/4] hwmon: iio: add label for channels read by iio_hwmon Jonathan Cameron <jic23@kernel.org> - 2016-07-19 07:40 +0200
Re: [PATCH v2 4/4] hwmon: iio: add label for channels read by iio_hwmon Quentin Schulz <quentin.schulz@free-electrons.com> - 2016-07-19 09:00 +0200
Re: [PATCH v2 4/4] hwmon: iio: add label for channels read by iio_hwmon Jonathan Cameron <jic23@kernel.org> - 2016-07-20 16:50 +0200
csiph-web