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

[Daniel Vetter]

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 
> > Cc: Sebastian Reichel 
> > Cc: Tomi Valkeinen 
> > Cc: Maarten Lankhorst 
> > Cc: Maxime Ripard 
> > Cc: Sean Paul 
> > Cc: Gerd Hoffmann 
> > Cc: David Lechner 
> > Cc: Noralf Trønnes 
> > Cc: Hans de Goede 
> > Signed-off-by: Daniel Vetter 
> > ---
> 
> With Chris's comment adressed:

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

> 
> Reviewed-by: Noralf Trønnes 
> 
> 
> >  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@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

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

[Noralf Trønnes]

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 
> Cc: Sebastian Reichel 
> Cc: Tomi Valkeinen 
> Cc: Maarten Lankhorst 
> Cc: Maxime Ripard 
> Cc: Sean Paul 
> Cc: Gerd Hoffmann 
> Cc: David Lechner 
> Cc: Noralf Trønnes 
> Cc: Hans de Goede 
> Signed-off-by: Daniel Vetter 
> ---

With Chris's comment adressed:

Reviewed-by: Noralf Trønnes 


>  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.
> 
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

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

[Chris Wilson]

Quoting Daniel Vetter (2019-06-11 12:28:59)
> Apparently little none fact that there's no need to hand-roll your own
s/none/known/
> anymore. Cc'ing a bunch of driver people who might want to know this
> too.
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

[PATCH] drm/fb: document dirty helper better

[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 
Cc: Sebastian Reichel 
Cc: Tomi Valkeinen 
Cc: Maarten Lankhorst 
Cc: Maxime Ripard 
Cc: Sean Paul 
Cc: Gerd Hoffmann 
Cc: David Lechner 
Cc: Noralf Trønnes 
Cc: Hans de Goede 
Signed-off-by: Daniel Vetter 
---
 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.
-- 
2.20.1

___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel