Package Macro potential issue

[nikola-kovacevic-ns]

Hello,

Caliptra in package definitions has following macro:

`ifndef PRIM_ASSERT
`define CALIPTRA_PRIM_ASSERT

Is this intentional? Shouldn't it be like this:

`ifndef CALIPTRA_PRIM_ASSERT
`define CALIPTRA_PRIM_ASSERT

It is causing problems when integrating caliptra with other cores wich have the same lowrisc ibex base RTL code.

Thanks in advance,
Kovacevic Nikola

[calebofearth]

Hi, this is a duplicate of #332, which is a follow-up from #56.
This is intended to be exclusively a reference to CALIPTRA_PRIM_ASSERT as you pointed out and will be fixed for the 2.0 release.