Skip to content
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

macropad rev B definion and fix for SH110x addressing #4864

Merged
merged 6 commits into from
Jun 8, 2021

Conversation

ladyada
Copy link
Member

@ladyada ladyada commented Jun 6, 2021

the SH1107 chipset is actually column not row mode (like the SSD1306!) BUT the SH1107 module we use on the featherwing is vertical orientation (confusing!). We should stick to the column mode addressing, but 'rotate' the SH1107 module.

we will have to fix the SH1107 driver to match

this fix required for SH1106 (which uses the same page mode commands but ISNT rotated). Tested on Macropad hardware

ladyada added 3 commits June 6, 2021 15:57
…D1107 module we use is vertical orientation (confusing!) so row/col are NOT swapped. we will have to fix the SH1107 driver to match. this fix required for SH1106 (which uses the same page mode commands but ISNT rotated)
@ladyada ladyada requested a review from tannewt June 6, 2021 22:32
Copy link
Member

@tannewt tannewt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Have you tried rotation with this? Seems like it'd be broken before and after this change.

Could you link me to the display datasheets?

@ladyada
Copy link
Member Author

ladyada commented Jun 7, 2021

SH1106.pdf

@ladyada
Copy link
Member Author

ladyada commented Jun 7, 2021

SH1107.pdf

@ladyada
Copy link
Member Author

ladyada commented Jun 7, 2021

i will try rotation now (i also have to try the col offset thing)

Copy link
Member

@tannewt tannewt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One minor thing plus the colstart and rowstart fix.

shared-module/displayio/Display.c Outdated Show resolved Hide resolved
Copy link
Member

@tannewt tannewt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me! Go ahead and merge once CI gives the OK.

@ladyada ladyada merged commit c1c101c into adafruit:main Jun 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants