New issue

Jump to bottom

[tools,ipgen] Improve VLNV renaming #26540

Closed

matutem opened this issue Mar 6, 2025 · 0 comments · Fixed by #26541

Contributor

matutem commented Mar 6, 2025

Description

IPGEN blocks have their fusesoc VLNV modified so they become top-specific. The renaming scheme needs to retain the information already existing in the VLNV and just add some extra tokens to uniquify the top-specific block. The current renaming scheme removes some information, like the library, and adds some irrelevant tokens, like 'opentitan'. In addition, we need stronger support for other block renaming needed to further uniquify block artifacts, essentially applying uniquified names to any core artifact.

matutem added a commit to matutem/opentitan that referenced this issue


          [tools,ipgen] Improve VLNV renaming

e379f24

Fixes lowRISC#26540

Signed-off-by: Guillermo Maturana <maturana@opentitan.org>

matutem mentioned this issue

[tools,ipgen] Improve VLNV renaming #26541

Merged

matutem added a commit to matutem/opentitan that referenced this issue


          [tools,ipgen] Improve VLNV renaming

e5329bf

Fixes lowRISC#26540

Signed-off-by: Guillermo Maturana <maturana@opentitan.org>

matutem added a commit to matutem/opentitan that referenced this issue


          [tools,ipgen] Improve VLNV renaming

97950ce

Change IpTemplateRendererBase._tplfunc_instance_vlnv per the new
VLNV scheme. This only needs the template name and the info in "self".
Remove the extra "prefix" parameter.
Change top core files to use the new naming.

Fixes lowRISC#26540

Signed-off-by: Guillermo Maturana <maturana@opentitan.org>

matutem added a commit to matutem/opentitan that referenced this issue


          [tools,ipgen] Improve VLNV renaming

4dab755

Change IpTemplateRendererBase._tplfunc_instance_vlnv per the new
VLNV scheme. This only needs the template name and the info in "self".
Remove the extra "prefix" parameter.
Change top core files to use the new naming.

Fixes lowRISC#26540

Signed-off-by: Guillermo Maturana <maturana@opentitan.org>

matutem added a commit to matutem/opentitan that referenced this issue


          [tools,ipgen] Improve VLNV renaming

73fd87d

- Change IpTemplateRendererBase._tplfunc_instance_vlnv per the new
  VLNV scheme. This only needs the template name and the info in "self".
  Remove the extra "prefix" parameter.
- Remove some unnecessary vlnv related ipgen parameters.
- Change top core files to use the new naming.

Fixes lowRISC#26540

Signed-off-by: Guillermo Maturana <maturana@opentitan.org>

matutem added a commit to matutem/opentitan that referenced this issue


          [tools,ipgen] Improve VLNV renaming

1894a95

- Change IpTemplateRendererBase._tplfunc_instance_vlnv per the new
  VLNV scheme. This only needs the template name and the info in "self".
  Remove the extra "prefix" parameter.
- Remove some unnecessary vlnv related ipgen parameters.
- Change top core files to use the new naming.

Fixes lowRISC#26540

Signed-off-by: Guillermo Maturana <maturana@opentitan.org>

matutem added a commit to matutem/opentitan that referenced this issue


          [tools,ipgen] Improve VLNV renaming

2d57342

- Change IpTemplateRendererBase._tplfunc_instance_vlnv per the new
  VLNV scheme. This only needs the template name and the info in "self".
  Remove the extra "prefix" parameter.
- Remove some unnecessary vlnv related ipgen parameters.
- Change top core files to use the new naming.

Fixes lowRISC#26540

Signed-off-by: Guillermo Maturana <maturana@opentitan.org>

matutem added a commit to matutem/opentitan that referenced this issue


          [tools,ipgen] Improve VLNV renaming

0de1417

- Change IpTemplateRendererBase._tplfunc_instance_vlnv per the new
  VLNV scheme. This only needs the template name and the info in "self".
  Remove the extra "prefix" parameter.
- Remove some unnecessary vlnv related ipgen parameters.
- Change top core files to use the new naming.

Fixes lowRISC#26540

Signed-off-by: Guillermo Maturana <maturana@opentitan.org>

matutem added a commit to matutem/opentitan that referenced this issue


          [tools,ipgen] Improve VLNV renaming

2d25076

- Change IpTemplateRendererBase._tplfunc_instance_vlnv per the new
  VLNV scheme. This only needs the template name and the info in "self".
  Remove the extra "prefix" parameter.
