From: whitequark Date: Wed, 3 Jul 2019 13:25:12 +0000 (+0000) Subject: compat.fhdl.specials: use "sync" as default domain, not "sys". X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=c72e65db65d71a436f9f966ae5c0c12eba89d3ec;p=nmigen.git compat.fhdl.specials: use "sync" as default domain, not "sys". In compat.fhdl.module, we already default to "sync" as the default clocked domain. Using "sys" in memories only would be inconsistent and result in more bugs. --- diff --git a/nmigen/compat/fhdl/specials.py b/nmigen/compat/fhdl/specials.py index c18c14e..a98afbc 100644 --- a/nmigen/compat/fhdl/specials.py +++ b/nmigen/compat/fhdl/specials.py @@ -65,7 +65,7 @@ class Tristate(Elaboratable): class _MemoryPort(CompatModule): def __init__(self, adr, dat_r, we=None, dat_w=None, async_read=False, re=None, - we_granularity=0, mode=WRITE_FIRST, clock_domain="sys"): + we_granularity=0, mode=WRITE_FIRST, clock_domain="sync"): self.adr = adr self.dat_r = dat_r self.we = we @@ -86,7 +86,7 @@ def elaborate(self, platform): class CompatMemory(NativeMemory): @deprecated("instead of `get_port()`, use `read_port()` and `write_port()`") def get_port(self, write_capable=False, async_read=False, has_re=False, we_granularity=0, - mode=WRITE_FIRST, clock_domain="sys"): + mode=WRITE_FIRST, clock_domain="sync"): if we_granularity >= self.width: warnings.warn("do not specify `we_granularity` greater than memory width, as it " "is a hard error in non-compatibility mode",