From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id E98EEC761AF for ; Wed, 22 Mar 2023 12:14:51 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230156AbjCVMOt (ORCPT ); Wed, 22 Mar 2023 08:14:49 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:41248 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229922AbjCVMOr (ORCPT ); Wed, 22 Mar 2023 08:14:47 -0400 Received: from hosting.gsystem.sk (hosting.gsystem.sk [212.5.213.30]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id 2C6B936478; Wed, 22 Mar 2023 05:14:46 -0700 (PDT) Received: from [192.168.0.2] (chello089173232159.chello.sk [89.173.232.159]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by hosting.gsystem.sk (Postfix) with ESMTPSA id 04F0C7A00F2; Wed, 22 Mar 2023 13:14:45 +0100 (CET) From: Ondrej Zary To: Jiapeng Chong Subject: Re: [PATCH] pata_parport: Clean up some inconsistent indenting Date: Wed, 22 Mar 2023 13:14:41 +0100 User-Agent: KMail/1.9.10 Cc: damien.lemoal@opensource.wdc.com, linux-ide@vger.kernel.org, linux-kernel@vger.kernel.org, Abaci Robot References: <20230322055519.67900-1-jiapeng.chong@linux.alibaba.com> In-Reply-To: <20230322055519.67900-1-jiapeng.chong@linux.alibaba.com> X-KMail-QuotePrefix: > MIME-Version: 1.0 Content-Type: Text/Plain; charset="utf-8" Content-Transfer-Encoding: 7bit Content-Disposition: inline Message-Id: <202303221314.41470.linux@zary.sk> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wednesday 22 March 2023 06:55:19 Jiapeng Chong wrote: > No functional modification involved. > > drivers/ata/pata_parport/comm.c:159 comm_write_block() warn: inconsistent indenting. This patch does not fix any real problem. All the protocol drivers have coding style problems but they need other work first. > Reported-by: Abaci Robot > Link: https://bugzilla.openanolis.cn/show_bug.cgi?id=4587 > Signed-off-by: Jiapeng Chong > --- > drivers/ata/pata_parport/comm.c | 227 ++++++++++++++++---------------- > 1 file changed, 116 insertions(+), 111 deletions(-) > > diff --git a/drivers/ata/pata_parport/comm.c b/drivers/ata/pata_parport/comm.c > index 4c2f9ad60ad8..b474bda59323 100644 > --- a/drivers/ata/pata_parport/comm.c > +++ b/drivers/ata/pata_parport/comm.c > @@ -1,12 +1,13 @@ > -/* > - comm.c (c) 1997-8 Grant R. Guenther > - Under the terms of the GNU General Public License. > - > - comm.c is a low-level protocol driver for some older models > - of the DataStor "Commuter" parallel to IDE adapter. Some of > - the parallel port devices marketed by Arista currently > - use this adapter. > -*/ > +// SPDX-License-Identifier: GPL-2.0-or-later > +/* > + * comm.c (c) 1997-8 Grant R. Guenther > + * Under the terms of the GNU General Public License. > + * > + * comm.c is a low-level protocol driver for some older models > + * of the DataStor "Commuter" parallel to IDE adapter. Some of > + * the parallel port devices marketed by Arista currently > + * use this adapter. > + */ > > #include > #include > @@ -18,120 +19,126 @@ > #include "pata_parport.h" > > /* mode codes: 0 nybble reads, 8-bit writes > - 1 8-bit reads and writes > - 2 8-bit EPP mode > -*/ > + * 1 8-bit reads and writes > + * 2 8-bit EPP mode > + */ > > -#define j44(a,b) (((a>>3)&0x0f)|((b<<1)&0xf0)) > +#define j44(a, b) (((a>>3)&0x0f)|((b<<1)&0xf0)) > > -#define P1 w2(5);w2(0xd);w2(0xd);w2(5);w2(4); > -#define P2 w2(5);w2(7);w2(7);w2(5);w2(4); > +#define P1 w2(5); w2(0xd); w2(0xd); w2(5); w2(4); > +#define P2 w2(5); w2(7); w2(7); w2(5); w2(4); > > -/* cont = 0 - access the IDE register file > - cont = 1 - access the IDE command set > -*/ > +/* > + * cont = 0 - access the IDE register file > + * cont = 1 - access the IDE command set > + */ > > static int cont_map[2] = { 0x08, 0x10 }; > > static int comm_read_regr(struct pi_adapter *pi, int cont, int regr) > +{ > + int l, h, r; > > -{ int l, h, r; > - > - r = regr + cont_map[cont]; > + r = regr + cont_map[cont]; > > - switch (pi->mode) { > + switch (pi->mode) { > > - case 0: w0(r); P1; w0(0); > - w2(6); l = r1(); w0(0x80); h = r1(); w2(4); > - return j44(l,h); > + case 0: w0(r); P1; w0(0); > + w2(6); l = r1(); w0(0x80); h = r1(); w2(4); > + return j44(l, h); > > - case 1: w0(r+0x20); P1; > - w0(0); w2(0x26); h = r0(); w2(4); > - return h; > + case 1: w0(r+0x20); P1; > + w0(0); w2(0x26); h = r0(); w2(4); > + return h; > > case 2: > case 3: > - case 4: w3(r+0x20); (void)r1(); > - w2(0x24); h = r4(); w2(4); > - return h; > + case 4: w3(r+0x20); (void)r1(); > + w2(0x24); h = r4(); w2(4); > + return h; > > - } > - return -1; > -} > + } > > -static void comm_write_regr(struct pi_adapter *pi, int cont, int regr, int val) > + return -1; > +} > > -{ int r; > +static void comm_write_regr(struct pi_adapter *pi, int cont, int regr, int val) > +{ > + int r; > > - r = regr + cont_map[cont]; > + r = regr + cont_map[cont]; > > - switch (pi->mode) { > + switch (pi->mode) { > > - case 0: > - case 1: w0(r); P1; w0(val); P2; > + case 0: > + case 1: w0(r); P1; w0(val); P2; > break; > > case 2: > case 3: > - case 4: w3(r); (void)r1(); w4(val); > - break; > - } > + case 4: w3(r); (void)r1(); w4(val); > + break; > + } > } > > static void comm_connect(struct pi_adapter *pi) > - > -{ pi->saved_r0 = r0(); > - pi->saved_r2 = r2(); > - w2(4); w0(0xff); w2(6); > - w2(4); w0(0xaa); w2(6); > - w2(4); w0(0x00); w2(6); > - w2(4); w0(0x87); w2(6); > - w2(4); w0(0xe0); w2(0xc); w2(0xc); w2(4); > +{ > + pi->saved_r0 = r0(); > + pi->saved_r2 = r2(); > + w2(4); w0(0xff); w2(6); > + w2(4); w0(0xaa); w2(6); > + w2(4); w0(0x00); w2(6); > + w2(4); w0(0x87); w2(6); > + w2(4); w0(0xe0); w2(0xc); w2(0xc); w2(4); > } > > static void comm_disconnect(struct pi_adapter *pi) > - > -{ w2(0); w2(0); w2(0); w2(4); > +{ > + w2(0); w2(0); w2(0); w2(4); > w0(pi->saved_r0); > - w2(pi->saved_r2); > -} > + w2(pi->saved_r2); > +} > > static void comm_read_block(struct pi_adapter *pi, char *buf, int count) > > -{ int i, l, h; > - > - switch (pi->mode) { > - > - case 0: w0(0x48); P1; > - for(i=0;i - w0(0); w2(6); l = r1(); > - w0(0x80); h = r1(); w2(4); > - buf[i] = j44(l,h); > - } > - break; > - > - case 1: w0(0x68); P1; w0(0); > - for(i=0;i - w2(0x26); buf[i] = r0(); w2(0x24); > - } > +{ int i, l, h; > + > + switch (pi->mode) { > + case 0: w0(0x48); P1; > + for (i = 0; i < count; i++) { > + w0(0); w2(6); l = r1(); > + w0(0x80); h = r1(); w2(4); > + buf[i] = j44(l, h); > + } > + > + break; > + case 1: w0(0x68); P1; w0(0); > + for (i = 0; i < count; i++) { > + w2(0x26); > + buf[i] = r0(); > + w2(0x24); > + } > + > w2(4); > break; > - > case 2: w3(0x68); (void)r1(); w2(0x24); > - for (i=0;i + for (i = 0; i < count; i++) > + buf[i] = r4(); > + > w2(4); > break; > + case 3: w3(0x68); (void)r1(); w2(0x24); > + for (i = 0; i < count/2; i++) > + ((u16 *)buf)[i] = r4w(); > > - case 3: w3(0x68); (void)r1(); w2(0x24); > - for (i=0;i - w2(4); > - break; > + w2(4); > + break; > + case 4: w3(0x68); (void)r1(); w2(0x24); > + for (i = 0; i < count/4; i++) > + ((u32 *)buf)[i] = r4l(); > > - case 4: w3(0x68); (void)r1(); w2(0x24); > - for (i=0;i - w2(4); > - break; > - > + w2(4); > + break; > } > } > > @@ -139,43 +146,41 @@ static void comm_read_block(struct pi_adapter *pi, char *buf, int count) > > static void comm_write_block(struct pi_adapter *pi, char *buf, int count) > > -{ int k; > +{ > + int k; > > - switch (pi->mode) { > + switch (pi->mode) { > + case 0: > + case 1: w0(0x68); P1; > + for (k = 0; k < count; k++) { > + w2(5); > + w0(buf[k^1]); > + w2(7); > + } > > - case 0: > - case 1: w0(0x68); P1; > - for (k=0;k - w2(5); w0(buf[k^1]); w2(7); > - } > - w2(5); w2(4); > - break; > - > - case 2: w3(0x48); (void)r1(); > - for (k=0;k - break; > - > - case 3: w3(0x48); (void)r1(); > - for (k = 0; k < count / 2; k++) > + w2(5); w2(4); > + break; > + case 2: w3(0x48); (void)r1(); > + for (k = 0; k < count; k++) > + w4(buf[k^1]); > + break; > + case 3: w3(0x48); (void)r1(); > + for (k = 0; k < count/2; k++) > w4w(swab16(((u16 *)buf)[k])); > - break; > - > - case 4: w3(0x48); (void)r1(); > - for (k = 0; k < count / 4; k++) > - w4l(swab16(((u16 *)buf)[2 * k]) | > - swab16(((u16 *)buf)[2 * k + 1]) << 16); > - break; > - > - > - } > + break; > + case 4: w3(0x48); (void)r1(); > + for (k = 0; k < count/4; k++) > + w4l(swab16(((u16 *)buf)[2 * k]) | swab16(((u16 *)buf)[2 * k + 1]) << 16); > + break; > + } > } > > static void comm_log_adapter(struct pi_adapter *pi) > - > -{ char *mode_string[5] = {"4-bit","8-bit","EPP-8","EPP-16","EPP-32"}; > +{ > + char *mode_string[5] = {"4-bit", "8-bit", "EPP-8", "EPP-16", "EPP-32"}; > > dev_info(&pi->dev, "DataStor Commuter at 0x%x, mode %d (%s), delay %d\n", > - pi->port, pi->mode, mode_string[pi->mode], pi->delay); > + pi->port, pi->mode, mode_string[pi->mode], pi->delay); > } > > static struct pi_protocol comm = { -- Ondrej Zary