On Sun, Apr 14, 2024 at 5:45 PM Florian Fainelli <florian.fainelli@xxxxxxxxxxxx> wrote: > On 4/13/2024 3:14 PM, Andrea della Porta wrote: > > Signed-off-by: Andrea della Porta <andrea.porta@xxxxxxxx> > > +++ b/Documentation/devicetree/bindings/pinctrl/brcm,bcm2712-pinctrl.yaml (...) > > +$id: http://devicetree.org/schemas/pinctrl/brcm,bcm2712-pinctrl.yaml# (...) > > +title: Broadcom BCM2712 pin controller > > This is not strictly speaking BCM2712 specific, the pin controller you > describe is a Broadcom STB product line pin controller. > > Please describe it as such as and make BCM2712 a specific instance of > the chip using that pin controller, see more comments on patch #4. So also the name of the bindings namespace should not be this one controller IMO but the name of the family, bcm-stb-pinctrl.yaml or so. Yours, Linus Walleij
