[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <1495829844-69341-15-git-send-email-keescook@chromium.org>
Date: Fri, 26 May 2017 13:17:18 -0700
From: Kees Cook <keescook@...omium.org>
To: kernel-hardening@...ts.openwall.com
Cc: Kees Cook <keescook@...omium.org>,
Christian König <christian.koenig@....com>,
Eric Huang <JinHuiEric.Huang@....com>,
Alex Deucher <alexander.deucher@....com>,
Laura Abbott <labbott@...hat.com>,
x86@...nel.org,
linux-kernel@...r.kernel.org
Subject: [PATCH v2 14/20] drm/amd/powerplay: Use designated initializers
The randstruct plugin requires designated initializers for structures
that are entirely function pointers.
Cc: Christian König <christian.koenig@....com>
Cc: Eric Huang <JinHuiEric.Huang@....com>
Cc: Alex Deucher <alexander.deucher@....com>
Signed-off-by: Kees Cook <keescook@...omium.org>
---
.../gpu/drm/amd/powerplay/hwmgr/vega10_thermal.c | 28 ++++++++++++++--------
1 file changed, 18 insertions(+), 10 deletions(-)
diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_thermal.c b/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_thermal.c
index d5f53d04fa08..cfa187185dcc 100644
--- a/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_thermal.c
+++ b/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_thermal.c
@@ -709,17 +709,22 @@ static int tf_vega10_thermal_disable_alert(struct pp_hwmgr *hwmgr,
static struct phm_master_table_item
vega10_thermal_start_thermal_controller_master_list[] = {
- {NULL, tf_vega10_thermal_initialize},
- {NULL, tf_vega10_thermal_set_temperature_range},
- {NULL, tf_vega10_thermal_enable_alert},
+ { .isFunctionNeededInRuntimeTable = NULL,
+ .tableFunction = tf_vega10_thermal_initialize },
+ { .isFunctionNeededInRuntimeTable = NULL,
+ .tableFunction = tf_vega10_thermal_set_temperature_range },
+ { .isFunctionNeededInRuntimeTable = NULL,
+ .tableFunction = tf_vega10_thermal_enable_alert },
/* We should restrict performance levels to low before we halt the SMC.
* On the other hand we are still in boot state when we do this
* so it would be pointless.
* If this assumption changes we have to revisit this table.
*/
- {NULL, tf_vega10_thermal_setup_fan_table},
- {NULL, tf_vega10_thermal_start_smc_fan_control},
- {NULL, NULL}
+ { .isFunctionNeededInRuntimeTable = NULL,
+ .tableFunction = tf_vega10_thermal_setup_fan_table },
+ { .isFunctionNeededInRuntimeTable = NULL,
+ .tableFunction = tf_vega10_thermal_start_smc_fan_control },
+ { }
};
static struct phm_master_table_header
@@ -731,10 +736,13 @@ vega10_thermal_start_thermal_controller_master = {
static struct phm_master_table_item
vega10_thermal_set_temperature_range_master_list[] = {
- {NULL, tf_vega10_thermal_disable_alert},
- {NULL, tf_vega10_thermal_set_temperature_range},
- {NULL, tf_vega10_thermal_enable_alert},
- {NULL, NULL}
+ { .isFunctionNeededInRuntimeTable = NULL,
+ .tableFunction = tf_vega10_thermal_disable_alert },
+ { .isFunctionNeededInRuntimeTable = NULL,
+ .tableFunction = tf_vega10_thermal_set_temperature_range },
+ { .isFunctionNeededInRuntimeTable = NULL,
+ .tableFunction = tf_vega10_thermal_enable_alert },
+ { }
};
struct phm_master_table_header
--
2.7.4
Powered by blists - more mailing lists
Confused about mailing lists and their use?
Read about mailing lists on Wikipedia
and check out these
guidelines on proper formatting of your messages.
