linux-kernel - Re: [PATCH 1/2] dt-bindings: touchscreen: trivial-touch: Drop 'interrupts' requirement for old Ilitek

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <68333066-2065-422d-9582-11a4bbb41e59@mailbox.org>
Date: Thu, 15 Jan 2026 03:09:11 +0100
From: Marek Vasut <marek.vasut@...lbox.org>
To: Frank Li <Frank.li@....com>, Marek Vasut <marek.vasut+renesas@...lbox.org>
Cc: linux-input@...r.kernel.org, Conor Dooley <conor+dt@...nel.org>,
 Dmitry Torokhov <dmitry.torokhov@...il.com>, Job Noorman <job@...rman.info>,
 Krzysztof Kozlowski <krzk+dt@...nel.org>, Rob Herring <robh@...nel.org>,
 devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
 linux-renesas-soc@...r.kernel.org
Subject: Re: [PATCH 1/2] dt-bindings: touchscreen: trivial-touch: Drop
 'interrupts' requirement for old Ilitek

On 1/13/26 5:35 PM, Frank Li wrote:

Hello Frank,

>> +++ b/Documentation/devicetree/bindings/input/touchscreen/trivial-touch.yaml
>> @@ -57,13 +57,25 @@ properties:
>>
>>     wakeup-source: true
>>
>> -allOf:
>> -  - $ref: touchscreen.yaml
>> -
>>   required:
>>     - compatible
>>     - reg
>> -  - interrupts
>> +
>> +allOf:
>> +  - $ref: touchscreen.yaml
>> +  - if:
>> +      not:
>> +        properties:
>> +          compatible:
>> +            contains:
>> +              enum:
>> +                - ilitek,ili210x
>> +                - ilitek,ili2117
>> +                - ilitek,ili2120
>> +                - ilitek,ili251x
>> +    then:
>> +      required:
>> +        - interrupts
> 
> Generally, if there are special requirements, move these to dedicated
> yaml file to avoid complex if-else in trivial-touch.yaml.
Done in V2, thanks.