lkml.org 
[lkml]   [2022]   [Jan]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    Date
    SubjectRe: [PATCH v1 0/4] fbtft: Unorphan the driver for maintenance
    On Wed, Jan 26, 2022 at 3:24 PM Greg Kroah-Hartman
    <gregkh@linuxfoundation.org> wrote:
    > On Wed, Jan 26, 2022 at 03:18:14PM +0100, Javier Martinez Canillas wrote:
    > > On 1/26/22 15:11, Andy Shevchenko wrote:
    > > > On Wed, Jan 26, 2022 at 02:47:33PM +0100, Javier Martinez Canillas wrote:
    > > >> On 1/26/22 14:27, Andy Shevchenko wrote:
    > > >>> On Wed, Jan 26, 2022 at 12:18:30PM +0100, Javier Martinez Canillas wrote:
    > > >>>> On 1/26/22 11:59, Helge Deller wrote:
    > > >>>>> On 1/26/22 11:02, Andy Shevchenko wrote:
    > > >
    > > > ...
    > > >
    > > >>>>>> P.S. For the record, I will personally NAK any attempts to remove that
    > > >>>>>> driver from the kernel. And this is another point why it's better not
    > > >>>>>> to be under the staging.
    > > >>>>>
    > > >>>>> I agree. Same as for me to NAK the disabling of fbcon's acceleration
    > > >>>>> features or even attempting to remove fbdev altogether (unless all
    > > >>>>> relevant drivers are ported to DRM).
    > > >>>>
    > > >>>> But that will never happen if we keep moving the goal post.
    > > >>>>
    > > >>>> At some point new fbdev drivers should not be added anymore, otherwise
    > > >>>> the number of existing drivers that need conversion will keep growing.
    > > >>>
    > > >>> This thread is not about adding a new driver.
    > > >>
    > > >> It was about adding a new drivers to drivers/video/ (taken from staging).
    > > >
    > > > Does it mean gates are open to take any new fbdev drivers to the staging?
    > > > If not, I do not see a point here.
    > > >
    > >
    > > Good question. I don't know really.
    > >
    > > But staging has always been more flexible in what's accepted there and
    > > that's why some distros avoid to enable CONFIG_STAGING=y in the kernel.
    >
    > And that's why if you load a staging driver, it enables TAINT_CRAP in
    > your runtime flags :)

    fwiw I'm fine with adding new fbdev drivers to staging, that really
    doesn't hurt anyone. Adding drm drivers to staging tends to be pain,
    least because if we need to do any changes to helpers there's a
    cross-tree cordination problem usually, and the benefit of staging
    hasn't in the past really outweighted that. Plus I try for us to land
    new drivers when they're good enough directly into drivers/gpu, and
    not aim for perfect.
    -Daniel
    --
    Daniel Vetter
    Software Engineer, Intel Corporation
    http://blog.ffwll.ch

    \
     
     \ /
      Last update: 2022-01-26 23:38    [W:3.415 / U:0.004 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site