Am Donnerstag, 5. Mai 2022, 08:41:31 CEST schrieb Sascha Hauer: > On Thu, May 05, 2022 at 02:28:24AM +0200, Heiko Stübner wrote: > > Am Freitag, 22. April 2022, 09:28:33 CEST schrieb Sascha Hauer: > > > The VOP2 is the display output controller on the RK3568. Add the node > > > for it to the dtsi file along with the required display-subsystem node > > > and the iommu node. > > > > > > Signed-off-by: Sascha Hauer <s.hauer@xxxxxxxxxxxxxx> > > > Acked-by: Rob Herring <robh@xxxxxxxxxx> > > > --- > > > + vop: vop@fe040000 { > > > + reg = <0x0 0xfe040000 0x0 0x3000>, <0x0 0xfe044000 0x0 0x1000>; > > > + reg-names = "regs", "gamma_lut"; > > > > reg-names does not seem to be part of the dt-binding. > > Though now in the vop2-driver I see that the code uses that naming. > > > > I guess we might want to just go the vop1-way by using numbered > > platform-resources via a follow-up patch and drop the regnames here? > > Or we could document the reg-names properties in a follow-up patch. > Wouldn't that be better? Also a possibility. Although that of course leads us into the naming bike-shed ;-) ... i.e. for reg-names I'd do "vop" + "gamme-lut" (dash instead of underscore) - instead of the ones used right now. While using numbered resources would keep the commonality between both vops