Skip to content

Commit

Permalink
[rv_core_ibex, top_earlgrey] Enable SecureIbex for CW340
Browse files Browse the repository at this point in the history
As the `SecureIbex` configuration is the one we use for the
Earl Grey ASIC, we also should enable it for the FPGA boards.

Due to resource constraints, this is only possible for the CW340
but not the CW310.

Resource utilization stays still will below 70%.

Closes #25137

Signed-off-by: Pascal Nasahl <[email protected]>
(cherry picked from commit ce82d80)
  • Loading branch information
nasahlpa authored and engdoreis committed Dec 9, 2024
1 parent 1767bd7 commit a8b52f6
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 3 deletions.
2 changes: 1 addition & 1 deletion hw/top_earlgrey/rtl/autogen/chip_earlgrey_cw310.sv
Original file line number Diff line number Diff line change
Expand Up @@ -1086,9 +1086,9 @@ module chip_earlgrey_cw310 #(
.KeymgrKmacEnMasking(0),
.SecKmacCmdDelay(0),
.SecKmacIdleAcceptSwMsg(1'b0),
.RvCoreIbexSecureIbex(0),
.RomCtrlBootRomInitFile(BootRomInitFile),
.RvCoreIbexRegFile(ibex_pkg::RegFileFPGA),
.RvCoreIbexSecureIbex(0),
.SramCtrlMainInstrExec(1),
.PinmuxAonTargetCfg(PinmuxTargetCfg)
) top_earlgrey (
Expand Down
2 changes: 1 addition & 1 deletion hw/top_earlgrey/rtl/autogen/chip_earlgrey_cw340.sv
Original file line number Diff line number Diff line change
Expand Up @@ -1070,9 +1070,9 @@ module chip_earlgrey_cw340 #(
.KmacEnMasking(1),
.KmacSwKeyMasked(1),
.KeymgrKmacEnMasking(1),
.RvCoreIbexSecureIbex(1),
.RomCtrlBootRomInitFile(BootRomInitFile),
.RvCoreIbexRegFile(ibex_pkg::RegFileFPGA),
.RvCoreIbexSecureIbex(0),
.SramCtrlMainInstrExec(1),
.PinmuxAonTargetCfg(PinmuxTargetCfg)
) top_earlgrey (
Expand Down
4 changes: 3 additions & 1 deletion util/topgen/templates/chiplevel.sv.tpl
Original file line number Diff line number Diff line change
Expand Up @@ -1165,6 +1165,7 @@ module chip_${top["name"]}_${target["name"]} #(
.SecAesAllowForcingMasks(1'b1),
.SecAesSkipPRNGReseeding(1'b1),
.UsbdevStub(1'b1),
.RvCoreIbexSecureIbex(0),
% else:
.SecAesMasking(1'b0),
.SecAesSBoxImpl(aes_pkg::SBoxImplLut),
Expand All @@ -1185,16 +1186,17 @@ module chip_${top["name"]}_${target["name"]} #(
.KmacEnMasking(1),
.KmacSwKeyMasked(1),
.KeymgrKmacEnMasking(1),
.RvCoreIbexSecureIbex(1),
% elif target["name"] == "cw310":
.KmacEnMasking(0),
.KmacSwKeyMasked(1),
.KeymgrKmacEnMasking(0),
.SecKmacCmdDelay(0),
.SecKmacIdleAcceptSwMsg(1'b0),
.RvCoreIbexSecureIbex(0),
% endif
.RomCtrlBootRomInitFile(BootRomInitFile),
.RvCoreIbexRegFile(ibex_pkg::RegFileFPGA),
.RvCoreIbexSecureIbex(0),
.SramCtrlMainInstrExec(1),
.PinmuxAonTargetCfg(PinmuxTargetCfg)
) top_${top["name"]} (
Expand Down

0 comments on commit a8b52f6

Please sign in to comment.