Re: [PATCH 5/5] drm/i915: Fix atomic state when reusing the firmware fb

Damien Lespiau
2015-02-05T19:20:02Z

On Thu, Feb 05, 2015 at 11:12:20AM -0800, Matt Roper wrote:
> I think the only thing we need to close on is Ville's concern about the
> plane winding up with non-NULL fb and crtc pointers even though a crazy
> firmware or bootloader somehow left the plane turned off. I'm not super
> familiar with the fastboot stuff, but maybe our
> get_initial_plane_config() calls need a test to ensure the plane is
> actually turned on before they start trying to read out the FB base and
> such?

I was also looking at get_initial_plane_config() and was thinking the
same, will add a patch on top.

--
Damien