2015-07-08 16:47:39

by Chris Metcalf

[permalink] [raw]
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<[email protected]>
> ---
> 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