linux-kernel - RE: [PATCH v10 4/4] irqchip/qeic: remove PPCisms for QEIC

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]

Date:   Wed, 9 Aug 2017 02:35:51 +0000
From:   Qiang Zhao <qiang.zhao@....com>
To:     Michael Ellerman <mpe@...erman.id.au>,
        "tglx@...utronix.de" <tglx@...utronix.de>
CC:     "oss@...error.net" <oss@...error.net>,
        "linuxppc-dev@...ts.ozlabs.org" <linuxppc-dev@...ts.ozlabs.org>,
        Xiaobo Xie <xiaobo.xie@....com>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: RE: [PATCH v10 4/4] irqchip/qeic: remove PPCisms for QEIC

On Tue 8/8/2017 6:05 PM, Michael Ellerman <mpe@...erman.id.au> wrote:

> -----Original Message-----
> From: Michael Ellerman [mailto:mpe@...erman.id.au]
> Sent: Tuesday, August 08, 2017 6:05 PM
> To: Qiang Zhao <qiang.zhao@....com>; tglx@...utronix.de
> Cc: oss@...error.net; linuxppc-dev@...ts.ozlabs.org; Xiaobo Xie
> <xiaobo.xie@....com>; linux-kernel@...r.kernel.org
> Subject: RE: [PATCH v10 4/4] irqchip/qeic: remove PPCisms for QEIC
> 
> Qiang Zhao <qiang.zhao@....com> writes:
> 
> > On Mon 8/7/2017 3:02 PM, Michael Ellerman <mpe@...erman.id.au> wrote:
> >
> >> -----Original Message-----
> >> From: Michael Ellerman [mailto:mpe@...erman.id.au]
> >> Sent: Monday, August 07, 2017 3:02 PM
> >> To: Qiang Zhao <qiang.zhao@....com>; tglx@...utronix.de
> >> Cc: oss@...error.net; Qiang Zhao <qiang.zhao@....com>; linuxppc-
> >> dev@...ts.ozlabs.org; Xiaobo Xie <xiaobo.xie@....com>; linux-
> >> kernel@...r.kernel.org
> >> Subject: Re: [PATCH v10 4/4] irqchip/qeic: remove PPCisms for QEIC
> >>
> >> Zhao Qiang <qiang.zhao@....com> writes:
> >>
> >> > QEIC 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>
> >> > ---
> >> >  arch/powerpc/platforms/83xx/km83xx.c          |   1 -
> >> >  arch/powerpc/platforms/83xx/misc.c            |   1 -
> >> >  arch/powerpc/platforms/83xx/mpc832x_mds.c     |   1 -
> >> >  arch/powerpc/platforms/83xx/mpc832x_rdb.c     |   1 -
> >> >  arch/powerpc/platforms/83xx/mpc836x_mds.c     |   1 -
> >> >  arch/powerpc/platforms/83xx/mpc836x_rdk.c     |   1 -
> >> >  arch/powerpc/platforms/85xx/corenet_generic.c |   1 -
> >> >  arch/powerpc/platforms/85xx/mpc85xx_mds.c     |   1 -
> >> >  arch/powerpc/platforms/85xx/mpc85xx_rdb.c     |   1 -
> >> >  arch/powerpc/platforms/85xx/twr_p102x.c       |   1 -
> >> >  drivers/irqchip/irq-qeic.c                    | 188 +++++++++++---------------
> >> >  include/soc/fsl/qe/qe_ic.h                    | 132 ------------------
> >> >  12 files changed, 80 insertions(+), 250 deletions(-)  delete mode
> >> > 100644 include/soc/fsl/qe/qe_ic.h
> >> >
> >> > diff --git a/arch/powerpc/platforms/83xx/km83xx.c
> >> > b/arch/powerpc/platforms/83xx/km83xx.c
> >> > index d8642a4..b1cef0a 100644
> >> > --- a/arch/powerpc/platforms/83xx/km83xx.c
> >> > +++ b/arch/powerpc/platforms/83xx/km83xx.c
> >> > @@ -38,7 +38,6 @@
> >> >  #include <sysdev/fsl_soc.h>
> >> >  #include <sysdev/fsl_pci.h>
> >> >  #include <soc/fsl/qe/qe.h>
> >> > -#include <soc/fsl/qe/qe_ic.h>
> >>
> >> You deleted that file in patch 2. So didn't you just break the build
> >> for the last two commits?
> >
> > Sorry, I am not sure what you said. Could you explain?
> 
> Don't worry about it. I was confused by the fact that we have both:
> 
>   drivers/soc/fsl/qe/qe_ic.h
> 
> and:
> 
>   include/soc/fsl/qe/qe_ic.h
> 
> cheers

I think this is a issue left over by history.
In patch with commit id 7aa1aa6ecec2af19d9aa85430ce3e56119e21626, I just move them out from arch/powerpc.  
Maybe need to ask the original author why there are 2 qe_ic.h.

Best Regards
Qiang Zhao