Re: [PATCH v9 RESEND 4/4] ARM: dts: add sram for imx53 and imx6q

From: Philipp Zabel
Date: Thu Mar 28 2013 - 05:05:48 EST


Am Donnerstag, den 28.03.2013, 10:54 +0800 schrieb Shawn Guo:
> On Wed, Mar 20, 2013 at 11:52:47AM +0100, Philipp Zabel wrote:
> > Signed-off-by: Philipp Zabel <p.zabel@xxxxxxxxxxxxxx>
> > Reviewed-by: Shawn Guo <shawn.guo@xxxxxxxxxx>
> > Acked-by: Grant Likely <grant.likely@xxxxxxxxxxxx>
> > ---
> > Changes since v8:
> > - Changed device tree compatible string to "mmio-sram"
> > ---
> > arch/arm/boot/dts/imx53.dtsi | 5 +++++
> > arch/arm/boot/dts/imx6q.dtsi | 6 ++++++
> > 2 files changed, 11 insertions(+)
> >
> > diff --git a/arch/arm/boot/dts/imx53.dtsi b/arch/arm/boot/dts/imx53.dtsi
> > index edc3f1e..69d0680 100644
> > --- a/arch/arm/boot/dts/imx53.dtsi
> > +++ b/arch/arm/boot/dts/imx53.dtsi
> > @@ -664,5 +664,10 @@
> > status = "disabled";
> > };
> > };
> > +
> > + ocram: ocram@f8000000 {
> > + compatible = "fsl,imx-ocram", "mmio-sram";
>
> We should probably just drop "fsl,imx-ocram".
>
> Shawn

I thought that in the future somebody might want to implement some i.MX
OCRAM specifics in a different driver, like the L2 cache / OCRAM mode
switching, the timing settings, or the TrustZone bits that are
configurable in the IOMUXC GPR registers.

regards
Philipp

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/