> @@ -3400,7 +3400,7 @@ static int smu7_read_sensor(struct pp_hwmgr *hwmgr, int idx, > static int smu7_find_dpm_states_clocks_in_dpm_table(struct pp_hwmgr *hwmgr, const void *input) > { > const struct phm_set_power_state_input *states = > - (const struct phm_set_power_state_input *)input; > + input; Actually, there are some more cleanup opportunities heer and in some other cases. There is no need for the newline before input. julia