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

fixed missing hartsel reference when determining HARTSELLEN #1002

Merged
merged 1 commit into from
Apr 18, 2024

Conversation

stefano-codasip
Copy link
Contributor

There is a paragraph in section 3.3 that explains how to determined HARTSELLEN. However, the word "hartsel" has been left out, and the sentence no longer makes sense. The fix is for adding "hartsel" back.

Copy link
Contributor

@JanMatCodasip JanMatCodasip left a comment

Choose a reason for hiding this comment

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

Thank you for filing this PR with the fix.

Copy link
Collaborator

@pdonahue-ventana pdonahue-ventana left a comment

Choose a reason for hiding this comment

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

Another Asciidoc conversion problem.

@stefano-codasip
Copy link
Contributor Author

@pdonahue-ventana this PR requires a further approval for the Workflow
image

@pdonahue-ventana
Copy link
Collaborator

I normally wait for @rtwfroody to do the final merge.

Copy link
Collaborator

@rtwfroody rtwfroody left a comment

Choose a reason for hiding this comment

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

Thank you!

@rtwfroody rtwfroody merged commit e3f0aac into riscv:main Apr 18, 2024
1 check passed
@stefano-codasip stefano-codasip deleted the fix/added_missing_hartsel branch April 19, 2024 09:01
@rtwfroody rtwfroody added 1.0 asciidoc asciidoc conversion labels May 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1.0 asciidoc asciidoc conversion
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants