[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