Le 11/05/2017 à 19:14, christophe leroy a écrit :
Le 11/05/2017 à 17:15, Nicholas Piggin a écrit :Cc: Scott Wood <o...@buserror.net> Cc: Christophe Leroy <christophe.le...@c-s.fr> Signed-off-by: Nicholas Piggin <npig...@gmail.com> --- arch/powerpc/xmon/xmon.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/arch/powerpc/xmon/xmon.c b/arch/powerpc/xmon/xmon.c index f11f65634aab..ec420b0e6e88 100644 --- a/arch/powerpc/xmon/xmon.c +++ b/arch/powerpc/xmon/xmon.c @@ -1242,10 +1242,13 @@ bpt_cmds(void) { int cmd; unsigned long a; - int mode, i; + int i; struct bpt *bp; +#ifndef CONFIG_8xxWould be better to use CONFIG_PPC_8xx As stated in arch/powerpc/platform/Kconfig.cputype, CONFIG_8xx is temp to handle compat with arch/ppc, and we are trying to get rid of it
I had the same comment in https://patchwork.ozlabs.org/patch/700354/ I also suggested to move the relevant declarations inside the switch() Christophe
Christophe+ int mode; const char badaddr[] = "Only kernel addresses are permitted " "for breakpoints\n"; +#endif cmd = inchar(); switch (cmd) {--- L'absence de virus dans ce courrier électronique a été vérifiée par le logiciel antivirus Avast. https://www.avast.com/antivirus
--- L'absence de virus dans ce courrier électronique a été vérifiée par le logiciel antivirus Avast. https://www.avast.com/antivirus
