From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: dri-devel@lists.freedesktop.org
Cc: linux-kernel@vger.kernel.org
Subject: Re: [PATCH v4 0/3] drm: Add various helpers for simple drivers
Date: Thu, 12 May 2016 22:05:37 +0300 [thread overview]
Message-ID: <10920756.EEokP9U6oI@avalon> (raw)
In-Reply-To: <1463077523-23959-1-git-send-email-noralf@tronnes.org>
Hi Noralf,
Thank you for the patches.
For 1/3 and 2/3,
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
On Thursday 12 May 2016 20:25:20 Noralf Trønnes wrote:
> This patchset adds various helpers that was originally part of the
> tinydrm patchset.
>
> Essentially it adds 2 functions:
> - drm_fb_cma_create_with_funcs()
> CMA backed framebuffer supporting a dirty() callback.
> - drm_simple_display_pipe_init()
> Plane, crtc and encoder are collapsed into one entity.
>
>
> Changes since v3:
> - Add patch 'drm/fb-cma-helper: Use const for drm_framebuffer_funcs
> argument' - drm/fb-cma-helper: Add function drm_fb_cma_create_with_funcs()
> - funcs argument should be const
> - drm: Add helper for simple display pipeline
> - (struct drm_simple_display_pipe *)->funcs should be const
>
> Changes since v2:
> - drm: Add helper for simple display pipeline
> - Drop Kconfig knob DRM_KMS_HELPER
> - Expand documentation
>
> Changes since v1:
> - Drop patch: drm/panel: Add helper for simple panel connector
> - Add fb-helper and fb-cma-helper doc patches
> - Add drm/atomic: Don't skip drm_bridge_*() calls if
> !drm_encoder_helper_funcs - Add drm_atomic_helper_best_encoder()
> - drm/fb-cma-helper: Add function drm_fb_cma_create_with_funcs()
> - Expand docs
> - drm: Add helper for simple display pipeline
> - Add DOC header and add to gpu.tmpl
> - Fix docs: @funcs is optional, "negative error code",
> "This hook is optional."
> - Add checks to drm_simple_kms_plane_atomic_check()
>
>
> Noralf Trønnes (3):
> drm/fb-cma-helper: Use const for drm_framebuffer_funcs argument
> drm/fb-cma-helper: Add function drm_fb_cma_create_with_funcs()
> drm: Add helper for simple display pipeline
>
> Documentation/DocBook/gpu.tmpl | 6 +
> drivers/gpu/drm/Makefile | 2 +-
> drivers/gpu/drm/drm_fb_cma_helper.c | 35 ++++--
> drivers/gpu/drm/drm_simple_kms_helper.c | 208
> ++++++++++++++++++++++++++++++++ include/drm/drm_fb_cma_helper.h |
> 5 +-
> include/drm/drm_simple_kms_helper.h | 94 +++++++++++++++
> 6 files changed, 340 insertions(+), 10 deletions(-)
> create mode 100644 drivers/gpu/drm/drm_simple_kms_helper.c
> create mode 100644 include/drm/drm_simple_kms_helper.h
>
> --
> 2.8.2
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
--
Regards,
Laurent Pinchart
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
WARNING: multiple messages have this Message-ID (diff)
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: dri-devel@lists.freedesktop.org
Cc: "Noralf Trønnes" <noralf@tronnes.org>, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v4 0/3] drm: Add various helpers for simple drivers
Date: Thu, 12 May 2016 22:05:37 +0300 [thread overview]
Message-ID: <10920756.EEokP9U6oI@avalon> (raw)
In-Reply-To: <1463077523-23959-1-git-send-email-noralf@tronnes.org>
Hi Noralf,
Thank you for the patches.
For 1/3 and 2/3,
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
On Thursday 12 May 2016 20:25:20 Noralf Trønnes wrote:
> This patchset adds various helpers that was originally part of the
> tinydrm patchset.
>
> Essentially it adds 2 functions:
> - drm_fb_cma_create_with_funcs()
> CMA backed framebuffer supporting a dirty() callback.
> - drm_simple_display_pipe_init()
> Plane, crtc and encoder are collapsed into one entity.
>
>
> Changes since v3:
> - Add patch 'drm/fb-cma-helper: Use const for drm_framebuffer_funcs
> argument' - drm/fb-cma-helper: Add function drm_fb_cma_create_with_funcs()
> - funcs argument should be const
> - drm: Add helper for simple display pipeline
> - (struct drm_simple_display_pipe *)->funcs should be const
>
> Changes since v2:
> - drm: Add helper for simple display pipeline
> - Drop Kconfig knob DRM_KMS_HELPER
> - Expand documentation
>
> Changes since v1:
> - Drop patch: drm/panel: Add helper for simple panel connector
> - Add fb-helper and fb-cma-helper doc patches
> - Add drm/atomic: Don't skip drm_bridge_*() calls if
> !drm_encoder_helper_funcs - Add drm_atomic_helper_best_encoder()
> - drm/fb-cma-helper: Add function drm_fb_cma_create_with_funcs()
> - Expand docs
> - drm: Add helper for simple display pipeline
> - Add DOC header and add to gpu.tmpl
> - Fix docs: @funcs is optional, "negative error code",
> "This hook is optional."
> - Add checks to drm_simple_kms_plane_atomic_check()
>
>
> Noralf Trønnes (3):
> drm/fb-cma-helper: Use const for drm_framebuffer_funcs argument
> drm/fb-cma-helper: Add function drm_fb_cma_create_with_funcs()
> drm: Add helper for simple display pipeline
>
> Documentation/DocBook/gpu.tmpl | 6 +
> drivers/gpu/drm/Makefile | 2 +-
> drivers/gpu/drm/drm_fb_cma_helper.c | 35 ++++--
> drivers/gpu/drm/drm_simple_kms_helper.c | 208
> ++++++++++++++++++++++++++++++++ include/drm/drm_fb_cma_helper.h |
> 5 +-
> include/drm/drm_simple_kms_helper.h | 94 +++++++++++++++
> 6 files changed, 340 insertions(+), 10 deletions(-)
> create mode 100644 drivers/gpu/drm/drm_simple_kms_helper.c
> create mode 100644 include/drm/drm_simple_kms_helper.h
>
> --
> 2.8.2
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
--
Regards,
Laurent Pinchart
next prev parent reply other threads:[~2016-05-12 19:05 UTC|newest]
Thread overview: 37+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-05-12 18:25 [PATCH v4 0/3] drm: Add various helpers for simple drivers Noralf Trønnes
2016-05-12 18:25 ` Noralf Trønnes
2016-05-12 18:25 ` [PATCH v4 1/3] drm/fb-cma-helper: Use const for drm_framebuffer_funcs argument Noralf Trønnes
2016-05-12 18:25 ` Noralf Trønnes
2016-05-12 18:25 ` [PATCH v4 2/3] drm/fb-cma-helper: Add function drm_fb_cma_create_with_funcs() Noralf Trønnes
2016-05-12 18:25 ` Noralf Trønnes
2016-05-17 7:08 ` Daniel Vetter
2016-05-17 7:08 ` Daniel Vetter
2016-05-12 18:25 ` [PATCH v4 3/3] drm: Add helper for simple display pipeline Noralf Trønnes
2016-05-12 18:25 ` Noralf Trønnes
2016-05-12 18:36 ` Ville Syrjälä
2016-05-12 18:36 ` Ville Syrjälä
2016-05-17 7:05 ` Daniel Vetter
2016-05-17 7:05 ` Daniel Vetter
2016-05-17 7:46 ` Ville Syrjälä
2016-05-17 7:46 ` Ville Syrjälä
2016-05-17 7:59 ` Daniel Vetter
2016-05-17 7:59 ` Daniel Vetter
2016-05-17 12:00 ` Noralf Trønnes
2016-05-17 12:00 ` Noralf Trønnes
2016-05-17 12:12 ` Ville Syrjälä
2016-05-17 12:22 ` Noralf Trønnes
2016-05-17 12:22 ` Noralf Trønnes
2016-05-17 13:04 ` Daniel Vetter
2016-05-17 13:04 ` Daniel Vetter
2016-05-17 13:14 ` Ville Syrjälä
2016-05-17 13:14 ` Ville Syrjälä
2016-05-17 13:19 ` Daniel Vetter
2016-05-17 13:19 ` Daniel Vetter
2016-05-17 14:41 ` Ville Syrjälä
2016-05-17 14:41 ` Ville Syrjälä
2016-05-29 15:38 ` Noralf Trønnes
2016-05-29 15:38 ` Noralf Trønnes
2016-05-30 8:10 ` Daniel Vetter
2016-06-06 15:41 ` Noralf Trønnes
2016-05-12 19:05 ` Laurent Pinchart [this message]
2016-05-12 19:05 ` [PATCH v4 0/3] drm: Add various helpers for simple drivers Laurent Pinchart
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=10920756.EEokP9U6oI@avalon \
--to=laurent.pinchart@ideasonboard.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=linux-kernel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.