On Tue, Aug 14, 2012 at 8:11 PM, David Gibson [off-list ref] wrote:

On Mon, Aug 13, 2012 at 09:01:53AM +0100, Srinivas KANDAGATLA wrote:

quoted

From: Srinivas Kandagatla <redacted>

This patch add pre-processing capablity to dtc based on status property.
Now the dtc has additional option -P to enable Pre-processing based on
status property.

The SOCS have lot of device tree infrastructure files which mark the
device nodes as disabled and the board level device tree enables them if
required. However while creating device tree blob, the compiler can
preprocess the nodes and exclude nodes marked as disabled, doing this
way will reduce the size of device tree blob.

IMHO, many devices that are marked as "disabled" in the DTS are
expecting to be enabled by the boot loader, so just because a node is
disabled in the DTS does not mean that it will be disabled when Linux
sees it.

quoted

In our case this has reduced the blob size from 29K to 15K.

I don't see that as significant.

quoted

Also nodes with status="disabled" is are never probed by dt platform bus
code.

Again, Preprocessing is optional parameter to dtc.

Using this option would break a lot of our device trees.  Perhaps it
should be given a better name, like --strip-disabled.

-- 
Timur Tabi
Linux kernel developer at Freescale