<feed xmlns='http://www.w3.org/2005/Atom'>
<title>kernel/linux.git/drivers/mtd/ofpart.c, branch v4.4.8</title>
<subtitle>Linux kernel stable tree (mirror)</subtitle>
<id>https://git.radix-linux.su/kernel/linux.git/atom?h=v4.4.8</id>
<link rel='self' href='https://git.radix-linux.su/kernel/linux.git/atom?h=v4.4.8'/>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/'/>
<updated>2015-12-09T01:10:20+00:00</updated>
<entry>
<title>doc: dt: mtd: partitions: add compatible property to "partitions" node</title>
<updated>2015-12-09T01:10:20+00:00</updated>
<author>
<name>Brian Norris</name>
<email>computersforpeace@gmail.com</email>
</author>
<published>2015-12-03T22:47:32+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=e488ca9f8d4f62c2dc36bfa5c32f68e7f05ab381'/>
<id>urn:sha1:e488ca9f8d4f62c2dc36bfa5c32f68e7f05ab381</id>
<content type='text'>
As noted here [1], there are potentially future conflicts if we try to
use MTD's "partitions" subnode to describe anything besides just the
fixed-in-the-device-tree partitions currently described in this
document. Particularly, there was a proposal to use this node for the
AFS parser too.

It can pose a (small) problem to try to differentiate the following
nodes:

	// using binding as currently specified
	partitions {
		#address-cells = &lt;x&gt;;
		#size-cells = &lt;y&gt;;
		partition@0 {
			...;
		};
	};

and

	// proposed future binding
	partitions {
		compatible = "arm,arm-flash-structure";
	};

It's especially difficult if other uses of this node start having
subnodes.

So, since the "partitions" node is new in v4.4, let's fixup the binding
before release so that it requires a compatible property, so it's much
clearer to distinguish. e.g.:

	// proposed
	partitions {
		compatible = "fixed-partitions";
		#address-cells = &lt;x&gt;;
		#size-cells = &lt;y&gt;;
		partition@0 {
			...;
		};
	};

[1] Subject: "mtd: create a partition type device tree binding"
    http://lkml.kernel.org/g/20151113220039.GA74382@google.com
    http://lists.infradead.org/pipermail/linux-mtd/2015-November/063355.html
    http://lists.infradead.org/pipermail/linux-mtd/2015-November/063364.html

Cc: Michal Suchanek &lt;hramrach@gmail.com&gt;
Signed-off-by: Brian Norris &lt;computersforpeace@gmail.com&gt;
Acked-by: Rob Herring &lt;robh@kernel.org&gt;
</content>
</entry>
<entry>
<title>mtd: ofpart: don't complain about missing 'partitions' node too loudly</title>
<updated>2015-12-04T01:50:36+00:00</updated>
<author>
<name>Brian Norris</name>
<email>computersforpeace@gmail.com</email>
</author>
<published>2015-12-03T22:26:52+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=8c62b4e118cfa7a3c906c01d4ba2c78a5bd97531'/>
<id>urn:sha1:8c62b4e118cfa7a3c906c01d4ba2c78a5bd97531</id>
<content type='text'>
The ofpart partition parser might be run on DT-enabled systems that
don't have any "ofpart" partition subnodes at all, since "ofpart" is in
the default parser list. So don't complain loudly on every boot.

Example: using m25p80.c with no intent to use ofpart:

&amp;spi2 {
	status = "okay";

	flash@0 {
		compatible = "jedec,spi-nor";
		reg = &lt;0&gt;;
	};
};

I see this warning:

[    0.588471] m25p80 spi2.0: gd25q32 (4096 Kbytes)
[    0.593091] spi2.0: 'partitions' subnode not found on /spi@ff130000/flash@0. Trying to parse direct subnodes as partitions.

Cc: Michal Suchanek &lt;hramrach@gmail.com&gt;
Signed-off-by: Brian Norris &lt;computersforpeace@gmail.com&gt;
</content>
</entry>
<entry>
<title>mtd: ofpart: move ofpart partitions to a dedicated dt node</title>
<updated>2015-10-30T21:01:39+00:00</updated>
<author>
<name>Michal Suchanek</name>
<email>hramrach@gmail.com</email>
</author>
<published>2015-08-18T15:34:09+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=5cfdedb7b9a0fe38aa4838bfe66fb9ebc2c9ce15'/>
<id>urn:sha1:5cfdedb7b9a0fe38aa4838bfe66fb9ebc2c9ce15</id>
<content type='text'>
Parsing direct subnodes of a mtd device as partitions is unreliable
since the mtd device is also part of its bus subsystem and can contain
bus data in subnodes.

Move ofpart data to a subnode of its own so it is clear which data is
part of the partition layout.

Signed-off-by: Michal Suchanek &lt;hramrach@gmail.com&gt;
Signed-off-by: Brian Norris &lt;computersforpeace@gmail.com&gt;
</content>
</entry>
<entry>
<title>mtd: make register_mtd_parser return void</title>
<updated>2014-01-03T19:22:22+00:00</updated>
<author>
<name>Axel Lin</name>
<email>axel.lin@ingics.com</email>
</author>
<published>2013-12-01T11:01:06+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=6e14a61d412eb87ef7bdcec8b08a95bead771a78'/>
<id>urn:sha1:6e14a61d412eb87ef7bdcec8b08a95bead771a78</id>
<content type='text'>
register_mtd_parser never fails; hence make it return void.

