diff mbox

[2/2] dt-bindings: clock: Move exynos-audss-clk.h to dt-bindings/clock

Message ID 1394085884-13081-3-git-send-email-tushar.behera@linaro.org (mailing list archive)
State New, archived
Headers show

Commit Message

Tushar Behera March 6, 2014, 6:04 a.m. UTC
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.

Signed-off-by: Tushar Behera <tushar.behera@linaro.org>
CC: Kukjin Kim <kgene.kim@samsung.com>
CC: Tomasz Figa <t.figa@samsung.com>
---
 arch/arm/boot/dts/exynos5250.dtsi                  |    2 +-
 arch/arm/boot/dts/exynos5420.dtsi                  |    2 +-
 drivers/clk/samsung/clk-exynos-audss.c             |    2 +-
 .../dt-bindings/{clk => clock}/exynos-audss-clk.h  |    0
 4 files changed, 3 insertions(+), 3 deletions(-)
 rename include/dt-bindings/{clk => clock}/exynos-audss-clk.h (100%)

diff --git a/include/dt-bindings/clk/exynos-audss-clk.h b/include/dt-bindings/clock/exynos-audss-clk.h
similarity index 100%
rename from include/dt-bindings/clk/exynos-audss-clk.h
rename to include/dt-bindings/clock/exynos-audss-clk.h

Comments

Sachin Kamat March 6, 2014, 6:34 a.m. UTC | #1
On 6 March 2014 11:34, Tushar Behera <tushar.behera@linaro.org> 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.
>
> Signed-off-by: Tushar Behera <tushar.behera@linaro.org>
> CC: Kukjin Kim <kgene.kim@samsung.com>
> CC: Tomasz Figa <t.figa@samsung.com>
> ---

Sensible move.
Reviewed-by: Sachin Kamat <sachin.kamat@linaro.org>

Now that the dt-bindings/clk folder is empty (with these 2 patches
applied), it can be deleted as well.
Tushar Behera March 6, 2014, 8:17 a.m. UTC | #2
On 6 March 2014 12:04, Sachin Kamat <sachin.kamat@linaro.org> wrote:
> On 6 March 2014 11:34, Tushar Behera <tushar.behera@linaro.org> 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.
>>
>> Signed-off-by: Tushar Behera <tushar.behera@linaro.org>
>> CC: Kukjin Kim <kgene.kim@samsung.com>
>> CC: Tomasz Figa <t.figa@samsung.com>
>> ---
>
> Sensible move.
> Reviewed-by: Sachin Kamat <sachin.kamat@linaro.org>
>

Thanks for the review.

> Now that the dt-bindings/clk folder is empty (with these 2 patches
> applied), it can be deleted as well.
>

Sure, I will wait till both these patches are merged and send another
patch for that later.

> --
> With warm regards,
> Sachin
Tomasz Figa March 10, 2014, 12:20 a.m. UTC | #3
On 06.03.2014 07:04, Tushar Behera 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.
>
> Signed-off-by: Tushar Behera <tushar.behera@linaro.org>
> CC: Kukjin Kim <kgene.kim@samsung.com>
> CC: Tomasz Figa <t.figa@samsung.com>
> ---
>   arch/arm/boot/dts/exynos5250.dtsi                  |    2 +-
>   arch/arm/boot/dts/exynos5420.dtsi                  |    2 +-
>   drivers/clk/samsung/clk-exynos-audss.c             |    2 +-
>   .../dt-bindings/{clk => clock}/exynos-audss-clk.h  |    0
>   4 files changed, 3 insertions(+), 3 deletions(-)
>   rename include/dt-bindings/{clk => clock}/exynos-audss-clk.h (100%)

Acked-by: Tomasz Figa <t.figa@samsung.com>

Best regards,
Tomasz
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Kim Kukjin March 17, 2014, 10:20 p.m. UTC | #4
On 03/10/14 09:20, Tomasz Figa wrote:
> On 06.03.2014 07:04, Tushar Behera 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.
>>
>> Signed-off-by: Tushar Behera <tushar.behera@linaro.org>
>> CC: Kukjin Kim <kgene.kim@samsung.com>

Acked-by: Kukjin Kim <kgene.kim@samsung.com>

