From 273a3ea15df5a81b306755adff7e1cac08c165c7 Mon Sep 17 00:00:00 2001 From: "Gabriel L. Somlo" Date: Wed, 29 May 2019 10:07:43 -0400 Subject: [PATCH] soc/integration/cpu_interface: improve code legibility Factor out code appearing in both branches of an if/else. --- litex/soc/integration/cpu_interface.py | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/litex/soc/integration/cpu_interface.py b/litex/soc/integration/cpu_interface.py index c80dbccd6..d20f06ebe 100644 --- a/litex/soc/integration/cpu_interface.py +++ b/litex/soc/integration/cpu_interface.py @@ -140,12 +140,9 @@ def get_csr_header(regions, constants, with_access_functions=True, with_shadow_b for name, origin, busword, obj in regions: if not with_shadow_base: origin &= (~shadow_base) - if isinstance(obj, Memory): - r += "\n/* "+name+" */\n" - r += "#define CSR_"+name.upper()+"_BASE "+hex(origin)+"L\n" - else: - r += "\n/* "+name+" */\n" - r += "#define CSR_"+name.upper()+"_BASE "+hex(origin)+"L\n" + r += "\n/* "+name+" */\n" + r += "#define CSR_"+name.upper()+"_BASE "+hex(origin)+"L\n" + if not isinstance(obj, Memory): for csr in obj: nr = (csr.size + busword - 1)//busword r += _get_rw_functions_c(name + "_" + csr.name, origin, nr, busword, isinstance(csr, CSRStatus), with_access_functions)