Flare576 / sofle

Tracking my adventures in split-keyboard land with the Sofle V2
MIT License
6 stars 1 forks source link

Oled down, need some advice #1

Open ScyMarshall opened 1 year ago

ScyMarshall commented 1 year ago

Hi Flare, I found your great work with the sofle and wanted to give it a try.Especially because of your display layout! Unfortunatley after flashing, both oleds show nothing - And they do work with my other keymaps. Am I missing a secret ingredient somewhere? I would be grateful if you could spare your time for me. You can look at my butchered version on my fork https://github.com/ScyMarshall/ScyMarshalls-QMK-Keymaps/tree/main/sofle/keymaps/ScyMarshallFlare

Thank you very much! Best regards, Scy

Flare576 commented 1 year ago

I'm so sorry for leaving you hanging - Truth is that I shouldn't have pushed the code in the state it's in - there's two levels of bugs happening.

Firstly, the ones I fixed:

https://github.com/Flare576/qmk_firmware/blob/4daed683f93a11d60bc10cb91c7c057b1215c61e/keyboards/sofle/keymaps/flare576/graphics.c#L23

-#ifdef OLED_DRIVER_ENABLE
+#ifdef OLED_ENABLE

and https://github.com/Flare576/qmk_firmware/blob/4daed683f93a11d60bc10cb91c7c057b1215c61e/keyboards/sofle/keymaps/flare576/graphics.c#L272

-void oled_task_user(void) {
+bool oled_task_user(void) {

and lastly https://github.com/Flare576/qmk_firmware/blob/4daed683f93a11d60bc10cb91c7c057b1215c61e/keyboards/sofle/keymaps/flare576/graphics.c#L283

+    return false;

That will get the screens working.... but THE SECONDARY OLED WILL NEVER TURN OFF. For some reason with the version of the code I'm running right now (that's in this repo), the secondary MCU doesn't get the WPM values, so it never drops to zero, so the scrolling logos never stop. I had to add a line to disable them and haven't had time to circle back to either downgrade back to the older working code or debug the issue.

I plan to build another Sofle soon™, and not having to hack the firmware on my main keyboard should lower the fear wall a bit, but hopefully this at least gets you started!