| Message ID | 20200709152200.10039-1-contact@artur-rojek.eu (mailing list archive) |
|---|---|
| Headers | show |
| Series | iio/adc: ingenic: Cleanups & add touchscreen mode. | expand |
On Thu, Jul 9, 2020 at 6:22 PM Artur Rojek <contact@artur-rojek.eu> wrote: > > Hi all, > > v8 of this patchset introduces some structural changes, which I deemed > worthy highlighting here: Can you remind me if I gave you tags on the previous version? If so, is the above the reason to drop them? > > - adc-joystick related changes have been dropped from this patchset and > will be upstreamed separately. Their only connection to this patchset > was that they used INGENIC_ADC_TOUCH_* defines in the DTS example, > causing trouble to Rob's scripts. > > - Integrated Paul's changes, which introduce an ADCMD low-level command > feature. These changes affect patches 5/6 and 6/6, with the former > requiring Rob to re-ack. > > Cheers, > Artur > > Artur Rojek (5): > dt-bindings: iio/adc: Convert ingenic-adc docs to YAML. > IIO: Ingenic JZ47xx: Error check clk_enable calls. > IIO: Ingenic JZ47xx: Add xlate cb to retrieve correct channel idx > dt-bindings: iio/adc: Add touchscreen idx for JZ47xx SoC ADC > IIO: Ingenic JZ47xx: Add touchscreen mode. > > Paul Cercueil (1): > iio/adc: ingenic: Retrieve channels list from soc data struct > > .../bindings/iio/adc/ingenic,adc.txt | 49 --- > .../bindings/iio/adc/ingenic,adc.yaml | 71 ++++ > drivers/iio/adc/Kconfig | 1 + > drivers/iio/adc/ingenic-adc.c | 386 ++++++++++++++++-- > include/dt-bindings/iio/adc/ingenic,adc.h | 6 + > 5 files changed, 426 insertions(+), 87 deletions(-) > delete mode 100644 Documentation/devicetree/bindings/iio/adc/ingenic,adc.txt > create mode 100644 Documentation/devicetree/bindings/iio/adc/ingenic,adc.yaml > > -- > 2.27.0 >
Hey Andy, On 2020-07-09 17:43, Andy Shevchenko wrote: > On Thu, Jul 9, 2020 at 6:22 PM Artur Rojek <contact@artur-rojek.eu> > wrote: >> >> Hi all, >> >> v8 of this patchset introduces some structural changes, which I deemed >> worthy highlighting here: > > Can you remind me if I gave you tags on the previous version? I received no tags from you on this patchset so far. Cheers, Artur > If so, is the above the reason to drop them? > >> >> - adc-joystick related changes have been dropped from this patchset >> and >> will be upstreamed separately. Their only connection to this >> patchset >> was that they used INGENIC_ADC_TOUCH_* defines in the DTS example, >> causing trouble to Rob's scripts. >> >> - Integrated Paul's changes, which introduce an ADCMD low-level >> command >> feature. These changes affect patches 5/6 and 6/6, with the former >> requiring Rob to re-ack. >> >> Cheers, >> Artur >> >> Artur Rojek (5): >> dt-bindings: iio/adc: Convert ingenic-adc docs to YAML. >> IIO: Ingenic JZ47xx: Error check clk_enable calls. >> IIO: Ingenic JZ47xx: Add xlate cb to retrieve correct channel idx >> dt-bindings: iio/adc: Add touchscreen idx for JZ47xx SoC ADC >> IIO: Ingenic JZ47xx: Add touchscreen mode. >> >> Paul Cercueil (1): >> iio/adc: ingenic: Retrieve channels list from soc data struct >> >> .../bindings/iio/adc/ingenic,adc.txt | 49 --- >> .../bindings/iio/adc/ingenic,adc.yaml | 71 ++++ >> drivers/iio/adc/Kconfig | 1 + >> drivers/iio/adc/ingenic-adc.c | 386 >> ++++++++++++++++-- >> include/dt-bindings/iio/adc/ingenic,adc.h | 6 + >> 5 files changed, 426 insertions(+), 87 deletions(-) >> delete mode 100644 >> Documentation/devicetree/bindings/iio/adc/ingenic,adc.txt >> create mode 100644 >> Documentation/devicetree/bindings/iio/adc/ingenic,adc.yaml >> >> -- >> 2.27.0 >>
On Thu, Jul 9, 2020 at 7:05 PM Artur Rojek <contact@artur-rojek.eu> wrote: > > Hey Andy, > > On 2020-07-09 17:43, Andy Shevchenko wrote: > > On Thu, Jul 9, 2020 at 6:22 PM Artur Rojek <contact@artur-rojek.eu> > > wrote: > >> > >> Hi all, > >> > >> v8 of this patchset introduces some structural changes, which I deemed > >> worthy highlighting here: > > > > Can you remind me if I gave you tags on the previous version? > I received no tags from you on this patchset so far. Thanks for reminding me!
Hi Artur, Am Donnerstag, 9. Juli 2020, 17:21:54 CEST schrieb Artur Rojek: > Hi all, > > v8 of this patchset introduces some structural changes, which I deemed > worthy highlighting here: > > - adc-joystick related changes have been dropped from this patchset and > will be upstreamed separately. Their only connection to this patchset > was that they used INGENIC_ADC_TOUCH_* defines in the DTS example, > causing trouble to Rob's scripts. as I'm mainly eyeing your adc-joystick patch ... did you post that already somewhere - separately as you wrote? Thanks Heiko
On 2020-07-14 20:33, Heiko Stuebner wrote: > Hi Artur, > > Am Donnerstag, 9. Juli 2020, 17:21:54 CEST schrieb Artur Rojek: >> Hi all, >> >> v8 of this patchset introduces some structural changes, which I deemed >> worthy highlighting here: >> >> - adc-joystick related changes have been dropped from this patchset >> and >> will be upstreamed separately. Their only connection to this >> patchset >> was that they used INGENIC_ADC_TOUCH_* defines in the DTS example, >> causing trouble to Rob's scripts. > > as I'm mainly eyeing your adc-joystick patch ... did you post that > already > somewhere - separately as you wrote? > > Thanks > Heiko Hi Heiko, sorry for a late reply. As you have surely noticed in your mailbox, I have sent the adc-joystick changes in the following series: https://lore.kernel.org/linux-input/20200719221103.91644-1-contact@artur-rojek.eu/ Regards, Artur