Hi Linus, On Tue, Oct 4, 2022 at 10:50 AM Linus Walleij <linus.walleij@xxxxxxxxxx> wrote: > On Fri, Sep 30, 2022 at 11:28 PM Rob Herring <robh@xxxxxxxxxx> wrote: > > On Fri, 30 Sep 2022 14:14:04 +0800, Hal Feng wrote: > > > From: Jianlong Huang <jianlong.huang@xxxxxxxxxxxxxxxx> > > > > > > Add the SoC name to make it more clear. Also the next generation StarFive > > > SoCs will use "pinctrl-starfive" as the core of StarFive pinctrl driver. > > > No functional change. > > > > > > Signed-off-by: Jianlong Huang <jianlong.huang@xxxxxxxxxxxxxxxx> > > > Signed-off-by: Hal Feng <hal.feng@xxxxxxxxxxxxxxxxxxxxxx> > > > --- > > > .../bindings/pinctrl/starfive,jh7100-pinctrl.yaml | 2 +- > > > arch/riscv/boot/dts/starfive/jh7100-beaglev-starlight.dts | 2 +- > > > drivers/pinctrl/starfive/Kconfig | 2 +- > > > drivers/pinctrl/starfive/Makefile | 2 +- > > > .../{pinctrl-starfive.c => pinctrl-starfive-jh7100.c} | 2 +- > > > .../{pinctrl-starfive.h => pinctrl-starfive-jh7100.h} | 6 +++--- > > > 6 files changed, 8 insertions(+), 8 deletions(-) > > > rename drivers/pinctrl/starfive/{pinctrl-starfive.c => pinctrl-starfive-jh7100.c} (99%) > > > rename include/dt-bindings/pinctrl/{pinctrl-starfive.h => pinctrl-starfive-jh7100.h} (98%) > > > > > > > Reviewed-by: Rob Herring <robh@xxxxxxxxxx> > > > > Would be good to pull this out separately and apply for 6.1. It's kind > > of messy with cross tree dependencies. > > OK I applied this for V6.1. Isn't the name of the DT bindings header file part of the DT bindings, i.e. it cannot be changed afterwards? As of v5.17, it is in active use by arch/riscv/boot/dts/starfive/jh7100-beaglev-starlight.dts. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds