From: Scott Wood <scottwood@freescale.com>
To: Zhao Qiang-B45475 <qiang.zhao@freescale.com>
Cc: "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"netdev@vger.kernel.org" <netdev@vger.kernel.org>,
"linuxppc-dev@lists.ozlabs.org" <linuxppc-dev@lists.ozlabs.org>,
"Xie Xiaobo-R63061" <X.Xie@freescale.com>
Subject: Re: [v4] QE: Move QE from arch/powerpc to drivers/soc
Date: Mon, 1 Jun 2015 17:06:44 -0500 [thread overview]
Message-ID: <1433196404.4089.187.camel@freescale.com> (raw)
In-Reply-To: <SN1PR0301MB1550F0744C4F5B84D6B6DA9D9BB60@SN1PR0301MB1550.namprd03.prod.outlook.com>
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: linux-kernel@vger.kernel.org; netdev@vger.kernel.org; linuxppc-
> > dev@lists.ozlabs.org; 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 <B45475@freescale.com>
> > > ---
> > > 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
prev parent reply other threads:[~2015-06-01 22:07 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-11-12 3:40 [PATCH v4] QE: Move QE from arch/powerpc to drivers/soc Zhao Qiang
2014-11-12 3:40 ` Zhao Qiang
2015-01-30 5:22 ` [v4] " Scott Wood
2015-01-30 5:22 ` Scott Wood
2015-06-01 4:09 ` Zhao Qiang
2015-06-01 4:09 ` Zhao Qiang
2015-06-01 22:06 ` Scott Wood [this message]
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=1433196404.4089.187.camel@freescale.com \
--to=scottwood@freescale.com \
--cc=X.Xie@freescale.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linuxppc-dev@lists.ozlabs.org \
--cc=netdev@vger.kernel.org \
--cc=qiang.zhao@freescale.com \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.