[<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/
Powered by blists - more mailing lists