Skip to content
This repository has been archived by the owner on Dec 18, 2024. It is now read-only.

Light all leds when flash is empty. #372

Closed
wants to merge 5 commits into from

Conversation

kareltucek
Copy link
Contributor

Closes UltimateHackingKeyboard/firmware#971

Tested only with uhk80.

@mondalaci
Copy link
Member

I ran nrfjprog --eraseall on both halves, then flashed the firmware. This is the result:

IMG20241203105922

Some of the left LEDs are yellow, and all of the right LEDs are dim.

@kareltucek
Copy link
Contributor Author

Ah. I naively assumed the left half leds to be controlled by right side 🤦.

Will fix that in a minute.

@kareltucek
Copy link
Contributor Author

Should be fixed.

@mondalaci
Copy link
Member

The UHK 80 works as expected.

But UHK 60 LEDs are lit according to their functional backlighting colors. User config is set. Hardware config is unset. Uninitialized EEPROM seems to contain 0xff, not 0x00, which may be the issue.

@kareltucek
Copy link
Contributor Author

User config is set.

This is the issue. I didnt read the specification carefully and require both regions to be empty.

@kareltucek
Copy link
Contributor Author

Please test now.

@mondalaci
Copy link
Member

Almost perfect, but the LEDs of the left ANSI Shift are disabled.

@kareltucek
Copy link
Contributor Author

kareltucek commented Dec 4, 2024

Aargh. Is that important? Switches can't be tested at this point anyways.

(If so, I will have to figure out another way to implement that...)

Edit: nevermind, will deal with it.

@mondalaci
Copy link
Member

At this stage, switches aren't even put into the PCB. The operator must see whether all LEDs work.

@kareltucek
Copy link
Contributor Author

Here you go.

@kareltucek kareltucek force-pushed the always_on_when_config_is_empty branch from 3b718ec to c77218b Compare December 5, 2024 00:08
@kareltucek
Copy link
Contributor Author

Migrated to UltimateHackingKeyboard/firmware#821

@kareltucek kareltucek closed this Dec 6, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

All LEDs must be white after factory flashing
2 participants