-
Notifications
You must be signed in to change notification settings - Fork 78
feat(linux): Add initial documentation for EV Charging User Guide #618
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
base: master
Are you sure you want to change the base?
Conversation
|
New warnings found with rstcheck: |
cshilwant
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are few valid warnings from vale which needs to be resolved - https://github.com/TexasInstruments/processor-sdk-doc/pull/618/files
source/devices/AM62LX/linux/Release_Specific_Yocto_layer_Configuration.rst
Outdated
Show resolved
Hide resolved
|
Also, general comment about https://github.com/TexasInstruments/processor-sdk-doc/blob/master/CONTRIBUTING.md#indentation-and-whitespace |
|
New warnings found with rstcheck: |
e37d9c2 to
64bd6af
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Better to highlight this EV Charging User Guide under Key Release References for AM62L [0]
|
New warnings found with rstcheck: |
Add initial documentation for building EV charging WIC image on AM62L using yocto. Signed-off-by: Yogesh Hegde <y-hegde@ti.com>
StaticRocket
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some nitpicks about use of parsed literals vs inline code, but otherwise this is fine
| This documentation assumes you have an existing EV charging software stack ready for integration. If you do not have a charging stack, | ||
| consider evaluating `EVerest`_, an open-source EV charging software platform. | ||
|
|
||
| ---- |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What's up with these line breaks? I guess they are fine, but this would be the first document to start using them.
|
|
||
| Follow the steps given below to add `EVerest`_ stack into the WIC image. | ||
|
|
||
| 1. Edit sources/meta-tisdk/meta-ti-foundational/recipes-core/images/tisdk-evse-image.bb and add everest-core & tida-010939-everest-config packages: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| 1. Edit sources/meta-tisdk/meta-ti-foundational/recipes-core/images/tisdk-evse-image.bb and add everest-core & tida-010939-everest-config packages: | |
| 1. Edit :file:`sources/meta-tisdk/meta-ti-foundational/recipes-core/images/tisdk-evse-image.bb` and add ``everest-core`` & ``tida-010939-everest-config`` packages: |
| .. important:: | ||
|
|
||
| Find the existing `IMAGE_INSTALL:append` section in the file. **Do not remove or modify the existing packages**. Add the two new packages to the list. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| Find the existing `IMAGE_INSTALL:append` section in the file. **Do not remove or modify the existing packages**. Add the two new packages to the list. | |
| Find the existing ``IMAGE_INSTALL:append`` section in the file. **Do not remove or modify the existing packages**. Add the two new packages to the list. |
| $ systemctl status everest | ||
| If `EVerest`_ is added then the output should show `Active: active (running)`, If it shows `Unit everest.service could not be found.` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| If `EVerest`_ is added then the output should show `Active: active (running)`, If it shows `Unit everest.service could not be found.` | |
| If `EVerest`_ is added then the output should show ``Active: active (running)``, If it shows ``Unit everest.service could not be found.`` |
Add initial documentation for building EV charging WIC image on AM62L using yocto.
Preview can be found here - https://yogeshhegde.github.io/processor-sdk-doc/processor-sdk-linux-AM62LX/esd/docs/master/linux/Demo_User_Guides/EV_charging_Demo_User_Guide.html