From patchwork Tue Dec 5 18:26:38 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Arvind Yadav X-Patchwork-Id: 844876 X-Patchwork-Delegate: davem@davemloft.net Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Authentication-Results: ozlabs.org; spf=none (mailfrom) smtp.mailfrom=vger.kernel.org (client-ip=209.132.180.67; helo=vger.kernel.org; envelope-from=linux-ide-owner@vger.kernel.org; receiver=) Authentication-Results: ozlabs.org; dkim=fail reason="signature verification failed" (2048-bit key; unprotected) header.d=gmail.com header.i=@gmail.com header.b="sLhr8IIt"; dkim-atps=neutral Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 3yrqvw04jWz9rxl for ; Wed, 6 Dec 2017 05:26:52 +1100 (AEDT) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752649AbdLES0u (ORCPT ); Tue, 5 Dec 2017 13:26:50 -0500 Received: from mail-pf0-f194.google.com ([209.85.192.194]:47101 "EHLO mail-pf0-f194.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752610AbdLES0t (ORCPT ); Tue, 5 Dec 2017 13:26:49 -0500 Received: by mail-pf0-f194.google.com with SMTP id c204so755920pfc.13; Tue, 05 Dec 2017 10:26:49 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id; bh=+7Zb5FUlOU7Mjmj89uusZTJMga7JxkSXvrwt6CmTdws=; b=sLhr8IItCIxPX/c6nXGZaPiv/s3RH770BiLUxVn+nfEkh/Pd01OgnXWC77L9dkwHI7 ljH5n9iRhsgxJ+tk76EpDg/p+hBW/auJ5Z+Yb20x+bq6siAbFrfB5zCMDLNRFZ2u3nXW w9QAjhms07KT5ScW172ypvoUrdzWz35kNb8wFNntwnMlDhyeRWy0SKXHeX+OgB/MXYDt OEl8B6Dv81Ti5CEudh1A3LaMB1tt4smxS++wkn8gL2COEr2OyizcvRcM3CtZJoK9XzN4 EaVBBYkLVnAs+BG9vY0VNxFfXZ55zm1GYQrtlEKa06yH4+TDeixPJNBK8E1NXavhTfSC OvSA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id; bh=+7Zb5FUlOU7Mjmj89uusZTJMga7JxkSXvrwt6CmTdws=; b=ID+xPuySD7un+1jNU5trTv12c7v8YmVL4gEAqKQkDm2/FcVSn5ijf0iFh7i6WbOfNB vAbboES9Ayh/55wu+rMRB0zhvhporbEqcviIKVzDl0JMOx8IPYCxELGKnNnjA8k9MxRJ yOf6pqpd4MDemhE0ngjAtuWa0VXu6Z2lQF9Ip36SmRE6npMwl5hw6fOAxkXHWoBnxCRg kxd1s07wIZXC1VG5kRUzMKdQRMYB3qzIOzmS7M0+ZBJ5ge6nsiDSobSkAEMgn6BIA31f Neup2D1TJZUhr9DYcKw82X2PqKgPln8erSAkU7ztmCRWSJJEOqA5Hrl4ceIeoMiHdNd1 09jQ== X-Gm-Message-State: AJaThX7xaUViGRwAkRYRoZzgI2Ay4oV0mh1HKR+NQ0kQAndQHiGrf1JQ ZxVQrz78NpKWR6WkenjMrIY= X-Google-Smtp-Source: AGs4zMab0i/ZOqE0Lj+vVsvBIrk7facqJeZcnil9lsfGuENBL5ARYPYrTCl/UJZHvKDQ4kfbJk0ReQ== X-Received: by 10.99.55.92 with SMTP id g28mr18754274pgn.293.1512498409287; Tue, 05 Dec 2017 10:26:49 -0800 (PST) Received: from localhost.localdomain ([103.16.68.147]) by smtp.gmail.com with ESMTPSA id v82sm1191565pfd.111.2017.12.05.10.26.47 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Tue, 05 Dec 2017 10:26:48 -0800 (PST) From: Arvind Yadav To: b.zolnierkie@samsung.com, tj@kernel.org Cc: linux-kernel@vger.kernel.org, linux-ide@vger.kernel.org Subject: [PATCH v2] pata_pdc2027x: Fix coding sytle errors Date: Tue, 5 Dec 2017 23:56:38 +0530 Message-Id: <1512498399-23577-1-git-send-email-arvind.yadav.cs@gmail.com> X-Mailer: git-send-email 2.7.4 Sender: linux-ide-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-ide@vger.kernel.org Fix these checkpatch.pl errors: Fix checkpatch.pl error: ERROR: space prohibited before open square bracket '['. ERROR: space prohibited after that '~' (ctx:WxW) + mask &= ~ (1 << (6 + ATA_SHIFT_UDMA)); ERROR: spaces required around that '?' (ctx:VxW) + long pout_required = board_idx? PDC_133_MHZ:PDC_100_MHZ; ERROR: that open brace { should be on the previous line + const struct ata_port_info *ppi[] = + { &pdc2027x_port_info[board_idx], NULL }; Signed-off-by: Arvind Yadav --- changes in v2: Subject and description was not correct. *s/sytle error/style errors/ in the patch summary *s/error/errors/ in the patch description drivers/ata/pata_pdc2027x.c | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/ata/pata_pdc2027x.c b/drivers/ata/pata_pdc2027x.c index 6348a83..d1e8b63 100644 --- a/drivers/ata/pata_pdc2027x.c +++ b/drivers/ata/pata_pdc2027x.c @@ -84,7 +84,7 @@ static int pdc2027x_set_mode(struct ata_link *link, struct ata_device **r_failed */ static struct pdc2027x_pio_timing { u8 value0, value1, value2; -} pdc2027x_pio_timing_tbl [] = { +} pdc2027x_pio_timing_tbl[] = { { 0xfb, 0x2b, 0xac }, /* PIO mode 0 */ { 0x46, 0x29, 0xa4 }, /* PIO mode 1 */ { 0x23, 0x26, 0x64 }, /* PIO mode 2 */ @@ -94,7 +94,7 @@ static struct pdc2027x_pio_timing { static struct pdc2027x_mdma_timing { u8 value0, value1; -} pdc2027x_mdma_timing_tbl [] = { +} pdc2027x_mdma_timing_tbl[] = { { 0xdf, 0x5f }, /* MDMA mode 0 */ { 0x6b, 0x27 }, /* MDMA mode 1 */ { 0x69, 0x25 }, /* MDMA mode 2 */ @@ -102,7 +102,7 @@ static struct pdc2027x_mdma_timing { static struct pdc2027x_udma_timing { u8 value0, value1, value2; -} pdc2027x_udma_timing_tbl [] = { +} pdc2027x_udma_timing_tbl[] = { { 0x4a, 0x0f, 0xd5 }, /* UDMA mode 0 */ { 0x3a, 0x0a, 0xd0 }, /* UDMA mode 1 */ { 0x2a, 0x07, 0xcd }, /* UDMA mode 2 */ @@ -277,7 +277,7 @@ static unsigned long pdc2027x_mode_filter(struct ata_device *adev, unsigned long ATA_ID_PROD_LEN + 1); /* If the master is a maxtor in UDMA6 then the slave should not use UDMA 6 */ if (strstr(model_num, "Maxtor") == NULL && pair->dma_mode == XFER_UDMA_6) - mask &= ~ (1 << (6 + ATA_SHIFT_UDMA)); + mask &= ~(1 << (6 + ATA_SHIFT_UDMA)); return mask; } @@ -520,7 +520,7 @@ static void pdc_adjust_pll(struct ata_host *host, long pll_clock, unsigned int b void __iomem *mmio_base = host->iomap[PDC_MMIO_BAR]; u16 pll_ctl; long pll_clock_khz = pll_clock / 1000; - long pout_required = board_idx? PDC_133_MHZ:PDC_100_MHZ; + long pout_required = board_idx ? PDC_133_MHZ : PDC_100_MHZ; long ratio = pout_required / pll_clock_khz; int F, R; @@ -705,8 +705,8 @@ static int pdc2027x_init_one(struct pci_dev *pdev, static const unsigned long cmd_offset[] = { 0x17c0, 0x15c0 }; static const unsigned long bmdma_offset[] = { 0x1000, 0x1008 }; unsigned int board_idx = (unsigned int) ent->driver_data; - const struct ata_port_info *ppi[] = - { &pdc2027x_port_info[board_idx], NULL }; + const struct ata_port_info *ppi[] = { + &pdc2027x_port_info[board_idx], NULL }; struct ata_host *host; void __iomem *mmio_base; int i, rc;