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

[Chris Metcalf]

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@xxxxxxxxx>
> ---
>   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@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/