Or shall I take this into samsung tree?

Thanks,
Kukjin

>> CC: Tomasz Figa <t.figa@samsung.com>
>> ---
>> arch/arm/boot/dts/exynos5250.dtsi | 2 +-
>> arch/arm/boot/dts/exynos5420.dtsi | 2 +-
>> drivers/clk/samsung/clk-exynos-audss.c | 2 +-
>> .../dt-bindings/{clk => clock}/exynos-audss-clk.h | 0
>> 4 files changed, 3 insertions(+), 3 deletions(-)
>> rename include/dt-bindings/{clk => clock}/exynos-audss-clk.h (100%)
>
> Acked-by: Tomasz Figa <t.figa@samsung.com>
>
> Best regards,
> Tomasz
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Tomasz Figa March 18, 2014, 2:32 p.m. UTC | #5
On 17.03.2014 23:20, Kukjin Kim wrote:
> On 03/10/14 09:20, Tomasz Figa wrote:
>> On 06.03.2014 07:04, Tushar Behera 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.
>>>
>>> Signed-off-by: Tushar Behera <tushar.behera@linaro.org>
>>> CC: Kukjin Kim <kgene.kim@samsung.com>
>
> Acked-by: Kukjin Kim <kgene.kim@samsung.com>
>
> Or shall I take this into samsung tree?

I guess so.

Or should this whole series go through DT tree?

Best regards,
Tomasz
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Kim Kukjin March 20, 2014, 7:32 p.m. UTC | #6
On 03/18/14 23:32, Tomasz Figa wrote:
> On 17.03.2014 23:20, Kukjin Kim wrote:
>> On 03/10/14 09:20, Tomasz Figa wrote:
>>> On 06.03.2014 07:04, Tushar Behera 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.
>>>>
>>>> Signed-off-by: Tushar Behera <tushar.behera@linaro.org>
>>>> CC: Kukjin Kim <kgene.kim@samsung.com>
>>
>> Acked-by: Kukjin Kim <kgene.kim@samsung.com>
>>
>> Or shall I take this into samsung tree?
>
> I guess so.
>
> Or should this whole series go through DT tree?
>
OK, I've applied into samsung tree.

Thanks,
Kukjin
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/arch/arm/boot/dts/exynos5250.dtsi b/arch/arm/boot/dts/exynos5250.dtsi
index 8f6300f..2048012 100644
--- a/arch/arm/boot/dts/exynos5250.dtsi
+++ b/arch/arm/boot/dts/exynos5250.dtsi
@@ -20,7 +20,7 @@ 
 #include "exynos5.dtsi"
 #include "exynos5250-pinctrl.dtsi"
 
-#include <dt-bindings/clk/exynos-audss-clk.h>
+#include <dt-bindings/clock/exynos-audss-clk.h>
 
 / {
 	compatible = "samsung,exynos5250";
diff --git a/arch/arm/boot/dts/exynos5420.dtsi b/arch/arm/boot/dts/exynos5420.dtsi
index 45e2e65..64c7f6d 100644
--- a/arch/arm/boot/dts/exynos5420.dtsi
+++ b/arch/arm/boot/dts/exynos5420.dtsi
@@ -16,7 +16,7 @@ 
 #include "exynos5.dtsi"
 #include "exynos5420-pinctrl.dtsi"
 
-#include <dt-bindings/clk/exynos-audss-clk.h>
+#include <dt-bindings/clock/exynos-audss-clk.h>
 
 / {
 	compatible = "samsung,exynos5420";
diff --git a/drivers/clk/samsung/clk-exynos-audss.c b/drivers/clk/samsung/clk-exynos-audss.c
index 884187f..13eae14c 100644
--- a/drivers/clk/samsung/clk-exynos-audss.c
+++ b/drivers/clk/samsung/clk-exynos-audss.c
@@ -17,7 +17,7 @@ 
 #include <linux/module.h>
 #include <linux/platform_device.h>
 
-#include <dt-bindings/clk/exynos-audss-clk.h>
+#include <dt-bindings/clock/exynos-audss-clk.h>
 
 enum exynos_audss_clk_type {
 	TYPE_EXYNOS4210,