Skip to content
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.

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

Adding ISO15118-20 dynamic mode support #912

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

SebaLukas
Copy link
Contributor

Describe your changes

This PR adds with this libiso15118 Adding basic dynamic mode PR a basic support for ISO15118-20 dynamic mode.

Issue ticket number and link

Right now only a basic scheduled mode in the Evse15118D20 module was supported.

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

D20 dynamic mode works now for IsoMux

Signed-off-by: Sebastian Lukas <sebastian.lukas@pionix.de>
@corneliusclaussen corneliusclaussen force-pushed the feat/adding-d20-dynamic-mode-support branch from 3ce818f to e9c7992 Compare October 25, 2024 09:32
@@ -46,7 +46,7 @@ libfsm:
# libiso15118
libiso15118:
git: https://github.com/EVerest/libiso15118.git
git_tag: v0.2.0
git_tag: a135d7d3cc62a2206b1128b267a388e209b3cd2c
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should point to a release before merge

@@ -71,7 +71,7 @@ libocpp:
# Josev
Josev:
git: https://github.com/EVerest/ext-switchev-iso15118.git
git_tag: 2024.9.0
git_tag: 4abfad56331332d56a87e40cc361744f87e65c39
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same as above

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants