From: Hyun Kwon <hyunk-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org>
To: Daniel Vetter <daniel-/w4YWyX8dFk@public.gmane.org>
Cc: "dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org"
<dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org>,
"devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
<devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
Michal Simek
<michal.simek-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org>,
Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
Daniel Vetter <daniel.vetter-/w4YWyX8dFk@public.gmane.org>,
Laurent Pinchart
<laurent.pinchart-ryLnwIuWjnjg/C1BVhZhaw@public.gmane.org>
Subject: RE: [PATCH v3 2/8] drm: xlnx: Xilinx DRM KMS driver
Date: Fri, 26 Jan 2018 01:36:15 +0000 [thread overview]
Message-ID: <MWHPR02MB2493AA0C2AE0E1F80D57225BD6E00@MWHPR02MB2493.namprd02.prod.outlook.com> (raw)
In-Reply-To: <20180117082028.GC2759-dv86pmgwkMBes7Z6vYuT8azUEOm+Xw19@public.gmane.org>
Hi Daniel,
> -----Original Message-----
> From: Daniel Vetter [mailto:daniel.vetter@ffwll.ch] On Behalf Of Daniel
> Vetter
> Sent: Wednesday, January 17, 2018 12:20 AM
> To: Hyun Kwon <hyunk@xilinx.com>
> Cc: dri-devel@lists.freedesktop.org; devicetree@vger.kernel.org; Michal
> Simek <michal.simek@xilinx.com>; Rob Herring <robh@kernel.org>; Daniel
> Vetter <daniel.vetter@ffwll.ch>; Laurent Pinchart
> <laurent.pinchart@ideasonboard.com>
> Subject: Re: [PATCH v3 2/8] drm: xlnx: Xilinx DRM KMS driver
>
> On Mon, Jan 15, 2018 at 05:57:05PM -0800, Hyun Kwon wrote:
> > Xilinx has various platforms for display, where users can create
> > using multiple IPs in the programmable FPGA fabric, or where
> > some hardened piepline is available on the chip. Furthermore,
> > hardened pipeline can also interact with soft logics in FPGA.
> >
> > The Xilinx DRM KMS is to integrate multiple subdevices and to represent
> > the entire pipeline as a single DRM device. The driver includes helpers
> > (ex, framebuffer and gem helpers) and glue logics (ex, crtc interface).
> >
> > Signed-off-by: Hyun Kwon <hyun.kwon@xilinx.com>
> > ---
> > v3
> > - Add Laurent as a maintainer
> > - Fix multiple-reference on gem objects
> > v2
> > - Change the SPDX identifier format
> > - Merge patches(crtc, gem, fb) into single one
> > v2 of xlnx_drv
> > - Rename kms to display in xlnx_drv
> > - Replace some xlnx specific fb helper with common helpers in xlnx_drv
> > - Don't set the commit tail callback in xlnx_drv
> > - Support 'ports' graph binding in xlnx_drv
> > v2 of xlnx_fb
> > - Remove wrappers in xlnx_fb
> > - Replace some functions with drm core helpers in xlnx_fb
> > ---
> > ---
> > MAINTAINERS | 9 +
> > drivers/gpu/drm/Kconfig | 2 +
> > drivers/gpu/drm/Makefile | 1 +
> > drivers/gpu/drm/xlnx/Kconfig | 12 ++
> > drivers/gpu/drm/xlnx/Makefile | 2 +
> > drivers/gpu/drm/xlnx/xlnx_crtc.c | 203 ++++++++++++++++++
> > drivers/gpu/drm/xlnx/xlnx_crtc.h | 78 +++++++
> > drivers/gpu/drm/xlnx/xlnx_drv.c | 447
> +++++++++++++++++++++++++++++++++++++++
> > drivers/gpu/drm/xlnx/xlnx_drv.h | 30 +++
> > drivers/gpu/drm/xlnx/xlnx_fb.c | 298 ++++++++++++++++++++++++++
> > drivers/gpu/drm/xlnx/xlnx_fb.h | 33 +++
> > drivers/gpu/drm/xlnx/xlnx_gem.c | 47 ++++
> > drivers/gpu/drm/xlnx/xlnx_gem.h | 26 +++
> > 13 files changed, 1188 insertions(+)
> > create mode 100644 drivers/gpu/drm/xlnx/Kconfig
> > create mode 100644 drivers/gpu/drm/xlnx/Makefile
> > create mode 100644 drivers/gpu/drm/xlnx/xlnx_crtc.c
> > create mode 100644 drivers/gpu/drm/xlnx/xlnx_crtc.h
> > create mode 100644 drivers/gpu/drm/xlnx/xlnx_drv.c
> > create mode 100644 drivers/gpu/drm/xlnx/xlnx_drv.h
> > create mode 100644 drivers/gpu/drm/xlnx/xlnx_fb.c
> > create mode 100644 drivers/gpu/drm/xlnx/xlnx_fb.h
> > create mode 100644 drivers/gpu/drm/xlnx/xlnx_gem.c
> > create mode 100644 drivers/gpu/drm/xlnx/xlnx_gem.h
> >
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index 2811a21..dc84346 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -4682,6 +4682,15 @@ F: drivers/gpu/drm/etnaviv/
> > F: include/uapi/drm/etnaviv_drm.h
> > F: Documentation/devicetree/bindings/display/etnaviv/
> >
> > +DRM DRIVERS FOR XILINX
> > +M: Hyun Kwon <hyun.kwon@xilinx.com>
> > +M: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> > +L: dri-devel@lists.freedesktop.org
> > +S: Maintained
> > +F: drivers/gpu/drm/xlnx/
> > +F: Documentation/devicetree/bindings/display/xlnx/
> > +T: git git://anongit.freedesktop.org/drm/drm-misc
>
> Ok, I scrolled through the patches again and besides the one nit for the
> enable/disable_vblank callback it looks all good. On patches 2 and 4-6:
>
> Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>
>
> These are ready for merging. The 2 dt patches need an ack from Rob
> Herring. The final 2 patches with the new properties need the usual uapi
> work and userspace enabling per
>
> https://dri.freedesktop.org/docs/drm/gpu/drm-uapi.html#open-source-
> userspace-requirements
>
> Since Laurent is listed as co-maintainer, and Laurent is already drm-misc
> committer I think it's indeed best to maintain it in there. Please request
> a new account per
>
> https://www.freedesktop.org/wiki/AccountRequests/
>
> and check out quickstart in
>
> https://01.org/linuxgraphics/gfx-docs/maintainer-
> tools/dim.html#quickstart
>
> For any questions I or Laurent should be able to help out. My expectation
> is that once you have the ack from DT maintainers you'll merge patches 1-6
> yourself directly.
>
> Thanks, Daniel
>
Thanks for the pointers and help.
I'll follow the process once bindings are accepted.
Thanks,
-hyun
> > +
> > DRM DRIVERS FOR ZTE ZX
> > M: Shawn Guo <shawnguo@kernel.org>
> > L: dri-devel@lists.freedesktop.org
> > diff --git a/drivers/gpu/drm/Kconfig b/drivers/gpu/drm/Kconfig
> > index 22ee5a1..f3c71e3 100644
> > --- a/drivers/gpu/drm/Kconfig
> > +++ b/drivers/gpu/drm/Kconfig
> > @@ -282,6 +282,8 @@ source "drivers/gpu/drm/tinydrm/Kconfig"
> >
> > source "drivers/gpu/drm/pl111/Kconfig"
> >
> > +source "drivers/gpu/drm/xlnx/Kconfig"
> > +
> > # Keep legacy drivers last
> >
> > menuconfig DRM_LEGACY
> > diff --git a/drivers/gpu/drm/Makefile b/drivers/gpu/drm/Makefile
> > index 92550dd..2f44c77 100644
> > --- a/drivers/gpu/drm/Makefile
> > +++ b/drivers/gpu/drm/Makefile
> > @@ -103,3 +103,4 @@ obj-$(CONFIG_DRM_ZTE) += zte/
> > obj-$(CONFIG_DRM_MXSFB) += mxsfb/
> > obj-$(CONFIG_DRM_TINYDRM) += tinydrm/
> > obj-$(CONFIG_DRM_PL111) += pl111/
> > +obj-$(CONFIG_DRM_XLNX) += xlnx/
> > diff --git a/drivers/gpu/drm/xlnx/Kconfig b/drivers/gpu/drm/xlnx/Kconfig
> > new file mode 100644
> > index 0000000..19fd7cd
> > --- /dev/null
> > +++ b/drivers/gpu/drm/xlnx/Kconfig
> > @@ -0,0 +1,12 @@
> > +config DRM_XLNX
> > + tristate "Xilinx DRM KMS Driver"
> > + depends on DRM && OF
> > + select DRM_KMS_HELPER
> > + select DRM_KMS_CMA_HELPER
> > + select DRM_GEM_CMA_HELPER
> > + help
> > + Xilinx DRM KMS driver. Choose this option if you have
> > + a Xilinx SoCs with hardened display pipeline or soft
> > + display pipeline using Xilinx IPs in FPGA. This module
> > + provides the kernel mode setting functionalities
> > + for Xilinx display drivers.
> > diff --git a/drivers/gpu/drm/xlnx/Makefile
> b/drivers/gpu/drm/xlnx/Makefile
> > new file mode 100644
> > index 0000000..c60a281
> > --- /dev/null
> > +++ b/drivers/gpu/drm/xlnx/Makefile
> > @@ -0,0 +1,2 @@
> > +xlnx_drm-objs += xlnx_crtc.o xlnx_drv.o xlnx_fb.o xlnx_gem.o
> > +obj-$(CONFIG_DRM_XLNX) += xlnx_drm.o
> > diff --git a/drivers/gpu/drm/xlnx/xlnx_crtc.c
> b/drivers/gpu/drm/xlnx/xlnx_crtc.c
> > new file mode 100644
> > index 0000000..89beb9e
> > --- /dev/null
> > +++ b/drivers/gpu/drm/xlnx/xlnx_crtc.c
> > @@ -0,0 +1,203 @@
> > +// SPDX-License-Identifier: GPL-2.0
> > +/*
> > + * Xilinx DRM crtc driver
> > + *
> > + * Copyright (C) 2017 - 2018 Xilinx, Inc.
> > + *
> > + * Author: Hyun Woo Kwon <hyun.kwon@xilinx.com>
> > + *
> > + * This software is licensed under the terms of the GNU General Public
> > + * License version 2, as published by the Free Software Foundation, and
> > + * may be copied, distributed, and modified under those terms.
> > + *
> > + * This program is distributed in the hope that it will be useful,
> > + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> > + * GNU General Public License for more details.
> > + */
> > +
> > +#include <drm/drmP.h>
> > +
> > +#include <linux/list.h>
> > +
> > +#include "xlnx_crtc.h"
> > +#include "xlnx_drv.h"
> > +
> > +/*
> > + * Overview
> > + * --------
> > + *
> > + * The Xilinx CRTC layer is to enable the custom interface to CRTC drivers.
> > + * The interface is used by Xilinx DRM driver where it needs CRTC
> > + * functionailty. CRTC drivers should attach the desired callbacks
> > + * to struct xlnx_crtc and register the xlnx_crtc with correcsponding
> > + * drm_device. It's highly recommended CRTC drivers register all
> callbacks
> > + * even though many of them are optional.
> > + * The CRTC helper simply walks through the registered CRTC device,
> > + * and call the callbacks.
> > + */
> > +
> > +/**
> > + * struct xlnx_crtc_helper - Xilinx CRTC helper
> > + * @xlnx_crtcs: list of Xilinx CRTC devices
> > + * @lock: lock to protect @xlnx_crtcs
> > + * @drm: back pointer to DRM core
> > + */
> > +struct xlnx_crtc_helper {
> > + struct list_head xlnx_crtcs;
> > + struct mutex lock; /* lock for @xlnx_crtcs */
> > + struct drm_device *drm;
> > +};
> > +
> > +#define XLNX_CRTC_MAX_HEIGHT_WIDTH UINT_MAX
> > +
> > +int xlnx_crtc_helper_enable_vblank(struct xlnx_crtc_helper *helper,
> > + unsigned int crtc_id)
> > +{
> > + struct xlnx_crtc *crtc;
> > +
> > + list_for_each_entry(crtc, &helper->xlnx_crtcs, list)
> > + if (drm_crtc_index(&crtc->crtc) == crtc_id)
> > + if (crtc->enable_vblank)
> > + return crtc->enable_vblank(crtc);
> > + return -ENODEV;
> > +}
> > +
> > +void xlnx_crtc_helper_disable_vblank(struct xlnx_crtc_helper *helper,
> > + unsigned int crtc_id)
> > +{
> > + struct xlnx_crtc *crtc;
> > +
> > + list_for_each_entry(crtc, &helper->xlnx_crtcs, list) {
> > + if (drm_crtc_index(&crtc->crtc) == crtc_id) {
> > + if (crtc->disable_vblank)
> > + crtc->disable_vblank(crtc);
> > + return;
> > + }
> > + }
> > +}
> > +
> > +unsigned int xlnx_crtc_helper_get_align(struct xlnx_crtc_helper *helper)
> > +{
> > + struct xlnx_crtc *crtc;
> > + unsigned int align = 1, tmp;
> > +
> > + list_for_each_entry(crtc, &helper->xlnx_crtcs, list) {
> > + if (crtc->get_align) {
> > + tmp = crtc->get_align(crtc);
> > + align = ALIGN(align, tmp);
> > + }
> > + }
> > +
> > + return align;
> > +}
> > +
> > +u64 xlnx_crtc_helper_get_dma_mask(struct xlnx_crtc_helper *helper)
> > +{
> > + struct xlnx_crtc *crtc;
> > + u64 mask = DMA_BIT_MASK(sizeof(dma_addr_t) * 8), tmp;
> > +
> > + list_for_each_entry(crtc, &helper->xlnx_crtcs, list) {
> > + if (crtc->get_dma_mask) {
> > + tmp = crtc->get_dma_mask(crtc);
> > + mask = min(mask, tmp);
> > + }
> > + }
> > +
> > + return mask;
> > +}
> > +
> > +int xlnx_crtc_helper_get_max_width(struct xlnx_crtc_helper *helper)
> > +{
> > + struct xlnx_crtc *crtc;
> > + unsigned int width = XLNX_CRTC_MAX_HEIGHT_WIDTH, tmp;
> > +
> > + list_for_each_entry(crtc, &helper->xlnx_crtcs, list) {
> > + if (crtc->get_max_width) {
> > + tmp = crtc->get_max_width(crtc);
> > + width = min(width, tmp);
> > + }
> > + }
> > +
> > + return width;
> > +}
> > +
> > +int xlnx_crtc_helper_get_max_height(struct xlnx_crtc_helper *helper)
> > +{
> > + struct xlnx_crtc *crtc;
> > + unsigned int height = XLNX_CRTC_MAX_HEIGHT_WIDTH, tmp;
> > +
> > + list_for_each_entry(crtc, &helper->xlnx_crtcs, list) {
> > + if (crtc->get_max_height) {
> > + tmp = crtc->get_max_height(crtc);
> > + height = min(height, tmp);
> > + }
> > + }
> > +
> > + return height;
> > +}
> > +
> > +uint32_t xlnx_crtc_helper_get_format(struct xlnx_crtc_helper *helper)
> > +{
> > + struct xlnx_crtc *crtc;
> > + u32 format = 0, tmp;
> > +
> > + list_for_each_entry(crtc, &helper->xlnx_crtcs, list) {
> > + if (crtc->get_format) {
> > + tmp = crtc->get_format(crtc);
> > + if (format && format != tmp)
> > + return 0;
> > + format = tmp;
> > + }
> > + }
> > +
> > + return format;
> > +}
> > +
> > +struct xlnx_crtc_helper *xlnx_crtc_helper_init(struct drm_device *drm)
> > +{
> > + struct xlnx_crtc_helper *helper;
> > +
> > + helper = devm_kzalloc(drm->dev, sizeof(*helper), GFP_KERNEL);
> > + if (!helper)
> > + return ERR_PTR(-ENOMEM);
> > +
> > + INIT_LIST_HEAD(&helper->xlnx_crtcs);
> > + mutex_init(&helper->lock);
> > + helper->drm = drm;
> > +
> > + return helper;
> > +}
> > +
> > +void xlnx_crtc_helper_fini(struct drm_device *drm,
> > + struct xlnx_crtc_helper *helper)
> > +{
> > + if (WARN_ON(helper->drm != drm))
> > + return;
> > +
> > + if (WARN_ON(!list_empty(&helper->xlnx_crtcs)))
> > + return;
> > +
> > + mutex_destroy(&helper->lock);
> > + devm_kfree(drm->dev, helper);
> > +}
> > +
> > +void xlnx_crtc_register(struct drm_device *drm, struct xlnx_crtc *crtc)
> > +{
> > + struct xlnx_crtc_helper *helper = xlnx_get_crtc_helper(drm);
> > +
> > + mutex_lock(&helper->lock);
> > + list_add_tail(&crtc->list, &helper->xlnx_crtcs);
> > + mutex_unlock(&helper->lock);
> > +}
> > +EXPORT_SYMBOL_GPL(xlnx_crtc_register);
> > +
> > +void xlnx_crtc_unregister(struct drm_device *drm, struct xlnx_crtc *crtc)
> > +{
> > + struct xlnx_crtc_helper *helper = xlnx_get_crtc_helper(drm);
> > +
> > + mutex_lock(&helper->lock);
> > + list_del(&crtc->list);
> > + mutex_unlock(&helper->lock);
> > +}
> > +EXPORT_SYMBOL_GPL(xlnx_crtc_unregister);
> > diff --git a/drivers/gpu/drm/xlnx/xlnx_crtc.h
> b/drivers/gpu/drm/xlnx/xlnx_crtc.h
> > new file mode 100644
> > index 0000000..1498e10
> > --- /dev/null
> > +++ b/drivers/gpu/drm/xlnx/xlnx_crtc.h
> > @@ -0,0 +1,78 @@
> > +// SPDX-License-Identifier: GPL-2.0
> > +/*
> > + * Xilinx DRM crtc header
> > + *
> > + * Copyright (C) 2017 - 2018 Xilinx, Inc.
> > + *
> > + * Author: Hyun Woo Kwon <hyun.kwon@xilinx.com>
> > + *
> > + * This software is licensed under the terms of the GNU General Public
> > + * License version 2, as published by the Free Software Foundation, and
> > + * may be copied, distributed, and modified under those terms.
> > + *
> > + * This program is distributed in the hope that it will be useful,
> > + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> > + * GNU General Public License for more details.
> > + */
> > +
> > +#ifndef _XLNX_CRTC_H_
> > +#define _XLNX_CRTC_H_
> > +
> > +/**
> > + * struct xlnx_crtc - Xilinx CRTC device
> > + * @crtc: DRM CRTC device
> > + * @list: list node for Xilinx CRTC device list
> > + * @enable_vblank: Enable vblank
> > + * @disable_vblank: Disable vblank
> > + * @get_align: Get the alignment requirement of CRTC device
> > + * @get_dma_mask: Get the dma mask of CRTC device
> > + * @get_max_width: Get the maximum supported width
> > + * @get_max_height: Get the maximum supported height
> > + * @get_format: Get the current format of CRTC device
> > + */
> > +struct xlnx_crtc {
> > + struct drm_crtc crtc;
> > + struct list_head list;
> > + int (*enable_vblank)(struct xlnx_crtc *crtc);
> > + void (*disable_vblank)(struct xlnx_crtc *crtc);
> > + unsigned int (*get_align)(struct xlnx_crtc *crtc);
> > + u64 (*get_dma_mask)(struct xlnx_crtc *crtc);
> > + int (*get_max_width)(struct xlnx_crtc *crtc);
> > + int (*get_max_height)(struct xlnx_crtc *crtc);
> > + uint32_t (*get_format)(struct xlnx_crtc *crtc);
> > +};
> > +
> > +/*
> > + * Helper functions: used within Xlnx DRM
> > + */
> > +
> > +struct xlnx_crtc_helper;
> > +
> > +int xlnx_crtc_helper_enable_vblank(struct xlnx_crtc_helper *helper,
> > + unsigned int crtc_id);
> > +void xlnx_crtc_helper_disable_vblank(struct xlnx_crtc_helper *helper,
> > + unsigned int crtc_id);
> > +unsigned int xlnx_crtc_helper_get_align(struct xlnx_crtc_helper *helper);
> > +u64 xlnx_crtc_helper_get_dma_mask(struct xlnx_crtc_helper *helper);
> > +int xlnx_crtc_helper_get_max_width(struct xlnx_crtc_helper *helper);
> > +int xlnx_crtc_helper_get_max_height(struct xlnx_crtc_helper *helper);
> > +uint32_t xlnx_crtc_helper_get_format(struct xlnx_crtc_helper *helper);
> > +
> > +struct xlnx_crtc_helper *xlnx_crtc_helper_init(struct drm_device *drm);
> > +void xlnx_crtc_helper_fini(struct drm_device *drm,
> > + struct xlnx_crtc_helper *helper);
> > +
> > +/*
> > + * CRTC registration: used by other sub-driver modules
> > + */
> > +
> > +static inline struct xlnx_crtc *to_xlnx_crtc(struct drm_crtc *crtc)
> > +{
> > + return container_of(crtc, struct xlnx_crtc, crtc);
> > +}
> > +
> > +void xlnx_crtc_register(struct drm_device *drm, struct xlnx_crtc *crtc);
> > +void xlnx_crtc_unregister(struct drm_device *drm, struct xlnx_crtc
> *crtc);
> > +
> > +#endif /* _XLNX_CRTC_H_ */
> > diff --git a/drivers/gpu/drm/xlnx/xlnx_drv.c
> b/drivers/gpu/drm/xlnx/xlnx_drv.c
> > new file mode 100644
> > index 0000000..a350492
> > --- /dev/null
> > +++ b/drivers/gpu/drm/xlnx/xlnx_drv.c
> > @@ -0,0 +1,447 @@
> > +// SPDX-License-Identifier: GPL-2.0
> > +/*
> > + * Xilinx DRM KMS Driver
> > + *
> > + * Copyright (C) 2013 - 2018 Xilinx, Inc.
> > + *
> > + * Author: Hyun Woo Kwon <hyun.kwon@xilinx.com>
> > + *
> > + * This software is licensed under the terms of the GNU General Public
> > + * License version 2, as published by the Free Software Foundation, and
> > + * may be copied, distributed, and modified under those terms.
> > + *
> > + * This program is distributed in the hope that it will be useful,
> > + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> > + * GNU General Public License for more details.
> > + */
> > +
> > +#include <drm/drmP.h>
> > +#include <drm/drm_atomic.h>
> > +#include <drm/drm_atomic_helper.h>
> > +#include <drm/drm_crtc_helper.h>
> > +#include <drm/drm_fb_helper.h>
> > +#include <drm/drm_gem_cma_helper.h>
> > +#include <drm/drm_of.h>
> > +
> > +#include <linux/component.h>
> > +#include <linux/device.h>
> > +#include <linux/dma-buf.h>
> > +#include <linux/module.h>
> > +#include <linux/of_graph.h>
> > +#include <linux/platform_device.h>
> > +#include <linux/reservation.h>
> > +
> > +#include "xlnx_crtc.h"
> > +#include "xlnx_fb.h"
> > +#include "xlnx_gem.h"
> > +
> > +#define DRIVER_NAME "xlnx"
> > +#define DRIVER_DESC "Xilinx DRM KMS Driver"
> > +#define DRIVER_DATE "20130509"
> > +#define DRIVER_MAJOR 1
> > +#define DRIVER_MINOR 0
> > +
> > +static uint xlnx_fbdev_vres = 2;
> > +module_param_named(fbdev_vres, xlnx_fbdev_vres, uint, 0444);
> > +MODULE_PARM_DESC(fbdev_vres,
> > + "fbdev virtual resolution multiplier for fb (default: 2)");
> > +
> > +/**
> > + * struct xlnx_drm - Xilinx DRM private data
> > + * @drm: DRM core
> > + * @crtc: Xilinx DRM CRTC helper
> > + * @fb: DRM fb helper
> > + * @pdev: platform device
> > + * @suspend_state: atomic state for suspend / resume
> > + */
> > +struct xlnx_drm {
> > + struct drm_device *drm;
> > + struct xlnx_crtc_helper *crtc;
> > + struct drm_fb_helper *fb;
> > + struct platform_device *pdev;
> > + struct drm_atomic_state *suspend_state;
> > +};
> > +
> > +/**
> > + * xlnx_get_crtc_helper - Return the crtc helper instance
> > + * @drm: DRM device
> > + *
> > + * Return: the crtc helper instance
> > + */
> > +struct xlnx_crtc_helper *xlnx_get_crtc_helper(struct drm_device *drm)
> > +{
> > + struct xlnx_drm *xlnx_drm = drm->dev_private;
> > +
> > + return xlnx_drm->crtc;
> > +}
> > +
> > +/**
> > + * xlnx_get_align - Return the align requirement through CRTC helper
> > + * @drm: DRM device
> > + *
> > + * Return: the alignment requirement
> > + */
> > +unsigned int xlnx_get_align(struct drm_device *drm)
> > +{
> > + struct xlnx_drm *xlnx_drm = drm->dev_private;
> > +
> > + return xlnx_crtc_helper_get_align(xlnx_drm->crtc);
> > +}
> > +
> > +/**
> > + * xlnx_get_format - Return the current format of CRTC
> > + * @drm: DRM device
> > + *
> > + * Return: the current CRTC format
> > + */
> > +uint32_t xlnx_get_format(struct drm_device *drm)
> > +{
> > + struct xlnx_drm *xlnx_drm = drm->dev_private;
> > +
> > + return xlnx_crtc_helper_get_format(xlnx_drm->crtc);
> > +}
> > +
> > +static void xlnx_output_poll_changed(struct drm_device *drm)
> > +{
> > + struct xlnx_drm *xlnx_drm = drm->dev_private;
> > +
> > + if (xlnx_drm->fb)
> > + drm_fb_helper_hotplug_event(xlnx_drm->fb);
> > +}
> > +
> > +static const struct drm_mode_config_funcs xlnx_mode_config_funcs = {
> > + .fb_create = xlnx_fb_create,
> > + .output_poll_changed = xlnx_output_poll_changed,
> > + .atomic_check = drm_atomic_helper_check,
> > + .atomic_commit = drm_atomic_helper_commit,
> > +};
> > +
> > +static int xlnx_enable_vblank(struct drm_device *drm, unsigned int crtc)
> > +{
> > + struct xlnx_drm *xlnx_drm = drm->dev_private;
> > +
> > + return xlnx_crtc_helper_enable_vblank(xlnx_drm->crtc, crtc);
> > +}
> > +
> > +static void xlnx_disable_vblank(struct drm_device *drm, unsigned int
> crtc)
> > +{
> > + struct xlnx_drm *xlnx_drm = drm->dev_private;
> > +
> > + xlnx_crtc_helper_disable_vblank(xlnx_drm->crtc, crtc);
> > +}
> > +
> > +static void xlnx_mode_config_init(struct drm_device *drm)
> > +{
> > + struct xlnx_drm *xlnx_drm = drm->dev_private;
> > + struct xlnx_crtc_helper *crtc = xlnx_drm->crtc;
> > +
> > + drm->mode_config.min_width = 0;
> > + drm->mode_config.min_height = 0;
> > + drm->mode_config.max_width =
> xlnx_crtc_helper_get_max_width(crtc);
> > + drm->mode_config.max_height =
> xlnx_crtc_helper_get_max_height(crtc);
> > +}
> > +
> > +static void xlnx_lastclose(struct drm_device *drm)
> > +{
> > + struct xlnx_drm *xlnx_drm = drm->dev_private;
> > +
> > + if (xlnx_drm->fb)
> > + drm_fb_helper_restore_fbdev_mode_unlocked(xlnx_drm->fb);
> > +}
> > +
> > +static const struct file_operations xlnx_fops = {
> > + .owner = THIS_MODULE,
> > + .open = drm_open,
> > + .release = drm_release,
> > + .unlocked_ioctl = drm_ioctl,
> > + .mmap = drm_gem_cma_mmap,
> > + .poll = drm_poll,
> > + .read = drm_read,
> > +#ifdef CONFIG_COMPAT
> > + .compat_ioctl = drm_compat_ioctl,
> > +#endif
> > + .llseek = noop_llseek,
> > +};
> > +
> > +static struct drm_driver xlnx_drm_driver = {
> > + .driver_features = DRIVER_MODESET | DRIVER_GEM |
> > + DRIVER_ATOMIC | DRIVER_PRIME,
> > + .lastclose = xlnx_lastclose,
> > +
> > + .enable_vblank = xlnx_enable_vblank,
> > + .disable_vblank = xlnx_disable_vblank,
> > +
> > + .prime_handle_to_fd = drm_gem_prime_handle_to_fd,
> > + .prime_fd_to_handle = drm_gem_prime_fd_to_handle,
> > + .gem_prime_export = drm_gem_prime_export,
> > + .gem_prime_import = drm_gem_prime_import,
> > + .gem_prime_get_sg_table =
> drm_gem_cma_prime_get_sg_table,
> > + .gem_prime_import_sg_table =
> drm_gem_cma_prime_import_sg_table,
> > + .gem_prime_vmap = drm_gem_cma_prime_vmap,
> > + .gem_prime_vunmap = drm_gem_cma_prime_vunmap,
> > + .gem_prime_mmap = drm_gem_cma_prime_mmap,
> > + .gem_free_object = drm_gem_cma_free_object,
> > + .gem_vm_ops = &drm_gem_cma_vm_ops,
> > + .dumb_create = xlnx_gem_cma_dumb_create,
> > + .dumb_destroy = drm_gem_dumb_destroy,
> > +
> > + .fops = &xlnx_fops,
> > +
> > + .name = DRIVER_NAME,
> > + .desc = DRIVER_DESC,
> > + .date = DRIVER_DATE,
> > + .major = DRIVER_MAJOR,
> > + .minor = DRIVER_MINOR,
> > +};
> > +
> > +static int xlnx_bind(struct device *dev)
> > +{
> > + struct xlnx_drm *xlnx_drm;
> > + struct drm_device *drm;
> > + const struct drm_format_info *info;
> > + struct platform_device *pdev = to_platform_device(dev);
> > + int ret;
> > + u32 format;
> > +
> > + drm = drm_dev_alloc(&xlnx_drm_driver, &pdev->dev);
> > + if (IS_ERR(drm))
> > + return PTR_ERR(drm);
> > +
> > + xlnx_drm = devm_kzalloc(drm->dev, sizeof(*xlnx_drm),
> GFP_KERNEL);
> > + if (!xlnx_drm) {
> > + ret = -ENOMEM;
> > + goto err_drm;
> > + }
> > +
> > + drm_mode_config_init(drm);
> > + drm->mode_config.funcs = &xlnx_mode_config_funcs;
> > +
> > + ret = drm_vblank_init(drm, 1);
> > + if (ret) {
> > + dev_err(&pdev->dev, "failed to initialize vblank\n");
> > + goto err_xlnx_drm;
> > + }
> > +
> > + drm->irq_enabled = 1;
> > + drm->dev_private = xlnx_drm;
> > + xlnx_drm->drm = drm;
> > + drm_kms_helper_poll_init(drm);
> > + platform_set_drvdata(pdev, xlnx_drm);
> > +
> > + xlnx_drm->crtc = xlnx_crtc_helper_init(drm);
> > + if (IS_ERR(xlnx_drm->crtc)) {
> > + ret = PTR_ERR(xlnx_drm->crtc);
> > + goto err_xlnx_drm;
> > + }
> > +
> > + ret = component_bind_all(drm->dev, drm);
> > + if (ret)
> > + goto err_crtc;
> > +
> > + xlnx_mode_config_init(drm);
> > + drm_mode_config_reset(drm);
> > + dma_set_mask(drm->dev,
> xlnx_crtc_helper_get_dma_mask(xlnx_drm->crtc));
> > +
> > + format = xlnx_crtc_helper_get_format(xlnx_drm->crtc);
> > + info = drm_format_info(format);
> > + if (info && info->depth && info->cpp[0]) {
> > + unsigned int align;
> > +
> > + align = xlnx_crtc_helper_get_align(xlnx_drm->crtc);
> > + xlnx_drm->fb = xlnx_fb_init(drm, info->cpp[0] * 8, 1, align,
> > + xlnx_fbdev_vres);
> > + if (IS_ERR(xlnx_drm->fb)) {
> > + dev_err(&pdev->dev,
> > + "failed to initialize drm fb\n");
> > + xlnx_drm->fb = NULL;
> > + }
> > + } else {
> > + /* fbdev emulation is optional */
> > + dev_info(&pdev->dev, "fbdev is not initialized\n");
> > + }
> > +
> > + ret = drm_dev_register(drm, 0);
> > + if (ret < 0)
> > + goto err_fb;
> > +
> > + return 0;
> > +
> > +err_fb:
> > + if (xlnx_drm->fb)
> > + xlnx_fb_fini(xlnx_drm->fb);
> > + component_unbind_all(drm->dev, drm);
> > +err_crtc:
> > + xlnx_crtc_helper_fini(drm, xlnx_drm->crtc);
> > +err_xlnx_drm:
> > + drm_mode_config_cleanup(drm);
> > +err_drm:
> > + drm_dev_unref(drm);
> > + return ret;
> > +}
> > +
> > +static void xlnx_unbind(struct device *dev)
> > +{
> > + struct xlnx_drm *xlnx_drm = dev_get_drvdata(dev);
> > + struct drm_device *drm = xlnx_drm->drm;
> > +
> > + drm_dev_unregister(drm);
> > + if (xlnx_drm->fb)
> > + xlnx_fb_fini(xlnx_drm->fb);
> > + component_unbind_all(drm->dev, drm);
> > + xlnx_crtc_helper_fini(drm, xlnx_drm->crtc);
> > + drm_kms_helper_poll_fini(drm);
> > + drm_mode_config_cleanup(drm);
> > + drm_dev_unref(drm);
> > +}
> > +
> > +static const struct component_master_ops xlnx_master_ops = {
> > + .bind = xlnx_bind,
> > + .unbind = xlnx_unbind,
> > +};
> > +
> > +static int xlnx_of_component_probe(struct device *dev,
> > + int (*compare_of)(struct device *, void *),
> > + const struct component_master_ops *m_ops)
> > +{
> > + struct device_node *ep, *port, *remote, *parent;
> > + struct component_match *match = NULL;
> > + int i;
> > +
> > + if (!dev->of_node)
> > + return -EINVAL;
> > +
> > + for (i = 0; ; i++) {
> > + port = of_parse_phandle(dev->of_node, "ports", i);
> > + if (!port)
> > + break;
> > +
> > + parent = of_graph_get_port_parent(port);
> > + if (!of_device_is_available(parent)) {
> > + of_node_put(port);
> > + continue;
> > + }
> > +
> > + component_match_add(dev, &match, compare_of, parent);
> > + of_node_put(parent);
> > + of_node_put(port);
> > + }
> > +
> > + if (i == 0) {
> > + dev_err(dev, "missing 'ports' property\n");
> > + return -ENODEV;
> > + }
> > +
> > + if (!match) {
> > + dev_err(dev, "no available port\n");
> > + return -ENODEV;
> > + }
> > +
> > + for (i = 0; ; i++) {
> > + port = of_parse_phandle(dev->of_node, "ports", i);
> > + if (!port)
> > + break;
> > +
> > + parent = of_graph_get_port_parent(port);
> > + if (!of_device_is_available(parent)) {
> > + of_node_put(port);
> > + continue;
> > + }
> > +
> > + for_each_child_of_node(port, ep) {
> > + remote = of_graph_get_remote_port_parent(ep);
> > + if (!remote || !of_device_is_available(remote)) {
> > + of_node_put(remote);
> > + continue;
> > + } else if (!of_device_is_available(remote->parent)) {
> > + dev_warn(dev, "parent dev of %s unavailable\n",
> > + remote->full_name);
> > + of_node_put(remote);
> > + continue;
> > + }
> > + component_match_add(dev, &match, compare_of,
> remote);
> > + of_node_put(remote);
> > + }
> > + of_node_put(port);
> > + }
> > +
> > + return component_master_add_with_match(dev, m_ops, match);
> > +}
> > +
> > +static int xlnx_compare_of(struct device *dev, void *data)
> > +{
> > + return dev->of_node == data;
> > +}
> > +
> > +static int xlnx_platform_probe(struct platform_device *pdev)
> > +{
> > + return xlnx_of_component_probe(&pdev->dev, xlnx_compare_of,
> > + &xlnx_master_ops);
> > +}
> > +
> > +static int xlnx_platform_remove(struct platform_device *pdev)
> > +{
> > + component_master_del(&pdev->dev, &xlnx_master_ops);
> > + return 0;
> > +}
> > +
> > +static void xlnx_platform_shutdown(struct platform_device *pdev)
> > +{
> > + struct xlnx_drm *xlnx_drm = platform_get_drvdata(pdev);
> > +
> > + drm_put_dev(xlnx_drm->drm);
> > +}
> > +
> > +static int __maybe_unused xlnx_pm_suspend(struct device *dev)
> > +{
> > + struct xlnx_drm *xlnx_drm = dev_get_drvdata(dev);
> > + struct drm_device *drm = xlnx_drm->drm;
> > +
> > + drm_kms_helper_poll_disable(drm);
> > +
> > + xlnx_drm->suspend_state = drm_atomic_helper_suspend(drm);
> > + if (IS_ERR(xlnx_drm->suspend_state)) {
> > + drm_kms_helper_poll_enable(drm);
> > + return PTR_ERR(xlnx_drm->suspend_state);
> > + }
> > +
> > + return 0;
> > +}
> > +
> > +static int __maybe_unused xlnx_pm_resume(struct device *dev)
> > +{
> > + struct xlnx_drm *xlnx_drm = dev_get_drvdata(dev);
> > + struct drm_device *drm = xlnx_drm->drm;
> > +
> > + drm_atomic_helper_resume(drm, xlnx_drm->suspend_state);
> > + drm_kms_helper_poll_enable(drm);
> > +
> > + return 0;
> > +}
> > +
> > +static const struct dev_pm_ops xlnx_pm_ops = {
> > + SET_SYSTEM_SLEEP_PM_OPS(xlnx_pm_suspend, xlnx_pm_resume)
> > +};
> > +
> > +static const struct of_device_id xlnx_of_match[] = {
> > + { .compatible = "xlnx,display", },
> > + { /* end of table */ },
> > +};
> > +MODULE_DEVICE_TABLE(of, xlnx_of_match);
> > +
> > +static struct platform_driver xlnx_driver = {
> > + .probe = xlnx_platform_probe,
> > + .remove = xlnx_platform_remove,
> > + .shutdown = xlnx_platform_shutdown,
> > + .driver = {
> > + .name = "xlnx-drm",
> > + .pm = &xlnx_pm_ops,
> > + .of_match_table = xlnx_of_match,
> > + },
> > +};
> > +
> > +module_platform_driver(xlnx_driver);
> > +
> > +MODULE_AUTHOR("Xilinx, Inc.");
> > +MODULE_DESCRIPTION("Xilinx DRM KMS Driver");
> > +MODULE_LICENSE("GPL v2");
> > diff --git a/drivers/gpu/drm/xlnx/xlnx_drv.h
> b/drivers/gpu/drm/xlnx/xlnx_drv.h
> > new file mode 100644
> > index 0000000..8c320fd
> > --- /dev/null
> > +++ b/drivers/gpu/drm/xlnx/xlnx_drv.h
> > @@ -0,0 +1,30 @@
> > +// SPDX-License-Identifier: GPL-2.0
> > +/*
> > + * Xilinx DRM KMS Header for Xilinx
> > + *
> > + * Copyright (C) 2013 - 2018 Xilinx, Inc.
> > + *
> > + * Author: Hyun Woo Kwon <hyunk@xilinx.com>
> > + *
> > + * This software is licensed under the terms of the GNU General Public
> > + * License version 2, as published by the Free Software Foundation, and
> > + * may be copied, distributed, and modified under those terms.
> > + *
> > + * This program is distributed in the hope that it will be useful,
> > + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> > + * GNU General Public License for more details.
> > + */
> > +
> > +#ifndef _XLNX_DRV_H_
> > +#define _XLNX_DRV_H_
> > +
> > +struct drm_device;
> > +struct xlnx_crtc_helper;
> > +
> > +uint32_t xlnx_get_format(struct drm_device *drm);
> > +unsigned int xlnx_get_align(struct drm_device *drm);
> > +struct xlnx_crtc_helper *xlnx_get_crtc_helper(struct drm_device *drm);
> > +struct xlnx_bridge_helper *xlnx_get_bridge_helper(struct drm_device
> *drm);
> > +
> > +#endif /* _XLNX_DRV_H_ */
> > diff --git a/drivers/gpu/drm/xlnx/xlnx_fb.c
> b/drivers/gpu/drm/xlnx/xlnx_fb.c
> > new file mode 100644
> > index 0000000..e72087d
> > --- /dev/null
> > +++ b/drivers/gpu/drm/xlnx/xlnx_fb.c
> > @@ -0,0 +1,298 @@
> > +// SPDX-License-Identifier: GPL-2.0
> > +/*
> > + * Xilinx DRM KMS Framebuffer helper
> > + *
> > + * Copyright (C) 2015 - 2018 Xilinx, Inc.
> > + *
> > + * Author: Hyun Woo Kwon <hyun.kwon@xilinx.com>
> > + *
> > + * Based on drm_fb_cma_helper.c
> > + *
> > + * Copyright (C) 2012 Analog Device Inc.
> > + *
> > + * This software is licensed under the terms of the GNU General Public
> > + * License version 2, as published by the Free Software Foundation, and
> > + * may be copied, distributed, and modified under those terms.
> > + *
> > + * This program is distributed in the hope that it will be useful,
> > + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> > + * GNU General Public License for more details.
> > + */
> > +
> > +#include <drm/drmP.h>
> > +#include <drm/drm_crtc.h>
> > +#include <drm/drm_crtc_helper.h>
> > +#include <drm/drm_fb_helper.h>
> > +#include <drm/drm_gem_cma_helper.h>
> > +#include <drm/drm_gem_framebuffer_helper.h>
> > +
> > +#include "xlnx_drv.h"
> > +#include "xlnx_fb.h"
> > +
> > +#define XLNX_MAX_PLANES 4
> > +
> > +struct xlnx_fbdev {
> > + struct drm_fb_helper fb_helper;
> > + struct drm_framebuffer *fb;
> > + unsigned int align;
> > + unsigned int vres_mult;
> > +};
> > +
> > +static inline struct xlnx_fbdev *to_fbdev(struct drm_fb_helper
> *fb_helper)
> > +{
> > + return container_of(fb_helper, struct xlnx_fbdev, fb_helper);
> > +}
> > +
> > +static struct drm_framebuffer_funcs xlnx_fb_funcs = {
> > + .destroy = drm_gem_fb_destroy,
> > + .create_handle = drm_gem_fb_create_handle,
> > +};
> > +
> > +static int
> > +xlnx_fb_ioctl(struct fb_info *info, unsigned int cmd, unsigned long arg)
> > +{
> > + struct drm_fb_helper *fb_helper = info->par;
> > + unsigned int i;
> > + int ret = 0;
> > +
> > + switch (cmd) {
> > + case FBIO_WAITFORVSYNC:
> > + for (i = 0; i < fb_helper->crtc_count; i++) {
> > + struct drm_mode_set *mode_set;
> > + struct drm_crtc *crtc;
> > +
> > + mode_set = &fb_helper->crtc_info[i].mode_set;
> > + crtc = mode_set->crtc;
> > + ret = drm_crtc_vblank_get(crtc);
> > + if (!ret) {
> > + drm_crtc_wait_one_vblank(crtc);
> > + drm_crtc_vblank_put(crtc);
> > + }
> > + }
> > + return ret;
> > + default:
> > + return -ENOTTY;
> > + }
> > +
> > + return 0;
> > +}
> > +
> > +static struct fb_ops xlnx_fbdev_ops = {
> > + .owner = THIS_MODULE,
> > + .fb_fillrect = sys_fillrect,
> > + .fb_copyarea = sys_copyarea,
> > + .fb_imageblit = sys_imageblit,
> > + .fb_check_var = drm_fb_helper_check_var,
> > + .fb_set_par = drm_fb_helper_set_par,
> > + .fb_blank = drm_fb_helper_blank,
> > + .fb_pan_display = drm_fb_helper_pan_display,
> > + .fb_setcmap = drm_fb_helper_setcmap,
> > + .fb_ioctl = xlnx_fb_ioctl,
> > +};
> > +
> > +/**
> > + * xlnx_fbdev_create - Create the fbdev with a framebuffer
> > + * @fb_helper: fb helper structure
> > + * @size: framebuffer size info
> > + *
> > + * This function is based on drm_fbdev_cma_create().
> > + *
> > + * Return: 0 if successful, or the error code.
> > + */
> > +static int xlnx_fbdev_create(struct drm_fb_helper *fb_helper,
> > + struct drm_fb_helper_surface_size *size)
> > +{
> > + struct xlnx_fbdev *fbdev = to_fbdev(fb_helper);
> > + struct drm_device *drm = fb_helper->dev;
> > + struct drm_gem_cma_object *obj;
> > + struct drm_framebuffer *fb;
> > + unsigned int bytes_per_pixel;
> > + unsigned long offset;
> > + struct fb_info *fbi;
> > + size_t bytes;
> > + int ret;
> > +
> > + dev_dbg(drm->dev, "surface width(%d), height(%d) and bpp(%d)\n",
> > + size->surface_width, size->surface_height, size->surface_bpp);
> > +
> > + size->surface_height *= fbdev->vres_mult;
> > + bytes_per_pixel = DIV_ROUND_UP(size->surface_bpp, 8);
> > + bytes = ALIGN(size->surface_width * bytes_per_pixel, fbdev->align);
> > + bytes *= size->surface_height;
> > +
> > + obj = drm_gem_cma_create(drm, bytes);
> > + if (IS_ERR(obj))
> > + return PTR_ERR(obj);
> > +
> > + fbi = framebuffer_alloc(0, drm->dev);
> > + if (!fbi) {
> > + dev_err(drm->dev, "Failed to allocate framebuffer info.\n");
> > + ret = -ENOMEM;
> > + goto err_drm_gem_cma_free_object;
> > + }
> > +
> > + fbdev->fb = drm_gem_fbdev_fb_create(drm, size, fbdev->align,
> &obj->base,
> > + &xlnx_fb_funcs);
> > + if (IS_ERR(fbdev->fb)) {
> > + dev_err(drm->dev, "Failed to allocate DRM framebuffer.\n");
> > + ret = PTR_ERR(fbdev->fb);
> > + goto err_framebuffer_release;
> > + }
> > +
> > + fb = fbdev->fb;
> > + fb_helper->fb = fb;
> > + fb_helper->fbdev = fbi;
> > + fbi->par = fb_helper;
> > + fbi->flags = FBINFO_FLAG_DEFAULT;
> > + fbi->fbops = &xlnx_fbdev_ops;
> > +
> > + ret = fb_alloc_cmap(&fbi->cmap, 256, 0);
> > + if (ret) {
> > + dev_err(drm->dev, "Failed to allocate color map.\n");
> > + goto err_fb_destroy;
> > + }
> > +
> > + drm_fb_helper_fill_fix(fbi, fb->pitches[0], fb->format->depth);
> > + drm_fb_helper_fill_var(fbi, fb_helper, fb->width, fb->height);
> > + fbi->var.yres = fb->height / fbdev->vres_mult;
> > +
> > + offset = fbi->var.xoffset * bytes_per_pixel;
> > + offset += fbi->var.yoffset * fb->pitches[0];
> > +
> > + drm->mode_config.fb_base = (resource_size_t)obj->paddr;
> > + fbi->screen_base = (char __iomem *)(obj->vaddr + offset);
> > + fbi->fix.smem_start = (unsigned long)(obj->paddr + offset);
> > + fbi->screen_size = bytes;
> > + fbi->fix.smem_len = bytes;
> > +
> > + return 0;
> > +
> > +err_fb_destroy:
> > + drm_framebuffer_unregister_private(fb);
> > + drm_gem_fb_destroy(fb);
> > +err_framebuffer_release:
> > + framebuffer_release(fbi);
> > +err_drm_gem_cma_free_object:
> > + drm_gem_cma_free_object(&obj->base);
> > + return ret;
> > +}
> > +
> > +static struct drm_fb_helper_funcs xlnx_fb_helper_funcs = {
> > + .fb_probe = xlnx_fbdev_create,
> > +};
> > +
> > +/**
> > + * xlnx_fb_init - Allocate and initializes the Xilinx framebuffer
> > + * @drm: DRM device
> > + * @preferred_bpp: preferred bits per pixel for the device
> > + * @max_conn_count: maximum number of connectors
> > + * @align: alignment value for pitch
> > + * @vres_mult: multiplier for virtual resolution
> > + *
> > + * This function is based on drm_fbdev_cma_init().
> > + *
> > + * Return: a newly allocated drm_fb_helper struct or a ERR_PTR.
> > + */
> > +struct drm_fb_helper *
> > +xlnx_fb_init(struct drm_device *drm, int preferred_bpp,
> > + unsigned int max_conn_count, unsigned int align,
> > + unsigned int vres_mult)
> > +{
> > + struct xlnx_fbdev *fbdev;
> > + struct drm_fb_helper *fb_helper;
> > + int ret;
> > +
> > + fbdev = kzalloc(sizeof(*fbdev), GFP_KERNEL);
> > + if (!fbdev)
> > + return ERR_PTR(-ENOMEM);
> > +
> > + fbdev->vres_mult = vres_mult;
> > + fbdev->align = align;
> > + fb_helper = &fbdev->fb_helper;
> > + drm_fb_helper_prepare(drm, fb_helper, &xlnx_fb_helper_funcs);
> > +
> > + ret = drm_fb_helper_init(drm, fb_helper, max_conn_count);
> > + if (ret < 0) {
> > + dev_err(drm->dev, "Failed to initialize drm fb helper.\n");
> > + goto err_free;
> > + }
> > +
> > + ret = drm_fb_helper_single_add_all_connectors(fb_helper);
> > + if (ret < 0) {
> > + dev_err(drm->dev, "Failed to add connectors.\n");
> > + goto err_drm_fb_helper_fini;
> > + }
> > +
> > + ret = drm_fb_helper_initial_config(fb_helper, preferred_bpp);
> > + if (ret < 0) {
> > + dev_err(drm->dev, "Failed to set initial hw configuration.\n");
> > + goto err_drm_fb_helper_fini;
> > + }
> > +
> > + return fb_helper;
> > +
> > +err_drm_fb_helper_fini:
> > + drm_fb_helper_fini(fb_helper);
> > +err_free:
> > + kfree(fbdev);
> > + return ERR_PTR(ret);
> > +}
> > +
> > +/**
> > + * xlnx_fbdev_defio_fini - Free the defio fb
> > + * @fbi: fb_info struct
> > + *
> > + * This function is based on drm_fbdev_cma_defio_fini().
> > + */
> > +static void xlnx_fbdev_defio_fini(struct fb_info *fbi)
> > +{
> > + if (!fbi->fbdefio)
> > + return;
> > +
> > + fb_deferred_io_cleanup(fbi);
> > + kfree(fbi->fbdefio);
> > + kfree(fbi->fbops);
> > +}
> > +
> > +/**
> > + * xlnx_fbdev_fini - Free the Xilinx framebuffer
> > + * @fb_helper: drm_fb_helper struct
> > + *
> > + * This function is based on drm_fbdev_cma_fini().
> > + */
> > +void xlnx_fb_fini(struct drm_fb_helper *fb_helper)
> > +{
> > + struct xlnx_fbdev *fbdev = to_fbdev(fb_helper);
> > +
> > + drm_fb_helper_unregister_fbi(&fbdev->fb_helper);
> > + if (fbdev->fb_helper.fbdev)
> > + xlnx_fbdev_defio_fini(fbdev->fb_helper.fbdev);
> > +
> > + if (fbdev->fb_helper.fb)
> > + drm_framebuffer_remove(fbdev->fb_helper.fb);
> > +
> > + drm_fb_helper_fini(&fbdev->fb_helper);
> > + kfree(fbdev);
> > +}
> > +
> > +/**
> > + * xlnx_fb_create - (struct drm_mode_config_funcs *)->fb_create
> callback
> > + * @drm: DRM device
> > + * @file_priv: drm file private data
> > + * @mode_cmd: mode command for fb creation
> > + *
> > + * This functions creates a drm_framebuffer with xlnx_fb_funcs for given
> mode
> > + * @mode_cmd. This functions is intended to be used for the fb_create
> callback
> > + * function of drm_mode_config_funcs.
> > + *
> > + * Return: a drm_framebuffer object if successful, or
> > + * ERR_PTR from drm_gem_fb_create_with_funcs().
> > + */
> > +struct drm_framebuffer *
> > +xlnx_fb_create(struct drm_device *drm, struct drm_file *file_priv,
> > + const struct drm_mode_fb_cmd2 *mode_cmd)
> > +{
> > + return drm_gem_fb_create_with_funcs(drm, file_priv, mode_cmd,
> > + &xlnx_fb_funcs);
> > +}
> > diff --git a/drivers/gpu/drm/xlnx/xlnx_fb.h
> b/drivers/gpu/drm/xlnx/xlnx_fb.h
> > new file mode 100644
> > index 0000000..6efc985
> > --- /dev/null
> > +++ b/drivers/gpu/drm/xlnx/xlnx_fb.h
> > @@ -0,0 +1,33 @@
> > +// SPDX-License-Identifier: GPL-2.0
> > +/*
> > + * Xilinx DRM KMS Framebuffer helper header
> > + *
> > + * Copyright (C) 2015 - 2018 Xilinx, Inc.
> > + *
> > + * Author: Hyun Woo Kwon <hyun.kwon@xilinx.com>
> > + *
> > + * This software is licensed under the terms of the GNU General Public
> > + * License version 2, as published by the Free Software Foundation, and
> > + * may be copied, distributed, and modified under those terms.
> > + *
> > + * This program is distributed in the hope that it will be useful,
> > + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> > + * GNU General Public License for more details.
> > + */
> > +
> > +#ifndef _XLNX_FB_H_
> > +#define _XLNX_FB_H_
> > +
> > +struct drm_fb_helper;
> > +
> > +struct drm_framebuffer *
> > +xlnx_fb_create(struct drm_device *drm, struct drm_file *file_priv,
> > + const struct drm_mode_fb_cmd2 *mode_cmd);
> > +struct drm_fb_helper *
> > +xlnx_fb_init(struct drm_device *drm, int preferred_bpp,
> > + unsigned int max_conn_count, unsigned int align,
> > + unsigned int vres_mult);
> > +void xlnx_fb_fini(struct drm_fb_helper *fb_helper);
> > +
> > +#endif /* _XLNX_FB_H_ */
> > diff --git a/drivers/gpu/drm/xlnx/xlnx_gem.c
> b/drivers/gpu/drm/xlnx/xlnx_gem.c
> > new file mode 100644
> > index 0000000..4a5d533
> > --- /dev/null
> > +++ b/drivers/gpu/drm/xlnx/xlnx_gem.c
> > @@ -0,0 +1,47 @@
> > +// SPDX-License-Identifier: GPL-2.0
> > +/*
> > + * Xilinx DRM KMS GEM helper
> > + *
> > + * Copyright (C) 2015 - 2018 Xilinx, Inc.
> > + *
> > + * Author: Hyun Woo Kwon <hyun.kwon@xilinx.com>
> > + *
> > + * This software is licensed under the terms of the GNU General Public
> > + * License version 2, as published by the Free Software Foundation, and
> > + * may be copied, distributed, and modified under those terms.
> > + *
> > + * This program is distributed in the hope that it will be useful,
> > + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> > + * GNU General Public License for more details.
> > + */
> > +
> > +#include <drm/drmP.h>
> > +#include <drm/drm_gem_cma_helper.h>
> > +
> > +#include "xlnx_drv.h"
> > +#include "xlnx_gem.h"
> > +
> > +/*
> > + * xlnx_gem_cma_dumb_create - (struct drm_driver)->dumb_create
> callback
> > + * @file_priv: drm_file object
> > + * @drm: DRM object
> > + * @args: info for dumb scanout buffer creation
> > + *
> > + * This function is for dumb_create callback of drm_driver struct. Simply
> > + * it wraps around drm_gem_cma_dumb_create() and sets the pitch
> value
> > + * by retrieving the value from the device.
> > + *
> > + * Return: The return value from drm_gem_cma_dumb_create()
> > + */
> > +int xlnx_gem_cma_dumb_create(struct drm_file *file_priv, struct
> drm_device *drm,
> > + struct drm_mode_create_dumb *args)
> > +{
> > + int pitch = DIV_ROUND_UP(args->width * args->bpp, 8);
> > + unsigned int align = xlnx_get_align(drm);
> > +
> > + if (!args->pitch || !IS_ALIGNED(args->pitch, align))
> > + args->pitch = ALIGN(pitch, align);
> > +
> > + return drm_gem_cma_dumb_create_internal(file_priv, drm, args);
> > +}
> > diff --git a/drivers/gpu/drm/xlnx/xlnx_gem.h
> b/drivers/gpu/drm/xlnx/xlnx_gem.h
> > new file mode 100644
> > index 0000000..f380de9
> > --- /dev/null
> > +++ b/drivers/gpu/drm/xlnx/xlnx_gem.h
> > @@ -0,0 +1,26 @@
> > +// SPDX-License-Identifier: GPL-2.0
> > +/*
> > + * Xilinx DRM KMS GEM helper header
> > + *
> > + * Copyright (C) 2015 - 2018 Xilinx, Inc.
> > + *
> > + * Author: Hyun Woo Kwon <hyun.kwon@xilinx.com>
> > + *
> > + * This software is licensed under the terms of the GNU General Public
> > + * License version 2, as published by the Free Software Foundation, and
> > + * may be copied, distributed, and modified under those terms.
> > + *
> > + * This program is distributed in the hope that it will be useful,
> > + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> > + * GNU General Public License for more details.
> > + */
> > +
> > +#ifndef _XLNX_GEM_H_
> > +#define _XLNX_GEM_H_
> > +
> > +int xlnx_gem_cma_dumb_create(struct drm_file *file_priv,
> > + struct drm_device *drm,
> > + struct drm_mode_create_dumb *args);
> > +
> > +#endif /* _XLNX_GEM_H_ */
> > --
> > 2.7.4
> >
> > This email and any attachments are intended for the sole use of the
> named recipient(s) and contain(s) confidential information that may be
> proprietary, privileged or copyrighted under applicable law. If you are not
> the intended recipient, do not read, copy, or forward this email message or
> any attachments. Delete this email message and any attachments
> immediately.
>
> --
> Daniel Vetter
> Software Engineer, Intel Corporation
> http://blog.ffwll.ch
next prev parent reply other threads:[~2018-01-26 1:36 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-01-16 1:57 [PATCH v3 1/8] dt-bindings: display: xlnx: Add bindings for Xilinx display pipeline Hyun Kwon
[not found] ` <1516067831-11382-1-git-send-email-hyun.kwon-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org>
2018-01-16 1:57 ` [PATCH v3 2/8] drm: xlnx: Xilinx DRM KMS driver Hyun Kwon
[not found] ` <1516067831-11382-2-git-send-email-hyun.kwon-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org>
2018-01-17 8:11 ` Daniel Vetter
2018-01-17 8:20 ` Daniel Vetter
2018-01-19 23:33 ` Rob Herring
[not found] ` <20180117082028.GC2759-dv86pmgwkMBes7Z6vYuT8azUEOm+Xw19@public.gmane.org>
2018-01-26 1:36 ` Hyun Kwon [this message]
2018-01-16 1:57 ` [PATCH v3 3/8] dt-bindings: display: xlnx: Add ZynqMP DP subsystem bindings Hyun Kwon
[not found] ` <1516067831-11382-3-git-send-email-hyun.kwon-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org>
2018-01-20 0:31 ` Rob Herring
2018-01-20 2:22 ` Hyun Kwon
2018-01-22 14:36 ` Rob Herring
2018-01-16 1:57 ` [PATCH v3 4/8] drm: xlnx: DRM KMS driver for Xilinx ZynqMP DP subsystem display Hyun Kwon
2018-01-16 1:57 ` [PATCH v3 5/8] drm: xlnx: DRM KMS driver for Xilinx ZynqMP DisplayPort Hyun Kwon
2018-01-16 1:57 ` [PATCH v3 6/8] drm: xlnx: ZynqMP DP subsystem DRM KMS driver Hyun Kwon
2018-01-16 1:57 ` [PATCH v3 7/8] drm: xlnx: zynqmp_disp: Add drm properties Hyun Kwon
2018-01-16 1:57 ` [PATCH v3 8/8] drm: xlnx: zynqmp_dp: " Hyun Kwon
2018-01-19 23:32 ` [PATCH v3 1/8] dt-bindings: display: xlnx: Add bindings for Xilinx display pipeline Rob Herring
2018-01-20 2:22 ` Hyun Kwon
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=MWHPR02MB2493AA0C2AE0E1F80D57225BD6E00@MWHPR02MB2493.namprd02.prod.outlook.com \
--to=hyunk-gjffaj9ahvfqt0dzr+alfa@public.gmane.org \
--cc=daniel-/w4YWyX8dFk@public.gmane.org \
--cc=daniel.vetter-/w4YWyX8dFk@public.gmane.org \
--cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org \
--cc=laurent.pinchart-ryLnwIuWjnjg/C1BVhZhaw@public.gmane.org \
--cc=michal.simek-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org \
--cc=robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).