Re: [PATCH v2 1/5] dt-bindings: input: touchscreen: hynitron,cst816x: axes and panel
From: Krzysztof Kozlowski
Date: Sun Oct 04 2026 - 03:34:03 EST
On 04/10/2026 09:29, Daniel Golle wrote:
> On Sun, Oct 04, 2026 at 09:24:26AM +0200, Krzysztof Kozlowski wrote:
>> On Fri, Oct 02, 2026 at 01:47:21PM +0100, Daniel Golle wrote:
>>> diff --git a/Documentation/devicetree/bindings/input/touchscreen/hynitron,cst816x.yaml b/Documentation/devicetree/bindings/input/touchscreen/hynitron,cst816x.yaml
>>> index 72d4da636881e..027a2b3dcf2e8 100644
>>> --- a/Documentation/devicetree/bindings/input/touchscreen/hynitron,cst816x.yaml
>>> +++ b/Documentation/devicetree/bindings/input/touchscreen/hynitron,cst816x.yaml
>>> @@ -36,6 +36,18 @@ properties:
>>> - description: Slide right gesture
>>> - description: Long press gesture
>>>
>>> + panel: true
>>> +
>>> + touchscreen-min-x: true
>>> + touchscreen-min-y: true
>>> + touchscreen-size-x: true
>>> + touchscreen-size-y: true
>>> + touchscreen-fuzz-x: true
>>> + touchscreen-fuzz-y: true
>>> + touchscreen-inverted-x: true
>>> + touchscreen-inverted-y: true
>>> + touchscreen-swapped-x-y: true
>>> +
>>> required:
>>> - compatible
>>> - reg
>>> @@ -43,6 +55,9 @@ required:
>>>
>>> additionalProperties: false
>>
>> So just use unevaluatedProperties instead of listing almost entire
>> contents of touchscreen.yaml.
>>
>
> "almost" is the keyword here. If it is acceptable to include properties
> the driver doesn't (and will never) parse, I'll do that instead.
And if you add them to the driver, you would be changing the binding? We
ask instead to post complete binding (see also writing-bindings).
Best regards,
Krzysztof