From 01754a82c8f86e28271bd0845fe660aaf8a13117 Mon Sep 17 00:00:00 2001 From: Gabriel Somlo Date: Thu, 18 Aug 2022 16:50:41 -0400 Subject: [PATCH] integration/soc: fix sata irq initialization The first argument to `self.irq.add()` should match the name of the `EventManager()` object being added, i.e., "sata_irq" rather than just plain "sata". This is necessary for interrupt signals to be asserted as intended. Signed-off-by: Gabriel Somlo --- litex/soc/integration/soc.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/litex/soc/integration/soc.py b/litex/soc/integration/soc.py index a9c7f5eee..ae4af627a 100644 --- a/litex/soc/integration/soc.py +++ b/litex/soc/integration/soc.py @@ -1906,7 +1906,7 @@ class LiteXSoC(SoC): if "write" in mode: self.comb += self.sata_irq.mem2sector_dma.trigger.eq(self.sata_mem2sector.irq) if self.irq.enabled: - self.irq.add("sata", use_loc_if_exists=True) + self.irq.add("sata_irq", use_loc_if_exists=True) # Timing constraints. self.platform.add_period_constraint(self.sata_phy.crg.cd_sata_tx.clk, 1e9/sata_clk_freq)