Signed-off-by: Axel Lin &lt;axel.lin@ingics.com&gt;
Signed-off-by: Brian Norris &lt;computersforpeace@gmail.com&gt;
</content>
</entry>
<entry>
<title>mtd: make mtd_partition.name const</title>
<updated>2014-01-03T19:22:09+00:00</updated>
<author>
<name>Geert Uytterhoeven</name>
<email>geert@linux-m68k.org</email>
</author>
<published>2013-11-12T19:11:26+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=26a6d240e2a1480a33d76ac4db8855b6a7f2bd89'/>
<id>urn:sha1:26a6d240e2a1480a33d76ac4db8855b6a7f2bd89</id>
<content type='text'>
This allows to drop a few casts.

Signed-off-by: Geert Uytterhoeven &lt;geert@linux-m68k.org&gt;
Signed-off-by: Brian Norris &lt;computersforpeace@gmail.com&gt;
</content>
</entry>
<entry>
<title>mtd: ofpart: use for_each_child_of_node() macro</title>
<updated>2013-08-30T20:52:57+00:00</updated>
<author>
<name>Wei Yongjun</name>
<email>yongjun_wei@trendmicro.com.cn</email>
</author>
<published>2013-08-23T03:04:27+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=60ea89e22aa5de529d0fdb9de07504afa8e5d4b0'/>
<id>urn:sha1:60ea89e22aa5de529d0fdb9de07504afa8e5d4b0</id>
<content type='text'>
Use for_each_child_of_node() macro instead of open coding it.

Signed-off-by: Wei Yongjun &lt;yongjun_wei@trendmicro.com.cn&gt;
Acked-by: Huang Shijie &lt;b32955@freescale.com&gt;
Signed-off-by: David Woodhouse &lt;David.Woodhouse@intel.com&gt;
</content>
</entry>
<entry>
<title>mtd: ofpart: add compatible check for child nodes</title>
<updated>2013-08-05T20:12:32+00:00</updated>
<author>
<name>Josh Wu</name>
<email>josh.wu@atmel.com</email>
</author>
<published>2013-08-05T11:14:38+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=e79265ba6bdb31437bd4c3e7911950f9d1262a07'/>
<id>urn:sha1:e79265ba6bdb31437bd4c3e7911950f9d1262a07</id>
<content type='text'>
In case that the nand device will support some features like Nand Flash
Controller, we want to make the sub feature as a sub node of nand device.

Use such organization it is easy to enable/disable feature, also it is back
compatible and more readable.

If the sub-node has a compatible property then it is a driver not partition.

Signed-off-by: Josh Wu &lt;josh.wu@atmel.com&gt;
Acked-by: Brian Norris &lt;computersforpeace@gmail.com&gt;
Acked-by: Jean-Christophe PLAGNIOL-VILLARD &lt;plagnioj@jcrosoft.com&gt;
Signed-off-by: Artem Bityutskiy &lt;artem.bityutskiy@linux.intel.com&gt;
[ added a missing newline -Brian ]
Signed-off-by: Brian Norris &lt;computersforpeace@gmail.com&gt;
Signed-off-by: David Woodhouse &lt;David.Woodhouse@intel.com&gt;
</content>
</entry>
<entry>
<title>mtd: ofpart: support partitions of 4 GiB and larger</title>
<updated>2013-04-05T11:05:53+00:00</updated>
<author>
<name>Joe Schaack</name>
<email>jschaack@xes-inc.com</email>
</author>
<published>2013-02-21T22:29:45+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=05ff8c258c76dd469232a03ec8d45f7e7267e0ac'/>
<id>urn:sha1:05ff8c258c76dd469232a03ec8d45f7e7267e0ac</id>
<content type='text'>
Previously, partitions were limited to less than 4 GiB in size because
the address and size were read as 32-bit values. Add support for 64-bit
values to support devices of 4 GiB and larger.

Signed-off-by: Joe Schaack &lt;jschaack@xes-inc.com&gt;
Signed-off-by: Nate Case &lt;ncase@xes-inc.com&gt;
Signed-off-by: Aaron Sierra &lt;asierra@xes-inc.com&gt;
Signed-off-by: Artem Bityutskiy &lt;artem.bityutskiy@linux.intel.com&gt;
Signed-off-by: David Woodhouse &lt;David.Woodhouse@intel.com&gt;
</content>
</entry>
<entry>
<title>mtd: Allow removal of partitioning modules</title>
<updated>2013-02-04T07:27:33+00:00</updated>
<author>
<name>Lubomir Rintel</name>
<email>lkundrak@v3.sk</email>
</author>
<published>2013-01-16T01:12:50+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=422f3890a68333cfd39f2133cf7255ed067e92e6'/>
<id>urn:sha1:422f3890a68333cfd39f2133cf7255ed067e92e6</id>
<content type='text'>
Signed-off-by: Lubomir Rintel &lt;lkundrak@v3.sk&gt;
Signed-off-by: Artem Bityutskiy &lt;artem.bityutskiy@linux.intel.com&gt;
</content>
</entry>
<entry>
<title>mtd: ofpart: Replicate mtd cmdline "lk" option with device tree "lock" property</title>
<updated>2012-11-22T08:14:22+00:00</updated>
<author>
<name>Josh Radel</name>
<email>jradel@gmail.com</email>
</author>
<published>2012-11-14T22:11:32+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=ab0b00bc6ebcfbff42dedb3bf53f0ece79b27e88'/>
<id>urn:sha1:ab0b00bc6ebcfbff42dedb3bf53f0ece79b27e88</id>
<content type='text'>
The mtd partition command line parser already supports a "lk" option to mask
MTD_POWERUP_LOCK. This extends that same functionality to device tree
partition specifications.

Signed-off-by: Josh Radel &lt;jradel@gmail.com&gt;
Signed-off-by: Artem Bityutskiy &lt;artem.bityutskiy@linux.intel.com&gt;
</content>
</entry>
</feed>
