From mboxrd@z Thu Jan 1 00:00:00 1970 From: daniel@ffwll.ch (Daniel Vetter) Date: Wed, 15 Jun 2016 17:23:10 +0200 Subject: [PATCH v5 2/3] drm/arm: Add support for Mali Display Processors In-Reply-To: <1466002295-24813-3-git-send-email-Liviu.Dudau@arm.com> References: <1466002295-24813-1-git-send-email-Liviu.Dudau@arm.com> <1466002295-24813-3-git-send-email-Liviu.Dudau@arm.com> Message-ID: <20160615152309.GI1338@phenom.ffwll.local> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.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