Re: [PATCH v7 2/2] dt-bindings: input: Goodix SPI HID Touchscreen

From: Jiri Kosina
Date: Wed Sep 25 2024 - 17:49:09 EST


On Wed, 25 Sep 2024, Dmitry Torokhov wrote:

> I see that Krzysztof sent a revert, but what a proper fix would be?
> Apparently Goodix is using the same product ID gt7986u for both I2C and
> SPI parts, and covering them in one binding is not really easy (well, I
> guess it is possible with a big ugly "if"). Do we just slap "-spi"
> suffix on the compatible, so it becomes "goodix,gt7986u-spi" and go on
> on our merry way?

I actually see this as a viable option, given the circumstances.

However, given the non-responsiveness of the original author, I am now
proceeding with the revert, and we can try to sort it out later.

> Is there a better option for such products that support multiple
> interfaces/transports?

I unfortunately currently don't see one.

--
Jiri Kosina
SUSE Labs