Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1233486

Re: No more new fbdev drivers, please

Show all headers | View raw

On 27 September 2015 at 06:49, Rob Clark <robdclark@gmail.com> wrote:
> On Sat, Sep 26, 2015 at 2:46 PM, Geert Uytterhoeven
> <geert@linux-m68k.org> wrote:
>> Hi David,
>>
>> On Sat, Sep 26, 2015 at 8:13 PM, David Herrmann <dh.herrmann@gmail.com> wrote:
>>> On Sat, Sep 26, 2015 at 8:01 PM, Geert Uytterhoeven
>>> <geert@linux-m68k.org> wrote:
>>>> On Sat, Sep 26, 2015 at 7:07 PM, Alex Deucher <alexdeucher@gmail.com> wrote:
>>>>> On Sat, Sep 26, 2015 at 4:28 AM, Geert Uytterhoeven
>>>>> <geert@linux-m68k.org> wrote:
>>>>>> For the (mailing list) record, can you please provide some explicit pointers
>>>>>> to these existing really simple drivers?
>>>>>
>>>>> See the tilcdc, ast, mgag200, and udl drivers for example.
>>>>
>>>> Thanks for the list!
>>>>
>>>> The smallest of these (udl) still counts in at ca. 2800 LoC, while there are
>>>> several fbdev drivers that have less than 200 LoC.
>>>> Granted, these really small ones support a single fixed video mode only, but
>>>> you can write a simple fbdev driver with mode setting in less than 1000 LoC.
>>>>
>>>> I'm sure DRM can do better?
>>>
>>> Is counting lines really the level of the discussion to go here?
>>
>> LoC is not the most important. But if the smallest DRM driver needs an order
>> of magnitude more LoC than the smallest fbdev driver, I start to wonder.
>
> I think most of the drm/kms drivers are bigger due to more features..
> iirc original tilcdc was ~2k loc (compared to ~1.6kloc for da8xx-fb),
> but it already supported multiple modes, page flipping, vblank
> notification, etc.  It has grown since then.  Although still probably
> smaller than downstream da8xx-fb + tda998x hdmi bridge (and re-using
> the same tda998x bridge code with several other drivers too, compared
> to downstream solution for the same)..
>
> Probably there is room for more helpers for even more restrictive hw.

My main worry for having helpers for "simple" hw, is that people start
using them
to have a minimal 400loc driver, but once they add any feature outside
the helper
they have to rewrite their driver to avoid the helpers.

Most of the drm driver is boilerplate, we could possibly reduce the boilerplate,
but I'm not sure it's worth the effort to save somebody a small bit of
trouble at
bringup.

loc is a pointless tool for measuring this, a small drm driver will be as simple
as a small fbdev driver, and will likely provide more features that people need.

Dave.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

No more new fbdev drivers, please Tomi Valkeinen <tomi.valkeinen@ti.com> - 2015-09-24 14:30 +0200
  Re: No more new fbdev drivers, please Thomas Petazzoni <thomas.petazzoni@free-electrons.com> - 2015-09-24 14:50 +0200
    Re: No more new fbdev drivers, please Austin S Hemmelgarn <ahferroin7@gmail.com> - 2015-09-24 17:30 +0200
      Re: No more new fbdev drivers, please Alex Deucher <alexdeucher@gmail.com> - 2015-09-24 17:40 +0200
      Re: No more new fbdev drivers, please Daniel Vetter <daniel@ffwll.ch> - 2015-09-24 18:00 +0200
        Re: No more new fbdev drivers, please Austin S Hemmelgarn <ahferroin7@gmail.com> - 2015-09-24 18:20 +0200
        Re: No more new fbdev drivers, please Ondrej Zary <linux@rainbow-software.org> - 2015-09-24 19:20 +0200
          Re: No more new fbdev drivers, please Daniel Vetter <daniel@ffwll.ch> - 2015-09-24 20:10 +0200
    Re: No more new fbdev drivers, please Daniel Vetter <daniel@ffwll.ch> - 2015-09-24 17:30 +0200
      Re: No more new fbdev drivers, please Geert Uytterhoeven <geert@linux-m68k.org> - 2015-09-26 10:30 +0200
        Re: No more new fbdev drivers, please Alex Deucher <alexdeucher@gmail.com> - 2015-09-26 19:10 +0200
          Re: No more new fbdev drivers, please Geert Uytterhoeven <geert@linux-m68k.org> - 2015-09-26 20:10 +0200
            Re: No more new fbdev drivers, please David Herrmann <dh.herrmann@gmail.com> - 2015-09-26 20:20 +0200
              Re: No more new fbdev drivers, please Geert Uytterhoeven <geert@linux-m68k.org> - 2015-09-26 20:50 +0200
                Re: No more new fbdev drivers, please Rob Clark <robdclark@gmail.com> - 2015-09-26 22:50 +0200
                Re: No more new fbdev drivers, please Dave Airlie <airlied@gmail.com> - 2015-09-27 00:00 +0200
                Re: No more new fbdev drivers, please Emil Velikov <emil.l.velikov@gmail.com> - 2015-09-30 14:00 +0200
              Re: No more new fbdev drivers, please Gerd Hoffmann <kraxel@redhat.com> - 2015-09-28 09:50 +0200
                Re: No more new fbdev drivers, please Daniel Vetter <daniel@ffwll.ch> - 2015-09-28 14:40 +0200
                Re: No more new fbdev drivers, please Gerd Hoffmann <kraxel@redhat.com> - 2015-09-29 10:30 +0200
                Re: No more new fbdev drivers, please Laurent Pinchart <laurent.pinchart@ideasonboard.com> - 2015-09-29 10:40 +0200
            Re: No more new fbdev drivers, please Bernie Thompson <bernie@plugable.com> - 2015-09-28 23:00 +0200
            Re: No more new fbdev drivers, please Daniel Vetter <daniel@ffwll.ch> - 2015-09-29 09:10 +0200
  Re: No more new fbdev drivers, please Aaro Koskinen <aaro.koskinen@iki.fi> - 2015-09-25 11:00 +0200
    Re: No more new fbdev drivers, please Ondrej Zary <linux@rainbow-software.org> - 2015-09-25 13:10 +0200
  Re: No more new fbdev drivers, please Sudip Mukherjee <sudipm.mukherjee@gmail.com> - 2015-09-26 09:20 +0200
    Re: No more new fbdev drivers, please Ilia Mirkin <imirkin@alum.mit.edu> - 2015-09-26 09:40 +0200
  Re: No more new fbdev drivers, please Noralf Trønnes <noralf@tronnes.org> - 2015-09-27 15:20 +0200
    Re: No more new fbdev drivers, please Emil Velikov <emil.l.velikov@gmail.com> - 2015-09-27 18:10 +0200
      Re: No more new fbdev drivers, please Noralf Trønnes <noralf@tronnes.org> - 2015-09-29 01:00 +0200
        Re: No more new fbdev drivers, please Daniel Vetter <daniel@ffwll.ch> - 2015-09-29 09:10 +0200

csiph-web