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

On Thu, Apr 16, 2009 at 6:19 AM, Artem Bityutskiy
[off-list ref] wrote:

On Thu, 2009-04-16 at 14:05 +0200, Stefan Roese wrote:

quoted

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. Additionall=

y

quoted

partitions now can span over multiple chips:

=9A =9A mtd: physmap_of: Add multiple regions and concatenation support
=9A =9A mtd/powerpc: Factor out MTD physmap bindings into mtd-physmap.tx=

t

quoted

=9A =9A 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 (=E2=C9=D4=C0=C3=CB=C9=CA =E1=D2=D4=A3=CD)

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