Messages in this thread | | | Date | Thu, 28 Mar 2013 22:22:08 +0800 | From | Shawn Guo <> | Subject | Re: [PATCH v9 RESEND 4/4] ARM: dts: add sram for imx53 and imx6q |
| |
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.
Shawn
| |