create a power_supply for battery information (upower support)

[babbaj]

Only implemented for mice because I only have a viper v2. Still need to only register a power_supply for mice that have a battery and generate a unique name for each power_supply.
It also looks like for some reason when the mouse is plugged in upower says the battery is at 0% and the state is unknown.

[z3ntu]

Looks somewhat reasonable from a very quick first look. Could you please look into the CI failure and fix it?

[babbaj]

the ci is complaining about the formatting even after i ran the autoformatter and only on the alpine ci

[z3ntu]

Are you running astyle 3.4.10 on your machine also?

Also the Ubuntu CI is just build-testing, all the other checks are only in the Alpine CI

[babbaj]

astyle 3.1 from nixos unstable

[babbaj]

formatting shouldn't be a problem anymore and it looks like the ci breaking isn't my fault this time

[z3ntu]

Yeah it's okay now. That failure I'll look to fix, your change broke some silent expectations from this shell script

[babbaj]

I tried to create a unique name for every device with an id allocator but it ended up creating 5 devices for the same mouse and creating 3 duplicate batteries in upower so I just used the usb id instead. Is there a better way to fix this?

[babbaj]

I'm mostly confident in this now but when I plug in my mouse it shows a battery for the wired device and one for the wireless. I don't think that's incorrect but it seems unnecessary so should the wired version of wireless mice be ignored?

[babbaj]

Ignoring wireless mouse is maybe not ideal because they can be used as wired mice without the dongle but in that case the battery level is useless anyways.
It also seems to cause upower to detect them as duplicate devices and sometimes no battery appears but that's fixable by giving them unique serial numbers.
I think ignoring the wired device would be good but sometimes the dongle will report 0% charge while the wired device reports the actual charge. (is this a known problem?)

[babbaj]

ci fixed now but I think it would be a lot simpler for the script and the C code if every case statement just called a simple macro

[babbaj]

@z3ntu got time to review this?

[z3ntu]

Give me some time, I hope to find some time soon :)