adafruit/Adafruit_SSD1306

Define OLED address instead of hard-coding

Closed this issue · 6 comments

flywire commented

Define OLED address like Screen Width/Height rather than hard-coding.

https://github.com/adafruit/Adafruit_SSD1306/blob/master/examples/ssd1306_128x32_i2c/ssd1306_128x32_i2c.ino#L63

if(!display.begin(SSD1306_SWITCHCAPVCC, 0x3C)) { // Address 0x3C for 128x32

https://github.com/adafruit/Adafruit_SSD1306/blob/master/examples/ssd1306_128x64_i2c/ssd1306_128x64_i2c.ino#L59

if(!display.begin(SSD1306_SWITCHCAPVCC, 0x3D)) { // Address 0x3D for 128x64

ladyada commented

you can set the width/height on instantiation, thats where the allocation happens as well.

flywire commented

Agree but it would be nicer to use like:

...
#define SCREEN_HEIGHT 32 // OLED display height, in pixels
#define SCREEN_ADDRESS 0x3D ///< See datasheet for Address; 0x3D for 128x64, 0x3C for 128x32
...
  if(!display.begin(SSD1306_SWITCHCAPVCC, SCREEN_ADDRESS)) {
...

I'd be happy to do a PR.

ladyada commented

that is not possible to do, a #define does not change the allocation inside the library

ladyada commented

wait, do mean just changing the examples? or the API?

flywire commented

Just the examples so SCREEN_ADDRESS (or preferred name) is identified more clearly.

ladyada commented

oooh ok yes thats ok! sorry we get tons of people trying to change the API... yes a PR is welcome, please note it has to pass CI
https://learn.adafruit.com/contribute-to-arduino-with-git-and-github

  • 👍1