From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751207AbdCQKnI (ORCPT ); Fri, 17 Mar 2017 06:43:08 -0400 Received: from mail-wm0-f65.google.com ([74.125.82.65]:34873 "EHLO mail-wm0-f65.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751029AbdCQKnF (ORCPT ); Fri, 17 Mar 2017 06:43:05 -0400 From: Daniel Perez de Andres To: sre@kernel.org Cc: linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org, nicolassaenzj@gmail.com, Daniel Perez Subject: [PATCH v3] Power: supply: sbs-charger: simplified bool function Date: Fri, 17 Mar 2017 11:42:41 +0100 Message-Id: <1489747361-8957-1-git-send-email-danielperezdeandres@gmail.com> X-Mailer: git-send-email 2.7.4 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Daniel Perez This patch rewrites the implementation of the regmap's readable function sbs_readable_reg Signed-off-by: Daniel Perez --- v2 -> v3: - v3 is needed because version history was misplaced in the patch message. v1 -> v2: - given Joe Perches' comment, I've further simplified the syntax of the bool function, removing the ternary conditional drivers/power/supply/sbs-charger.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/power/supply/sbs-charger.c b/drivers/power/supply/sbs-charger.c index 353765a..15947db 100644 --- a/drivers/power/supply/sbs-charger.c +++ b/drivers/power/supply/sbs-charger.c @@ -137,10 +137,7 @@ static enum power_supply_property sbs_properties[] = { static bool sbs_readable_reg(struct device *dev, unsigned int reg) { - if (reg < SBS_CHARGER_REG_SPEC_INFO) - return false; - else - return true; + return reg >= SBS_CHARGER_REG_SPEC_INFO; } static bool sbs_volatile_reg(struct device *dev, unsigned int reg) -- 2.7.4