From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pb0-f46.google.com (mail-pb0-f46.google.com [209.85.160.46]) by dpdk.org (Postfix) with ESMTP id 06778B0CC for ; Wed, 14 May 2014 18:35:06 +0200 (CEST) Received: by mail-pb0-f46.google.com with SMTP id rq2so1901178pbb.19 for ; Wed, 14 May 2014 09:35:14 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:message-id:user-agent:date:from:to:cc:subject :references:mime-version:content-type:content-disposition; bh=66bIgpbypgO+D5gnd1aYVEqhn+NHtsAkpDf/jPSser8=; b=QXavsZ3JxnxarkSjBkT9XzCjlzU5/7UF9O8zfkS7gDrCyzBc6Wdr7UKupoAKIRRXoU xMLHuzrdMSQpnMvUkd2MkSJgk/kZvjp2VR4dN1fiYAP16bThf+TiCJf6P8Y1nPZWeCBb rQgwV3HZmVBEdv8ROVIKol7CxxqHIl8M9EHT1vmghkog9IqJN36RZj6X4YKXJbL/ZXJp mmZvLOKE+Lixy2+h+A2Hs0YK7rfUnhpJewuw6/7Nwk+Mnkz9QdH6f0jRQJDdfzF5R62C UBiBxliK8TQMH595U/KtXFwLAexF7bbLDFBiz7MJEEl3HZIYV9ozXllXLM/v7HDu80pC 9yLw== X-Gm-Message-State: ALoCoQlYYdXmxXMx6UR3Fd/LDHdWyDnZXgPx1bDxx5Esk98WCgk+TWGWDH561/ypZcUAljTF5m4e X-Received: by 10.66.139.38 with SMTP id qv6mr5644132pab.123.1400085314403; Wed, 14 May 2014 09:35:14 -0700 (PDT) Received: from localhost (static-50-53-83-51.bvtn.or.frontiernet.net. [50.53.83.51]) by mx.google.com with ESMTPSA id fu12sm9846988pad.42.2014.05.14.09.35.13 for (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Wed, 14 May 2014 09:35:13 -0700 (PDT) Message-Id: <20140514163512.445351102@networkplumber.org> User-Agent: quilt/0.61-1 Date: Wed, 14 May 2014 09:34:06 -0700 From: Stephen Hemminger To: dev@dpdk.org References: <20140514163401.079384157@networkplumber.org> MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-15 Content-Disposition: inline; filename=igb_uio-checkpatch.diff Cc: Stephen Hemminger Subject: [dpdk-dev] [PATCH 5/5] igb_uio: fix checkpatch warnings X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 14 May 2014 16:35:07 -0000 Fix whitespace and other style issues reported by checkpatch. Signed-off-by: Stephen Hemminger --- a/lib/librte_eal/linuxapp/igb_uio/igb_uio.c 2014-05-14 09:32:25.366870293 -0700 +++ b/lib/librte_eal/linuxapp/igb_uio/igb_uio.c 2014-05-14 09:32:25.366870293 -0700 @@ -1,23 +1,23 @@ /*- * GPL LICENSE SUMMARY - * + * * Copyright(c) 2010-2014 Intel Corporation. All rights reserved. - * + * * This program is free software; you can redistribute it and/or modify * it under the terms of version 2 of the GNU General Public License as * published by the Free Software Foundation. - * + * * This program is distributed in the hope that it will be useful, but * WITHOUT ANY WARRANTY; without even the implied warranty of * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU * General Public License for more details. - * + * * You should have received a copy of the GNU General Public License * along with this program; if not, write to the Free Software * Foundation, Inc., 51 Franklin St - Fifth Floor, Boston, MA 02110-1301 USA. * The full GNU General Public License is included in this distribution * in the file called LICENSE.GPL. - * + * * Contact Information: * Intel Corporation */ @@ -32,7 +32,7 @@ #include #include -#ifdef CONFIG_XEN_DOM0 +#ifdef CONFIG_XEN_DOM0 #include #endif @@ -98,11 +98,11 @@ int local_pci_num_vf(struct pci_dev *dev u16 total; u16 initial; u16 nr_virtfn; - } *iov = (struct iov*)dev->sriov; + } *iov = (struct iov *)dev->sriov; if (!dev->is_physfn) return 0; - + return iov->nr_virtfn; #else return pci_num_vf(dev); @@ -135,13 +135,13 @@ store_max_vfs(struct device *dev, struct else /* do nothing if change max_vfs number */ err = -EINVAL; - return err ? err : count; + return err ? err : count; } static DEVICE_ATTR(max_vfs, S_IRUGO | S_IWUSR, show_max_vfs, store_max_vfs); static struct attribute *dev_attrs[] = { &dev_attr_max_vfs.attr, - NULL, + NULL, }; static const struct attribute_group dev_attr_grp = { @@ -283,9 +283,9 @@ igbuio_dom0_pci_mmap(struct uio_info *in { int idx; - if (vma->vm_pgoff >= MAX_UIO_MAPS) + if (vma->vm_pgoff >= MAX_UIO_MAPS) return -EINVAL; - if(info->mem[vma->vm_pgoff].size == 0) + if (info->mem[vma->vm_pgoff].size == 0) return -EINVAL; idx = (int)vma->vm_pgoff; @@ -297,7 +297,7 @@ igbuio_dom0_pci_mmap(struct uio_info *in default: return -EINVAL; } -} +} #endif /* Remap pci resources described by bar #pci_bar in uio resource n. */ @@ -308,8 +308,8 @@ igbuio_pci_setup_iomem(struct pci_dev *d unsigned long addr, len; void *internal_addr; - if (sizeof(info->mem) / sizeof (info->mem[0]) <= n) - return (EINVAL); + if (sizeof(info->mem) / sizeof(info->mem[0]) <= n) + return -EINVAL; addr = pci_resource_start(dev, pci_bar); len = pci_resource_len(dev, pci_bar); @@ -333,20 +333,20 @@ igbuio_pci_setup_ioport(struct pci_dev * { unsigned long addr, len; - if (sizeof(info->port) / sizeof (info->port[0]) <= n) - return (EINVAL); + if (sizeof(info->port) / sizeof(info->port[0]) <= n) + return -EINVAL; addr = pci_resource_start(dev, pci_bar); len = pci_resource_len(dev, pci_bar); if (addr == 0 || len == 0) - return (-1); + return -1; info->port[n].name = name; info->port[n].start = addr; info->port[n].size = len; info->port[n].porttype = UIO_PORT_X86; - return (0); + return 0; } /* Unmap previously ioremap'd resources */ @@ -382,14 +382,18 @@ igbuio_setup_bars(struct pci_dev *dev, s pci_resource_start(dev, i) != 0) { flags = pci_resource_flags(dev, i); if (flags & IORESOURCE_MEM) { - if ((ret = igbuio_pci_setup_iomem(dev, info, - iom, i, bar_names[i])) != 0) - return (ret); + ret = igbuio_pci_setup_iomem(dev, info, + iom, i, + bar_names[i]); + if (ret != 0) + return ret; iom++; } else if (flags & IORESOURCE_IO) { - if ((ret = igbuio_pci_setup_ioport(dev, info, - iop, i, bar_names[i])) != 0) - return (ret); + ret = igbuio_pci_setup_ioport(dev, info, + iop, i, + bar_names[i]); + if (ret != 0) + return ret; iop++; } }