linux-kernel - Re: [PATCH v3 6/9] dt-bindings: interrupt-controller: realtek,rtl-intc: Add rtl9300-intc

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: <0ffcb229-0feb-4bc8-bbb2-6a9143401e88@alliedtelesis.co.nz>
Date: Mon, 1 Jul 2024 03:02:27 +0000
From: Chris Packham <Chris.Packham@...iedtelesis.co.nz>
To: Krzysztof Kozlowski <krzk@...nel.org>, "tglx@...utronix.de"
	<tglx@...utronix.de>, "robh@...nel.org" <robh@...nel.org>,
	"krzk+dt@...nel.org" <krzk+dt@...nel.org>, "conor+dt@...nel.org"
	<conor+dt@...nel.org>, "tsbogend@...ha.franken.de"
	<tsbogend@...ha.franken.de>, "daniel.lezcano@...aro.org"
	<daniel.lezcano@...aro.org>, "paulburton@...nel.org" <paulburton@...nel.org>,
	"peterz@...radead.org" <peterz@...radead.org>, "mail@...ger-koblitz.de"
	<mail@...ger-koblitz.de>, "bert@...t.com" <bert@...t.com>, "john@...ozen.org"
	<john@...ozen.org>, "sander@...nheule.net" <sander@...nheule.net>
CC: "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
	"linux-mips@...r.kernel.org" <linux-mips@...r.kernel.org>, "kabel@...nel.org"
	<kabel@...nel.org>, "ericwouds@...il.com" <ericwouds@...il.com>
Subject: Re: [PATCH v3 6/9] dt-bindings: interrupt-controller:
 realtek,rtl-intc: Add rtl9300-intc


On 1/07/24 10:12, Chris Packham wrote:
>
> On 27/06/24 19:41, Krzysztof Kozlowski wrote:
>> On 27/06/2024 06:33, Chris Packham wrote:
>>> Add a compatible string for the interrupt controller found on the
>>> rtl930x SoCs. The interrupt controller has registers for VPE1 so these
>>> are added as a second reg cell.
>>>
>>> Signed-off-by: Chris Packham <chris.packham@...iedtelesis.co.nz>
>>> ---
>>>
>>> Notes:
>>>      Changes in v3:
>>>      - Use items to describe the regs property
>>>      Changes in v2:
>>>      - Set reg:maxItems to 2 to allow for VPE1 registers on the 
>>> rtl9300. Add
>>>        a condition to enforce the old limit on other SoCs.
>>>      - Connor and Krzysztof offered acks on v1 but I think the 
>>> changes here
>>>        are big enough to void those.
>>>
>>>   .../interrupt-controller/realtek,rtl-intc.yaml | 18 
>>> +++++++++++++++++-
>>>   1 file changed, 17 insertions(+), 1 deletion(-)
>>>
>>> diff --git 
>>> a/Documentation/devicetree/bindings/interrupt-controller/realtek,rtl-intc.yaml 
>>> b/Documentation/devicetree/bindings/interrupt-controller/realtek,rtl-intc.yaml 
>>>
>>> index fb5593724059..d0e5bdf45d05 100644
>>> --- 
>>> a/Documentation/devicetree/bindings/interrupt-controller/realtek,rtl-intc.yaml
>>> +++ 
>>> b/Documentation/devicetree/bindings/interrupt-controller/realtek,rtl-intc.yaml
>>> @@ -25,6 +25,7 @@ properties:
>>>         - items:
>>>             - enum:
>>>                 - realtek,rtl8380-intc
>>> +              - realtek,rtl9300-intc
>>>             - const: realtek,rtl-intc
>>>         - const: realtek,rtl-intc
>>>           deprecated: true
>>> @@ -35,7 +36,9 @@ properties:
>>>       const: 1
>>>       reg:
>>> -    maxItems: 1
>> Missing minItems (as testing would tell you)
>
> Hmm I really did test this. Even now running
>
> `make ARCH=mips O=build_mips 
> DT_SCHEMA_FILES=realtek,rtl-intc.yaml:realtek-rtl.yaml:realtek,otto-timer.yaml 
> clean dt_binding_check` doesn't seem to yield any complaints. Am I 
> testing this properly?

Looks like there is a difference between 
`DT_SCHEMA_FILES=realtek,rtl-intc.yaml` and 
`DT_SCHEMA_FILES=realtek,rtl-intc.yaml:realtek-rtl.yaml:realtek,otto-timer.yaml`.

>
>
>>
>>> +    items:
>>> +      - description: vpe0 registers
>>> +      - description: vpe1 registers
>>>       interrupts:
>>>       minItems: 1
>>> @@ -71,6 +74,19 @@ allOf:
>>>       else:
>>>         required:
>>>           - interrupts
>>> +  - if:
>>> +      properties:
>>> +        compatible:
>>> +          contains:
>>> +            const: realtek,rtl9300-intc
>>> +    then:
>>> +      properties:
>>> +        reg:
>> Needed is: minItems: 2
>>
>>> +          maxItems: 2
>>> +    else:
>>> +      properties:
>>> +        reg:
>>> +          maxItems: 1
>>>     additionalProperties: false
>> Best regards,
>> Krzysztof
>>