Skip to content

Conversation

@Drugantibus
Copy link
Owner

Made some changes to implement emulated EEPROM in NVM to persist settings on power loss as seen in qmk's #6068, using @daltona's mdloader fork and his qmk alt-eeprom branch. Thank you so much Alexandre!

Tested it for several weeks, seems to be stable and behave as expected. Still a couple of bugs to fix and features to add, but so far so good.

daltona and others added 6 commits April 27, 2020 12:37
Signed-off-by: Alexandre d Alton <[email protected]>
- Use define for SmartEEPROM buffer address
- Check buffer overflow
- Do not perform operation when timeout occurs

Signed-off-by: Alexandre d'Alton <[email protected]>
Merge changes to support saving to NVM
@Drugantibus Drugantibus merged commit 72966bc into master Jul 24, 2021
@daltona
Copy link

daltona commented Jul 24, 2021

You're welcome Alessandro !

I also have a branch somewhere with VIA support should you be interested, I can create a pull request on your repo !

@Drugantibus
Copy link
Owner Author

Hey!

Oh awesome, that would be nice to try out :)
Let me push a new via branch, as it will probably break my existing raw HID implementation I assume?
Thanks again both for your work and for your kindness :)

@Drugantibus
Copy link
Owner Author

Hey @daltona, sorry for getting back to you after all this time, the EEPROM works perfectly and I forgot about your comment 😅
I was messing around with your VIA patch but I never got it anywhere close to working. Everytime I flash a VIA firmware I get really weird behaviour on my board, i.e. some keys don't work at all (usually in "zones", like bnm,./), some keys are mapped incorrectly, and the RGB is in a weird state. Since these errors are not consistent (dead keys are not always the same, RGB bootup state is different), I suspect it has to do with the content of my EEPROM. I guess I'd need to resize the NVM as tzarc mentions here? Is this possible to do using your mdloader or would I need a J-Link?

Thanks :)

Drugantibus pushed a commit that referenced this pull request Sep 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants