From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934065AbcAYSUh (ORCPT ); Mon, 25 Jan 2016 13:20:37 -0500 Received: from mout.web.de ([212.227.17.12]:57248 "EHLO mout.web.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932792AbcAYSUd (ORCPT ); Mon, 25 Jan 2016 13:20:33 -0500 Subject: Re: [PATCH 1/2] [media] m88rs6000t: Better exception handling in five functions To: Mauro Carvalho Chehab References: <566ABCD9.1060404@users.sourceforge.net> <5680FDB3.7060305@users.sourceforge.net> <56810F56.4080306@users.sourceforge.net> <568148FD.7080209@users.sourceforge.net> <5681497E.7030702@users.sourceforge.net> <20160125150136.449f2593@recife.lan> Cc: linux-media@vger.kernel.org, LKML , kernel-janitors@vger.kernel.org, Julia Lawall From: SF Markus Elfring X-Enigmail-Draft-Status: N1110 Message-ID: <56A6662C.3000804@users.sourceforge.net> Date: Mon, 25 Jan 2016 19:15:08 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.5.1 MIME-Version: 1.0 In-Reply-To: <20160125150136.449f2593@recife.lan> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit X-Provags-ID: V03:K0:VyhAxDCT2orM2izU2U6NlZx939JgJ+xWU+KVpHjFKTCe5+CVTGZ 6RMxGGFj18q4l4m/huMlL1zEHoCHcihV3D8QWE51KNrE2GSsTpBH1nK7FS/J9gGS+e8dyAe XYbRXqgrX2eN7D2UvO6srpf6qtsCTRAWW2x3Cr2cAiKsohq7vdWPhNXbEOanq0NLMa+PVSx Hg0QOA8AgR6QVoJYhrjgA== X-UI-Out-Filterresults: notjunk:1;V01:K0:4K9R4WAOdQo=:6rU7qLQkPeI9rmuDNYYhrZ XzuuUCD2eLNptvceMNGNMa9qF7GnPNutmqnLI1gP7H6AvZP6Ns330s96BlNpJiwu39ejHu9Hg gChlm9jmB39NgwHDAI+G6zbm2dyODBeyRKduJZhMSrLO1WaT4eAeHhj6qAe/7+FPtFVgL0bNW b7DEYTKUuQLNLutEZU4Qu82lvInsKPNKDhiXiclGrMGw4uSyFMW+WCLvAWQAbaq9Op/z9xBtc 1E8prW7nMArtmnXAwu0ry+GUwUq5IFBCqPo3dePWlPjD74pDLhAfhT5hUnPQX+2JR+UTR4d0h aWoVAWjoJg8umibYOSC8t4nhr7olMhyF3IQYLLWa++jWoupYnRCYbsTZkln24Qupplrayk0XM WObWTnlHcjxAui8AwL6aux31b7HHGuUihiKJZ9qP5fymiibx1cQLlVvgOPmtT9/UiRLHDCDtv LhVXetwGEw5S4plMQ0nLbps76koyN1qAGrL7LF8gR3AaBPbHrm2Lh/TdBVbE2N2d261urX1Yr WwJPeqglo1VqhbT2552aOL3PYtqeH6id/HD73+JV6DfGDRi1WuGvUnO02iKWigX8SAusp4sjJ BRyoJ6sPoRawsV1H4YdZQ/1fu9Pe8F8k2xEFICduLUpO23ABwsvbRozceE/PZhEBhY/++D+E9 Bzv/+9dL5pBz97E+4jbfXgx/YpOt8IqiZKVz3JV0tSNPpfPr/RN0HV59auFGen7s490PdunZu dU4A4DagP6jby6Y+w84MjyARqJmVESJYfhuTAWUG2MelfTw6BU/tpaq7s6sbqSsLR+CJrltTw k9dId9A Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org >> This issue was detected by using the Coccinelle software. >> >> Move the jump label directly before the desired log statement >> so that the variable "ret" will not be checked once more >> after a function call. >> Use the identifier "report_failure" instead of "err". >> >> Suggested-by: Julia Lawall >> Signed-off-by: Markus Elfring >> --- >> drivers/media/tuners/m88rs6000t.c | 154 +++++++++++++++++++------------------- >> 1 file changed, 78 insertions(+), 76 deletions(-) >> >> diff --git a/drivers/media/tuners/m88rs6000t.c b/drivers/media/tuners/m88rs6000t.c >> index 504bfbc..7e59a9f 100644 >> --- a/drivers/media/tuners/m88rs6000t.c >> +++ b/drivers/media/tuners/m88rs6000t.c >> @@ -44,7 +44,7 @@ static int m88rs6000t_set_demod_mclk(struct dvb_frontend *fe) >> /* select demod main mclk */ >> ret = regmap_read(dev->regmap, 0x15, &utmp); >> if (ret) >> - goto err; >> + goto report_failure; > > Why to be so verbose? Does the document "CodingStyle" give an indication in the section "Chapter 7: Centralized exiting of functions"? > Calling it as "err" is enough, It seems that some short identifiers are popular during software development. > and it means less code to type if we need to add another goto. Would you like to increase the usage of jump labels which will contain only a single character? Regards, Markus