Why M_NOWAIT?

Should this be under bootverbose?

Why an array?

if ((hpd & HPD_DET) == HPD_DET)

return ((edid[CEA_DTD] & DTD_BASIC_AUDIO) == DTD_BASIC_AUDIO)

or

return ((edid[CEA_DTD] & DTD_BASIC_AUDIO) != 0)

tab

This line looks quite wide, is it past 80 characters?

Where did this number come from? (131072 == 0x20000)

Are these a10 specific?

Where did these compat strings come from?

It's a lot of memory and it needs to be physically contiguous (and no GART or IOMMU as far as I can tell). If the HDMI cable is plugged in some time after boot, memory may be fragmented and can fail.

The alternative is to reserve memory early at boot to account for the largest possible framebuffer. Not sure what the top end should be if we want to support hotplug (DEBE can support up to 8Kx8K layers -- a single layer at 32bpp would be 256MB!).

You're right, I'll fix it.

No specific reason, I will fix it.

I pulled it out of the air. Will update this to use pcm_getbuffersize.

a10_fb.c: a10, a20, a31, and a80 share the same display engine. Later SoCs (e.g. a83t) are different (datasheet refers to it as "DE2.0").

a10_hdmi.c: a10/a20 have the same hdmi ("HDMI 1.3") block. a31 has a block that is mostly the same ("HDMI 1.4") with a different DDC interface and a few other minor changes. Later SoCs (a80, a83t, etc) have something completely different (not documented in the datasheet).

I made them up, they are not in Linux mainline and the vendor tree doesn't use FDT.