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

Groups > linux.kernel > #1229986

Re: [PATCH RFC 0/8] Add New DRM Driver for Hisilicon's Hi6220 SoC

Show all headers | View raw

On Thu, Sep 17, 2015 at 05:51:28PM +0800, Xinliang Liu wrote:
> On 17 September 2015 at 04:16, Daniel Vetter <daniel@ffwll.ch> wrote:
> 
> > On Wed, Sep 16, 2015 at 04:23:35PM +0100, Daniel Stone wrote:
> > > The biggest issue though, is that this driver should become an atomic
> > > modesetting driver. Atomic modesetting, rather than sending small
> > > individual commands (enable CRTC, change plane position, etc) is based
> > > on validating and passing around complete sets of hardware state.
> > > Daniel Vetter's blog has an article on how to convert your driver:
> > > http://blog.ffwll.ch/2014/11/atomic-modeset-support-for-kms-drivers.html
> >
> > Yeah, any new driver should really be built on top of atomic - it's a lot
> > more flexible than the old thing and it's also what you want long-term.
> >
> > I've also just done a presenation about atomic for drivers:
> >
> > http://people.freedesktop.org/~danvet/presentations/xdc-2015.pdf
> 
> 
> Hi Daniel,
> This is a good presentation. It gives us very detail and good suggection on
> implementation.
> Thank you for sharing this.
> 
> We have a open source KMS hwc:
> wiki:
> https://wiki.linaro.org/BenjaminGaignard/HWComposer_DRM?highlight=%28hwcomposer%2
> source code: git://git.linaro.org/people/benjamin.gaignard/hwcomposer.git
> Now only STI and Hikey boards are tested on it. And atomic mode setting is
> not support now.
> I think we should add support for atomic mode setting next.
> 
> One difficulty I am facing is that one setting should be made sure is ok in
> the prepare function of hwc.
> If not, the set function of hwc may be fail and display will not properly.
> I don't know atomic mode setting how to handle this situation. And it seems
> that in the prepare function,
> it should check the hardware's capabilities, such as clip, scale, rotation,
> blending and so on.

Specifically to support things like hwc's ->prepare callback atomic
supports a TEST_ONLY mode. Hence in your the prepare code build up the
atomic state, but set the TEST_ONLY flag when calling the ioctl. When the
kernel is happy you can store it somewhere and tell surface flinger the
configuration so it can render the remaining buffers with GL. The idea is
that generic userspace does use TEST_ONLY mode iterative to build up the
maximal configuration of hw planes that a given kms driver supports,
leaving no hw-specific code in userspace. For optimized hwc drivers
running on atomic you then just need to tune the selection, but detailed
constraint checking would still be done by the kernel.

The weston patches from Daniel Stone have a similar design, would be worth
checking those out.

Cheers, Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
--
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

[PATCH RFC 0/8] Add New DRM Driver for Hisilicon's Hi6220 SoC Xinwei Kong <kong.kongxinwei@hisilicon.com> - 2015-09-15 11:50 +0200
  [PATCH RFC 6/8] drm: hisilicon: Add support for fbdev Xinwei Kong <kong.kongxinwei@hisilicon.com> - 2015-09-15 11:50 +0200
    Re: [PATCH RFC 6/8] drm: hisilicon: Add support for fbdev Rob Herring <robh@kernel.org> - 2015-09-15 20:30 +0200
      Re: [PATCH RFC 6/8] drm: hisilicon: Add support for fbdev Xinwei Kong <kong.kongxinwei@hisilicon.com> - 2015-09-16 05:50 +0200
      Re: [PATCH RFC 6/8] drm: hisilicon: Add support for fbdev Rob Clark <robdclark@gmail.com> - 2015-09-17 14:00 +0200
  [PATCH RFC 5/8] drm: hisilicon: fill interface function of encoder\connector part Xinwei Kong <kong.kongxinwei@hisilicon.com> - 2015-09-15 11:50 +0200
    Re: [PATCH RFC 5/8] drm: hisilicon: fill interface function of  encoder\connector part Archit Taneja <architt@codeaurora.org> - 2015-09-17 13:50 +0200
  [PATCH RFC 3/8] drm: hisilicon: Add the link to DRM/KMS interface Xinwei Kong <kong.kongxinwei@hisilicon.com> - 2015-09-15 11:50 +0200
  [PATCH RFC 2/8] drm: hisilicon: Add new DRM driver for hisilicon Soc Xinwei Kong <kong.kongxinwei@hisilicon.com> - 2015-09-15 11:50 +0200
    Re: [PATCH RFC 2/8] drm: hisilicon: Add new DRM driver for hisilicon  Soc Archit Taneja <architt@codeaurora.org> - 2015-09-17 13:20 +0200
  Re: [PATCH RFC 0/8] Add New DRM Driver for Hisilicon's Hi6220 SoC Daniel Stone <daniel@fooishbar.org> - 2015-09-16 17:30 +0200
    Re: [PATCH RFC 0/8] Add New DRM Driver for Hisilicon's Hi6220 SoC Daniel Vetter <daniel@ffwll.ch> - 2015-09-16 22:20 +0200
      Re: [PATCH RFC 0/8] Add New DRM Driver for Hisilicon's Hi6220 SoC Daniel Vetter <daniel@ffwll.ch> - 2015-09-22 10:50 +0200
    Re: [PATCH RFC 0/8] Add New DRM Driver for Hisilicon's Hi6220 SoC Xinwei Kong <kong.kongxinwei@hisilicon.com> - 2015-09-18 12:50 +0200

csiph-web