search

ucb-bar / chipyard

An Agile RISC-V SoC Design Framework with in-order cores, out-of-order cores, accelerators, and more
https://chipyard.readthedocs.io/en/stable/
BSD 3-Clause "New" or "Revised" License
1.67k stars 656 forks source link

how to combine hwacha and gemmini in the same tile #531

Closed sq4693 closed 4 years ago

sq4693 commented 4 years ago

HI,

my CONFIG is to put hwacha and gemmini in the same tile, but I can not generate the hwacha RTL, How to solve this, thanks.

class GemminiHwachaRocketConfig extends Config( new chipyard.iobinders.WithUARTAdapter ++ new chipyard.iobinders.WithTieOffInterrupts ++ new chipyard.iobinders.WithBlackBoxSimMem ++ new chipyard.iobinders.WithTiedOffDebug ++ new chipyard.iobinders.WithSimSerial ++ new testchipip.WithTSI ++ new chipyard.config.WithNoGPIO ++ new chipyard.config.WithBootROM ++ new chipyard.config.WithUART ++ new chipyard.config.WithL2TLBs(1024) ++ new gemmini.DefaultGemminiConfig ++ new hwacha.DefaultHwachaConfig ++ // use Hwacha vector accelerator new freechips.rocketchip.subsystem.WithNoMMIOPort ++ new freechips.rocketchip.subsystem.WithNoSlavePort ++ new freechips.rocketchip.subsystem.WithInclusiveCache ++ new freechips.rocketchip.subsystem.WithNExtTopInterrupts(0) ++ new freechips.rocketchip.subsystem.WithNBigCores(1) ++ new freechips.rocketchip.system.BaseConfig)

jerryz123 commented 4 years ago

Can you clarify what error you are seeing?

jerryz123 commented 4 years ago

The error was that the config fragments defined in Gemmini and Hwacha overrode the list of accelerators, instead of appending to it. The fixes are here: https://github.com/ucb-bar/gemmini/pull/18/commits/ef0276086504c09f3cea2a30025f85ce999b3458 https://github.com/ucb-bar/hwacha/pull/19/commits/9cfcd4ddeda386536f100c22609b318079b64cec

You can apply these patches to the local Gemmini and Hwacha repos you are using. The fix will appear in Chipyard master in a future release.

sq4693 commented 4 years ago

Yes,i found its been overrided. and i have sovle it. Thanks!

sq4693 commented 4 years ago

Yes,i found its been overrided. and i have sovle it. Thanks!