* [PATCH v5 00/10] Input: synaptics-rmi4: Synaptics RMI4 Driver
@ 2016-02-10 22:23 Andrew Duggan
2016-02-11 10:05 ` Benjamin Tissoires
` (2 more replies)
0 siblings, 3 replies; 11+ messages in thread
From: Andrew Duggan @ 2016-02-10 22:23 UTC (permalink / raw)
To: linux-input, linux-kernel
Cc: Andrew Duggan, Dmitry Torokhov, Linus Walleij, Jiri Kosina,
Benjamin Tissoires, Christopher Heiny, Stephen Chandler Paul,
Vincent Huang, Chris Healy, Andrey Gusakov, Rob Herring
This is the v5 patch series which fixes the issues identified by kbuild
and coccinelle in the previous patch series. These include:
- Using ARRAY_SIZE instead of a custom macro to calculate the size of the
function handler array.
- Remove setting the .owner field for the driver in rmi_i2c and rmi_spi
since the i2c-core and spi set that field.
- Fix if statment in rmi_f30 which was testing has_led twice instead of
testing has_gpio.
Thanks,
Andrew
Andrew Duggan (10):
Input: synaptics-rmi4: Add support for Synaptics RMI4 devices
Input: synaptics-rmi4: Add I2C transport driver
Input: synaptics-rmi4: Add device tree support for RMI4 I2C devices
Input: synaptics-rmi4: Add support for 2D sensors and F11
Input: synaptics-rmi4: Add device tree support for 2d sensors and F11
Input: synaptics-rmi4: Add support for F12
Input: synaptics-rmi4: Add support for F30
Input: synaptics-rmi4: Add SPI transport driver
Input: synaptics-rmi4: Add device tree support to the SPI transport
driver
HID: rmi: Make hid-rmi a transport driver for synaptics-rmi4
.../bindings/input/rmi4/rmi_2d_sensor.txt | 56 +
.../devicetree/bindings/input/rmi4/rmi_f01.txt | 39 +
.../devicetree/bindings/input/rmi4/rmi_i2c.txt | 53 +
.../devicetree/bindings/input/rmi4/rmi_spi.txt | 57 +
.../bindings/input/touchscreen/touchscreen.txt | 2 +
Documentation/devicetree/bindings/spi/spi-bus.txt | 2 +
.../devicetree/bindings/vendor-prefixes.txt | 1 +
drivers/hid/Kconfig | 2 +-
drivers/hid/hid-rmi.c | 891 ++-----------
drivers/input/Kconfig | 2 +
drivers/input/Makefile | 2 +
drivers/input/rmi4/Kconfig | 63 +
drivers/input/rmi4/Makefile | 13 +
drivers/input/rmi4/rmi_2d_sensor.c | 329 +++++
drivers/input/rmi4/rmi_2d_sensor.h | 87 ++
drivers/input/rmi4/rmi_bus.c | 419 +++++++
drivers/input/rmi4/rmi_bus.h | 182 +++
drivers/input/rmi4/rmi_driver.c | 1055 ++++++++++++++++
drivers/input/rmi4/rmi_driver.h | 105 ++
drivers/input/rmi4/rmi_f01.c | 624 ++++++++++
drivers/input/rmi4/rmi_f11.c | 1317 ++++++++++++++++++++
drivers/input/rmi4/rmi_f12.c | 457 +++++++
drivers/input/rmi4/rmi_f30.c | 407 ++++++
drivers/input/rmi4/rmi_i2c.c | 397 ++++++
drivers/input/rmi4/rmi_spi.c | 589 +++++++++
include/linux/rmi.h | 359 ++++++
include/uapi/linux/input.h | 1 +
27 files changed, 6693 insertions(+), 818 deletions(-)
create mode 100644 Documentation/devicetree/bindings/input/rmi4/rmi_2d_sensor.txt
create mode 100644 Documentation/devicetree/bindings/input/rmi4/rmi_f01.txt
create mode 100644 Documentation/devicetree/bindings/input/rmi4/rmi_i2c.txt
create mode 100644 Documentation/devicetree/bindings/input/rmi4/rmi_spi.txt
create mode 100644 drivers/input/rmi4/Kconfig
create mode 100644 drivers/input/rmi4/Makefile
create mode 100644 drivers/input/rmi4/rmi_2d_sensor.c
create mode 100644 drivers/input/rmi4/rmi_2d_sensor.h
create mode 100644 drivers/input/rmi4/rmi_bus.c
create mode 100644 drivers/input/rmi4/rmi_bus.h
create mode 100644 drivers/input/rmi4/rmi_driver.c
create mode 100644 drivers/input/rmi4/rmi_driver.h
create mode 100644 drivers/input/rmi4/rmi_f01.c
create mode 100644 drivers/input/rmi4/rmi_f11.c
create mode 100644 drivers/input/rmi4/rmi_f12.c
create mode 100644 drivers/input/rmi4/rmi_f30.c
create mode 100644 drivers/input/rmi4/rmi_i2c.c
create mode 100644 drivers/input/rmi4/rmi_spi.c
create mode 100644 include/linux/rmi.h
--
2.5.0
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v5 00/10] Input: synaptics-rmi4: Synaptics RMI4 Driver
2016-02-10 22:23 [PATCH v5 00/10] Input: synaptics-rmi4: Synaptics RMI4 Driver Andrew Duggan
@ 2016-02-11 10:05 ` Benjamin Tissoires
2016-02-11 21:37 ` Andrew Duggan
2016-02-12 12:54 ` Linus Walleij
2016-02-15 22:48 ` Bjorn Andersson
2 siblings, 1 reply; 11+ messages in thread
From: Benjamin Tissoires @ 2016-02-11 10:05 UTC (permalink / raw)
To: Andrew Duggan
Cc: linux-input, linux-kernel, Dmitry Torokhov, Linus Walleij,
Jiri Kosina, Christopher Heiny, Stephen Chandler Paul,
Vincent Huang, Chris Healy, Andrey Gusakov, Rob Herring
On Feb 10 2016 or thereabouts, Andrew Duggan wrote:
> This is the v5 patch series which fixes the issues identified by kbuild
> and coccinelle in the previous patch series. These include:
>
> - Using ARRAY_SIZE instead of a custom macro to calculate the size of the
> function handler array.
> - Remove setting the .owner field for the driver in rmi_i2c and rmi_spi
> since the i2c-core and spi set that field.
> - Fix if statment in rmi_f30 which was testing has_led twice instead of
> testing has_gpio.
>
Thanks Andrew for the (hopefully) last respin.
For the rmi4 core/bus:
Tested-by: Benjamin Tissoires <benjamin.tissoires@redhat.com>
The HID part does not apply on top of Jiri's tree, so we might want to
prevent Dmitri to pick this patch up for now (the 10/10).
Regarding the SMBus part, I have pushed my rebase on top of your v5
here:
https://github.com/bentiss/linux/tree/synaptics-rmi4-smbus-v4.5-rc3%2B
There is no trackpoint support (no PS/2 redirection) nor support for the
embedded buttons. I think it would be good to already merge the current
RMI4 bits so at least I2C and SPI could use it, and then figure out the
remaining parts for HID and SMBus.
Cheers,
Benjamin
> Thanks,
> Andrew
>
> Andrew Duggan (10):
> Input: synaptics-rmi4: Add support for Synaptics RMI4 devices
> Input: synaptics-rmi4: Add I2C transport driver
> Input: synaptics-rmi4: Add device tree support for RMI4 I2C devices
> Input: synaptics-rmi4: Add support for 2D sensors and F11
> Input: synaptics-rmi4: Add device tree support for 2d sensors and F11
> Input: synaptics-rmi4: Add support for F12
> Input: synaptics-rmi4: Add support for F30
> Input: synaptics-rmi4: Add SPI transport driver
> Input: synaptics-rmi4: Add device tree support to the SPI transport
> driver
> HID: rmi: Make hid-rmi a transport driver for synaptics-rmi4
>
> .../bindings/input/rmi4/rmi_2d_sensor.txt | 56 +
> .../devicetree/bindings/input/rmi4/rmi_f01.txt | 39 +
> .../devicetree/bindings/input/rmi4/rmi_i2c.txt | 53 +
> .../devicetree/bindings/input/rmi4/rmi_spi.txt | 57 +
> .../bindings/input/touchscreen/touchscreen.txt | 2 +
> Documentation/devicetree/bindings/spi/spi-bus.txt | 2 +
> .../devicetree/bindings/vendor-prefixes.txt | 1 +
> drivers/hid/Kconfig | 2 +-
> drivers/hid/hid-rmi.c | 891 ++-----------
> drivers/input/Kconfig | 2 +
> drivers/input/Makefile | 2 +
> drivers/input/rmi4/Kconfig | 63 +
> drivers/input/rmi4/Makefile | 13 +
> drivers/input/rmi4/rmi_2d_sensor.c | 329 +++++
> drivers/input/rmi4/rmi_2d_sensor.h | 87 ++
> drivers/input/rmi4/rmi_bus.c | 419 +++++++
> drivers/input/rmi4/rmi_bus.h | 182 +++
> drivers/input/rmi4/rmi_driver.c | 1055 ++++++++++++++++
> drivers/input/rmi4/rmi_driver.h | 105 ++
> drivers/input/rmi4/rmi_f01.c | 624 ++++++++++
> drivers/input/rmi4/rmi_f11.c | 1317 ++++++++++++++++++++
> drivers/input/rmi4/rmi_f12.c | 457 +++++++
> drivers/input/rmi4/rmi_f30.c | 407 ++++++
> drivers/input/rmi4/rmi_i2c.c | 397 ++++++
> drivers/input/rmi4/rmi_spi.c | 589 +++++++++
> include/linux/rmi.h | 359 ++++++
> include/uapi/linux/input.h | 1 +
> 27 files changed, 6693 insertions(+), 818 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/input/rmi4/rmi_2d_sensor.txt
> create mode 100644 Documentation/devicetree/bindings/input/rmi4/rmi_f01.txt
> create mode 100644 Documentation/devicetree/bindings/input/rmi4/rmi_i2c.txt
> create mode 100644 Documentation/devicetree/bindings/input/rmi4/rmi_spi.txt
> create mode 100644 drivers/input/rmi4/Kconfig
> create mode 100644 drivers/input/rmi4/Makefile
> create mode 100644 drivers/input/rmi4/rmi_2d_sensor.c
> create mode 100644 drivers/input/rmi4/rmi_2d_sensor.h
> create mode 100644 drivers/input/rmi4/rmi_bus.c
> create mode 100644 drivers/input/rmi4/rmi_bus.h
> create mode 100644 drivers/input/rmi4/rmi_driver.c
> create mode 100644 drivers/input/rmi4/rmi_driver.h
> create mode 100644 drivers/input/rmi4/rmi_f01.c
> create mode 100644 drivers/input/rmi4/rmi_f11.c
> create mode 100644 drivers/input/rmi4/rmi_f12.c
> create mode 100644 drivers/input/rmi4/rmi_f30.c
> create mode 100644 drivers/input/rmi4/rmi_i2c.c
> create mode 100644 drivers/input/rmi4/rmi_spi.c
> create mode 100644 include/linux/rmi.h
>
> --
> 2.5.0
>
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v5 00/10] Input: synaptics-rmi4: Synaptics RMI4 Driver
2016-02-11 10:05 ` Benjamin Tissoires
@ 2016-02-11 21:37 ` Andrew Duggan
2016-02-12 7:56 ` Benjamin Tissoires
0 siblings, 1 reply; 11+ messages in thread
From: Andrew Duggan @ 2016-02-11 21:37 UTC (permalink / raw)
To: Benjamin Tissoires
Cc: linux-input, linux-kernel, Dmitry Torokhov, Linus Walleij,
Jiri Kosina, Christopher Heiny, Stephen Chandler Paul,
Vincent Huang, Chris Healy, Andrey Gusakov, Rob Herring
On 02/11/2016 02:05 AM, Benjamin Tissoires wrote:
> On Feb 10 2016 or thereabouts, Andrew Duggan wrote:
>> This is the v5 patch series which fixes the issues identified by kbuild
>> and coccinelle in the previous patch series. These include:
>>
>> - Using ARRAY_SIZE instead of a custom macro to calculate the size of the
>> function handler array.
>> - Remove setting the .owner field for the driver in rmi_i2c and rmi_spi
>> since the i2c-core and spi set that field.
>> - Fix if statment in rmi_f30 which was testing has_led twice instead of
>> testing has_gpio.
>>
> Thanks Andrew for the (hopefully) last respin.
>
> For the rmi4 core/bus:
> Tested-by: Benjamin Tissoires <benjamin.tissoires@redhat.com>
>
> The HID part does not apply on top of Jiri's tree, so we might want to
> prevent Dmitri to pick this patch up for now (the 10/10).
>
> Regarding the SMBus part, I have pushed my rebase on top of your v5
> here:
> https://github.com/bentiss/linux/tree/synaptics-rmi4-smbus-v4.5-rc3%2B
>
> There is no trackpoint support (no PS/2 redirection) nor support for the
> embedded buttons. I think it would be good to already merge the current
> RMI4 bits so at least I2C and SPI could use it, and then figure out the
> remaining parts for HID and SMBus.
Yes, my hid-rmi patch b786ae8e269b (HID: rmi: Check that the device is a
RMI device in suspend and resume callbacks) is in Jiri's tree, but not
yet anywhere else. Including the HID patch in this patch series has
probably unnecessarily complicated things since HID is a separate tree.
It is self contained and I can resubmit HID support as a standalone
patch once the core has been accepted.
But, I do think that HID support is ready to go and the changes need to
get the patch to apply cleanly on top of b786ae8e269b are minimal.
Andrew
> Cheers,
> Benjamin
>
>> Thanks,
>> Andrew
>>
>> Andrew Duggan (10):
>> Input: synaptics-rmi4: Add support for Synaptics RMI4 devices
>> Input: synaptics-rmi4: Add I2C transport driver
>> Input: synaptics-rmi4: Add device tree support for RMI4 I2C devices
>> Input: synaptics-rmi4: Add support for 2D sensors and F11
>> Input: synaptics-rmi4: Add device tree support for 2d sensors and F11
>> Input: synaptics-rmi4: Add support for F12
>> Input: synaptics-rmi4: Add support for F30
>> Input: synaptics-rmi4: Add SPI transport driver
>> Input: synaptics-rmi4: Add device tree support to the SPI transport
>> driver
>> HID: rmi: Make hid-rmi a transport driver for synaptics-rmi4
>>
>> .../bindings/input/rmi4/rmi_2d_sensor.txt | 56 +
>> .../devicetree/bindings/input/rmi4/rmi_f01.txt | 39 +
>> .../devicetree/bindings/input/rmi4/rmi_i2c.txt | 53 +
>> .../devicetree/bindings/input/rmi4/rmi_spi.txt | 57 +
>> .../bindings/input/touchscreen/touchscreen.txt | 2 +
>> Documentation/devicetree/bindings/spi/spi-bus.txt | 2 +
>> .../devicetree/bindings/vendor-prefixes.txt | 1 +
>> drivers/hid/Kconfig | 2 +-
>> drivers/hid/hid-rmi.c | 891 ++-----------
>> drivers/input/Kconfig | 2 +
>> drivers/input/Makefile | 2 +
>> drivers/input/rmi4/Kconfig | 63 +
>> drivers/input/rmi4/Makefile | 13 +
>> drivers/input/rmi4/rmi_2d_sensor.c | 329 +++++
>> drivers/input/rmi4/rmi_2d_sensor.h | 87 ++
>> drivers/input/rmi4/rmi_bus.c | 419 +++++++
>> drivers/input/rmi4/rmi_bus.h | 182 +++
>> drivers/input/rmi4/rmi_driver.c | 1055 ++++++++++++++++
>> drivers/input/rmi4/rmi_driver.h | 105 ++
>> drivers/input/rmi4/rmi_f01.c | 624 ++++++++++
>> drivers/input/rmi4/rmi_f11.c | 1317 ++++++++++++++++++++
>> drivers/input/rmi4/rmi_f12.c | 457 +++++++
>> drivers/input/rmi4/rmi_f30.c | 407 ++++++
>> drivers/input/rmi4/rmi_i2c.c | 397 ++++++
>> drivers/input/rmi4/rmi_spi.c | 589 +++++++++
>> include/linux/rmi.h | 359 ++++++
>> include/uapi/linux/input.h | 1 +
>> 27 files changed, 6693 insertions(+), 818 deletions(-)
>> create mode 100644 Documentation/devicetree/bindings/input/rmi4/rmi_2d_sensor.txt
>> create mode 100644 Documentation/devicetree/bindings/input/rmi4/rmi_f01.txt
>> create mode 100644 Documentation/devicetree/bindings/input/rmi4/rmi_i2c.txt
>> create mode 100644 Documentation/devicetree/bindings/input/rmi4/rmi_spi.txt
>> create mode 100644 drivers/input/rmi4/Kconfig
>> create mode 100644 drivers/input/rmi4/Makefile
>> create mode 100644 drivers/input/rmi4/rmi_2d_sensor.c
>> create mode 100644 drivers/input/rmi4/rmi_2d_sensor.h
>> create mode 100644 drivers/input/rmi4/rmi_bus.c
>> create mode 100644 drivers/input/rmi4/rmi_bus.h
>> create mode 100644 drivers/input/rmi4/rmi_driver.c
>> create mode 100644 drivers/input/rmi4/rmi_driver.h
>> create mode 100644 drivers/input/rmi4/rmi_f01.c
>> create mode 100644 drivers/input/rmi4/rmi_f11.c
>> create mode 100644 drivers/input/rmi4/rmi_f12.c
>> create mode 100644 drivers/input/rmi4/rmi_f30.c
>> create mode 100644 drivers/input/rmi4/rmi_i2c.c
>> create mode 100644 drivers/input/rmi4/rmi_spi.c
>> create mode 100644 include/linux/rmi.h
>>
>> --
>> 2.5.0
>>
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v5 00/10] Input: synaptics-rmi4: Synaptics RMI4 Driver
2016-02-11 21:37 ` Andrew Duggan
@ 2016-02-12 7:56 ` Benjamin Tissoires
0 siblings, 0 replies; 11+ messages in thread
From: Benjamin Tissoires @ 2016-02-12 7:56 UTC (permalink / raw)
To: Andrew Duggan
Cc: linux-input, linux-kernel, Dmitry Torokhov, Linus Walleij,
Jiri Kosina, Christopher Heiny, Stephen Chandler Paul,
Vincent Huang, Chris Healy, Andrey Gusakov, Rob Herring
On Feb 11 2016 or thereabouts, Andrew Duggan wrote:
> On 02/11/2016 02:05 AM, Benjamin Tissoires wrote:
> >On Feb 10 2016 or thereabouts, Andrew Duggan wrote:
> >>This is the v5 patch series which fixes the issues identified by kbuild
> >>and coccinelle in the previous patch series. These include:
> >>
> >>- Using ARRAY_SIZE instead of a custom macro to calculate the size of the
> >> function handler array.
> >>- Remove setting the .owner field for the driver in rmi_i2c and rmi_spi
> >> since the i2c-core and spi set that field.
> >>- Fix if statment in rmi_f30 which was testing has_led twice instead of
> >> testing has_gpio.
> >>
> >Thanks Andrew for the (hopefully) last respin.
> >
> >For the rmi4 core/bus:
> >Tested-by: Benjamin Tissoires <benjamin.tissoires@redhat.com>
> >
> >The HID part does not apply on top of Jiri's tree, so we might want to
> >prevent Dmitri to pick this patch up for now (the 10/10).
> >
> >Regarding the SMBus part, I have pushed my rebase on top of your v5
> >here:
> >https://github.com/bentiss/linux/tree/synaptics-rmi4-smbus-v4.5-rc3%2B
> >
> >There is no trackpoint support (no PS/2 redirection) nor support for the
> >embedded buttons. I think it would be good to already merge the current
> >RMI4 bits so at least I2C and SPI could use it, and then figure out the
> >remaining parts for HID and SMBus.
>
> Yes, my hid-rmi patch b786ae8e269b (HID: rmi: Check that the device is a RMI
> device in suspend and resume callbacks) is in Jiri's tree, but not yet
> anywhere else. Including the HID patch in this patch series has probably
> unnecessarily complicated things since HID is a separate tree. It is self
> contained and I can resubmit HID support as a standalone patch once the core
> has been accepted.
>
> But, I do think that HID support is ready to go and the changes need to get
> the patch to apply cleanly on top of b786ae8e269b are minimal.
I agree that the HID part should use rmi4 core as soon as possible. It's
just that the patch in this series conflicts with the upstream HID tree.
And this usually requires coordination between Jiri and Dmitry. I was
mentioning it to avoid having Linus to handle the merge himself, or at
least so we can explain to him how the merge should be done because of
the conflict.
Cheers,
Benjamin
>
> Andrew
>
> >Cheers,
> >Benjamin
> >
> >>Thanks,
> >>Andrew
> >>
> >>Andrew Duggan (10):
> >> Input: synaptics-rmi4: Add support for Synaptics RMI4 devices
> >> Input: synaptics-rmi4: Add I2C transport driver
> >> Input: synaptics-rmi4: Add device tree support for RMI4 I2C devices
> >> Input: synaptics-rmi4: Add support for 2D sensors and F11
> >> Input: synaptics-rmi4: Add device tree support for 2d sensors and F11
> >> Input: synaptics-rmi4: Add support for F12
> >> Input: synaptics-rmi4: Add support for F30
> >> Input: synaptics-rmi4: Add SPI transport driver
> >> Input: synaptics-rmi4: Add device tree support to the SPI transport
> >> driver
> >> HID: rmi: Make hid-rmi a transport driver for synaptics-rmi4
> >>
> >> .../bindings/input/rmi4/rmi_2d_sensor.txt | 56 +
> >> .../devicetree/bindings/input/rmi4/rmi_f01.txt | 39 +
> >> .../devicetree/bindings/input/rmi4/rmi_i2c.txt | 53 +
> >> .../devicetree/bindings/input/rmi4/rmi_spi.txt | 57 +
> >> .../bindings/input/touchscreen/touchscreen.txt | 2 +
> >> Documentation/devicetree/bindings/spi/spi-bus.txt | 2 +
> >> .../devicetree/bindings/vendor-prefixes.txt | 1 +
> >> drivers/hid/Kconfig | 2 +-
> >> drivers/hid/hid-rmi.c | 891 ++-----------
> >> drivers/input/Kconfig | 2 +
> >> drivers/input/Makefile | 2 +
> >> drivers/input/rmi4/Kconfig | 63 +
> >> drivers/input/rmi4/Makefile | 13 +
> >> drivers/input/rmi4/rmi_2d_sensor.c | 329 +++++
> >> drivers/input/rmi4/rmi_2d_sensor.h | 87 ++
> >> drivers/input/rmi4/rmi_bus.c | 419 +++++++
> >> drivers/input/rmi4/rmi_bus.h | 182 +++
> >> drivers/input/rmi4/rmi_driver.c | 1055 ++++++++++++++++
> >> drivers/input/rmi4/rmi_driver.h | 105 ++
> >> drivers/input/rmi4/rmi_f01.c | 624 ++++++++++
> >> drivers/input/rmi4/rmi_f11.c | 1317 ++++++++++++++++++++
> >> drivers/input/rmi4/rmi_f12.c | 457 +++++++
> >> drivers/input/rmi4/rmi_f30.c | 407 ++++++
> >> drivers/input/rmi4/rmi_i2c.c | 397 ++++++
> >> drivers/input/rmi4/rmi_spi.c | 589 +++++++++
> >> include/linux/rmi.h | 359 ++++++
> >> include/uapi/linux/input.h | 1 +
> >> 27 files changed, 6693 insertions(+), 818 deletions(-)
> >> create mode 100644 Documentation/devicetree/bindings/input/rmi4/rmi_2d_sensor.txt
> >> create mode 100644 Documentation/devicetree/bindings/input/rmi4/rmi_f01.txt
> >> create mode 100644 Documentation/devicetree/bindings/input/rmi4/rmi_i2c.txt
> >> create mode 100644 Documentation/devicetree/bindings/input/rmi4/rmi_spi.txt
> >> create mode 100644 drivers/input/rmi4/Kconfig
> >> create mode 100644 drivers/input/rmi4/Makefile
> >> create mode 100644 drivers/input/rmi4/rmi_2d_sensor.c
> >> create mode 100644 drivers/input/rmi4/rmi_2d_sensor.h
> >> create mode 100644 drivers/input/rmi4/rmi_bus.c
> >> create mode 100644 drivers/input/rmi4/rmi_bus.h
> >> create mode 100644 drivers/input/rmi4/rmi_driver.c
> >> create mode 100644 drivers/input/rmi4/rmi_driver.h
> >> create mode 100644 drivers/input/rmi4/rmi_f01.c
> >> create mode 100644 drivers/input/rmi4/rmi_f11.c
> >> create mode 100644 drivers/input/rmi4/rmi_f12.c
> >> create mode 100644 drivers/input/rmi4/rmi_f30.c
> >> create mode 100644 drivers/input/rmi4/rmi_i2c.c
> >> create mode 100644 drivers/input/rmi4/rmi_spi.c
> >> create mode 100644 include/linux/rmi.h
> >>
> >>--
> >>2.5.0
> >>
>
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v5 00/10] Input: synaptics-rmi4: Synaptics RMI4 Driver
2016-02-10 22:23 [PATCH v5 00/10] Input: synaptics-rmi4: Synaptics RMI4 Driver Andrew Duggan
2016-02-11 10:05 ` Benjamin Tissoires
@ 2016-02-12 12:54 ` Linus Walleij
2016-02-12 18:10 ` Andrew Duggan
2016-02-15 22:48 ` Bjorn Andersson
2 siblings, 1 reply; 11+ messages in thread
From: Linus Walleij @ 2016-02-12 12:54 UTC (permalink / raw)
To: Andrew Duggan
Cc: Linux Input, linux-kernel, Dmitry Torokhov, Jiri Kosina,
Benjamin Tissoires, Christopher Heiny, Stephen Chandler Paul,
Vincent Huang, Chris Healy, Andrey Gusakov, Rob Herring
On Wed, Feb 10, 2016 at 11:23 PM, Andrew Duggan <aduggan@synaptics.com> wrote:
> This is the v5 patch series which fixes the issues identified by kbuild
> and coccinelle in the previous patch series. These include:
>
> - Using ARRAY_SIZE instead of a custom macro to calculate the size of the
> function handler array.
> - Remove setting the .owner field for the driver in rmi_i2c and rmi_spi
> since the i2c-core and spi set that field.
> - Fix if statment in rmi_f30 which was testing has_led twice instead of
> testing has_gpio.
Works like a charm.
Tested-by: Linus Walleij <linus.walleij@linaro.org>
Yours,
Linus Walleij
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v5 00/10] Input: synaptics-rmi4: Synaptics RMI4 Driver
2016-02-12 12:54 ` Linus Walleij
@ 2016-02-12 18:10 ` Andrew Duggan
2016-03-03 17:11 ` Benjamin Tissoires
0 siblings, 1 reply; 11+ messages in thread
From: Andrew Duggan @ 2016-02-12 18:10 UTC (permalink / raw)
To: Linus Walleij, Dmitry Torokhov
Cc: Linux Input, linux-kernel, Jiri Kosina, Benjamin Tissoires,
Christopher Heiny, Stephen Chandler Paul, Vincent Huang,
Chris Healy, Andrey Gusakov, Rob Herring
On 02/12/2016 04:54 AM, Linus Walleij wrote:
> On Wed, Feb 10, 2016 at 11:23 PM, Andrew Duggan <aduggan@synaptics.com> wrote:
>
>> This is the v5 patch series which fixes the issues identified by kbuild
>> and coccinelle in the previous patch series. These include:
>>
>> - Using ARRAY_SIZE instead of a custom macro to calculate the size of the
>> function handler array.
>> - Remove setting the .owner field for the driver in rmi_i2c and rmi_spi
>> since the i2c-core and spi set that field.
>> - Fix if statment in rmi_f30 which was testing has_led twice instead of
>> testing has_gpio.
> Works like a charm.
> Tested-by: Linus Walleij <linus.walleij@linaro.org>
Thanks for testing. At this point except for the touchscreen binding
rename fix needed in patch 5, all of the outstanding issues have been
addressed. If the rest of the patch series is acceptable, then I can do
a final v6 respin which fixes the binding and also removes the HID
patch. But, if more work needs to be done on the patch series I can wait
to do the respin. Dmitry, what do you think of the patch series at this
point?
Thanks,
Andrew
> Yours,
> Linus Walleij
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v5 00/10] Input: synaptics-rmi4: Synaptics RMI4 Driver
2016-02-10 22:23 [PATCH v5 00/10] Input: synaptics-rmi4: Synaptics RMI4 Driver Andrew Duggan
2016-02-11 10:05 ` Benjamin Tissoires
2016-02-12 12:54 ` Linus Walleij
@ 2016-02-15 22:48 ` Bjorn Andersson
2 siblings, 0 replies; 11+ messages in thread
From: Bjorn Andersson @ 2016-02-15 22:48 UTC (permalink / raw)
To: Andrew Duggan
Cc: linux-input, lkml, Dmitry Torokhov, Linus Walleij, Jiri Kosina,
Benjamin Tissoires, Christopher Heiny, Stephen Chandler Paul,
Vincent Huang, Chris Healy, Andrey Gusakov, Rob Herring
On Wed, Feb 10, 2016 at 2:23 PM, Andrew Duggan <aduggan@synaptics.com> wrote:
> This is the v5 patch series which fixes the issues identified by kbuild
> and coccinelle in the previous patch series. These include:
>
> - Using ARRAY_SIZE instead of a custom macro to calculate the size of the
> function handler array.
> - Remove setting the .owner field for the driver in rmi_i2c and rmi_spi
> since the i2c-core and spi set that field.
> - Fix if statment in rmi_f30 which was testing has_led twice instead of
> testing has_gpio.
>
> Thanks,
> Andrew
>
> Andrew Duggan (10):
> Input: synaptics-rmi4: Add support for Synaptics RMI4 devices
> Input: synaptics-rmi4: Add I2C transport driver
> Input: synaptics-rmi4: Add device tree support for RMI4 I2C devices
> Input: synaptics-rmi4: Add support for 2D sensors and F11
> Input: synaptics-rmi4: Add device tree support for 2d sensors and F11
> Input: synaptics-rmi4: Add support for F12
> Input: synaptics-rmi4: Add support for F30
> Input: synaptics-rmi4: Add SPI transport driver
> Input: synaptics-rmi4: Add device tree support to the SPI transport
> driver
> HID: rmi: Make hid-rmi a transport driver for synaptics-rmi4
>
My rmi4 device is powered by two regulators that are left disabled by
the boot, so I have a patch to acquire those and toggle them
appropriately. I plan to send that out once your patches lands.
Tested-by: Bjorn Andersson <bjorn.andersson@linaro.org>
Regards,
Bjorn
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v5 00/10] Input: synaptics-rmi4: Synaptics RMI4 Driver
2016-02-12 18:10 ` Andrew Duggan
@ 2016-03-03 17:11 ` Benjamin Tissoires
2016-03-11 18:05 ` Dmitry Torokhov
0 siblings, 1 reply; 11+ messages in thread
From: Benjamin Tissoires @ 2016-03-03 17:11 UTC (permalink / raw)
To: Dmitry Torokhov
Cc: Andrew Duggan, Linus Walleij, Linux Input, linux-kernel,
Jiri Kosina, Christopher Heiny, Stephen Chandler Paul,
Vincent Huang, Chris Healy, Andrey Gusakov, Rob Herring
On Feb 12 2016 or thereabouts, Andrew Duggan wrote:
> On 02/12/2016 04:54 AM, Linus Walleij wrote:
> >On Wed, Feb 10, 2016 at 11:23 PM, Andrew Duggan <aduggan@synaptics.com> wrote:
> >
> >>This is the v5 patch series which fixes the issues identified by kbuild
> >>and coccinelle in the previous patch series. These include:
> >>
> >>- Using ARRAY_SIZE instead of a custom macro to calculate the size of the
> >> function handler array.
> >>- Remove setting the .owner field for the driver in rmi_i2c and rmi_spi
> >> since the i2c-core and spi set that field.
> >>- Fix if statment in rmi_f30 which was testing has_led twice instead of
> >> testing has_gpio.
> >Works like a charm.
> >Tested-by: Linus Walleij <linus.walleij@linaro.org>
>
> Thanks for testing. At this point except for the touchscreen binding rename
> fix needed in patch 5, all of the outstanding issues have been addressed. If
> the rest of the patch series is acceptable, then I can do a final v6 respin
> which fixes the binding and also removes the HID patch. But, if more work
> needs to be done on the patch series I can wait to do the respin. Dmitry,
> what do you think of the patch series at this point?
Dmitry, ping on this one...
I'd like to see the series in 4.6, and we are getting closer of the
merge window. Could we get some of your time on this?
Cheers,
Benjamin
>
> Thanks,
> Andrew
>
> >Yours,
> >Linus Walleij
>
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v5 00/10] Input: synaptics-rmi4: Synaptics RMI4 Driver
2016-03-03 17:11 ` Benjamin Tissoires
@ 2016-03-11 18:05 ` Dmitry Torokhov
2016-03-11 18:36 ` Andrew Duggan
2016-03-16 11:53 ` Linus Walleij
0 siblings, 2 replies; 11+ messages in thread
From: Dmitry Torokhov @ 2016-03-11 18:05 UTC (permalink / raw)
To: Benjamin Tissoires
Cc: Andrew Duggan, Linus Walleij, Linux Input, linux-kernel,
Jiri Kosina, Christopher Heiny, Stephen Chandler Paul,
Vincent Huang, Chris Healy, Andrey Gusakov, Rob Herring
On Thu, Mar 03, 2016 at 06:11:38PM +0100, Benjamin Tissoires wrote:
> On Feb 12 2016 or thereabouts, Andrew Duggan wrote:
> > On 02/12/2016 04:54 AM, Linus Walleij wrote:
> > >On Wed, Feb 10, 2016 at 11:23 PM, Andrew Duggan <aduggan@synaptics.com> wrote:
> > >
> > >>This is the v5 patch series which fixes the issues identified by kbuild
> > >>and coccinelle in the previous patch series. These include:
> > >>
> > >>- Using ARRAY_SIZE instead of a custom macro to calculate the size of the
> > >> function handler array.
> > >>- Remove setting the .owner field for the driver in rmi_i2c and rmi_spi
> > >> since the i2c-core and spi set that field.
> > >>- Fix if statment in rmi_f30 which was testing has_led twice instead of
> > >> testing has_gpio.
> > >Works like a charm.
> > >Tested-by: Linus Walleij <linus.walleij@linaro.org>
> >
> > Thanks for testing. At this point except for the touchscreen binding rename
> > fix needed in patch 5, all of the outstanding issues have been addressed. If
> > the rest of the patch series is acceptable, then I can do a final v6 respin
> > which fixes the binding and also removes the HID patch. But, if more work
> > needs to be done on the patch series I can wait to do the respin. Dmitry,
> > what do you think of the patch series at this point?
>
> Dmitry, ping on this one...
>
> I'd like to see the series in 4.6, and we are getting closer of the
> merge window. Could we get some of your time on this?
I adjusted the binding on patch 5 (to read -x-mm and y-mm) and applied
the first 9 leaving the HID out for now.
Thanks.
--
Dmitry
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v5 00/10] Input: synaptics-rmi4: Synaptics RMI4 Driver
2016-03-11 18:05 ` Dmitry Torokhov
@ 2016-03-11 18:36 ` Andrew Duggan
2016-03-16 11:53 ` Linus Walleij
1 sibling, 0 replies; 11+ messages in thread
From: Andrew Duggan @ 2016-03-11 18:36 UTC (permalink / raw)
To: Dmitry Torokhov
Cc: Benjamin Tissoires, Andrew Duggan, Linus Walleij, Linux Input,
linux-kernel, Jiri Kosina, Christopher Heiny,
Stephen Chandler Paul, Vincent Huang, Chris Healy,
Andrey Gusakov, Rob Herring
On Fri, Mar 11, 2016 at 10:05 AM, Dmitry Torokhov
<dmitry.torokhov@gmail.com> wrote:
> On Thu, Mar 03, 2016 at 06:11:38PM +0100, Benjamin Tissoires wrote:
>> On Feb 12 2016 or thereabouts, Andrew Duggan wrote:
>> > On 02/12/2016 04:54 AM, Linus Walleij wrote:
>> > >On Wed, Feb 10, 2016 at 11:23 PM, Andrew Duggan <aduggan@synaptics.com> wrote:
>> > >
>> > >>This is the v5 patch series which fixes the issues identified by kbuild
>> > >>and coccinelle in the previous patch series. These include:
>> > >>
>> > >>- Using ARRAY_SIZE instead of a custom macro to calculate the size of the
>> > >> function handler array.
>> > >>- Remove setting the .owner field for the driver in rmi_i2c and rmi_spi
>> > >> since the i2c-core and spi set that field.
>> > >>- Fix if statment in rmi_f30 which was testing has_led twice instead of
>> > >> testing has_gpio.
>> > >Works like a charm.
>> > >Tested-by: Linus Walleij <linus.walleij@linaro.org>
>> >
>> > Thanks for testing. At this point except for the touchscreen binding rename
>> > fix needed in patch 5, all of the outstanding issues have been addressed. If
>> > the rest of the patch series is acceptable, then I can do a final v6 respin
>> > which fixes the binding and also removes the HID patch. But, if more work
>> > needs to be done on the patch series I can wait to do the respin. Dmitry,
>> > what do you think of the patch series at this point?
>>
>> Dmitry, ping on this one...
>>
>> I'd like to see the series in 4.6, and we are getting closer of the
>> merge window. Could we get some of your time on this?
>
> I adjusted the binding on patch 5 (to read -x-mm and y-mm) and applied
> the first 9 leaving the HID out for now.
>
Great! Thanks everyone for testing and reviewing.
Andrew
> Thanks.
>
> --
> Dmitry
> --
> To unsubscribe from this list: send the line "unsubscribe linux-input" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v5 00/10] Input: synaptics-rmi4: Synaptics RMI4 Driver
2016-03-11 18:05 ` Dmitry Torokhov
2016-03-11 18:36 ` Andrew Duggan
@ 2016-03-16 11:53 ` Linus Walleij
1 sibling, 0 replies; 11+ messages in thread
From: Linus Walleij @ 2016-03-16 11:53 UTC (permalink / raw)
To: Dmitry Torokhov
Cc: Benjamin Tissoires, Andrew Duggan, Linux Input, linux-kernel,
Jiri Kosina, Christopher Heiny, Stephen Chandler Paul,
Vincent Huang, Chris Healy, Andrey Gusakov, Rob Herring
On Fri, Mar 11, 2016 at 7:05 PM, Dmitry Torokhov
<dmitry.torokhov@gmail.com> wrote:
> On Thu, Mar 03, 2016 at 06:11:38PM +0100, Benjamin Tissoires wrote:
>> I'd like to see the series in 4.6, and we are getting closer of the
>> merge window. Could we get some of your time on this?
>
> I adjusted the binding on patch 5 (to read -x-mm and y-mm) and applied
> the first 9 leaving the HID out for now.
Thanks to everyone involved that we finally got this in upstream shape
after all these years!
Yours,
Linus Walleij
^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2016-03-16 11:53 UTC | newest]
Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-02-10 22:23 [PATCH v5 00/10] Input: synaptics-rmi4: Synaptics RMI4 Driver Andrew Duggan
2016-02-11 10:05 ` Benjamin Tissoires
2016-02-11 21:37 ` Andrew Duggan
2016-02-12 7:56 ` Benjamin Tissoires
2016-02-12 12:54 ` Linus Walleij
2016-02-12 18:10 ` Andrew Duggan
2016-03-03 17:11 ` Benjamin Tissoires
2016-03-11 18:05 ` Dmitry Torokhov
2016-03-11 18:36 ` Andrew Duggan
2016-03-16 11:53 ` Linus Walleij
2016-02-15 22:48 ` Bjorn Andersson
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).