Op 22-10-2019 om 20:16 schreef Ville Syrjälä: > On Fri, Oct 18, 2019 at 10:13:23AM +0200, Maarten Lankhorst wrote: >> intel_get_load_detect_pipe() needs to set uapi active, >> uapi enable is set by the call to drm_atomic_set_mode_for_crtc(), >> so we can remove it. >> >> intel_pipe_config_compare() needs to look at hw state, but I didn't >> change spatch to look at it. It's easy enough to do manually. >> >> intel_atomic_check() definitely needs to check for uapi enable, >> otherwise intel_modeset_pipe_config cannot copy uapi state to hw. > We seem to have three totally separate things in this one patch. The patch is about the manual conversions that need to be done because the automated checks get them wrong. So it touches 3 spots but does 1 thing, if you want I can split it into 3 patches. Will do so if required. ~Maarten _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
