Skip to content

RP2350 datasheet: HSTX_CTRL.EXPAND_TMDS register poorly documented regarding unused bits #454

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
WolfWings opened this issue Mar 16, 2025 · 1 comment
Labels
documentation Improvements or additions to documentation pdf rp2350 Concerning the RP2350 chip

Comments

@WolfWings
Copy link

The documentation doesn't specify if the unused data input bits are zero-filled or bit-repeated from the valid bits.

Experimenting I confirmed they are zero-filled; this prevents various otherwise useful configurations as lower-precision settings have reduced (sometimes greatly) value ranges.

This is both why 1bpp modes look quite dim because they're only displaying 0x80 not 0xFF with even 2bpp greyscale modes jumping to 0xC0 as a maximum value, and the source of the 'tinting' that RGB332 and RGB565 modes end up reported with because 'white' is not 0xFFFFFF but 0xE0E0C0 (RGB332) or 0xF8FCF8 (RGB565).

Documenting that the unused bits are zero-filled versus bit-repeated from the valid bits would be appreciated in the datasheet if possible.

@lurch lurch added documentation Improvements or additions to documentation pdf rp2350 Concerning the RP2350 chip labels Mar 17, 2025
@WolfWings
Copy link
Author

As a minor addendum to this the TMDS_CTRL register documents this effect with good phrasing for the L0_NBITS/L1_NBITS/L2_NBITS:

Remaining LSBs are masked to 0 after the rotate.

Making the phrasing match between the TMDS_CTRL fields and the EXPAND_TMDS fields would be the best resolution for this I believe.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation pdf rp2350 Concerning the RP2350 chip
Projects
None yet
Development

No branches or pull requests

2 participants