[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-id: <11380780.Ccg1NEX8q5@amdc3058>
Date: Fri, 17 Mar 2017 15:08:56 +0100
From: Bartlomiej Zolnierkiewicz <b.zolnierkie@...sung.com>
To: Tejun Heo <tj@...nel.org>
Cc: Russell King <rmk+kernel@...linux.org.uk>,
Sergei Shtylyov <sergei.shtylyov@...entembedded.com>,
Sekhar Nori <nsekhar@...com>, Arnd Bergmann <arnd@...db.de>,
Dmitry Eremin-Solenikov <dbaryshkov@...il.com>,
Kevin Hilman <khilman@...libre.com>,
Olof Johansson <olof@...om.net>, linux-ide@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] pata_pcmcia: add EBSA110's PCMCIA slot support
On Friday, March 17, 2017 09:39:00 AM Tejun Heo wrote:
> Hello,
>
> On Fri, Mar 17, 2017 at 12:59:56PM +0100, Bartlomiej Zolnierkiewicz wrote:
> > There is no support for this device in the upstream ide driver but
> > Russell has a hacky patch to make it work by redefining inb()/outb()
> > operations globally for the whole ide subsystem, please see:
> >
> > https://www.spinics.net/lists/arm-kernel/msg567454.html
>
> Well, that mail says that the hardware is a museum piece. What's the
> point here? This can't even be tested, can it?
To be honest, the main point here is to make Russell happy. ;-)
This is the second time he brought EBSA110 issue while talking about
conversion of ARM platforms to use libata PATA so I've finally decided
to do the patch just to have a reference example of how to deal with
this and similar devices in a proper way.
Even though the patch can't be tested EBSA110 is still a supported ARM
platform and the patch should work fine in theory (+ it cannot break
anything). OTOH it adds an extra code which has some cost from the
long-term maintenance POV. Anyway, it is up to you to decide what to
do with it. :-)
Best regards,
--
Bartlomiej Zolnierkiewicz
Samsung R&D Institute Poland
Samsung Electronics
Powered by blists - more mailing lists