linux-kernel - Re: [v6,2/2] QE: remove PPCisms for QE

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20160925041906.GA16719@home.buserror.net>
Date:   Sat, 24 Sep 2016 23:19:06 -0500
From:   Scott Wood <oss@...error.net>
To:     Zhao Qiang <qiang.zhao@....com>
Cc:     linuxppc-dev@...ts.ozlabs.org, pku.leo@...il.com,
        xiaobo.xie@....com, linux-kernel@...r.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 <qiang.zhao@....com>
> > ---
> > 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.

-Scott