[U-Boot] [PATCH 2/3] video: mb862xx: add option VIDEO_FB_16BPP_WORD_SWAP for IPEK01

Wolfgang Denk wd at denx.de
Mon Oct 19 13:56:53 CEST 2009


Dear Wolfgang Grandegger,

In message <20091019111913.621578739 at denx.de> you wrote:
> In 16 bpp mode, the new IPEK01 board only requires swapping of D16 words
> for D32 accesses due to the diffferent connecting to the GDC bus. This
> patch introduces the configuration option VIDEO_FB_16BPP_WORD_SWAP,
> which should be set for all board using the mb862xx in 16 bpp mode. For
> the IPEK01, VIDEO_FB_16BPP_PIXEL_SWAP should not be set.

I don't see any functional change in this patch - all you do is
renaming VIDEO_FB_16BPP_PIXEL_SWAP into VIDEO_FB_16BPP_WORD_SWAP.

This makes no sense to me.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
The universe, they said, depended for its operation on the balance of
four forces which they identified as charm,  persuasion,  uncertainty
and bloody-mindedness.      -- Terry Pratchett, "The Light Fantastic"


More information about the U-Boot mailing list