https://bugzilla.kernel.org/show_bug.cgi?id=205393 --- Comment #4 from haro41@xxxxxx --- Yes, your patch works and has the same effect, apparently. What confused me and the reason why i prefered to leave the vega10_update_avfs() call before the flag modification, was the code inside vega10_update_avfs(): static int vega10_update_avfs(struct pp_hwmgr *hwmgr) { struct vega10_hwmgr *data = hwmgr->backend; if (data->need_update_dpm_table & DPMTABLE_OD_UPDATE_VDDC) { vega10_avfs_enable(hwmgr, false); } else if (data->need_update_dpm_table) { vega10_avfs_enable(hwmgr, false); vega10_avfs_enable(hwmgr, true); } else { vega10_avfs_enable(hwmgr, true); } return 0; } Here is a disable/enable sequence inserted, perhaps for a reason? -- You are receiving this mail because: You are watching the assignee of the bug. _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
