From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1763462AbdKRVRD (ORCPT ); Sat, 18 Nov 2017 16:17:03 -0500 Received: from mout.web.de ([217.72.192.78]:57341 "EHLO mout.web.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1763436AbdKRVQy (ORCPT ); Sat, 18 Nov 2017 16:16:54 -0500 To: alsa-devel@alsa-project.org, Jaroslav Kysela , Takashi Iwai Cc: LKML , kernel-janitors@vger.kernel.org From: SF Markus Elfring Subject: [PATCH] ALSA: cs5530: Use common error handling code in snd_cs5530_probe() Message-ID: Date: Sat, 18 Nov 2017 22:15:57 +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:0QMLiA2hBA7TgaK+7/NJFOkl/o4CK18KTUm7ym+2cPWCGXXDp86 5ZLYuKzA2vq7h2uYdgs2vtCXdncDkZ/o6aDUkajQrYQHMgOawRUbxu8Nr9wBDbuVdM+KBY/ p1wHO9Nzd9BpoOIjIe23NsJcNSeaxXSz9E4XFm1kUF9lXTtlIUIsohlsAr2x2D0YQdX6r/v dM/PonayfwHFLwSs+mIiQ== X-UI-Out-Filterresults: notjunk:1;V01:K0:dE7J1kRiwfY=:+4wcCC2siuNYocuQfWl2pv 2xJ1rqHLXKUG8MFWl0SurZ/f3OV4JlOXkPZDKJl2JP2YQKD5hNjOxDdDUe1lHN6w1ej72gRP9 W6y97IEc/1Ha08DoIMf03Eih+gtPls8UgfqkE7IWRnG8hY/6kSnT2hHhaFcalXFLeuE/oFuE3 cSAO7nn/HUThfaUSH+ZgyuQBbXHoBoyNfrgLWC+XxhlkxWdiYmAl8uV2dR7o/OlJbt+IiIn9L AHjBqwa6pYCasXz8zZxWHhJ6uQFZ3klbrdJSrBl1fwdxXT5N+7woKnWeNOiU1TJV2SHL+3mqA QvpItotYG1cZQ0vcpEsc7Y/7twrqHrAvwVEsMQbxWzIIxXf/UgdHTlnmFh9UAtWGMdHEQxeHX q4wCxfUBVgZnvB82K7uoW90QB2f8wOC1A/2QckmjIONJG2j6Z3YH3vxcZt2O0encBN5cKQIDZ 2nSfIEmTScNQ9P7xWmfrVBRgo4gtSfXaEaOdhTP1cJc+M2irDNSckTAcI3zbG4cPdSoymYbBf 9JpI4mNGOsOu4s7GwyNqTq3ZE/Wm/ZkSlkLU2GdzS7qC+ext/CUmqNlCt9TylA7ISdCP1UkbA W+H/k011qkKPL/J2/tyvc5eA7C7zvz0Awe/Ipe2KjNySllp2BcxFPu2NQ4R96mc/j7RPmvm2Y 9yAEGBDEw9MylFkBhSUQigmFCOB/ds2olMjMIPtTNhWMD5CZqSSY1zaGB6cVud56X0Zj14Ej1 cB2BZi1pt+mCnA41mxUfZetiZzKPVRA5kexEMI7MRPov/m5CZMoixLSSRxQZwvzYgVOiaaE1N /xhk8txA1P9A4IrnhxPAefu8vr0/pa8F401DL31yCxbYNoucM8= Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Markus Elfring Date: Sat, 18 Nov 2017 22:10:19 +0100 Add a jump target so that a bit of exception handling can be better reused at the end of this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- sound/pci/cs5530.c | 17 +++++++++-------- 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/sound/pci/cs5530.c b/sound/pci/cs5530.c index 0a8cf94c4858..674572edf2d2 100644 --- a/sound/pci/cs5530.c +++ b/sound/pci/cs5530.c @@ -269,23 +269,24 @@ static int snd_cs5530_probe(struct pci_dev *pci, return err; err = snd_cs5530_create(card, pci, &chip); - if (err < 0) { - snd_card_free(card); - return err; - } + if (err < 0) + goto free_card; strcpy(card->driver, "CS5530"); strcpy(card->shortname, "CS5530 Audio"); sprintf(card->longname, "%s at 0x%lx", card->shortname, chip->pci_base); err = snd_card_register(card); - if (err < 0) { - snd_card_free(card); - return err; - } + if (err < 0) + goto free_card; + pci_set_drvdata(pci, card); dev++; return 0; + +free_card: + snd_card_free(card); + return err; } static struct pci_driver cs5530_driver = { -- 2.15.0 From mboxrd@z Thu Jan 1 00:00:00 1970 From: SF Markus Elfring Date: Sat, 18 Nov 2017 21:15:57 +0000 Subject: [PATCH] ALSA: cs5530: Use common error handling code in snd_cs5530_probe() Message-Id: List-Id: References: <6acf9697-a5df-0cd4-6cac-1198980c943b@users.sourceforge.net> In-Reply-To: <6acf9697-a5df-0cd4-6cac-1198980c943b@users.sourceforge.net> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: alsa-devel@alsa-project.org, Jaroslav Kysela , Takashi Iwai Cc: kernel-janitors@vger.kernel.org, LKML From: Markus Elfring Date: Sat, 18 Nov 2017 22:10:19 +0100 Add a jump target so that a bit of exception handling can be better reused at the end of this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- sound/pci/cs5530.c | 17 +++++++++-------- 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/sound/pci/cs5530.c b/sound/pci/cs5530.c index 0a8cf94c4858..674572edf2d2 100644 --- a/sound/pci/cs5530.c +++ b/sound/pci/cs5530.c @@ -269,23 +269,24 @@ static int snd_cs5530_probe(struct pci_dev *pci, return err; err = snd_cs5530_create(card, pci, &chip); - if (err < 0) { - snd_card_free(card); - return err; - } + if (err < 0) + goto free_card; strcpy(card->driver, "CS5530"); strcpy(card->shortname, "CS5530 Audio"); sprintf(card->longname, "%s at 0x%lx", card->shortname, chip->pci_base); err = snd_card_register(card); - if (err < 0) { - snd_card_free(card); - return err; - } + if (err < 0) + goto free_card; + pci_set_drvdata(pci, card); dev++; return 0; + +free_card: + snd_card_free(card); + return err; } static struct pci_driver cs5530_driver = { -- 2.15.0 From mboxrd@z Thu Jan 1 00:00:00 1970 From: SF Markus Elfring Subject: [PATCH] ALSA: cs5530: Use common error handling code in snd_cs5530_probe() Date: Sat, 18 Nov 2017 22:15:57 +0100 Message-ID: Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mout.web.de (mout.web.de [217.72.192.78]) by alsa0.perex.cz (Postfix) with ESMTP id 90F2A266EFA for ; Sat, 18 Nov 2017 22:16:18 +0100 (CET) Content-Language: en-GB List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: alsa-devel-bounces@alsa-project.org To: alsa-devel@alsa-project.org, Jaroslav Kysela , Takashi Iwai Cc: kernel-janitors@vger.kernel.org, LKML List-Id: alsa-devel@alsa-project.org From: Markus Elfring Date: Sat, 18 Nov 2017 22:10:19 +0100 Add a jump target so that a bit of exception handling can be better reused at the end of this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- sound/pci/cs5530.c | 17 +++++++++-------- 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/sound/pci/cs5530.c b/sound/pci/cs5530.c index 0a8cf94c4858..674572edf2d2 100644 --- a/sound/pci/cs5530.c +++ b/sound/pci/cs5530.c @@ -269,23 +269,24 @@ static int snd_cs5530_probe(struct pci_dev *pci, return err; err = snd_cs5530_create(card, pci, &chip); - if (err < 0) { - snd_card_free(card); - return err; - } + if (err < 0) + goto free_card; strcpy(card->driver, "CS5530"); strcpy(card->shortname, "CS5530 Audio"); sprintf(card->longname, "%s at 0x%lx", card->shortname, chip->pci_base); err = snd_card_register(card); - if (err < 0) { - snd_card_free(card); - return err; - } + if (err < 0) + goto free_card; + pci_set_drvdata(pci, card); dev++; return 0; + +free_card: + snd_card_free(card); + return err; } static struct pci_driver cs5530_driver = { -- 2.15.0