Re: [PATCH] dt-bindings: input: atmel,captouch: convert bindings to YAML

[Dharma.B]

Hi Krzysztof,

On 07/02/24 8:45 pm, Krzysztof Kozlowski wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
> 
> On 07/02/2024 10:08, Dharma Balasubiramani wrote:
>> Convert the Atmel capacitive touchscreen bindings to YAML format.
>>
>> Signed-off-by: Dharma Balasubiramani <dharma.b@xxxxxxxxxxxx
> 
> Thank you for your patch. There is something to discuss/improve.
> 
>> +
>> +properties:
>> +  compatible:
>> +    const: atmel,captouch
>> +
>> +  reg:
>> +    maxItems: 1
>> +
>> +  interrupts:
>> +    maxItems: 1
>> +
>> +  linux,keycodes:
>> +    minItems: 1
>> +    maxItems: 8
>> +
>> +  autorepeat:
>> +    type: boolean
> 
> You can drop entire property, coming from input.yaml.

Sure, I will drop the 'linux,keycodes' and 'autorepeat' properties.
> 
>> +
>> +required:
>> +  - compatible
>> +  - reg
>> +  - interrupts
>> +  - linux,keycodes
>> +
>> +additionalProperties: false
> 
> Instead:
> unevaluatedProperties: false

Noted, Thanks.
> 
>> +
>> +examples:
>> +  - |
>> +    #include <dt-bindings/interrupt-controller/irq.h>
>> +    #include <dt-bindings/input/linux-event-codes.h>
>> +    i2c {
>> +      #address-cells = <1>;
>> +      #size-cells = <0>;
>> +      atmel-captouch@51 {
> 
> Instead:
> touchscreen? touchpad? if none of these, then just "touch@51"

I will go with touch@51.

Thanks.
> 
>> +        compatible = "atmel,captouch";
>> +        reg = <0x51>;
> 
> 
> Best regards,
> Krzysztof
> 

-- 
With Best Regards,
Dharma B.