[PATCH] ARM: imx6q: add support for IRAM

Russell King - ARM Linux linux at arm.linux.org.uk
Tue Jan 3 04:10:15 EST 2012


On Mon, Dec 26, 2011 at 09:41:30PM +0800, Shawn Guo wrote:
> On Mon, Dec 26, 2011 at 07:15:23PM +0800, Jason Chen wrote:
> > Signed-off-by: Jason Chen <jason.chen at linaro.org>
> > Signed-off-by: Eric Miao <eric.miao at linaro.org>
> > ---
> >  arch/arm/boot/dts/imx6q.dtsi    |    7 +++++++
> >  arch/arm/mach-imx/Kconfig       |    1 +
> >  arch/arm/mach-imx/clock-imx6q.c |    3 ++-
> >  arch/arm/mach-imx/mach-imx6q.c  |   17 +++++++++++++++++
> >  4 files changed, 27 insertions(+), 1 deletions(-)
> > 
> > diff --git a/arch/arm/boot/dts/imx6q.dtsi b/arch/arm/boot/dts/imx6q.dtsi
> > index 263e8f3..01646b8 100644
> > --- a/arch/arm/boot/dts/imx6q.dtsi
> > +++ b/arch/arm/boot/dts/imx6q.dtsi
> > @@ -80,6 +80,13 @@
> >  		};
> >  	};
> >  
> > +	ocram at 00900000 {
> > +		compatible = "fsl,imx6q-iram";
> 
> It should have nothing specific to imx6q, and could be "fsl,iram".
> Then we can have some common code across different SoCs to match it.

Why is this even specific to fsl?  Isn't it something that could be
specified in a totally generic way?

As I showed with my original set of sram patches, there is not much
specific about this on-board RAM: what is specific is how a SoC uses
it, and that's up to the rest of the SoC code.



More information about the linux-arm-kernel mailing list