Return-Path: Subject: [PATCH 5/5] Bluetooth: btmrvl: Use common error handling code in btmrvl_sdio_download_fw_w_helper() From: SF Markus Elfring To: linux-bluetooth@vger.kernel.org, Johan Hedberg , Marcel Holtmann Cc: LKML , kernel-janitors@vger.kernel.org References: <8d050f76-aed1-7b6b-8778-a78aef34d4ea@users.sourceforge.net> Message-ID: Date: Mon, 12 Mar 2018 12:35:33 +0100 MIME-Version: 1.0 In-Reply-To: <8d050f76-aed1-7b6b-8778-a78aef34d4ea@users.sourceforge.net> Content-Type: text/plain; charset=utf-8 List-ID: From: Markus Elfring Date: Mon, 12 Mar 2018 11:30:28 +0100 Add a jump target so that the setting of a specific error code is stored only once at the end of this function. Signed-off-by: Markus Elfring --- drivers/bluetooth/btmrvl_sdio.c | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/drivers/bluetooth/btmrvl_sdio.c b/drivers/bluetooth/btmrvl_sdio.c index 05c78fcc13ff..24ed62fe2aeb 100644 --- a/drivers/bluetooth/btmrvl_sdio.c +++ b/drivers/bluetooth/btmrvl_sdio.c @@ -601,8 +601,7 @@ static int btmrvl_sdio_download_fw_w_helper(struct btmrvl_sdio_card *card) " base0 = 0x%04X(%d)." " Terminating download", base0, base0); - ret = -EIO; - goto done; + goto e_io; } base1 = sdio_readb(card->func, card->reg->sq_read_base_addr_a1, &ret); @@ -611,8 +610,7 @@ static int btmrvl_sdio_download_fw_w_helper(struct btmrvl_sdio_card *card) " base1 = 0x%04X(%d)." " Terminating download", base1, base1); - ret = -EIO; - goto done; + goto e_io; } len = (((u16) base1) << 8) | base0; @@ -638,8 +636,7 @@ static int btmrvl_sdio_download_fw_w_helper(struct btmrvl_sdio_card *card) if (count > MAX_WRITE_IOMEM_RETRY) { BT_ERR("FW download failure @%d, " "over max retry count", offset); - ret = -EIO; - goto done; + goto e_io; } BT_ERR("FW CRC error indicated by the helper: " "len = 0x%04X, txlen = %d", len, txlen); @@ -681,6 +678,10 @@ static int btmrvl_sdio_download_fw_w_helper(struct btmrvl_sdio_card *card) kfree(tmpfwbuf); release_firmware(fw_firmware); return ret; + +e_io: + ret = -EIO; + goto done; } static int btmrvl_sdio_card_to_host(struct btmrvl_private *priv) -- 2.16.2