Steffan Posted February 4, 2017 Report Share Posted February 4, 2017 While working with the LTDC I stumbled up on two issues in the LTDC driver: See https://git.ugfx.io/uGFX/uGFX/src/master/drivers/gdisp/STM32LTDC/gdisp_lld_STM32LTDC.c#L60 the sync field is 8-bit, while it seems it should be 32-bit as the corresponding defines are > 8 bit. "PIXIL_POS" ( see https://git.ugfx.io/uGFX/uGFX/src/master/drivers/gdisp/STM32LTDC/gdisp_lld_STM32LTDC.c#L98 ). Why is it called PIXIL and not PIXEL? Is this on purpose or is this a mistake? The same macro ended up in gdisp_lld_fb24bpp.c and gdisp_lld_framebuffer.c Link to comment Share on other sites More sharing options...
Joel Bodenmann Posted February 4, 2017 Report Share Posted February 4, 2017 Thank you very much for bringing this to our attention, we'll have a look at that in the following days. Link to comment Share on other sites More sharing options...
inmarket Posted February 4, 2017 Report Share Posted February 4, 2017 2. The spelling of pixel as pixil is a private and historical joke. A pix-el is a picture (pix) element (el). A pixil is then an pix with an ill ie a broken pixel. Thus the spelling of the word pixel as pixil is both an easter egg (hidden treasure) and a comment on the pixels being broken (buggy) hence the joke. The private part refers to a funny incident way back in my early programming history (roughly 35 years ago) that has led me to consistently and intentionally mispell it in my code since then. Well done on finding it. We will probably leave the spelling as it is as it is internal code only, not something a user would normally play with and it always gives me a chuckle. Link to comment Share on other sites More sharing options...
Steffan Posted February 4, 2017 Author Report Share Posted February 4, 2017 Hah, Ok. I'll never get over this. Im sure of it. Link to comment Share on other sites More sharing options...
inmarket Posted February 4, 2017 Report Share Posted February 4, 2017 Link to comment Share on other sites More sharing options...
Recommended Posts
Create an account or sign in to comment
You need to be a member in order to leave a comment
Create an account
Sign up for a new account in our community. It's easy!
Register a new accountSign in
Already have an account? Sign in here.
Sign In Now