* [PATCH 0/2] Move headers from dt-bindings/clk to clock
@ 2022-06-13 8:16 Lukas Bulwahn
2022-06-13 8:16 ` [PATCH 1/2] dt-bindings: clock: Move lochnagar.h to dt-bindings/clock Lukas Bulwahn
2022-06-13 8:16 ` [PATCH 2/2] dt-bindings: clock: Move versaclock.h " Lukas Bulwahn
0 siblings, 2 replies; 9+ messages in thread
From: Lukas Bulwahn @ 2022-06-13 8:16 UTC (permalink / raw)
To: Michael Turquette, Stephen Boyd, Joe Perches, linux-clk
Cc: Luca Ceresoli, Rob Herring, Krzysztof Kozlowski, Charles Keepax,
Richard Fitzgerald, devicetree, patches, kernel-janitors,
linux-kernel, Lukas Bulwahn
Michael, Stephen,
I was investigating the state of ownership of the files in
include/dt-bindings/ according to the MAINTAINERS file and noticed that
there are two directories, clk and clock, that contain the include files
for clock drivers mentioned in dt-bindings.
It seems that by mistake, files were added to include/dt-bindings/clk/
instead of include/dt-bindings/clock/ where the majority of files are
located.
In the past, others already took care of moving some files from
clk to clock in the commit 8e28918a85a0 ("dt-bindings: clock: Move
ti-dra7-atl.h to dt-bindings/clock") and commit 35d35aae8177
("dt-bindings: clock: Move at91.h to dt-bindigs/clock").
This patch series repeats this activity of moving files from clk to clock
for the latest state of the kernel repository.
Michael, Stephen, please pick this minor clean-up patches into your common
clock tree.
The references that needed to be adapted was identified with git grep;
I have not compile-tested these patches and I am relying that the
specific compile-test builds for these drivers will detect any potential
issues with this refactoring.
Joe, I was also considering to create a rule in checkpatch.pl to check if a
patch creates some file in and warn about it. What do you think? You think
this is worth a rule in checkpatch? If so, I will create a first RFC patch
for such a rule.
Best regards,
Lukas
Lukas Bulwahn (2):
dt-bindings: clock: Move lochnagar.h to dt-bindings/clock
dt-bindings: clock: Move versaclock.h to dt-bindings/clock
Documentation/devicetree/bindings/clock/idt,versaclock5.yaml | 4 ++--
Documentation/devicetree/bindings/mfd/cirrus,lochnagar.yaml | 2 +-
MAINTAINERS | 2 +-
arch/arm64/boot/dts/renesas/beacon-renesom-baseboard.dtsi | 2 +-
arch/arm64/boot/dts/renesas/beacon-renesom-som.dtsi | 2 +-
drivers/clk/clk-lochnagar.c | 2 +-
drivers/clk/clk-versaclock5.c | 2 +-
include/dt-bindings/{clk => clock}/lochnagar.h | 0
include/dt-bindings/{clk => clock}/versaclock.h | 0
9 files changed, 8 insertions(+), 8 deletions(-)
rename include/dt-bindings/{clk => clock}/lochnagar.h (100%)
rename include/dt-bindings/{clk => clock}/versaclock.h (100%)
--
2.17.1
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH 1/2] dt-bindings: clock: Move lochnagar.h to dt-bindings/clock
2022-06-13 8:16 [PATCH 0/2] Move headers from dt-bindings/clk to clock Lukas Bulwahn
@ 2022-06-13 8:16 ` Lukas Bulwahn
2022-06-13 9:52 ` Charles Keepax
` (2 more replies)
2022-06-13 8:16 ` [PATCH 2/2] dt-bindings: clock: Move versaclock.h " Lukas Bulwahn
1 sibling, 3 replies; 9+ messages in thread
From: Lukas Bulwahn @ 2022-06-13 8:16 UTC (permalink / raw)
To: Michael Turquette, Stephen Boyd, Joe Perches, linux-clk
Cc: Luca Ceresoli, Rob Herring, Krzysztof Kozlowski, Charles Keepax,
Richard Fitzgerald, devicetree, patches, kernel-janitors,
linux-kernel, Lukas Bulwahn
Most of the clock-related dt-binding header files are located in
include/dt-bindings/clock. It would be good to keep all the similar
header files at a single location.
This was discovered while investigating the state of ownership of the files
in include/dt-bindings/ according to the MAINTAINERS file.
This change here is similar to commit 8e28918a85a0 ("dt-bindings: clock:
Move ti-dra7-atl.h to dt-bindings/clock") and commit 35d35aae8177
("dt-bindings: clock: Move at91.h to dt-bindigs/clock").
Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>
---
Documentation/devicetree/bindings/mfd/cirrus,lochnagar.yaml | 2 +-
MAINTAINERS | 2 +-
drivers/clk/clk-lochnagar.c | 2 +-
include/dt-bindings/{clk => clock}/lochnagar.h | 0
4 files changed, 3 insertions(+), 3 deletions(-)
rename include/dt-bindings/{clk => clock}/lochnagar.h (100%)
diff --git a/Documentation/devicetree/bindings/mfd/cirrus,lochnagar.yaml b/Documentation/devicetree/bindings/mfd/cirrus,lochnagar.yaml
index ad285cb480c9..ef4814ced8db 100644
--- a/Documentation/devicetree/bindings/mfd/cirrus,lochnagar.yaml
+++ b/Documentation/devicetree/bindings/mfd/cirrus,lochnagar.yaml
@@ -261,7 +261,7 @@ additionalProperties: false
examples:
- |
- #include <dt-bindings/clk/lochnagar.h>
+ #include <dt-bindings/clock/lochnagar.h>
#include <dt-bindings/pinctrl/lochnagar.h>
i2c@e0004000 {
#address-cells = <1>;
diff --git a/MAINTAINERS b/MAINTAINERS
index 91e9cd30326d..86e6198b206f 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -4801,7 +4801,7 @@ F: drivers/hwmon/lochnagar-hwmon.c
F: drivers/mfd/lochnagar-i2c.c
F: drivers/pinctrl/cirrus/pinctrl-lochnagar.c
F: drivers/regulator/lochnagar-regulator.c
-F: include/dt-bindings/clk/lochnagar.h
+F: include/dt-bindings/clock/lochnagar.h
F: include/dt-bindings/pinctrl/lochnagar.h
F: include/linux/mfd/lochnagar*
F: sound/soc/codecs/lochnagar-sc.c
diff --git a/drivers/clk/clk-lochnagar.c b/drivers/clk/clk-lochnagar.c
index 565bcd0cdde9..80944bf482e9 100644
--- a/drivers/clk/clk-lochnagar.c
+++ b/drivers/clk/clk-lochnagar.c
@@ -19,7 +19,7 @@
#include <linux/mfd/lochnagar1_regs.h>
#include <linux/mfd/lochnagar2_regs.h>
-#include <dt-bindings/clk/lochnagar.h>
+#include <dt-bindings/clock/lochnagar.h>
#define LOCHNAGAR_NUM_CLOCKS (LOCHNAGAR_SPDIF_CLKOUT + 1)
diff --git a/include/dt-bindings/clk/lochnagar.h b/include/dt-bindings/clock/lochnagar.h
similarity index 100%
rename from include/dt-bindings/clk/lochnagar.h
rename to include/dt-bindings/clock/lochnagar.h
--
2.17.1
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH 2/2] dt-bindings: clock: Move versaclock.h to dt-bindings/clock
2022-06-13 8:16 [PATCH 0/2] Move headers from dt-bindings/clk to clock Lukas Bulwahn
2022-06-13 8:16 ` [PATCH 1/2] dt-bindings: clock: Move lochnagar.h to dt-bindings/clock Lukas Bulwahn
@ 2022-06-13 8:16 ` Lukas Bulwahn
2022-06-13 20:53 ` Luca Ceresoli
` (2 more replies)
1 sibling, 3 replies; 9+ messages in thread
From: Lukas Bulwahn @ 2022-06-13 8:16 UTC (permalink / raw)
To: Michael Turquette, Stephen Boyd, Joe Perches, linux-clk
Cc: Luca Ceresoli, Rob Herring, Krzysztof Kozlowski, Charles Keepax,
Richard Fitzgerald, devicetree, patches, kernel-janitors,
linux-kernel, Lukas Bulwahn
Most of the clock related dt-binding header files are located in
dt-bindings/clock folder. It would be good to keep all the similar
header files at a single location.
This was discovered while investigating the state of ownership of the
files in include/dt-bindings/ according to the MAINTAINERS file.
This change here is similar to commit 8e28918a85a0 ("dt-bindings: clock:
Move ti-dra7-atl.h to dt-bindings/clock") and commit 35d35aae8177
("dt-bindings: clock: Move at91.h to dt-bindigs/clock").
Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>
---
Documentation/devicetree/bindings/clock/idt,versaclock5.yaml | 4 ++--
arch/arm64/boot/dts/renesas/beacon-renesom-baseboard.dtsi | 2 +-
arch/arm64/boot/dts/renesas/beacon-renesom-som.dtsi | 2 +-
drivers/clk/clk-versaclock5.c | 2 +-
include/dt-bindings/{clk => clock}/versaclock.h | 0
5 files changed, 5 insertions(+), 5 deletions(-)
rename include/dt-bindings/{clk => clock}/versaclock.h (100%)
diff --git a/Documentation/devicetree/bindings/clock/idt,versaclock5.yaml b/Documentation/devicetree/bindings/clock/idt,versaclock5.yaml
index 7c331bfbe370..c1b838524064 100644
--- a/Documentation/devicetree/bindings/clock/idt,versaclock5.yaml
+++ b/Documentation/devicetree/bindings/clock/idt,versaclock5.yaml
@@ -108,7 +108,7 @@ patternProperties:
properties:
idt,mode:
description:
- The output drive mode. Values defined in dt-bindings/clk/versaclock.h
+ The output drive mode. Values defined in dt-bindings/clock/versaclock.h
$ref: /schemas/types.yaml#/definitions/uint32
minimum: 0
maximum: 6
@@ -151,7 +151,7 @@ additionalProperties: false
examples:
- |
- #include <dt-bindings/clk/versaclock.h>
+ #include <dt-bindings/clock/versaclock.h>
/* 25MHz reference crystal */
ref25: ref25m {
diff --git a/arch/arm64/boot/dts/renesas/beacon-renesom-baseboard.dtsi b/arch/arm64/boot/dts/renesas/beacon-renesom-baseboard.dtsi
index 63e7a39e100e..8166e3c1ff4e 100644
--- a/arch/arm64/boot/dts/renesas/beacon-renesom-baseboard.dtsi
+++ b/arch/arm64/boot/dts/renesas/beacon-renesom-baseboard.dtsi
@@ -5,7 +5,7 @@
#include <dt-bindings/gpio/gpio.h>
#include <dt-bindings/input/input.h>
-#include <dt-bindings/clk/versaclock.h>
+#include <dt-bindings/clock/versaclock.h>
/ {
backlight_lvds: backlight-lvds {
diff --git a/arch/arm64/boot/dts/renesas/beacon-renesom-som.dtsi b/arch/arm64/boot/dts/renesas/beacon-renesom-som.dtsi
index f5c1d74b738b..d3fc8ffd5b4c 100644
--- a/arch/arm64/boot/dts/renesas/beacon-renesom-som.dtsi
+++ b/arch/arm64/boot/dts/renesas/beacon-renesom-som.dtsi
@@ -4,7 +4,7 @@
*/
#include <dt-bindings/gpio/gpio.h>
-#include <dt-bindings/clk/versaclock.h>
+#include <dt-bindings/clock/versaclock.h>
/ {
memory@48000000 {
diff --git a/drivers/clk/clk-versaclock5.c b/drivers/clk/clk-versaclock5.c
index e7be3e54b9be..b8c556dbeba0 100644
--- a/drivers/clk/clk-versaclock5.c
+++ b/drivers/clk/clk-versaclock5.c
@@ -24,7 +24,7 @@
#include <linux/regmap.h>
#include <linux/slab.h>
-#include <dt-bindings/clk/versaclock.h>
+#include <dt-bindings/clock/versaclock.h>
/* VersaClock5 registers */
#define VC5_OTP_CONTROL 0x00
diff --git a/include/dt-bindings/clk/versaclock.h b/include/dt-bindings/clock/versaclock.h
similarity index 100%
rename from include/dt-bindings/clk/versaclock.h
rename to include/dt-bindings/clock/versaclock.h
--
2.17.1
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH 1/2] dt-bindings: clock: Move lochnagar.h to dt-bindings/clock
2022-06-13 8:16 ` [PATCH 1/2] dt-bindings: clock: Move lochnagar.h to dt-bindings/clock Lukas Bulwahn
@ 2022-06-13 9:52 ` Charles Keepax
2022-06-16 15:14 ` Rob Herring
2022-08-22 23:06 ` Stephen Boyd
2 siblings, 0 replies; 9+ messages in thread
From: Charles Keepax @ 2022-06-13 9:52 UTC (permalink / raw)
To: Lukas Bulwahn
Cc: Michael Turquette, Stephen Boyd, Joe Perches, linux-clk,
Luca Ceresoli, Rob Herring, Krzysztof Kozlowski,
Richard Fitzgerald, devicetree, patches, kernel-janitors,
linux-kernel
On Mon, Jun 13, 2022 at 10:16:31AM +0200, Lukas Bulwahn wrote:
> Most of the clock-related dt-binding header files are located in
> include/dt-bindings/clock. It would be good to keep all the similar
> header files at a single location.
>
> This was discovered while investigating the state of ownership of the files
> in include/dt-bindings/ according to the MAINTAINERS file.
>
> This change here is similar to commit 8e28918a85a0 ("dt-bindings: clock:
> Move ti-dra7-atl.h to dt-bindings/clock") and commit 35d35aae8177
> ("dt-bindings: clock: Move at91.h to dt-bindigs/clock").
>
> Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>
> ---
Acked-by: Charles Keepax <ckeepax@opensource.cirrus.com>
Thanks,
Charles
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 2/2] dt-bindings: clock: Move versaclock.h to dt-bindings/clock
2022-06-13 8:16 ` [PATCH 2/2] dt-bindings: clock: Move versaclock.h " Lukas Bulwahn
@ 2022-06-13 20:53 ` Luca Ceresoli
2022-06-16 15:14 ` Rob Herring
2022-08-22 23:07 ` Stephen Boyd
2 siblings, 0 replies; 9+ messages in thread
From: Luca Ceresoli @ 2022-06-13 20:53 UTC (permalink / raw)
To: Lukas Bulwahn, Michael Turquette, Stephen Boyd, Joe Perches, linux-clk
Cc: Rob Herring, Krzysztof Kozlowski, Charles Keepax,
Richard Fitzgerald, devicetree, patches, kernel-janitors,
linux-kernel
Hi Lukas,
On 13/06/22 10:16, Lukas Bulwahn wrote:
> Most of the clock related dt-binding header files are located in
> dt-bindings/clock folder. It would be good to keep all the similar
> header files at a single location.
>
> This was discovered while investigating the state of ownership of the
> files in include/dt-bindings/ according to the MAINTAINERS file.
>
> This change here is similar to commit 8e28918a85a0 ("dt-bindings: clock:
> Move ti-dra7-atl.h to dt-bindings/clock") and commit 35d35aae8177
> ("dt-bindings: clock: Move at91.h to dt-bindigs/clock").
>
> Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>
Reviewed-by: Luca Ceresoli <luca@lucaceresoli.net>
Reviewed-by: Luca Ceresoli <luca.ceresoli@bootlin.com>
(I'm switching to the Bootlin address, patches are out but the
MAINTAINERS one is not yet on master(
--
Luca
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 1/2] dt-bindings: clock: Move lochnagar.h to dt-bindings/clock
2022-06-13 8:16 ` [PATCH 1/2] dt-bindings: clock: Move lochnagar.h to dt-bindings/clock Lukas Bulwahn
2022-06-13 9:52 ` Charles Keepax
@ 2022-06-16 15:14 ` Rob Herring
2022-08-22 23:06 ` Stephen Boyd
2 siblings, 0 replies; 9+ messages in thread
From: Rob Herring @ 2022-06-16 15:14 UTC (permalink / raw)
To: Lukas Bulwahn
Cc: Luca Ceresoli, Krzysztof Kozlowski, Joe Perches,
Michael Turquette, linux-clk, linux-kernel, Richard Fitzgerald,
devicetree, Rob Herring, patches, Stephen Boyd, kernel-janitors,
Charles Keepax
On Mon, 13 Jun 2022 10:16:31 +0200, Lukas Bulwahn wrote:
> Most of the clock-related dt-binding header files are located in
> include/dt-bindings/clock. It would be good to keep all the similar
> header files at a single location.
>
> This was discovered while investigating the state of ownership of the files
> in include/dt-bindings/ according to the MAINTAINERS file.
>
> This change here is similar to commit 8e28918a85a0 ("dt-bindings: clock:
> Move ti-dra7-atl.h to dt-bindings/clock") and commit 35d35aae8177
> ("dt-bindings: clock: Move at91.h to dt-bindigs/clock").
>
> Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>
> ---
> Documentation/devicetree/bindings/mfd/cirrus,lochnagar.yaml | 2 +-
> MAINTAINERS | 2 +-
> drivers/clk/clk-lochnagar.c | 2 +-
> include/dt-bindings/{clk => clock}/lochnagar.h | 0
> 4 files changed, 3 insertions(+), 3 deletions(-)
> rename include/dt-bindings/{clk => clock}/lochnagar.h (100%)
>
Acked-by: Rob Herring <robh@kernel.org>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 2/2] dt-bindings: clock: Move versaclock.h to dt-bindings/clock
2022-06-13 8:16 ` [PATCH 2/2] dt-bindings: clock: Move versaclock.h " Lukas Bulwahn
2022-06-13 20:53 ` Luca Ceresoli
@ 2022-06-16 15:14 ` Rob Herring
2022-08-22 23:07 ` Stephen Boyd
2 siblings, 0 replies; 9+ messages in thread
From: Rob Herring @ 2022-06-16 15:14 UTC (permalink / raw)
To: Lukas Bulwahn
Cc: Michael Turquette, Krzysztof Kozlowski, devicetree,
Richard Fitzgerald, Luca Ceresoli, linux-kernel, Charles Keepax,
kernel-janitors, Rob Herring, patches, linux-clk, Stephen Boyd,
Joe Perches
On Mon, 13 Jun 2022 10:16:32 +0200, Lukas Bulwahn wrote:
> Most of the clock related dt-binding header files are located in
> dt-bindings/clock folder. It would be good to keep all the similar
> header files at a single location.
>
> This was discovered while investigating the state of ownership of the
> files in include/dt-bindings/ according to the MAINTAINERS file.
>
> This change here is similar to commit 8e28918a85a0 ("dt-bindings: clock:
> Move ti-dra7-atl.h to dt-bindings/clock") and commit 35d35aae8177
> ("dt-bindings: clock: Move at91.h to dt-bindigs/clock").
>
> Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>
> ---
> Documentation/devicetree/bindings/clock/idt,versaclock5.yaml | 4 ++--
> arch/arm64/boot/dts/renesas/beacon-renesom-baseboard.dtsi | 2 +-
> arch/arm64/boot/dts/renesas/beacon-renesom-som.dtsi | 2 +-
> drivers/clk/clk-versaclock5.c | 2 +-
> include/dt-bindings/{clk => clock}/versaclock.h | 0
> 5 files changed, 5 insertions(+), 5 deletions(-)
> rename include/dt-bindings/{clk => clock}/versaclock.h (100%)
>
Acked-by: Rob Herring <robh@kernel.org>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 1/2] dt-bindings: clock: Move lochnagar.h to dt-bindings/clock
2022-06-13 8:16 ` [PATCH 1/2] dt-bindings: clock: Move lochnagar.h to dt-bindings/clock Lukas Bulwahn
2022-06-13 9:52 ` Charles Keepax
2022-06-16 15:14 ` Rob Herring
@ 2022-08-22 23:06 ` Stephen Boyd
2 siblings, 0 replies; 9+ messages in thread
From: Stephen Boyd @ 2022-08-22 23:06 UTC (permalink / raw)
To: Joe Perches, Lukas Bulwahn, Michael Turquette, linux-clk
Cc: Luca Ceresoli, Rob Herring, Krzysztof Kozlowski, Charles Keepax,
Richard Fitzgerald, devicetree, patches, kernel-janitors,
linux-kernel, Lukas Bulwahn
Quoting Lukas Bulwahn (2022-06-13 01:16:31)
> Most of the clock-related dt-binding header files are located in
> include/dt-bindings/clock. It would be good to keep all the similar
> header files at a single location.
>
> This was discovered while investigating the state of ownership of the files
> in include/dt-bindings/ according to the MAINTAINERS file.
>
> This change here is similar to commit 8e28918a85a0 ("dt-bindings: clock:
> Move ti-dra7-atl.h to dt-bindings/clock") and commit 35d35aae8177
> ("dt-bindings: clock: Move at91.h to dt-bindigs/clock").
>
> Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>
> ---
Applied to clk-next
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 2/2] dt-bindings: clock: Move versaclock.h to dt-bindings/clock
2022-06-13 8:16 ` [PATCH 2/2] dt-bindings: clock: Move versaclock.h " Lukas Bulwahn
2022-06-13 20:53 ` Luca Ceresoli
2022-06-16 15:14 ` Rob Herring
@ 2022-08-22 23:07 ` Stephen Boyd
2 siblings, 0 replies; 9+ messages in thread
From: Stephen Boyd @ 2022-08-22 23:07 UTC (permalink / raw)
To: Joe Perches, Lukas Bulwahn, Michael Turquette, linux-clk
Cc: Luca Ceresoli, Rob Herring, Krzysztof Kozlowski, Charles Keepax,
Richard Fitzgerald, devicetree, patches, kernel-janitors,
linux-kernel, Lukas Bulwahn
Quoting Lukas Bulwahn (2022-06-13 01:16:32)
> Most of the clock related dt-binding header files are located in
> dt-bindings/clock folder. It would be good to keep all the similar
> header files at a single location.
>
> This was discovered while investigating the state of ownership of the
> files in include/dt-bindings/ according to the MAINTAINERS file.
>
> This change here is similar to commit 8e28918a85a0 ("dt-bindings: clock:
> Move ti-dra7-atl.h to dt-bindings/clock") and commit 35d35aae8177
> ("dt-bindings: clock: Move at91.h to dt-bindigs/clock").
>
> Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>
> ---
Applied to clk-next
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2022-08-22 23:07 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-06-13 8:16 [PATCH 0/2] Move headers from dt-bindings/clk to clock Lukas Bulwahn
2022-06-13 8:16 ` [PATCH 1/2] dt-bindings: clock: Move lochnagar.h to dt-bindings/clock Lukas Bulwahn
2022-06-13 9:52 ` Charles Keepax
2022-06-16 15:14 ` Rob Herring
2022-08-22 23:06 ` Stephen Boyd
2022-06-13 8:16 ` [PATCH 2/2] dt-bindings: clock: Move versaclock.h " Lukas Bulwahn
2022-06-13 20:53 ` Luca Ceresoli
2022-06-16 15:14 ` Rob Herring
2022-08-22 23:07 ` Stephen Boyd
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.