From mboxrd@z Thu Jan 1 00:00:00 1970 From: Johannes Berg Subject: Re: [PATCH 08/11] net/fsl: use of_property_read_bool Date: Fri, 05 Aug 2016 12:12:28 +0200 Message-ID: <1470391948.2977.33.camel@sipsolutions.net> References: <1470387411-15879-1-git-send-email-Julia.Lawall@lip6.fr> <1470387411-15879-9-git-send-email-Julia.Lawall@lip6.fr> (sfid-20160805_111640_944584_00A38E89) <1470389433.2977.31.camel@sipsolutions.net> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 8bit Cc: netdev@vger.kernel.org, kernel-janitors@vger.kernel.org, linux-kernel@vger.kernel.org To: Julia Lawall Return-path: In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org On Fri, 2016-08-05 at 12:08 +0200, Julia Lawall wrote: >  > Can I do the same for: > >         if (of_property_read_bool(np, "phy-clk-valid")) >                 pdata->check_phy_clk_valid = 1; >         else >                 pdata->check_phy_clk_valid = 0; > > The type is not bool, but: > > include/linux/fsl_devices.h:    unsigned        check_phy_clk_valid:1; > The type doesn't even matter, does it? Any bool->int conversion should result in 0/1. johannes