[Intel-gfx] [PATCH 0/8] drm/i915: Give crtcs and planes actual names (v5)
Ville Syrjälä
ville.syrjala at linux.intel.com
Mon May 30 14:47:07 UTC 2016
On Fri, May 27, 2016 at 09:21:45PM +0100, Chris Wilson wrote:
> On Fri, May 27, 2016 at 08:59:18PM +0300, ville.syrjala at linux.intel.com wrote:
> > From: Ville Syrjälä <ville.syrjala at linux.intel.com>
> >
> > This is the remainder (the i915 specific parts) of my earlier
> > {crtc,plane}->name series [1]. I think at least one i915 specific patch
> > was already merged, and I think I added one or two new ones at some
> > point.
>
> Making the message more informative is always appreciated, and I
> couldn't spot anything blatantly wrong,
>
> Reviewed-by: Chris Wilson <chris at chris-wilson.co.uk>
>
> > I'm not so sure about the last patch. It was somewhat useful in narrowing
> > down some underrun locations and whatnot for me, but normally it might
> > be just noise. So I'm happy to keep it in my private stash for a rainy day
> > if people are fed up with dmesg spam.
>
> If you have a compelling usecase, it's not the worst thing we've ever
> done. If there's no rush, we should try to leverage ftrace, or something
> similar, rather than roll our own partial solution.
I've pushed everything but the last patch to dinq. Thanks for the
review.
--
Ville Syrjälä
Intel OTC
More information about the Intel-gfx
mailing list