Skip to content

Commit

Permalink
soc: change default CSR bus data-width to 32.
Browse files Browse the repository at this point in the history
A CSR bus data-width of 32 has been validated on very various design and is
now recommended. It provides better performance without impacting resource
usage (even in iCE40).
  • Loading branch information
enjoy-digital committed Oct 7, 2020
1 parent 4f30a5b commit 898743c
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 3 deletions.
1 change: 1 addition & 0 deletions CHANGES
Expand Up @@ -27,6 +27,7 @@
--------------------------
- BIOS: commands have been renamed/reorganized.
- LiteDRAM: rdcmdphase/wrcmdphase no longer exposed.
- CSR: change default csr_data_width from 8 to 32.

[> 2020.08, planned for July 2020
---------------------------------
Expand Down
2 changes: 1 addition & 1 deletion litex/soc/doc/csr.py
Expand Up @@ -63,7 +63,7 @@ def __init__(self, name, address,
f.description = self.trim(f.description)

class DocumentedCSRRegion:
def __init__(self, name, region, module=None, submodules=[], csr_data_width=8):
def __init__(self, name, region, module=None, submodules=[], csr_data_width=32):
self.name = name
self.origin = region.origin
self.busword = region.busword
Expand Down
4 changes: 2 additions & 2 deletions litex/soc/integration/soc_core.py
Expand Up @@ -80,7 +80,7 @@ def __init__(self, platform, clk_freq,
integrated_main_ram_size = 0,
integrated_main_ram_init = [],
# CSR parameters
csr_data_width = 8,
csr_data_width = 32,
csr_address_width = 14,
csr_paging = 0x800,
csr_ordering = "big",
Expand Down Expand Up @@ -282,7 +282,7 @@ def soc_core_args(parser):
help="size/enable the integrated main RAM")
# CSR parameters
parser.add_argument("--csr-data-width", default=None, type=auto_int,
help="CSR bus data-width (8 or 32, default=8)")
help="CSR bus data-width (8 or 32, default=32)")
parser.add_argument("--csr-address-width", default=14, type=auto_int,
help="CSR bus address-width")
parser.add_argument("--csr-paging", default=0x800, type=auto_int,
Expand Down

0 comments on commit 898743c

Please sign in to comment.