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:23:13 +0200 Message-ID: <20161004122313.GA4832@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> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: 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 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. Thanks, Lukas