From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754753AbcETG5y (ORCPT ); Fri, 20 May 2016 02:57:54 -0400 Received: from mail-wm0-f66.google.com ([74.125.82.66]:33700 "EHLO mail-wm0-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753334AbcETG5u (ORCPT ); Fri, 20 May 2016 02:57:50 -0400 Date: Fri, 20 May 2016 08:57:47 +0200 From: Daniel Vetter To: Liviu Dudau Cc: David Airlie , DRI devel , LKML , Daniel Vetter , Brian Starkey , David Brown Subject: Re: [PATCH v2 2/3] drm/arm: Add support for Mali Display Processors Message-ID: <20160520065747.GP27098@phenom.ffwll.local> Mail-Followup-To: Liviu Dudau , David Airlie , DRI devel , LKML , Brian Starkey , David Brown References: <1461593964-27722-1-git-send-email-Liviu.Dudau@arm.com> <1461593964-27722-3-git-send-email-Liviu.Dudau@arm.com> <20160519152648.GG23566@e106497-lin.cambridge.arm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20160519152648.GG23566@e106497-lin.cambridge.arm.com> X-Operating-System: Linux phenom 4.6.0-rc5+ User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, May 19, 2016 at 04:26:49PM +0100, Liviu Dudau wrote: > On Mon, Apr 25, 2016 at 03:19:23PM +0100, Liviu Dudau wrote: > > Add support for the new family of Display Processors from ARM Ltd. > > This commit adds basic support for Mali DP500, DP550 and DP650 > > parts, with only the display engine being supported at the moment. > > > > Cc: David Brown > > Cc: Brian Starkey > > > > Signed-off-by: Liviu Dudau > > Hello, > > I'm fully aware that the merge window is open and people are busy, however if > any of the DRM sub-maintainers could provide some Acks or Reviewed-bys I would > really appreciate it. Asking just the maintainers for review in a big subsystem like drm doesn't scale. There's lots other people who know how to write drm drivers, and for community building it's better to have a wide network of contacts anyway. A little bit of "I review yours and you review mine" is usually all it takes. Dave & I and others will do some sanity checks, but there's no way we can review all submissions individually. -Daniel > > I plan to make the code available under a new branch to go into linux-next next > week. > > Best regards, > Liviu > > > --- > > drivers/gpu/drm/arm/Kconfig | 16 + > > drivers/gpu/drm/arm/Makefile | 2 + > > drivers/gpu/drm/arm/malidp_crtc.c | 259 ++++++++++++ > > drivers/gpu/drm/arm/malidp_drv.c | 538 +++++++++++++++++++++++++ > > drivers/gpu/drm/arm/malidp_drv.h | 54 +++ > > drivers/gpu/drm/arm/malidp_hw.c | 774 ++++++++++++++++++++++++++++++++++++ > > drivers/gpu/drm/arm/malidp_hw.h | 189 +++++++++ > > drivers/gpu/drm/arm/malidp_planes.c | 337 ++++++++++++++++ > > drivers/gpu/drm/arm/malidp_regs.h | 172 ++++++++ > > 9 files changed, 2341 insertions(+) > > create mode 100644 drivers/gpu/drm/arm/malidp_crtc.c > > create mode 100644 drivers/gpu/drm/arm/malidp_drv.c > > create mode 100644 drivers/gpu/drm/arm/malidp_drv.h > > create mode 100644 drivers/gpu/drm/arm/malidp_hw.c > > create mode 100644 drivers/gpu/drm/arm/malidp_hw.h > > create mode 100644 drivers/gpu/drm/arm/malidp_planes.c > > create mode 100644 drivers/gpu/drm/arm/malidp_regs.h > > > > diff --git a/drivers/gpu/drm/arm/Kconfig b/drivers/gpu/drm/arm/Kconfig > > index eaed454..1b29065 100644 > > --- a/drivers/gpu/drm/arm/Kconfig > > +++ b/drivers/gpu/drm/arm/Kconfig > > @@ -25,3 +25,19 @@ config DRM_HDLCD_SHOW_UNDERRUN > > Enable this option to show in red colour the pixels that the > > HDLCD device did not fetch from framebuffer due to underrun > > conditions. > > + > > +config DRM_MALI_DISPLAY > > + tristate "ARM Mali Display Processor" > > + depends on DRM && OF && (ARM || ARM64) > > + depends on COMMON_CLK > > + select DRM_ARM > > + select DRM_KMS_HELPER > > + select DRM_KMS_CMA_HELPER > > + select DRM_GEM_CMA_HELPER > > + select VIDEOMODE_HELPERS > > + help > > + Choose this option if you want to compile the ARM Mali Display > > + Processor driver. It supports the DP500, DP550 and DP650 variants > > + of the hardware. > > + > > + If compiled as a module it will be called mali-dp. > > diff --git a/drivers/gpu/drm/arm/Makefile b/drivers/gpu/drm/arm/Makefile > > index 89dcb7b..bb8b158 100644 > > --- a/drivers/gpu/drm/arm/Makefile > > +++ b/drivers/gpu/drm/arm/Makefile > > @@ -1,2 +1,4 @@ > > hdlcd-y := hdlcd_drv.o hdlcd_crtc.o > > obj-$(CONFIG_DRM_HDLCD) += hdlcd.o > > +mali-dp-y := malidp_drv.o malidp_hw.o malidp_planes.o malidp_crtc.o > > +obj-$(CONFIG_DRM_MALI_DISPLAY) += mali-dp.o > > diff --git a/drivers/gpu/drm/arm/malidp_crtc.c b/drivers/gpu/drm/arm/malidp_crtc.c > > new file mode 100644 > > index 0000000..12893d0 > > --- /dev/null > > +++ b/drivers/gpu/drm/arm/malidp_crtc.c > > @@ -0,0 +1,259 @@ > > +/* > > + * (C) COPYRIGHT 2016 ARM Limited. All rights reserved. > > + * Author: Liviu Dudau > > + * > > + * This program is free software and is provided to you under the terms of the > > + * GNU General Public License version 2 as published by the Free Software > > + * Foundation, and any use by you of this program is subject to the terms > > + * of such GNU licence. > > + * > > + * ARM Mali DP500/DP550/DP650 driver (crtc operations) > > + */ > > + > > +#include > > +#include > > +#include > > +#include > > +#include > > +#include > > +#include