platform/x86: ISST: Ignore minor version change

The hardware definition of every TPMI feature contains a major and minor
version. When there is a change in the MMIO offset or change in the
definition of a field, hardware will change major version. For addition
of new fields without modifying existing MMIO offsets or fields, only the
minor version is changed.

Driver is developed to support SST functionality for a major and minor
version. If the hardware changes major version, since offsets and
definitions are changed, driver cannot continue to provide SST interface
to users. Driver can still function with a minor version change as it will
just miss the new functionality added by the hardware. The current
implementation doesn't ignore any version change.

If there is mismatch with the minor version, continue with an information
log message. If there is mismatch with the major version, log error and
exit.

Signed-off-by: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
Link: https://lore.kernel.org/r/20231003184916.1860084-3-srinivas.pandruvada@linux.intel.com
Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>

diff --git a/drivers/platform/x86/intel/speed_select_if/isst_tpmi_core.c b/drivers/platform/x86/intel/speed_select_if/isst_tpmi_core.c
index ac5c6a812592b9b8f5ee965af9b96f67c25200d1..0b6d2c8644372d1b0bdc090c59337e9718865f3c 100644 (file)
--- a/drivers/platform/x86/intel/speed_select_if/isst_tpmi_core.c
+++ b/drivers/platform/x86/intel/speed_select_if/isst_tpmi_core.c
@@ -30,7 +30,8 @@
 #include "isst_if_common.h"
 
 /* Supported SST hardware version by this driver */
-#define ISST_HEADER_VERSION            1
+#define ISST_MAJOR_VERSION     0
+#define ISST_MINOR_VERSION     1
 
 /*
  * Used to indicate if value read from MMIO needs to get multiplied
@@ -352,12 +353,19 @@ static int sst_main(struct auxiliary_device *auxdev, struct tpmi_per_power_domai
        pd_info->sst_header.cp_offset *= 8;
        pd_info->sst_header.pp_offset *= 8;
 
-       if (pd_info->sst_header.interface_version != ISST_HEADER_VERSION) {
-               dev_err(&auxdev->dev, "SST: Unsupported version:%x\n",
-                       pd_info->sst_header.interface_version);
+       if (pd_info->sst_header.interface_version == TPMI_VERSION_INVALID)
+               return -ENODEV;
+
+       if (TPMI_MAJOR_VERSION(pd_info->sst_header.interface_version) != ISST_MAJOR_VERSION) {
+               dev_err(&auxdev->dev, "SST: Unsupported major version:%lx\n",
+                       TPMI_MAJOR_VERSION(pd_info->sst_header.interface_version));
                return -ENODEV;
        }
 
+       if (TPMI_MINOR_VERSION(pd_info->sst_header.interface_version) != ISST_MINOR_VERSION)
+               dev_info(&auxdev->dev, "SST: Ignore: Unsupported minor version:%lx\n",
+                        TPMI_MINOR_VERSION(pd_info->sst_header.interface_version));
+
        /* Read SST CP Header */
        *((u64 *)&pd_info->cp_header) = readq(pd_info->sst_base + pd_info->sst_header.cp_offset);