Skip to content

feat: huawei v100 bsp error publishing #1614

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
markxoe wants to merge 38 commits into
base: main
Choose a base branch
from
Draft

feat: huawei v100 bsp error publishing #1614

markxoe wants to merge 38 commits into from
+19,896 −1

Conversation

@markxoe
Copy link
Contributor

@markxoe markxoe commented Dec 5, 2025 •
edited
Loading

Describe your changes

  • adds a few errors to the BSP error set
  • adds handlers to BSP errors and publishes them to the PSU

Todo:

Note: diff between #1593 and this PR is here: feature/add-huawei-v100..feature/huawei-v100/bsp-error-publishing

Issue ticket number and link

Checklist before requesting a review

  • I have performed a self-review of my code
  • I have made corresponding changes to the documentation
  • I read the contribution documentation and made sure that my changes meet its requirements

@markxoe markxoe force-pushed the feature/huawei-v100/bsp-error-publishing branch from 2fb4e96 to 619cee4 Compare December 5, 2025 16:37
corneliusclaussen and others added 29 commits December 12, 2025 08:45
@corneliusclaussen @Pietfried
Add Huawei power stack driver (#107)
4d01442 
* Add Huawei power stack driver

Co-authored-by: Fabian Gajek <[email protected]>
Co-authored-by: Mark Oude Elberink <[email protected]>

Huawei V100R023C10 Updates (#137)

Huawei V100R023C10: multiple connector support

Co-authored-by: Tim <[email protected]>

feat: huawei v100r023c10 improvements (#271)

* feat: squashed huawei mock improvements
* wip: convert some info logs to debug
* wip: fix quad config
* wip: use psu capabilities instead of hardcoded values
* wip: fix log connector number
* wip: goose socket filter
* wip: mock send min rated current/voltage
* wip: capabilities missing error
* wip: fix log messages
* wip: mock: wait sending capabilities for 10s
* wip: fix module placeholder allocation response acknowleged in every state
* wip: log running mode and working status on change
* wip: update changelog
* wip: fix capabilities not being limited by config
* wip: improve log prefix in connector base
* wip: add goose filter documentation
* wip: remove log double prefix; update mock to wait only 5s; update changelog
* wip: add over voltage monitor hack
* wip: change missing hmac log to verbosity
* wip: add PowermeterOVMSimulator module
* wip: log rated output power, voltage and current on change as info
* wip: add units for added logs
* wip: remove wrongly-added log
* wip: callbacks
* wip: improve mock structure (a bit)
* wip: add mqtt client
* wip: add mqtt feature
* wip: reformat
* wip: fix capabilities not resetted when PSU gets disconnected
* wip: rebase fixing powermeterovmsimulator module
* wip: add tls/tcp log
* refactor: clang-format
* wip: add flexible upstream voltage config
* wip: apply suggestion
* Apply suggestions from code review

---------

Signed-off-by: Mark Oude Elberink <[email protected]>
Signed-off-by: corneliusclaussen <[email protected]>
Co-authored-by: Fabian Gajek <[email protected]>
Co-authored-by: Kai Hermann <[email protected]>

Updated licenses

Signed-off-by: Piet Gömpel <[email protected]>

Moved files into modules/

Signed-off-by: Piet Gömpel <[email protected]>
@Pietfried
Replaced usage of designated initializers
7c67dee 
Signed-off-by: Piet Gömpel <[email protected]>
@Pietfried
Removed gitlab ci files
01c8e44 
Signed-off-by: Piet Gömpel <[email protected]>
@Pietfried
Replace README with doc.rst
fd946c3 
Signed-off-by: Piet Gömpel <[email protected]>
@Pietfried @markxoe
Update modules/HardwareDrivers/PowerSupplies/Huawei_V100R023C10/Huawe…
b62f1d1 
…i_V100R023C10.cpp

Co-authored-by: Mark Oude Elberink <[email protected]>
Signed-off-by: Piet Gömpel <[email protected]>
@Pietfried
Fix initiializers and add module to CMakeLists.txt
38d4c63 
Signed-off-by: Piet Gömpel <[email protected]>
@Pietfried @hikinggrass
Apply suggestions from code review
811dd4e 
Co-authored-by: Kai Hermann <[email protected]>
Signed-off-by: Piet Gömpel <[email protected]>
@Pietfried @markxoe
Apply suggestions from code review
7665b2b 
Co-authored-by: Mark Oude Elberink <[email protected]>
Signed-off-by: Piet Gömpel <[email protected]>
@Pietfried @markxoe
Update modules/HardwareDrivers/PowerSupplies/Huawei_V100R023C10/fusio…
12f010b 
…n_charger_lib/fusion-charger-dispenser-library/README.md

Co-authored-by: Mark Oude Elberink <[email protected]>
Signed-off-by: Piet Gömpel <[email protected]>
@Pietfried @markxoe
Update modules/HardwareDrivers/PowerSupplies/Huawei_V100R023C10/fusio…
39a5f53 
…n_charger_lib/fusion-charger-dispenser-library/README.md

Co-authored-by: Mark Oude Elberink <[email protected]>
Signed-off-by: Piet Gömpel <[email protected]>
@Pietfried @markxoe
Update modules/HardwareDrivers/PowerSupplies/Huawei_V100R023C10/fusio…
148878f 
…n_charger_lib/fusion-charger-dispenser-library/README.md

Co-authored-by: Mark Oude Elberink <[email protected]>
Signed-off-by: Piet Gömpel <[email protected]>
@Pietfried @markxoe
Update modules/HardwareDrivers/PowerSupplies/Huawei_V100R023C10/fusio…
31ef1b9 
…n_charger_lib/fusion-charger-dispenser-library/README.md

Co-authored-by: Mark Oude Elberink <[email protected]>
Signed-off-by: Piet Gömpel <[email protected]>
@Pietfried
Remove Auffaeligkeiten and renamed readme.md to README.md
66edda2 
Signed-off-by: Piet Gömpel <[email protected]>
@Pietfried
Use std:: prefixed integer types
2c413b6 
Signed-off-by: Piet Gömpel <[email protected]>
@Pietfried
Remove clang format files
583b8d7 
Signed-off-by: Piet Gömpel <[email protected]>
@Pietfried
Fix language
eceef58 
Signed-off-by: Piet Gömpel <[email protected]>
@Pietfried
Removed unused file
e1c02e4 
Signed-off-by: Piet Gömpel <[email protected]>
@Pietfried
Removed designated initializers
ffd7ee0 
Signed-off-by: Piet Gömpel <[email protected]>
@Pietfried
Add missing newline to .gitignore
44731ef 
Signed-off-by: Piet Gömpel <[email protected]>
@Pietfried
Removed code that was commented out
2151987 
Signed-off-by: Piet Gömpel <[email protected]>
@Pietfried
Use modern instead of c-style casts
15c163b 
Signed-off-by: Piet Gömpel <[email protected]>
@Pietfried
Removed code that was commented out
7bdfa6e 
Signed-off-by: Piet Gömpel <[email protected]>
@Pietfried
Removed unused files
c29f0f8 
Signed-off-by: Piet Gömpel <[email protected]>
@Pietfried
Call get_mac_address only once
05abe04 
Signed-off-by: Piet Gömpel <[email protected]>
@Pietfried
add accidentally removed }
2e5edf3 
Signed-off-by: Piet Gömpel <[email protected]>
@Pietfried
Use reinterpretcast instead of c-style
d2408e3 
Signed-off-by: Piet Gömpel <[email protected]>
@Pietfried
Refactor CMakeLists: Refactor test setup and register libraries with …
077df96 
…ev_register_library_target to include those in EVerest coverage generation

Signed-off-by: Piet Gömpel <[email protected]>
@Pietfried
Fix Indentation
a5d12c7 
Signed-off-by: Piet Gömpel <[email protected]>
@Pietfried
Use reinterpretcast instead of c-style
353635d 
Signed-off-by: Piet Gömpel <[email protected]>
@Pietfried
Renamed and refactored library README to address the updated CMake st…
4abeafc 
…ructure. The module internally libraries can currently not be built standalone

Signed-off-by: Piet Gömpel <[email protected]>
@Pietfried
clang format
165e670 
Signed-off-by: Piet Gömpel <[email protected]>
@markxoe markxoe force-pushed the feature/huawei-v100/bsp-error-publishing branch from 7d2f740 to f4f605f Compare December 12, 2025 09:37
@markxoe
feat: initial telemetry
c3e6a90 
Signed-off-by: Mark Oude Elberink <[email protected]>
@markxoe
feat: add BSP errors
9812c95 
Signed-off-by: Mark Oude Elberink <[email protected]>
@markxoe
feat: add errors to YetiSimulator module
e13f503 
Signed-off-by: Mark Oude Elberink <[email protected]>
@markxoe
feat: add error handling for Dispenser alarms
cf309cf 
Signed-off-by: Mark Oude Elberink <[email protected]>
@markxoe
doc: add doc to changelog + doc.rst
ad1a1e4 
Signed-off-by: Mark Oude Elberink <[email protected]>
@markxoe
refactor: use callbacks for alarms
bd7cbe5 
Signed-off-by: Mark Oude Elberink <[email protected]>
@markxoe
feat: add error telemetry
0e64004 
Signed-off-by: Mark Oude Elberink <[email protected]>
@markxoe markxoe force-pushed the feature/huawei-v100/bsp-error-publishing branch from f4f605f to 0e64004 Compare December 12, 2025 09:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@SebaLukas SebaLukas Awaiting requested review from SebaLukas SebaLukas will be requested when the pull request is marked ready for review SebaLukas is a code owner

@Pietfried Pietfried Awaiting requested review from Pietfried Pietfried will be requested when the pull request is marked ready for review Pietfried is a code owner

@corneliusclaussen corneliusclaussen Awaiting requested review from corneliusclaussen corneliusclaussen will be requested when the pull request is marked ready for review corneliusclaussen is a code owner

@james-ctc james-ctc Awaiting requested review from james-ctc james-ctc will be requested when the pull request is marked ready for review james-ctc is a code owner

@mlitre mlitre Awaiting requested review from mlitre mlitre will be requested when the pull request is marked ready for review mlitre is a code owner

@hikinggrass hikinggrass Awaiting requested review from hikinggrass hikinggrass will be requested when the pull request is marked ready for review hikinggrass is a code owner

@florinmihut florinmihut Awaiting requested review from florinmihut florinmihut will be requested when the pull request is marked ready for review florinmihut is a code owner

@cburandt cburandt Awaiting requested review from cburandt cburandt will be requested when the pull request is marked ready for review cburandt is a code owner

@djchhp djchhp Awaiting requested review from djchhp djchhp will be requested when the pull request is marked ready for review djchhp is a code owner

@andistorm andistorm Awaiting requested review from andistorm andistorm will be requested when the pull request is marked ready for review andistorm is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@markxoe @corneliusclaussen @Pietfried