From patchwork Tue Mar 7 22:46:27 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Ondrej Zary X-Patchwork-Id: 65922 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a5d:5915:0:0:0:0:0 with SMTP id v21csp5122wrd; Tue, 7 Mar 2023 14:51:13 -0800 (PST) X-Google-Smtp-Source: AK7set8u+SDw4uam2DYPtBErpw7boa0gliksUPhlaN1OXiORVq3jt6cr8oOyZDX76N9YD8xa2Vtr X-Received: by 2002:a17:90b:1e4f:b0:237:aed6:fff3 with SMTP id pi15-20020a17090b1e4f00b00237aed6fff3mr16389219pjb.44.1678229473231; Tue, 07 Mar 2023 14:51:13 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1678229473; cv=none; d=google.com; s=arc-20160816; b=xQNxm4l5p7CuRYU5Mx3Li4/Vy5e9idvWO86+NT6sNeiCgrW9vMwwKCA0JxGz1xRNAH GgTFOXb9AjwEdmrD3UuVcvXxLRi2bEtZ0ugXdlnNE0Q09U0IpWhCSHA1v1pN3yeQtaSs 82GHxLQFRxSdledL2TLS/Yf4ZQVvVovGWj6nnUZ/Bn4VX1cAb9lRxQvhPY55z3bJSZlD HuXdWIGf+D8ibD7xh6RLGKyxBYh1gmajYcCAbXI9COzoTe9dH4wOMimDQvSTtto3myLD Z2q42baxiFy3X9dYfQkGn1CgpQGyMUTAOwAe0GKwF36QqgBmpUfYTXRjKfpQzIet+pEO zofg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from; bh=+xqPo6RzLMdLLcYZOFoY6b6kZUEC22dSSFVHlwNNi40=; b=TGWb/M0+UQE+mDuF4BY/XY6h6va8VafnJzFdHhF5kY5ASANYWxKVP699GSjFsYvb98 DVVl2DAmPRrOlUzR9GzCE/cMUbXGdVncA5unt9euH+6kyhGq7r4NiQemwbOrGuWxxAaX KJo7VtG8Uu+2j82Ohk9hSHNaSOkwyJ9PFiNsKsdyFaiThrgSLB7RE40GsHhnXLvHsYBf QSInQ1IPokfnIujA753zgOzd4pOsHRKhRasFsewIv25oCU6/+GohsWt/uugoFf2BCA7+ hCb2c+JvgDKTZ2DHqBSM8sKAQUpGOFifj5Rltemp1q2RDsz8HFYl6hc9xe5oO5LysVwK yrjQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id 79-20020a630252000000b00502b278306fsi12790554pgc.568.2023.03.07.14.51.00; Tue, 07 Mar 2023 14:51:13 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231802AbjCGWtU (ORCPT + 99 others); Tue, 7 Mar 2023 17:49:20 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:35128 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231785AbjCGWrp (ORCPT ); Tue, 7 Mar 2023 17:47:45 -0500 Received: from hosting.gsystem.sk (hosting.gsystem.sk [212.5.213.30]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id 0EE4B9FE78; Tue, 7 Mar 2023 14:47:08 -0800 (PST) Received: from gsql.ggedos.sk (off-20.infotel.telecom.sk [212.5.213.20]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by hosting.gsystem.sk (Postfix) with ESMTPSA id 1D93D7A077D; Tue, 7 Mar 2023 23:46:51 +0100 (CET) From: Ondrej Zary To: Damien Le Moal , Sudip Mukherjee Cc: Christoph Hellwig , Sergey Shtylyov , Jens Axboe , Tim Waugh , linux-block@vger.kernel.org, linux-parport@lists.infradead.org, linux-ide@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH 32/32] pata_parport-bpck6: reduce indents in bpck6_open Date: Tue, 7 Mar 2023 23:46:27 +0100 Message-Id: <20230307224627.28011-33-linux@zary.sk> X-Mailer: git-send-email 2.20.1 In-Reply-To: <20230307224627.28011-1-linux@zary.sk> References: <20230307224627.28011-1-linux@zary.sk> MIME-Version: 1.0 X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,SPF_HELO_NONE, SPF_PASS autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1759751147737814313?= X-GMAIL-MSGID: =?utf-8?q?1759751147737814313?= Reverse error handling conditions to reduce code indents in bpck6_open. Signed-off-by: Ondrej Zary --- drivers/ata/pata_parport/bpck6.c | 48 +++++++++++++++++--------------- 1 file changed, 25 insertions(+), 23 deletions(-) diff --git a/drivers/ata/pata_parport/bpck6.c b/drivers/ata/pata_parport/bpck6.c index 53d0ad1d6e89..295dbe5cdaa4 100644 --- a/drivers/ata/pata_parport/bpck6.c +++ b/drivers/ata/pata_parport/bpck6.c @@ -318,30 +318,32 @@ static int bpck6_open(struct pi_adapter *pi) j = ((i & 0x08) << 4) | ((i & 0x07) << 3); k = parport_read_status(pi->pardev->port) & 0xB8; - if (j == k) { - parport_frob_control(pi->pardev->port, PARPORT_CONTROL_AUTOFD, 0); - k = (parport_read_status(pi->pardev->port) & 0xB8) ^ 0xB8; - if (j == k) { - if (i & 4) // EPP - parport_frob_control(pi->pardev->port, - PARPORT_CONTROL_SELECT | PARPORT_CONTROL_INIT, 0); - else // PPC/ECP - parport_frob_control(pi->pardev->port, - PARPORT_CONTROL_SELECT, 0); - - pi->private = 0; - - bpck6_send_cmd(pi, ACCESS_REG | ACCESS_WRITE | REG_RAMSIZE); - bpck6_wr_data_byte(pi, RAMSIZE_128K); - - bpck6_send_cmd(pi, ACCESS_REG | ACCESS_READ | REG_VERSION); - if ((bpck6_rd_data_byte(pi) & 0x3F) == 0x0C) - pi->private |= fifo_wait; - - return 1; - } - } + if (j != k) + goto fail; + + parport_frob_control(pi->pardev->port, PARPORT_CONTROL_AUTOFD, 0); + k = (parport_read_status(pi->pardev->port) & 0xB8) ^ 0xB8; + if (j != k) + goto fail; + + if (i & 4) // EPP + parport_frob_control(pi->pardev->port, + PARPORT_CONTROL_SELECT | PARPORT_CONTROL_INIT, 0); + else // PPC/ECP + parport_frob_control(pi->pardev->port, PARPORT_CONTROL_SELECT, 0); + + pi->private = 0; + + bpck6_send_cmd(pi, ACCESS_REG | ACCESS_WRITE | REG_RAMSIZE); + bpck6_wr_data_byte(pi, RAMSIZE_128K); + + bpck6_send_cmd(pi, ACCESS_REG | ACCESS_READ | REG_VERSION); + if ((bpck6_rd_data_byte(pi) & 0x3F) == 0x0C) + pi->private |= fifo_wait; + + return 1; +fail: parport_write_control(pi->pardev->port, pi->saved_r2); parport_write_data(pi->pardev->port, pi->saved_r0);