Qiang Zhao <[email protected]> writes: > On Mon 8/7/2017 3:02 PM, Michael Ellerman <[email protected]> wrote: > >> -----Original Message----- >> From: Michael Ellerman [mailto:[email protected]] >> Sent: Monday, August 07, 2017 3:02 PM >> To: Qiang Zhao <[email protected]>; [email protected] >> Cc: [email protected]; Qiang Zhao <[email protected]>; linuxppc- >> [email protected]; Xiaobo Xie <[email protected]>; linux- >> [email protected] >> Subject: Re: [PATCH v10 4/4] irqchip/qeic: remove PPCisms for QEIC >> >> Zhao Qiang <[email protected]> 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 <[email protected]> >> > --- >> > 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
