From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752815AbbCXXGF (ORCPT ); Tue, 24 Mar 2015 19:06:05 -0400 Received: from mail-wi0-f169.google.com ([209.85.212.169]:34082 "EHLO mail-wi0-f169.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752273AbbCXXGD (ORCPT ); Tue, 24 Mar 2015 19:06:03 -0400 Message-ID: <5511EDD7.40202@linaro.org> Date: Tue, 24 Mar 2015 23:05:59 +0000 From: Srinivas Kandagatla User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.4.0 MIME-Version: 1.0 To: Mark Brown CC: linux-arm-kernel@lists.infradead.org, Maxime Ripard , Rob Herring , Kumar Gala , Greg Kroah-Hartman , linux-api@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-msm@vger.kernel.org, arnd@arndb.de, sboyd@codeaurora.org Subject: Re: [PATCH v3 1/9] regmap: Introduce regmap_get_max_register. References: <1427236116-18531-1-git-send-email-srinivas.kandagatla@linaro.org> <1427236179-18580-1-git-send-email-srinivas.kandagatla@linaro.org> <20150324223637.GB28997@sirena.org.uk> In-Reply-To: <20150324223637.GB28997@sirena.org.uk> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 24/03/15 22:36, Mark Brown wrote: > On Tue, Mar 24, 2015 at 10:29:39PM +0000, Srinivas Kandagatla wrote: > >> This patch introduces regmap_get_max_register() function which would be >> used by the infrastructures like eeprom framework built on top of >> regmap. > > In what way would it be used? Its used in 2 purposes 1> It is used for sanity check purposes of the register ranges provided via DT/non-DT eeprom data cells. 2> To stop user reading when eeprom binary from /sys/class/eeprom/*/eeprom > >> +int regmap_get_max_register(struct regmap *map) >> +{ >> + return map->max_register ? : -EINVAL; >> +} > > Please write the logic out properly, don't abuse the ternery operator. Am happy to change it to your preference in next version, but this GNU extensions widely used in the kernel $ grep -rR "return.*? :" ./linux | wc -l 115 >