From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755300AbdKAUuq (ORCPT ); Wed, 1 Nov 2017 16:50:46 -0400 Received: from mout.web.de ([212.227.15.14]:56106 "EHLO mout.web.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751922AbdKAUun (ORCPT ); Wed, 1 Nov 2017 16:50:43 -0400 To: linux-pwm@vger.kernel.org, Thierry Reding Cc: LKML , kernel-janitors@vger.kernel.org From: SF Markus Elfring Subject: [PATCH] pwm-twl: Use common error handling code in twl6030_pwm_disable() Message-ID: <97ad7b8f-a5ca-b867-41a9-c3b46507eddb@users.sourceforge.net> Date: Wed, 1 Nov 2017 21:50:37 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0 MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Language: en-GB Content-Transfer-Encoding: 7bit X-Provags-ID: V03:K0:WuPP37VKQf1yohg8G9dmPfpCCuFUujEHsBkK3M7RkHxTqY8502C hXVdIuC1DgF8k8+fj0ftyHjomV+wx6c+jal4AdID871tkfmuOcqCbpsWtVlsPPhqxi0vonB Y3iYn14ZjsYUhLwrnzLyXt9UAF/G+5bMmFBoM7oidbIgVo+8YuIEnsA/UBNqBgNvz769kSt U7NWamwHTBjH5fnuEVeYQ== X-UI-Out-Filterresults: notjunk:1;V01:K0:DHnfxmjT59g=:Q2b1rGCPxsqLFPrtYU0XYJ Ju0+Gxmnu/UltpsWBQ/3j3YiX5EX444yxWsmWY5DBxgU4AvKW/lSz41wtSWsSHpz44MZLVPtS v20PjkCooSLOStuWvIrjwaYGgVTslNOhlUtrI8yUSGc0RUn/XwcLV86yDvplTS0KJaAgPOub5 m4Sgm2aaQxw4o+HRa/Aq0wbktkVbaqgiFdRMPn6IyEf9ybiB0J30ElmVYBbIdp2d6/pWZi13D kieDAXytPc86baAw+dQCvcla1GgxenrSxsnVNrsOIyUeSQhB0CubdWakHTUfuICpz14aE3Ygr xYNc1CTKJsS/v2r1dvNpCQmPFZLD4FG4LTDZe3oQEtNmbhHXXBAkAvQ10NCwVqxvODBFjNFRY Oo6QJKnQeCYuBz7yGUaF38A6kXKZ5xUNlOhKvavTIV6M//PxNJ1BO741LV4ytHwbtUx7WWfnj /a5bIaE7xo2Br490pk3cePZUsxp2HXZY2uVhNrGCha6bzqOcoSRn+ibsvCOdyDR9+dt9PuqCr Opf7NIq1/LG7J2EkbXwK8Zk1vzx4jEl4y+H8o7ZLHuGiqCYn7vVfG0TPbeYycECl1j0BQ8P2h +K26V9HmmBuVbbtm4gK+o2JkmQv4eiKFPTlqro/R1gzAUMljYxLu0WMeAzLxssxpFUjknHGUm bTSmJZ2BdEhB63DpBjlSDQwzxwoJzlgz5EPDo6ujIegin+YAgAOJiJXmGKEA1TpYZnpMT3kAj oYHDrsMgeP+vyz4UfVWLk8BkD/hD+W5rCGHi8gTOl8JJUjSNAd/+l47UMJtcrPyb8itinxMwG wrNanByfLbNc6NzQ4z1JavGD0fDkhEuTGe3fKe0bQ1JrdjQZh8= Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Markus Elfring Date: Wed, 1 Nov 2017 21:44:30 +0100 * Adjust jump targets so that a specific error message is stored only once at the end of this function implementation. * Replace three calls of the function "dev_err" by goto statements. * Adjust three condition checks. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/pwm/pwm-twl.c | 25 ++++++++++++------------- 1 file changed, 12 insertions(+), 13 deletions(-) diff --git a/drivers/pwm/pwm-twl.c b/drivers/pwm/pwm-twl.c index b7a45be99815..0367fb8e425b 100644 --- a/drivers/pwm/pwm-twl.c +++ b/drivers/pwm/pwm-twl.c @@ -264,30 +264,29 @@ static void twl6030_pwm_disable(struct pwm_chip *chip, struct pwm_device *pwm) val &= ~TWL6030_PWM_TOGGLE(pwm->hwpwm, TWL6030_PWMXS | TWL6030_PWMXEN); ret = twl_i2c_write_u8(TWL6030_MODULE_ID1, val, TWL6030_TOGGLE3_REG); - if (ret < 0) { - dev_err(chip->dev, "%s: Failed to disable PWM\n", pwm->label); - goto out; - } + if (ret) + goto report_failure; val |= TWL6030_PWM_TOGGLE(pwm->hwpwm, TWL6030_PWMXEN); ret = twl_i2c_write_u8(TWL6030_MODULE_ID1, val, TWL6030_TOGGLE3_REG); - if (ret < 0) { - dev_err(chip->dev, "%s: Failed to disable PWM\n", pwm->label); - goto out; - } + if (ret) + goto report_failure; val &= ~TWL6030_PWM_TOGGLE(pwm->hwpwm, TWL6030_PWMXEN); ret = twl_i2c_write_u8(TWL6030_MODULE_ID1, val, TWL6030_TOGGLE3_REG); - if (ret < 0) { - dev_err(chip->dev, "%s: Failed to disable PWM\n", pwm->label); - goto out; - } + if (ret) + goto report_failure; twl->twl6030_toggle3 = val; -out: +unlock: mutex_unlock(&twl->mutex); + return; + +report_failure: + dev_err(chip->dev, "%s: Failed to disable PWM\n", pwm->label); + goto unlock; } static const struct pwm_ops twl4030_pwm_ops = { -- 2.14.3