bslathi19/PeakRDL-regblock
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

remove dead-code

Browse Source
This commit is contained in:
Alex Mykyta
2025-03-06 22:08:47 -08:00
parent c53b11cf28
commit f882e155d1
1 changed files with 3 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
src/peakrdl_regblock/validate_design.py
View File

@@ -1,4 +1,4 @@
from typing import TYPE_CHECKING, Optional, List from typing import TYPE_CHECKING, Optional, List, Union
from systemrdl.walker import RDLListener, RDLWalker, WalkerAction from systemrdl.walker import RDLListener, RDLWalker, WalkerAction
from systemrdl.rdltypes import PropertyReference from systemrdl.rdltypes import PropertyReference
@@ -97,7 +97,7 @@ class DesignValidator(RDLListener):
def enter_Addrmap(self, node: AddrmapNode) -> None: def enter_Addrmap(self, node: AddrmapNode) -> None:
self._check_sharedextbus(node) self._check_sharedextbus(node)
def _check_sharedextbus(self, node: Node) -> None: def _check_sharedextbus(self, node: Union[RegfileNode, AddrmapNode]) -> None:
if node.get_property('sharedextbus'): if node.get_property('sharedextbus'):
self.msg.error( self.msg.error(
"This exporter does not support enabling the 'sharedextbus' property yet.", "This exporter does not support enabling the 'sharedextbus' property yet.",
@@ -114,7 +114,7 @@ class DesignValidator(RDLListener):
if accesswidth != self.exp.cpuif.data_width: if accesswidth != self.exp.cpuif.data_width:
self.msg.error( self.msg.error(
f"Multi-word registers that have an accesswidth ({accesswidth}) " f"Multi-word registers that have an accesswidth ({accesswidth}) "
"that is inconsistent with this regblock's CPU bus width " "that are inconsistent with this regblock's CPU bus width "
f"({self.exp.cpuif.data_width}) are not supported.", f"({self.exp.cpuif.data_width}) are not supported.",
node.inst.inst_src_ref node.inst.inst_src_ref
) )
@@ -128,15 +128,6 @@ class DesignValidator(RDLListener):
and (node.lsb // parent_accesswidth) != (node.msb // parent_accesswidth) and (node.lsb // parent_accesswidth) != (node.msb // parent_accesswidth)
): ):
# field spans multiple sub-words # field spans multiple sub-words
if node.external:
# External fields that span multiple subwords is not supported
self.msg.error(
"External fields that span multiple software-accessible "
"subwords are not supported.",
node.inst.inst_src_ref
)
# Skip remaining validation rules for external fields
return
if node.is_sw_writable and not node.parent.get_property('buffer_writes'): if node.is_sw_writable and not node.parent.get_property('buffer_writes'):
# ... and is writable without the protection of double-buffering # ... and is writable without the protection of double-buffering