linux-kernel - Re: [PATCH] tile:Remove unused marcos from the file tilepro.c

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [day] [month] [year] [list]

Message-ID: <559D541A.5050504@ezchip.com>
Date:	Wed, 8 Jul 2015 12:47:22 -0400
From:	Chris Metcalf <cmetcalf@...hip.com>
To:	Nicholas Krause <xerofoify@...il.com>
CC:	<netdev@...r.kernel.org>, <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] tile:Remove unused marcos from the file tilepro.c

On 07/06/2015 09:22 PM, Nicholas Krause wrote:
> This removes no longer used marcos at the top of the driver
> due tilepro.c due to having no proper values nor never being
> used by this driver code.
>
> Signed-off-by: Nicholas Krause<xerofoify@...il.com>
> ---
>   drivers/net/ethernet/tile/tilepro.c | 13 -------------
>   1 file changed, 13 deletions(-)

I'm not sure what you mean by "no longer used".  They are all
potentially usable to enable chunks of code lower down in
the driver, they're just not enabled by default.  So, nack.

Also, "no proper values" doesn't make a lot of sense; if you
specify "#define FOO" it implicitly defines it to the empty
string.  Regardless, it doesn't matter what value it is
defined to if the only uses are "#ifdef", as is true here.

-- 
Chris Metcalf, EZChip Semiconductor
http://www.ezchip.com

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/