From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jean-Christophe PLAGNIOL-VILLARD Subject: Re: [PATCH 1/1] of: introduce helper to manage boolean Date: Tue, 13 Mar 2012 04:17:39 +0100 Message-ID: <20120313031739.GA18320@game.jcrosoft.org> References: <1328588028-24766-1-git-send-email-plagnioj@jcrosoft.com> <20120309014441.8236B3E0903@localhost> <20120309100435.GN27213@game.jcrosoft.org> <20120309162638.92E813E0880@localhost> <20120309163608.GQ27213@game.jcrosoft.org> <4F5E50DA.6080000@freescale.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <4F5E50DA.6080000-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-bounces+gldd-devicetree-discuss=m.gmane.org-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org To: Scott Wood Cc: devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org List-Id: devicetree@vger.kernel.org On 14:39 Mon 12 Mar , Scott Wood wrote: > On 03/09/2012 10:36 AM, Jean-Christophe PLAGNIOL-VILLARD wrote: > >>>> Ugh. so any value other than 1 returns false? I think that will surprise > >>>> most people. > >>>> > >>>> I don't like this api or binding. If it is a bool property, then why isn't > >>>> simply testing for the property existance sufficient? > >>> no if you want to disable it > >>> > >>> if a bool is define in the dtsi and want to disable it int the dts > >>> > >>> if you we can do the the invert > >>> > >>> if !0 => true > >>> > >>> is-ok; => true > >>> is-ok = ; => true > >>> is-ok = <0>; => false > >> > >> This is a failure of the dtc tool, not the binding. Accepting this binding > >> means we have to live with it for a very long time. It needs to be fixed > >> in dtc instead so that properties can be deleted instead of only modified. > > I understand your idea but today if you put and value in the property it's true. > > > > So is-ok = <0>; is true also which is illogical as in any language a boolean is > > true (1) or false (0). When I read the property I will understand false not true > > You could say similar things about is-ok = "no" or is-ok = "" or is-ok = > "I'd rather you didn't"... it's expected that violating the binding may > produce illogical results. today is most of the binding people use a number whe the want to be able to delete it and it's the same in most of the promgramming language Best Regards, J.