commit 53b946bbb6c87072d6f57697155f56fd41372a4b
author Eunchan Kim <eunchan@opentitan.org> Thu Apr 15 12:27:57 2021 -0700
committer Eunchan Kim <eunchan@opentitan.org> Thu Apr 15 17:26:55 2021 -0700
tree e1bfc06c630811a12457b9ce15fd34acfdae3271
parent 9ffffe099c3605b27e21993dd8cb857e717566f7

[topgen] Fix enum type warnings

enum for top package did not define any type. Change them to `int
unsigned`.

Signed-off-by: Eunchan Kim <eunchan@opentitan.org>

hw/top_earlgrey/rtl/autogen/top_earlgrey_pkg.sv

diff --git a/hw/top_earlgrey/rtl/autogen/top_earlgrey_pkg.sv b/hw/top_earlgrey/rtl/autogen/top_earlgrey_pkg.sv
index 4037c0f..42e937e 100644
--- a/hw/top_earlgrey/rtl/autogen/top_earlgrey_pkg.sv
+++ b/hw/top_earlgrey/rtl/autogen/top_earlgrey_pkg.sv
@@ -482,7 +482,7 @@
   } pwr_dom_e;
 
   // Enumeration for MIO signals on the top-level.
-  typedef enum {
+  typedef enum int unsigned {
     MioInGpioGpio0 = 0,
     MioInGpioGpio1 = 1,
     MioInGpioGpio2 = 2,
@@ -619,7 +619,7 @@
   } mio_out_e;
 
   // Enumeration for DIO signals, used on both the top and chip-levels.
-  typedef enum {
+  typedef enum int unsigned {
     DioSpiHost0Sd0 = 0,
     DioSpiHost0Sd1 = 1,
     DioSpiHost0Sd2 = 2,
@@ -650,7 +650,7 @@
   // Raw MIO/DIO input array indices on chip-level.
   // TODO: Does not account for target specific stubbed/added pads.
   // Need to make a target-specific package for those.
-  typedef enum {
+  typedef enum int unsigned {
     MioPadIoa0 = 0,
     MioPadIoa1 = 1,
     MioPadIoa2 = 2,
@@ -701,7 +701,7 @@
     MioPadCount
   } mio_pad_e;
 
-  typedef enum {
+  typedef enum int unsigned {
     DioPadPorN = 0,
     DioPadUsbP = 1,
     DioPadUsbN = 2,

util/topgen/templates/toplevel_pkg.sv.tpl

diff --git a/util/topgen/templates/toplevel_pkg.sv.tpl b/util/topgen/templates/toplevel_pkg.sv.tpl
index 234c618..a25d4fd 100644
--- a/util/topgen/templates/toplevel_pkg.sv.tpl
+++ b/util/topgen/templates/toplevel_pkg.sv.tpl
@@ -50,7 +50,7 @@
   } pwr_dom_e;
 
   // Enumeration for MIO signals on the top-level.
-  typedef enum {
+  typedef enum int unsigned {
 % for sig in top["pinmux"]["ios"]:
   % if sig['type'] in ['inout', 'input'] and sig['connection'] == 'muxed':
     ${lib.get_io_enum_literal(sig, 'mio_in')} = ${sig['glob_idx']},
@@ -73,7 +73,7 @@
   } mio_out_e;
 
   // Enumeration for DIO signals, used on both the top and chip-levels.
-  typedef enum {
+  typedef enum int unsigned {
 % for sig in top["pinmux"]["ios"]:
   % if sig['connection'] != 'muxed':
     ${lib.get_io_enum_literal(sig, 'dio')} = ${sig['glob_idx']},
@@ -88,7 +88,7 @@
   // Raw MIO/DIO input array indices on chip-level.
   // TODO: Does not account for target specific stubbed/added pads.
   // Need to make a target-specific package for those.
-  typedef enum {
+  typedef enum int unsigned {
 % for pad in top["pinout"]["pads"]:
   % if pad["connection"] == "muxed":
     ${lib.Name.from_snake_case("mio_pad_" + pad["name"]).as_camel_case()} = ${pad["idx"]},
@@ -97,7 +97,7 @@
     ${lib.Name.from_snake_case("mio_pad_count").as_camel_case()}
   } mio_pad_e;
 
-  typedef enum {
+  typedef enum int unsigned {
 % for pad in top["pinout"]["pads"]:
   % if pad["connection"] != "muxed":
     ${lib.Name.from_snake_case("dio_pad_" + pad["name"]).as_camel_case()} = ${pad["idx"]},