public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH v3] imx27: dt: only map 4 Kbyte for fec registers
@ 2015-05-12 22:18 Philippe Reynes
  2015-05-13  6:40 ` Uwe Kleine-König
  2015-05-14  3:09 ` Shawn Guo
  0 siblings, 2 replies; 3+ messages in thread
From: Philippe Reynes @ 2015-05-12 22:18 UTC (permalink / raw)
  To: shawn.guo, kernel, robh+dt, pawel.moll, mark.rutland,
	ijc+devicetree, galak, linux
  Cc: linux-arm-kernel, devicetree, linux-kernel, Philippe Reynes

According to the imx27 documentation, fec has a 4 Kbyte
memory space map. Moreover, the actual 16 Kbyte mapping
overlaps the SCC (Security Controller) memory register
space. So, we reduce the memory register space to 4 Kbyte.

Signed-off-by: Philippe Reynes <tremyfr@gmail.com>
---
 arch/arm/boot/dts/imx27.dtsi |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

Changelog:
v2: (thanks Shawn Guo and Uwe Kleine-Konig)
- map 4 Kbyte instead of 1 Kbyte
v3:
- point that actual fec mapping overlap scc mapping (I forget it in v2)

diff --git a/arch/arm/boot/dts/imx27.dtsi b/arch/arm/boot/dts/imx27.dtsi
index 6951b66..bc215e4 100644
--- a/arch/arm/boot/dts/imx27.dtsi
+++ b/arch/arm/boot/dts/imx27.dtsi
@@ -533,7 +533,7 @@
 
 			fec: ethernet@1002b000 {
 				compatible = "fsl,imx27-fec";
-				reg = <0x1002b000 0x4000>;
+				reg = <0x1002b000 0x1000>;
 				interrupts = <50>;
 				clocks = <&clks IMX27_CLK_FEC_IPG_GATE>,
 					 <&clks IMX27_CLK_FEC_AHB_GATE>;
-- 
1.7.4.4


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

* Re: [PATCH v3] imx27: dt: only map 4 Kbyte for fec registers
  2015-05-12 22:18 [PATCH v3] imx27: dt: only map 4 Kbyte for fec registers Philippe Reynes
@ 2015-05-13  6:40 ` Uwe Kleine-König
  2015-05-14  3:09 ` Shawn Guo
  1 sibling, 0 replies; 3+ messages in thread
From: Uwe Kleine-König @ 2015-05-13  6:40 UTC (permalink / raw)
  To: Philippe Reynes
  Cc: shawn.guo, kernel, robh+dt, pawel.moll, mark.rutland,
	ijc+devicetree, galak, linux, devicetree, linux-kernel,
	linux-arm-kernel

On Wed, May 13, 2015 at 12:18:26AM +0200, Philippe Reynes wrote:
> According to the imx27 documentation, fec has a 4 Kbyte
> memory space map. Moreover, the actual 16 Kbyte mapping
> overlaps the SCC (Security Controller) memory register
> space. So, we reduce the memory register space to 4 Kbyte.
> 
> Signed-off-by: Philippe Reynes <tremyfr@gmail.com>
Acked-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>

Thanks
Uwe

-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |

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

* Re: [PATCH v3] imx27: dt: only map 4 Kbyte for fec registers
  2015-05-12 22:18 [PATCH v3] imx27: dt: only map 4 Kbyte for fec registers Philippe Reynes
  2015-05-13  6:40 ` Uwe Kleine-König
@ 2015-05-14  3:09 ` Shawn Guo
  1 sibling, 0 replies; 3+ messages in thread
From: Shawn Guo @ 2015-05-14  3:09 UTC (permalink / raw)
  To: Philippe Reynes
  Cc: kernel, robh+dt, pawel.moll, mark.rutland, ijc+devicetree, galak,
	linux, linux-arm-kernel, devicetree, linux-kernel

On Wed, May 13, 2015 at 12:18:26AM +0200, Philippe Reynes wrote:
> According to the imx27 documentation, fec has a 4 Kbyte
> memory space map. Moreover, the actual 16 Kbyte mapping
> overlaps the SCC (Security Controller) memory register
> space. So, we reduce the memory register space to 4 Kbyte.
> 
> Signed-off-by: Philippe Reynes <tremyfr@gmail.com>

Updated the patch prefix as "ARM: dts: imx27: ", and applied.  Thanks.

Shawn

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

end of thread, other threads:[~2015-05-14  3:10 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-05-12 22:18 [PATCH v3] imx27: dt: only map 4 Kbyte for fec registers Philippe Reynes
2015-05-13  6:40 ` Uwe Kleine-König
2015-05-14  3:09 ` Shawn Guo

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox