mbox series

[v3,0/5] himax_hx83112b: add support for HX83100A

Message ID 20240620145019.156187-1-felix@kaechele.ca
Headers show
Series himax_hx83112b: add support for HX83100A | expand

Message

Felix Kaechele June 20, 2024, 2:50 p.m. UTC 
Original discussion around v2 here: https://lore.kernel.org/all/20240511121245.109644-1-felix@kaechele.ca/

Changes from v2:
- reworded dt-bindings commit message to specifiy how the HX83100A is different
  from other chips in this series to justify having its own compatible string
- added Acked-By from Conor that was given in the v2 thread to commit message

I've done some more testing in the meantime and haven't found any issues.
I haven't heard back from anyone with an HX83112B, but the HX83100A works as
intended with these changes and I assume the HX83112B will too.

Thanks,
Felix

Felix Kaechele (5):
  dt-bindings: input: touchscreen: himax,hx83112b: add HX83100A
  input: himax_hx83112b: use more descriptive register defines
  input: himax_hx83112b: implement MCU register reading
  input: himax_hx83112b: add himax_chip struct for multi-chip support
  input: himax_hx83112b: add support for HX83100A

 .../input/touchscreen/himax,hx83112b.yaml     |   1 +
 drivers/input/touchscreen/himax_hx83112b.c    | 135 ++++++++++++++----
 2 files changed, 110 insertions(+), 26 deletions(-)


base-commit: 9b9247397e2e20016031e59f76dae563b79b6ee2

Comments

Dmitry Torokhov July 8, 2024, 11:16 p.m. UTC | #1 
On Thu, Jun 20, 2024 at 10:50:01AM -0400, Felix Kaechele wrote:
> Original discussion around v2 here: https://lore.kernel.org/all/20240511121245.109644-1-felix@kaechele.ca/
> 
> Changes from v2:
> - reworded dt-bindings commit message to specifiy how the HX83100A is different
>   from other chips in this series to justify having its own compatible string
> - added Acked-By from Conor that was given in the v2 thread to commit message
> 
> I've done some more testing in the meantime and haven't found any issues.
> I haven't heard back from anyone with an HX83112B, but the HX83100A works as
> intended with these changes and I assume the HX83112B will too.

Applied the lot, thank you.