devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/3] pwm: renesas-tpu: Misc Fixes
@ 2017-12-19 16:02 Geert Uytterhoeven
       [not found] ` <1513699327-27357-1-git-send-email-geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
  2017-12-19 16:02 ` [PATCH 3/3] dt-bindings: pwm: renesas-tpu: Correct SoC part numbers and family names Geert Uytterhoeven
  0 siblings, 2 replies; 10+ messages in thread
From: Geert Uytterhoeven @ 2017-12-19 16:02 UTC (permalink / raw)
  To: Thierry Reding, Rob Herring, Mark Rutland, Simon Horman, Magnus Damm
  Cc: linux-pwm-u79uwXL29TY76Z2rM5mHXA,
	devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-renesas-soc-u79uwXL29TY76Z2rM5mHXA, Geert Uytterhoeven

	Hi all,

Here are three small fixes related to the Renesas Timer Pulse Unit PWM
driver.

The first patch is meant for Simon's tree.
The other two patches are meant for either the PWM or DT tree.

Thanks!

Geert Uytterhoeven (3):
  ARM: dts: r8a7740: Correct TPU register block size
  dt-bindings: pwm: renesas-tpu: Correct example TPU register block size
  dt-bindings: pwm: renesas-tpu: Correct SoC part numbers and family
    names

 Documentation/devicetree/bindings/pwm/renesas,tpu-pwm.txt | 6 +++---
 arch/arm/boot/dts/r8a7740.dtsi                            | 2 +-
 2 files changed, 4 insertions(+), 4 deletions(-)

-- 
2.7.4

Gr{oetje,eeting}s,

						Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert-Td1EMuHUCqxL1ZNQvxDV9g@public.gmane.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
							    -- Linus Torvalds
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

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

* [PATCH 1/3] ARM: dts: r8a7740: Correct TPU register block size
       [not found] ` <1513699327-27357-1-git-send-email-geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
@ 2017-12-19 16:02   ` Geert Uytterhoeven
  2017-12-20  8:23     ` Simon Horman
  2017-12-19 16:02   ` [PATCH 2/3] dt-bindings: pwm: renesas-tpu: Correct example " Geert Uytterhoeven
  1 sibling, 1 reply; 10+ messages in thread
From: Geert Uytterhoeven @ 2017-12-19 16:02 UTC (permalink / raw)
  To: Thierry Reding, Rob Herring, Mark Rutland, Simon Horman, Magnus Damm
  Cc: linux-pwm-u79uwXL29TY76Z2rM5mHXA,
	devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-renesas-soc-u79uwXL29TY76Z2rM5mHXA, Geert Uytterhoeven

The Timer Pulse Unit has registers that lie outside the declared
register block.  Enlarge the register block size to fix this.

Signed-off-by: Geert Uytterhoeven <geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
---
This was probably based on the old platform code, which also assumed a
register block size of 0x100.

 arch/arm/boot/dts/r8a7740.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/r8a7740.dtsi b/arch/arm/boot/dts/r8a7740.dtsi
index 95c408b11991fb77..afd3bc5e6cf2e23f 100644
--- a/arch/arm/boot/dts/r8a7740.dtsi
+++ b/arch/arm/boot/dts/r8a7740.dtsi
@@ -317,7 +317,7 @@
 
 	tpu: pwm@e6600000 {
 		compatible = "renesas,tpu-r8a7740", "renesas,tpu";
-		reg = <0xe6600000 0x100>;
+		reg = <0xe6600000 0x148>;
 		clocks = <&mstp3_clks R8A7740_CLK_TPU0>;
 		power-domains = <&pd_a3sp>;
 		status = "disabled";
-- 
2.7.4

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

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

* [PATCH 2/3] dt-bindings: pwm: renesas-tpu: Correct example TPU register block size
       [not found] ` <1513699327-27357-1-git-send-email-geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
  2017-12-19 16:02   ` [PATCH 1/3] ARM: dts: r8a7740: Correct TPU register block size Geert Uytterhoeven
@ 2017-12-19 16:02   ` Geert Uytterhoeven
  2017-12-20  8:21     ` Simon Horman
  2017-12-20 21:19     ` Rob Herring
  1 sibling, 2 replies; 10+ messages in thread
From: Geert Uytterhoeven @ 2017-12-19 16:02 UTC (permalink / raw)
  To: Thierry Reding, Rob Herring, Mark Rutland, Simon Horman, Magnus Damm
  Cc: linux-pwm-u79uwXL29TY76Z2rM5mHXA,
	devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-renesas-soc-u79uwXL29TY76Z2rM5mHXA, Geert Uytterhoeven

The Timer Pulse Unit on R-Mobile A1 has registers that lie outside the
declared register block.  Enlarge the register block size to fix this.

Signed-off-by: Geert Uytterhoeven <geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
---
 Documentation/devicetree/bindings/pwm/renesas,tpu-pwm.txt | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/pwm/renesas,tpu-pwm.txt b/Documentation/devicetree/bindings/pwm/renesas,tpu-pwm.txt
index 1aadc804dae4e30b..641b5b4c648340ee 100644
--- a/Documentation/devicetree/bindings/pwm/renesas,tpu-pwm.txt
+++ b/Documentation/devicetree/bindings/pwm/renesas,tpu-pwm.txt
@@ -22,6 +22,6 @@ Example: R8A7740 (R-Car A1) TPU controller node
 
 	tpu: pwm@e6600000 {
 		compatible = "renesas,tpu-r8a7740", "renesas,tpu";
-		reg = <0xe6600000 0x100>;
+		reg = <0xe6600000 0x148>;
 		#pwm-cells = <3>;
 	};
-- 
2.7.4

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

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

* [PATCH 3/3] dt-bindings: pwm: renesas-tpu: Correct SoC part numbers and family names
  2017-12-19 16:02 [PATCH 0/3] pwm: renesas-tpu: Misc Fixes Geert Uytterhoeven
       [not found] ` <1513699327-27357-1-git-send-email-geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
@ 2017-12-19 16:02 ` Geert Uytterhoeven
  2017-12-20  8:13   ` Simon Horman
  2017-12-20 21:19   ` Rob Herring
  1 sibling, 2 replies; 10+ messages in thread
From: Geert Uytterhoeven @ 2017-12-19 16:02 UTC (permalink / raw)
  To: Thierry Reding, Rob Herring, Mark Rutland, Simon Horman, Magnus Damm
  Cc: linux-pwm, devicetree, linux-renesas-soc, Geert Uytterhoeven

R8A73A4 (not R8A77A4) is R-Mobile APE6,
R8A7740 is R-Mobile (not R-Car) A1.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 Documentation/devicetree/bindings/pwm/renesas,tpu-pwm.txt | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/Documentation/devicetree/bindings/pwm/renesas,tpu-pwm.txt b/Documentation/devicetree/bindings/pwm/renesas,tpu-pwm.txt
index 641b5b4c648340ee..fb633129d845a893 100644
--- a/Documentation/devicetree/bindings/pwm/renesas,tpu-pwm.txt
+++ b/Documentation/devicetree/bindings/pwm/renesas,tpu-pwm.txt
@@ -3,7 +3,7 @@
 Required Properties:
 
   - compatible: should be one of the following.
-    - "renesas,tpu-r8a73a4": for R8A77A4 (R-Mobile APE6) compatible PWM controller.
+    - "renesas,tpu-r8a73a4": for R8A73A4 (R-Mobile APE6) compatible PWM controller.
     - "renesas,tpu-r8a7740": for R8A7740 (R-Mobile A1) compatible PWM controller.
     - "renesas,tpu-r8a7790": for R8A7790 (R-Car H2) compatible PWM controller.
     - "renesas,tpu": for generic R-Car TPU PWM controller.
@@ -18,7 +18,7 @@ Required Properties:
 Please refer to pwm.txt in this directory for details of the common PWM bindings
 used by client devices.
 
-Example: R8A7740 (R-Car A1) TPU controller node
+Example: R8A7740 (R-Mobile A1) TPU controller node
 
 	tpu: pwm@e6600000 {
 		compatible = "renesas,tpu-r8a7740", "renesas,tpu";
-- 
2.7.4

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

* Re: [PATCH 3/3] dt-bindings: pwm: renesas-tpu: Correct SoC part numbers and family names
  2017-12-19 16:02 ` [PATCH 3/3] dt-bindings: pwm: renesas-tpu: Correct SoC part numbers and family names Geert Uytterhoeven
@ 2017-12-20  8:13   ` Simon Horman
  2017-12-20 21:19   ` Rob Herring
  1 sibling, 0 replies; 10+ messages in thread
From: Simon Horman @ 2017-12-20  8:13 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Thierry Reding, Rob Herring, Mark Rutland, Magnus Damm,
	linux-pwm, devicetree, linux-renesas-soc

On Tue, Dec 19, 2017 at 05:02:07PM +0100, Geert Uytterhoeven wrote:
> R8A73A4 (not R8A77A4) is R-Mobile APE6,
> R8A7740 is R-Mobile (not R-Car) A1.
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>

Acked-by: Simon Horman <horms+renesas@verge.net.au>

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

* Re: [PATCH 2/3] dt-bindings: pwm: renesas-tpu: Correct example TPU register block size
  2017-12-19 16:02   ` [PATCH 2/3] dt-bindings: pwm: renesas-tpu: Correct example " Geert Uytterhoeven
@ 2017-12-20  8:21     ` Simon Horman
  2017-12-20 21:19     ` Rob Herring
  1 sibling, 0 replies; 10+ messages in thread
From: Simon Horman @ 2017-12-20  8:21 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Thierry Reding, Rob Herring, Mark Rutland, Magnus Damm,
	linux-pwm, devicetree, linux-renesas-soc

On Tue, Dec 19, 2017 at 05:02:06PM +0100, Geert Uytterhoeven wrote:
> The Timer Pulse Unit on R-Mobile A1 has registers that lie outside the
> declared register block.  Enlarge the register block size to fix this.
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>

Reviewed-by: Simon Horman <simon.horman@netronome.com>

With reference to "User's Manual: Hardware" v2.00.

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

* Re: [PATCH 1/3] ARM: dts: r8a7740: Correct TPU register block size
  2017-12-19 16:02   ` [PATCH 1/3] ARM: dts: r8a7740: Correct TPU register block size Geert Uytterhoeven
@ 2017-12-20  8:23     ` Simon Horman
  2017-12-21  9:32       ` Simon Horman
  0 siblings, 1 reply; 10+ messages in thread
From: Simon Horman @ 2017-12-20  8:23 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Thierry Reding, Rob Herring, Mark Rutland, Magnus Damm,
	linux-pwm, devicetree, linux-renesas-soc

On Tue, Dec 19, 2017 at 05:02:05PM +0100, Geert Uytterhoeven wrote:
> The Timer Pulse Unit has registers that lie outside the declared
> register block.  Enlarge the register block size to fix this.
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> ---
> This was probably based on the old platform code, which also assumed a
> register block size of 0x100.

Reviewed-by: Simon Horman <simon.horman@netronome.com>

With reference to "User's Manual: Hardware" v2.00.

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

* Re: [PATCH 2/3] dt-bindings: pwm: renesas-tpu: Correct example TPU register block size
  2017-12-19 16:02   ` [PATCH 2/3] dt-bindings: pwm: renesas-tpu: Correct example " Geert Uytterhoeven
  2017-12-20  8:21     ` Simon Horman
@ 2017-12-20 21:19     ` Rob Herring
  1 sibling, 0 replies; 10+ messages in thread
From: Rob Herring @ 2017-12-20 21:19 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Thierry Reding, Mark Rutland, Simon Horman, Magnus Damm,
	linux-pwm, devicetree, linux-renesas-soc

On Tue, Dec 19, 2017 at 05:02:06PM +0100, Geert Uytterhoeven wrote:
> The Timer Pulse Unit on R-Mobile A1 has registers that lie outside the
> declared register block.  Enlarge the register block size to fix this.
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> ---
>  Documentation/devicetree/bindings/pwm/renesas,tpu-pwm.txt | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Rob Herring <robh@kernel.org>

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

* Re: [PATCH 3/3] dt-bindings: pwm: renesas-tpu: Correct SoC part numbers and family names
  2017-12-19 16:02 ` [PATCH 3/3] dt-bindings: pwm: renesas-tpu: Correct SoC part numbers and family names Geert Uytterhoeven
  2017-12-20  8:13   ` Simon Horman
@ 2017-12-20 21:19   ` Rob Herring
  1 sibling, 0 replies; 10+ messages in thread
From: Rob Herring @ 2017-12-20 21:19 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Thierry Reding, Mark Rutland, Simon Horman, Magnus Damm,
	linux-pwm, devicetree, linux-renesas-soc

On Tue, Dec 19, 2017 at 05:02:07PM +0100, Geert Uytterhoeven wrote:
> R8A73A4 (not R8A77A4) is R-Mobile APE6,
> R8A7740 is R-Mobile (not R-Car) A1.
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> ---
>  Documentation/devicetree/bindings/pwm/renesas,tpu-pwm.txt | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

Reviewed-by: Rob Herring <robh@kernel.org>

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

* Re: [PATCH 1/3] ARM: dts: r8a7740: Correct TPU register block size
  2017-12-20  8:23     ` Simon Horman
@ 2017-12-21  9:32       ` Simon Horman
  0 siblings, 0 replies; 10+ messages in thread
From: Simon Horman @ 2017-12-21  9:32 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Thierry Reding, Rob Herring, Mark Rutland, Magnus Damm,
	linux-pwm, devicetree, linux-renesas-soc

On Wed, Dec 20, 2017 at 09:23:07AM +0100, Simon Horman wrote:
> On Tue, Dec 19, 2017 at 05:02:05PM +0100, Geert Uytterhoeven wrote:
> > The Timer Pulse Unit has registers that lie outside the declared
> > register block.  Enlarge the register block size to fix this.
> > 
> > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> > ---
> > This was probably based on the old platform code, which also assumed a
> > register block size of 0x100.
> 
> Reviewed-by: Simon Horman <simon.horman@netronome.com>
> 
> With reference to "User's Manual: Hardware" v2.00.

... and applied for v4.16.

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

end of thread, other threads:[~2017-12-21  9:32 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-12-19 16:02 [PATCH 0/3] pwm: renesas-tpu: Misc Fixes Geert Uytterhoeven
     [not found] ` <1513699327-27357-1-git-send-email-geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
2017-12-19 16:02   ` [PATCH 1/3] ARM: dts: r8a7740: Correct TPU register block size Geert Uytterhoeven
2017-12-20  8:23     ` Simon Horman
2017-12-21  9:32       ` Simon Horman
2017-12-19 16:02   ` [PATCH 2/3] dt-bindings: pwm: renesas-tpu: Correct example " Geert Uytterhoeven
2017-12-20  8:21     ` Simon Horman
2017-12-20 21:19     ` Rob Herring
2017-12-19 16:02 ` [PATCH 3/3] dt-bindings: pwm: renesas-tpu: Correct SoC part numbers and family names Geert Uytterhoeven
2017-12-20  8:13   ` Simon Horman
2017-12-20 21:19   ` Rob Herring

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).