Re: [PATCH] drm/fb: document dirty helper better

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Jun 11, 2019 at 03:34:52PM +0200, Noralf Trønnes wrote:
> 
> 
> Den 11.06.2019 13.28, skrev Daniel Vetter:
> > Apparently little none fact that there's no need to hand-roll your own
> > anymore. Cc'ing a bunch of driver people who might want to know this
> > too.
> > 
> > Cc: Rob Clark <robdclark@xxxxxxxxx>
> > Cc: Sebastian Reichel <sebastian.reichel@xxxxxxxxxxxxx>
> > Cc: Tomi Valkeinen <tomi.valkeinen@xxxxxx>
> > Cc: Maarten Lankhorst <maarten.lankhorst@xxxxxxxxxxxxxxx>
> > Cc: Maxime Ripard <maxime.ripard@xxxxxxxxxxx>
> > Cc: Sean Paul <sean@xxxxxxxxxx>
> > Cc: Gerd Hoffmann <kraxel@xxxxxxxxxx>
> > Cc: David Lechner <david@xxxxxxxxxxxxxx>
> > Cc: Noralf Trønnes <noralf@xxxxxxxxxxx>
> > Cc: Hans de Goede <hdegoede@xxxxxxxxxx>
> > Signed-off-by: Daniel Vetter <daniel.vetter@xxxxxxxxx>
> > ---
> 
> With Chris's comment adressed:

Done&merged, thanks to both of you for taking a look.
-Daniel

> 
> Reviewed-by: Noralf Trønnes <noralf@xxxxxxxxxxx>
> 
> 
> >  include/drm/drm_framebuffer.h | 3 +++
> >  1 file changed, 3 insertions(+)
> > 
> > diff --git a/include/drm/drm_framebuffer.h b/include/drm/drm_framebuffer.h
> > index c23016748e3f..c0e0256e3e98 100644
> > --- a/include/drm/drm_framebuffer.h
> > +++ b/include/drm/drm_framebuffer.h
> > @@ -87,6 +87,9 @@ struct drm_framebuffer_funcs {
> >  	 * for more information as all the semantics and arguments have a one to
> >  	 * one mapping on this function.
> >  	 *
> > +	 * Atomic drivers should use drm_atomic_helper_dirtyfb() to implement
> > +	 * this hook.
> > +	 *
> >  	 * RETURNS:
> >  	 *
> >  	 * 0 on success or a negative error code on failure.
> > 

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux