[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <E5313AF6F2BFD14293E5FD0F94750F86A5CFD83694@HQ1-EXCH01.corp.brocade.com>
Date: Thu, 23 Dec 2010 10:00:32 -0800
From: Rasesh Mody <rmody@...cade.com>
To: David Miller <davem@...emloft.net>
CC: "netdev@...r.kernel.org" <netdev@...r.kernel.org>,
Debashis Dutt <ddutt@...cade.COM>,
Jing Huang <huangj@...cade.COM>
Subject: RE: [PATCH] bna: bnad_udelay macro cleanup
>From: David Miller [mailto:davem@...emloft.net]
>Sent: Thursday, December 23, 2010 9:38 AM
>
>From: Rasesh Mody <rmody@...cade.com>
>Date: Wed, 22 Dec 2010 23:05:49 -0800
>
>> Change details:
>> - Removed unnecessary bnad_udelay macro for ia64
>>
>> Signed-off-by: Debashis Dutt <ddutt@...cade.com>
>> Signed-off-by: Rasesh Mody <rmody@...cade.com>
>
>Please don't do it this way.
>
>We don't add known crap, then add a patch to remove the crap.
>
>Respin your BNA patch series so that this IA64 stuff never shows
>up in the first place.
>
>Thanks.
We'll respin and resubmit the patches with ia64 macro removed as
per your suggestion. In future we'll take care to avoid this.
--Rasesh
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists