Bump Snitch cluster to latest revision

[Paolo309]

This PR updates the Snitch cluster to pulp-platform/snitch_cluster@5b2fccd.

Main changes

• Updated Bender yaml and lock files to match upstream dependencies.
  • Notable change: switched axi to https://github.com/colluca/axi (branch multicast).
• Updated chimera_pkg to match the current Snitch cluster's defaults.
• Added TCDM-related configuration to match the updated Snitch cluster.
• Updated chimera_cluster, adding connections to new Snitch cluster's ports.

Verification

• All tests under sw/tests pass in vsim.

[Lore0599]

Thanks a lot for the valuable work.
Overall LGTM, I left some small comments.

[Lore0599]

These parameters are not related to the SoC. but are mainly related to the cluster. Maybe it might make sense to move them somewhere else?

Also we are not using the external TCDM ports, so I think that actually those parameters can be removed in general and when instantiating the Snitch cluster we don't specify them and use the default types.

[Lore0599]

In general, since we are not using the new ports, we can avoid generating intermediate types. If we do not specify the type at the interface, the default types will be used. Then we can simply leave the port unconnected, as you already did.
What do you think?

[Paolo309]

Yeah, I agree it would be cleaner the way you suggest. However, I've tried not defining the tcdm_dma_*_t types, and it can't compile. The default type for them in snitch_cluster module is logic, which doesn't seem to be supported by snitch_tcdm_interconnect and other modules down the hierarchy.

I don't think we can avoid defining tcdm_dma_*_t. In that case, since it's cluster-specific, probably it's better to move the definition of TcdmSize and TcdmAddrWidth inside chimera_cluster, right?

[Lore0599]

okay, then it makes sense to open an issue on Snitch.
As a temporary fix to merge this PR the parameters can be moved inside the chimera_cluster to make sure that it is cluster specific