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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Tue, 26 Oct 2021 18:20:13 -0500
From:   Rob Herring <robh@...nel.org>
To:     Paul Cercueil <paul@...pouillou.net>
Cc:     Michael Turquette <mturquette@...libre.com>,
        Stephen Boyd <sboyd@...nel.org>, list@...ndingux.net,
        linux-clk@...r.kernel.org, devicetree@...r.kernel.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2 1/2] dt-bindings: Rename Ingenic CGU headers to
 ingenic,*.h

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@...pouillou.net>
> ---
> 
> 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@...nel.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.

Rob