All of lore.kernel.org
 help / color / mirror / Atom feed
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: Archit Taneja <architt@codeaurora.org>
Cc: dri-devel@lists.freedesktop.org, daniel@ffwll.ch,
	linux-arm-msm@vger.kernel.org, airlied@linux.ie,
	Tomi Valkeinen <tomi.valkeinen@ti.com>
Subject: Re: [RFC 05/21] drm/omap: Remove FB_KMS_HELPER and FB related config options
Date: Sat, 01 Aug 2015 13:02:05 +0300	[thread overview]
Message-ID: <2190055.OdQA4o6Iei@avalon> (raw)
In-Reply-To: <1436769848-4990-6-git-send-email-architt@codeaurora.org>

Hi Archit,

Thank you for the patch.

On Monday 13 July 2015 12:13:52 Archit Taneja wrote:
> Remove FB_* config options since the driver doesn't call any fbdev
> functions directly.
> 
> Remove FB_KMS_HELPER as this would now be selected by the top level
> FBDEV_EMULATION config option. If the fbdev emulation isn't selected,
> the drm_fb_helper functions will be stubbed out.
> 
> Cc: Tomi Valkeinen <tomi.valkeinen@ti.com>
> Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> 
> Signed-off-by: Archit Taneja <architt@codeaurora.org>

I like that, one step closer to removing fbdev altogether (even if there are 
many steps left) :-)

Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

> ---
>  drivers/gpu/drm/omapdrm/Kconfig | 5 -----
>  1 file changed, 5 deletions(-)
> 
> diff --git a/drivers/gpu/drm/omapdrm/Kconfig
> b/drivers/gpu/drm/omapdrm/Kconfig index 6c220cd..340ece7 100644
> --- a/drivers/gpu/drm/omapdrm/Kconfig
> +++ b/drivers/gpu/drm/omapdrm/Kconfig
> @@ -5,11 +5,6 @@ config DRM_OMAP
>  	depends on ARCH_OMAP2PLUS || ARCH_MULTIPLATFORM
>  	depends on OMAP2_DSS
>  	select DRM_KMS_HELPER
> -	select DRM_KMS_FB_HELPER
> -	select FB_SYS_FILLRECT
> -	select FB_SYS_COPYAREA
> -	select FB_SYS_IMAGEBLIT
> -	select FB_SYS_FOPS
>  	default n
>  	help
>  	  DRM display driver for OMAP2/3/4 based boards.

-- 
Regards,

Laurent Pinchart

  reply	other threads:[~2015-08-01 10:01 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-07-13  6:43 [RFC 00/21] drm: fb emulation: Step 3: Remove FB_KMS_HELPER config from drivers Archit Taneja
2015-07-13  6:43 ` [RFC 01/21] drm/cirrus: Remove FB_KMS_HELPER and FB related config options Archit Taneja
2015-07-13  6:43 ` [RFC 02/21] drm/rockchip: " Archit Taneja
2015-07-13  6:43 ` [RFC 03/21] drm/armada: " Archit Taneja
2015-07-13  6:43 ` [RFC 04/21] drm/ast: " Archit Taneja
2015-07-13  6:43 ` [RFC 05/21] drm/omap: " Archit Taneja
2015-08-01 10:02   ` Laurent Pinchart [this message]
2015-08-05  7:04     ` Archit Taneja
2015-07-13  6:43 ` [RFC 06/21] drm/exynos: " Archit Taneja
2015-07-13  6:43 ` [RFC 07/21] drm/gma500: " Archit Taneja
2015-07-13  6:43 ` [RFC 08/21] drm/mgag200: " Archit Taneja
2015-07-13  6:43 ` [RFC 09/21] drm/radeon: " Archit Taneja
2015-07-13  6:43 ` [RFC 10/21] drm/qxl: " Archit Taneja
2015-07-13  7:38   ` Daniel Vetter
2015-07-13  6:43 ` [RFC 11/21] drm/nouveau: " Archit Taneja
2015-07-13  6:43 ` [RFC 12/21] drm/udl: " Archit Taneja
2015-07-13  6:44 ` [RFC 13/21] drm/bochs: " Archit Taneja
2015-07-13  6:44 ` [RFC 14/21] drm/amdgpu: " Archit Taneja
2015-07-13  6:44 ` [RFC 15/21] drm/virtio: " Archit Taneja
2015-07-13  6:44 ` [RFC 16/21] drm/fb_cma_helper: " Archit Taneja
2015-07-13  6:44 ` [RFC 17/21] drm/atmel-hlcdc: Remove FB_KMS_HELPER config option Archit Taneja
2015-07-13  6:44 ` [RFC 18/21] drm/imx: " Archit Taneja
2015-07-13  6:44 ` [RFC 19/21] drm/rcar-du: " Archit Taneja
2015-08-01 10:02   ` Laurent Pinchart
2015-07-13  6:44 ` [RFC 20/21] drm/shmobile: " Archit Taneja
2015-08-01 10:02   ` Laurent Pinchart
2015-07-13  6:44 ` [RFC 21/21] drm/tilcdc: " Archit Taneja
2015-07-13 15:30 ` [RFC 00/21] drm: fb emulation: Step 3: Remove FB_KMS_HELPER config from drivers Alex Deucher
2015-07-13 15:37   ` Daniel Vetter
2015-07-14  6:31     ` Archit Taneja
2015-07-14  7:32       ` Daniel Vetter
2015-07-14  8:02       ` Thierry Reding
2015-07-14  8:23         ` Archit Taneja

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=2190055.OdQA4o6Iei@avalon \
    --to=laurent.pinchart@ideasonboard.com \
    --cc=airlied@linux.ie \
    --cc=architt@codeaurora.org \
    --cc=daniel@ffwll.ch \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=tomi.valkeinen@ti.com \
    /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.