All of lore.kernel.org
 help / color / mirror / Atom feed
* [GIT PULL] Renesas ARM Based SoC DT Fixes for v4.8
@ 2016-07-18  2:39 ` Simon Horman
  0 siblings, 0 replies; 16+ messages in thread
From: Simon Horman @ 2016-07-18  2:39 UTC (permalink / raw)
  To: arm
  Cc: linux-renesas-soc, Olof Johansson, Kevin Hilman, Arnd Bergmann,
	linux-arm-kernel, Magnus Damm, Simon Horman

Hi Olof, Hi Kevin, Hi Arnd,

Please consider these Renesas ARM based SoC DT fixes for v4.8.

This fixes some problems with clock descriptions in DT for the r8a7792 SoC
which is new in v4.8.

This pull request is based on "Second Round of Renesas ARM Based SoC DT
Updates for v4.8", tagged as renesas-dt2-for-v4.8, which you have already
pulled.


The following changes since commit 8fd763c75c3ab8e72e5d7f0d4c53531e6ff76197:

  ARM: dts: r8a7792: add SMP support (2016-06-30 14:26:14 +0200)

are available in the git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas.git tags/renesas-dt-fixes-for-v4.8

for you to fetch changes up to e0c3f92a08f3e0a95024d0d032564fdc1ee96f54:

  ARM: dts: r8a7792: remove ADSP clock (2016-07-15 13:22:45 +0900)

----------------------------------------------------------------
Renesas ARM Based SoC DT Fixes for v4.8

* Corrections to r8a7792

----------------------------------------------------------------
Sergei Shtylyov (2):
      ARM: dts: r8a7792: add PLL1 divided by 2 clock
      ARM: dts: r8a7792: remove ADSP clock

 arch/arm/boot/dts/r8a7792.dtsi            | 9 ++++++++-
 include/dt-bindings/clock/r8a7792-clock.h | 1 -
 2 files changed, 8 insertions(+), 2 deletions(-)

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

* [GIT PULL] Renesas ARM Based SoC DT Fixes for v4.8
@ 2016-07-18  2:39 ` Simon Horman
  0 siblings, 0 replies; 16+ messages in thread
From: Simon Horman @ 2016-07-18  2:39 UTC (permalink / raw)
  To: linux-arm-kernel

Hi Olof, Hi Kevin, Hi Arnd,

Please consider these Renesas ARM based SoC DT fixes for v4.8.

This fixes some problems with clock descriptions in DT for the r8a7792 SoC
which is new in v4.8.

This pull request is based on "Second Round of Renesas ARM Based SoC DT
Updates for v4.8", tagged as renesas-dt2-for-v4.8, which you have already
pulled.


The following changes since commit 8fd763c75c3ab8e72e5d7f0d4c53531e6ff76197:

  ARM: dts: r8a7792: add SMP support (2016-06-30 14:26:14 +0200)

are available in the git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas.git tags/renesas-dt-fixes-for-v4.8

for you to fetch changes up to e0c3f92a08f3e0a95024d0d032564fdc1ee96f54:

  ARM: dts: r8a7792: remove ADSP clock (2016-07-15 13:22:45 +0900)

----------------------------------------------------------------
Renesas ARM Based SoC DT Fixes for v4.8

* Corrections to r8a7792

----------------------------------------------------------------
Sergei Shtylyov (2):
      ARM: dts: r8a7792: add PLL1 divided by 2 clock
      ARM: dts: r8a7792: remove ADSP clock

 arch/arm/boot/dts/r8a7792.dtsi            | 9 ++++++++-
 include/dt-bindings/clock/r8a7792-clock.h | 1 -
 2 files changed, 8 insertions(+), 2 deletions(-)

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

* [PATCH 1/2] ARM: dts: r8a7792: add PLL1 divided by 2 clock
  2016-07-18  2:39 ` Simon Horman
@ 2016-07-18  2:39   ` Simon Horman
  -1 siblings, 0 replies; 16+ messages in thread
From: Simon Horman @ 2016-07-18  2:39 UTC (permalink / raw)
  To: linux-renesas-soc
  Cc: linux-arm-kernel, Magnus Damm, Sergei Shtylyov, Simon Horman

From: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>

Despite the fact that QSPI clock has PLL1/VCOx1/4 clock as a parent, the
latter hasn't been added to the R8A7792 device tree. This patch corrects
that oversight.

Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
---
 arch/arm/boot/dts/r8a7792.dtsi | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/arch/arm/boot/dts/r8a7792.dtsi b/arch/arm/boot/dts/r8a7792.dtsi
index 75256ef4a04d..d5fd0762e2d6 100644
--- a/arch/arm/boot/dts/r8a7792.dtsi
+++ b/arch/arm/boot/dts/r8a7792.dtsi
@@ -284,6 +284,13 @@
 		};
 
 		/* Fixed factor clocks */
+		pll1_div2_clk: pll1_div2 {
+			compatible = "fixed-factor-clock";
+			clocks = <&cpg_clocks R8A7792_CLK_PLL1>;
+			#clock-cells = <0>;
+			clock-div = <2>;
+			clock-mult = <1>;
+		};
 		zs_clk: zs {
 			compatible = "fixed-factor-clock";
 			clocks = <&cpg_clocks R8A7792_CLK_PLL1>;
-- 
2.7.0.rc3.207.g0ac5344

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

* [PATCH 1/2] ARM: dts: r8a7792: add PLL1 divided by 2 clock
@ 2016-07-18  2:39   ` Simon Horman
  0 siblings, 0 replies; 16+ messages in thread
From: Simon Horman @ 2016-07-18  2:39 UTC (permalink / raw)
  To: linux-arm-kernel

From: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>

Despite the fact that QSPI clock has PLL1/VCOx1/4 clock as a parent, the
latter hasn't been added to the R8A7792 device tree. This patch corrects
that oversight.

Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
---
 arch/arm/boot/dts/r8a7792.dtsi | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/arch/arm/boot/dts/r8a7792.dtsi b/arch/arm/boot/dts/r8a7792.dtsi
index 75256ef4a04d..d5fd0762e2d6 100644
--- a/arch/arm/boot/dts/r8a7792.dtsi
+++ b/arch/arm/boot/dts/r8a7792.dtsi
@@ -284,6 +284,13 @@
 		};
 
 		/* Fixed factor clocks */
+		pll1_div2_clk: pll1_div2 {
+			compatible = "fixed-factor-clock";
+			clocks = <&cpg_clocks R8A7792_CLK_PLL1>;
+			#clock-cells = <0>;
+			clock-div = <2>;
+			clock-mult = <1>;
+		};
 		zs_clk: zs {
 			compatible = "fixed-factor-clock";
 			clocks = <&cpg_clocks R8A7792_CLK_PLL1>;
-- 
2.7.0.rc3.207.g0ac5344

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

* [PATCH 2/2] ARM: dts: r8a7792: remove ADSP clock
  2016-07-18  2:39 ` Simon Horman
@ 2016-07-18  2:39   ` Simon Horman
  -1 siblings, 0 replies; 16+ messages in thread
From: Simon Horman @ 2016-07-18  2:39 UTC (permalink / raw)
  To: linux-renesas-soc
  Cc: linux-arm-kernel, Magnus Damm, Sergei Shtylyov, Simon Horman

From: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>

Simon Horman told me that R8A7792 has ADSP clock based on an incorrect
table in the most recent R-Car gen2 manual. But when I received that manual
I discovered that this is false: R8A7792 is the only Gen 2 SoC that doesn't
have ADSP at all.  Accordingly remove the ADSP clock from DT for the
r8a7792.

Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
---
 arch/arm/boot/dts/r8a7792.dtsi            | 2 +-
 include/dt-bindings/clock/r8a7792-clock.h | 1 -
 2 files changed, 1 insertion(+), 2 deletions(-)

diff --git a/arch/arm/boot/dts/r8a7792.dtsi b/arch/arm/boot/dts/r8a7792.dtsi
index d5fd0762e2d6..3fd61d7ab906 100644
--- a/arch/arm/boot/dts/r8a7792.dtsi
+++ b/arch/arm/boot/dts/r8a7792.dtsi
@@ -279,7 +279,7 @@
 			clocks = <&extal_clk>;
 			#clock-cells = <1>;
 			clock-output-names = "main", "pll0", "pll1", "pll3",
-					     "lb", "qspi", "z", "adsp";
+					     "lb", "qspi", "z";
 			#power-domain-cells = <0>;
 		};
 
diff --git a/include/dt-bindings/clock/r8a7792-clock.h b/include/dt-bindings/clock/r8a7792-clock.h
index 89a5155913f6..9a8b392ceb00 100644
--- a/include/dt-bindings/clock/r8a7792-clock.h
+++ b/include/dt-bindings/clock/r8a7792-clock.h
@@ -18,7 +18,6 @@
 #define R8A7792_CLK_LB			4
 #define R8A7792_CLK_QSPI		5
 #define R8A7792_CLK_Z			6
-#define R8A7792_CLK_ADSP		7
 
 /* MSTP0 */
 #define R8A7792_CLK_MSIOF0		0
-- 
2.7.0.rc3.207.g0ac5344

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

* [PATCH 2/2] ARM: dts: r8a7792: remove ADSP clock
@ 2016-07-18  2:39   ` Simon Horman
  0 siblings, 0 replies; 16+ messages in thread
From: Simon Horman @ 2016-07-18  2:39 UTC (permalink / raw)
  To: linux-arm-kernel

From: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>

Simon Horman told me that R8A7792 has ADSP clock based on an incorrect
table in the most recent R-Car gen2 manual. But when I received that manual
I discovered that this is false: R8A7792 is the only Gen 2 SoC that doesn't
have ADSP at all.  Accordingly remove the ADSP clock from DT for the
r8a7792.

Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
---
 arch/arm/boot/dts/r8a7792.dtsi            | 2 +-
 include/dt-bindings/clock/r8a7792-clock.h | 1 -
 2 files changed, 1 insertion(+), 2 deletions(-)

diff --git a/arch/arm/boot/dts/r8a7792.dtsi b/arch/arm/boot/dts/r8a7792.dtsi
index d5fd0762e2d6..3fd61d7ab906 100644
--- a/arch/arm/boot/dts/r8a7792.dtsi
+++ b/arch/arm/boot/dts/r8a7792.dtsi
@@ -279,7 +279,7 @@
 			clocks = <&extal_clk>;
 			#clock-cells = <1>;
 			clock-output-names = "main", "pll0", "pll1", "pll3",
-					     "lb", "qspi", "z", "adsp";
+					     "lb", "qspi", "z";
 			#power-domain-cells = <0>;
 		};
 
diff --git a/include/dt-bindings/clock/r8a7792-clock.h b/include/dt-bindings/clock/r8a7792-clock.h
index 89a5155913f6..9a8b392ceb00 100644
--- a/include/dt-bindings/clock/r8a7792-clock.h
+++ b/include/dt-bindings/clock/r8a7792-clock.h
@@ -18,7 +18,6 @@
 #define R8A7792_CLK_LB			4
 #define R8A7792_CLK_QSPI		5
 #define R8A7792_CLK_Z			6
-#define R8A7792_CLK_ADSP		7
 
 /* MSTP0 */
 #define R8A7792_CLK_MSIOF0		0
-- 
2.7.0.rc3.207.g0ac5344

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

* Re: [GIT PULL] Renesas ARM Based SoC DT Fixes for v4.8
  2016-07-18  2:39 ` Simon Horman
@ 2016-07-21 12:41   ` Arnd Bergmann
  -1 siblings, 0 replies; 16+ messages in thread
From: Arnd Bergmann @ 2016-07-21 12:41 UTC (permalink / raw)
  To: linux-arm-kernel
  Cc: Simon Horman, arm, Kevin Hilman, Magnus Damm, linux-renesas-soc,
	Olof Johansson

On Monday, July 18, 2016 11:39:35 AM CEST Simon Horman wrote:
> Renesas ARM Based SoC DT Fixes for v4.8
> 
> * Corrections to r8a7792
> 

Merged into next/dt, thanks!

	Arnd

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

* [GIT PULL] Renesas ARM Based SoC DT Fixes for v4.8
@ 2016-07-21 12:41   ` Arnd Bergmann
  0 siblings, 0 replies; 16+ messages in thread
From: Arnd Bergmann @ 2016-07-21 12:41 UTC (permalink / raw)
  To: linux-arm-kernel

On Monday, July 18, 2016 11:39:35 AM CEST Simon Horman wrote:
> Renesas ARM Based SoC DT Fixes for v4.8
> 
> * Corrections to r8a7792
> 

Merged into next/dt, thanks!

	Arnd

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

* Re: [GIT PULL] Renesas ARM Based SoC DT Fixes for v4.8
  2016-07-21 12:41   ` Arnd Bergmann
@ 2016-07-22  1:59     ` Simon Horman
  -1 siblings, 0 replies; 16+ messages in thread
From: Simon Horman @ 2016-07-22  1:59 UTC (permalink / raw)
  To: Arnd Bergmann
  Cc: linux-arm-kernel, arm, Kevin Hilman, Magnus Damm,
	linux-renesas-soc, Olof Johansson

On Thu, Jul 21, 2016 at 02:41:52PM +0200, Arnd Bergmann wrote:
> On Monday, July 18, 2016 11:39:35 AM CEST Simon Horman wrote:
> > Renesas ARM Based SoC DT Fixes for v4.8
> > 
> > * Corrections to r8a7792
> > 
> 
> Merged into next/dt, thanks!

Thanks!

I have queued up another fix for v4.8 since sending the above to you.
This time it is an SoC (C-code) rather than a DT fix. I am wondering
if you could offer guidance on:

* If you would prefer me to split fixes out into separate (DT, SoC, ...)
  branches? If so is that until around rc1 when everything has been
  merged into the forthcoming release and thereafter you would prefer
  a single fixes branch?

* What pace you would like me to send fixes. I guess it depends on
  where we are in the cycle. Currently I put the changes in next and
  send what I have about once a week (or not if there are none :).

Typically there are very few fixes. But as I already have 3 for v4.8,
including those in this pull request, I get the feeling a few more could
emerge before v4.8 is released.

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

* [GIT PULL] Renesas ARM Based SoC DT Fixes for v4.8
@ 2016-07-22  1:59     ` Simon Horman
  0 siblings, 0 replies; 16+ messages in thread
From: Simon Horman @ 2016-07-22  1:59 UTC (permalink / raw)
  To: linux-arm-kernel

On Thu, Jul 21, 2016 at 02:41:52PM +0200, Arnd Bergmann wrote:
> On Monday, July 18, 2016 11:39:35 AM CEST Simon Horman wrote:
> > Renesas ARM Based SoC DT Fixes for v4.8
> > 
> > * Corrections to r8a7792
> > 
> 
> Merged into next/dt, thanks!

Thanks!

I have queued up another fix for v4.8 since sending the above to you.
This time it is an SoC (C-code) rather than a DT fix. I am wondering
if you could offer guidance on:

* If you would prefer me to split fixes out into separate (DT, SoC, ...)
  branches? If so is that until around rc1 when everything has been
  merged into the forthcoming release and thereafter you would prefer
  a single fixes branch?

* What pace you would like me to send fixes. I guess it depends on
  where we are in the cycle. Currently I put the changes in next and
  send what I have about once a week (or not if there are none :).

Typically there are very few fixes. But as I already have 3 for v4.8,
including those in this pull request, I get the feeling a few more could
emerge before v4.8 is released.

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

* Re: [GIT PULL] Renesas ARM Based SoC DT Fixes for v4.8
  2016-07-22  1:59     ` Simon Horman
@ 2016-07-22 19:30       ` Arnd Bergmann
  -1 siblings, 0 replies; 16+ messages in thread
From: Arnd Bergmann @ 2016-07-22 19:30 UTC (permalink / raw)
  To: Simon Horman
  Cc: linux-arm-kernel, arm, Kevin Hilman, Magnus Damm,
	linux-renesas-soc, Olof Johansson

On Friday, July 22, 2016 10:59:24 AM CEST Simon Horman wrote:
> On Thu, Jul 21, 2016 at 02:41:52PM +0200, Arnd Bergmann wrote:
> > On Monday, July 18, 2016 11:39:35 AM CEST Simon Horman wrote:
> > > Renesas ARM Based SoC DT Fixes for v4.8
> > > 
> > > * Corrections to r8a7792
> > > 
> > 
> > Merged into next/dt, thanks!
> 
> Thanks!
> 
> I have queued up another fix for v4.8 since sending the above to you.
> This time it is an SoC (C-code) rather than a DT fix. I am wondering
> if you could offer guidance on:
> 
> * If you would prefer me to split fixes out into separate (DT, SoC, ...)
>   branches? If so is that until around rc1 when everything has been
>   merged into the forthcoming release and thereafter you would prefer
>   a single fixes branch?

I prefer a separate fixes branch. It's fine for all other branches
to be based on top of this branch so you have a working baseline
for testing.

> * What pace you would like me to send fixes. I guess it depends on
>   where we are in the cycle. Currently I put the changes in next and
>   send what I have about once a week (or not if there are none :).
> 
> Typically there are very few fixes. But as I already have 3 for v4.8,
> including those in this pull request, I get the feeling a few more could
> emerge before v4.8 is released.

