Messages in this thread | | | Subject | Re: [PATCH v9 RESEND 4/4] ARM: dts: add sram for imx53 and imx6q | From | Philipp Zabel <> | Date | Thu, 28 Mar 2013 16:23:06 +0100 |
| |
Am Donnerstag, den 28.03.2013, 22:22 +0800 schrieb Shawn Guo: > On Thu, Mar 28, 2013 at 10:05:18AM +0100, Philipp Zabel wrote: > > > > + > > > > + 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. > > We can add i.MX OCRAM specific compatible when we see that real need. > Also, generally we should use a specific chip/osc name rather than "imx" > for compatible to specify the particular hardware version.
Alright, I'll drop it, then.
thanks Philipp
| |