On 25-03-20 15:28:49, Kajetan Puchalski wrote: > > > > > +ARM/CIX SKY ARM ARCHITECTURE > > > > +M: Peter Chen <peter.chen@xxxxxxxxxxx> > > > > +M: Fugang Duan <fugang.duan@xxxxxxxxxxx> > > > > +R: CIX Linux Kernel Upstream Group <cix-kernel-upstream@xxxxxxxxxxx> > > > > +L: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx (moderated for non-subscribers) > > > > +S: Maintained > > > > +F: Documentation/devicetree/bindings/arm/cix.yaml > > > > +F: arch/arm64/boot/dts/cix/sky1-orion-o6.dts > > > > +F: arch/arm64/boot/dts/cix/sky1.dtsi > > > > > > Instead: > > > arch/arm64/boot/dts/cix/ > > > > > > or if you think there will be more different architectures coming from > > > cix, which you will not maintain, then useful would be a "sky" subdirectory. > > > > > Hi Krzysztof, > > > > Thanks for your comment, CIX is Arm SoC based silicon design > > company, we may have different SoC style for middle end in future. > > So, for that SoC series, it may not name sky. > > I believe the idea is that the file should either say > > +F: arch/arm64/boot/dts/cix/ > > if the two of you intend to be maintainers for everything under cix/. > You don't need to list every specific file. > > Otherwise it should say > > +F: arch/arm64/boot/dts/cix/sky/ > > with its own subdirectory if you only intend to be maintaining the sky family. Hi Kajetan & Krzysztof, Thanks for your valuable comments. We prefer maintain the entire CIX architecture under one entry, I will change maintain information like below: ARM/CIX SOC SUPPORT M: Peter Chen <peter.chen@xxxxxxxxxxx> M: Fugang Duan <fugang.duan@xxxxxxxxxxx> R: CIX Linux Kernel Upstream Group <cix-kernel-upstream@xxxxxxxxxxx> L: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx (moderated for non-subscribers) S: Maintained F: Documentation/devicetree/bindings/arm/cix.yaml F: arch/arm64/boot/dts/cix/ -- Best regards, Peter