Thread (10 messages) 10 messages, 2 authors, 2021-11-10

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

From: Paul Cercueil <paul@crapouillou.net>
Date: 2021-10-28 11:46:24
Also in: linux-clk, lkml

Hi Rob,

Le mar., oct. 26 2021 at 18:20:13 -0500, Rob Herring [off-list ref] 
a écrit :
On Sat, Oct 16, 2021 at 02:33:21PM +0100, Paul Cercueil wrote:
quoted
 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 [off-list ref]
 ---

 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@kernel.org>

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.
I get Cc'd on all ingenic-related patches, so far I don't see anything 
that will cause a breakage. I think it would be okay to merge it now 
(we're at -rc7 after all).

Cheers,
-Paul

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help