I'd say don't wait longer than a week before forwarding bugfixes.
If you find something urgent, just send that right away along with
whatever less urgent patches you have accumulated.

It's possible we won't merge it right away as we might all be busy
for a couple of days, but in the worst case that means you send
another fixes pull request that is a superset before we have merged
the first one.

	Arnd

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

* [GIT PULL] Renesas ARM Based SoC DT Fixes for v4.8
@ 2016-07-22 19:30       ` Arnd Bergmann
  0 siblings, 0 replies; 16+ messages in thread
From: Arnd Bergmann @ 2016-07-22 19:30 UTC (permalink / raw)
  To: linux-arm-kernel

On Friday, July 22, 2016 10:59:24 AM CEST Simon Horman wrote:
> On Thu, Jul 21, 2016 at 02:41:52PM +0200, Arnd Bergmann wrote:
> > On Monday, July 18, 2016 11:39:35 AM CEST Simon Horman wrote:
> > > Renesas ARM Based SoC DT Fixes for v4.8
> > > 
> > > * Corrections to r8a7792
> > > 
> > 
> > Merged into next/dt, thanks!
> 
> Thanks!
> 
> I have queued up another fix for v4.8 since sending the above to you.
> This time it is an SoC (C-code) rather than a DT fix. I am wondering
> if you could offer guidance on:
> 
> * If you would prefer me to split fixes out into separate (DT, SoC, ...)
>   branches? If so is that until around rc1 when everything has been
>   merged into the forthcoming release and thereafter you would prefer
>   a single fixes branch?

I prefer a separate fixes branch. It's fine for all other branches
to be based on top of this branch so you have a working baseline
for testing.

> * What pace you would like me to send fixes. I guess it depends on
>   where we are in the cycle. Currently I put the changes in next and
>   send what I have about once a week (or not if there are none :).
> 
> Typically there are very few fixes. But as I already have 3 for v4.8,
> including those in this pull request, I get the feeling a few more could
> emerge before v4.8 is released.

I'd say don't wait longer than a week before forwarding bugfixes.
If you find something urgent, just send that right away along with
whatever less urgent patches you have accumulated.

It's possible we won't merge it right away as we might all be busy
for a couple of days, but in the worst case that means you send
another fixes pull request that is a superset before we have merged
the first one.

	Arnd

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

* Re: [GIT PULL] Renesas ARM Based SoC DT Fixes for v4.8
  2016-07-22 19:30       ` Arnd Bergmann
@ 2016-07-25  2:15         ` Simon Horman
  -1 siblings, 0 replies; 16+ messages in thread
From: Simon Horman @ 2016-07-25  2:15 UTC (permalink / raw)
  To: Arnd Bergmann
  Cc: linux-arm-kernel, arm, Kevin Hilman, Magnus Damm,
	linux-renesas-soc, Olof Johansson

On Fri, Jul 22, 2016 at 09:30:58PM +0200, Arnd Bergmann wrote:
> On Friday, July 22, 2016 10:59:24 AM CEST Simon Horman wrote:
> > On Thu, Jul 21, 2016 at 02:41:52PM +0200, Arnd Bergmann wrote:
> > > On Monday, July 18, 2016 11:39:35 AM CEST Simon Horman wrote:
> > > > Renesas ARM Based SoC DT Fixes for v4.8
> > > > 
> > > > * Corrections to r8a7792
> > > > 
> > > 
> > > Merged into next/dt, thanks!
> > 
> > Thanks!
> > 
> > I have queued up another fix for v4.8 since sending the above to you.
> > This time it is an SoC (C-code) rather than a DT fix. I am wondering
> > if you could offer guidance on:
> > 
> > * If you would prefer me to split fixes out into separate (DT, SoC, ...)
> >   branches? If so is that until around rc1 when everything has been
> >   merged into the forthcoming release and thereafter you would prefer
> >   a single fixes branch?
> 
> I prefer a separate fixes branch. It's fine for all other branches
> to be based on top of this branch so you have a working baseline
> for testing.

Is some sort of minimal base for the fixes branch desired?
If should the base be a merge of all the tags you have pulled for v4.8?

Currently I have DT fixes and SoC fixes which depend on
respective tags that have been accepted for v4.8.

> > * What pace you would like me to send fixes. I guess it depends on
> >   where we are in the cycle. Currently I put the changes in next and
> >   send what I have about once a week (or not if there are none :).
> > 
> > Typically there are very few fixes. But as I already have 3 for v4.8,
> > including those in this pull request, I get the feeling a few more could
> > emerge before v4.8 is released.
> 
> I'd say don't wait longer than a week before forwarding bugfixes.
> If you find something urgent, just send that right away along with
> whatever less urgent patches you have accumulated.
> 
> It's possible we won't merge it right away as we might all be busy
> for a couple of days, but in the worst case that means you send
> another fixes pull request that is a superset before we have merged
> the first one.
> 
> 	Arnd
> 

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

* [GIT PULL] Renesas ARM Based SoC DT Fixes for v4.8
@ 2016-07-25  2:15         ` Simon Horman
  0 siblings, 0 replies; 16+ messages in thread
From: Simon Horman @ 2016-07-25  2:15 UTC (permalink / raw)
  To: linux-arm-kernel

On Fri, Jul 22, 2016 at 09:30:58PM +0200, Arnd Bergmann wrote:
> On Friday, July 22, 2016 10:59:24 AM CEST Simon Horman wrote:
> > On Thu, Jul 21, 2016 at 02:41:52PM +0200, Arnd Bergmann wrote:
> > > On Monday, July 18, 2016 11:39:35 AM CEST Simon Horman wrote:
> > > > Renesas ARM Based SoC DT Fixes for v4.8
> > > > 
> > > > * Corrections to r8a7792
> > > > 
> > > 
> > > Merged into next/dt, thanks!
> > 
> > Thanks!
> > 
> > I have queued up another fix for v4.8 since sending the above to you.
> > This time it is an SoC (C-code) rather than a DT fix. I am wondering
> > if you could offer guidance on:
> > 
> > * If you would prefer me to split fixes out into separate (DT, SoC, ...)
> >   branches? If so is that until around rc1 when everything has been
> >   merged into the forthcoming release and thereafter you would prefer
> >   a single fixes branch?
> 
> I prefer a separate fixes branch. It's fine for all other branches
> to be based on top of this branch so you have a working baseline
> for testing.

Is some sort of minimal base for the fixes branch desired?
If should the base be a merge of all the tags you have pulled for v4.8?

Currently I have DT fixes and SoC fixes which depend on
respective tags that have been accepted for v4.8.

> > * What pace you would like me to send fixes. I guess it depends on
> >   where we are in the cycle. Currently I put the changes in next and
> >   send what I have about once a week (or not if there are none :).
> > 
> > Typically there are very few fixes. But as I already have 3 for v4.8,
> > including those in this pull request, I get the feeling a few more could
> > emerge before v4.8 is released.
> 
> I'd say don't wait longer than a week before forwarding bugfixes.
> If you find something urgent, just send that right away along with
> whatever less urgent patches you have accumulated.
> 
> It's possible we won't merge it right away as we might all be busy
> for a couple of days, but in the worst case that means you send
> another fixes pull request that is a superset before we have merged
> the first one.
> 
> 	Arnd
> 

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

* Re: [GIT PULL] Renesas ARM Based SoC DT Fixes for v4.8
  2016-07-25  2:15         ` Simon Horman
@ 2016-07-25  4:30           ` Olof Johansson
  -1 siblings, 0 replies; 16+ messages in thread
From: Olof Johansson @ 2016-07-25  4:30 UTC (permalink / raw)
  To: Simon Horman
  Cc: Arnd Bergmann, linux-arm-kernel, arm, Kevin Hilman, Magnus Damm,
	linux-renesas-soc

[resending as non-HTML]

On Sun, Jul 24, 2016 at 7:15 PM, Simon Horman <horms@verge.net.au> wrote:
> On Fri, Jul 22, 2016 at 09:30:58PM +0200, Arnd Bergmann wrote:
>> On Friday, July 22, 2016 10:59:24 AM CEST Simon Horman wrote:
>> > On Thu, Jul 21, 2016 at 02:41:52PM +0200, Arnd Bergmann wrote:
>> > > On Monday, July 18, 2016 11:39:35 AM CEST Simon Horman wrote:
>> > > > Renesas ARM Based SoC DT Fixes for v4.8
>> > > >
>> > > > * Corrections to r8a7792
>> > > >
>> > >
>> > > Merged into next/dt, thanks!
>> >
>> > Thanks!
>> >
>> > I have queued up another fix for v4.8 since sending the above to you.
>> > This time it is an SoC (C-code) rather than a DT fix. I am wondering
>> > if you could offer guidance on:
>> >
>> > * If you would prefer me to split fixes out into separate (DT, SoC, ...)
>> >   branches? If so is that until around rc1 when everything has been
>> >   merged into the forthcoming release and thereafter you would prefer
>> >   a single fixes branch?
>>
>> I prefer a separate fixes branch. It's fine for all other branches
>> to be based on top of this branch so you have a working baseline
>> for testing.
>
> Is some sort of minimal base for the fixes branch desired?
> If should the base be a merge of all the tags you have pulled for v4.8?
>
> Currently I have DT fixes and SoC fixes which depend on
> respective tags that have been accepted for v4.8.


It's hard to make those one branch without doing your own merge. You
can't take a merge out of our for-next branch as a base, since those
aren't stable.

Wait for our merges to go into Linus' tree, then use his last merge
commit as the base (before -rc1 is out).


-Olof

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

* [GIT PULL] Renesas ARM Based SoC DT Fixes for v4.8
@ 2016-07-25  4:30           ` Olof Johansson
  0 siblings, 0 replies; 16+ messages in thread
From: Olof Johansson @ 2016-07-25  4:30 UTC (permalink / raw)
  To: linux-arm-kernel

[resending as non-HTML]

On Sun, Jul 24, 2016 at 7:15 PM, Simon Horman <horms@verge.net.au> wrote:
> On Fri, Jul 22, 2016 at 09:30:58PM +0200, Arnd Bergmann wrote:
>> On Friday, July 22, 2016 10:59:24 AM CEST Simon Horman wrote:
>> > On Thu, Jul 21, 2016 at 02:41:52PM +0200, Arnd Bergmann wrote:
>> > > On Monday, July 18, 2016 11:39:35 AM CEST Simon Horman wrote:
>> > > > Renesas ARM Based SoC DT Fixes for v4.8
>> > > >
>> > > > * Corrections to r8a7792
>> > > >
>> > >
>> > > Merged into next/dt, thanks!
>> >
>> > Thanks!
>> >
>> > I have queued up another fix for v4.8 since sending the above to you.
>> > This time it is an SoC (C-code) rather than a DT fix. I am wondering
>> > if you could offer guidance on:
>> >
>> > * If you would prefer me to split fixes out into separate (DT, SoC, ...)
>> >   branches? If so is that until around rc1 when everything has been
>> >   merged into the forthcoming release and thereafter you would prefer
>> >   a single fixes branch?
>>
>> I prefer a separate fixes branch. It's fine for all other branches
>> to be based on top of this branch so you have a working baseline
>> for testing.
>
> Is some sort of minimal base for the fixes branch desired?
> If should the base be a merge of all the tags you have pulled for v4.8?
>
> Currently I have DT fixes and SoC fixes which depend on
> respective tags that have been accepted for v4.8.


It's hard to make those one branch without doing your own merge. You
can't take a merge out of our for-next branch as a base, since those
aren't stable.

Wait for our merges to go into Linus' tree, then use his last merge
commit as the base (before -rc1 is out).


-Olof

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

end of thread, other threads:[~2016-07-25  4:30 UTC | newest]

Thread overview: 16+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-07-18  2:39 [GIT PULL] Renesas ARM Based SoC DT Fixes for v4.8 Simon Horman
2016-07-18  2:39 ` Simon Horman
2016-07-18  2:39 ` [PATCH 1/2] ARM: dts: r8a7792: add PLL1 divided by 2 clock Simon Horman
2016-07-18  2:39   ` Simon Horman
2016-07-18  2:39 ` [PATCH 2/2] ARM: dts: r8a7792: remove ADSP clock Simon Horman
2016-07-18  2:39   ` Simon Horman
2016-07-21 12:41 ` [GIT PULL] Renesas ARM Based SoC DT Fixes for v4.8 Arnd Bergmann
2016-07-21 12:41   ` Arnd Bergmann
2016-07-22  1:59   ` Simon Horman
2016-07-22  1:59     ` Simon Horman
2016-07-22 19:30     ` Arnd Bergmann
2016-07-22 19:30       ` Arnd Bergmann
2016-07-25  2:15       ` Simon Horman
2016-07-25  2:15         ` Simon Horman
2016-07-25  4:30         ` Olof Johansson
2016-07-25  4:30           ` Olof Johansson

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.