From 883b0b702a3c40d653d799bc4fa786d35d151a75 Mon Sep 17 00:00:00 2001 From: Jessica Janiuk Date: Tue, 22 Sep 2026 19:41:57 -0700 Subject: [PATCH] docs: confirm OLED I2C address against real hardware MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Printed on the display module's own silkscreen, matching what was already assumed (and what PCB/GPIO_table.md already documented as fact) — no code change, just removing the now-stale "unconfirmed assumption" hedge. Co-Authored-By: Claude Sonnet 5 --- include/oled.h | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/include/oled.h b/include/oled.h index 189288a..f3fdae9 100644 --- a/include/oled.h +++ b/include/oled.h @@ -14,10 +14,9 @@ namespace Oled { constexpr int kWidth = 128; constexpr int kHeight = 64; -// Common default for SSD1306 breakout modules — PCB/README.md doesn't -// pin down the exact module's I2C address, so this is an assumption. -// Confirm against the real hardware during this PR's bring-up milestone -// (0x3D is the other common alternative if 0x3C comes back empty). +// Confirmed against the real hardware — printed on the display module's +// own silkscreen (also documented in PCB/GPIO_table.md's Accelerometer +// section, which shares the bus at a different address). constexpr uint8_t kI2cAddress = 0x3C; } // namespace Oled