Re: [PATCH V2 2/2] mtd/maps/mtd-ram: add an of-platform driver

Grant Likely
2009-06-16T12:53:50Z

On Tue, Jun 16, 2009 at 3:18 AM, Wolfram Sang<w.sang@pengutronix.de> wrote:
> On Mon, Jun 15, 2009 at 07:43:20PM +0200, Albrecht Dreß wrote:
>> Am 05.06.09 14:05 schrieb(en) Wolfram Sang:
>>> Create an of-aware driver using the now exported generic functions
>>> from plat-ram.c. Also add the documentation for the binding.
>>> Partitions are not yet supported. Tested on a phyCORE-MPC5200B-IO.
>>
>> Dumb question: what is the current status of this patch?  I ask because
>
> I don't know.
>
> David wondered if 'of_physmap' could be used, so I wrote what I experienced
> during development. I can't tell if this helped the case.
>
> Grant wondered if we need a bankwidth. IMHO it is needed for now, but I don't
> know if this is a common agreement.

I'm not happy about the use case though. It probably shouldn't appear
in this binding, or if it does it should be tagged as an optional
property. It is only in the 5200 localplus case that bank-width is
needed to figure out how to apply the workaround.

g.

--
Grant Likely, B.Sc., P.Eng.
Secret Lab Technologies Ltd.