From mboxrd@z Thu Jan 1 00:00:00 1970 From: Wolfgang Grandegger Subject: Re: [PATCH net-next-2.6 v4] can: Topcliff: PCH_CAN driver: Add Flow control/Fix Endianess issue/Separate IF register/Enumerate LEC macro/Move MSI processing/Use BIT(X)/Change Message Object index/Add prefix PCH_ Date: Tue, 16 Nov 2010 21:39:47 +0100 Message-ID: <4CE2EC13.9090707@grandegger.com> References: <4CE275A4.9010400@dsn.okisemi.com> <20101116.091443.226753420.davem@davemloft.net> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: andrew.chih.howe.khor-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org, socketcan-core-0fE9KPoRgkgATYTw5x5z8w@public.gmane.org, sameo-VuQAYsv1563Yd54FQh9/CA@public.gmane.org, margie.foster-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org, netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, yong.y.wang-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org, masa-korg-ECg8zkTtlr0C6LszWs/t0g@public.gmane.org, kok.howg.ewe-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org, chripell-VaTbYqLCNhc@public.gmane.org, joel.clark-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org, qi.wang-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org To: David Miller Return-path: In-Reply-To: <20101116.091443.226753420.davem-fT/PcQaiUtIeIZ0/mPfg9Q@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: socketcan-core-bounces-0fE9KPoRgkgATYTw5x5z8w@public.gmane.org Errors-To: socketcan-core-bounces-0fE9KPoRgkgATYTw5x5z8w@public.gmane.org List-Id: netdev.vger.kernel.org Hi David, On 11/16/2010 06:14 PM, David Miller wrote: > > Way too many changes in one patch. > > Please post one that fixes the endianness issues. > > One that fixes the register interface seperation. > > One that adds flow control processing. > > etc. > > When you combine many tasks into one patch it's impossible > to bisect through your changes to debug problems in order > to figure out which changed introduced a bug. > > I am not applying this, and I will not apply your patches > until you split them up properly. > > You may think that there is zero value in this, but there > is huge value in it for anyone who tries to debug your > changes in the future. Right now you are making that a > nearly impossible task. Please take into account that this patch got accepted by accident (because the maintainer did not respond properly in time). At that time the driver was incomplete, not ready for mainline and did not even work properly. Therefore it makes little sense to debug or even bisec these changes. Just for that reason I made an exemption and added my "Acked-by". Hope you can share my arguments. Wolfgang.