Re: [PATCH v8 2/4] drm: Add API for capturing frame CRCs

From	Lukas Wunner <lukas@wunner.de>
Newsgroups	linux.kernel
Subject	Re: [PATCH v8 2/4] drm: Add API for capturing frame CRCs
Date	2016-10-04 14:50 +0200
Message-ID	<soxkt-2M1-3@gated-at.bofh.it> (permalink)
References	<sfqLf-1c3-3@gated-at.bofh.it> <sfqLf-1c3-11@gated-at.bofh.it> <souZj-1my-11@gated-at.bofh.it> <sow53-204-21@gated-at.bofh.it> <sox17-2F6-7@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

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 <jonathanh@nvidia.com> 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

Thread

Re: [PATCH v8 2/4] drm: Add API for capturing frame CRCs Jon Hunter <jonathanh@nvidia.com> - 2016-10-04 12:20 +0200
  Re: [PATCH v8 2/4] drm: Add API for capturing frame CRCs Daniel Vetter <daniel@ffwll.ch> - 2016-10-04 13:30 +0200
    Re: [PATCH v8 2/4] drm: Add API for capturing frame CRCs Lukas Wunner <lukas@wunner.de> - 2016-10-04 14:30 +0200
      Re: [PATCH v8 2/4] drm: Add API for capturing frame CRCs Lukas Wunner <lukas@wunner.de> - 2016-10-04 14:50 +0200
    Re: [PATCH v8 2/4] drm: Add API for capturing frame CRCs Jon Hunter <jonathanh@nvidia.com> - 2016-10-04 14:40 +0200

csiph-web