Skip to content

Conversation

@kathrynsuraya
Copy link

Kathryn Kalouf - PME supporting AM64/AM243 with Sitara. Updating the introduction and supported features.

Kathryn Kalouf - PME supporting AM64/AM243 with Sitara. Updating the introduction and supported features. 

Signed-off-by: kathrynsuraya <[email protected]>
Copy link
Member

@cshilwant cshilwant left a comment

Choose a reason for hiding this comment

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

Copy link
Member

@cshilwant cshilwant left a comment

Choose a reason for hiding this comment

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

"Kathryn Kalouf -" in commit message is not intended to the content added in the PR

…work/HSR_Offload.rst

Co-authored-by: Chirag Shilwant <[email protected]>
Signed-off-by: kathrynsuraya <[email protected]>
@kathrynsuraya
Copy link
Author

This is my first time doing this. In order to change the commit message, what would be the next steps? Do i submit another pull request? And is this all that needs to be resolved?

@cshilwant
Copy link
Member

@kathrynsuraya You can use the same PR and git commot --amend inorder to update the commit message and force push in same PR.

I see two commits in this PR now. Can you also make it into a single commit with updated commit message

Copy link
Member

@StaticRocket StaticRocket left a comment

Choose a reason for hiding this comment

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

Otherwise, this seems fine. Please squash your commits and change the subject to follow conventionalcommits (just change it to something like "feat: describe pru issg offloading")


HSR framework in Linux allows offloading below functionalities to the device:

• hsr-fwd-offload: For forwarding HSR frames one port to another i.e. port-to-port forwarding
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
hsr-fwd-offload: For forwarding HSR frames one port to another i.e. port-to-port forwarding
- hsr-fwd-offload: For forwarding HSR frames one port to another i.e. port-to-port forwarding

Simply using a - for list items creates a sphinx list, which adjusts formatting automatically based on the output document.

https://www.sphinx-doc.org/en/master/usage/restructuredtext/basics.html#lists-and-quote-like-blocks

@praneethbajjuri
Copy link
Collaborator

@kathrynsuraya address the review comments and push the update on this pr.

@kathrynsuraya
Copy link
Author

@praneethbajjuri Hi, because I am not familiar with github, I would have to download git to my computer to do this. I was hoping I could be able to do this on the web. This will take me some time.

@praneethbajjuri
Copy link
Collaborator

marking as closed. submitter will send a new PR later.

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.

8 participants