All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] clk: sunxi-ng: sun50i-h6-r: Fix incorrect W1 clock gate register
@ 2019-06-04 15:00 ` megous
  0 siblings, 0 replies; 10+ messages in thread
From: megous @ 2019-06-04 15:00 UTC (permalink / raw)
  To: linux-sunxi
  Cc: Ondrej Jirman, Maxime Ripard, Chen-Yu Tsai, Michael Turquette,
	Stephen Boyd, moderated list:ARM/Allwinner sunXi SoC support,
	open list:COMMON CLK FRAMEWORK, open list

From: Ondrej Jirman <megous@megous.com>

The current code defines W1 clock gate to be at 0x1cc, overlaying it
with the IR gate.

Clock gate for r-apb1-w1 is at 0x1ec. This fixes issues with IR receiver
causing interrupt floods on H6 (because interrupt flags can't be cleared,
due to IR module's bus being disabled).

Signed-off-by: Ondrej Jirman <megous@megous.com>
---
 drivers/clk/sunxi-ng/ccu-sun50i-h6-r.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/clk/sunxi-ng/ccu-sun50i-h6-r.c b/drivers/clk/sunxi-ng/ccu-sun50i-h6-r.c
index 27554eaf6929..8d05d4f1f8a1 100644
--- a/drivers/clk/sunxi-ng/ccu-sun50i-h6-r.c
+++ b/drivers/clk/sunxi-ng/ccu-sun50i-h6-r.c
@@ -104,7 +104,7 @@ static SUNXI_CCU_GATE(r_apb2_i2c_clk,	"r-apb2-i2c",	"r-apb2",
 static SUNXI_CCU_GATE(r_apb1_ir_clk,	"r-apb1-ir",	"r-apb1",
 		      0x1cc, BIT(0), 0);
 static SUNXI_CCU_GATE(r_apb1_w1_clk,	"r-apb1-w1",	"r-apb1",
-		      0x1cc, BIT(0), 0);
+		      0x1ec, BIT(0), 0);
 
 /* Information of IR(RX) mod clock is gathered from BSP source code */
 static const char * const r_mod0_default_parents[] = { "osc32k", "osc24M" };
-- 
2.21.0


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

* [PATCH] clk: sunxi-ng: sun50i-h6-r: Fix incorrect W1 clock gate register
@ 2019-06-04 15:00 ` megous
  0 siblings, 0 replies; 10+ messages in thread
From: megous @ 2019-06-04 15:00 UTC (permalink / raw)
  To: linux-sunxi
  Cc: Ondrej Jirman, Maxime Ripard, Michael Turquette, open list,
	Stephen Boyd, Chen-Yu Tsai, open list:COMMON CLK FRAMEWORK,
	moderated list:ARM/Allwinner sunXi SoC support

From: Ondrej Jirman <megous@megous.com>

The current code defines W1 clock gate to be at 0x1cc, overlaying it
with the IR gate.

