2015-05-12 22:18:36

by Philippe Reynes

[permalink] [raw]
Subject: [PATCH v3] imx27: dt: only map 4 Kbyte for fec registers

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 <[email protected]>
---
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


2015-05-13 06:40:47

by Uwe Kleine-König

[permalink] [raw]
Subject: Re: [PATCH v3] imx27: dt: only map 4 Kbyte for fec registers

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 <[email protected]>
Acked-by: Uwe Kleine-K?nig <[email protected]>

Thanks
Uwe

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

2015-05-14 03:10:21

by Shawn Guo

[permalink] [raw]
Subject: Re: [PATCH v3] imx27: dt: only map 4 Kbyte for fec registers

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 <[email protected]>

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

Shawn