Skip to content

Fix screen geometry update for SH1107 display#10444

Merged
thebentern merged 1 commit into
meshtastic:developfrom
Bjk8kds:128x128
May 9, 2026
Merged

Fix screen geometry update for SH1107 display#10444
thebentern merged 1 commit into
meshtastic:developfrom
Bjk8kds:128x128

Conversation

@Bjk8kds

@Bjk8kds Bjk8kds commented May 9, 2026

Copy link
Copy Markdown
Contributor

Added conditional block to update screen geometry for SH1107 128x128.

before WhatsApp Image 2026-05-09 at 16 45 21 (1)
after WhatsApp Image 2026-05-09 at 16 45 21

Fix #9751

🤝 Attestations

  • I have tested that my proposed changes behave as described.
  • I have tested that my proposed changes do not cause any obvious regressions on the following devices:
    • Heltec (Lora32) V3
    • LilyGo T-Deck
    • LilyGo T-Beam
    • RAK WisBlock 4631
    • Seeed Studio T-1000E tracker card
    • Other (please specify below)
      • NRF52 Pro-micro DIY
      • Seeed Xiao ESP32-S3

Added conditional block to update screen geometry for SH1107 128x128.
@CLAassistant

CLAassistant commented May 9, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@github-actions github-actions Bot added needs-review Needs human review bugfix Pull request that fixes bugs labels May 9, 2026
@Bjk8kds Bjk8kds marked this pull request as ready for review May 9, 2026 09:41
@thebentern thebentern requested a review from HarukiToreda May 9, 2026 11:58

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes incorrect rendering on SH1107 128x128 OLEDs when the display type is selected via the device menu (config), by ensuring the runtime screen geometry is updated to match the 128x128 panel. This addresses issue #9751 where only a quarter of the screen was being used due to the geometry staying at the default 128x64.

Changes:

  • When config.display.oled is explicitly set (not OLED_AUTO), detect OLED_SH1107_128_128 and set screen_geometry = GEOMETRY_128_128.
  • Normalize the model value back to OLED_SH1107 after applying geometry, keeping the driver selection consistent with existing SH1107 handling.

@thebentern thebentern merged commit fefd424 into meshtastic:develop May 9, 2026
74 of 77 checks passed
thebentern pushed a commit that referenced this pull request May 9, 2026
Added conditional block to update screen geometry for SH1107 128x128.
@Bjk8kds Bjk8kds deleted the 128x128 branch May 10, 2026 11:22
balya pushed a commit to balya/meshtastic-firmware that referenced this pull request May 14, 2026
Added conditional block to update screen geometry for SH1107 128x128.
Evil8it pushed a commit to Evil8it/ME4TACTNK that referenced this pull request Jun 10, 2026
Added conditional block to update screen geometry for SH1107 128x128.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix Pull request that fixes bugs needs-review Needs human review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: SH1107 128x128 OLED display issue

5 participants