All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/2] clk: samsung: exynos5422: add missing parent GSCL block clocks
@ 2015-12-08 13:46 Marek Szyprowski
  2015-12-08 13:46 ` [PATCH 2/2] ARM: dts: exynos542x: add GSCL block parent clock management to pm domain Marek Szyprowski
  2015-12-09  4:49 ` [PATCH 1/2] clk: samsung: exynos5422: add missing parent GSCL block clocks Krzysztof Kozlowski
  0 siblings, 2 replies; 14+ messages in thread
From: Marek Szyprowski @ 2015-12-08 13:46 UTC (permalink / raw)
  To: linux-samsung-soc, linux-kernel
  Cc: Marek Szyprowski, Sylwester Nawrocki, Tomasz Figa,
	Mike Turquette, Krzysztof Kozlowski, Kukjin Kim

This patch adds clocks, which are required for preserving parent clock
configuration on GSCL power domain on/off.

Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
---
 drivers/clk/samsung/clk-exynos5420.c   | 8 ++++----
 include/dt-bindings/clock/exynos5420.h | 2 ++
 2 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/drivers/clk/samsung/clk-exynos5420.c b/drivers/clk/samsung/clk-exynos5420.c
index 389af3c..4bae2e3 100644
--- a/drivers/clk/samsung/clk-exynos5420.c
+++ b/drivers/clk/samsung/clk-exynos5420.c
@@ -677,8 +677,8 @@ static struct samsung_mux_clock exynos5x_mux_clks[] __initdata = {
 			SRC_TOP5, 20, 1),
 	MUX(CLK_MOUT_USER_ACLK300_DISP1, "mout_user_aclk300_disp1",
 			mout_user_aclk300_disp1_p, SRC_TOP5, 24, 1),
-	MUX(0, "mout_user_aclk300_gscl", mout_user_aclk300_gscl_p,
-			SRC_TOP5, 28, 1),
+	MUX(CLK_MOUT_USER_ACLK300_GSCL, "mout_user_aclk300_gscl",
+			mout_user_aclk300_gscl_p, SRC_TOP5, 28, 1),
 
 	MUX(0, "mout_sclk_mpll", mout_mpll_p, SRC_TOP6, 0, 1),
 	MUX(CLK_MOUT_VPLL, "mout_sclk_vpll", mout_vpll_p, SRC_TOP6, 4, 1),
@@ -729,8 +729,8 @@ static struct samsung_mux_clock exynos5x_mux_clks[] __initdata = {
 			SRC_TOP12, 20, 1),
 	MUX(CLK_MOUT_SW_ACLK300, "mout_sw_aclk300_disp1",
 			mout_sw_aclk300_disp1_p, SRC_TOP12, 24, 1),
-	MUX(0, "mout_sw_aclk300_gscl", mout_sw_aclk300_gscl_p,
-			SRC_TOP12, 28, 1),
+	MUX(CLK_MOUT_SW_ACLK300_GSCL, "mout_sw_aclk300_gscl",
+			mout_sw_aclk300_gscl_p, SRC_TOP12, 28, 1),
 
 	/* DISP1 Block */
 	MUX(0, "mout_mipi1", mout_group2_p, SRC_DISP10, 16, 3),
diff --git a/include/dt-bindings/clock/exynos5420.h b/include/dt-bindings/clock/exynos5420.h
index 99da0d1..b5af23a 100644
--- a/include/dt-bindings/clock/exynos5420.h
+++ b/include/dt-bindings/clock/exynos5420.h
@@ -210,6 +210,8 @@
 #define CLK_MOUT_SW_ACLK300     649
 #define CLK_MOUT_USER_ACLK400_DISP1     650
 #define CLK_MOUT_SW_ACLK400     651
+#define CLK_MOUT_USER_ACLK300_GSCL	652
+#define CLK_MOUT_SW_ACLK300_GSCL	653
 
 /* divider clocks */
 #define CLK_DOUT_PIXEL		768
-- 
1.9.2


^ permalink raw reply related	[flat|nested] 14+ messages in thread

* [PATCH 2/2] ARM: dts: exynos542x: add GSCL block parent clock management to pm domain
  2015-12-08 13:46 [PATCH 1/2] clk: samsung: exynos5422: add missing parent GSCL block clocks Marek Szyprowski
@ 2015-12-08 13:46 ` Marek Szyprowski
  2015-12-09  4:52   ` Krzysztof Kozlowski
  2015-12-09  4:49 ` [PATCH 1/2] clk: samsung: exynos5422: add missing parent GSCL block clocks Krzysztof Kozlowski
  1 sibling, 1 reply; 14+ messages in thread
From: Marek Szyprowski @ 2015-12-08 13:46 UTC (permalink / raw)
  To: linux-samsung-soc, linux-kernel
  Cc: Marek Szyprowski, Sylwester Nawrocki, Tomasz Figa,
	Mike Turquette, Krzysztof Kozlowski, Kukjin Kim

Add support for restoring GScaler parent clocks configuration when GSCL
power domain is turned on.

Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
---
 arch/arm/boot/dts/exynos5420.dtsi | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/arch/arm/boot/dts/exynos5420.dtsi b/arch/arm/boot/dts/exynos5420.dtsi
index 48a0a55..912143e 100644
--- a/arch/arm/boot/dts/exynos5420.dtsi
+++ b/arch/arm/boot/dts/exynos5420.dtsi
@@ -252,8 +252,10 @@
 		compatible = "samsung,exynos4210-pd";
 		reg = <0x10044000 0x20>;
 		#power-domain-cells = <0>;
-		clocks = <&clock CLK_GSCL0>, <&clock CLK_GSCL1>;
-		clock-names = "asb0", "asb1";
+		clocks = <&clock CLK_FIN_PLL>, <&clock CLK_MOUT_SW_ACLK300_GSCL>,
+			<&clock CLK_MOUT_USER_ACLK300_GSCL>, <&clock CLK_GSCL0>,
+			<&clock CLK_GSCL1>;
+		clock-names = "oscclk", "pclk0", "clk0", "asb0", "asb1";
 	};
 
 	isp_pd: power-domain@10044020 {
-- 
1.9.2


^ permalink raw reply related	[flat|nested] 14+ messages in thread

* Re: [PATCH 1/2] clk: samsung: exynos5422: add missing parent GSCL block clocks
  2015-12-08 13:46 [PATCH 1/2] clk: samsung: exynos5422: add missing parent GSCL block clocks Marek Szyprowski
  2015-12-08 13:46 ` [PATCH 2/2] ARM: dts: exynos542x: add GSCL block parent clock management to pm domain Marek Szyprowski
@ 2015-12-09  4:49 ` Krzysztof Kozlowski
  2015-12-09 10:14   ` Sylwester Nawrocki
  1 sibling, 1 reply; 14+ messages in thread
From: Krzysztof Kozlowski @ 2015-12-09  4:49 UTC (permalink / raw)
  To: Marek Szyprowski, linux-samsung-soc, linux-kernel
  Cc: Sylwester Nawrocki, Tomasz Figa, Mike Turquette, Kukjin Kim

On 08.12.2015 22:46, Marek Szyprowski wrote:
> This patch adds clocks, which are required for preserving parent clock
> configuration on GSCL power domain on/off.
> 
> Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
> ---
>  drivers/clk/samsung/clk-exynos5420.c   | 8 ++++----
>  include/dt-bindings/clock/exynos5420.h | 2 ++
>  2 files changed, 6 insertions(+), 4 deletions(-)

I suppose that, with ack from clock folks, this can go through samsung-soc?

Reviewed-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>

Best regards,
Krzysztof

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 2/2] ARM: dts: exynos542x: add GSCL block parent clock management to pm domain
  2015-12-08 13:46 ` [PATCH 2/2] ARM: dts: exynos542x: add GSCL block parent clock management to pm domain Marek Szyprowski
@ 2015-12-09  4:52   ` Krzysztof Kozlowski
  2015-12-09  8:07     ` [PATCH v2 " Marek Szyprowski
  0 siblings, 1 reply; 14+ messages in thread
From: Krzysztof Kozlowski @ 2015-12-09  4:52 UTC (permalink / raw)
  To: Marek Szyprowski, linux-samsung-soc, linux-kernel
  Cc: Sylwester Nawrocki, Tomasz Figa, Mike Turquette, Kukjin Kim

On 08.12.2015 22:46, Marek Szyprowski wrote:
> Add support for restoring GScaler parent clocks configuration when GSCL
> power domain is turned on.
> 
> Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
> ---
>  arch/arm/boot/dts/exynos5420.dtsi | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/exynos5420.dtsi b/arch/arm/boot/dts/exynos5420.dtsi
> index 48a0a55..912143e 100644
> --- a/arch/arm/boot/dts/exynos5420.dtsi
> +++ b/arch/arm/boot/dts/exynos5420.dtsi
> @@ -252,8 +252,10 @@
>  		compatible = "samsung,exynos4210-pd";
>  		reg = <0x10044000 0x20>;
>  		#power-domain-cells = <0>;
> -		clocks = <&clock CLK_GSCL0>, <&clock CLK_GSCL1>;
> -		clock-names = "asb0", "asb1";
> +		clocks = <&clock CLK_FIN_PLL>, <&clock CLK_MOUT_SW_ACLK300_GSCL>,
> +			<&clock CLK_MOUT_USER_ACLK300_GSCL>, <&clock CLK_GSCL0>,
> +			<&clock CLK_GSCL1>;
> +		clock-names = "oscclk", "pclk0", "clk0", "asb0", "asb1";

The pclkN name is not used.

Best regards,
Krzysztof


^ permalink raw reply	[flat|nested] 14+ messages in thread

* [PATCH v2 2/2] ARM: dts: exynos542x: add GSCL block parent clock management to pm domain
  2015-12-09  4:52   ` Krzysztof Kozlowski
@ 2015-12-09  8:07     ` Marek Szyprowski
  2015-12-09 13:38       ` Krzysztof Kozlowski
  2016-01-28 12:59       ` Krzysztof Kozlowski
  0 siblings, 2 replies; 14+ messages in thread
From: Marek Szyprowski @ 2015-12-09  8:07 UTC (permalink / raw)
  To: linux-samsung-soc, linux-kernel
  Cc: Marek Szyprowski, Sylwester Nawrocki, Tomasz Figa,
	Mike Turquette, Krzysztof Kozlowski, Kukjin Kim

Add support for restoring GScaler parent clocks configuration when GSCL
power domain is turned on.

Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
---
 arch/arm/boot/dts/exynos5420.dtsi | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/arch/arm/boot/dts/exynos5420.dtsi b/arch/arm/boot/dts/exynos5420.dtsi
index 1b3d6c7..5d00c18 100644
--- a/arch/arm/boot/dts/exynos5420.dtsi
+++ b/arch/arm/boot/dts/exynos5420.dtsi
@@ -252,8 +252,10 @@
 		compatible = "samsung,exynos4210-pd";
 		reg = <0x10044000 0x20>;
 		#power-domain-cells = <0>;
-		clocks = <&clock CLK_GSCL0>, <&clock CLK_GSCL1>;
-		clock-names = "asb0", "asb1";
+		clocks = <&clock CLK_FIN_PLL>,
+			 <&clock CLK_MOUT_USER_ACLK300_GSCL>,
+			 <&clock CLK_GSCL0>, <&clock CLK_GSCL1>;
+		clock-names = "oscclk", "clk0", "asb0", "asb1";
 	};
 
 	isp_pd: power-domain@10044020 {
-- 
1.9.2


^ permalink raw reply related	[flat|nested] 14+ messages in thread

* Re: [PATCH 1/2] clk: samsung: exynos5422: add missing parent GSCL block clocks
  2015-12-09  4:49 ` [PATCH 1/2] clk: samsung: exynos5422: add missing parent GSCL block clocks Krzysztof Kozlowski
@ 2015-12-09 10:14   ` Sylwester Nawrocki
  2015-12-09 13:36     ` Krzysztof Kozlowski
  0 siblings, 1 reply; 14+ messages in thread
From: Sylwester Nawrocki @ 2015-12-09 10:14 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Marek Szyprowski, linux-samsung-soc, linux-kernel, Tomasz Figa,
	Mike Turquette, Kukjin Kim, linux-clk, Stephen Boyd

Adding Stephen and linux-clk at Cc.

On 09/12/15 05:49, Krzysztof Kozlowski wrote:
> On 08.12.2015 22:46, Marek Szyprowski wrote:
>> > This patch adds clocks, which are required for preserving parent clock
>> > configuration on GSCL power domain on/off.
>> > 
>> > Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
>> > ---
>> >  drivers/clk/samsung/clk-exynos5420.c   | 8 ++++----
>> >  include/dt-bindings/clock/exynos5420.h | 2 ++
>> >  2 files changed, 6 insertions(+), 4 deletions(-)
>
> I suppose that, with ack from clock folks, this can go through samsung-soc?

I guess it makes more sense that making a stable branch with just
this patch to be pulled into arm-soc and clk tree. I'm fine with
applying this patch through arm-soc, but I think we also need
Mike's or Stephen ack for this.

Acked-by: Sylwester Nawrocki <s.nawrocki@samsung.com>

-- 
Thanks,
Sylwester

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 1/2] clk: samsung: exynos5422: add missing parent GSCL block clocks
  2015-12-09 10:14   ` Sylwester Nawrocki
@ 2015-12-09 13:36     ` Krzysztof Kozlowski
  2015-12-16 17:23       ` Sylwester Nawrocki
  0 siblings, 1 reply; 14+ messages in thread
From: Krzysztof Kozlowski @ 2015-12-09 13:36 UTC (permalink / raw)
  To: Sylwester Nawrocki
  Cc: k.kozlowski.k, Marek Szyprowski, linux-samsung-soc, linux-kernel,
	Tomasz Figa, Mike Turquette, Kukjin Kim, linux-clk, Stephen Boyd

W dniu 09.12.2015 o 19:14, Sylwester Nawrocki pisze:
> Adding Stephen and linux-clk at Cc.
> 
> On 09/12/15 05:49, Krzysztof Kozlowski wrote:
>> On 08.12.2015 22:46, Marek Szyprowski wrote:
>>>> This patch adds clocks, which are required for preserving parent clock
>>>> configuration on GSCL power domain on/off.
>>>>
>>>> Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
>>>> ---
>>>>  drivers/clk/samsung/clk-exynos5420.c   | 8 ++++----
>>>>  include/dt-bindings/clock/exynos5420.h | 2 ++
>>>>  2 files changed, 6 insertions(+), 4 deletions(-)
>>
>> I suppose that, with ack from clock folks, this can go through samsung-soc?
> 
> I guess it makes more sense that making a stable branch with just
> this patch to be pulled into arm-soc and clk tree. I'm fine with
> applying this patch through arm-soc, but I think we also need
> Mike's or Stephen ack for this.
> 
> Acked-by: Sylwester Nawrocki <s.nawrocki@samsung.com>

I am fine with the branch approach (actually in such cases I make them
anyway just in case).

As you suggested I'll wait for Mike's or Stepen's acks.

BR,
Krzysztof


^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH v2 2/2] ARM: dts: exynos542x: add GSCL block parent clock management to pm domain
  2015-12-09  8:07     ` [PATCH v2 " Marek Szyprowski
@ 2015-12-09 13:38       ` Krzysztof Kozlowski
  2016-01-28 12:59       ` Krzysztof Kozlowski
  1 sibling, 0 replies; 14+ messages in thread
From: Krzysztof Kozlowski @ 2015-12-09 13:38 UTC (permalink / raw)
  To: Marek Szyprowski, linux-samsung-soc, linux-kernel
  Cc: k.kozlowski.k, Sylwester Nawrocki, Tomasz Figa, Mike Turquette,
	Kukjin Kim

W dniu 09.12.2015 o 17:07, Marek Szyprowski pisze:
> Add support for restoring GScaler parent clocks configuration when GSCL
> power domain is turned on.
> 
> Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
> ---
>  arch/arm/boot/dts/exynos5420.dtsi | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)

Changelog is always nice, even though the traffic on samsung-soc is not
huge now.

Anyway, thanks for update:

Reviewed-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>

Best regards,
Krzysztof

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 1/2] clk: samsung: exynos5422: add missing parent GSCL block clocks
  2015-12-09 13:36     ` Krzysztof Kozlowski
@ 2015-12-16 17:23       ` Sylwester Nawrocki
  2015-12-17  0:58         ` Krzysztof Kozlowski
  0 siblings, 1 reply; 14+ messages in thread
From: Sylwester Nawrocki @ 2015-12-16 17:23 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: k.kozlowski.k, Marek Szyprowski, linux-samsung-soc, linux-kernel,
	Tomasz Figa, Mike Turquette, Kukjin Kim, linux-clk, Stephen Boyd

Krzysztof,

On 09/12/15 14:36, Krzysztof Kozlowski wrote:
> W dniu 09.12.2015 o 19:14, Sylwester Nawrocki pisze:
>> > Adding Stephen and linux-clk at Cc.
>> > 
>> > On 09/12/15 05:49, Krzysztof Kozlowski wrote:
>>> >> On 08.12.2015 22:46, Marek Szyprowski wrote:
>>>>> >>>> This patch adds clocks, which are required for preserving parent clock
>>>>> >>>> configuration on GSCL power domain on/off.
>>>>> >>>>
>>>>> >>>> Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
>>>>> >>>> ---
>>>>> >>>>  drivers/clk/samsung/clk-exynos5420.c   | 8 ++++----
>>>>> >>>>  include/dt-bindings/clock/exynos5420.h | 2 ++
>>>>> >>>>  2 files changed, 6 insertions(+), 4 deletions(-)
>>> >>
>>> >> I suppose that, with ack from clock folks, this can go through samsung-soc?
>> > 
>> > I guess it makes more sense that making a stable branch with just
>> > this patch to be pulled into arm-soc and clk tree. I'm fine with
>> > applying this patch through arm-soc, but I think we also need
>> > Mike's or Stephen ack for this.
>> > 
>> > Acked-by: Sylwester Nawrocki <s.nawrocki@samsung.com>
>
> I am fine with the branch approach (actually in such cases I make them
> anyway just in case).
> 
> As you suggested I'll wait for Mike's or Stepen's acks.

I've put the $subject patch and the other exynos5420 patch which 
was a dependency for dts patches (both touching include/dt-bindings/
clock/exynos5420.h) onto a stable branch. Below are details if 
you need to pull.


The following changes since commit 9f9499ae8e6415cefc4fe0a96ad0e27864353c89:

  Linux 4.4-rc5 (2015-12-13 17:42:58 -0800)

are available in the git repository at:

  git://linuxtv.org/snawrocki/samsung.git tags/for-4.5-clk-exynos5420

for you to fetch changes up to bee4f87f01dc30fcf9e05eb55b833f89fd9bb4f4:

  clk: samsung: exynos5420: add cpu clock configuration data and instantiate cpu clock (2015-12-16 16:35:26 +0100)

----------------------------------------------------------------
Samsung exynos5420 SoC clk subsystem support updates:
instantiation of the cpu clocks and addition of the GSCL
IP parent clocks to the list of available consumer clocks.

----------------------------------------------------------------
Marek Szyprowski (1):
      clk: samsung: exynos542x: add missing parent GSCL block clocks

Thomas Abraham (1):
      clk: samsung: exynos5420: add cpu clock configuration data and instantiate cpu clock

 drivers/clk/samsung/clk-exynos5420.c   |   66 +++++++++++++++++++++++++++++---
 include/dt-bindings/clock/exynos5420.h |    4 ++
 2 files changed, 64 insertions(+), 6 deletions(-)

-- 
Thanks
Sylwester

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 1/2] clk: samsung: exynos5422: add missing parent GSCL block clocks
  2015-12-16 17:23       ` Sylwester Nawrocki
@ 2015-12-17  0:58         ` Krzysztof Kozlowski
  2015-12-22 18:44           ` Mike Turquette
  0 siblings, 1 reply; 14+ messages in thread
From: Krzysztof Kozlowski @ 2015-12-17  0:58 UTC (permalink / raw)
  To: Sylwester Nawrocki
  Cc: k.kozlowski.k, Marek Szyprowski, linux-samsung-soc, linux-kernel,
	Tomasz Figa, Mike Turquette, Kukjin Kim, linux-clk, Stephen Boyd

On 17.12.2015 02:23, Sylwester Nawrocki wrote:
> Krzysztof,
> 
> On 09/12/15 14:36, Krzysztof Kozlowski wrote:
>> W dniu 09.12.2015 o 19:14, Sylwester Nawrocki pisze:
>>>> Adding Stephen and linux-clk at Cc.
>>>>
>>>> On 09/12/15 05:49, Krzysztof Kozlowski wrote:
>>>>>> On 08.12.2015 22:46, Marek Szyprowski wrote:
>>>>>>>>>> This patch adds clocks, which are required for preserving parent clock
>>>>>>>>>> configuration on GSCL power domain on/off.
>>>>>>>>>>
>>>>>>>>>> Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
>>>>>>>>>> ---
>>>>>>>>>>  drivers/clk/samsung/clk-exynos5420.c   | 8 ++++----
>>>>>>>>>>  include/dt-bindings/clock/exynos5420.h | 2 ++
>>>>>>>>>>  2 files changed, 6 insertions(+), 4 deletions(-)
>>>>>>
>>>>>> I suppose that, with ack from clock folks, this can go through samsung-soc?
>>>>
>>>> I guess it makes more sense that making a stable branch with just
>>>> this patch to be pulled into arm-soc and clk tree. I'm fine with
>>>> applying this patch through arm-soc, but I think we also need
>>>> Mike's or Stephen ack for this.
>>>>
>>>> Acked-by: Sylwester Nawrocki <s.nawrocki@samsung.com>
>>
>> I am fine with the branch approach (actually in such cases I make them
>> anyway just in case).
>>
>> As you suggested I'll wait for Mike's or Stepen's acks.
> 
> I've put the $subject patch and the other exynos5420 patch which 
> was a dependency for dts patches (both touching include/dt-bindings/
> clock/exynos5420.h) onto a stable branch. Below are details if 
> you need to pull.

Great! Thanks Sylwester.
BR,
Krzysztof


> 
> 
> The following changes since commit 9f9499ae8e6415cefc4fe0a96ad0e27864353c89:
> 
>   Linux 4.4-rc5 (2015-12-13 17:42:58 -0800)
> 
> are available in the git repository at:
> 
>   git://linuxtv.org/snawrocki/samsung.git tags/for-4.5-clk-exynos5420
> 
> for you to fetch changes up to bee4f87f01dc30fcf9e05eb55b833f89fd9bb4f4:
> 
>   clk: samsung: exynos5420: add cpu clock configuration data and instantiate cpu clock (2015-12-16 16:35:26 +0100)
> 
> ----------------------------------------------------------------
> Samsung exynos5420 SoC clk subsystem support updates:
> instantiation of the cpu clocks and addition of the GSCL
> IP parent clocks to the list of available consumer clocks.
> 
> ----------------------------------------------------------------
> Marek Szyprowski (1):
>       clk: samsung: exynos542x: add missing parent GSCL block clocks
> 
> Thomas Abraham (1):
>       clk: samsung: exynos5420: add cpu clock configuration data and instantiate cpu clock
> 
>  drivers/clk/samsung/clk-exynos5420.c   |   66 +++++++++++++++++++++++++++++---
>  include/dt-bindings/clock/exynos5420.h |    4 ++
>  2 files changed, 64 insertions(+), 6 deletions(-)
> 


^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 1/2] clk: samsung: exynos5422: add missing parent GSCL block clocks
  2015-12-17  0:58         ` Krzysztof Kozlowski
@ 2015-12-22 18:44           ` Mike Turquette
  2015-12-22 19:39             ` Sylwester Nawrocki
  0 siblings, 1 reply; 14+ messages in thread
From: Mike Turquette @ 2015-12-22 18:44 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Sylwester Nawrocki, k.kozlowski.k, Marek Szyprowski,
	linux-samsung-soc, linux-kernel, Tomasz Figa, Kukjin Kim,
	linux-clk, Stephen Boyd

On 12/17, Krzysztof Kozlowski wrote:
> On 17.12.2015 02:23, Sylwester Nawrocki wrote:
> > Krzysztof,
> > 
> > On 09/12/15 14:36, Krzysztof Kozlowski wrote:
> >> W dniu 09.12.2015 o 19:14, Sylwester Nawrocki pisze:
> >>>> Adding Stephen and linux-clk at Cc.
> >>>>
> >>>> On 09/12/15 05:49, Krzysztof Kozlowski wrote:
> >>>>>> On 08.12.2015 22:46, Marek Szyprowski wrote:
> >>>>>>>>>> This patch adds clocks, which are required for preserving parent clock
> >>>>>>>>>> configuration on GSCL power domain on/off.
> >>>>>>>>>>
> >>>>>>>>>> Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
> >>>>>>>>>> ---
> >>>>>>>>>>  drivers/clk/samsung/clk-exynos5420.c   | 8 ++++----
> >>>>>>>>>>  include/dt-bindings/clock/exynos5420.h | 2 ++
> >>>>>>>>>>  2 files changed, 6 insertions(+), 4 deletions(-)
> >>>>>>
> >>>>>> I suppose that, with ack from clock folks, this can go through samsung-soc?
> >>>>
> >>>> I guess it makes more sense that making a stable branch with just
> >>>> this patch to be pulled into arm-soc and clk tree. I'm fine with
> >>>> applying this patch through arm-soc, but I think we also need
> >>>> Mike's or Stephen ack for this.
> >>>>
> >>>> Acked-by: Sylwester Nawrocki <s.nawrocki@samsung.com>
> >>
> >> I am fine with the branch approach (actually in such cases I make them
> >> anyway just in case).
> >>
> >> As you suggested I'll wait for Mike's or Stepen's acks.
> > 
> > I've put the $subject patch and the other exynos5420 patch which 
> > was a dependency for dts patches (both touching include/dt-bindings/
> > clock/exynos5420.h) onto a stable branch. Below are details if 
> > you need to pull.

This is superseded by the pull request found in Message-ID:
<5671A456.9030203@samsung.com>, correct?

Thanks,
Mike

> 
> Great! Thanks Sylwester.
> BR,
> Krzysztof
> 
> 
> > 
> > 
> > The following changes since commit 9f9499ae8e6415cefc4fe0a96ad0e27864353c89:
> > 
> >   Linux 4.4-rc5 (2015-12-13 17:42:58 -0800)
> > 
> > are available in the git repository at:
> > 
> >   git://linuxtv.org/snawrocki/samsung.git tags/for-4.5-clk-exynos5420
> > 
> > for you to fetch changes up to bee4f87f01dc30fcf9e05eb55b833f89fd9bb4f4:
> > 
> >   clk: samsung: exynos5420: add cpu clock configuration data and instantiate cpu clock (2015-12-16 16:35:26 +0100)
> > 
> > ----------------------------------------------------------------
> > Samsung exynos5420 SoC clk subsystem support updates:
> > instantiation of the cpu clocks and addition of the GSCL
> > IP parent clocks to the list of available consumer clocks.
> > 
> > ----------------------------------------------------------------
> > Marek Szyprowski (1):
> >       clk: samsung: exynos542x: add missing parent GSCL block clocks
> > 
> > Thomas Abraham (1):
> >       clk: samsung: exynos5420: add cpu clock configuration data and instantiate cpu clock
> > 
> >  drivers/clk/samsung/clk-exynos5420.c   |   66 +++++++++++++++++++++++++++++---
> >  include/dt-bindings/clock/exynos5420.h |    4 ++
> >  2 files changed, 64 insertions(+), 6 deletions(-)
> > 
> 

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 1/2] clk: samsung: exynos5422: add missing parent GSCL block clocks
  2015-12-22 18:44           ` Mike Turquette
@ 2015-12-22 19:39             ` Sylwester Nawrocki
  2015-12-22 20:45               ` Mike Turquette
  0 siblings, 1 reply; 14+ messages in thread
From: Sylwester Nawrocki @ 2015-12-22 19:39 UTC (permalink / raw)
  To: Mike Turquette
  Cc: Krzysztof Kozlowski, k.kozlowski.k, Marek Szyprowski,
	linux-samsung-soc, linux-kernel, Tomasz Figa, Kukjin Kim,
	linux-clk, Stephen Boyd

Mike,

On 22/12/15 19:44, Mike Turquette wrote:
> This is superseded by the pull request found in Message-ID:
> <5671A456.9030203@samsung.com>, correct?

The two pull requests are based on same branch, first 2 commits are
tagged with for-4.5-clk-exynos5420 tag and the whole branch is tagged
as clk-samsung-4.5. I assumed it's fine to use just the below tag for
arm-soc.

>>> > > The following changes since commit 9f9499ae8e6415cefc4fe0a96ad0e27864353c89:
>>> > > 
>>> > >   Linux 4.4-rc5 (2015-12-13 17:42:58 -0800)
>>> > > 
>>> > > are available in the git repository at:
>>> > > 
>>> > >   git://linuxtv.org/snawrocki/samsung.git tags/for-4.5-clk-exynos5420
>>> > > 
>>> > > for you to fetch changes up to bee4f87f01dc30fcf9e05eb55b833f89fd9bb4f4:
>>> > > 
>>> > >   clk: samsung: exynos5420: add cpu clock configuration data and instantiate cpu clock (2015-12-16 16:35:26 +0100)
>>> > > 
>>> > > ----------------------------------------------------------------
>>> > > Samsung exynos5420 SoC clk subsystem support updates:
>>> > > instantiation of the cpu clocks and addition of the GSCL
>>> > > IP parent clocks to the list of available consumer clocks.
>>> > > 
>>> > > ----------------------------------------------------------------
>>> > > Marek Szyprowski (1):
>>> > >       clk: samsung: exynos542x: add missing parent GSCL block clocks
>>> > > 
>>> > > Thomas Abraham (1):
>>> > >       clk: samsung: exynos5420: add cpu clock configuration data and instantiate cpu clock
>>> > > 
>>> > >  drivers/clk/samsung/clk-exynos5420.c   |   66 +++++++++++++++++++++++++++++---
>>> > >  include/dt-bindings/clock/exynos5420.h |    4 ++
>>> > >  2 files changed, 64 insertions(+), 6 deletions(-)

--
Regards,
Sylwester


^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 1/2] clk: samsung: exynos5422: add missing parent GSCL block clocks
  2015-12-22 19:39             ` Sylwester Nawrocki
@ 2015-12-22 20:45               ` Mike Turquette
  0 siblings, 0 replies; 14+ messages in thread
From: Mike Turquette @ 2015-12-22 20:45 UTC (permalink / raw)
  To: Sylwester Nawrocki
  Cc: Krzysztof Kozlowski, k.kozlowski.k, Marek Szyprowski,
	linux-samsung-soc, linux-kernel, Tomasz Figa, Kukjin Kim,
	linux-clk, Stephen Boyd

