Technical reminder about this patch: usually, for electronic
stability, you'll have to raise the voltage BEFORE increasing the
clock speed; on the other, you'll have to lower the voltage AFTER
lowering the clock speed. You may encounter stability problem if you
don't follow this order.

The patch fixes the first case, but you may end up creating the
opposite problem by only moving
ci_apply_disp_minimum_voltage_request(rdev) earlier in the process.
But it may actually be covered by ci_send_msg_to_smc_with_parameter()
or it may not be actually what ci_upload_dpm_level_enable_mask() is
about.

Is this patch linked to a specific bug?

Alexandre Demers

Reply via email to