linux-kernel - Re:Re: [PATCH v2 5/8] dt-bindings: display: vop2: Add rk3576 support

Open Source and information security mailing list archives

Message-ID: <4847f6b1.1c60.19412092a7a.Coremail.andyshrk@163.com>
Date: Sun, 29 Dec 2024 18:49:38 +0800 (CST)
From: "Andy Yan" <andyshrk@....com>
To: "Krzysztof Kozlowski" <krzk@...nel.org>
Cc: robh@...nel.org, iommu@...ts.linux.dev, heiko@...ech.de, 
	devicetree@...r.kernel.org, joro@...tes.org, hjc@...k-chips.com, 
	dri-devel@...ts.freedesktop.org, linux-kernel@...r.kernel.org, 
	linux-phy@...ts.infradead.org, vkoul@...nel.org, 
	detlev.casanova@...labora.com, linux-arm-kernel@...ts.infradead.org, 
	"Andy Yan" <andy.yan@...k-chips.com>, 
	linux-rockchip@...ts.infradead.org, krzk+dt@...nel.org, 
	cl@...k-chips.com
Subject: Re:Re: [PATCH v2 5/8] dt-bindings: display: vop2: Add rk3576
 support

Hi Krzysztof,

At 2024-12-29 18:13:39, "Krzysztof Kozlowski" <krzk@...nel.org> wrote:
>On Sat, Dec 28, 2024 at 08:21:48PM +0800, Andy Yan wrote:
>> From: Andy Yan <andy.yan@...k-chips.com>
>> 
>> Add vop found on rk3576, the main difference between rk3576 and the
>> previous vop is that each VP has its own interrupt line.
>> 
>> Signed-off-by: Andy Yan <andy.yan@...k-chips.com>
>> 
>> ---
>> 
>> Changes in v2:
>> - describe constraint SOC by SOC, as interrupts of rk3576 is very
>>   different from others
>> - Drop Krzysztof's Reviewed-by, as this version changed a lot.
>> 
>> Changes in v1:
>> - ordered by soc name
>> - Add description for newly added interrupt
>> 
>>  .../display/rockchip/rockchip-vop2.yaml       | 70 +++++++++++++++----
>>  1 file changed, 56 insertions(+), 14 deletions(-)
>> 
>> diff --git a/Documentation/devicetree/bindings/display/rockchip/rockchip-vop2.yaml b/Documentation/devicetree/bindings/display/rockchip/rockchip-vop2.yaml
>> index 2531726af306..f28cea4a6d82 100644
>> --- a/Documentation/devicetree/bindings/display/rockchip/rockchip-vop2.yaml
>> +++ b/Documentation/devicetree/bindings/display/rockchip/rockchip-vop2.yaml
>> @@ -20,6 +20,7 @@ properties:
>>      enum:
>>        - rockchip,rk3566-vop
>>        - rockchip,rk3568-vop
>> +      - rockchip,rk3576-vop
>>        - rockchip,rk3588-vop
>>  
>>    reg:
>> @@ -37,10 +38,22 @@ properties:
>>        - const: gamma-lut
>>  
>>    interrupts:
>> -    maxItems: 1
>> -    description:
>> -      The VOP interrupt is shared by several interrupt sources, such as
>> -      frame start (VSYNC), line flag and other status interrupts.
>> +    minItems: 1
>> +    maxItems: 4
>> +    items:
>> +      - description:
>
>That's contradictory. min/max say 1-4, but here you list two interrupts.

There are 4 interrupts on rk3576, and the other SOC only have one.
I am sorry that I am not familiar with this schema language; I have read many
other YAML bindings these days and try to write it correctly.

Maybe I should not list the interrupts-names here(then list the interrupt-name by soc at allOf:  part),
 like the clocks handles in samsung,usb3-drd-phy.yaml  ?

>
>> +          vop system interrupt, such as bus error, and vsync for vop version under
>> +          rk3576.
>> +      - description:
>> +          independent interrupts for each video port, such as vsync and other video
>> +          port related error interrupts.
>> +
>> +  interrupt-names:
>> +    items:
>> +      - const: vop-sys
>> +      - const: vop-vp0
>> +      - const: vop-vp1
>> +      - const: vop-vp2
>
>And this says exactly 4.
>
>Best regards,
>Krzysztof
>
>
>_______________________________________________
>Linux-rockchip mailing list
>Linux-rockchip@...ts.infradead.org
>http://lists.infradead.org/mailman/listinfo/linux-rockchip

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