From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lukas Wunner Subject: Re: [PATCH v8 2/4] drm: Add API for capturing frame CRCs Date: Tue, 4 Oct 2016 14:44:00 +0200 Message-ID: <20161004124400.GA4913@wunner.de> References: <1473415010-1389-1-git-send-email-tomeu.vizoso@collabora.com> <1473415010-1389-3-git-send-email-tomeu.vizoso@collabora.com> <48ef75ce-061f-d9b0-d5fb-4503af2e5f71@nvidia.com> <20161004122313.GA4832@wunner.de> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20161004122313.GA4832@wunner.de> Sender: linux-kernel-owner@vger.kernel.org To: Daniel Vetter Cc: Jon Hunter , Tomeu Vizoso , Jonathan Corbet , linux-doc@vger.kernel.org, Emil Velikov , Linux Kernel Mailing List , dri-devel , Daniel Vetter List-Id: dri-devel@lists.freedesktop.org On Tue, Oct 04, 2016 at 02:23:13PM +0200, Lukas Wunner wrote: > On Tue, Oct 04, 2016 at 01:25:23PM +0200, Daniel Vetter wrote: > > On Tue, Oct 4, 2016 at 12:10 PM, Jon Hunter wrote: > > > Looks like crtc is a errno in the above case. I see this function is > > > called by looping through all the crtc and we never check to see if > > > they are valid. Should we? > > nouveau is exploding as well on driver load! > > > Tegra is still using the load/unload hooks. That didn't mesh well with > > Tomeu's patches (and Tomeu's patches have been thrown out meanwhile > > because of that). > > They're still on drm-intel-nightly, please revert if they're broken. Never mind, I hadn't rebased on today's drm-intel-nightly yet, I see it's gone now. Thanks, Lukas