[PATCH 0/3 v3] mtd: physmap_of: Add multiple regions and concatenation support

Grant Likely grant.likely at secretlab.ca
Thu Apr 16 22:58:22 EST 2009


On Thu, Apr 16, 2009 at 6:19 AM, Artem Bityutskiy
<dedekind at infradead.org> wrote:
> On Thu, 2009-04-16 at 14:05 +0200, Stefan Roese wrote:
>> This patchset adds support to handle multiple non-identical chips in one
>> flash device tree node. It also adds concat support to physmap_of. This
>> makes it possible to support e.g. the Intel P30 48F4400 chip which
>> internally consists of 2 non-identical NOR chips on one die. Additionally
>> partitions now can span over multiple chips:
>>
>>     mtd: physmap_of: Add multiple regions and concatenation support
>>     mtd/powerpc: Factor out MTD physmap bindings into mtd-physmap.txt
>>     mtd/powerpc: Describe multiple "reg" tuples usage
>>
>> v2 addresses all comments from Grant Likely, including factoring out the
>> MTD dts bindings documentation into a separate file.
>>
>> v3 addresses all comments (again from Grant).
>
> Sorry for my ignorance, but could you please explain why regions
> exists as an MTD concept? Why different regions could not be
> represented as different MTD devices? The benefit is - simplicity.

Trying to describe MTD partitions that span device boundaries in the
device tree is far simpler if a single node is used.  It is also a
common case for hardware designers to drop down multiple NOR devices
to be used as a single flash region, so it makes sense to describe it
as a single device.

g.

>
> Thanks.
>
> --
> Best regards,
> Artem Bityutskiy (Битюцкий Артём)
>
>



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



More information about the devicetree-discuss mailing list