Re: [PATCH 2/2] dt-bindings: touchscreen: Convert Goodix touchscreen to json-schema

From: Rob Herring
Date: Tue Jan 07 2020 - 13:03:09 EST


On Tue, Jan 7, 2020 at 7:38 AM Bastien Nocera <hadess@xxxxxxxxxx> wrote:
>
> On Tue, 2020-01-07 at 14:09 +0100, Benjamin Gaignard wrote:
> > Convert the Goodix binding to DT schema format using json-schema
>
> I don't have an opinion on the migration itself, but this really should
> not lose any of the descriptions that were in the old text file.

To some extent, yes. Any information specific to the device should be.
Anything generic can be dropped. I see 2 cases that should be kept:

> > - - reg : I2C address of the chip. Should be
> > 0x5d or 0x14

'I2C address of the chip' can be dropped as that's every 'reg'
property for I2C devices. The addresses can be expressed as
constraints.

> > - - irq-gpios : GPIO pin used for IRQ. The driver uses the
> > - interrupt gpio pin as output to reset the
> > device.

Also useful info.

Rob