Issue with the code

Question

Issue with the code

5Volts opened this issue 5 years ago · 8 comments

So I tried to use my 128x64 I2C OLED with an Arduino and it worked without a problem. I really wanted to use pure C code so I came across your library. I tried running it on my ATMEGA328P but this is what I got.

Answer 1 · 2019-09-18T08:03:26.000Z

Wich controller have you configured in lcd.h (line 61)? It looks like configured for SH1106 controller/display (default when you download library from github) and you use a SSD1306 controller/display.

Answer 2 · 2019-09-18T15:30:38.000Z

Yes I am indeed using a SSD1306. What are the things that I need to change in that header file so that it works on the SSD1306 controller?

Answer 3 · 2019-09-18T17:27:09.000Z

You have to replace "#define SH1106" by "#define SSD1306" in lcd.h (line 61 in actual version that updated today)

Answer 4 · 2019-09-18T18:24:11.000Z

It worked now. Another thing I have to change is the LCD_I2C_ADR to 0x78 so that it get directed to the 0X3C address. Thanks for the help!

Answer 5 · 2019-12-11T23:33:40.000Z

Maybe this issue could be in a FAQ? This is not obvious by reading the example...

Answer 6 · 2019-12-12T00:31:53.000Z

Maybe this issue could be in a FAQ? This is not obvious by reading the example...

Ähm...why not clear from TODOs in header-file of lcd-library? All TODOs in the header files tells you, what you have to do for running display. Your prefered IDE should show you all TODOs.

Answer 7 · 2019-12-12T12:14:17.000Z

Sorry, but try to consider that I am not the only one thinking this way: the TODO word is commonly used to document what should be implemented next by the development team. For example it is not clear if the programmer (your client) should expect the library to work with graphical by changing the respective macro definition because the one maybe thinks TODO is something to be developed. I like the way this lib is implemented, it is really simple to integrate, but I think the documentation can be improved. Thanks!

…

On Wed, Dec 11, 2019, 9:31 PM Sylaina ***@***.***> wrote: Maybe this issue could be in a FAQ? This is not obvious by reading the example... Ähm...why not clear from TODOs in header-file of lcd-library? All TODOs in the header files tells you, what you have to do for running display. Your prefered IDE should show you all TODOs. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#6?email_source=notifications&email_token=ABNFMHXNEPPA3KCZT5NS5SLQYGA7VA5CNFSM4IXKSTP2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGVBQZQ#issuecomment-564795494>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABNFMHWHGVWJ5I44CY6YHM3QYGA7VANCNFSM4IXKSTPQ> .

Answer 8 · 2020-12-20T21:04:27.000Z

Had the same Issue... I tried different Controllers etc...and now after 4 hours, I read your Issue.
Should really be in the description of the Lib. Overall its pretty nice an working well now.

PS: You could add the defined(AVR_ATmega32)|| to your lib, cause its working well and has the same registers.