Re: [PATCH 0/9] ARM: dts: Various OMAP2+ device-tree updates

2013-03-14 Thread Benoit Cousson
Salut Jon,

On 03/08/2013 06:27 PM, Jon Hunter wrote:
 Various OMAP device-tree updates for PMU, DMA, GPIO, GPMC and boards.
 
 The DMA, PMU and OMAP3430 SDP board changes have been sent before
 individually but re-sending here as a complete series for v3.10.
 
 This is based upon v3.9-rc1 and the OMAP3 GPMC binding from Florian
 Vaussard [1] and OMAP5 DT SPI patch from Felipe Balbi [2].
 
 [1] https://patchwork.kernel.org/patch/2057111/

I've tried to follow the series review, and it seems that Florian was
considering sending some other patches. It is not clear if this is a new
version of the series or some additional patches.

I still did not merge this series wondering what to do with it.

Felipe's patch on this other hand is already applied in my branch.

Regards,
Benoit

 [2] https://patchwork.kernel.org/patch/2134711/
 
 Jon Hunter (9):
   ARM: OMAP2+: Prepare for device-tree PMU support
   ARM: dts: OMAP2+: Add PMU nodes
   ARM: dts: OMAP2+: Add SDMA controller bindings and nodes
   ARM: dts: OMAP3: Add support for OMAP3430 SDP board
   ARM: dts: Add GPMC node for OMAP2, OMAP4 and OMAP5
   ARM: dts: Add OMAP3430 SDP flash memory bindings
   ARM: dts: Add OMAP2 gpio bindings
   ARM: dts: OMAP3+: Correct gpio #interrupts-cells property
   ARM: dts: OMAP3: Add reg and interrupt properties for gpio
 
  .../devicetree/bindings/dma/omap-sdma.txt  |   51 +++
  arch/arm/boot/dts/Makefile |1 +
  arch/arm/boot/dts/omap2.dtsi   |   17 +++
  arch/arm/boot/dts/omap2420.dtsi|   55 
  arch/arm/boot/dts/omap2430.dtsi|   66 +
  arch/arm/boot/dts/omap3.dtsi   |   70 +-
  arch/arm/boot/dts/omap3430-sdp.dts |  141 
 
  arch/arm/boot/dts/omap4-panda-es.dts   |2 +
  arch/arm/boot/dts/omap4.dtsi   |   64 -
  arch/arm/boot/dts/omap4460.dtsi|   18 +++
  arch/arm/boot/dts/omap5.dtsi   |   68 --
  arch/arm/mach-omap2/pmu.c  |   14 +-
  12 files changed, 544 insertions(+), 23 deletions(-)
  create mode 100644 Documentation/devicetree/bindings/dma/omap-sdma.txt
  create mode 100644 arch/arm/boot/dts/omap3430-sdp.dts
  create mode 100644 arch/arm/boot/dts/omap4460.dtsi
 

___
devicetree-discuss mailing list
devicetree-discuss@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/devicetree-discuss


Re: [PATCH 0/9] ARM: dts: Various OMAP2+ device-tree updates

2013-03-14 Thread Florian Vaussard

Hi Benoit,

On 03/14/2013 03:57 PM, Benoit Cousson wrote:

Salut Jon,

On 03/08/2013 06:27 PM, Jon Hunter wrote:

Various OMAP device-tree updates for PMU, DMA, GPIO, GPMC and boards.

The DMA, PMU and OMAP3430 SDP board changes have been sent before
individually but re-sending here as a complete series for v3.10.

This is based upon v3.9-rc1 and the OMAP3 GPMC binding from Florian
Vaussard [1] and OMAP5 DT SPI patch from Felipe Balbi [2].

[1] https://patchwork.kernel.org/patch/2057111/


I've tried to follow the series review, and it seems that Florian was
considering sending some other patches. It is not clear if this is a new
version of the series or some additional patches.



Concerning patch [1], it can be merged to add the GPMC binding for OMAP3.
But please discard the rest of my original series, I will repost something
later.

Regards,

Florian


I still did not merge this series wondering what to do with it.

Felipe's patch on this other hand is already applied in my branch.

Regards,
Benoit


[2] https://patchwork.kernel.org/patch/2134711/

Jon Hunter (9):
   ARM: OMAP2+: Prepare for device-tree PMU support
   ARM: dts: OMAP2+: Add PMU nodes
   ARM: dts: OMAP2+: Add SDMA controller bindings and nodes
   ARM: dts: OMAP3: Add support for OMAP3430 SDP board
   ARM: dts: Add GPMC node for OMAP2, OMAP4 and OMAP5
   ARM: dts: Add OMAP3430 SDP flash memory bindings
   ARM: dts: Add OMAP2 gpio bindings
   ARM: dts: OMAP3+: Correct gpio #interrupts-cells property
   ARM: dts: OMAP3: Add reg and interrupt properties for gpio

  .../devicetree/bindings/dma/omap-sdma.txt  |   51 +++
  arch/arm/boot/dts/Makefile |1 +
  arch/arm/boot/dts/omap2.dtsi   |   17 +++
  arch/arm/boot/dts/omap2420.dtsi|   55 
  arch/arm/boot/dts/omap2430.dtsi|   66 +
  arch/arm/boot/dts/omap3.dtsi   |   70 +-
  arch/arm/boot/dts/omap3430-sdp.dts |  141 
  arch/arm/boot/dts/omap4-panda-es.dts   |2 +
  arch/arm/boot/dts/omap4.dtsi   |   64 -
  arch/arm/boot/dts/omap4460.dtsi|   18 +++
  arch/arm/boot/dts/omap5.dtsi   |   68 --
  arch/arm/mach-omap2/pmu.c  |   14 +-
  12 files changed, 544 insertions(+), 23 deletions(-)
  create mode 100644 Documentation/devicetree/bindings/dma/omap-sdma.txt
  create mode 100644 arch/arm/boot/dts/omap3430-sdp.dts
  create mode 100644 arch/arm/boot/dts/omap4460.dtsi



___
devicetree-discuss mailing list
devicetree-discuss@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/devicetree-discuss


Re: [PATCH 0/9] ARM: dts: Various OMAP2+ device-tree updates

2013-03-14 Thread Jon Hunter

On 03/14/2013 10:45 AM, Florian Vaussard wrote:
 Hi Benoit,
 
 On 03/14/2013 03:57 PM, Benoit Cousson wrote:
 Salut Jon,

 On 03/08/2013 06:27 PM, Jon Hunter wrote:
 Various OMAP device-tree updates for PMU, DMA, GPIO, GPMC and boards.

 The DMA, PMU and OMAP3430 SDP board changes have been sent before
 individually but re-sending here as a complete series for v3.10.

 This is based upon v3.9-rc1 and the OMAP3 GPMC binding from Florian
 Vaussard [1] and OMAP5 DT SPI patch from Felipe Balbi [2].

 [1] https://patchwork.kernel.org/patch/2057111/

 I've tried to follow the series review, and it seems that Florian was
 considering sending some other patches. It is not clear if this is a new
 version of the series or some additional patches.

A bit of both. Sorry for the confusion :-)

 Concerning patch [1], it can be merged to add the GPMC binding for OMAP3.
 But please discard the rest of my original series, I will repost something
 later.

Benoit, if you pick up Florian's patch, I will re-post my remaining
patches for you to pick up.

By the way, I see that you have picked up the PMU bindings, however,
ideally we should merge the pmu prepare patch first. We agreed with
Tony for you to take this with his ack [1].

Let me know if you just want me to re-post that one on top of your branch.

Cheers
Jon

[1] http://comments.gmane.org/gmane.linux.ports.arm.omap/91036



___
devicetree-discuss mailing list
devicetree-discuss@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/devicetree-discuss


Re: [PATCH 0/9] ARM: dts: Various OMAP2+ device-tree updates

2013-03-14 Thread Javier Martinez Canillas
On Thu, Mar 14, 2013 at 3:57 PM, Benoit Cousson b-cous...@ti.com wrote:
 Salut Jon,

 On 03/08/2013 06:27 PM, Jon Hunter wrote:
 Various OMAP device-tree updates for PMU, DMA, GPIO, GPMC and boards.

 The DMA, PMU and OMAP3430 SDP board changes have been sent before
 individually but re-sending here as a complete series for v3.10.

 This is based upon v3.9-rc1 and the OMAP3 GPMC binding from Florian
 Vaussard [1] and OMAP5 DT SPI patch from Felipe Balbi [2].

 [1] https://patchwork.kernel.org/patch/2057111/

 I've tried to follow the series review, and it seems that Florian was
 considering sending some other patches. It is not clear if this is a new
 version of the series or some additional patches.


Hi Benoit,

According to [1] Jon suggested that it was not necessary to map all
the 16MB for the GPMC mapped register address space since in practice
is a very small fraction of that size is used.

I had the following patch but I did never post it because Jon said
that the I/O memory mapping is page-aligned and the minimum page
size for ARM is 4KB anyways, so there is no functional difference
between using 0x1000 or 0x02d0.

But now reading [2] I see that you prefer to do what the documentation
said and don't assume any the page size / alignment.

[2]: https://patchwork.kernel.org/patch/2239741/

From 68edff5a102bb8fc81e006738baa456eb69f080a Mon Sep 17 00:00:00 2001
From: Javier Martinez Canillas javier.marti...@collabora.co.uk
Date: Wed, 27 Feb 2013 02:30:51 +0100
Subject: [PATCH] ARM: dts: OMAP3: reduce GPMC mapped registers address space

