fusesoc icon indicating copy to clipboard operation
fusesoc copied to clipboard

Published 20 hours ago • verified publisher icon olofk

Integrate openTitan `virtual` keyword (proposal)

Open joennlae opened this issue 3 years ago • 1 comments

@imphil I tried to rebase this such that this addition is integrated into the current version of fusesoc. I got stuck during rebasing as I do not have enough experience with all the code. Is it possible to rebase that into the master branch here 👍 ? If that could be rebased, one does not have to use this (lowRISC/ot) branch to integrate lowRISC IP.

joennlae avatar Aug 03 '22 07:08 joennlae

The ot branch contains various things.

  • Support for the virtual keyword. That's already merged in upstream FuseSoC at https://github.com/olofk/fusesoc/pull/524.
  • Support for dependency resolution in generators. That's https://github.com/olofk/fusesoc/pull/391, and has significant design problems which require this feature to be redesigned. It works for OKish OpenTitan, but it's not ready for general use.

imphil avatar Aug 03 '22 15:08 imphil

Thank you for clarifying 👍

joennlae avatar Aug 08 '22 16:08 joennlae