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

Groups > linux.kernel > #1579618

Re: [PATCH v2 1/2] drm/cma-helper: Add multi buffer support for cma fbdev

Show all headers | View raw

[Multipart message — attachments visible in raw view] - view raw

Hi Laurent,

On Sun, Feb 12, 2017 at 02:28:11PM +0200, Laurent Pinchart wrote:
> Hi Maxime,
> 
> Thank you for the patch.
> 
> On Thursday 02 Feb 2017 11:31:56 Maxime Ripard wrote:
> > From: Xinliang Liu <xinliang.liu@linaro.org>
> > 
> > This patch add a config to support to create multi buffer for cma fbdev.
> > Such as double buffer and triple buffer.
> > 
> > Cma fbdev is convient to add a legency fbdev. And still many Android
> > devices use fbdev now and at least double buffer is needed for these
> > Android devices, so that a buffer flip can be operated. It will need
> > some time for Android device vendors to abondon legency fbdev. So multi
> > buffer for fbdev is needed.
> 
> How exactly do we expect Android to move away from fbdev if we add features to 
> the fbdev compat layer ? I'd much rather make it clear to them that fbdev is a 
> thing from the past and that they'd better migrate now.

If your point is that merging this patch will slow down the Android
move away from fbdev, I disagree with that (obviously).

I don't care at all about Android on my platform of choice, but don't
see how that merging this patch will change anything.

Let's be honest, Android trees typically have thousands of patches on
top of mainline. Do you think a simple, 15 LoC, patch will make any
difference to vendors? If they want to stay on fbdev and have that
feature, they'll just merge this patch, done.

However, what I do see is that three different people/organisations
have now expressed interest in that feature, on three different
SoCs. If that patch needed a significant rework of the fbdev layer,
then yes, I might agree that it's not worth it. But in this case, it's
pretty trivial.

The only people you're "punishing" here with that kind of concern are
the people who actually play fair and want not to have any patches and
everything upstream.

I guess a much better strategy would be to provide an incentive to
moving to KMS. And I truely think there's one already, so it's just a
matter of time before people switch over. Fbdev emulation or not.

Ma

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

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

Thread

[PATCH v2 1/2] drm/cma-helper: Add multi buffer support for cma fbdev Maxime Ripard <maxime.ripard@free-electrons.com> - 2017-02-02 11:40 +0100
  Re: [PATCH v2 1/2] drm/cma-helper: Add multi buffer support for cma  fbdev Daniel Vetter <daniel@ffwll.ch> - 2017-02-09 18:20 +0100
    Re: [PATCH v2 1/2] drm/cma-helper: Add multi buffer support for cma  fbdev Maxime Ripard <maxime.ripard@free-electrons.com> - 2017-02-10 17:00 +0100
  Re: [PATCH v2 1/2] drm/cma-helper: Add multi buffer support for cma fbdev Laurent Pinchart <laurent.pinchart@ideasonboard.com> - 2017-02-12 13:30 +0100
    Re: [PATCH v2 1/2] drm/cma-helper: Add multi buffer support for cma  fbdev Maxime Ripard <maxime.ripard@free-electrons.com> - 2017-02-13 12:00 +0100
      Re: [PATCH v2 1/2] drm/cma-helper: Add multi buffer support for cma fbdev Daniel Stone <daniel@fooishbar.org> - 2017-02-13 12:30 +0100
        Re: [PATCH v2 1/2] drm/cma-helper: Add multi buffer support for cma  fbdev Daniel Vetter <daniel@ffwll.ch> - 2017-02-14 21:20 +0100
          Re: [PATCH v2 1/2] drm/cma-helper: Add multi buffer support for cma fbdev Laurent Pinchart <laurent.pinchart@ideasonboard.com> - 2017-02-14 22:30 +0100
            Re: [PATCH v2 1/2] drm/cma-helper: Add multi buffer support for cma  fbdev Maxime Ripard <maxime.ripard@free-electrons.com> - 2017-02-15 14:00 +0100
              Re: [PATCH v2 1/2] drm/cma-helper: Add multi buffer support for cma fbdev Laurent Pinchart <laurent.pinchart@ideasonboard.com> - 2017-02-17 12:40 +0100
        Re: [PATCH v2 1/2] drm/cma-helper: Add multi buffer support for cma  fbdev Maxime Ripard <maxime.ripard@free-electrons.com> - 2017-02-15 13:40 +0100
          Re: [PATCH v2 1/2] drm/cma-helper: Add multi buffer support for cma fbdev Laurent Pinchart <laurent.pinchart@ideasonboard.com> - 2017-02-17 12:30 +0100

csiph-web