From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lars-Peter Clausen Subject: Re: [PATCH 4/4] drm: Renesas SH Mobile DRM driver Date: Wed, 30 May 2012 15:48:04 +0200 Message-ID: <4FC62514.5040600@metafoo.de> References: <1338381179-13290-1-git-send-email-laurent.pinchart@ideasonboard.com> <1338381179-13290-5-git-send-email-laurent.pinchart@ideasonboard.com> <20120530134324.GK30400@pengutronix.de> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from smtp-out-102.synserver.de (smtp-out-102.synserver.de [212.40.185.102]) by gabe.freedesktop.org (Postfix) with ESMTP id BC92B9E954 for ; Wed, 30 May 2012 06:44:53 -0700 (PDT) In-Reply-To: <20120530134324.GK30400@pengutronix.de> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org Errors-To: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org To: Sascha Hauer Cc: Laurent Pinchart , dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org On 05/30/2012 03:43 PM, Sascha Hauer wrote: > Hi Laurent, > > On Wed, May 30, 2012 at 02:32:59PM +0200, Laurent Pinchart wrote: >> The SH Mobile LCD controller (LCDC) DRM driver supports the main >> graphics plane in RGB and YUV formats, as well as the overlay planes (in >> alpha-blending mode only). >> >> Only flat panel outputs using the parallel interface are supported. >> Support for SYS panels, HDMI and DSI is currently not implemented. > > Have you seen > > http://lists.freedesktop.org/archives/dri-devel/2012-May/023536.html > > and > > http://lists.freedesktop.org/archives/dri-devel/2012-May/023544.html > > The first patch is a generic gem helper for dma_alloc_* backed memory > which could serve as a drop in replacement for your gem code. The > latter is a generic helper for the framebuffer handling on these > devices. I saw that your driver supports planes which Lars' patch > currently doesn't, but it should be easy to add support for this. > Yup, just added plane support, will repost the patch in a moment. - Lars