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

Fix literal width #827

Merged
merged 1 commit into from
Apr 6, 2023
Merged

Fix literal width #827

merged 1 commit into from
Apr 6, 2023

Conversation

mole99
Copy link
Contributor

@mole99 mole99 commented Apr 6, 2023

This commit fixes a minor issue with the width of a literal that triggered a warning with the sv2v converter.

@Silabs-ArjanB
Copy link
Contributor

Hello @mole99 Thank you very much for your contribution! Before merging it I would kindly like to ask you to sign the so-called Eclipse Contributor Agreement (ECA). You can find more information on https://www.eclipse.org/projects/handbook/#contributing-eca and https://www.eclipse.org/legal/ECA.php.

Best regards,
Arjan

@Silabs-ArjanB Silabs-ArjanB added the Component:RTL For issues in the RTL (e.g. for files in the rtl directory) label Apr 6, 2023
@mole99
Copy link
Contributor Author

mole99 commented Apr 6, 2023

Hi Arjan, I created an account and signed the ECA:

grafik

But it seems to not recognize my ECA sign-off status. I made sure to use the same e-mail address.

Do you have a suggestion what I could try? Thanks!

@Silabs-ArjanB
Copy link
Contributor

Did you do the commit using the -s option for git commit?

Signed-off-by: mole99 <[email protected]>
@mole99
Copy link
Contributor Author

mole99 commented Apr 6, 2023

Thanks, that was the issue :D

@Silabs-ArjanB
Copy link
Contributor

Thank you very much @mole99 !

@Silabs-ArjanB Silabs-ArjanB merged commit 56d1332 into openhwgroup:master Apr 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component:RTL For issues in the RTL (e.g. for files in the rtl directory)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants