diff mbox

drm-misc: Next phase in small drivers in drm-misc

Message ID 20170329073815.10410-1-daniel.vetter@ffwll.ch (mailing list archive)
State New, archived
Headers show

Commit Message

Daniel Vetter March 29, 2017, 7:38 a.m. UTC
I think the initial experiment worked fairly well, and the
kickstarting of a review (well, ack/best practices really) market
seems to work ok-ish too. I think we can try to make it (slowly) scale
to more drivers, with driver teams interested in being part of
drm-misc. I do think we need to be careful though, to make sure we
don't overstretch the review/mentoring bandwidth and can improve
process&tooling as we go.

Cc: Sumit Semwal <sumit.semwal@linaro.org>
Cc: Gustavo Padovan <gustavo@padovan.org>
Cc: Jani Nikula <jani.nikula@linux.intel.com>
Cc: Sean Paul <seanpaul@chromium.org>
Cc: Archit Taneja <architt@codeaurora.org>
Cc: Gerd Hoffmann <kraxel@redhat.com>
Cc: Boris Brezillon <boris.brezillon@free-electrons.com>
Cc: Neil Armstrong <narmstrong@baylibre.com>
Cc: Mark Yao <mark.yao@rock-chips.com>
Cc: Eric Anholt <eric@anholt.net>
Cc: Shawn Guo <shawnguo@kernel.org>
Cc: Dave Airlie <airlied@redhat.com>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
---
 drm-misc.rst | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Boris BREZILLON March 29, 2017, 7:46 a.m. UTC | #1
On Wed, 29 Mar 2017 09:38:15 +0200
Daniel Vetter <daniel.vetter@ffwll.ch> wrote:

> I think the initial experiment worked fairly well, and the
> kickstarting of a review (well, ack/best practices really) market
> seems to work ok-ish too. I think we can try to make it (slowly) scale
> to more drivers, with driver teams interested in being part of
> drm-misc. I do think we need to be careful though, to make sure we
> don't overstretch the review/mentoring bandwidth and can improve
> process&tooling as we go.
> 
> Cc: Sumit Semwal <sumit.semwal@linaro.org>
> Cc: Gustavo Padovan <gustavo@padovan.org>
> Cc: Jani Nikula <jani.nikula@linux.intel.com>
> Cc: Sean Paul <seanpaul@chromium.org>
> Cc: Archit Taneja <architt@codeaurora.org>
> Cc: Gerd Hoffmann <kraxel@redhat.com>
> Cc: Boris Brezillon <boris.brezillon@free-electrons.com>

Acked-by: Boris Brezillon <boris.brezillon@free-electrons.com>

> Cc: Neil Armstrong <narmstrong@baylibre.com>
> Cc: Mark Yao <mark.yao@rock-chips.com>
> Cc: Eric Anholt <eric@anholt.net>
> Cc: Shawn Guo <shawnguo@kernel.org>
> Cc: Dave Airlie <airlied@redhat.com>
> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
> ---
>  drm-misc.rst | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drm-misc.rst b/drm-misc.rst
> index b6d01f2c7c2b..e4409cce8178 100644
> --- a/drm-misc.rst
> +++ b/drm-misc.rst
> @@ -105,8 +105,8 @@ Small Drivers
>  =============
>  
>  Small drivers, where a full tree is overkill, can be maintained in drm-misc. For
> -now it's just an experiment with a few drivers to figure out a working process.
> -Slightly different rules apply:
> +now there's just a few drivers maintained in drm-misc, but we can slowly add
> +more to figure out how to make this scale. Slightly different rules apply:
>  
>  * Small is measured in patches merged per kernel release. The occasional big
>    patch series is still acceptable if it's not a common thing (e.g. new hw
Shawn Guo March 29, 2017, 8:26 a.m. UTC | #2
On Wed, Mar 29, 2017 at 09:38:15AM +0200, Daniel Vetter wrote:
> I think the initial experiment worked fairly well, and the
> kickstarting of a review (well, ack/best practices really) market
> seems to work ok-ish too. I think we can try to make it (slowly) scale
> to more drivers, with driver teams interested in being part of
> drm-misc. I do think we need to be careful though, to make sure we
> don't overstretch the review/mentoring bandwidth and can improve
> process&tooling as we go.
> 
> Cc: Sumit Semwal <sumit.semwal@linaro.org>
> Cc: Gustavo Padovan <gustavo@padovan.org>
> Cc: Jani Nikula <jani.nikula@linux.intel.com>
> Cc: Sean Paul <seanpaul@chromium.org>
> Cc: Archit Taneja <architt@codeaurora.org>
> Cc: Gerd Hoffmann <kraxel@redhat.com>
> Cc: Boris Brezillon <boris.brezillon@free-electrons.com>
> Cc: Neil Armstrong <narmstrong@baylibre.com>
> Cc: Mark Yao <mark.yao@rock-chips.com>
> Cc: Eric Anholt <eric@anholt.net>
> Cc: Shawn Guo <shawnguo@kernel.org>

Acked-by: Shawn Guo <shawnguo@kernel.org>

> Cc: Dave Airlie <airlied@redhat.com>
> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
Sean Paul April 3, 2017, 2:23 p.m. UTC | #3
On Wed, Mar 29, 2017 at 09:38:15AM +0200, Daniel Vetter wrote:
> I think the initial experiment worked fairly well, and the
> kickstarting of a review (well, ack/best practices really) market
> seems to work ok-ish too. I think we can try to make it (slowly) scale
> to more drivers, with driver teams interested in being part of
> drm-misc. I do think we need to be careful though, to make sure we
> don't overstretch the review/mentoring bandwidth and can improve
> process&tooling as we go.
> 
> Cc: Sumit Semwal <sumit.semwal@linaro.org>
> Cc: Gustavo Padovan <gustavo@padovan.org>
> Cc: Jani Nikula <jani.nikula@linux.intel.com>
> Cc: Sean Paul <seanpaul@chromium.org>

Acked-by: Sean Paul <seanpaul@chromium.org>

> Cc: Archit Taneja <architt@codeaurora.org>
> Cc: Gerd Hoffmann <kraxel@redhat.com>
> Cc: Boris Brezillon <boris.brezillon@free-electrons.com>
> Cc: Neil Armstrong <narmstrong@baylibre.com>
> Cc: Mark Yao <mark.yao@rock-chips.com>
> Cc: Eric Anholt <eric@anholt.net>
> Cc: Shawn Guo <shawnguo@kernel.org>
> Cc: Dave Airlie <airlied@redhat.com>
> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
> ---
>  drm-misc.rst | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drm-misc.rst b/drm-misc.rst
> index b6d01f2c7c2b..e4409cce8178 100644
> --- a/drm-misc.rst
> +++ b/drm-misc.rst
> @@ -105,8 +105,8 @@ Small Drivers
>  =============
>  
>  Small drivers, where a full tree is overkill, can be maintained in drm-misc. For
> -now it's just an experiment with a few drivers to figure out a working process.
> -Slightly different rules apply:
> +now there's just a few drivers maintained in drm-misc, but we can slowly add
> +more to figure out how to make this scale. Slightly different rules apply:
>  
>  * Small is measured in patches merged per kernel release. The occasional big
>    patch series is still acceptable if it's not a common thing (e.g. new hw
> -- 
> 2.11.0
Daniel Vetter April 3, 2017, 2:32 p.m. UTC | #4
On Mon, Apr 03, 2017 at 10:23:08AM -0400, Sean Paul wrote:
> On Wed, Mar 29, 2017 at 09:38:15AM +0200, Daniel Vetter wrote:
> > I think the initial experiment worked fairly well, and the
> > kickstarting of a review (well, ack/best practices really) market
> > seems to work ok-ish too. I think we can try to make it (slowly) scale
> > to more drivers, with driver teams interested in being part of
> > drm-misc. I do think we need to be careful though, to make sure we
> > don't overstretch the review/mentoring bandwidth and can improve
> > process&tooling as we go.
> > 
> > Cc: Sumit Semwal <sumit.semwal@linaro.org>
> > Cc: Gustavo Padovan <gustavo@padovan.org>
> > Cc: Jani Nikula <jani.nikula@linux.intel.com>
> > Cc: Sean Paul <seanpaul@chromium.org>
> 
> Acked-by: Sean Paul <seanpaul@chromium.org>

And pushed with Jani's irc-ack.
-Daniel

> 
> > Cc: Archit Taneja <architt@codeaurora.org>
> > Cc: Gerd Hoffmann <kraxel@redhat.com>
> > Cc: Boris Brezillon <boris.brezillon@free-electrons.com>
> > Cc: Neil Armstrong <narmstrong@baylibre.com>
> > Cc: Mark Yao <mark.yao@rock-chips.com>
> > Cc: Eric Anholt <eric@anholt.net>
> > Cc: Shawn Guo <shawnguo@kernel.org>
> > Cc: Dave Airlie <airlied@redhat.com>
> > Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
> > ---
> >  drm-misc.rst | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> > 
> > diff --git a/drm-misc.rst b/drm-misc.rst
> > index b6d01f2c7c2b..e4409cce8178 100644
> > --- a/drm-misc.rst
> > +++ b/drm-misc.rst
> > @@ -105,8 +105,8 @@ Small Drivers
> >  =============
> >  
> >  Small drivers, where a full tree is overkill, can be maintained in drm-misc. For
> > -now it's just an experiment with a few drivers to figure out a working process.
> > -Slightly different rules apply:
> > +now there's just a few drivers maintained in drm-misc, but we can slowly add
> > +more to figure out how to make this scale. Slightly different rules apply:
> >  
> >  * Small is measured in patches merged per kernel release. The occasional big
> >    patch series is still acceptable if it's not a common thing (e.g. new hw
> > -- 
> > 2.11.0
> 
> -- 
> Sean Paul, Software Engineer, Google / Chromium OS
diff mbox

Patch

diff --git a/drm-misc.rst b/drm-misc.rst
index b6d01f2c7c2b..e4409cce8178 100644
--- a/drm-misc.rst
+++ b/drm-misc.rst
@@ -105,8 +105,8 @@  Small Drivers
 =============
 
 Small drivers, where a full tree is overkill, can be maintained in drm-misc. For
-now it's just an experiment with a few drivers to figure out a working process.
-Slightly different rules apply:
+now there's just a few drivers maintained in drm-misc, but we can slowly add
+more to figure out how to make this scale. Slightly different rules apply:
 
 * Small is measured in patches merged per kernel release. The occasional big
   patch series is still acceptable if it's not a common thing (e.g. new hw