Re: [PATCH v3 04/19] dt-bindings: soc: starfive: Add starfive,jh7110-vout-syscon

[Michal Wilczynski]

On 9/10/26 10:18, Krzysztof Kozlowski wrote:
> On Fri, Sep 04, 2026 at 03:27:05PM +0200, Michal Wilczynski wrote:
>> The video output subsystem has its own system controller, documented as
>> dom_vout_syscon. Among other things it holds the mux that selects which
>> DC8200 output drives the HDMI transmitter, so the HDMI controller needs
>> to reach it.
>>
>> Signed-off-by: Michal Wilczynski <m.wilczynski@xxxxxxxxxxx>
>> ---
>>  .../devicetree/bindings/soc/starfive/starfive,jh7110-syscon.yaml    | 6 ++++++
>>  1 file changed, 6 insertions(+)
>>
>> diff --git a/Documentation/devicetree/bindings/soc/starfive/starfive,jh7110-syscon.yaml b/Documentation/devicetree/bindings/soc/starfive/starfive,jh7110-syscon.yaml
>> index 0039319e91fe6e6aadc4300114626348e4dfb7ce..47334f6c1243c9950e6fba44daacd41f42b55781 100644
>> --- a/Documentation/devicetree/bindings/soc/starfive/starfive,jh7110-syscon.yaml
>> +++ b/Documentation/devicetree/bindings/soc/starfive/starfive,jh7110-syscon.yaml
>> @@ -24,6 +24,7 @@ properties:
>>            - enum:
>>                - starfive,jh7110-aon-syscon
>>                - starfive,jh7110-stg-syscon
>> +              - starfive,jh7110-vout-syscon
>>            - const: syscon
>>  
>>    reg:
>> @@ -90,4 +91,9 @@ examples:
>>          #power-domain-cells = <1>;
>>      };
>>  
>> +    syscon@295b0000 {
> 
> Don't need new example for every compatible. You already have three...
> and this one is exactly the same as the first one. Drop.
> 
> Best regards,
> Krzysztof
> 
> 

Will do, thanks !

Best regards,
-- 
Michal Wilczynski <m.wilczynski@xxxxxxxxxxx>