Clock gate for r-apb1-w1 is at 0x1ec. This fixes issues with IR receiver
causing interrupt floods on H6 (because interrupt flags can't be cleared,
due to IR module's bus being disabled).

Signed-off-by: Ondrej Jirman <megous@megous.com>
---
 drivers/clk/sunxi-ng/ccu-sun50i-h6-r.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/clk/sunxi-ng/ccu-sun50i-h6-r.c b/drivers/clk/sunxi-ng/ccu-sun50i-h6-r.c
index 27554eaf6929..8d05d4f1f8a1 100644
--- a/drivers/clk/sunxi-ng/ccu-sun50i-h6-r.c
+++ b/drivers/clk/sunxi-ng/ccu-sun50i-h6-r.c
@@ -104,7 +104,7 @@ static SUNXI_CCU_GATE(r_apb2_i2c_clk,	"r-apb2-i2c",	"r-apb2",
 static SUNXI_CCU_GATE(r_apb1_ir_clk,	"r-apb1-ir",	"r-apb1",
 		      0x1cc, BIT(0), 0);
 static SUNXI_CCU_GATE(r_apb1_w1_clk,	"r-apb1-w1",	"r-apb1",
-		      0x1cc, BIT(0), 0);
+		      0x1ec, BIT(0), 0);
 
 /* Information of IR(RX) mod clock is gathered from BSP source code */
 static const char * const r_mod0_default_parents[] = { "osc32k", "osc24M" };
-- 
2.21.0


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* Re: [linux-sunxi] [PATCH] clk: sunxi-ng: sun50i-h6-r: Fix incorrect W1 clock gate register
  2019-06-04 15:00 ` megous
@ 2019-06-04 15:03   ` Jernej Škrabec
  -1 siblings, 0 replies; 10+ messages in thread
From: Jernej Škrabec @ 2019-06-04 15:03 UTC (permalink / raw)
  To: linux-sunxi, megous
  Cc: Maxime Ripard, Chen-Yu Tsai, Michael Turquette, Stephen Boyd,
	moderated list:ARM/Allwinner sunXi SoC support,
	open list:COMMON CLK FRAMEWORK, open list

Dne torek, 04. junij 2019 ob 17:00:54 CEST je megous via linux-sunxi 
napisal(a):
> From: Ondrej Jirman <megous@megous.com>
> 
> The current code defines W1 clock gate to be at 0x1cc, overlaying it
> with the IR gate.
> 
> Clock gate for r-apb1-w1 is at 0x1ec. This fixes issues with IR receiver
> causing interrupt floods on H6 (because interrupt flags can't be cleared,
> due to IR module's bus being disabled).
> 
> Signed-off-by: Ondrej Jirman <megous@megous.com>

You should add Fixes tag and CC stable with this.

Best regards,
Jernej



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

* Re: [linux-sunxi] [PATCH] clk: sunxi-ng: sun50i-h6-r: Fix incorrect W1 clock gate register
@ 2019-06-04 15:03   ` Jernej Škrabec
  0 siblings, 0 replies; 10+ messages in thread
From: Jernej Škrabec @ 2019-06-04 15:03 UTC (permalink / raw)
  To: linux-sunxi, megous
  Cc: Maxime Ripard, Michael Turquette, open list, Stephen Boyd,
	Chen-Yu Tsai, open list:COMMON CLK FRAMEWORK,
	moderated list:ARM/Allwinner sunXi SoC support

Dne torek, 04. junij 2019 ob 17:00:54 CEST je megous via linux-sunxi 
napisal(a):
> From: Ondrej Jirman <megous@megous.com>
> 
> The current code defines W1 clock gate to be at 0x1cc, overlaying it
> with the IR gate.
> 
> Clock gate for r-apb1-w1 is at 0x1ec. This fixes issues with IR receiver
> causing interrupt floods on H6 (because interrupt flags can't be cleared,
> due to IR module's bus being disabled).
> 
> Signed-off-by: Ondrej Jirman <megous@megous.com>

You should add Fixes tag and CC stable with this.

Best regards,
Jernej



_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* Re: [linux-sunxi] [PATCH] clk: sunxi-ng: sun50i-h6-r: Fix incorrect W1 clock gate register
  2019-06-04 15:03   ` Jernej Škrabec
@ 2019-06-04 15:31     ` Ondřej Jirman
  -1 siblings, 0 replies; 10+ messages in thread
From: Ondřej Jirman @ 2019-06-04 15:31 UTC (permalink / raw)
  To: Jernej Škrabec
  Cc: linux-sunxi, Maxime Ripard, Michael Turquette, open list,
	Stephen Boyd, Chen-Yu Tsai, open list:COMMON CLK FRAMEWORK,
	moderated list:ARM/Allwinner sunXi SoC support

Hi Jernej,

On Tue, Jun 04, 2019 at 05:03:55PM +0200, Jernej Škrabec wrote:
> Dne torek, 04. junij 2019 ob 17:00:54 CEST je megous via linux-sunxi 
> napisal(a):
> > From: Ondrej Jirman <megous@megous.com>
> > 
> > The current code defines W1 clock gate to be at 0x1cc, overlaying it
> > with the IR gate.
> > 
> > Clock gate for r-apb1-w1 is at 0x1ec. This fixes issues with IR receiver
> > causing interrupt floods on H6 (because interrupt flags can't be cleared,
> > due to IR module's bus being disabled).
> > 
> > Signed-off-by: Ondrej Jirman <megous@megous.com>
> 
> You should add Fixes tag and CC stable with this.

Not necessary, since H6 IR is not yet supported on mainline.

regards,
	o.

> Best regards,
> Jernej
> 
> 
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* Re: [linux-sunxi] [PATCH] clk: sunxi-ng: sun50i-h6-r: Fix incorrect W1 clock gate register
@ 2019-06-04 15:31     ` Ondřej Jirman
  0 siblings, 0 replies; 10+ messages in thread
From: Ondřej Jirman @ 2019-06-04 15:31 UTC (permalink / raw)
  To: Jernej Škrabec
  Cc: Maxime Ripard, Michael Turquette, Chen-Yu Tsai, open list,
	Stephen Boyd, linux-sunxi, open list:COMMON CLK FRAMEWORK,
	moderated list:ARM/Allwinner sunXi SoC support

Hi Jernej,

On Tue, Jun 04, 2019 at 05:03:55PM +0200, Jernej Škrabec wrote:
> Dne torek, 04. junij 2019 ob 17:00:54 CEST je megous via linux-sunxi 
> napisal(a):
> > From: Ondrej Jirman <megous@megous.com>
> > 
> > The current code defines W1 clock gate to be at 0x1cc, overlaying it
> > with the IR gate.
> > 
> > Clock gate for r-apb1-w1 is at 0x1ec. This fixes issues with IR receiver
> > causing interrupt floods on H6 (because interrupt flags can't be cleared,
> > due to IR module's bus being disabled).
> > 
> > Signed-off-by: Ondrej Jirman <megous@megous.com>
> 
> You should add Fixes tag and CC stable with this.

Not necessary, since H6 IR is not yet supported on mainline.

regards,
	o.

> Best regards,
> Jernej
> 
> 
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* Re: [linux-sunxi] [PATCH] clk: sunxi-ng: sun50i-h6-r: Fix incorrect W1 clock gate register
  2019-06-04 15:31     ` Ondřej Jirman
@ 2019-06-04 15:35       ` Jernej Škrabec
  -1 siblings, 0 replies; 10+ messages in thread
From: Jernej Škrabec @ 2019-06-04 15:35 UTC (permalink / raw)
  To: linux-sunxi, megous
  Cc: Maxime Ripard, Michael Turquette, open list, Stephen Boyd,
	Chen-Yu Tsai, open list:COMMON CLK FRAMEWORK,
	moderated list:ARM/Allwinner sunXi SoC support

Hi!

Dne torek, 04. junij 2019 ob 17:31:20 CEST je 'Ondřej Jirman' via linux-sunxi 
napisal(a):
> Hi Jernej,
> 
> On Tue, Jun 04, 2019 at 05:03:55PM +0200, Jernej Škrabec wrote:
> > Dne torek, 04. junij 2019 ob 17:00:54 CEST je megous via linux-sunxi
> > 
> > napisal(a):
> > > From: Ondrej Jirman <megous@megous.com>
> > > 
> > > The current code defines W1 clock gate to be at 0x1cc, overlaying it
> > > with the IR gate.
> > > 
> > > Clock gate for r-apb1-w1 is at 0x1ec. This fixes issues with IR receiver
> > > causing interrupt floods on H6 (because interrupt flags can't be
> > > cleared,
> > > due to IR module's bus being disabled).
> > > 
> > > Signed-off-by: Ondrej Jirman <megous@megous.com>
> > 
> > You should add Fixes tag and CC stable with this.
> 
> Not necessary, since H6 IR is not yet supported on mainline.

Well, CCing stable is probably really not necessary, but you are fixing bug in 
existing driver (clk), fixes tag still apply.

Best regards,
Jernej

> 
> regards,
> 	o.
> 
> > Best regards,
> > Jernej
> > 
> > 
> > 
> > _______________________________________________
> > linux-arm-kernel mailing list
> > linux-arm-kernel@lists.infradead.org
> > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel





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

* Re: [linux-sunxi] [PATCH] clk: sunxi-ng: sun50i-h6-r: Fix incorrect W1 clock gate register
@ 2019-06-04 15:35       ` Jernej Škrabec
  0 siblings, 0 replies; 10+ messages in thread
From: Jernej Škrabec @ 2019-06-04 15:35 UTC (permalink / raw)
  To: linux-sunxi, megous
  Cc: Maxime Ripard, Michael Turquette, open list, Stephen Boyd,
	Chen-Yu Tsai, open list:COMMON CLK FRAMEWORK,
	moderated list:ARM/Allwinner sunXi SoC support

Hi!

Dne torek, 04. junij 2019 ob 17:31:20 CEST je 'Ondřej Jirman' via linux-sunxi 
napisal(a):
> Hi Jernej,
> 
> On Tue, Jun 04, 2019 at 05:03:55PM +0200, Jernej Škrabec wrote:
> > Dne torek, 04. junij 2019 ob 17:00:54 CEST je megous via linux-sunxi
> > 
> > napisal(a):
> > > From: Ondrej Jirman <megous@megous.com>
> > > 
> > > The current code defines W1 clock gate to be at 0x1cc, overlaying it
> > > with the IR gate.
> > > 
> > > Clock gate for r-apb1-w1 is at 0x1ec. This fixes issues with IR receiver
> > > causing interrupt floods on H6 (because interrupt flags can't be
> > > cleared,
> > > due to IR module's bus being disabled).
> > > 
> > > Signed-off-by: Ondrej Jirman <megous@megous.com>
> > 
> > You should add Fixes tag and CC stable with this.
> 
> Not necessary, since H6 IR is not yet supported on mainline.

Well, CCing stable is probably really not necessary, but you are fixing bug in 
existing driver (clk), fixes tag still apply.

Best regards,
Jernej

> 
> regards,
> 	o.
> 
> > Best regards,
> > Jernej
> > 
> > 
> > 
> > _______________________________________________
> > linux-arm-kernel mailing list
> > linux-arm-kernel@lists.infradead.org
> > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel





_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* Re: [linux-sunxi] [PATCH] clk: sunxi-ng: sun50i-h6-r: Fix incorrect W1 clock gate register
  2019-06-04 15:35       ` Jernej Škrabec
@ 2019-06-04 15:41         ` Ondřej Jirman
  -1 siblings, 0 replies; 10+ messages in thread
From: Ondřej Jirman @ 2019-06-04 15:41 UTC (permalink / raw)
  To: Jernej Škrabec
  Cc: linux-sunxi, Maxime Ripard, Michael Turquette, open list,
	Stephen Boyd, Chen-Yu Tsai, open list:COMMON CLK FRAMEWORK,
	moderated list:ARM/Allwinner sunXi SoC support

Hello Jernej,

On Tue, Jun 04, 2019 at 05:35:48PM +0200, Jernej Škrabec wrote:
> Hi!
> 
> Dne torek, 04. junij 2019 ob 17:31:20 CEST je 'Ondřej Jirman' via linux-sunxi 
> napisal(a):
> > Hi Jernej,
> > 
> > On Tue, Jun 04, 2019 at 05:03:55PM +0200, Jernej Škrabec wrote:
> > > Dne torek, 04. junij 2019 ob 17:00:54 CEST je megous via linux-sunxi
> > > 
> > > napisal(a):
> > > > From: Ondrej Jirman <megous@megous.com>
> > > > 
> > > > The current code defines W1 clock gate to be at 0x1cc, overlaying it
> > > > with the IR gate.
> > > > 
> > > > Clock gate for r-apb1-w1 is at 0x1ec. This fixes issues with IR receiver
> > > > causing interrupt floods on H6 (because interrupt flags can't be
> > > > cleared,
> > > > due to IR module's bus being disabled).
> > > > 
> > > > Signed-off-by: Ondrej Jirman <megous@megous.com>
> > > 
> > > You should add Fixes tag and CC stable with this.
> > 
> > Not necessary, since H6 IR is not yet supported on mainline.
> 
> Well, CCing stable is probably really not necessary, but you are fixing bug in 
> existing driver (clk), fixes tag still apply.

Right, resent v2.

thank you and regards,
	o.

> Best regards,
> Jernej
> 
> > 
> > regards,
> > 	o.
> > 
> > > Best regards,
> > > Jernej
> > > 
> > > 
> > > 
> > > _______________________________________________
> > > linux-arm-kernel mailing list
> > > linux-arm-kernel@lists.infradead.org
> > > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
> 
> 
> 
> 
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* Re: [linux-sunxi] [PATCH] clk: sunxi-ng: sun50i-h6-r: Fix incorrect W1 clock gate register
@ 2019-06-04 15:41         ` Ondřej Jirman
  0 siblings, 0 replies; 10+ messages in thread
From: Ondřej Jirman @ 2019-06-04 15:41 UTC (permalink / raw)
  To: Jernej Škrabec
  Cc: Maxime Ripard, Michael Turquette, Chen-Yu Tsai, open list,
	Stephen Boyd, linux-sunxi, open list:COMMON CLK FRAMEWORK,
	moderated list:ARM/Allwinner sunXi SoC support

Hello Jernej,

On Tue, Jun 04, 2019 at 05:35:48PM +0200, Jernej Škrabec wrote:
> Hi!
> 
> Dne torek, 04. junij 2019 ob 17:31:20 CEST je 'Ondřej Jirman' via linux-sunxi 
> napisal(a):
> > Hi Jernej,
> > 
> > On Tue, Jun 04, 2019 at 05:03:55PM +0200, Jernej Škrabec wrote:
> > > Dne torek, 04. junij 2019 ob 17:00:54 CEST je megous via linux-sunxi
> > > 
> > > napisal(a):
> > > > From: Ondrej Jirman <megous@megous.com>
> > > > 
> > > > The current code defines W1 clock gate to be at 0x1cc, overlaying it
> > > > with the IR gate.
> > > > 
> > > > Clock gate for r-apb1-w1 is at 0x1ec. This fixes issues with IR receiver
> > > > causing interrupt floods on H6 (because interrupt flags can't be
> > > > cleared,
> > > > due to IR module's bus being disabled).
> > > > 
> > > > Signed-off-by: Ondrej Jirman <megous@megous.com>
> > > 
> > > You should add Fixes tag and CC stable with this.
> > 
> > Not necessary, since H6 IR is not yet supported on mainline.
> 
> Well, CCing stable is probably really not necessary, but you are fixing bug in 
> existing driver (clk), fixes tag still apply.

Right, resent v2.

thank you and regards,
	o.

> Best regards,
> Jernej
> 
> > 
> > regards,
> > 	o.
> > 
> > > Best regards,
> > > Jernej
> > > 
> > > 
> > > 
> > > _______________________________________________
> > > linux-arm-kernel mailing list
> > > linux-arm-kernel@lists.infradead.org
> > > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
> 
> 
> 
> 
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

end of thread, other threads:[~2019-06-04 15:42 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-06-04 15:00 [PATCH] clk: sunxi-ng: sun50i-h6-r: Fix incorrect W1 clock gate register megous
2019-06-04 15:00 ` megous
2019-06-04 15:03 ` [linux-sunxi] " Jernej Škrabec
2019-06-04 15:03   ` Jernej Škrabec
2019-06-04 15:31   ` Ondřej Jirman
2019-06-04 15:31     ` Ondřej Jirman
2019-06-04 15:35     ` Jernej Škrabec
2019-06-04 15:35       ` Jernej Škrabec
2019-06-04 15:41       ` Ondřej Jirman
2019-06-04 15:41         ` Ondřej Jirman

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.