platform/x86: hp-bioscfg: Change how enum possible values size is evaluated

Updates steps how enum possible values size is evaluated

Signed-off-by: Jorge Lopez <jorge.lopez2@hp.com>
Link: https://lore.kernel.org/r/20230731203141.30044-7-jorge.lopez2@hp.com
Reviewed-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>

diff --git a/drivers/platform/x86/hp/hp-bioscfg/enum-attributes.c b/drivers/platform/x86/hp/hp-bioscfg/enum-attributes.c
index 89bb039..a2402d3 100644 (file)
--- a/drivers/platform/x86/hp/hp-bioscfg/enum-attributes.c
+++ b/drivers/platform/x86/hp/hp-bioscfg/enum-attributes.c
@@ -240,9 +240,11 @@ static int hp_populate_enumeration_elements_from_package(union acpi_object *enum
                                str_value, sizeof(enum_data->current_value));
                        break;
                case ENUM_SIZE:
-                       enum_data->possible_values_size = int_value;
-                       if (int_value > MAX_VALUES_SIZE)
+                       if (int_value > MAX_VALUES_SIZE) {
                                pr_warn("Possible number values size value exceeded the maximum number of elements supported or data may be malformed\n");
+                               int_value = MAX_VALUES_SIZE;
+                       }
+                       enum_data->possible_values_size = int_value;
 
                        /*
                         * This step is needed to keep the expected