On 12/22, Sylwester Nawrocki wrote:
> Mike,
> 
> On 22/12/15 19:44, Mike Turquette wrote:
> > This is superseded by the pull request found in Message-ID:
> > <5671A456.9030203@samsung.com>, correct?
> 
> The two pull requests are based on same branch, first 2 commits are
> tagged with for-4.5-clk-exynos5420 tag and the whole branch is tagged
> as clk-samsung-4.5. I assumed it's fine to use just the below tag for
> arm-soc.

OK, I'll interpret that as a "yes".

Thanks,
Mike

> 
> >>> > > The following changes since commit 9f9499ae8e6415cefc4fe0a96ad0e27864353c89:
> >>> > > 
> >>> > >   Linux 4.4-rc5 (2015-12-13 17:42:58 -0800)
> >>> > > 
> >>> > > are available in the git repository at:
> >>> > > 
> >>> > >   git://linuxtv.org/snawrocki/samsung.git tags/for-4.5-clk-exynos5420
> >>> > > 
> >>> > > for you to fetch changes up to bee4f87f01dc30fcf9e05eb55b833f89fd9bb4f4:
> >>> > > 
> >>> > >   clk: samsung: exynos5420: add cpu clock configuration data and instantiate cpu clock (2015-12-16 16:35:26 +0100)
> >>> > > 
> >>> > > ----------------------------------------------------------------
> >>> > > Samsung exynos5420 SoC clk subsystem support updates:
> >>> > > instantiation of the cpu clocks and addition of the GSCL
> >>> > > IP parent clocks to the list of available consumer clocks.
> >>> > > 
> >>> > > ----------------------------------------------------------------
> >>> > > Marek Szyprowski (1):
> >>> > >       clk: samsung: exynos542x: add missing parent GSCL block clocks
> >>> > > 
> >>> > > Thomas Abraham (1):
> >>> > >       clk: samsung: exynos5420: add cpu clock configuration data and instantiate cpu clock
> >>> > > 
> >>> > >  drivers/clk/samsung/clk-exynos5420.c   |   66 +++++++++++++++++++++++++++++---
> >>> > >  include/dt-bindings/clock/exynos5420.h |    4 ++
> >>> > >  2 files changed, 64 insertions(+), 6 deletions(-)
> 
> --
> Regards,
> Sylwester
> 

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH v2 2/2] ARM: dts: exynos542x: add GSCL block parent clock management to pm domain
  2015-12-09  8:07     ` [PATCH v2 " Marek Szyprowski
  2015-12-09 13:38       ` Krzysztof Kozlowski
@ 2016-01-28 12:59       ` Krzysztof Kozlowski
  1 sibling, 0 replies; 14+ messages in thread
From: Krzysztof Kozlowski @ 2016-01-28 12:59 UTC (permalink / raw)
  To: Marek Szyprowski
  Cc: linux-samsung-soc, linux-kernel, Sylwester Nawrocki, Tomasz Figa,
	Mike Turquette, Kukjin Kim

2015-12-09 17:07 GMT+09:00 Marek Szyprowski <m.szyprowski@samsung.com>:
> Add support for restoring GScaler parent clocks configuration when GSCL
> power domain is turned on.
>
> Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
> ---
>  arch/arm/boot/dts/exynos5420.dtsi | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
>

Applied, thanks!

BR,
Krzysztof

^ permalink raw reply	[flat|nested] 14+ messages in thread

end of thread, other threads:[~2016-01-28 12:59 UTC | newest]

Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-12-08 13:46 [PATCH 1/2] clk: samsung: exynos5422: add missing parent GSCL block clocks Marek Szyprowski
2015-12-08 13:46 ` [PATCH 2/2] ARM: dts: exynos542x: add GSCL block parent clock management to pm domain Marek Szyprowski
2015-12-09  4:52   ` Krzysztof Kozlowski
2015-12-09  8:07     ` [PATCH v2 " Marek Szyprowski
2015-12-09 13:38       ` Krzysztof Kozlowski
2016-01-28 12:59       ` Krzysztof Kozlowski
2015-12-09  4:49 ` [PATCH 1/2] clk: samsung: exynos5422: add missing parent GSCL block clocks Krzysztof Kozlowski
2015-12-09 10:14   ` Sylwester Nawrocki
2015-12-09 13:36     ` Krzysztof Kozlowski
2015-12-16 17:23       ` Sylwester Nawrocki
2015-12-17  0:58         ` Krzysztof Kozlowski
2015-12-22 18:44           ` Mike Turquette
2015-12-22 19:39             ` Sylwester Nawrocki
2015-12-22 20:45               ` Mike Turquette

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.