From mboxrd@z Thu Jan 1 00:00:00 1970 From: Michael Grzeschik Subject: Re: [PATCH 00/21] ARCNET: Defibrillation Date: Tue, 5 May 2015 18:43:10 +0200 Message-ID: <20150505164310.GA26406@pengutronix.de> References: <1429896055-31680-1-git-send-email-m.grzeschik@pengutronix.de> <1429901273.23327.5.camel@perches.com> <553AB241.3080004@pengutronix.de> <20150424.235853.1997287895150693202.davem@davemloft.net> <20150427145715.GC23151@pengutronix.de> <1430365765.2575.5.camel@perches.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: netdev@vger.kernel.org, mkl@pengutronix.de, David Miller , kernel@pengutronix.de, linux-kernel@vger.kernel.org To: Joe Perches Return-path: Content-Disposition: inline In-Reply-To: <1430365765.2575.5.camel@perches.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org Hi, and sorry for the delay. On Wed, Apr 29, 2015 at 08:49:25PM -0700, Joe Perches wrote: > On Mon, 2015-04-27 at 16:57 +0200, Michael Grzeschik wrote: > > Hi! > > > > On Fri, Apr 24, 2015 at 11:58:53PM -0400, David Miller wrote: > > > From: Marc Kleine-Budde > > > Date: Fri, 24 Apr 2015 23:14:41 +0200 > > > > > > > On 04/24/2015 08:47 PM, Joe Perches wrote: > > > >> On Fri, 2015-04-24 at 19:20 +0200, Michael Grzeschik wrote: > > > >>> Hi! > > > >> > > > >> Hello. > > > >> > > > >>> This patch series tries to reanimate the ARCNET hardware layer to be > > > >>> somehow readable and maintainable again. It includes a lot of cleanup > > > >>> patches. It also adds some fixes which leads the layer to become usable > > > >>> again. And as a special treatment it adds more features like correct > > > >>> loading and unloading of the com20020 card. > > > >> > > > >> Wow. Good for you, but why? Does anyone still use these? > > > > > > > > Yes, there are parts of the industry where "old" machines are > > > > retrofitted with new hardware...and a lot of these machines still talk > > > > ARCNET :) > > > > > > But the real issue is, this layer is development wise in the same > > > category as the IDE layer. > > > > > > Any non-trivial change is nothing but pure risk, especially given the > > > low level of test coverage the code gets. > > > > Do you count coding style patches as non-trivial or trivial > > patches? > > > > > So I really only want to see the most critical obvious bug fixes > > > submitted for this layer and drivers. > > > > The cleanup changes I submitted should not change the actual behaviour. > > Replacing the register access macros with their equivalent outb/inb > > seems pretty obvious. What is your opinion on those? > > > > I see that the "ARCNET: whitespace, tab and codingstyle fixes" patch is > > pretty mixed up and not very reliable. But Joe has sent me a nice series > > for the cleanup. > > > > > And no I will not accept an argument stating that you have to > > > restructure and clean this code up in order to fix the bugs. That's > > > bogus. > > > > OK. > > > > I have the following patches in this series which fix bugs found during > > my development: > > > > com20020-pci: add dev_port for udev handling > > ARCNET: fix hard_header_len limit > > ARCNET: com20020: add enable and disable device on open/close > > > > I would send a new series containing only those patches with more > > detailed patch descriptions, if that's fine with you? > > I was away for a few days and while out I did another > restructuring of all the inb/outb/readb/writeb code > removing all the A macros. It removes all the > dependencies on ioaddr and names all the offsets with > new defines. > > I could post if you like. Sure, you should do that! I will rework my patches on top of them. > I think it's a lot cleaner and easier to read. OK! I would like to know what David things about the whole work. Thanks, Michael -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |