[RFC, v1, 1/1] powerpc/85xx: Wakeup kexec smp slave cpus in second kernel
Yu Chen
chenyu105 at gmail.com
Sun Sep 1 23:21:01 EST 2013
thx, I think I've sent this patch using gmail web gui and got some
format problems, I'll try resend this patch tomorrow
2013/9/1 Wang Shilong <wangshilong1991 at gmail.com>:
> Hello, Using checkpatch.pl, i get the following warnings(errors):
> ERROR: patch seems to be corrupt (line wrapped?)
> #108: FILE: :418:
> mpc85xx_smp_flush_dcache_kexec(struct kimage *image)
>
> ERROR: do not set execute permissions for source files
> #197: FILE: arch/powerpc/kernel/head_fsl_booke.S
>
> ERROR: do not set execute permissions for source files
> #217: FILE: arch/powerpc/kernel/misc_32.S
>
> ERROR: do not set execute permissions for source files
> #316: FILE: arch/powerpc/platforms/85xx/smp.c
>
> WARNING: externs should be avoided in .c files
> #329: FILE: arch/powerpc/platforms/85xx/smp.c:148:
> +extern void reserve_kexec_bootmem(unsigned long poll_phy, int size);
>
> WARNING: please, no spaces at the start of a line
> #336: FILE: arch/powerpc/platforms/85xx/smp.c:155:
> + unsigned long kexec_poll_virt;$
>
> WARNING: please, no spaces at the start of a line
> #337: FILE: arch/powerpc/platforms/85xx/smp.c:156:
> + unsigned long *kexec_magic_virt;$
>
> WARNING: please, no spaces at the start of a line
> #339: FILE: arch/powerpc/platforms/85xx/smp.c:158:
> + if (!kexec_poll_phy ||$
>
> ERROR: code indent should use tabs where possible
> #340: FILE: arch/powerpc/platforms/85xx/smp.c:159:
> + kexec_poll_phy >= __max_low_memory)$
>
> WARNING: please, no spaces at the start of a line
> #340: FILE: arch/powerpc/platforms/85xx/smp.c:159:
> + kexec_poll_phy >= __max_low_memory)$
>
> ERROR: code indent should use tabs where possible
> #341: FILE: arch/powerpc/platforms/85xx/smp.c:160:
> + return;$
>
> WARNING: please, no spaces at the start of a line
> #341: FILE: arch/powerpc/platforms/85xx/smp.c:160:
> + return;$
>
> WARNING: please, no spaces at the start of a line
> #343: FILE: arch/powerpc/platforms/85xx/smp.c:162:
> + kexec_poll_virt = (unsigned long)phys_to_virt(kexec_poll_phy);$
>
> WARNING: please, no spaces at the start of a line
> #344: FILE: arch/powerpc/platforms/85xx/smp.c:163:
> + kexec_magic_virt = (unsigned long *)kexec_poll_virt;$
>
> WARNING: please, no spaces at the start of a line
> #346: FILE: arch/powerpc/platforms/85xx/smp.c:165:
> + if (*kexec_magic_virt == KEXEC_MAGIC)$
>
> ERROR: code indent should use tabs where possible
> #347: FILE: arch/powerpc/platforms/85xx/smp.c:166:
> + reserve_kexec_bootmem(kexec_poll_phy, KEXEC_RESERVE_LIMIT);$
>
> WARNING: please, no spaces at the start of a line
> #347: FILE: arch/powerpc/platforms/85xx/smp.c:166:
> + reserve_kexec_bootmem(kexec_poll_phy, KEXEC_RESERVE_LIMIT);$
>
> WARNING: please, no spaces at the start of a line
> #355: FILE: arch/powerpc/platforms/85xx/smp.c:174:
> + unsigned long kexec_poll_virt;$
>
> WARNING: please, no spaces at the start of a line
> #356: FILE: arch/powerpc/platforms/85xx/smp.c:175:
> + unsigned long *kexec_flag_virt;$
>
> WARNING: please, no spaces at the start of a line
> #357: FILE: arch/powerpc/platforms/85xx/smp.c:176:
> + unsigned long *kexec_magic_virt;$
>
> WARNING: please, no spaces at the start of a line
> #358: FILE: arch/powerpc/platforms/85xx/smp.c:177:
> + unsigned long *kexec_jump_virt;$
>
> WARNING: please, no spaces at the start of a line
> #361: FILE: arch/powerpc/platforms/85xx/smp.c:180:
> + if (!kexec_poll_phy ||$
>
> ERROR: code indent should use tabs where possible
> #362: FILE: arch/powerpc/platforms/85xx/smp.c:181:
> + kexec_poll_phy >= __max_low_memory)$
>
> WARNING: please, no spaces at the start of a line
> #362: FILE: arch/powerpc/platforms/85xx/smp.c:181:
> + kexec_poll_phy >= __max_low_memory)$
>
> ERROR: code indent should use tabs where possible
> #363: FILE: arch/powerpc/platforms/85xx/smp.c:182:
> + return -EBUSY;$
>
> WARNING: please, no spaces at the start of a line
> #363: FILE: arch/powerpc/platforms/85xx/smp.c:182:
> + return -EBUSY;$
>
> WARNING: please, no spaces at the start of a line
> #365: FILE: arch/powerpc/platforms/85xx/smp.c:184:
> + kexec_poll_virt = (unsigned long)phys_to_virt(kexec_poll_phy);$
>
> WARNING: please, no spaces at the start of a line
> #367: FILE: arch/powerpc/platforms/85xx/smp.c:186:
> + kexec_magic_virt = (unsigned long *)kexec_poll_virt;$
>
> WARNING: please, no spaces at the start of a line
> #368: FILE: arch/powerpc/platforms/85xx/smp.c:187:
> + kexec_flag_virt = (unsigned long *)kexec_poll_virt + 1;$
>
> WARNING: please, no spaces at the start of a line
> #369: FILE: arch/powerpc/platforms/85xx/smp.c:188:
> + kexec_jump_virt = (unsigned long *)kexec_poll_virt + 2;$
>
> WARNING: please, no spaces at the start of a line
> #372: FILE: arch/powerpc/platforms/85xx/smp.c:191:
> + if (*kexec_magic_virt == KEXEC_MAGIC) {$
>
> ERROR: code indent should use tabs where possible
> #373: FILE: arch/powerpc/platforms/85xx/smp.c:192:
> + flush_dcache_range((ulong)kexec_poll_virt,$
>
> WARNING: please, no spaces at the start of a line
> #373: FILE: arch/powerpc/platforms/85xx/smp.c:192:
> + flush_dcache_range((ulong)kexec_poll_virt,$
>
> ERROR: code indent should use tabs where possible
> #374: FILE: arch/powerpc/platforms/85xx/smp.c:193:
> + (ulong)kexec_poll_virt + L1_CACHE_BYTES-1);$
>
> WARNING: please, no spaces at the start of a line
> #374: FILE: arch/powerpc/platforms/85xx/smp.c:193:
> + (ulong)kexec_poll_virt + L1_CACHE_BYTES-1);$
>
> ERROR: code indent should use tabs where possible
> #375: FILE: arch/powerpc/platforms/85xx/smp.c:194:
> + *kexec_jump_virt = (unsigned long)__early_start;$
>
> WARNING: please, no spaces at the start of a line
> #375: FILE: arch/powerpc/platforms/85xx/smp.c:194:
> + *kexec_jump_virt = (unsigned long)__early_start;$
>
> ERROR: code indent should use tabs where possible
> #376: FILE: arch/powerpc/platforms/85xx/smp.c:195:
> + mb();$
>
> WARNING: please, no spaces at the start of a line
> #376: FILE: arch/powerpc/platforms/85xx/smp.c:195:
> + mb();$
>
> ERROR: code indent should use tabs where possible
> #377: FILE: arch/powerpc/platforms/85xx/smp.c:196:
> + /*kick cpu[nr] up*/$
>
> ERROR: code indent should use tabs where possible
> #378: FILE: arch/powerpc/platforms/85xx/smp.c:197:
> + *kexec_flag_virt = nr;$
>
> WARNING: please, no spaces at the start of a line
> #378: FILE: arch/powerpc/platforms/85xx/smp.c:197:
> + *kexec_flag_virt = nr;$
>
> ERROR: code indent should use tabs where possible
> #379: FILE: arch/powerpc/platforms/85xx/smp.c:198:
> + mb();$
>
> WARNING: please, no spaces at the start of a line
> #379: FILE: arch/powerpc/platforms/85xx/smp.c:198:
> + mb();$
>
> ERROR: code indent should use tabs where possible
> #380: FILE: arch/powerpc/platforms/85xx/smp.c:199:
> + flush_dcache_range((ulong)kexec_poll_virt,$
>
> WARNING: please, no spaces at the start of a line
> #380: FILE: arch/powerpc/platforms/85xx/smp.c:199:
> + flush_dcache_range((ulong)kexec_poll_virt,$
>
> ERROR: code indent should use tabs where possible
> #381: FILE: arch/powerpc/platforms/85xx/smp.c:200:
> + (ulong)kexec_poll_virt + L1_CACHE_BYTES-1);$
>
> WARNING: please, no spaces at the start of a line
> #381: FILE: arch/powerpc/platforms/85xx/smp.c:200:
> + (ulong)kexec_poll_virt + L1_CACHE_BYTES-1);$
>
> ERROR: code indent should use tabs where possible
> #383: FILE: arch/powerpc/platforms/85xx/smp.c:202:
> + return 0;$
>
> WARNING: please, no spaces at the start of a line
> #383: FILE: arch/powerpc/platforms/85xx/smp.c:202:
> + return 0;$
>
> WARNING: please, no spaces at the start of a line
> #384: FILE: arch/powerpc/platforms/85xx/smp.c:203:
> + }$
>
> WARNING: please, no spaces at the start of a line
> #385: FILE: arch/powerpc/platforms/85xx/smp.c:204:
> + return -EBUSY;$
>
> WARNING: please, no spaces at the start of a line
> #396: FILE: arch/powerpc/platforms/85xx/smp.c:249:
> + if (!mpc85xx_smp_kick_kexec_cpus(nr))$
>
> ERROR: code indent should use tabs where possible
> #397: FILE: arch/powerpc/platforms/85xx/smp.c:250:
> + goto kexec_kick_done;$
>
> WARNING: please, no spaces at the start of a line
> #397: FILE: arch/powerpc/platforms/85xx/smp.c:250:
> + goto kexec_kick_done;$
>
> WARNING: externs should be avoided in .c files
> #423: FILE: arch/powerpc/platforms/85xx/smp.c:345:
> +extern const unsigned int relocate_smp_cpu_size;
>
> ERROR: code indent should use tabs where possible
> #434: FILE: arch/powerpc/platforms/85xx/smp.c:353:
> + mb();$
>
> WARNING: please, no spaces at the start of a line
> #434: FILE: arch/powerpc/platforms/85xx/smp.c:353:
> + mb();$
>
> ERROR: code indent should use tabs where possible
> #436: FILE: arch/powerpc/platforms/85xx/smp.c:355:
> + if (crash_shutdown) {$
>
> WARNING: please, no spaces at the start of a line
> #436: FILE: arch/powerpc/platforms/85xx/smp.c:355:
> + if (crash_shutdown) {$
>
> WARNING: suspect code indent for conditional statements (8, 12)
> #436: FILE: arch/powerpc/platforms/85xx/smp.c:355:
> + if (crash_shutdown) {
> + /* loop forever */
>
> ERROR: code indent should use tabs where possible
> #437: FILE: arch/powerpc/platforms/85xx/smp.c:356:
> + /* loop forever */$
>
> ERROR: code indent should use tabs where possible
> #438: FILE: arch/powerpc/platforms/85xx/smp.c:357:
> + while (1)$
>
> WARNING: please, no spaces at the start of a line
> #438: FILE: arch/powerpc/platforms/85xx/smp.c:357:
> + while (1)$
>
> ERROR: code indent should use tabs where possible
> #439: FILE: arch/powerpc/platforms/85xx/smp.c:358:
> + ;$
>
> WARNING: please, no spaces at the start of a line
> #439: FILE: arch/powerpc/platforms/85xx/smp.c:358:
> + ;$
>
> ERROR: code indent should use tabs where possible
> #440: FILE: arch/powerpc/platforms/85xx/smp.c:359:
> + } else {$
>
> WARNING: please, no spaces at the start of a line
> #440: FILE: arch/powerpc/platforms/85xx/smp.c:359:
> + } else {$
>
> ERROR: code indent should use tabs where possible
> #441: FILE: arch/powerpc/platforms/85xx/smp.c:360:
> + while (!atomic_read(&kexec_ready_to_reboot))$
>
> WARNING: please, no spaces at the start of a line
> #441: FILE: arch/powerpc/platforms/85xx/smp.c:360:
> + while (!atomic_read(&kexec_ready_to_reboot))$
>
> ERROR: code indent should use tabs where possible
> #442: FILE: arch/powerpc/platforms/85xx/smp.c:361:
> + cpu_relax();$
>
> WARNING: please, no spaces at the start of a line
> #442: FILE: arch/powerpc/platforms/85xx/smp.c:361:
> + cpu_relax();$
>
> ERROR: code indent should use tabs where possible
> #443: FILE: arch/powerpc/platforms/85xx/smp.c:362:
> + /*flush destination*/$
>
> ERROR: code indent should use tabs where possible
> #444: FILE: arch/powerpc/platforms/85xx/smp.c:363:
> + if (save_image)$
>
> WARNING: please, no spaces at the start of a line
> #444: FILE: arch/powerpc/platforms/85xx/smp.c:363:
> + if (save_image)$
>
> ERROR: code indent should use tabs where possible
> #445: FILE: arch/powerpc/platforms/85xx/smp.c:364:
> + mpc85xx_smp_flush_dcache_kexec(save_image, 1);$
>
> WARNING: please, no spaces at the start of a line
> #445: FILE: arch/powerpc/platforms/85xx/smp.c:364:
> + mpc85xx_smp_flush_dcache_kexec(save_image, 1);$
>
> ERROR: code indent should use tabs where possible
> #447: FILE: arch/powerpc/platforms/85xx/smp.c:366:
> + flush_icache_range(wait_code_buffer,$
>
> WARNING: please, no spaces at the start of a line
> #447: FILE: arch/powerpc/platforms/85xx/smp.c:366:
> + flush_icache_range(wait_code_buffer,$
>
> ERROR: code indent should use tabs where possible
> #448: FILE: arch/powerpc/platforms/85xx/smp.c:367:
> + wait_code_buffer + relocate_smp_cpu_size);$
>
> WARNING: please, no spaces at the start of a line
> #448: FILE: arch/powerpc/platforms/85xx/smp.c:367:
> + wait_code_buffer + relocate_smp_cpu_size);$
>
> ERROR: code indent should use tabs where possible
> #449: FILE: arch/powerpc/platforms/85xx/smp.c:368:
> + flush_dcache_range(wait_code_buffer,$
>
> WARNING: please, no spaces at the start of a line
> #449: FILE: arch/powerpc/platforms/85xx/smp.c:368:
> + flush_dcache_range(wait_code_buffer,$
>
> ERROR: code indent should use tabs where possible
> #450: FILE: arch/powerpc/platforms/85xx/smp.c:369:
> + wait_code_buffer + relocate_smp_cpu_size);$
>
> WARNING: please, no spaces at the start of a line
> #450: FILE: arch/powerpc/platforms/85xx/smp.c:369:
> + wait_code_buffer + relocate_smp_cpu_size);$
>
> ERROR: code indent should use tabs where possible
> #452: FILE: arch/powerpc/platforms/85xx/smp.c:371:
> + atomic_inc(&kexec_slave_finish);$
>
> WARNING: please, no spaces at the start of a line
> #452: FILE: arch/powerpc/platforms/85xx/smp.c:371:
> + atomic_inc(&kexec_slave_finish);$
>
> ERROR: code indent should use tabs where possible
> #454: FILE: arch/powerpc/platforms/85xx/smp.c:373:
> + ((void (*)(void)) wait_code_buffer)();$
>
> WARNING: please, no spaces at the start of a line
> #454: FILE: arch/powerpc/platforms/85xx/smp.c:373:
> + ((void (*)(void)) wait_code_buffer)();$
>
> ERROR: code indent should use tabs where possible
> #455: FILE: arch/powerpc/platforms/85xx/smp.c:374:
> + /* NOTREACHED */$
>
> ERROR: code indent should use tabs where possible
> #456: FILE: arch/powerpc/platforms/85xx/smp.c:375:
> + }$
>
> WARNING: please, no spaces at the start of a line
> #456: FILE: arch/powerpc/platforms/85xx/smp.c:375:
> + }$
>
> total: 39 errors, 53 warnings, 352 lines checked
>
> NOTE: whitespace errors detected, you may wish to use scripts/cleanpatch or
> scripts/cleanfile
>
> patch has style problems, please review. If any of these errors
> are false positives report them to the maintainer, see
> CHECKPATCH in MAINTAINERS.
>
> Thanks, Wang
>
> Notic: this is an automated generated by shell script
> Any problems please contact: wangshilong1991 at gmail.com
--
Yu Chen
More information about the Linuxppc-dev
mailing list