Re: [PATCH V4 2/6] MIPS: Move Loongson GPIO driver to drivers/gpio

["陈华才" <chenhc@xxxxxxxxxx>]

Hi, Ralf and Alexandre,

So, I need three patches: clean up, move to drivers/gpio, and improve it to support Loonson-3A/3B?

Huacai
 
 
------------------ Original ------------------
From:  "Ralf Baechle"<ralf@xxxxxxxxxxxxxx>;
Date:  Wed, Nov 19, 2014 06:11 PM
To:  "Alexandre Courbot"<gnurou@xxxxxxxxx>;
Cc:  "Huacai Chen"<chenhc@xxxxxxxxxx>; "John Crispin"<john@xxxxxxxxxxx>; "Steven J. Hill"<Steven.Hill@xxxxxxxxxx>; "linux-mips"<linux-mips@xxxxxxxxxxxxxx>; "Fuxin Zhang"<zhangfx@xxxxxxxxxx>; "wuzhangjin"<wuzhangjin@xxxxxxxxx>; "linux-gpio@xxxxxxxxxxxxxxx"<linux-gpio@xxxxxxxxxxxxxxx>;
Subject:  Re: [PATCH V4 2/6] MIPS: Move Loongson GPIO driver to drivers/gpio
 
On Wed, Nov 19, 2014 at 05:21:56PM +0900, Alexandre Courbot wrote:

> > On Fri, Nov 14, 2014 at 12:12 AM, Huacai Chen <chenhc@xxxxxxxxxx> wrote:
> >> Move Loongson-2's GPIO driver to drivers/gpio and add Kconfig options.
> >
> > Acked-by: Alexandre Courbot <acourbot@xxxxxxxxxx>
> >
> > Guess this should go through the GPIO tree once the platform
> > maintainers have acked this?
> 
> Ouch. After looking at this driver's implementation I think I have to
> take my Ack back. This driver comes with custom definitions of
> gpio_get_value() and other functions, which we will want to get rid of
> before moving this into drivers/gpio. Can you port this to a proper
> gpiolib driver before doing the move?

As the arch maintainer I'm happy to say farewell to driver code in arch/mips.

Also, can we split the whole procedure into two patches, one for the cleanup
and one for the move?  I don't care in which order.

  Ralf��.n��������+%������w��{.n�����{��
b���ܨ}���Ơz�j:+v�����w����ޙ��&�)ߡ�a����z�ޗ���ݢj��w�f