exynos_hdmi.c fails to build with v3.13-10094-g9b0cd30
Hi All, After the DRM merge, the exynos_hdmi.c file fails to build with our ARM config. The error is: drivers/gpu/drm/exynos/exynos_hdmi.c:382:8: error: 'hdmi_infoframe' defined as wrong kind of tag struct hdmi_infoframe { ^ make[4]: *** [drivers/gpu/drm/exynos/exynos_hdmi.o] Error 1 make[3]: *** [drivers/gpu/drm/exynos] Error 2 make[2]: *** [drivers/gpu/drm] Error 2 which to me was a somewhat confusing error message. After digging further, I believe it means that there is a conflict with the definition in exynos_hdmi.c and the one found in include/linux/hdmi.h for what hdmi_infoframe is supposed to be. exynos_hdmi.c: struct hdmi_infoframe { enum HDMI_PACKET_TYPE type; u8 ver; u8 len; }; include/linux/hdmi.h: union hdmi_infoframe { struct hdmi_any_infoframe any; struct hdmi_avi_infoframe avi; struct hdmi_spd_infoframe spd; union hdmi_vendor_any_infoframe vendor; struct hdmi_audio_infoframe audio; }; Could someone take a look at this? I have no idea how this wasn't caught before being merged. josh -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
3.12 merge window exynos cpufreq driver fails to build
Using Linus' tree as of this morning (v3.11-2654-g458c3f6), the exynos-cpufreq driver doesn't build because it includes a file that doesn't exist. + make -s ARCH=arm V=1 -j4 bzImage KALLSYMS_EXTRA_PASS=1 arch/arm/kernel/return_address.c:63:2: warning: #warning "TODO: return_address should use unwind tables" [-Wcpp] #warning "TODO: return_address should use unwind tables" ^ arch/arm/mm/dma-mapping.c:253:2: warning: #warning ARM Coherent DMA allocator does not (yet) support huge TLB [-Wcpp] #warning ARM Coherent DMA allocator does not (yet) support huge TLB ^ drivers/cpufreq/exynos-cpufreq.c:21:22: fatal error: plat/cpu.h: No such file or directory #include ^ compilation terminated. make[2]: *** [drivers/cpufreq/exynos-cpufreq.o] Error 1 I'm guessing that file is supposed to come from arch/arm/plat-samsung/include/plat/, but I don't see anything that specifies a -I there. Thoughts? josh -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: 3.12 merge window exynos cpufreq driver fails to build
On Wed, Sep 4, 2013 at 11:04 AM, Tomasz Figa wrote: > On Wednesday 04 of September 2013 10:41:31 Josh Boyer wrote: >> On Wed, Sep 4, 2013 at 10:36 AM, Tomasz Figa wrote: >> > Hi Josh, >> > >> > On Wednesday 04 of September 2013 10:13:16 Josh Boyer wrote: >> >> Using Linus' tree as of this morning (v3.11-2654-g458c3f6), the >> >> exynos-cpufreq driver doesn't build because it includes a file that >> >> doesn't exist. >> >> >> >> + make -s ARCH=arm V=1 -j4 bzImage KALLSYMS_EXTRA_PASS=1 >> >> arch/arm/kernel/return_address.c:63:2: warning: #warning "TODO: >> >> return_address should use unwind tables" [-Wcpp] >> >> >> >> #warning "TODO: return_address should use unwind tables" >> >> >> >> ^ >> >> >> >> arch/arm/mm/dma-mapping.c:253:2: warning: #warning ARM Coherent DMA >> >> allocator does not (yet) support huge TLB [-Wcpp] >> >> >> >> #warning ARM Coherent DMA allocator does not (yet) support huge TLB >> >> >> >> ^ >> >> >> >> drivers/cpufreq/exynos-cpufreq.c:21:22: fatal error: plat/cpu.h: No >> >> such file or directory >> >> >> >> #include >> >> >> >> ^ >> >> >> >> compilation terminated. >> >> make[2]: *** [drivers/cpufreq/exynos-cpufreq.o] Error 1 >> >> >> >> I'm guessing that file is supposed to come from >> >> arch/arm/plat-samsung/include/plat/, but I don't see anything that >> >> specifies a -I there. >> >> >> >> Thoughts? >> > >> > Could you attach config that could be used to trigger this? Thanks in >> > advance. >> >> Attached. > > Hmm, this does not look like config for kernel neither from Linus' nor > linux-next tree. The first thing I can see is CONFIG_ARCH_EXYNOS_MULTI > selected, which is not available in any mainline tree. Sigh. Yes, I see the issue now. > As for the build error itself, headers from plat/ are available for drivers > only in single platform builds, they can't be included when multiplatform > is enabled. The exynos-cpufreq driver is not multiplatform-aware yet and so > you get the failure. Someone added some exynos multiplatform patches to the Fedora kernel. That seems to have caused these build failures. Our mistake and my apologies for the inconvenience. Thank you for the quick response. josh -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html