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

Possible minor error on the documentation related with the ROM usage #243

Open
PabloUlises opened this issue May 31, 2024 · 0 comments
Open

Comments

@PabloUlises
Copy link

Describe the bug

I am not sure, but I think that I have found a minor error on the documentation related with the ROM usage, to be precise under “Command Line Usage”.

In the part where it says:


To run macros, it is suggested to use, for example:

cd OpenRAM/macros/rom_configs
make sky130_rom_1kbyte

If I blindly follow what it says, I receive a make error. That seems to be caused by the fact that make file is inside “OpenRAM/macros/” while the configuration is inside “OpenRAM/macros/rom_configs”.
Therefore, “make sky130_rom_1kbyte” command works on the “macros” folder but not on the “rom_configs” folder.

I think that it should say:

"
To run macros, it is suggested to use, for example:

cd OpenRAM/macros/
make sky130_rom_1kbyte

"

Again, I am not sure if it is an error on the documentation.

Version
Which commit are you using?
b6a6f12

Additional context
https://github.com/VLSIDA/OpenRAM/blob/stable/docs/source/basic_rom_usage.md#go-back

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

No branches or pull requests

1 participant