Re: [linux-sunxi] Re: [PATCH v3 2/3] dt-bindings: input: Add R_LRADC support for A83T

From	Ziping Chen <techping.chan@gmail.com>
Newsgroups	linux.kernel
Subject	Re: [linux-sunxi] Re: [PATCH v3 2/3] dt-bindings: input: Add R_LRADC support for A83T
Date	2017-06-29 14:20 +0200
Message-ID	<tXGAp-7t8-5@gated-at.bofh.it> (permalink)
References	(3 earlier) <tX0Bc-2nz-19@gated-at.bofh.it> <tX0Bc-2nz-17@gated-at.bofh.it> <tX2D0-3Ko-21@gated-at.bofh.it> <tXzyV-30Y-1@gated-at.bofh.it> <tXCwP-4ON-31@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

2017-06-29 15:53 GMT+08:00 Maxime Ripard <maxime.ripard@free-electrons.com>:
> On Thu, Jun 29, 2017 at 12:45:51PM +0800, Ziping Chen wrote:
>> 2017-06-28 1:36 GMT+08:00 Maxime Ripard <maxime.ripard@free-electrons.com>:
>> > Hi,
>> >
>> > On Tue, Jun 27, 2017 at 11:29:10PM +0800, icenowy@aosc.io wrote:
>> >> Maxime, here's another problem: if we have already a GP LRADC driver,
>> >> how can we tell the kernel to use it as IIO ADC rather than keys?
>> >
>> > The GPADC IIO driver is not for the LRADC driver, but the GPADC /
>> > temperature sensor.
>> >
>> > We used to have an LRADC IIO driver in the CHIP BSP written by Alex
>> > (in CC):
>> > https://github.com/NextThingCo/CHIP-linux/commit/8675b761c54be73dc7cc0113209f02e10cc63a27
>> >
>> > But he never mainlined it.
>> >
>> >> Should we introduce a new property for this once ready?
>> >
>> > We need to keep the current binding. We can just check for the
>> > presence or not of child nodes to see if it has some keys, and we'd
>> > need an IIO-to-input driver that is yet to be written.
>> >
>>
>> Yes, then we need an iio-to-input driver...
>> So...whether the driver(a83t lradc keys) can be applied now,
>> or we should wait for the iio-to-input driver.
>
> This is a long term discussion, it shouldn't hold the patches you
> sent.

OK, I get it.

And I will think about how to implement the iio-to-input driver.

Thanks,
Ziping

>
> Maxime
>
> --
> Maxime Ripard, Free Electrons
> Embedded Linux and Kernel engineering
> http://free-electrons.com

Thread

[PATCH v3 2/3] dt-bindings: input: Add R_LRADC support for A83T Ziping Chen <techping.chan@gmail.com> - 2017-06-24 04:50 +0200
  Re: [PATCH v3 2/3] dt-bindings: input: Add R_LRADC support for A83T Maxime Ripard <maxime.ripard@free-electrons.com> - 2017-06-26 19:20 +0200
    Re: [linux-sunxi] Re: [PATCH v3 2/3] dt-bindings: input: Add R_LRADC  support for A83T icenowy@aosc.io - 2017-06-27 17:30 +0200
      Re: [linux-sunxi] Re: [PATCH v3 2/3] dt-bindings: input: Add R_LRADC  support for A83T Maxime Ripard <maxime.ripard@free-electrons.com> - 2017-06-27 19:40 +0200
        Re: [linux-sunxi] Re: [PATCH v3 2/3] dt-bindings: input: Add R_LRADC  support for A83T Alexandre Belloni <alexandre.belloni@free-electrons.com> - 2017-06-27 20:20 +0200
        Re: [linux-sunxi] Re: [PATCH v3 2/3] dt-bindings: input: Add R_LRADC  support for A83T Ziping Chen <techping.chan@gmail.com> - 2017-06-29 06:50 +0200
          Re: [linux-sunxi] Re: [PATCH v3 2/3] dt-bindings: input: Add R_LRADC  support for A83T Maxime Ripard <maxime.ripard@free-electrons.com> - 2017-06-29 10:00 +0200
            Re: [linux-sunxi] Re: [PATCH v3 2/3] dt-bindings: input: Add R_LRADC  support for A83T Ziping Chen <techping.chan@gmail.com> - 2017-06-29 14:20 +0200
    Re: [PATCH v3 2/3] dt-bindings: input: Add R_LRADC support for A83T Ziping Chen <techping.chan@gmail.com> - 2017-06-27 17:30 +0200
      Re: [PATCH v3 2/3] dt-bindings: input: Add R_LRADC support for A83T Maxime Ripard <maxime.ripard@free-electrons.com> - 2017-06-27 19:40 +0200
        Re: [PATCH v3 2/3] dt-bindings: input: Add R_LRADC support for A83T Ziping Chen <techping.chan@gmail.com> - 2017-06-29 06:40 +0200
          Re: [PATCH v3 2/3] dt-bindings: input: Add R_LRADC support for A83T Maxime Ripard <maxime.ripard@free-electrons.com> - 2017-06-29 10:00 +0200

csiph-web