From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mailrelay012.isp.belgacom.be (mailrelay012.isp.belgacom.be [195.238.6.179]) by ozlabs.org (Postfix) with ESMTP id 77E0EB6FCB for ; Fri, 25 May 2012 01:07:57 +1000 (EST) Message-ID: <4FBE4E02.1070306@computer.org> Date: Thu, 24 May 2012 17:04:34 +0200 From: Jan Ceuleers MIME-Version: 1.0 To: David Miller Subject: Re: [PATCH] gianfar:don't add FCB length to hard_header_len References: <1337677248-27792-1-git-send-email-b06378@freescale.com> <20120522.151830.2018953337014839778.davem@davemloft.net> In-Reply-To: <20120522.151830.2018953337014839778.davem@davemloft.net> Content-Type: text/plain; charset=ISO-8859-1 Cc: netdev@vger.kernel.org, b06378@freescale.com, linuxppc-dev@lists.ozlabs.org List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On 05/22/2012 09:18 PM, David Miller wrote: > From: Jiajun Wu > Date: Tue, 22 May 2012 17:00:48 +0800 > >> FCB(Frame Control Block) isn't the part of netdev hard header. >> Add FCB to hard_header_len will make GRO fail at MAC comparision stage. >> >> Signed-off-by: Jiajun Wu > > Applied, thanks. > > Someone needs to go through this driver when net-next opens up > and fix all of the indentation in this driver. May I give that a go?