- Remove some unnecessary vlnv related ipgen parameters.
- Change top core files to use the new naming.

Fixes lowRISC#26540

Signed-off-by: Guillermo Maturana <maturana@opentitan.org>

matutem added a commit to matutem/opentitan that referenced this issue


          [tools,ipgen] Update README.md per recent changes

02c1efa

Update description of VLNV renaming and module uniquification.

Fixes lowRISC#26540

Signed-off-by: Guillermo Maturana <maturana@opentitan.org>

matutem added a commit to matutem/opentitan that referenced this issue


          [tools,ipgen] Improve VLNV renaming

6a53e50

- Change IpTemplateRendererBase._tplfunc_instance_vlnv per the new
  VLNV scheme. This only needs the template name and the info in "self".
  Remove the extra "prefix" parameter.
- Remove some unnecessary vlnv related ipgen parameters.
- Change top core files to use the new naming.

Fixes lowRISC#26540

Signed-off-by: Guillermo Maturana <maturana@opentitan.org>

matutem added a commit to matutem/opentitan that referenced this issue


          [tools,ipgen] Update README.md per recent changes

00a1021

Update description of VLNV renaming and module uniquification.

Fixes lowRISC#26540

Signed-off-by: Guillermo Maturana <maturana@opentitan.org>

matutem added a commit to matutem/opentitan that referenced this issue


          [tools,ipgen] Improve VLNV renaming

9df3d4d

- Change IpTemplateRendererBase._tplfunc_instance_vlnv per the new
  VLNV scheme. This only needs the template name and the info in "self".
  Remove the extra "prefix" parameter.
- Remove some unnecessary vlnv related ipgen parameters.
- Change top core files to use the new naming.

Fixes lowRISC#26540

Signed-off-by: Guillermo Maturana <maturana@opentitan.org>

matutem added a commit to matutem/opentitan that referenced this issue


          [tools,ipgen] Update README.md per recent changes

edf3706

Update description of VLNV renaming and module uniquification.

Fixes lowRISC#26540

Signed-off-by: Guillermo Maturana <maturana@opentitan.org>

matutem added a commit to matutem/opentitan that referenced this issue


          [tools,ipgen] Improve VLNV renaming

c7ddcbb

- Change IpTemplateRendererBase._tplfunc_instance_vlnv per the new
  VLNV scheme. This only needs the template name and the info in "self".
  Remove the extra "prefix" parameter.
- Remove some unnecessary vlnv related ipgen parameters.
- Change top core files to use the new naming.

Fixes lowRISC#26540

Signed-off-by: Guillermo Maturana <maturana@opentitan.org>

matutem added a commit to matutem/opentitan that referenced this issue


          [tools,ipgen] Update README.md per recent changes

7aab0ba

Update description of VLNV renaming and module uniquification.

Fixes lowRISC#26540

Signed-off-by: Guillermo Maturana <maturana@opentitan.org>

matutem added a commit to matutem/opentitan that referenced this issue


          [tools,ipgen] Update README.md per recent changes

67441d1

Update description of VLNV renaming and module uniquification.

Fixes lowRISC#26540

Signed-off-by: Guillermo Maturana <maturana@opentitan.org>

matutem closed this as completed in #26541

matutem closed this as completed in

c385a20

matutem added a commit that referenced this issue


          [tools,ipgen] Update README.md per recent changes

8190c68

Update description of VLNV renaming and module uniquification.

Fixes #26540

Signed-off-by: Guillermo Maturana <maturana@opentitan.org>

vogelpi pushed a commit to vogelpi/opentitan that referenced this issue


          [tools,ipgen] Improve VLNV renaming

7b9143c

- Change IpTemplateRendererBase._tplfunc_instance_vlnv per the new
  VLNV scheme. This only needs the template name and the info in "self".
  Remove the extra "prefix" parameter.
- Remove some unnecessary vlnv related ipgen parameters.
- Change top core files to use the new naming.

Fixes lowRISC#26540

Signed-off-by: Guillermo Maturana <maturana@opentitan.org>

vogelpi pushed a commit to vogelpi/opentitan that referenced this issue


          [tools,ipgen] Update README.md per recent changes

8c60bdc

Update description of VLNV renaming and module uniquification.

Fixes lowRISC#26540

Signed-off-by: Guillermo Maturana <maturana@opentitan.org>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment