Bootloader reset improvements

[darkxst]

I have two more devices to add support for bootloader reset. The first is the Sonoff iHost included in this series, second will be a forthcoming device from Smlight.

The current bootloader reset implementation doesnt really scale cleanly for adding more devices, so I have split it out into a new --bootloader-reset option. I made it a global option so it can be shared with both probe and flash commands. Kept compatibility with the previous individual device flags, in case there is any usage from scripts currently.

[codecov-commenter]

Codecov Report

Attention: 22 lines in your changes are missing coverage. Please review.

Comparison is base (3dbfc51) 60.71% compared to head (16622d2) 60.67%.

Files	Patch %	Lines
universal_silabs_flasher/flash.py	15.38%	11 Missing ⚠️
universal_silabs_flasher/flasher.py	21.42%	11 Missing ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@            Coverage Diff             @@
##              dev      #51      +/-   ##
==========================================
- Coverage   60.71%   60.67%   -0.05%     
==========================================
  Files          14       14              
  Lines        1642     1653      +11     
==========================================
+ Hits          997     1003       +6     
- Misses        645      650       +5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[darkxst]

Rebased on changes in dev branch now

[puddly]

I've made a few changes to CI, as it looks like pre-commit hooks weren't running to check for formatting. You can re-format your code like so:

pip install pre-commit
pre-commit install
pre-commit run --all-files

[darkxst]

ok, I have run the pre commit hooks and fixed errors detected.

[puddly]

Thanks! Can you also revert the changes to gpio.py replacing the _send_gpio_pattern kwargs with pin_config: dict[str, typing.Any]? Passing an untyped dictionary to do the same isn't very Pythonic. It would be better to call the function with its original kwargs and parse the config dictionary within async def enter_bootloader_reset(self, target):.

[darkxst]

sure, I have made that change as well.