diff mbox series

drm/todo: Add item about modeset properties

Message ID 20200603170434.2363446-1-emil.l.velikov@gmail.com (mailing list archive)
State New, archived
Headers show
Series drm/todo: Add item about modeset properties | expand

Commit Message

Emil Velikov June 3, 2020, 5:04 p.m. UTC
Add some information about pre-atomic modeset properties alongside a
list of suggestions how to handle the different instances.

Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
---
 Documentation/gpu/todo.rst | 32 ++++++++++++++++++++++++++++++++
 1 file changed, 32 insertions(+)

Comments

Daniel Vetter June 4, 2020, 7:30 a.m. UTC | #1
On Wed, Jun 03, 2020 at 06:04:34PM +0100, Emil Velikov wrote:
> Add some information about pre-atomic modeset properties alongside a
> list of suggestions how to handle the different instances.
> 
> Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
> ---
>  Documentation/gpu/todo.rst | 32 ++++++++++++++++++++++++++++++++
>  1 file changed, 32 insertions(+)
> 
> diff --git a/Documentation/gpu/todo.rst b/Documentation/gpu/todo.rst
> index 658b52f7ffc6..6648fd13cc1e 100644
> --- a/Documentation/gpu/todo.rst
> +++ b/Documentation/gpu/todo.rst
> @@ -392,6 +392,38 @@ Contact: Laurent Pinchart, respective driver maintainers
>  
>  Level: Intermediate
>  
> +Consolidate custom driver modeset properties
> +--------------------------------------------
> +
> +Before atomic modeset took place, many drivers where creating their own 
> +properties. Among other things, atomic brought the requirement that custom,
> +driver specific properties should not be used.
> +
> +In for this task, we aim to introduce core helper or reuse the existing ones

s/In for/For/ I think ...

> +if available:
> +
> +A quick, unconfirmed, examples list.
> +
> +Introduce core helpers:
> +- audio (amdgpu, intel, gma500, radeon)
> +- brightness, contrast, etc (armada, nouveau) - overlay only (?)
> +- broadcast rgb (gma500, intel)
> +- colorkey (armada, nouveau, rcar) - overlay only (?)
> +- dither (amdgpu, nouveau, radeon) - varies across drivers
> +- underscan family (amdgpu, radeon, nouveau)
> +
> +Already in core:
> +- colorspace (sti)
> +- tv format names, enhancements (gma500, intel)
> +- tv overscan, margins, etc. (gma500, intel)
> +- zorder (omapdrm) - same as zpos (?)
> +
> +
> +Contact: Emil Velikov, respective driver maintainers
> +
> +Level: Intermediate

Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>
> +
> +
>  Core refactorings
>  =================
>  
> -- 
> 2.25.1
> 
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
Emil Velikov June 15, 2020, 7:23 p.m. UTC | #2
On Thu, 4 Jun 2020 at 08:30, Daniel Vetter <daniel@ffwll.ch> wrote:
>
> On Wed, Jun 03, 2020 at 06:04:34PM +0100, Emil Velikov wrote:
> > Add some information about pre-atomic modeset properties alongside a
> > list of suggestions how to handle the different instances.
> >
> > Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
> > ---
> >  Documentation/gpu/todo.rst | 32 ++++++++++++++++++++++++++++++++
> >  1 file changed, 32 insertions(+)
> >
> > diff --git a/Documentation/gpu/todo.rst b/Documentation/gpu/todo.rst
> > index 658b52f7ffc6..6648fd13cc1e 100644
> > --- a/Documentation/gpu/todo.rst
> > +++ b/Documentation/gpu/todo.rst
> > @@ -392,6 +392,38 @@ Contact: Laurent Pinchart, respective driver maintainers
> >
> >  Level: Intermediate
> >
> > +Consolidate custom driver modeset properties
> > +--------------------------------------------
> > +
> > +Before atomic modeset took place, many drivers where creating their own
> > +properties. Among other things, atomic brought the requirement that custom,
> > +driver specific properties should not be used.
> > +
> > +In for this task, we aim to introduce core helper or reuse the existing ones
>
> s/In for/For/ I think ...
>
Fixed up alongside s/core helper/core helpers/

> > +if available:
> > +
> > +A quick, unconfirmed, examples list.
> > +
> > +Introduce core helpers:
> > +- audio (amdgpu, intel, gma500, radeon)
> > +- brightness, contrast, etc (armada, nouveau) - overlay only (?)
> > +- broadcast rgb (gma500, intel)
> > +- colorkey (armada, nouveau, rcar) - overlay only (?)
> > +- dither (amdgpu, nouveau, radeon) - varies across drivers
> > +- underscan family (amdgpu, radeon, nouveau)
> > +
> > +Already in core:
> > +- colorspace (sti)
> > +- tv format names, enhancements (gma500, intel)
> > +- tv overscan, margins, etc. (gma500, intel)
> > +- zorder (omapdrm) - same as zpos (?)
> > +
> > +
> > +Contact: Emil Velikov, respective driver maintainers
> > +
> > +Level: Intermediate
>
> Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>

... and pushed to drm-misc. Thank you.

-Emil
diff mbox series

Patch

diff --git a/Documentation/gpu/todo.rst b/Documentation/gpu/todo.rst
index 658b52f7ffc6..6648fd13cc1e 100644
--- a/Documentation/gpu/todo.rst
+++ b/Documentation/gpu/todo.rst
@@ -392,6 +392,38 @@  Contact: Laurent Pinchart, respective driver maintainers
 
 Level: Intermediate
 
+Consolidate custom driver modeset properties
+--------------------------------------------
+
+Before atomic modeset took place, many drivers where creating their own 
+properties. Among other things, atomic brought the requirement that custom,
+driver specific properties should not be used.
+
+In for this task, we aim to introduce core helper or reuse the existing ones
+if available:
+
+A quick, unconfirmed, examples list.
+
+Introduce core helpers:
+- audio (amdgpu, intel, gma500, radeon)
+- brightness, contrast, etc (armada, nouveau) - overlay only (?)
+- broadcast rgb (gma500, intel)
+- colorkey (armada, nouveau, rcar) - overlay only (?)
+- dither (amdgpu, nouveau, radeon) - varies across drivers
+- underscan family (amdgpu, radeon, nouveau)
+
+Already in core:
+- colorspace (sti)
+- tv format names, enhancements (gma500, intel)
+- tv overscan, margins, etc. (gma500, intel)
+- zorder (omapdrm) - same as zpos (?)
+
+
+Contact: Emil Velikov, respective driver maintainers
+
+Level: Intermediate
+
+
 Core refactorings
 =================