Re: [RFC:PATCH dtc-1.3.0] dtc: Add --strip-disabled option to dtc.

On 08/20/2012 03:36 AM, Srinivas KANDAGATLA wrote:

On 17/08/12 16:36, Timur Tabi wrote:

quoted

Srinivas KANDAGATLA wrote:

quoted

If you know in advance that device on that SOC is broken, then I guess
"Fail"/"Failed" can be used in status property.

One user of this flag in kernel device trees is
./arch/powerpc/boot/dts/mpc8313erdb.dts

	/* Remove this (or change to "okay") if you have
	 * a REVA3 or later board, if you apply one of the
	 * workarounds listed in section 8.5 of the board
	 * manual, or if you are adapting this device tree
	 * to a different board.
	 */
	status = "fail";

I'm not sure this is the right way to do it. 

I agree, the way fail status is used is pretty much redundant to what
"disabled" is used for.
I think the device trees files should have status as "okay" or "ok" or
"disabled" or skip status property totally.

The distinction between "disabled" and "fail" is useful when the OS
knows how to enable a node (e.g. by manipulating muxing).

-Scott