From e5aa6a01b5bbe1675c3aa8872e1203413ded83d1 Mon Sep 17 00:00:00 2001 From: Luca Bertaccini <55843305+lucabertaccini@users.noreply.github.com> Date: Fri, 27 Sep 2024 11:52:27 +0200 Subject: [PATCH] Fix illegal Verilog assignment (#15) --- docs/CHANGELOG-PULP.md | 5 +++++ .../C910_RTL_FACTORY/gen_rtl/vfdsu/rtl/ct_vfdsu_round.v | 2 +- .../0001-Add-FP16ALT-support-to-THMULTI-DivSqrt-unit.patch | 2 +- 3 files changed, 7 insertions(+), 2 deletions(-) diff --git a/docs/CHANGELOG-PULP.md b/docs/CHANGELOG-PULP.md index 94d245be..17ef11f5 100644 --- a/docs/CHANGELOG-PULP.md +++ b/docs/CHANGELOG-PULP.md @@ -7,6 +7,11 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/) a In this sense, we interpret the "Public API" of a hardware module as its port/parameter list. Versions of the IP in the same major relase are "pin-compatible" with each other. Minor relases are permitted to add new parameters as long as their default bindings ensure backwards compatibility. +## [pulp-v0.2.3] - 2024-09-27 + +### Fix +- Fix illegal Verilog `'0` + ## [pulp-v0.2.2] - 2024-06-24 ### Added diff --git a/vendor/openc910/C910_RTL_FACTORY/gen_rtl/vfdsu/rtl/ct_vfdsu_round.v b/vendor/openc910/C910_RTL_FACTORY/gen_rtl/vfdsu/rtl/ct_vfdsu_round.v index cb3dc8e3..69462eb9 100644 --- a/vendor/openc910/C910_RTL_FACTORY/gen_rtl/vfdsu/rtl/ct_vfdsu_round.v +++ b/vendor/openc910/C910_RTL_FACTORY/gen_rtl/vfdsu/rtl/ct_vfdsu_round.v @@ -763,7 +763,7 @@ case(vfdsu_ex3_bfloat_expnt_rst[8:0]) 9'h17a:begin qt_result_bfloat_denorm_for_round[10:0] = {total_qt_rt_58[56:46]}; //-93 -6 bfloat_denorm_lst_frac = 1'b0; end//-1022 1 - default: begin qt_result_bfloat_denorm_for_round[10:0] = '0; + default: begin qt_result_bfloat_denorm_for_round[10:0] = 11'b0; bfloat_denorm_lst_frac = 1'b0; end//-1022 1 endcase diff --git a/vendor/patches/openc910/0001-Add-FP16ALT-support-to-THMULTI-DivSqrt-unit.patch b/vendor/patches/openc910/0001-Add-FP16ALT-support-to-THMULTI-DivSqrt-unit.patch index 7d1ce903..fab95f9d 100644 --- a/vendor/patches/openc910/0001-Add-FP16ALT-support-to-THMULTI-DivSqrt-unit.patch +++ b/vendor/patches/openc910/0001-Add-FP16ALT-support-to-THMULTI-DivSqrt-unit.patch @@ -817,7 +817,7 @@ index 6eece52..a419289 100644 + 9'h17a:begin qt_result_bfloat_denorm_for_round[10:0] = {total_qt_rt_58[56:46]}; //-93 -6 + bfloat_denorm_lst_frac = 1'b0; + end//-1022 1 -+ default: begin qt_result_bfloat_denorm_for_round[10:0] = '0; ++ default: begin qt_result_bfloat_denorm_for_round[10:0] = 11'b0; + bfloat_denorm_lst_frac = 1'b0; + end//-1022 1 +endcase