linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
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

      reply	other threads:[~2015-06-01 22:07 UTC|newest]

Thread overview: 4+ 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
2015-01-30  5:22 ` [v4] " Scott Wood
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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).