Re: [PATCH v2 1/2] dt-bindings: Rename Ingenic CGU headers to ingenic,*.h

From: Rob Herring
Date: Tue Oct 26 2021 - 19:20:29 EST

On Sat, Oct 16, 2021 at 02:33:21PM +0100, Paul Cercueil wrote:
> Tidy up a bit the tree, by prefixing all include/dt-bindings/clock/ files
> related to Ingenic SoCs with 'ingenic,'.
>
> Signed-off-by: Paul Cercueil <paul@xxxxxxxxxxxxxxx>
> ---
>
> Notes:
> v2: Fix x1000-cgu.h and x1830-cgu.h incorrectly renamed to
> x1000.h / x1830.h
>
> Documentation/devicetree/bindings/clock/ingenic,cgu.yaml | 2 +-
> Documentation/devicetree/bindings/display/ingenic,ipu.yaml | 2 +-
> Documentation/devicetree/bindings/display/ingenic,lcd.yaml | 4 ++--
> Documentation/devicetree/bindings/dma/ingenic,dma.yaml | 2 +-
> Documentation/devicetree/bindings/i2c/ingenic,i2c.yaml | 2 +-
> Documentation/devicetree/bindings/iio/adc/ingenic,adc.yaml | 2 +-
> .../devicetree/bindings/memory-controllers/ingenic,nemc.yaml | 2 +-
> .../devicetree/bindings/mips/ingenic/ingenic,cpu.yaml | 2 +-
> Documentation/devicetree/bindings/mmc/ingenic,mmc.yaml | 2 +-
> Documentation/devicetree/bindings/mtd/ingenic,nand.yaml | 2 +-
> Documentation/devicetree/bindings/net/ingenic,mac.yaml | 2 +-
> .../devicetree/bindings/nvmem/ingenic,jz4780-efuse.yaml | 2 +-
> Documentation/devicetree/bindings/phy/ingenic,phy-usb.yaml | 2 +-
> Documentation/devicetree/bindings/remoteproc/ingenic,vpu.yaml | 2 +-
> Documentation/devicetree/bindings/rng/ingenic,trng.yaml | 2 +-
> Documentation/devicetree/bindings/rtc/ingenic,rtc.yaml | 2 +-
> Documentation/devicetree/bindings/serial/ingenic,uart.yaml | 2 +-
> Documentation/devicetree/bindings/sound/ingenic,aic.yaml | 2 +-
> Documentation/devicetree/bindings/sound/ingenic,codec.yaml | 2 +-
> Documentation/devicetree/bindings/timer/ingenic,sysost.yaml | 2 +-
> Documentation/devicetree/bindings/timer/ingenic,tcu.yaml | 2 +-
> Documentation/devicetree/bindings/usb/ingenic,musb.yaml | 2 +-
> arch/mips/boot/dts/ingenic/jz4725b.dtsi | 2 +-
> arch/mips/boot/dts/ingenic/jz4740.dtsi | 2 +-
> arch/mips/boot/dts/ingenic/jz4770.dtsi | 2 +-
> arch/mips/boot/dts/ingenic/jz4780.dtsi | 2 +-
> arch/mips/boot/dts/ingenic/x1000.dtsi | 2 +-
> arch/mips/boot/dts/ingenic/x1830.dtsi | 2 +-
> drivers/clk/ingenic/jz4725b-cgu.c | 2 +-
> drivers/clk/ingenic/jz4740-cgu.c | 2 +-
> drivers/clk/ingenic/jz4760-cgu.c | 2 +-
> drivers/clk/ingenic/jz4770-cgu.c | 2 +-
> drivers/clk/ingenic/jz4780-cgu.c | 2 +-
> drivers/clk/ingenic/x1000-cgu.c | 2 +-
> drivers/clk/ingenic/x1830-cgu.c | 2 +-
> .../clock/{jz4725b-cgu.h => ingenic,jz4725b-cgu.h} | 0
> .../dt-bindings/clock/{jz4740-cgu.h => ingenic,jz4740-cgu.h} | 0
> .../dt-bindings/clock/{jz4760-cgu.h => ingenic,jz4760-cgu.h} | 0
> .../dt-bindings/clock/{jz4770-cgu.h => ingenic,jz4770-cgu.h} | 0
> .../dt-bindings/clock/{jz4780-cgu.h => ingenic,jz4780-cgu.h} | 0
> .../dt-bindings/clock/{x1000-cgu.h => ingenic,x1000-cgu.h} | 0
> .../dt-bindings/clock/{x1830-cgu.h => ingenic,x1830-cgu.h} | 0
> 42 files changed, 36 insertions(+), 36 deletions(-)
> rename include/dt-bindings/clock/{jz4725b-cgu.h => ingenic,jz4725b-cgu.h} (100%)
> rename include/dt-bindings/clock/{jz4740-cgu.h => ingenic,jz4740-cgu.h} (100%)
> rename include/dt-bindings/clock/{jz4760-cgu.h => ingenic,jz4760-cgu.h} (100%)
> rename include/dt-bindings/clock/{jz4770-cgu.h => ingenic,jz4770-cgu.h} (100%)
> rename include/dt-bindings/clock/{jz4780-cgu.h => ingenic,jz4780-cgu.h} (100%)
> rename include/dt-bindings/clock/{x1000-cgu.h => ingenic,x1000-cgu.h} (100%)
> rename include/dt-bindings/clock/{x1830-cgu.h => ingenic,x1830-cgu.h} (100%)

Acked-by: Rob Herring <robh@xxxxxxxxxx>

However, this will be a pain to merge if there are any new users coming
in during the merge window. If not, then Stephen can just take this. If
there are, then at the end of the merge window is the best time. For
that to work, you need to track when all dependent changes are merged
and we need to agree who will pick this up. I usually have fixes from
breakages in the merge window anyways.

Rob