Re: [PATCH RFC 01/13] dt-bindings: soc: starfive: Add vout-subsystem IP block

[Conor Dooley <conor@kernel.org>]

[-- Attachment #1: Type: text/plain, Size: 1494 bytes --]

On Wed, Nov 12, 2025 at 02:34:39PM +0800, Icenowy Zheng wrote:
> 在 2025-11-11星期二的 18:36 +0000，Conor Dooley写道：
> > On Tue, Nov 11, 2025 at 06:18:16PM +0000, Conor Dooley wrote:
> > > On Sat, Nov 08, 2025 at 02:04:35AM +0100, Michal Wilczynski wrote:
> > > > Add the dt-binding documentation for the StarFive JH7110 Video

> > > > +patternProperties:
> > > > +  "^display@[0-9a-f]+$":
> > > 
> > > Personally I'd like to see these being regular properties, since
> > > there's
> > > exactly one possible setup for this.
> > > 
> > > > +    type: object
> > > > +    description: Verisilicon DC8200 Display Controller node.
> > > 
> > > Can you add the relevant references here instead of allowing any
> > > object?
> > 
> > I don't think that if you did, this would pass the binding checks,
> > because there's no "verisilicon,dc" binding. I think I saw one in
> > progress, but without the soc-specific compatible that I am going to
> > require here - if for no reason other than making sure that the
> > clocks
> > etc are provided correctly for this device.
> 
> Well I didn't specify any soc-specific compatible because that IP has
> its own identification registers.

I still require one because I want to make sure that clocks etc are
handled correctly. You can ignore it in the driver if you wish, but when
the next user comes along with one more or less clock, I want the
jh7110 one to be forced to use the correct configuration.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]