exynos_hdmi.c fails to build with v3.13-10094-g9b0cd30

2014-01-30 Thread Josh Boyer
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

2013-09-04 Thread Josh Boyer
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

2013-09-04 Thread Josh Boyer
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