_PepcTpmi: Print feature ID in hex

Signed-off-by: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>

Author: dedekind

docs/guide-tpmi.md

@@ -216,25 +216,25 @@ Here is how to list all available TPMI features on a Granite Rapids system.
 ```bash
 $ pepc tpmi ls
  Supported TPMI features
-- rapl (0): Running Average Power Limit (RAPL) reporting and control
-- ufs (2): Processor uncore (fabric) monitoring and control
-- sst (5): Intel Speed Select Technology (SST) control
-- tpmi_info (129): TPMI Info Registers
+- rapl (0x00): Running Average Power Limit (RAPL) reporting and control
+- ufs (0x02): Processor uncore (fabric) monitoring and control
+- sst (0x05): Intel Speed Select Technology (SST) control
+- tpmi_info (0x81): TPMI Info Registers
 ```
 
 Check the topology of TPMI devices:
 
 ```bash
 $ pepc tpmi ls --topology
 Supported TPMI features
-- rapl (0): Running Average Power Limit (RAPL) reporting and control
+- rapl (0x00): Running Average Power Limit (RAPL) reporting and control
   - PCI address: 0000:00:03.1
     Package: 0
     Instances: 0
   - PCI address: 0000:80:03.1
     Package: 1
     Instances: 0
-- ufs (2): Processor uncore (fabric) monitoring and control
+- ufs (0x02): Processor uncore (fabric) monitoring and control
   - PCI address: 0000:00:03.1
     Package: 0
     - Instance: 0
@@ -257,13 +257,13 @@ TPMI spec directories information:
   Platform Name: Granite Rapids Xeon
   Spec Sub-directory Path: /home/dedekind/git/pepc/pepcdata/tpmi/gnr
 TPMI spec files:
-- rapl (0): Running Average Power Limit (RAPL) reporting and control
+- rapl (0x00): Running Average Power Limit (RAPL) reporting and control
   Spec file: /home/dedekind/git/pepc/pepcdata/tpmi/gnr/rapl.yml
-- ufs (2): Processor uncore (fabric) monitoring and control
+- ufs (0x02): Processor uncore (fabric) monitoring and control
   Spec file: /home/dedekind/git/pepc/pepcdata/tpmi/gnr/ufs.yml
-- sst (5): Intel Speed Select Technology (SST) control
+- sst (0x05): Intel Speed Select Technology (SST) control
   Spec file: /home/dedekind/git/pepc/pepcdata/tpmi/gnr/sst.yml
-- tpmi_info (129): TPMI Info Registers
+- tpmi_info (0x81): TPMI Info Registers
   Spec file: /home/dedekind/git/pepc/pepcdata/tpmi/gnr/tpmi_info.yml
 ```
 
@@ -491,13 +491,13 @@ TPMI spec directories information:
   Platform Name: Granite Rapids Xeon
   Spec Sub-directory Path: /home/dedekind/git/pepc/pepcdata/tpmi/gnr
 TPMI spec files:
-- rapl (0): Running Average Power Limit (RAPL) reporting and control
+- rapl (0x00): Running Average Power Limit (RAPL) reporting and control
   Spec file: /home/dedekind/git/pepc/pepcdata/tpmi/gnr/rapl.yml
-- ufs (2): Processor uncore (fabric) monitoring and control
+- ufs (0x02): Processor uncore (fabric) monitoring and control
   Spec file: /home/dedekind/git/pepc/pepcdata/tpmi/gnr/ufs.yml
-- sst (5): Intel Speed Select Technology (SST) control
+- sst (0x05): Intel Speed Select Technology (SST) control
   Spec file: /home/dedekind/git/pepc/pepcdata/tpmi/gnr/sst.yml
-- tpmi_info (129): TPMI Info Registers
+- tpmi_info (0x81): TPMI Info Registers
   Spec file: /home/dedekind/git/pepc/pepcdata/tpmi/gnr/tpmi_info.yml
 ```
 
@@ -539,8 +539,8 @@ pepc: notice: No VFM provided, assuming VFM 0x6ad (Granite Rapids Xeon) for deco
 pepc: warning: The 'tpmi_info' feature was not found in the debugfs dump
 pepc: notice: Using a dummy 'tpmi_info', assigning package number 0 to TPMI device 0000:00:00.1
 Supported TPMI features
-- ufs (2): Processor uncore (fabric) monitoring and control
-- tpmi_info (129): TPMI Info Registers
+- ufs (0x02): Processor uncore (fabric) monitoring and control
+- tpmi_info (0x81): TPMI Info Registers
 ```
 
 **Note:** Since the 'tpmi_info' feature is missing in this case, `pepc` creates a dummy 'tpmi_info'

pepctools/_PepcTpmi.py

@@ -428,7 +428,7 @@ def _list_specs(sdicts: dict[str, SDictTypedDict],
     _LOG.info("TPMI spec files:")
     for fname in sdicts:
         sdict = sdicts[fname]
-        _LOG.info("- %s (%d): %s", sdict["name"], sdict["feature_id"], sdict["desc"])
+        _LOG.info("- %s (%#04x): %s", sdict["name"], sdict["feature_id"], sdict["desc"])
         _LOG.info("  Spec file: %s", sdict["path"])
 
 def _get_ls_supported(cmdl: _LsCmdlineArgsTypedDict,
@@ -490,7 +490,7 @@ def _tpmi_ls_flat(cmdl: _LsCmdlineArgsTypedDict, tpmi: TPMI.TPMI):
         _LOG.info("Supported TPMI features")
 
         for fname, finfo in info["supported"].items():
-            _LOG.info("- %s (%s): %s", fname, finfo["feature_id"], finfo["desc"])
+            _LOG.info("- %s (%#04x): %s", fname, finfo["feature_id"], finfo["desc"])
 
     if info.get("unknown"):
         _LOG.info("TPMI features supported by the target platform, but no spec files found")
@@ -546,7 +546,7 @@ def _tpmi_ls_topology(cmdl: _LsCmdlineArgsTypedDict, tpmi: TPMI.TPMI):
     _LOG.info("Supported TPMI features")
 
     for fname, finfo in info["supported"].items():
-        _LOG.info("- %s (%d): %s", fname, finfo["feature_id"], finfo["desc"].strip())
+        _LOG.info("- %s (%#04x): %s", fname, finfo["feature_id"], finfo["desc"].strip())
 
         for addr, addr_info in topology[fname].items():
             _LOG.info("%sPCI address: %s", _pfx_bullet(1), addr)