From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1760043AbcHYQBQ (ORCPT ); Thu, 25 Aug 2016 12:01:16 -0400 Received: from mail-sn1nam02on0055.outbound.protection.outlook.com ([104.47.36.55]:49315 "EHLO NAM02-SN1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1760024AbcHYQBO (ORCPT ); Thu, 25 Aug 2016 12:01:14 -0400 Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=David.Daney@cavium.com; Message-ID: <57BF1644.5060207@caviumnetworks.com> Date: Thu, 25 Aug 2016 09:01:08 -0700 From: David Daney User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130625 Thunderbird/17.0.7 MIME-Version: 1.0 To: Javier Martinez Canillas CC: , , Aaro Koskinen , David Daney , Greg Kroah-Hartman , Cristina Moraru , Laura Garcia Liebana , Bhaktipriya Shridhar , Philippe Reynes Subject: Re: [RESEND PATCH] staging: octeon: Use IS_ENABLED() instead of checking for built-in or module References: <1472133315-14640-1-git-send-email-javier@osg.samsung.com> In-Reply-To: <1472133315-14640-1-git-send-email-javier@osg.samsung.com> Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit X-Originating-IP: [50.233.148.156] X-ClientProxiedBy: BLUPR07CA057.namprd07.prod.outlook.com (10.160.24.12) To BN4PR07MB2130.namprd07.prod.outlook.com (10.164.63.12) X-MS-Office365-Filtering-Correlation-Id: e5bd88a0-4f23-4685-3287-08d3cd0108b6 X-Microsoft-Exchange-Diagnostics: 1;BN4PR07MB2130;2:sqYs7pjF8g26KmO9vnpOoWlWXwZgnho/40ZnN4LtdBoaKm9arSuRY4fu8UbQZ9Fv1ARgmvj+JLT5uKBAvb0Zre3PVH3aQ4pj3HU+DbjhAte53cDAy9uKsefr7iQc+GwyPT49bkIUbN3al79S5dSr+RaZOrsTmZleMD5GOZgZqYcXQPlKuPwwfYuRKccmmphk;3:U/uC4WlC9pO04WpE3/tg5Ht0hWO6K3thmsWeWfWxP7DQ1ZFv2wwKYixZ3WViksEkJfTKUalVu5gyDCJuJdGVS+QQZw3n0B/16D12iNb5iAhw8YDPsfaGjvt+nJCDHq/b X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:BN4PR07MB2130; X-Microsoft-Exchange-Diagnostics: 1;BN4PR07MB2130;25:QksYSj7AqCqSMoiJDdPUN56hj+34oieyKwbICCzaZIfsspP7nMnInYm481hzc7FwnLtb/iBDegCW/R5tvsP8oaRGZ28f4F8vvpbaAKmxowlJhuNnGu6jxamZNR3kMb/nzR2L67IyxEKyN2VdF0sEN1DHZ3KrBKgasMrWdl1iO5UaJ0x9583OFWC9Sl4SOjsg1o9t1cypoa3a2AN8fdr7vCZNkDQXweMzUcF8pRDGQ+WEIxGOfxxOUIc5bHa3eqbukV5R/+BAVOKlyvq94mZHH25wewSfc1+/0J/GWXISf/HmE1InBenus2MHsjfcCkfSFuvO1ZclUkbD6jvWixmcGolYQFLRJ/k6+K2l2Q6+nd2h8E8NINiI2cuWvnPqNDBLBPeawgWzViH2PlLi4g4AA/LRuXoxdumFbAEIjREpDgPXExKLkJ6TmVXtzLYlOtpywMWhoatXGeZzTw5Ufwxj8AYtpCST1BELa3W1fj8YUd6HSxKk/cM08BaRYQ5l293a50HEKQzZI6bpNygYPri3pjWnYnYsg6F64iyvCevhbT8nDbnQCNJwZNCDnTLyDYmFgkhWnaBlYHOgGezOVi35o6oNh01b1NH+kb1DaWAaBrJFLYi7gI5/UREZlHlDUWE0WRSlVghytR3yWbBcgSJa4TN7wbao17fEvrK4Zsquq2dyv3unZ2vf+YOtDP+1EX2mcBeXvN8u9PkKKkA9mM+MtJlJx7n+7bhfS69j8RsuuwNIQpEwpWvU93mrDDiF2xuP X-Microsoft-Exchange-Diagnostics: 1;BN4PR07MB2130;31:xWAaEoy1WHePv4VcVqr1adl9LMa72zFr2KhEGEkuKcMiQIeiSWDsuToyNyE2/IO0ZgQtEU0Wi3KtoJgPvTj3QY25JSgZZoQfbjWQy9ao+++M0l9JKnDktyBBNTf9RMe1ube1fS/kkSSn2wefKWRySjBjAnrXskosAXzFNKza7OBaWW3iG6hRoQPrJKWwwekkQ0RKrWUNHJxAx3FK0jyLNd7wA4Xew3cFQmZdGtB/GuA=;20:Jo4+3je8mto6MQQWoHAonfI9cyR7iZ7lthlfLOvE1HxvnO3INDKja9/vZO9cRgiGS51FCPPRnNG0aO21YBz1ln0H7Sv091953ms/MSftZfcou7ZGSUKEztZZDJYZYYbuyNkWgA7noKArMNFC+1WXygFOUyvhrp5O8u5TJGzlfbc8pwppe5YEWFwFCrevfC8sMdcaHMzkZR5NBQYK+5VFdkt+T3BvseBjmCPyVMuQMRHxvq8Ue95U37xcHldny5KYD2GxWAguxXdFRLiTIVsbOF16nwIBZVCc/k601EBV5qaMTlgT9ALlwpFD5aVulOjjwSp6NwtY0J1W7DTzqXHR4neZIajxqNyy0fhHzcTZJ4IYqohZhwp6EBn1lx8JT2chh6Cm1sBVZV3u9qr6HjcoZzwRquzahpjteIlYE6yw90satgVa4eNfO0HmxKs0DE8wBKyYy0d+gfZtR24IfqFxj908jEFcxUilDgLIqIVWxsnIFJ7nKYfDT7crob/iL4k0n59XH3jI/ypnnheJs9qCGumWa727ccw1TJgZveVt+Dy7HWurOxKlAJ+bDkKNzoyFjHmTZ/3WEMwYDCNRZmSg6cZPsK+o/GTmINx8sdVo01s= X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:(7411616537696); X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(6040176)(601004)(2401047)(8121501046)(5005006)(3002001)(10201501046);SRVR:BN4PR07MB2130;BCL:0;PCL:0;RULEID:;SRVR:BN4PR07MB2130; X-Microsoft-Exchange-Diagnostics: 1;BN4PR07MB2130;4:t2jBtoEzKbzh0Vjt75x/ZstOxz09HqhmRSsfJUHvnS3IyEiGLooTDxplaji29mPGfXGzHgVR65SHpBYKuj91riMARrGe6Y3s4bmEDCtLn8cW9Gc5FTJCKsGaDx+KzFP94vaKxt7Vu7XoBGLPqVFB+7ANigmjPWqugmxSABMXmWiCkXCkXdrj2x1QarPjl+7TShXM5nLCjPIws0jvtZqmttR4VILN6CrVuC+0cANC4bX3jGCNk+5htwBgDPfbFgIOFSTLJikk2+W69+RF8Awl6Hn6RbRztag0lsK0jU9mThiFH76uOYgChIdpMl/TmnAqC/vfSzcYp+WUvj7ZSF9Qy38hrsVt+z16/4/Sb44MrnUaUvd/aNMdCvIQ3Bx0Ul/bDz13+8WKmyE/Go3CSR1c4bSlyfOt8THPSCHwYaxsDLLDcsLNaHUxRfnug30YXP1h X-Forefront-PRVS: 0045236D47 X-Forefront-Antispam-Report: SFV:NSPM;SFS:(10009020)(4630300001)(6009001)(7916002)(24454002)(377454003)(199003)(189002)(33656002)(19580395003)(80316001)(83506001)(19580405001)(65816999)(101416001)(50466002)(54356999)(76176999)(50986999)(87266999)(64126003)(23756003)(105586002)(69596002)(42186005)(53416004)(36756003)(77096005)(2950100001)(65956001)(66066001)(189998001)(65806001)(47776003)(6116002)(3846002)(68736007)(7736002)(7846002)(92566002)(305945005)(2906002)(4326007)(230700001)(586003)(110136002)(59896002)(81166006)(81156014)(4001350100001)(5660300001)(97736004)(8676002)(106356001);DIR:OUT;SFP:1101;SCL:1;SRVR:BN4PR07MB2130;H:dl.caveonetworks.com;FPR:;SPF:None;PTR:InfoNoRecords;A:1;MX:1;LANG:en; X-Microsoft-Exchange-Diagnostics: =?iso-8859-1?Q?1;BN4PR07MB2130;23:v+sIInviBDVKQDlWWSaQug4kjQHxB/sWCRLf9GR?= =?iso-8859-1?Q?u2cz7T5UY0l8yVSe4mZDmp370R9DjXiPxa6tiLjv6O6+6xduqbuSrEAfWS?= =?iso-8859-1?Q?iq47l7YRRJ5Qhcq+y3CcjMujyik96CpushLsX0VPQWrPTwcdM3CW1eqnED?= =?iso-8859-1?Q?JGQ87/sVyINzGQqAZOx4mf+yUlRa2ptPsh3vV+6OJlJjp/MrpnS1MaEQXD?= =?iso-8859-1?Q?XuejUCgoVH+j5bBxr5KsI4TSK7f2YZ0fl0kQakWpuJktpcGhT4mHamMC/H?= =?iso-8859-1?Q?qW/o3vBLhO7ODkL8Ty8dUxAuEp9/2yytwzeOYsdNJmH98dvCIwaM5FDL9P?= =?iso-8859-1?Q?RWlSVYbUZUJ4FKHayCOGXxInSlB/L50Nw5blCapuf0IIWmRQ1Sb2frEyi7?= =?iso-8859-1?Q?/Buodf8hRl48kvSr3BtZtpG57N4U/Ngj6FV5o3oR+Rwzh44Tgie9Tf84rQ?= =?iso-8859-1?Q?Nqzhhl8SVntaHoJBrwtlCa6Hg9SOS491Yl0hXsky4soLvSTPKU2BTWj5B8?= =?iso-8859-1?Q?fhawZjKoZv5SOPMSqPbtTPRDMvVaiGaVVOIPMYZqsAvbo+sTok6V3cDmoJ?= =?iso-8859-1?Q?0nqnSQ2eay0EDV8P5yXMmx2IoueHKifbrmqyrBiYXWQXw8hOyzLooMUG6D?= =?iso-8859-1?Q?ImkWeEzDQKCC47hOu0j6/sELwWqtgkRLv7H+qOb0NcUnvD/YYCajxc/S/7?= =?iso-8859-1?Q?NBez7Qf8ILcheqItPoSEQha+K17b3WLlI9zNkKFKRX9eSNjJUFWBI9ci/i?= =?iso-8859-1?Q?WUv74q1pKqBsTcrYJms8mHyZmPSt8VULdOogBfvBNCyTD4HYj0aAr49yi4?= =?iso-8859-1?Q?XGoRnsDyMFk6H/+3aH/uXgRFz0q6vXEBB1qe08RKsKwH1vMJgO0o71Bxj6?= =?iso-8859-1?Q?TRxTdYYVpnnFQgNKwazRMyi2aFMHAi1/Y6LL2uDg8F3jZhKxuUTO7QnILw?= =?iso-8859-1?Q?dJwhgz9V0y1pho0lV9248zoJyDNz9U2NMiUdDyhTDZUgYP6PQCYzM76YG3?= =?iso-8859-1?Q?NwEpZvwnBRA5gB2qBkYWSKQrObcfHUxuEUW7HBIefYCiG2r2xvK+EkkUOi?= =?iso-8859-1?Q?59ebs6OgeCLauMM8I+gYEFS3aco9HO/AB3KHOZwPPfqxjmi67ci0UKyBho?= =?iso-8859-1?Q?42N9mkYaKsa9Kj4+b3x6anoMM026LfI9cO0xPTjchiiXIa2IOZK9lznBXS?= =?iso-8859-1?Q?Wj8FEYH2IfBIcQstGAmGLryzzbQpWRH/SwLHWE02UDxN3U+zIzFbli7ndH?= =?iso-8859-1?Q?Q5ygWQ3HdW3XhZNdfoe9NAjF7EAWvp4NqBFlHLJp2hr8zTFV/+NckH4AIr?= =?iso-8859-1?Q?GXoaWTMIo0oIAnH5rYUKIGemg8PSbdCrjzMRbDneKByoHga7EnGdj3ewpN?= =?iso-8859-1?Q?3PQkLq1o=3D?= X-Microsoft-Exchange-Diagnostics: 1;BN4PR07MB2130;6:OCgZkcL5mThrU0JWPNlzFC82lcTJ/wyPzAig0tnFTi5mu45bBldSuuabS2V3lSpC3N/66u43fUyuwsgJBes4c5ORJtmQ+5/5L+itZL/GM9YRkRRZAuRE9EWtKsGJGt4jk+bKu5x95TgjhwDY0kKX7lyem5Bomc0rL0EAB0pJUYBpGzSuB/eiU+qzGOPSwgDe9XLeGNm84MsqaExCW9ykHEhwb8AxwIOEpw0UUPU7C6RoNajuiWZjAYfTsB3rRRfAH2GLrn5tHHZl6in/YVRuPKT8wzKqShhQ3XbqsceESDM=;5:vLYWc9JQSKULDPb1iZsysIICH9y9NyPOlHmxc25gX3c+IIltV5OJFPl0qmHbAkST0nDnSCxp2bcXDBSNu9TqXoL45Lbh6UehOpOzGQ1IDqkypuq+bMDssldJZa0SQt5WS3TbAnPRwm3eHqnDN6C+Wg==;24:UzPB+dXcwSAZAnoq9xaBIG5yyxjSTtnrSjR5A/mmZW7pvrwwrlNEGH8bgqjdxSEfowenklNUzdXn5bo+iu8S2hgjCQBTDVHyAG/VKEaI+GU=;7:K/cKpbKkqwK7oPtFnAvaynY/270ALwiT5OhsIMC851VlETYENfhhtzQ9Qpd/CJhC8Bz9S/Em0p/boAdn+PXV645nlAGMyWra1Au/kp6eQGIe8Ud232MnBlzxgQREM1YriACM/KYAjhQD8J2Wrp2DAqxh5xi6eNyWHp67pUYn1gnQW+0ELldlc2JcUUOxvaXz9+eeHrQk7amJzeCHLBHvsPDP8/wvq25mDsNpbz+BTaqUbP2cOVKNNQCe6nXL4IyW SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: caviumnetworks.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 25 Aug 2016 16:01:10.4370 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN4PR07MB2130 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 08/25/2016 06:55 AM, Javier Martinez Canillas wrote: > The IS_ENABLED() macro checks if a Kconfig symbol has been enabled either > built-in or as a module, use that macro instead of open coding the same. > > Using the macro makes the code more readable by helping abstract away some > of the Kconfig built-in and module enable details. > > Signed-off-by: Javier Martinez Canillas > Acked-by: David Daney > > --- > > Changes since first post: > - Added David Daney Acked-by tag. > - Rebased on top of latest master. > You changed something about the patch. In this case the Subject is not "RESEND", but rather "v2". Incorrect Subject lines like this can lead to the patch being inadvertently ignored. David Daney > drivers/staging/octeon/ethernet.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/staging/octeon/ethernet.c b/drivers/staging/octeon/ethernet.c > index 2eb97317f631..a473f034b01a 100644 > --- a/drivers/staging/octeon/ethernet.c > +++ b/drivers/staging/octeon/ethernet.c > @@ -237,7 +237,7 @@ static int cvm_oct_common_change_mtu(struct net_device *dev, int new_mtu) > { > struct octeon_ethernet *priv = netdev_priv(dev); > int interface = INTERFACE(priv->port); > -#if defined(CONFIG_VLAN_8021Q) || defined(CONFIG_VLAN_8021Q_MODULE) > +#if IS_ENABLED(CONFIG_VLAN_8021Q) > int vlan_bytes = 4; > #else > int vlan_bytes = 0; >