Currently the OMAP General-Purpose Memory Controller (GPMC) device
node maps 16 MB of address space for its hardware registers.

This is because the OMAP Technical Reference Manual says that the
GPMC module register address space size is 16 MB. But in practice
the maximum address offset used by a GPMC register is 0x02d0.

So, there is no need to map such a big address space for GPMC regs.

This change was suggested by Jon Hunter [1].

[1]: https://patchwork.kernel.org/patch/2057111/

Signed-off-by: Javier Martinez Canillas javier.marti...@collabora.co.uk
---
 arch/arm/boot/dts/omap3.dtsi |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/arch/arm/boot/dts/omap3.dtsi b/arch/arm/boot/dts/omap3.dtsi
index 2ddae38..a60eaf1 100644
--- a/arch/arm/boot/dts/omap3.dtsi
+++ b/arch/arm/boot/dts/omap3.dtsi
@@ -407,7 +407,7 @@
gpmc: gpmc@6e00 {
compatible = ti,omap3430-gpmc;
ti,hwmods = gpmc;
-   reg = 0x6e00 0x100;
+   reg = 0x6e00 0x02d0;
interrupts = 20;
gpmc,num-cs = 8;
gpmc,num-waitpins = 4;
-- 
1.7.7.6
___
devicetree-discuss mailing list
devicetree-discuss@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/devicetree-discuss


Re: [PATCH 0/9] ARM: dts: Various OMAP2+ device-tree updates

2013-03-14 Thread Benoit Cousson
On 03/14/2013 04:59 PM, Jon Hunter wrote:
 
 On 03/14/2013 10:45 AM, Florian Vaussard wrote:
 Hi Benoit,

 On 03/14/2013 03:57 PM, Benoit Cousson wrote:
 Salut Jon,

 On 03/08/2013 06:27 PM, Jon Hunter wrote:
 Various OMAP device-tree updates for PMU, DMA, GPIO, GPMC and boards.

 The DMA, PMU and OMAP3430 SDP board changes have been sent before
 individually but re-sending here as a complete series for v3.10.

 This is based upon v3.9-rc1 and the OMAP3 GPMC binding from Florian
 Vaussard [1] and OMAP5 DT SPI patch from Felipe Balbi [2].

 [1] https://patchwork.kernel.org/patch/2057111/

 I've tried to follow the series review, and it seems that Florian was
 considering sending some other patches. It is not clear if this is a new
 version of the series or some additional patches.
 
 A bit of both. Sorry for the confusion :-)
 
 Concerning patch [1], it can be merged to add the GPMC binding for OMAP3.
 But please discard the rest of my original series, I will repost something
 later.
 
 Benoit, if you pick up Florian's patch, I will re-post my remaining
 patches for you to pick up.
 
 By the way, I see that you have picked up the PMU bindings, however,
 ideally we should merge the pmu prepare patch first. We agreed with
 Tony for you to take this with his ack [1].

OK, no problem, I'll remove it from the for_3.10/dts and get it when
I'll merge your branch.

 Let me know if you just want me to re-post that one on top of your branch.

I guess you will have to repost your branch anyway?

Thanks,
Benoit

___
devicetree-discuss mailing list
devicetree-discuss@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/devicetree-discuss


Re: [PATCH 0/9] ARM: dts: Various OMAP2+ device-tree updates

2013-03-14 Thread Benoit Cousson
Hi Javier,

