Thread (20 messages) 20 messages, 5 authors, 2016-04-01

Re: [PATCH v3 3/8] clk: samsung: exynos3250: Add MMC2 clock

From: Sylwester Nawrocki <s.nawrocki@samsung.com>
Date: 2016-03-25 09:50:39
Also in: linux-arm-kernel, linux-samsung-soc, lkml

Hi Chanwoo,

On 03/25/2016 12:39 AM, Chanwoo Choi wrote:
quoted hunk ↗ jump to hunk
diff --git a/include/dt-bindings/clock/exynos3250.h 
b/include/dt-bindings/clock/exynos3250.h
quoted
index ddb874130d86..c796ff02ceeb 100644
--- a/include/dt-bindings/clock/exynos3250.h
+++ b/include/dt-bindings/clock/exynos3250.h
...
quoted
-#define CLK_NR_CLKS			249
+#define CLK_NR_CLKS			250
What do you think about putting all the changes to include/dt-bindings/
clock/exynos3250.h into a separate patch? Now in patch 3/8 there
is being changed again what was added in patch 2/8. However, my main
point is to have a minimum required in a common topic branch for the clk
and the arm-soc trees, so there is no need for arm-soc to pull all
the drives/clk changes.
It might not be that sensible in case of just those 2 patches in your
series, nevertheless it would be good to keep that in mind for future
submissions.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help