Thread (92 messages) 92 messages, 7 authors, 2017-12-29

Re: [PATCH 04/15] clk: Add Ingenic jz4770 CGU driver

From: Stephen Boyd <hidden>
Date: 2017-06-26 22:57:30
Also in: linux-clk, linux-mips, lkml

On 06/26, Paul Cercueil wrote:
Hi,

Le 2017-06-07 22:59, Stephen Boyd a écrit :
quoted
On 06/07, Paul Cercueil wrote:
quoted
Add support for the clocks provided by the CGU in the Ingenic JZ4770
SoC.

Signed-off-by: Paul Cercueil <paul@crapouillou.net>
Signed-off-by: Maarten ter Huurne <redacted>
[...]
quoted
quoted
diff --git a/include/dt-bindings/clock/jz4770-cgu.h
b/include/dt-bindings/clock/jz4770-cgu.h
new file mode 100644
index 000000000000..54b8b2ae4a73
--- /dev/null
+++ b/include/dt-bindings/clock/jz4770-cgu.h
Can you split this file off into a different patch? That way clk
tree can apply clk patches on top of a stable branch where this
file lives by itself.
Oops, I forgot that in the v2.

The jz4770-cgu.c file includes and uses
<dt-bindings/clock/jz4770-cgu.h>, so I don't think
it would make sense to split it, since it wouldn't compile without it.
I was suggesting this header file be a patch before the driver C
file patch, so that it would still compile. Does that change
anything?

-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help