[kbuild-all] [android-goldfish:android-3.18 442/460] drivers/mmc/host/sdhci-of-esdhc.c:361:11: error: dereferencing pointer to incomplete type 'struct sdhci_esdhc'

tree: https://android.googlesource.com/kernel/goldfish android-3.18 head: 03c65f790cc49ff38432f8c39f3cd1320fcb1f2a commit: dc29f368658a96818bc9c602b0f160e603a0d4a6 [442/460] mmc: sdhci-of-esdhc: add/remove some quirks according to vendor version config: powerpc64-randconfig-p002-20210213 (attached as .config) compiler: powerpc64-linux-gcc (GCC) 5.5.0 reproduce (this is a W=1 build): wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross git remote add android-goldfish https://android.googlesource.com/kernel/goldfish git fetch --no-tags android-goldfish android-3.18 git checkout dc29f368658a96818bc9c602b0f160e603a0d4a6 # save the attached .config to linux build tree COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-5.5.0 make.cross ARCH=powerpc64 If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <lkp(a)intel.com&gt; All errors (new ones prefixed by >>): In file included from arch/powerpc/kernel/cputable.c:14:0: const struct sched_group_energy * const(*sched_domain_energy_f)(int cpu); ^ arch/powerpc/kernel/cputable.c: In function 'identify_cpu': arch/powerpc/kernel/cputable.c:2166:16: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare] for (i = 0; i < ARRAY_SIZE(cpu_specs); i++,s++) { ^ cc1: all warnings being treated as errors -- In file included from arch/powerpc/kernel/ptrace.c:19:0: const struct sched_group_energy * const(*sched_domain_energy_f)(int cpu); ^ In file included from include/linux/bug.h:4:0, from include/linux/thread_info.h:11, from include/asm-generic/preempt.h:4, from arch/powerpc/include/generated/asm/preempt.h:1, from include/linux/preempt.h:18, from include/linux/spinlock.h:50, from include/linux/seqlock.h:35, from include/linux/time.h:5, from include/uapi/linux/timex.h:56, from include/linux/timex.h:56, from include/linux/sched.h:19, from arch/powerpc/kernel/ptrace.c:19: include/linux/regset.h: In function 'user_regset_copyout': include/linux/regset.h:227:14: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare] BUG_ON(*pos < start_pos); ^ arch/powerpc/include/asm/bug.h:75:27: note: in definition of macro 'BUG_ON' if (__builtin_constant_p(x)) { \ ^ include/linux/regset.h:227:14: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare] BUG_ON(*pos < start_pos); ^ arch/powerpc/include/asm/bug.h:76:7: note: in definition of macro 'BUG_ON' if (x) \ ^ include/linux/regset.h:227:14: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare] BUG_ON(*pos < start_pos); ^ arch/powerpc/include/asm/bug.h:84:25: note: in definition of macro 'BUG_ON' "r" ((__force long)(x))); \ ^ In file included from arch/powerpc/kernel/ptrace.c:24:0: include/linux/regset.h:228:26: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare] if (end_pos < 0 || *pos < end_pos) { ^ In file included from include/linux/bug.h:4:0, from include/linux/thread_info.h:11, from include/asm-generic/preempt.h:4, from arch/powerpc/include/generated/asm/preempt.h:1, from include/linux/preempt.h:18, from include/linux/spinlock.h:50, from include/linux/seqlock.h:35, from include/linux/time.h:5, from include/uapi/linux/timex.h:56, from include/linux/timex.h:56, from include/linux/sched.h:19, from arch/powerpc/kernel/ptrace.c:19: include/linux/regset.h: In function 'user_regset_copyin': include/linux/regset.h:252:14: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare] BUG_ON(*pos < start_pos); ^ arch/powerpc/include/asm/bug.h:75:27: note: in definition of macro 'BUG_ON' if (__builtin_constant_p(x)) { \ ^ include/linux/regset.h:252:14: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare] BUG_ON(*pos < start_pos); ^ arch/powerpc/include/asm/bug.h:76:7: note: in definition of macro 'BUG_ON' if (x) \ ^ include/linux/regset.h:252:14: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare] BUG_ON(*pos < start_pos); ^ arch/powerpc/include/asm/bug.h:84:25: note: in definition of macro 'BUG_ON' "r" ((__force long)(x))); \ ^ In file included from arch/powerpc/kernel/ptrace.c:24:0: include/linux/regset.h:253:26: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare] if (end_pos < 0 || *pos < end_pos) { ^ In file included from include/linux/bug.h:4:0, from include/linux/thread_info.h:11, from include/asm-generic/preempt.h:4, from arch/powerpc/include/generated/asm/preempt.h:1, from include/linux/preempt.h:18, from include/linux/spinlock.h:50, from include/linux/seqlock.h:35, from include/linux/time.h:5, from include/uapi/linux/timex.h:56, from include/linux/timex.h:56, from include/linux/sched.h:19, from arch/powerpc/kernel/ptrace.c:19: include/linux/regset.h: In function 'user_regset_copyout_zero': include/linux/regset.h:282:14: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare] BUG_ON(*pos < start_pos); ^ arch/powerpc/include/asm/bug.h:75:27: note: in definition of macro 'BUG_ON' if (__builtin_constant_p(x)) { \ ^ include/linux/regset.h:282:14: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare] BUG_ON(*pos < start_pos); ^ arch/powerpc/include/asm/bug.h:76:7: note: in definition of macro 'BUG_ON' if (x) \ ^ include/linux/regset.h:282:14: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare] BUG_ON(*pos < start_pos); ^ -- In file included from arch/powerpc/kernel/syscalls.c:23:0: const struct sched_group_energy * const(*sched_domain_energy_f)(int cpu); ^ arch/powerpc/kernel/syscalls.c:118:6: error: no previous prototype for 'ppc_fadvise64_64' [-Werror=missing-prototypes] long ppc_fadvise64_64(int fd, int advice, u32 offset_high, u32 offset_low, ^ arch/powerpc/kernel/syscalls.c:125:6: error: no previous prototype for 'do_show_syscall' [-Werror=missing-prototypes] void do_show_syscall(unsigned long r3, unsigned long r4, unsigned long r5, ^ arch/powerpc/kernel/syscalls.c:134:6: error: no previous prototype for 'do_show_syscall_exit' [-Werror=missing-prototypes] void do_show_syscall_exit(unsigned long r3) ^ cc1: all warnings being treated as errors -- In file included from include/linux/kernel_stat.h:9:0, from arch/powerpc/kernel/irq.c:35: const struct sched_group_energy * const(*sched_domain_energy_f)(int cpu); ^ arch/powerpc/kernel/irq.c:136:22: error: no previous prototype for '__check_irq_replay' [-Werror=missing-prototypes] notrace unsigned int __check_irq_replay(void) ^ arch/powerpc/kernel/irq.c:290:14: error: no previous prototype for 'restore_interrupts' [-Werror=missing-prototypes] void notrace restore_interrupts(void) ^ In file included from include/linux/interrupt.h:11:0, from include/linux/kernel_stat.h:8, from arch/powerpc/kernel/irq.c:35: arch/powerpc/kernel/irq.c: In function 'migrate_irqs': include/linux/irqnr.h:12:45: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare] for (irq = 0, desc = irq_to_desc(irq); irq < nr_irqs; \ ^ arch/powerpc/kernel/irq.c:435:2: note: in expansion of macro 'for_each_irq_desc' for_each_irq_desc(irq, desc) { ^ arch/powerpc/kernel/irq.c:446:25: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare] if (cpumask_any(mask) >= nr_cpu_ids) { ^ arch/powerpc/kernel/irq.c: At top level: arch/powerpc/kernel/irq.c:548:13: error: no previous prototype for 'init_IRQ' [-Werror=missing-prototypes] void __init init_IRQ(void) ^ cc1: all warnings being treated as errors -- In file included from arch/powerpc/include/asm/uaccess.h:7:0, from arch/powerpc/kernel/align.c:23: const struct sched_group_energy * const(*sched_domain_energy_f)(int cpu); ^ In file included from arch/powerpc/kernel/align.c:23:0: arch/powerpc/kernel/align.c: In function 'emulate_dcbz': arch/powerpc/include/asm/uaccess.h:51:40: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare] (((addr) <= (segment).seg) && ((size) <= (segment).seg)) ^ arch/powerpc/include/asm/uaccess.h:63:3: note: in expansion of macro '__access_ok' __access_ok((__force unsigned long)(addr), (size), get_fs())) ^ arch/powerpc/kernel/align.c:211:26: note: in expansion of macro 'access_ok' if (user_mode(regs) && !access_ok(VERIFY_WRITE, p, size)) ^ arch/powerpc/kernel/align.c:213:16: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare] for (i = 0; i < size / sizeof(long); ++i) ^ arch/powerpc/kernel/align.c: In function 'emulate_vsx': arch/powerpc/kernel/align.c:717:16: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare] for (j = 0; j < length; j += elsize) { ^ arch/powerpc/kernel/align.c:718:17: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare] for (i = 0; i < elsize; ++i) { ^ arch/powerpc/kernel/align.c: In function 'fix_alignment': arch/powerpc/kernel/align.c:992:17: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare] for (i = 0; i < nb; i++) ^ arch/powerpc/kernel/align.c:1067:17: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare] for (i = 0; i < nb; i++) ^ cc1: all warnings being treated as errors -- In file included from arch/powerpc/kernel/signal_32.c:20:0: const struct sched_group_energy * const(*sched_domain_energy_f)(int cpu); ^ arch/powerpc/kernel/signal_32.c:60:25: error: no previous prototype for 'compat_sys_swapcontext' [-Werror=missing-prototypes] #define sys_swapcontext compat_sys_swapcontext ^ arch/powerpc/kernel/signal_32.c:1130:6: note: in expansion of macro 'sys_swapcontext' long sys_swapcontext(struct ucontext __user *old_ctx, ^ arch/powerpc/kernel/signal_32.c: In function 'compat_sys_swapcontext': arch/powerpc/kernel/signal_32.c:1159:15: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare] if (ctx_size < UCONTEXTSIZEWITHOUTVSX) ^ arch/powerpc/kernel/signal_32.c:1165:16: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare] if ((ctx_size < sizeof(struct ucontext)) && ^ arch/powerpc/kernel/signal_32.c:1169:15: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare] if (ctx_size >= sizeof(struct ucontext)) ^ In file included from include/linux/poll.h:11:0, from include/linux/ring_buffer.h:7, from include/linux/ftrace_event.h:5, from include/trace/syscall.h:6, from include/linux/syscalls.h:81, from arch/powerpc/kernel/signal_32.c:30: arch/powerpc/include/asm/uaccess.h:51:40: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare] (((addr) <= (segment).seg) && ((size) <= (segment).seg)) ^ arch/powerpc/include/asm/uaccess.h:63:3: note: in expansion of macro '__access_ok' __access_ok((__force unsigned long)(addr), (size), get_fs())) ^ arch/powerpc/kernel/signal_32.c:1190:8: note: in expansion of macro 'access_ok' if (!access_ok(VERIFY_WRITE, old_ctx, ctx_size) ^ arch/powerpc/include/asm/uaccess.h:51:40: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare] (((addr) <= (segment).seg) && ((size) <= (segment).seg)) ^ arch/powerpc/include/asm/uaccess.h:63:3: note: in expansion of macro '__access_ok' __access_ok((__force unsigned long)(addr), (size), get_fs())) ^ arch/powerpc/kernel/signal_32.c:1198:7: note: in expansion of macro 'access_ok' if (!access_ok(VERIFY_READ, new_ctx, ctx_size) ^ arch/powerpc/kernel/signal_32.c:1134:16: error: variable 'tmp' set but not used [-Werror=unused-but-set-variable] unsigned char tmp; ^ arch/powerpc/kernel/signal_32.c: At top level: arch/powerpc/kernel/signal_32.c:59:26: error: no previous prototype for 'compat_sys_rt_sigreturn' [-Werror=missing-prototypes] #define sys_rt_sigreturn compat_sys_rt_sigreturn ^ arch/powerpc/kernel/signal_32.c:1221:6: note: in expansion of macro 'sys_rt_sigreturn' long sys_rt_sigreturn(int r3, int r4, int r5, int r6, int r7, int r8, ^ arch/powerpc/kernel/signal_32.c:61:23: error: no previous prototype for 'compat_sys_sigreturn' [-Werror=missing-prototypes] #define sys_sigreturn compat_sys_sigreturn ^ arch/powerpc/kernel/signal_32.c:1493:6: note: in expansion of macro 'sys_sigreturn' long sys_sigreturn(int r3, int r4, int r5, int r6, int r7, int r8, ^ cc1: all warnings being treated as errors -- In file included from arch/powerpc/kernel/vdso.c:13:0: const struct sched_group_energy * const(*sched_domain_energy_f)(int cpu); ^ arch/powerpc/kernel/vdso.c: In function 'vdso_fixup_alt_funcs': arch/powerpc/kernel/vdso.c:631:16: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare] for (i = 0; i < ARRAY_SIZE(vdso_patches); i++) { ^ arch/powerpc/kernel/vdso.c: In function 'vdso_init': arch/powerpc/kernel/vdso.c:812:16: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare] for (i = 0; i < vdso32_pages; i++) { ^ arch/powerpc/kernel/vdso.c:825:16: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare] for (i = 0; i < vdso64_pages; i++) { ^ cc1: all warnings being treated as errors -- In file included from arch/powerpc/kernel/process.c:18:0: const struct sched_group_energy * const(*sched_domain_energy_f)(int cpu); ^ In file included from include/linux/netfilter.h:5:0, from include/net/netns/netfilter.h:5, from include/net/net_namespace.h:22, from include/linux/init_task.h:15, from arch/powerpc/kernel/process.c:29: include/linux/skbuff.h: In function 'skb_can_coalesce': include/linux/skbuff.h:2535:14: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare] off == frag->page_offset + skb_frag_size(frag); ^ arch/powerpc/kernel/process.c: At top level: arch/powerpc/kernel/process.c:78:6: error: no previous prototype for 'giveup_fpu_maybe_transactional' [-Werror=missing-prototypes] void giveup_fpu_maybe_transactional(struct task_struct *tsk) ^ arch/powerpc/kernel/process.c:96:6: error: no previous prototype for 'giveup_altivec_maybe_transactional' [-Werror=missing-prototypes] void giveup_altivec_maybe_transactional(struct task_struct *tsk) ^ arch/powerpc/kernel/process.c:721:6: error: no previous prototype for 'restore_tm_state' [-Werror=missing-prototypes] void restore_tm_state(struct pt_regs *regs) ^ arch/powerpc/kernel/process.c:1076:5: error: no previous prototype for 'arch_dup_task_struct' [-Werror=missing-prototypes] int arch_dup_task_struct(struct task_struct *dst, struct task_struct *src) ^ arch/powerpc/kernel/process.c:1633:15: error: no previous prototype for 'arch_align_stack' [-Werror=missing-prototypes] unsigned long arch_align_stack(unsigned long sp) ^ cc1: all warnings being treated as errors .. vim +361 drivers/mmc/host/sdhci-of-esdhc.c 342 343 static int sdhci_esdhc_probe(struct platform_device *pdev) 344 { 345 struct sdhci_host *host; 346 struct device_node *np; 347 struct sdhci_pltfm_host *pltfm_host; 348 struct sdhci_esdhc *esdhc; 349 int ret; 350 351 host = sdhci_pltfm_init(pdev, &sdhci_esdhc_pdata, 0); 352 if (IS_ERR(host)) 353 return PTR_ERR(host); 354 355 sdhci_get_of_property(pdev); 356 357 np = pdev->dev.of_node; 358 359 pltfm_host = sdhci_priv(host); 360 esdhc = pltfm_host->priv; 362 host->quirks2 |= SDHCI_QUIRK2_HOST_NO_CMD23; 363 364 if (esdhc->vendor_ver > VENDOR_V_22) 365 host->quirks &= ~SDHCI_QUIRK_NO_BUSY_IRQ; 366 367 if (of_device_is_compatible(np, "fsl,p2020-esdhc")) { 368 /* 369 * Freescale messed up with P2020 as it has a non-standard 370 * host control register 371 */ 372 host->quirks2 |= SDHCI_QUIRK2_BROKEN_HOST_CONTROL; 373 } 374 375 /* call to generic mmc_of_parse to support additional capabilities */ 376 mmc_of_parse(host->mmc); 377 mmc_of_parse_voltage(np, &host->ocr_mask); 378 379 ret = sdhci_add_host(host); 380 if (ret) 381 sdhci_pltfm_free(pdev); 382 383 return ret; 384 } 385 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org