Hi Rob, Sorry to disturb you. Could you please help to check the comments in last mail when you are free? Best Regards, Yunfei Dong On Thu, 2022-11-17 at 10:16 +0800, yunfei.dong wrote: > Hi Rob, > > Thanks for your comments. > On Wed, 2022-11-16 at 11:29 -0600, Rob Herring wrote: > > On Thu, Nov 10, 2022 at 06:28:32PM +0800, Yunfei Dong wrote: > > > mt8195 and mt8192 have different clock numbers, can't write > > > 'clocks' and > > > 'clock-names' with const value. > > > > Not a compatible change. Explain why that is okay if it is. > > > > This change is used for mt8195 platform for some architecture > changed. > Need to separate vcodec-lat with vcodec-lat-soc into different child > node. > > At the same time, vcodec-lat-soc don't have interrupt, but having > power > domain and clks. > > > > > > Move 'assigned-clocks' and 'assigned-clock-parents' to parent > > > node. > > > > > > Signed-off-by: Yunfei Dong <yunfei.dong@xxxxxxxxxxxx> > > > --- > > > .../media/mediatek,vcodec-subdev-decoder.yaml | 119 +++++++++++- > > > -- > > > ---- > > > 1 file changed, 72 insertions(+), 47 deletions(-) > > > > > > diff --git > > > a/Documentation/devicetree/bindings/media/mediatek,vcodec-subdev- > > > decoder.yaml > > > b/Documentation/devicetree/bindings/media/mediatek,vcodec-subdev- > > > decoder.yaml > > > index c4f20acdc1f8..794012853834 100644 > > > --- a/Documentation/devicetree/bindings/media/mediatek,vcodec- > > > subdev-decoder.yaml > > > +++ b/Documentation/devicetree/bindings/media/mediatek,vcodec- > > > subdev-decoder.yaml > > > @@ -89,23 +89,33 @@ properties: > > > > > > ranges: true > > > > > > + clocks: > > > + minItems: 1 > > > + maxItems: 5 > > > + > > > + clock-names: > > > + minItems: 1 > > > + maxItems: 5 > > > > Why do both the parent and child have clocks? > > > > If move assigned-clock-parents to child node, need to add 'ssigned- > clock-parents' and 'assigned-clocks' for each child node. Only need > to > add one in parent node, child node no need to add if add 'ssigned- > clock-parents' and 'assigned-clocks' in parent node. > > Adding 'assigned-clock-parents' and 'assigned-clocks' need to add > 'clocks' and 'clock-names', or will check fail. > > > + > > > + assigned-clocks: > > > + maxItems: 1 > > > + > > > + assigned-clock-parents: > > > + maxItems: 1 > > > > You can just drop assigned-clock properties. They are allowed in > > any > > node with 'clocks'. > > > > Only need to add one in parent node, or need to add for each child > node. > > > + > > > # Required child node: > > > patternProperties: > > > - '^vcodec-lat@[0-9a-f]+$': > > > + '^vcodec-lat-soc@[0-9a-f]+$': > > > type: object > > > > > > properties: > > > compatible: > > > enum: > > > - - mediatek,mtk-vcodec-lat > > > - mediatek,mtk-vcodec-lat-soc > > > > > > reg: > > > maxItems: 1 > > > > > > - interrupts: > > > - maxItems: 1 > > > - > > > > Dropping interrupts? Not explained in the commit msg (why?). > > > > vcodec-lat-soc no need interrupts, will add detail commit message in > next patch. > > > iommus: > > > minItems: 1 > > > maxItems: 32 > > > @@ -114,22 +124,55 @@ patternProperties: > > > Refer to bindings/iommu/mediatek,iommu.yaml. > > > > > > clocks: > > > + minItems: 1 > > > maxItems: 5 > > > > > > clock-names: > > > - items: > > > - - const: sel > > > - - const: soc-vdec > > > - - const: soc-lat > > > - - const: vdec > > > - - const: top > > > + minItems: 1 > > > + maxItems: 5 > > > > We had names defined and now we don't. That's a step backwards. > > > > Mt8195/mt8192/mt8186/mt8188 have different clock number and clock > names, so change it like this, do you have any other suggestion? > > > > > > - assigned-clocks: > > > + power-domains: > > > > Adding power-domains? > > Vcodec-lat-soc need power domain and add one new child node vcodec- > lat- > soc. > > Best Regards, > Yunfei Dong > > > > Rob