From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jiri Slaby Subject: Re: [PATCH/RFC 3/8] of: Add helper function to check MMIO register endianness Date: Wed, 12 Nov 2014 10:23:36 +0100 Message-ID: <54632718.9030603@suse.cz> References: <1415781993-7755-1-git-send-email-cernekee@gmail.com> <1415781993-7755-4-git-send-email-cernekee@gmail.com> <54631F64.8080009@suse.cz> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: linux-serial-owner@vger.kernel.org To: Kevin Cernekee Cc: Greg KH , Rob Herring , tushar.behera@linaro.org, daniel@zonque.org, haojian.zhuang@gmail.com, robert.jarzmik@free.fr, Grant Likely , Florian Fainelli , Maxime Bizon , Jonas Gorski , Linux MIPS Mailing List , "linux-serial@vger.kernel.org" , "devicetree@vger.kernel.org" List-Id: devicetree@vger.kernel.org On 11/12/2014, 10:04 AM, Kevin Cernekee wrote: >> This should actually return bool and use true/false. > > Well, the other APIs currently return an int: > > extern int of_device_is_compatible(const struct device_node *device, > const char *); > extern int of_device_is_available(const struct device_node *device); > [...] > extern int of_machine_is_compatible(const char *compat); > > Do you think it is best to change all of them at once, or just the > newly introduced function? Possibly fix all these in a separate patch and then add the new one fixed :). -- js suse labs