On Thu, Oct 24, 2019 at 02:12:46PM +0200, Maarten Lankhorst wrote:
> 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.

Does that mean the series is not actually bisectable?

> 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

-- 
Ville Syrjälä
Intel
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to