2015-06-01 04:24:18

by Zhao Qiang

[permalink] [raw]
Subject: RE: [v4] QE: Move QE from arch/powerpc to drivers/soc

On Fri, Jan 30, 2015 at 1:22PM +0800, Wood Scott wrote:

> -----Original Message-----
> From: Wood Scott-B07421
> Sent: Friday, January 30, 2015 1:22 PM
> To: Zhao Qiang-B45475
> Cc: [email protected]; [email protected]; linuxppc-
> [email protected]; Wood Scott-B07421; Xie Xiaobo-R63061
> Subject: Re: [v4] QE: Move QE from arch/powerpc to drivers/soc
>
> On Wed, Nov 12, 2014 at 11:40:13AM +0800, Zhao Qiang wrote:
> > ls1 has qe and ls1 has arm cpu.
> > move qe from arch/powerpc to drivers/soc/fsl to adapt to powerpc and
> > arm
> >
> > Signed-off-by: Zhao Qiang <[email protected]>
> > ---
> > Changes for v2:
> > - move code to driver/soc
> > Changes for v3:
> > - change drivers/soc/qe to drivers/soc/fsl-qe Changes for v4:
> > - move drivers/soc/fsl-qe to drivers/soc/fsl/qe
> > - move head files for qe from include/linux/fsl to include/soc/fsl
> > - move qe_ic.c to drivers/irqchip/
>
> Need MAINTAINERS update for drivers/soc/fsl/qe, as previously discussed.

You mean, I need to finish the following work before move qe to public directory?

1. gpio.c -> needs to be converted to GPIO framework and placed in drivers/gpio
2. qe_ic* should probably move into drivers/irqchip
3. qe_io.c should be converted over to pinmux and put in drivers/pinctrl
4. Some of the clock could should be looked to be converted to use the clk framework

>
> -Scott


Best Regards
Zhao Qiang


2015-06-01 22:07:10

by Scott Wood

[permalink] [raw]
Subject: Re: [v4] QE: Move QE from arch/powerpc to drivers/soc

On Sun, 2015-05-31 at 23:09 -0500, Zhao Qiang-B45475 wrote:
> On Fri, Jan 30, 2015 at 1:22PM +0800, Wood Scott wrote:
>
> > -----Original Message-----
> > From: Wood Scott-B07421
> > Sent: Friday, January 30, 2015 1:22 PM
> > To: Zhao Qiang-B45475
> > Cc: [email protected]; [email protected]; linuxppc-
> > [email protected]; Wood Scott-B07421; Xie Xiaobo-R63061
> > Subject: Re: [v4] QE: Move QE from arch/powerpc to drivers/soc
> >
> > On Wed, Nov 12, 2014 at 11:40:13AM +0800, Zhao Qiang wrote:
> > > ls1 has qe and ls1 has arm cpu.
> > > move qe from arch/powerpc to drivers/soc/fsl to adapt to powerpc
> > > and
> > > arm
> > >
> > > Signed-off-by: Zhao Qiang <[email protected]>
> > > ---
> > > Changes for v2:
> > > - move code to driver/soc
> > > Changes for v3:
> > > - change drivers/soc/qe to drivers/soc/fsl-qe Changes for v4:
> > > - move drivers/soc/fsl-qe to drivers/soc/fsl/qe
> > > - move head files for qe from include/linux/fsl to
> > > include/soc/fsl
> > > - move qe_ic.c to drivers/irqchip/
> >
> > Need MAINTAINERS update for drivers/soc/fsl/qe, as previously
> > discussed.
>
> You mean, I need to finish the following work before move qe to
> public directory?
>
> 1. gpio.c -> needs to be converted to GPIO framework and placed in
> drivers/gpio
> 2. qe_ic* should probably move into drivers/irqchip
> 3. qe_io.c should be converted over to pinmux and put in
> drivers/pinctrl
> 4. Some of the clock could should be looked to be converted to use
> the clk framework

Those would be nice but at the very least:
1. Don't introduce code into the toplevel /lib that duplicates
existing functionality, and
2. Add a MAINTAINERS entry as described above

-Scott