Skip to content

extmod/modframebuf: add XRGB8888 format#4880

Closed
dlech wants to merge 1 commit intomicropython:masterfrom
dlech:xrgb8888
Closed

extmod/modframebuf: add XRGB8888 format#4880
dlech wants to merge 1 commit intomicropython:masterfrom
dlech:xrgb8888

Conversation

@dlech
Copy link
Copy Markdown
Contributor

@dlech dlech commented Jun 27, 2019

This adds a 32-bit per pixel XRGB format to the framebuf module.
This format is seen in some Linux display drivers.

This adds a 32-bit per pixel XRGB format to the framebuf module.
This format is seen in some Linux display drivers.
@jimmo
Copy link
Copy Markdown
Member

jimmo commented Jun 27, 2019

Like RGB565, I think this should be explicit about the byte order otherwise the behaviour will change between BE and LE CPUs (see #3536).

@peterhinch
Copy link
Copy Markdown
Contributor

I'd question whether 32-bit modes are relevant to any microcontroller port for these reasons:

  • RAM requirements for the framebuf
  • Transfer time for the framebuf contents to the display
  • Support for 32-bit modes on small display hardware

@mcauser
Copy link
Copy Markdown
Contributor

mcauser commented Jun 28, 2019

Most if not all displays support partial screen updates, so one could argue you don't need the full screen dimensions @ 32 bit represented in your framebuf - eg. you could be drawing a collection of sprites.

@dlech dlech closed this May 10, 2021
@dpgeorge dpgeorge added the extmod Relates to extmod/ directory in source label May 11, 2021
@jimmo
Copy link
Copy Markdown
Member

jimmo commented May 11, 2021

In addition to partial updates, I think this is a worthwhile feature for, e.g. APA102 and RGBW WS2812 LED strips. See #7253 for discussion.

tannewt pushed a commit to tannewt/circuitpython that referenced this pull request Jun 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

extmod Relates to extmod/ directory in source

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants