From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933192AbcFOPXX (ORCPT ); Wed, 15 Jun 2016 11:23:23 -0400 Received: from mail-lf0-f68.google.com ([209.85.215.68]:36207 "EHLO mail-lf0-f68.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932700AbcFOPXV (ORCPT ); Wed, 15 Jun 2016 11:23:21 -0400 Date: Wed, 15 Jun 2016 17:23:10 +0200 From: Daniel Vetter To: Liviu Dudau Cc: David Airlie , Rob Herring , Brian Starkey , Emil Velikov , Daniel Vetter , devicetree , DRI devel , LKML , David Brown , LAKML Subject: Re: [PATCH v5 2/3] drm/arm: Add support for Mali Display Processors Message-ID: <20160615152309.GI1338@phenom.ffwll.local> Mail-Followup-To: Liviu Dudau , David Airlie , Rob Herring , Brian Starkey , Emil Velikov , devicetree , DRI devel , LKML , David Brown , LAKML References: <1466002295-24813-1-git-send-email-Liviu.Dudau@arm.com> <1466002295-24813-3-git-send-email-Liviu.Dudau@arm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <1466002295-24813-3-git-send-email-Liviu.Dudau@arm.com> X-Operating-System: Linux phenom 4.6.0-rc5+ User-Agent: Mutt/1.6.0 (2016-04-01) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Jun 15, 2016 at 03:51:34PM +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 Small thing I noticed: drm_dev_register/connector_register_all should be the last step in your init code, and unregister the first. Atm it's somewhere in the middle. But perfectly fine to do that as a follow-up. Quickly scrolled through the driver, looks all nice. Acked-by: Daniel Vetter > --- > drivers/gpu/drm/arm/Kconfig | 16 + > drivers/gpu/drm/arm/Makefile | 2 + > drivers/gpu/drm/arm/malidp_crtc.c | 216 +++++++++++ > drivers/gpu/drm/arm/malidp_drv.c | 512 ++++++++++++++++++++++++++ > drivers/gpu/drm/arm/malidp_drv.h | 54 +++ > drivers/gpu/drm/arm/malidp_hw.c | 691 ++++++++++++++++++++++++++++++++++++ > drivers/gpu/drm/arm/malidp_hw.h | 241 +++++++++++++ > drivers/gpu/drm/arm/malidp_planes.c | 298 ++++++++++++++++ > drivers/gpu/drm/arm/malidp_regs.h | 172 +++++++++ > 9 files changed, 2202 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..08e6a71 > --- /dev/null > +++ b/drivers/gpu/drm/arm/malidp_crtc.c > @@ -0,0 +1,216 @@ > +/* > + * (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