From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mitch Bradley Subject: Re: [RFC:PATCH dtc-1.3.0] dtc: Add --strip-disabled option to dtc. Date: Mon, 20 Aug 2012 07:16:17 -1000 Message-ID: <503270E1.6050902@firmworks.com> References: <1345034325-26656-1-git-send-email-srinivas.kandagatla@st.com> <20120817060415.GC29724@truffula.fritz.box> <502E3632.70208@freescale.com> <502E52F3.7090404@st.com> <502E64F9.2020400@freescale.com> <5031F706.3050509@st.com> <50322F9C.2070403@freescale.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <50322F9C.2070403-KZfg59tc24xl57MIdRCFDg@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: devicetree-discuss-bounces+gldd-devicetree-discuss=m.gmane.org-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org Sender: "devicetree-discuss" To: Tabi Timur-B04825 Cc: "mmarek-AlSwsSmVLrQ@public.gmane.org" , Wood Scott-B07421 , "devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org" , David Gibson List-Id: devicetree@vger.kernel.org On 8/20/2012 2:37 AM, Tabi Timur-B04825 wrote: > Srinivas KANDAGATLA wrote: >>>> But assuming that this really is the best approach, then it would make >>>> sense for --strip-disabled to leave this node in the dtb, because >>>> otherwise there would be no way to re-enable it. >> --strip-disabled should still get rid for nodes marked as failed >> as-well, because fail means something serious and un-recoverable. > > Well, I don't know if that's true. Does status=fail really mean > unrecoverable? My intention when I first conceived of the status property is that "fail" means that something has determined that the device is not working properly and the software does not know how to make it work. That is distinct from "disabled", which means that the choice has been made not to use the device. In the modern world of SoC with physically-unconnected functional units, perhaps a new value would be appropriate: status="unused".