Re: [PATCH v8 0/6] iio/adc: ingenic: Cleanups & add touchscreen mode.
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-cont...@artur-rojek.eu/ Regards, Artur
Re: [PATCH v8 0/6] iio/adc: ingenic: Cleanups & add touchscreen mode.
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
Re: [PATCH v8 0/6] iio/adc: ingenic: Cleanups & add touchscreen mode.
On Thu, Jul 9, 2020 at 7:05 PM Artur Rojek wrote: > > Hey Andy, > > On 2020-07-09 17:43, Andy Shevchenko wrote: > > On Thu, Jul 9, 2020 at 6:22 PM Artur Rojek > > 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! -- With Best Regards, Andy Shevchenko
Re: [PATCH v8 0/6] iio/adc: ingenic: Cleanups & add touchscreen mode.
Hey Andy, On 2020-07-09 17:43, Andy Shevchenko wrote: On Thu, Jul 9, 2020 at 6:22 PM Artur Rojek 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
Re: [PATCH v8 0/6] iio/adc: ingenic: Cleanups & add touchscreen mode.
On Thu, Jul 9, 2020 at 6:22 PM Artur Rojek 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 > -- With Best Regards, Andy Shevchenko
[PATCH v8 0/6] iio/adc: ingenic: Cleanups & add touchscreen mode.
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. - 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