From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755149AbcIZXLv (ORCPT ); Mon, 26 Sep 2016 19:11:51 -0400 Received: from host.buserror.net ([209.198.135.123]:33182 "EHLO host.buserror.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753152AbcIZXLq (ORCPT ); Mon, 26 Sep 2016 19:11:46 -0400 Message-ID: <1474931495.4283.44.camel@buserror.net> From: Scott Wood To: Qiang Zhao Cc: "linuxppc-dev@lists.ozlabs.org" , "pku.leo@gmail.com" , "X.B. Xie" , "linux-kernel@vger.kernel.org" Date: Mon, 26 Sep 2016 18:11:35 -0500 In-Reply-To: References: <1474597232-33397-2-git-send-email-qiang.zhao@nxp.com> <20160925041411.GA16092@home.buserror.net> <20160925041906.GA16719@home.buserror.net> Organization: NXP Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.18.5.2-0ubuntu3 Mime-Version: 1.0 Content-Transfer-Encoding: 8bit X-SA-Exim-Connect-IP: 75.72.173.242 X-SA-Exim-Mail-From: oss@buserror.net X-Spam-Report: * -1.0 ALL_TRUSTED Passed through trusted hosts only via SMTP * -15 BAYES_00 BODY: Bayes spam probability is 0 to 1% * [score: 0.0000] Subject: Re: [v6,2/2] QE: remove PPCisms for QE X-SA-Exim-Version: 4.2.1 (built Mon, 26 Dec 2011 16:57:07 +0000) X-SA-Exim-Scanned: Yes (on host.buserror.net) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, 2016-09-26 at 01:46 +0000, Qiang Zhao wrote: > On Sun, Sep 25, 2016 at 12:19PM -0500, Scott Wood wrote: > > > > > -----Original Message----- > > From: Scott Wood [mailto:oss@buserror.net] > > Sent: Sunday, September 25, 2016 12:19 PM > > To: Qiang Zhao > > Cc: linuxppc-dev@lists.ozlabs.org; pku.leo@gmail.com; X.B. Xie > > ; linux-kernel@vger.kernel.org > > Subject: Re: [v6,2/2] QE: remove PPCisms for QE > > > > On Sat, Sep 24, 2016 at 11:14:11PM -0500, Scott Wood wrote: > > > > > > On Fri, Sep 23, 2016 at 10:20:32AM +0800, Zhao Qiang wrote: > > > > > > > > QE was supported on PowerPC, and dependent on PPC, Now it is > > > > supported on other platforms. so remove PPCisms. > > > > > > > > Signed-off-by: Zhao Qiang > > > > --- > > > > Changes for v2: > > > > - na > > > > Changes for v3: > > > > - add NO_IRQ > > > > Changes for v4: > > > > - modify spin_event_timeout to opencoded timeout loop > > > > - remove NO_IRQ > > > > - modify virq_to_hw to opencoed code Changes for v5: > > > > - modify commit msg > > > > - modify depends of QUICC_ENGINE > > > > - add kerneldoc header for qe_issue_cmd Changes for v6: > > > > - add dependency on FSL_SOC and PPC32 for drivers > > > >   depending on QUICC_ENGING but not available on ARM > > > > > > > >  drivers/irqchip/qe_ic.c                | 28 +++++++----- > > > >  drivers/net/ethernet/freescale/Kconfig | 10 ++--- > > > >  drivers/soc/fsl/qe/Kconfig             |  2 +- > > > >  drivers/soc/fsl/qe/qe.c                | 80 ++++++++++++++++++++----- > > > > --------- > > > >  drivers/soc/fsl/qe/qe_io.c             | 42 ++++++++---------- > > > >  drivers/soc/fsl/qe/qe_tdm.c            |  8 ++-- > > > >  drivers/soc/fsl/qe/ucc.c               | 10 ++--- > > > >  drivers/soc/fsl/qe/ucc_fast.c          | 68 ++++++++++++++----------- > > > > ---- > > > >  drivers/tty/serial/Kconfig             |  2 +- > > > >  drivers/usb/gadget/udc/Kconfig         |  2 +- > > > >  drivers/usb/host/Kconfig               |  2 +- > > > >  include/soc/fsl/qe/qe.h                |  1 - > > > >  include/soc/fsl/qe/qe_ic.h             | 12 ++--- > > > >  13 files changed, 141 insertions(+), 126 deletions(-) > > > I assume this means you'll be updating > > > http://patchwork.ozlabs.org/patch/654473/ > > > to be based on top of this... > > Apparently that assumption was wrong, since I now see that you're patching > > drivers/irqchip/qe_ic.c rather than drivers/soc/fsl/qe/qe_ic.c. > > Please keep the drivers/irqchip stuff separate and send to the appropriate > > maintainers. > > > You means separate drivers/irqchip/qe_ic.c part from this patch and send it > with the other qe_ic patches? > Is it acceptable if I modify qe_ic.c with drivers/soc/fsl/qe/qe_ic.c, then > send qe_ic patches to move qe_ic to drivers/irqchip? I'd recommend against it.  It would complicate getting the drivers/irqchip patchset merged.  In any case, it's too late for 4.9. -Scott