On 03/14/2013 05:02 PM, Javier Martinez Canillas wrote:
 On Thu, Mar 14, 2013 at 3:57 PM, Benoit Cousson b-cous...@ti.com wrote:
 Salut Jon,

 On 03/08/2013 06:27 PM, Jon Hunter wrote:
 Various OMAP device-tree updates for PMU, DMA, GPIO, GPMC and boards.

 The DMA, PMU and OMAP3430 SDP board changes have been sent before
 individually but re-sending here as a complete series for v3.10.

 This is based upon v3.9-rc1 and the OMAP3 GPMC binding from Florian
 Vaussard [1] and OMAP5 DT SPI patch from Felipe Balbi [2].

 [1] https://patchwork.kernel.org/patch/2057111/

 I've tried to follow the series review, and it seems that Florian was
 considering sending some other patches. It is not clear if this is a new
 version of the series or some additional patches.

 
 Hi Benoit,
 
 According to [1] Jon suggested that it was not necessary to map all
 the 16MB for the GPMC mapped register address space since in practice
 is a very small fraction of that size is used.
 
 I had the following patch but I did never post it because Jon said
 that the I/O memory mapping is page-aligned and the minimum page
 size for ARM is 4KB anyways, so there is no functional difference
 between using 0x1000 or 0x02d0.
 
 But now reading [2] I see that you prefer to do what the documentation
 said and don't assume any the page size / alignment.
 
 [2]: https://patchwork.kernel.org/patch/2239741/
 
 From 68edff5a102bb8fc81e006738baa456eb69f080a Mon Sep 17 00:00:00 2001
 From: Javier Martinez Canillas javier.marti...@collabora.co.uk
 Date: Wed, 27 Feb 2013 02:30:51 +0100
 Subject: [PATCH] ARM: dts: OMAP3: reduce GPMC mapped registers address space
 
 Currently the OMAP General-Purpose Memory Controller (GPMC) device
 node maps 16 MB of address space for its hardware registers.
 
 This is because the OMAP Technical Reference Manual says that the
 GPMC module register address space size is 16 MB. But in practice
 the maximum address offset used by a GPMC register is 0x02d0.
 
 So, there is no need to map such a big address space for GPMC regs.
 
 This change was suggested by Jon Hunter [1].
 
 [1]: https://patchwork.kernel.org/patch/2057111/
 
 Signed-off-by: Javier Martinez Canillas javier.marti...@collabora.co.uk

Thanks for that super fast patch :-)

Applied.

Regards,
Benoit

 ---
  arch/arm/boot/dts/omap3.dtsi |2 +-
  1 files changed, 1 insertions(+), 1 deletions(-)
 
 diff --git a/arch/arm/boot/dts/omap3.dtsi b/arch/arm/boot/dts/omap3.dtsi
 index 2ddae38..a60eaf1 100644
 --- a/arch/arm/boot/dts/omap3.dtsi
 +++ b/arch/arm/boot/dts/omap3.dtsi
 @@ -407,7 +407,7 @@
   gpmc: gpmc@6e00 {
   compatible = ti,omap3430-gpmc;
   ti,hwmods = gpmc;
 - reg = 0x6e00 0x100;
 + reg = 0x6e00 0x02d0;
   interrupts = 20;
   gpmc,num-cs = 8;
   gpmc,num-waitpins = 4;
 

___
devicetree-discuss mailing list
devicetree-discuss@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/devicetree-discuss


[PATCH 0/9] ARM: dts: Various OMAP2+ device-tree updates

2013-03-08 Thread Jon Hunter
Various OMAP device-tree updates for PMU, DMA, GPIO, GPMC and boards.

The DMA, PMU and OMAP3430 SDP board changes have been sent before
individually but re-sending here as a complete series for v3.10.

This is based upon v3.9-rc1 and the OMAP3 GPMC binding from Florian
Vaussard [1] and OMAP5 DT SPI patch from Felipe Balbi [2].

[1] https://patchwork.kernel.org/patch/2057111/
[2] https://patchwork.kernel.org/patch/2134711/

Jon Hunter (9):
  ARM: OMAP2+: Prepare for device-tree PMU support
  ARM: dts: OMAP2+: Add PMU nodes
  ARM: dts: OMAP2+: Add SDMA controller bindings and nodes
  ARM: dts: OMAP3: Add support for OMAP3430 SDP board
  ARM: dts: Add GPMC node for OMAP2, OMAP4 and OMAP5
  ARM: dts: Add OMAP3430 SDP flash memory bindings
  ARM: dts: Add OMAP2 gpio bindings
  ARM: dts: OMAP3+: Correct gpio #interrupts-cells property
  ARM: dts: OMAP3: Add reg and interrupt properties for gpio

 .../devicetree/bindings/dma/omap-sdma.txt  |   51 +++
 arch/arm/boot/dts/Makefile |1 +
 arch/arm/boot/dts/omap2.dtsi   |   17 +++
 arch/arm/boot/dts/omap2420.dtsi|   55 
 arch/arm/boot/dts/omap2430.dtsi|   66 +
 arch/arm/boot/dts/omap3.dtsi   |   70 +-
 arch/arm/boot/dts/omap3430-sdp.dts |  141 
 arch/arm/boot/dts/omap4-panda-es.dts   |2 +
 arch/arm/boot/dts/omap4.dtsi   |   64 -
 arch/arm/boot/dts/omap4460.dtsi|   18 +++
 arch/arm/boot/dts/omap5.dtsi   |   68 --
 arch/arm/mach-omap2/pmu.c  |   14 +-
 12 files changed, 544 insertions(+), 23 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/dma/omap-sdma.txt
 create mode 100644 arch/arm/boot/dts/omap3430-sdp.dts
 create mode 100644 arch/arm/boot/dts/omap4460.dtsi

-- 
1.7.10.4

___
devicetree-discuss mailing list
devicetree-discuss@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/devicetree-discuss