From: David Miller <davem@davemloft.net>
To: mkl@pengutronix.de
Cc: joe@perches.com, m.grzeschik@pengutronix.de,
netdev@vger.kernel.org, kernel@pengutronix.de,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH 00/21] ARCNET: Defibrillation
Date: Fri, 24 Apr 2015 23:58:53 -0400 (EDT) [thread overview]
Message-ID: <20150424.235853.1997287895150693202.davem@davemloft.net> (raw)
In-Reply-To: <553AB241.3080004@pengutronix.de>
From: Marc Kleine-Budde <mkl@pengutronix.de>
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.
So I really only want to see the most critical obvious bug fixes
submitted for this layer and drivers.
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.
I'm rejecting this patch series, sorry.
next prev parent reply other threads:[~2015-04-25 3:59 UTC|newest]
Thread overview: 33+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-04-24 17:20 Michael Grzeschik
2015-04-24 17:20 ` [PATCH 01/21] com20020-pci: add dev_port for udev handling Michael Grzeschik
2015-04-24 17:20 ` [PATCH 02/21] ARCNET: fix hard_header_len limit Michael Grzeschik
2015-04-24 17:20 ` [PATCH 03/21] ARCNET: capmode: fix transfer length Michael Grzeschik
2015-04-24 17:20 ` [PATCH 04/21] ARCNET: whitespace, tab and codingstyle fixes Michael Grzeschik
2015-04-24 17:20 ` [PATCH 05/21] ARCNET: remove unneeded macros Michael Grzeschik
2015-04-24 17:20 ` [PATCH 06/21] ARCNET: com20020: " Michael Grzeschik
2015-04-24 23:04 ` Joe Perches
2015-04-25 3:59 ` David Miller
2015-04-27 14:13 ` Michael Grzeschik
2015-04-24 17:20 ` [PATCH 07/21] ARCNET: rimi: " Michael Grzeschik
2015-04-24 17:20 ` [PATCH 08/21] ARCNET: com90io: " Michael Grzeschik
2015-04-24 17:20 ` [PATCH 09/21] ARCNET: com90xx: " Michael Grzeschik
2015-04-24 17:20 ` [PATCH 10/21] ARCNET: com20020: fix ioaddr prefixes Michael Grzeschik
2015-04-24 17:20 ` [PATCH 11/21] ARCNET: rimi: " Michael Grzeschik
2015-04-24 17:20 ` [PATCH 12/21] ARCNET: com90io: " Michael Grzeschik
2015-04-24 17:20 ` [PATCH 13/21] ARCNET: com90xx: " Michael Grzeschik
2015-04-24 17:20 ` [PATCH 14/21] ARCNET: arc-rawmode: reorder module functions Michael Grzeschik
2015-04-24 17:20 ` [PATCH 15/21] ARCNET: capmode: remove extra function and use C99 in struct Michael Grzeschik
2015-04-24 17:20 ` [PATCH 16/21] ARCNET: capmode: move dev_free_skb to its only user Michael Grzeschik
2015-04-24 17:20 ` [PATCH 17/21] ARCNET: com20020: replace magic numbers with readable macros Michael Grzeschik
2015-04-24 17:20 ` [PATCH 18/21] ARCNET: com20020: remove obsolete BUS_ALIGN offset factor Michael Grzeschik
2015-04-24 17:20 ` [PATCH 19/21] ARCNET: com20020: add enable and disable device on open/close Michael Grzeschik
2015-04-24 17:20 ` [PATCH 20/21] ARCNET: com20020-pci: reformat structs to C99 format Michael Grzeschik
2015-04-24 17:20 ` [PATCH 21/21] ARCNET: com20020-pci: add rotary index support Michael Grzeschik
2015-04-24 18:47 ` [PATCH 00/21] ARCNET: Defibrillation Joe Perches
2015-04-24 21:14 ` Marc Kleine-Budde
2015-04-25 3:58 ` David Miller [this message]
2015-04-25 4:52 ` Joe Perches
2015-04-27 14:57 ` Michael Grzeschik
2015-04-30 3:49 ` Joe Perches
2015-05-05 16:43 ` Michael Grzeschik
2015-05-05 17:02 ` Joe Perches
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20150424.235853.1997287895150693202.davem@davemloft.net \
--to=davem@davemloft.net \
--cc=joe@perches.com \
--cc=kernel@pengutronix.de \
--cc=linux-kernel@vger.kernel.org \
--cc=m.grzeschik@pengutronix.de \
--cc=mkl@pengutronix.de \
--cc=netdev@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®