Thread (4 messages) 4 messages, 3 authors, 2015-11-30

Re: [PATCH 0/4] gpio: lpc32xx: add new LPC32xx GPIO controller driver

From: Vladimir Zapolskiy <vz@mleia.com>
Date: 2015-11-30 09:13:17
Also in: linux-arm-kernel, linux-devicetree

Hi Linus,

On 30.11.2015 10:54, Linus Walleij wrote:
On Fri, Nov 20, 2015 at 2:29 AM, Vladimir Zapolskiy [off-list ref] wrote:
quoted
Unfortunately the old driver has too many hacks inherited from the
ancient legacy NXP BSP and I find it can not be maintained, the new
driver has compatible interface to device tree clients, so moving
to the new driver should not impact any GPIO consumers.
Has it been tested on the old hardware too?
the hardware here is the same, the driver is different.

With the existing LPC32xx DTS files I tested that the DTS GPIO client's
interface is unchanged and that GPIO client drivers correctly work with
the new GPIO controller driver.

As a summary about the change,
* DT binding of GPIO controller in lpc32xx.dtsi is extended (no removed
properties),
* client's DT binding to the GPIO controller are untouched,
* GPIO controller code is completely replaced,
* GPIO client's side is untouched.

As a reminder about the necessity of the change, the legacy driver
operates with hardware IRQs, this prevents switching of the platfrom to
SPARSE_IRQ model.

It is important that the change depends on IRQ chip driver, which is
also under review.

I would appreciate, if review of this change can be started, so that
both IRQ chip driver and GPIO controller driver are accepted in one
merge window.

--
With best wishes,
Vladimir
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help