[PATCH] Revert "video: fbdev: amba-clcd: Retire elder CLCD driver"

Peter Collingbourne pcc at google.com
Tue Sep 29 16:51:36 EDT 2020


On Tue, Sep 29, 2020 at 11:44 AM Daniel Vetter <daniel at ffwll.ch> wrote:
>
> On Tue, Sep 29, 2020 at 7:49 PM Peter Collingbourne <pcc at google.com> wrote:
> >
> > On Tue, Sep 29, 2020 at 9:52 AM Daniel Vetter <daniel at ffwll.ch> wrote:
> > >
> > > On Tue, Sep 29, 2020 at 06:48:28PM +0200, Daniel Vetter wrote:
> > > > On Tue, Sep 29, 2020 at 09:30:08AM -0700, Peter Collingbourne wrote:
> > > > > On Tue, Sep 29, 2020 at 2:32 AM Daniel Vetter <daniel at ffwll.ch> wrote:
> > > > > >
> > > > > > On Tue, Sep 29, 2020 at 09:28:56AM +0200, Neil Armstrong wrote:
> > > > > > > Hi,
> > > > > > >
> > > > > > > On 28/09/2020 22:08, Peter Collingbourne wrote:
> > > > > > > > Also revert the follow-up change "drm: pl111: Absorb the external
> > > > > > > > register header".
> > > > > > > >
> > > > > > > > This reverts commits 7e4e589db76a3cf4c1f534eb5a09cc6422766b93
> > > > > > > > and 0fb8125635e8eb5483fb095f98dcf0651206a7b8.
> > > > > > > >
> > > > > > > > The fbdev driver is used by Android's FVP configuration. Using the
> > > > > > > > DRM driver together with DRM's fbdev emulation results in a failure
> > > > > > > > to boot Android. The root cause is that Android's generic fbdev
> > > > > > > > userspace driver relies on the ability to set the pixel format via
> > > > > > > > FBIOPUT_VSCREENINFO, which is not supported by fbdev emulation.
> > > > > > >
> > > > > > > Can't Android FVP use drm-hwcomposer instead ?
> > > > >
> > > > > Not without kernel changes. See e.g.
> > > > > https://www.spinics.net/lists/dri-devel/msg255883.html
> > > >
> > > > That discussion seems to have died down with no further action.
> > > >
> > > > I also was kinda under the assumption that with Android these buffers
> > > > would be allocated directly from dma-buf heaps/ion, so this all seems not
> > > > that well baked out.
> >
> > The disagreement about whether these allocations should be made via
> > the render node or via dma-buf/ion is one reason why it was hard to
> > make progress on this, unfortunately.
>
> Yeah, using dumb buffer create to allocate random buffers for shared
> software rendering isn't super popular move.
>
> But aside from all this, why is this blocking the migration from fbdev
> to drm? With fbdev you don't have buffer allocations, nor dma-buf
> support, and somehow android can boot. But on drm you can't boot if
> these things are not available. That sounds like the bar for drm is
> maybe a tad too high for simple dumb kms drivers like pl1111 which are
> just there to get a picture on the screen/panel.

I would not intend to use dumb buffer create to allocate
non-framebuffer-destined buffers for software rendering.

With the generic fbdev driver any allocations not destined for the
framebuffer use ashmem, which is how the driver avoids making huge
allocations in fbdev space. I would intend to do something similar for
DRM where framebuffer-destined allocations use dumb buffer create and
non-framebuffer-destined allocations use dma-buf. At the time I
prototyped this approach on the Android side [1] and together with my
render-nodes-everywhere patch and some other hopefully-uncontroversial
changes it worked and let me boot to the home screen. I would be very
happy if this approach were allowed on render nodes so that Android
FVP would be unblocked from moving off fbdev, but at the point where
we left the thread off last time you weren't sure whether it would be
acceptable [2].

One thing that I might not have mentioned on the thread last time was
that render nodes have the advantage that unlike primary nodes,
opening them does not take master if it is not already taken. This
means that on Android, using primary nodes instead of render nodes in
the process responsible for creating frame buffers creates a startup
race condition for master between the hwcomposer process (which
normally opens the primary node and is responsible for flipping
frames) and the surfaceflinger process (which normally opens the
render node and is responsible for drawing onto the framebuffer and
exporting its frames to the composer).

Peter

[1] https://chromium-review.googlesource.com/c/chromiumos/platform/minigbm/+/2438955
[2] https://www.spinics.net/lists/dri-devel/msg256559.html

> -Daniel
>
> >
> > > > > > Also, if we need to add more random fbdev ioctls to the drm fbdev
> > > > > > emulation, then let's do that. Not keep fbdev drivers on life support for
> > > > > > longer than necessary.
> > > > >
> > > > > That should have been done *before* removing the old driver, which was
> > > > > a userspace break that was introduced in 5.9rc1. We shouldn't leave
> > > > > userspace broken for the period of time that it would take to develop
> > > > > that change. Even if such a change were developed before 5.9 is
> > > > > released, it probably wouldn't be considered a bug fix that would be
> > > > > eligible for 5.9.
> > > > >
> > > > > >
> > > > > > >
> > > > > > > Neil
> > > > > > >
> > > > > > > >
> > > > > > > > There have been other less critical behavioral differences identified
> > > > > > > > between the fbdev driver and the DRM driver with fbdev emulation. The
> > > > > > > > DRM driver exposes different values for the panel's width, height and
> > > > > > > > refresh rate, and the DRM driver fails a FBIOPUT_VSCREENINFO syscall
> > > > > > > > with yres_virtual greater than the maximum supported value instead
> > > > > > > > of letting the syscall succeed and setting yres_virtual based on yres.
> > > > > >
> > > > > > Also something that should be fixed I think in upstream, in the drm fbdev
> > > > > > emulation. At least doesn't sound like it's something unfixable.
> > > > >
> > > > > Again, it should have been fixed before removing the old driver.
> > > >
> > > > Yeah, but we also don't have a whole lot people who seem to push for this.
> > > > So the only way to find the people who still care is to remove the fbdev
> > > > drivers.
> > > >
> > > > fbdev is dead. I'm totally fine reverting the driver to shut up the
> > > > regression report, but it's not fixing anything long term. There's not
> > > > going to be new drivers, or new hw support in existing drivers in fbdev.
> > > >
> > > > Also note that there's not spec nor test suite for fbdev, so "you guys
> > > > should have known before removing drivers" doesn't work.
> > >
> > > btw revert itself is stuck somewhere, so I can't find it. And for the
> >
> > You should be able to download the commit like this (the commit
> > message will need amending to remove most of the trailers at the
> > bottom, though):
> >
> > git fetch https://linux.googlesource.com/linux/kernel/git/torvalds/linux
> > refs/changes/52/2952/1
> >
> > > revert can you pls just resurrect the files in drivers/video, without
> > > touching anything in drivers/gpu? Tying down drm drivers with stuff in
> > > fbdev doesn't make much sense to me.
> >
> > The touching of code in drm was a consequence of needing to revert a
> > follow-up commit which moved code previously shared between fbdev and
> > drm into the drm driver.
> >
> > Or do you think we should "manually" revert the first commit and as a
> > consequence end up with the two copies of the code?
> >
> > Peter
> >
> > >
> > > Thanks, Daniel
> > >
> > > > -Daniel
> > > >
> > > > >
> > > > > Peter
> > > > >
> > > > > > -Daniel
> > > > > >
> > > > > > > >
> > > > > > > > Signed-off-by: Peter Collingbourne <pcc at google.com>
> > > > > > > > ---
> > > > > > > > View this change in Gerrit: https://linux-review.googlesource.com/q/I2d7e59b0e693d9fec206d40df190c5aa02844b56
> > > > > > > >
> > > > > > > >  MAINTAINERS                             |   5 +
> > > > > > > >  drivers/gpu/drm/pl111/pl111_debugfs.c   |   1 +
> > > > > > > >  drivers/gpu/drm/pl111/pl111_display.c   |   1 +
> > > > > > > >  drivers/gpu/drm/pl111/pl111_drm.h       |  73 --
> > > > > > > >  drivers/gpu/drm/pl111/pl111_drv.c       |   1 +
> > > > > > > >  drivers/gpu/drm/pl111/pl111_versatile.c |   1 +
> > > > > > > >  drivers/video/fbdev/Kconfig             |  20 +
> > > > > > > >  drivers/video/fbdev/Makefile            |   1 +
> > > > > > > >  drivers/video/fbdev/amba-clcd.c         | 986 ++++++++++++++++++++++++
> > > > > > > >  include/linux/amba/clcd-regs.h          |  87 +++
> > > > > > > >  include/linux/amba/clcd.h               | 290 +++++++
> > > > > > > >  11 files changed, 1393 insertions(+), 73 deletions(-)
> > > > > > > >  create mode 100644 drivers/video/fbdev/amba-clcd.c
> > > > > > > >  create mode 100644 include/linux/amba/clcd-regs.h
> > > > > > > >  create mode 100644 include/linux/amba/clcd.h
> > > > > > > >
> > > > > > > > diff --git a/MAINTAINERS b/MAINTAINERS
> > > > > > > > index 190c7fa2ea01..671c1fa79e64 100644
> > > > > > > > --- a/MAINTAINERS
> > > > > > > > +++ b/MAINTAINERS
> > > > > > > > @@ -1460,6 +1460,11 @@ S:   Odd Fixes
> > > > > > > >  F: drivers/amba/
> > > > > > > >  F: include/linux/amba/bus.h
> > > > > > > >
> > > > > > > > +ARM PRIMECELL CLCD PL110 DRIVER
> > > > > > > > +M: Russell King <linux at armlinux.org.uk>
> > > > > > > > +S: Odd Fixes
> > > > > > > > +F: drivers/video/fbdev/amba-clcd.*
> > > > > > > > +
> > > > > > > >  ARM PRIMECELL KMI PL050 DRIVER
> > > > > > > >  M: Russell King <linux at armlinux.org.uk>
> > > > > > > >  S: Odd Fixes
> > > > > > > > diff --git a/drivers/gpu/drm/pl111/pl111_debugfs.c b/drivers/gpu/drm/pl111/pl111_debugfs.c
> > > > > > > > index 317f68abf18b..26ca8cdf3e60 100644
> > > > > > > > --- a/drivers/gpu/drm/pl111/pl111_debugfs.c
> > > > > > > > +++ b/drivers/gpu/drm/pl111/pl111_debugfs.c
> > > > > > > > @@ -3,6 +3,7 @@
> > > > > > > >   *  Copyright © 2017 Broadcom
> > > > > > > >   */
> > > > > > > >
> > > > > > > > +#include <linux/amba/clcd-regs.h>
> > > > > > > >  #include <linux/seq_file.h>
> > > > > > > >
> > > > > > > >  #include <drm/drm_debugfs.h>
> > > > > > > > diff --git a/drivers/gpu/drm/pl111/pl111_display.c b/drivers/gpu/drm/pl111/pl111_display.c
> > > > > > > > index b3e8697cafcf..703ddc803c55 100644
> > > > > > > > --- a/drivers/gpu/drm/pl111/pl111_display.c
> > > > > > > > +++ b/drivers/gpu/drm/pl111/pl111_display.c
> > > > > > > > @@ -9,6 +9,7 @@
> > > > > > > >   * Copyright (C) 2011 Texas Instruments
> > > > > > > >   */
> > > > > > > >
> > > > > > > > +#include <linux/amba/clcd-regs.h>
> > > > > > > >  #include <linux/clk.h>
> > > > > > > >  #include <linux/delay.h>
> > > > > > > >  #include <linux/version.h>
> > > > > > > > diff --git a/drivers/gpu/drm/pl111/pl111_drm.h b/drivers/gpu/drm/pl111/pl111_drm.h
> > > > > > > > index 2a46b5bd8576..ba399bcb792f 100644
> > > > > > > > --- a/drivers/gpu/drm/pl111/pl111_drm.h
> > > > > > > > +++ b/drivers/gpu/drm/pl111/pl111_drm.h
> > > > > > > > @@ -23,79 +23,6 @@
> > > > > > > >  #include <drm/drm_panel.h>
> > > > > > > >  #include <drm/drm_simple_kms_helper.h>
> > > > > > > >
> > > > > > > > -/*
> > > > > > > > - * CLCD Controller Internal Register addresses
> > > > > > > > - */
> > > > > > > > -#define CLCD_TIM0          0x00000000
> > > > > > > > -#define CLCD_TIM1          0x00000004
> > > > > > > > -#define CLCD_TIM2          0x00000008
> > > > > > > > -#define CLCD_TIM3          0x0000000c
> > > > > > > > -#define CLCD_UBAS          0x00000010
> > > > > > > > -#define CLCD_LBAS          0x00000014
> > > > > > > > -
> > > > > > > > -#define CLCD_PL110_IENB            0x00000018
> > > > > > > > -#define CLCD_PL110_CNTL            0x0000001c
> > > > > > > > -#define CLCD_PL110_STAT            0x00000020
> > > > > > > > -#define CLCD_PL110_INTR            0x00000024
> > > > > > > > -#define CLCD_PL110_UCUR            0x00000028
> > > > > > > > -#define CLCD_PL110_LCUR            0x0000002C
> > > > > > > > -
> > > > > > > > -#define CLCD_PL111_CNTL            0x00000018
> > > > > > > > -#define CLCD_PL111_IENB            0x0000001c
> > > > > > > > -#define CLCD_PL111_RIS             0x00000020
> > > > > > > > -#define CLCD_PL111_MIS             0x00000024
> > > > > > > > -#define CLCD_PL111_ICR             0x00000028
> > > > > > > > -#define CLCD_PL111_UCUR            0x0000002c
> > > > > > > > -#define CLCD_PL111_LCUR            0x00000030
> > > > > > > > -
> > > > > > > > -#define CLCD_PALL          0x00000200
> > > > > > > > -#define CLCD_PALETTE               0x00000200
> > > > > > > > -
> > > > > > > > -#define TIM2_PCD_LO_MASK   GENMASK(4, 0)
> > > > > > > > -#define TIM2_PCD_LO_BITS   5
> > > > > > > > -#define TIM2_CLKSEL                (1 << 5)
> > > > > > > > -#define TIM2_ACB_MASK              GENMASK(10, 6)
> > > > > > > > -#define TIM2_IVS           (1 << 11)
> > > > > > > > -#define TIM2_IHS           (1 << 12)
> > > > > > > > -#define TIM2_IPC           (1 << 13)
> > > > > > > > -#define TIM2_IOE           (1 << 14)
> > > > > > > > -#define TIM2_BCD           (1 << 26)
> > > > > > > > -#define TIM2_PCD_HI_MASK   GENMASK(31, 27)
> > > > > > > > -#define TIM2_PCD_HI_BITS   5
> > > > > > > > -#define TIM2_PCD_HI_SHIFT  27
> > > > > > > > -
> > > > > > > > -#define CNTL_LCDEN         (1 << 0)
> > > > > > > > -#define CNTL_LCDBPP1               (0 << 1)
> > > > > > > > -#define CNTL_LCDBPP2               (1 << 1)
> > > > > > > > -#define CNTL_LCDBPP4               (2 << 1)
> > > > > > > > -#define CNTL_LCDBPP8               (3 << 1)
> > > > > > > > -#define CNTL_LCDBPP16              (4 << 1)
> > > > > > > > -#define CNTL_LCDBPP16_565  (6 << 1)
> > > > > > > > -#define CNTL_LCDBPP16_444  (7 << 1)
> > > > > > > > -#define CNTL_LCDBPP24              (5 << 1)
> > > > > > > > -#define CNTL_LCDBW         (1 << 4)
> > > > > > > > -#define CNTL_LCDTFT                (1 << 5)
> > > > > > > > -#define CNTL_LCDMONO8              (1 << 6)
> > > > > > > > -#define CNTL_LCDDUAL               (1 << 7)
> > > > > > > > -#define CNTL_BGR           (1 << 8)
> > > > > > > > -#define CNTL_BEBO          (1 << 9)
> > > > > > > > -#define CNTL_BEPO          (1 << 10)
> > > > > > > > -#define CNTL_LCDPWR                (1 << 11)
> > > > > > > > -#define CNTL_LCDVCOMP(x)   ((x) << 12)
> > > > > > > > -#define CNTL_LDMAFIFOTIME  (1 << 15)
> > > > > > > > -#define CNTL_WATERMARK             (1 << 16)
> > > > > > > > -
> > > > > > > > -/* ST Microelectronics variant bits */
> > > > > > > > -#define CNTL_ST_1XBPP_444  0x0
> > > > > > > > -#define CNTL_ST_1XBPP_5551 (1 << 17)
> > > > > > > > -#define CNTL_ST_1XBPP_565  (1 << 18)
> > > > > > > > -#define CNTL_ST_CDWID_12   0x0
> > > > > > > > -#define CNTL_ST_CDWID_16   (1 << 19)
> > > > > > > > -#define CNTL_ST_CDWID_18   (1 << 20)
> > > > > > > > -#define CNTL_ST_CDWID_24   ((1 << 19) | (1 << 20))
> > > > > > > > -#define CNTL_ST_CEAEN              (1 << 21)
> > > > > > > > -#define CNTL_ST_LCDBPP24_PACKED    (6 << 1)
> > > > > > > > -
> > > > > > > >  #define CLCD_IRQ_NEXTBASE_UPDATE BIT(2)
> > > > > > > >
> > > > > > > >  struct drm_minor;
> > > > > > > > diff --git a/drivers/gpu/drm/pl111/pl111_drv.c b/drivers/gpu/drm/pl111/pl111_drv.c
> > > > > > > > index 96e58fda75d8..da0c39dae874 100644
> > > > > > > > --- a/drivers/gpu/drm/pl111/pl111_drv.c
> > > > > > > > +++ b/drivers/gpu/drm/pl111/pl111_drv.c
> > > > > > > > @@ -47,6 +47,7 @@
> > > > > > > >   */
> > > > > > > >
> > > > > > > >  #include <linux/amba/bus.h>
> > > > > > > > +#include <linux/amba/clcd-regs.h>
> > > > > > > >  #include <linux/dma-buf.h>
> > > > > > > >  #include <linux/module.h>
> > > > > > > >  #include <linux/of.h>
> > > > > > > > diff --git a/drivers/gpu/drm/pl111/pl111_versatile.c b/drivers/gpu/drm/pl111/pl111_versatile.c
> > > > > > > > index bdd883f4f0da..1127082e9a60 100644
> > > > > > > > --- a/drivers/gpu/drm/pl111/pl111_versatile.c
> > > > > > > > +++ b/drivers/gpu/drm/pl111/pl111_versatile.c
> > > > > > > > @@ -9,6 +9,7 @@
> > > > > > > >   * Major contributions and discoveries by Russell King.
> > > > > > > >   */
> > > > > > > >
> > > > > > > > +#include <linux/amba/clcd-regs.h>
> > > > > > > >  #include <linux/bitops.h>
> > > > > > > >  #include <linux/device.h>
> > > > > > > >  #include <linux/mfd/syscon.h>
> > > > > > > > diff --git a/drivers/video/fbdev/Kconfig b/drivers/video/fbdev/Kconfig
> > > > > > > > index b2c9dd4f0cb5..402e85450bb5 100644
> > > > > > > > --- a/drivers/video/fbdev/Kconfig
> > > > > > > > +++ b/drivers/video/fbdev/Kconfig
> > > > > > > > @@ -272,6 +272,26 @@ config FB_PM2_FIFO_DISCONNECT
> > > > > > > >     help
> > > > > > > >       Support the Permedia2 FIFO disconnect feature.
> > > > > > > >
> > > > > > > > +config FB_ARMCLCD
> > > > > > > > +   tristate "ARM PrimeCell PL110 support"
> > > > > > > > +   depends on ARM || ARM64 || COMPILE_TEST
> > > > > > > > +   depends on FB && ARM_AMBA && HAS_IOMEM
> > > > > > > > +   select FB_CFB_FILLRECT
> > > > > > > > +   select FB_CFB_COPYAREA
> > > > > > > > +   select FB_CFB_IMAGEBLIT
> > > > > > > > +   select FB_MODE_HELPERS if OF
> > > > > > > > +   select VIDEOMODE_HELPERS if OF
> > > > > > > > +   select BACKLIGHT_CLASS_DEVICE if OF
> > > > > > > > +   help
> > > > > > > > +     This framebuffer device driver is for the ARM PrimeCell PL110
> > > > > > > > +     Colour LCD controller.  ARM PrimeCells provide the building
> > > > > > > > +     blocks for System on a Chip devices.
> > > > > > > > +
> > > > > > > > +     If you want to compile this as a module (=code which can be
> > > > > > > > +     inserted into and removed from the running kernel), say M
> > > > > > > > +     here and read <file:Documentation/kbuild/modules.rst>.  The module
> > > > > > > > +     will be called amba-clcd.
> > > > > > > > +
> > > > > > > >  config FB_ACORN
> > > > > > > >     bool "Acorn VIDC support"
> > > > > > > >     depends on (FB = y) && ARM && ARCH_ACORN
> > > > > > > > diff --git a/drivers/video/fbdev/Makefile b/drivers/video/fbdev/Makefile
> > > > > > > > index cad4fb64442a..a0705b99e643 100644
> > > > > > > > --- a/drivers/video/fbdev/Makefile
> > > > > > > > +++ b/drivers/video/fbdev/Makefile
> > > > > > > > @@ -75,6 +75,7 @@ obj-$(CONFIG_FB_HIT)              += hitfb.o
> > > > > > > >  obj-$(CONFIG_FB_ATMEL)               += atmel_lcdfb.o
> > > > > > > >  obj-$(CONFIG_FB_PVR2)             += pvr2fb.o
> > > > > > > >  obj-$(CONFIG_FB_VOODOO1)          += sstfb.o
> > > > > > > > +obj-$(CONFIG_FB_ARMCLCD)     += amba-clcd.o
> > > > > > > >  obj-$(CONFIG_FB_GOLDFISH)         += goldfishfb.o
> > > > > > > >  obj-$(CONFIG_FB_68328)            += 68328fb.o
> > > > > > > >  obj-$(CONFIG_FB_GBE)              += gbefb.o
> > > > > > > > diff --git a/drivers/video/fbdev/amba-clcd.c b/drivers/video/fbdev/amba-clcd.c
> > > > > > > > new file mode 100644
> > > > > > > > index 000000000000..b7682de412d8
> > > > > > > > --- /dev/null
> > > > > > > > +++ b/drivers/video/fbdev/amba-clcd.c
> > > > > > > > @@ -0,0 +1,986 @@
> > > > > > > > +/*
> > > > > > > > + *  linux/drivers/video/amba-clcd.c
> > > > > > > > + *
> > > > > > > > + * Copyright (C) 2001 ARM Limited, by David A Rusling
> > > > > > > > + * Updated to 2.5, Deep Blue Solutions Ltd.
> > > > > > > > + *
> > > > > > > > + * This file is subject to the terms and conditions of the GNU General Public
> > > > > > > > + * License.  See the file COPYING in the main directory of this archive
> > > > > > > > + * for more details.
> > > > > > > > + *
> > > > > > > > + *  ARM PrimeCell PL110 Color LCD Controller
> > > > > > > > + */
> > > > > > > > +#include <linux/amba/bus.h>
> > > > > > > > +#include <linux/amba/clcd.h>
> > > > > > > > +#include <linux/backlight.h>
> > > > > > > > +#include <linux/clk.h>
> > > > > > > > +#include <linux/delay.h>
> > > > > > > > +#include <linux/dma-mapping.h>
> > > > > > > > +#include <linux/fb.h>
> > > > > > > > +#include <linux/init.h>
> > > > > > > > +#include <linux/ioport.h>
> > > > > > > > +#include <linux/list.h>
> > > > > > > > +#include <linux/mm.h>
> > > > > > > > +#include <linux/module.h>
> > > > > > > > +#include <linux/of_address.h>
> > > > > > > > +#include <linux/of_graph.h>
> > > > > > > > +#include <linux/slab.h>
> > > > > > > > +#include <linux/string.h>
> > > > > > > > +#include <video/display_timing.h>
> > > > > > > > +#include <video/of_display_timing.h>
> > > > > > > > +#include <video/videomode.h>
> > > > > > > > +
> > > > > > > > +#define to_clcd(info)      container_of(info, struct clcd_fb, fb)
> > > > > > > > +
> > > > > > > > +/* This is limited to 16 characters when displayed by X startup */
> > > > > > > > +static const char *clcd_name = "CLCD FB";
> > > > > > > > +
> > > > > > > > +/*
> > > > > > > > + * Unfortunately, the enable/disable functions may be called either from
> > > > > > > > + * process or IRQ context, and we _need_ to delay.  This is _not_ good.
> > > > > > > > + */
> > > > > > > > +static inline void clcdfb_sleep(unsigned int ms)
> > > > > > > > +{
> > > > > > > > +   if (in_atomic()) {
> > > > > > > > +           mdelay(ms);
> > > > > > > > +   } else {
> > > > > > > > +           msleep(ms);
> > > > > > > > +   }
> > > > > > > > +}
> > > > > > > > +
> > > > > > > > +static inline void clcdfb_set_start(struct clcd_fb *fb)
> > > > > > > > +{
> > > > > > > > +   unsigned long ustart = fb->fb.fix.smem_start;
> > > > > > > > +   unsigned long lstart;
> > > > > > > > +
> > > > > > > > +   ustart += fb->fb.var.yoffset * fb->fb.fix.line_length;
> > > > > > > > +   lstart = ustart + fb->fb.var.yres * fb->fb.fix.line_length / 2;
> > > > > > > > +
> > > > > > > > +   writel(ustart, fb->regs + CLCD_UBAS);
> > > > > > > > +   writel(lstart, fb->regs + CLCD_LBAS);
> > > > > > > > +}
> > > > > > > > +
> > > > > > > > +static void clcdfb_disable(struct clcd_fb *fb)
> > > > > > > > +{
> > > > > > > > +   u32 val;
> > > > > > > > +
> > > > > > > > +   if (fb->board->disable)
> > > > > > > > +           fb->board->disable(fb);
> > > > > > > > +
> > > > > > > > +   if (fb->panel->backlight) {
> > > > > > > > +           fb->panel->backlight->props.power = FB_BLANK_POWERDOWN;
> > > > > > > > +           backlight_update_status(fb->panel->backlight);
> > > > > > > > +   }
> > > > > > > > +
> > > > > > > > +   val = readl(fb->regs + fb->off_cntl);
> > > > > > > > +   if (val & CNTL_LCDPWR) {
> > > > > > > > +           val &= ~CNTL_LCDPWR;
> > > > > > > > +           writel(val, fb->regs + fb->off_cntl);
> > > > > > > > +
> > > > > > > > +           clcdfb_sleep(20);
> > > > > > > > +   }
> > > > > > > > +   if (val & CNTL_LCDEN) {
> > > > > > > > +           val &= ~CNTL_LCDEN;
> > > > > > > > +           writel(val, fb->regs + fb->off_cntl);
> > > > > > > > +   }
> > > > > > > > +
> > > > > > > > +   /*
> > > > > > > > +    * Disable CLCD clock source.
> > > > > > > > +    */
> > > > > > > > +   if (fb->clk_enabled) {
> > > > > > > > +           fb->clk_enabled = false;
> > > > > > > > +           clk_disable(fb->clk);
> > > > > > > > +   }
> > > > > > > > +}
> > > > > > > > +
> > > > > > > > +static void clcdfb_enable(struct clcd_fb *fb, u32 cntl)
> > > > > > > > +{
> > > > > > > > +   /*
> > > > > > > > +    * Enable the CLCD clock source.
> > > > > > > > +    */
> > > > > > > > +   if (!fb->clk_enabled) {
> > > > > > > > +           fb->clk_enabled = true;
> > > > > > > > +           clk_enable(fb->clk);
> > > > > > > > +   }
> > > > > > > > +
> > > > > > > > +   /*
> > > > > > > > +    * Bring up by first enabling..
> > > > > > > > +    */
> > > > > > > > +   cntl |= CNTL_LCDEN;
> > > > > > > > +   writel(cntl, fb->regs + fb->off_cntl);
> > > > > > > > +
> > > > > > > > +   clcdfb_sleep(20);
> > > > > > > > +
> > > > > > > > +   /*
> > > > > > > > +    * and now apply power.
> > > > > > > > +    */
> > > > > > > > +   cntl |= CNTL_LCDPWR;
> > > > > > > > +   writel(cntl, fb->regs + fb->off_cntl);
> > > > > > > > +
> > > > > > > > +   /*
> > > > > > > > +    * Turn on backlight
> > > > > > > > +    */
> > > > > > > > +   if (fb->panel->backlight) {
> > > > > > > > +           fb->panel->backlight->props.power = FB_BLANK_UNBLANK;
> > > > > > > > +           backlight_update_status(fb->panel->backlight);
> > > > > > > > +   }
> > > > > > > > +
> > > > > > > > +   /*
> > > > > > > > +    * finally, enable the interface.
> > > > > > > > +    */
> > > > > > > > +   if (fb->board->enable)
> > > > > > > > +           fb->board->enable(fb);
> > > > > > > > +}
> > > > > > > > +
> > > > > > > > +static int
> > > > > > > > +clcdfb_set_bitfields(struct clcd_fb *fb, struct fb_var_screeninfo *var)
> > > > > > > > +{
> > > > > > > > +   u32 caps;
> > > > > > > > +   int ret = 0;
> > > > > > > > +
> > > > > > > > +   if (fb->panel->caps && fb->board->caps)
> > > > > > > > +           caps = fb->panel->caps & fb->board->caps;
> > > > > > > > +   else {
> > > > > > > > +           /* Old way of specifying what can be used */
> > > > > > > > +           caps = fb->panel->cntl & CNTL_BGR ?
> > > > > > > > +                   CLCD_CAP_BGR : CLCD_CAP_RGB;
> > > > > > > > +           /* But mask out 444 modes as they weren't supported */
> > > > > > > > +           caps &= ~CLCD_CAP_444;
> > > > > > > > +   }
> > > > > > > > +
> > > > > > > > +   /* Only TFT panels can do RGB888/BGR888 */
> > > > > > > > +   if (!(fb->panel->cntl & CNTL_LCDTFT))
> > > > > > > > +           caps &= ~CLCD_CAP_888;
> > > > > > > > +
> > > > > > > > +   memset(&var->transp, 0, sizeof(var->transp));
> > > > > > > > +
> > > > > > > > +   var->red.msb_right = 0;
> > > > > > > > +   var->green.msb_right = 0;
> > > > > > > > +   var->blue.msb_right = 0;
> > > > > > > > +
> > > > > > > > +   switch (var->bits_per_pixel) {
> > > > > > > > +   case 1:
> > > > > > > > +   case 2:
> > > > > > > > +   case 4:
> > > > > > > > +   case 8:
> > > > > > > > +           /* If we can't do 5551, reject */
> > > > > > > > +           caps &= CLCD_CAP_5551;
> > > > > > > > +           if (!caps) {
> > > > > > > > +                   ret = -EINVAL;
> > > > > > > > +                   break;
> > > > > > > > +           }
> > > > > > > > +
> > > > > > > > +           var->red.length         = var->bits_per_pixel;
> > > > > > > > +           var->red.offset         = 0;
> > > > > > > > +           var->green.length       = var->bits_per_pixel;
> > > > > > > > +           var->green.offset       = 0;
> > > > > > > > +           var->blue.length        = var->bits_per_pixel;
> > > > > > > > +           var->blue.offset        = 0;
> > > > > > > > +           break;
> > > > > > > > +
> > > > > > > > +   case 16:
> > > > > > > > +           /* If we can't do 444, 5551 or 565, reject */
> > > > > > > > +           if (!(caps & (CLCD_CAP_444 | CLCD_CAP_5551 | CLCD_CAP_565))) {
> > > > > > > > +                   ret = -EINVAL;
> > > > > > > > +                   break;
> > > > > > > > +           }
> > > > > > > > +
> > > > > > > > +           /*
> > > > > > > > +            * Green length can be 4, 5 or 6 depending whether
> > > > > > > > +            * we're operating in 444, 5551 or 565 mode.
> > > > > > > > +            */
> > > > > > > > +           if (var->green.length == 4 && caps & CLCD_CAP_444)
> > > > > > > > +                   caps &= CLCD_CAP_444;
> > > > > > > > +           if (var->green.length == 5 && caps & CLCD_CAP_5551)
> > > > > > > > +                   caps &= CLCD_CAP_5551;
> > > > > > > > +           else if (var->green.length == 6 && caps & CLCD_CAP_565)
> > > > > > > > +                   caps &= CLCD_CAP_565;
> > > > > > > > +           else {
> > > > > > > > +                   /*
> > > > > > > > +                    * PL110 officially only supports RGB555,
> > > > > > > > +                    * but may be wired up to allow RGB565.
> > > > > > > > +                    */
> > > > > > > > +                   if (caps & CLCD_CAP_565) {
> > > > > > > > +                           var->green.length = 6;
> > > > > > > > +                           caps &= CLCD_CAP_565;
> > > > > > > > +                   } else if (caps & CLCD_CAP_5551) {
> > > > > > > > +                           var->green.length = 5;
> > > > > > > > +                           caps &= CLCD_CAP_5551;
> > > > > > > > +                   } else {
> > > > > > > > +                           var->green.length = 4;
> > > > > > > > +                           caps &= CLCD_CAP_444;
> > > > > > > > +                   }
> > > > > > > > +           }
> > > > > > > > +
> > > > > > > > +           if (var->green.length >= 5) {
> > > > > > > > +                   var->red.length = 5;
> > > > > > > > +                   var->blue.length = 5;
> > > > > > > > +           } else {
> > > > > > > > +                   var->red.length = 4;
> > > > > > > > +                   var->blue.length = 4;
> > > > > > > > +           }
> > > > > > > > +           break;
> > > > > > > > +   case 32:
> > > > > > > > +           /* If we can't do 888, reject */
> > > > > > > > +           caps &= CLCD_CAP_888;
> > > > > > > > +           if (!caps) {
> > > > > > > > +                   ret = -EINVAL;
> > > > > > > > +                   break;
> > > > > > > > +           }
> > > > > > > > +
> > > > > > > > +           var->red.length = 8;
> > > > > > > > +           var->green.length = 8;
> > > > > > > > +           var->blue.length = 8;
> > > > > > > > +           break;
> > > > > > > > +   default:
> > > > > > > > +           ret = -EINVAL;
> > > > > > > > +           break;
> > > > > > > > +   }
> > > > > > > > +
> > > > > > > > +   /*
> > > > > > > > +    * >= 16bpp displays have separate colour component bitfields
> > > > > > > > +    * encoded in the pixel data.  Calculate their position from
> > > > > > > > +    * the bitfield length defined above.
> > > > > > > > +    */
> > > > > > > > +   if (ret == 0 && var->bits_per_pixel >= 16) {
> > > > > > > > +           bool bgr, rgb;
> > > > > > > > +
> > > > > > > > +           bgr = caps & CLCD_CAP_BGR && var->blue.offset == 0;
> > > > > > > > +           rgb = caps & CLCD_CAP_RGB && var->red.offset == 0;
> > > > > > > > +
> > > > > > > > +           if (!bgr && !rgb)
> > > > > > > > +                   /*
> > > > > > > > +                    * The requested format was not possible, try just
> > > > > > > > +                    * our capabilities.  One of BGR or RGB must be
> > > > > > > > +                    * supported.
> > > > > > > > +                    */
> > > > > > > > +                   bgr = caps & CLCD_CAP_BGR;
> > > > > > > > +
> > > > > > > > +           if (bgr) {
> > > > > > > > +                   var->blue.offset = 0;
> > > > > > > > +                   var->green.offset = var->blue.offset + var->blue.length;
> > > > > > > > +                   var->red.offset = var->green.offset + var->green.length;
> > > > > > > > +           } else {
> > > > > > > > +                   var->red.offset = 0;
> > > > > > > > +                   var->green.offset = var->red.offset + var->red.length;
> > > > > > > > +                   var->blue.offset = var->green.offset + var->green.length;
> > > > > > > > +           }
> > > > > > > > +   }
> > > > > > > > +
> > > > > > > > +   return ret;
> > > > > > > > +}
> > > > > > > > +
> > > > > > > > +static int clcdfb_check_var(struct fb_var_screeninfo *var, struct fb_info *info)
> > > > > > > > +{
> > > > > > > > +   struct clcd_fb *fb = to_clcd(info);
> > > > > > > > +   int ret = -EINVAL;
> > > > > > > > +
> > > > > > > > +   if (fb->board->check)
> > > > > > > > +           ret = fb->board->check(fb, var);
> > > > > > > > +
> > > > > > > > +   if (ret == 0 &&
> > > > > > > > +       var->xres_virtual * var->bits_per_pixel / 8 *
> > > > > > > > +       var->yres_virtual > fb->fb.fix.smem_len)
> > > > > > > > +           ret = -EINVAL;
> > > > > > > > +
> > > > > > > > +   if (ret == 0)
> > > > > > > > +           ret = clcdfb_set_bitfields(fb, var);
> > > > > > > > +
> > > > > > > > +   return ret;
> > > > > > > > +}
> > > > > > > > +
> > > > > > > > +static int clcdfb_set_par(struct fb_info *info)
> > > > > > > > +{
> > > > > > > > +   struct clcd_fb *fb = to_clcd(info);
> > > > > > > > +   struct clcd_regs regs;
> > > > > > > > +
> > > > > > > > +   fb->fb.fix.line_length = fb->fb.var.xres_virtual *
> > > > > > > > +                            fb->fb.var.bits_per_pixel / 8;
> > > > > > > > +
> > > > > > > > +   if (fb->fb.var.bits_per_pixel <= 8)
> > > > > > > > +           fb->fb.fix.visual = FB_VISUAL_PSEUDOCOLOR;
> > > > > > > > +   else
> > > > > > > > +           fb->fb.fix.visual = FB_VISUAL_TRUECOLOR;
> > > > > > > > +
> > > > > > > > +   fb->board->decode(fb, &regs);
> > > > > > > > +
> > > > > > > > +   clcdfb_disable(fb);
> > > > > > > > +
> > > > > > > > +   writel(regs.tim0, fb->regs + CLCD_TIM0);
> > > > > > > > +   writel(regs.tim1, fb->regs + CLCD_TIM1);
> > > > > > > > +   writel(regs.tim2, fb->regs + CLCD_TIM2);
> > > > > > > > +   writel(regs.tim3, fb->regs + CLCD_TIM3);
> > > > > > > > +
> > > > > > > > +   clcdfb_set_start(fb);
> > > > > > > > +
> > > > > > > > +   clk_set_rate(fb->clk, (1000000000 / regs.pixclock) * 1000);
> > > > > > > > +
> > > > > > > > +   fb->clcd_cntl = regs.cntl;
> > > > > > > > +
> > > > > > > > +   clcdfb_enable(fb, regs.cntl);
> > > > > > > > +
> > > > > > > > +#ifdef DEBUG
> > > > > > > > +   printk(KERN_INFO
> > > > > > > > +          "CLCD: Registers set to\n"
> > > > > > > > +          "  %08x %08x %08x %08x\n"
> > > > > > > > +          "  %08x %08x %08x %08x\n",
> > > > > > > > +           readl(fb->regs + CLCD_TIM0), readl(fb->regs + CLCD_TIM1),
> > > > > > > > +           readl(fb->regs + CLCD_TIM2), readl(fb->regs + CLCD_TIM3),
> > > > > > > > +           readl(fb->regs + CLCD_UBAS), readl(fb->regs + CLCD_LBAS),
> > > > > > > > +           readl(fb->regs + fb->off_ienb), readl(fb->regs + fb->off_cntl));
> > > > > > > > +#endif
> > > > > > > > +
> > > > > > > > +   return 0;
> > > > > > > > +}
> > > > > > > > +
> > > > > > > > +static inline u32 convert_bitfield(int val, struct fb_bitfield *bf)
> > > > > > > > +{
> > > > > > > > +   unsigned int mask = (1 << bf->length) - 1;
> > > > > > > > +
> > > > > > > > +   return (val >> (16 - bf->length) & mask) << bf->offset;
> > > > > > > > +}
> > > > > > > > +
> > > > > > > > +/*
> > > > > > > > + *  Set a single color register. The values supplied have a 16 bit
> > > > > > > > + *  magnitude.  Return != 0 for invalid regno.
> > > > > > > > + */
> > > > > > > > +static int
> > > > > > > > +clcdfb_setcolreg(unsigned int regno, unsigned int red, unsigned int green,
> > > > > > > > +            unsigned int blue, unsigned int transp, struct fb_info *info)
> > > > > > > > +{
> > > > > > > > +   struct clcd_fb *fb = to_clcd(info);
> > > > > > > > +
> > > > > > > > +   if (regno < 16)
> > > > > > > > +           fb->cmap[regno] = convert_bitfield(transp, &fb->fb.var.transp) |
> > > > > > > > +                             convert_bitfield(blue, &fb->fb.var.blue) |
> > > > > > > > +                             convert_bitfield(green, &fb->fb.var.green) |
> > > > > > > > +                             convert_bitfield(red, &fb->fb.var.red);
> > > > > > > > +
> > > > > > > > +   if (fb->fb.fix.visual == FB_VISUAL_PSEUDOCOLOR && regno < 256) {
> > > > > > > > +           int hw_reg = CLCD_PALETTE + ((regno * 2) & ~3);
> > > > > > > > +           u32 val, mask, newval;
> > > > > > > > +
> > > > > > > > +           newval  = (red >> 11)  & 0x001f;
> > > > > > > > +           newval |= (green >> 6) & 0x03e0;
> > > > > > > > +           newval |= (blue >> 1)  & 0x7c00;
> > > > > > > > +
> > > > > > > > +           /*
> > > > > > > > +            * 3.2.11: if we're configured for big endian
> > > > > > > > +            * byte order, the palette entries are swapped.
> > > > > > > > +            */
> > > > > > > > +           if (fb->clcd_cntl & CNTL_BEBO)
> > > > > > > > +                   regno ^= 1;
> > > > > > > > +
> > > > > > > > +           if (regno & 1) {
> > > > > > > > +                   newval <<= 16;
> > > > > > > > +                   mask = 0x0000ffff;
> > > > > > > > +           } else {
> > > > > > > > +                   mask = 0xffff0000;
> > > > > > > > +           }
> > > > > > > > +
> > > > > > > > +           val = readl(fb->regs + hw_reg) & mask;
> > > > > > > > +           writel(val | newval, fb->regs + hw_reg);
> > > > > > > > +   }
> > > > > > > > +
> > > > > > > > +   return regno > 255;
> > > > > > > > +}
> > > > > > > > +
> > > > > > > > +/*
> > > > > > > > + *  Blank the screen if blank_mode != 0, else unblank. If blank == NULL
> > > > > > > > + *  then the caller blanks by setting the CLUT (Color Look Up Table) to all
> > > > > > > > + *  black. Return 0 if blanking succeeded, != 0 if un-/blanking failed due
> > > > > > > > + *  to e.g. a video mode which doesn't support it. Implements VESA suspend
> > > > > > > > + *  and powerdown modes on hardware that supports disabling hsync/vsync:
> > > > > > > > + *    blank_mode == 2: suspend vsync
> > > > > > > > + *    blank_mode == 3: suspend hsync
> > > > > > > > + *    blank_mode == 4: powerdown
> > > > > > > > + */
> > > > > > > > +static int clcdfb_blank(int blank_mode, struct fb_info *info)
> > > > > > > > +{
> > > > > > > > +   struct clcd_fb *fb = to_clcd(info);
> > > > > > > > +
> > > > > > > > +   if (blank_mode != 0) {
> > > > > > > > +           clcdfb_disable(fb);
> > > > > > > > +   } else {
> > > > > > > > +           clcdfb_enable(fb, fb->clcd_cntl);
> > > > > > > > +   }
> > > > > > > > +   return 0;
> > > > > > > > +}
> > > > > > > > +
> > > > > > > > +static int clcdfb_mmap(struct fb_info *info,
> > > > > > > > +                  struct vm_area_struct *vma)
> > > > > > > > +{
> > > > > > > > +   struct clcd_fb *fb = to_clcd(info);
> > > > > > > > +   unsigned long len, off = vma->vm_pgoff << PAGE_SHIFT;
> > > > > > > > +   int ret = -EINVAL;
> > > > > > > > +
> > > > > > > > +   len = info->fix.smem_len;
> > > > > > > > +
> > > > > > > > +   if (off <= len && vma->vm_end - vma->vm_start <= len - off &&
> > > > > > > > +       fb->board->mmap)
> > > > > > > > +           ret = fb->board->mmap(fb, vma);
> > > > > > > > +
> > > > > > > > +   return ret;
> > > > > > > > +}
> > > > > > > > +
> > > > > > > > +static const struct fb_ops clcdfb_ops = {
> > > > > > > > +   .owner          = THIS_MODULE,
> > > > > > > > +   .fb_check_var   = clcdfb_check_var,
> > > > > > > > +   .fb_set_par     = clcdfb_set_par,
> > > > > > > > +   .fb_setcolreg   = clcdfb_setcolreg,
> > > > > > > > +   .fb_blank       = clcdfb_blank,
> > > > > > > > +   .fb_fillrect    = cfb_fillrect,
> > > > > > > > +   .fb_copyarea    = cfb_copyarea,
> > > > > > > > +   .fb_imageblit   = cfb_imageblit,
> > > > > > > > +   .fb_mmap        = clcdfb_mmap,
> > > > > > > > +};
> > > > > > > > +
> > > > > > > > +static int clcdfb_register(struct clcd_fb *fb)
> > > > > > > > +{
> > > > > > > > +   int ret;
> > > > > > > > +
> > > > > > > > +   /*
> > > > > > > > +    * ARM PL111 always has IENB at 0x1c; it's only PL110
> > > > > > > > +    * which is reversed on some platforms.
> > > > > > > > +    */
> > > > > > > > +   if (amba_manf(fb->dev) == 0x41 && amba_part(fb->dev) == 0x111) {
> > > > > > > > +           fb->off_ienb = CLCD_PL111_IENB;
> > > > > > > > +           fb->off_cntl = CLCD_PL111_CNTL;
> > > > > > > > +   } else {
> > > > > > > > +           fb->off_ienb = CLCD_PL110_IENB;
> > > > > > > > +           fb->off_cntl = CLCD_PL110_CNTL;
> > > > > > > > +   }
> > > > > > > > +
> > > > > > > > +   fb->clk = clk_get(&fb->dev->dev, NULL);
> > > > > > > > +   if (IS_ERR(fb->clk)) {
> > > > > > > > +           ret = PTR_ERR(fb->clk);
> > > > > > > > +           goto out;
> > > > > > > > +   }
> > > > > > > > +
> > > > > > > > +   ret = clk_prepare(fb->clk);
> > > > > > > > +   if (ret)
> > > > > > > > +           goto free_clk;
> > > > > > > > +
> > > > > > > > +   fb->fb.device           = &fb->dev->dev;
> > > > > > > > +
> > > > > > > > +   fb->fb.fix.mmio_start   = fb->dev->res.start;
> > > > > > > > +   fb->fb.fix.mmio_len     = resource_size(&fb->dev->res);
> > > > > > > > +
> > > > > > > > +   fb->regs = ioremap(fb->fb.fix.mmio_start, fb->fb.fix.mmio_len);
> > > > > > > > +   if (!fb->regs) {
> > > > > > > > +           printk(KERN_ERR "CLCD: unable to remap registers\n");
> > > > > > > > +           ret = -ENOMEM;
> > > > > > > > +           goto clk_unprep;
> > > > > > > > +   }
> > > > > > > > +
> > > > > > > > +   fb->fb.fbops            = &clcdfb_ops;
> > > > > > > > +   fb->fb.flags            = FBINFO_FLAG_DEFAULT;
> > > > > > > > +   fb->fb.pseudo_palette   = fb->cmap;
> > > > > > > > +
> > > > > > > > +   strncpy(fb->fb.fix.id, clcd_name, sizeof(fb->fb.fix.id));
> > > > > > > > +   fb->fb.fix.type         = FB_TYPE_PACKED_PIXELS;
> > > > > > > > +   fb->fb.fix.type_aux     = 0;
> > > > > > > > +   fb->fb.fix.xpanstep     = 0;
> > > > > > > > +   fb->fb.fix.ypanstep     = 0;
> > > > > > > > +   fb->fb.fix.ywrapstep    = 0;
> > > > > > > > +   fb->fb.fix.accel        = FB_ACCEL_NONE;
> > > > > > > > +
> > > > > > > > +   fb->fb.var.xres         = fb->panel->mode.xres;
> > > > > > > > +   fb->fb.var.yres         = fb->panel->mode.yres;
> > > > > > > > +   fb->fb.var.xres_virtual = fb->panel->mode.xres;
> > > > > > > > +   fb->fb.var.yres_virtual = fb->panel->mode.yres;
> > > > > > > > +   fb->fb.var.bits_per_pixel = fb->panel->bpp;
> > > > > > > > +   fb->fb.var.grayscale    = fb->panel->grayscale;
> > > > > > > > +   fb->fb.var.pixclock     = fb->panel->mode.pixclock;
> > > > > > > > +   fb->fb.var.left_margin  = fb->panel->mode.left_margin;
> > > > > > > > +   fb->fb.var.right_margin = fb->panel->mode.right_margin;
> > > > > > > > +   fb->fb.var.upper_margin = fb->panel->mode.upper_margin;
> > > > > > > > +   fb->fb.var.lower_margin = fb->panel->mode.lower_margin;
> > > > > > > > +   fb->fb.var.hsync_len    = fb->panel->mode.hsync_len;
> > > > > > > > +   fb->fb.var.vsync_len    = fb->panel->mode.vsync_len;
> > > > > > > > +   fb->fb.var.sync         = fb->panel->mode.sync;
> > > > > > > > +   fb->fb.var.vmode        = fb->panel->mode.vmode;
> > > > > > > > +   fb->fb.var.activate     = FB_ACTIVATE_NOW;
> > > > > > > > +   fb->fb.var.nonstd       = 0;
> > > > > > > > +   fb->fb.var.height       = fb->panel->height;
> > > > > > > > +   fb->fb.var.width        = fb->panel->width;
> > > > > > > > +   fb->fb.var.accel_flags  = 0;
> > > > > > > > +
> > > > > > > > +   fb->fb.monspecs.hfmin   = 0;
> > > > > > > > +   fb->fb.monspecs.hfmax   = 100000;
> > > > > > > > +   fb->fb.monspecs.vfmin   = 0;
> > > > > > > > +   fb->fb.monspecs.vfmax   = 400;
> > > > > > > > +   fb->fb.monspecs.dclkmin = 1000000;
> > > > > > > > +   fb->fb.monspecs.dclkmax = 100000000;
> > > > > > > > +
> > > > > > > > +   /*
> > > > > > > > +    * Make sure that the bitfields are set appropriately.
> > > > > > > > +    */
> > > > > > > > +   clcdfb_set_bitfields(fb, &fb->fb.var);
> > > > > > > > +
> > > > > > > > +   /*
> > > > > > > > +    * Allocate colourmap.
> > > > > > > > +    */
> > > > > > > > +   ret = fb_alloc_cmap(&fb->fb.cmap, 256, 0);
> > > > > > > > +   if (ret)
> > > > > > > > +           goto unmap;
> > > > > > > > +
> > > > > > > > +   /*
> > > > > > > > +    * Ensure interrupts are disabled.
> > > > > > > > +    */
> > > > > > > > +   writel(0, fb->regs + fb->off_ienb);
> > > > > > > > +
> > > > > > > > +   fb_set_var(&fb->fb, &fb->fb.var);
> > > > > > > > +
> > > > > > > > +   dev_info(&fb->dev->dev, "%s hardware, %s display\n",
> > > > > > > > +            fb->board->name, fb->panel->mode.name);
> > > > > > > > +
> > > > > > > > +   ret = register_framebuffer(&fb->fb);
> > > > > > > > +   if (ret == 0)
> > > > > > > > +           goto out;
> > > > > > > > +
> > > > > > > > +   printk(KERN_ERR "CLCD: cannot register framebuffer (%d)\n", ret);
> > > > > > > > +
> > > > > > > > +   fb_dealloc_cmap(&fb->fb.cmap);
> > > > > > > > + unmap:
> > > > > > > > +   iounmap(fb->regs);
> > > > > > > > + clk_unprep:
> > > > > > > > +   clk_unprepare(fb->clk);
> > > > > > > > + free_clk:
> > > > > > > > +   clk_put(fb->clk);
> > > > > > > > + out:
> > > > > > > > +   return ret;
> > > > > > > > +}
> > > > > > > > +
> > > > > > > > +#ifdef CONFIG_OF
> > > > > > > > +static int clcdfb_of_get_dpi_panel_mode(struct device_node *node,
> > > > > > > > +           struct clcd_panel *clcd_panel)
> > > > > > > > +{
> > > > > > > > +   int err;
> > > > > > > > +   struct display_timing timing;
> > > > > > > > +   struct videomode video;
> > > > > > > > +
> > > > > > > > +   err = of_get_display_timing(node, "panel-timing", &timing);
> > > > > > > > +   if (err) {
> > > > > > > > +           pr_err("%pOF: problems parsing panel-timing (%d)\n", node, err);
> > > > > > > > +           return err;
> > > > > > > > +   }
> > > > > > > > +
> > > > > > > > +   videomode_from_timing(&timing, &video);
> > > > > > > > +
> > > > > > > > +   err = fb_videomode_from_videomode(&video, &clcd_panel->mode);
> > > > > > > > +   if (err)
> > > > > > > > +           return err;
> > > > > > > > +
> > > > > > > > +   /* Set up some inversion flags */
> > > > > > > > +   if (timing.flags & DISPLAY_FLAGS_PIXDATA_NEGEDGE)
> > > > > > > > +           clcd_panel->tim2 |= TIM2_IPC;
> > > > > > > > +   else if (!(timing.flags & DISPLAY_FLAGS_PIXDATA_POSEDGE))
> > > > > > > > +           /*
> > > > > > > > +            * To preserve backwards compatibility, the IPC (inverted
> > > > > > > > +            * pixel clock) flag needs to be set on any display that
> > > > > > > > +            * doesn't explicitly specify that the pixel clock is
> > > > > > > > +            * active on the negative or positive edge.
> > > > > > > > +            */
> > > > > > > > +           clcd_panel->tim2 |= TIM2_IPC;
> > > > > > > > +
> > > > > > > > +   if (timing.flags & DISPLAY_FLAGS_HSYNC_LOW)
> > > > > > > > +           clcd_panel->tim2 |= TIM2_IHS;
> > > > > > > > +
> > > > > > > > +   if (timing.flags & DISPLAY_FLAGS_VSYNC_LOW)
> > > > > > > > +           clcd_panel->tim2 |= TIM2_IVS;
> > > > > > > > +
> > > > > > > > +   if (timing.flags & DISPLAY_FLAGS_DE_LOW)
> > > > > > > > +           clcd_panel->tim2 |= TIM2_IOE;
> > > > > > > > +
> > > > > > > > +   return 0;
> > > > > > > > +}
> > > > > > > > +
> > > > > > > > +static int clcdfb_snprintf_mode(char *buf, int size, struct fb_videomode *mode)
> > > > > > > > +{
> > > > > > > > +   return snprintf(buf, size, "%ux%u@%u", mode->xres, mode->yres,
> > > > > > > > +                   mode->refresh);
> > > > > > > > +}
> > > > > > > > +
> > > > > > > > +static int clcdfb_of_get_backlight(struct device *dev,
> > > > > > > > +                              struct clcd_panel *clcd_panel)
> > > > > > > > +{
> > > > > > > > +   struct backlight_device *backlight;
> > > > > > > > +
> > > > > > > > +   /* Look up the optional backlight device */
> > > > > > > > +   backlight = devm_of_find_backlight(dev);
> > > > > > > > +   if (IS_ERR(backlight))
> > > > > > > > +           return PTR_ERR(backlight);
> > > > > > > > +
> > > > > > > > +   clcd_panel->backlight = backlight;
> > > > > > > > +   return 0;
> > > > > > > > +}
> > > > > > > > +
> > > > > > > > +static int clcdfb_of_get_mode(struct device *dev, struct device_node *panel,
> > > > > > > > +                         struct clcd_panel *clcd_panel)
> > > > > > > > +{
> > > > > > > > +   int err;
> > > > > > > > +   struct fb_videomode *mode;
> > > > > > > > +   char *name;
> > > > > > > > +   int len;
> > > > > > > > +
> > > > > > > > +   /* Only directly connected DPI panels supported for now */
> > > > > > > > +   if (of_device_is_compatible(panel, "panel-dpi"))
> > > > > > > > +           err = clcdfb_of_get_dpi_panel_mode(panel, clcd_panel);
> > > > > > > > +   else
> > > > > > > > +           err = -ENOENT;
> > > > > > > > +   if (err)
> > > > > > > > +           return err;
> > > > > > > > +   mode = &clcd_panel->mode;
> > > > > > > > +
> > > > > > > > +   len = clcdfb_snprintf_mode(NULL, 0, mode);
> > > > > > > > +   name = devm_kzalloc(dev, len + 1, GFP_KERNEL);
> > > > > > > > +   if (!name)
> > > > > > > > +           return -ENOMEM;
> > > > > > > > +
> > > > > > > > +   clcdfb_snprintf_mode(name, len + 1, mode);
> > > > > > > > +   mode->name = name;
> > > > > > > > +
> > > > > > > > +   return 0;
> > > > > > > > +}
> > > > > > > > +
> > > > > > > > +static int clcdfb_of_init_tft_panel(struct clcd_fb *fb, u32 r0, u32 g0, u32 b0)
> > > > > > > > +{
> > > > > > > > +   static struct {
> > > > > > > > +           unsigned int part;
> > > > > > > > +           u32 r0, g0, b0;
> > > > > > > > +           u32 caps;
> > > > > > > > +   } panels[] = {
> > > > > > > > +           { 0x110, 1,  7, 13, CLCD_CAP_5551 },
> > > > > > > > +           { 0x110, 0,  8, 16, CLCD_CAP_888 },
> > > > > > > > +           { 0x110, 16, 8, 0,  CLCD_CAP_888 },
> > > > > > > > +           { 0x111, 4, 14, 20, CLCD_CAP_444 },
> > > > > > > > +           { 0x111, 3, 11, 19, CLCD_CAP_444 | CLCD_CAP_5551 },
> > > > > > > > +           { 0x111, 3, 10, 19, CLCD_CAP_444 | CLCD_CAP_5551 |
> > > > > > > > +                               CLCD_CAP_565 },
> > > > > > > > +           { 0x111, 0,  8, 16, CLCD_CAP_444 | CLCD_CAP_5551 |
> > > > > > > > +                               CLCD_CAP_565 | CLCD_CAP_888 },
> > > > > > > > +   };
> > > > > > > > +   int i;
> > > > > > > > +
> > > > > > > > +   /* Bypass pixel clock divider */
> > > > > > > > +   fb->panel->tim2 |= TIM2_BCD;
> > > > > > > > +
> > > > > > > > +   /* TFT display, vert. comp. interrupt at the start of the back porch */
> > > > > > > > +   fb->panel->cntl |= CNTL_LCDTFT | CNTL_LCDVCOMP(1);
> > > > > > > > +
> > > > > > > > +   fb->panel->caps = 0;
> > > > > > > > +
> > > > > > > > +   /* Match the setup with known variants */
> > > > > > > > +   for (i = 0; i < ARRAY_SIZE(panels) && !fb->panel->caps; i++) {
> > > > > > > > +           if (amba_part(fb->dev) != panels[i].part)
> > > > > > > > +                   continue;
> > > > > > > > +           if (g0 != panels[i].g0)
> > > > > > > > +                   continue;
> > > > > > > > +           if (r0 == panels[i].r0 && b0 == panels[i].b0)
> > > > > > > > +                   fb->panel->caps = panels[i].caps;
> > > > > > > > +   }
> > > > > > > > +
> > > > > > > > +   /*
> > > > > > > > +    * If we actually physically connected the R lines to B and
> > > > > > > > +    * vice versa
> > > > > > > > +    */
> > > > > > > > +   if (r0 != 0 && b0 == 0)
> > > > > > > > +           fb->panel->bgr_connection = true;
> > > > > > > > +
> > > > > > > > +   return fb->panel->caps ? 0 : -EINVAL;
> > > > > > > > +}
> > > > > > > > +
> > > > > > > > +static int clcdfb_of_init_display(struct clcd_fb *fb)
> > > > > > > > +{
> > > > > > > > +   struct device_node *endpoint, *panel;
> > > > > > > > +   int err;
> > > > > > > > +   unsigned int bpp;
> > > > > > > > +   u32 max_bandwidth;
> > > > > > > > +   u32 tft_r0b0g0[3];
> > > > > > > > +
> > > > > > > > +   fb->panel = devm_kzalloc(&fb->dev->dev, sizeof(*fb->panel), GFP_KERNEL);
> > > > > > > > +   if (!fb->panel)
> > > > > > > > +           return -ENOMEM;
> > > > > > > > +
> > > > > > > > +   /*
> > > > > > > > +    * Fetch the panel endpoint.
> > > > > > > > +    */
> > > > > > > > +   endpoint = of_graph_get_next_endpoint(fb->dev->dev.of_node, NULL);
> > > > > > > > +   if (!endpoint)
> > > > > > > > +           return -ENODEV;
> > > > > > > > +
> > > > > > > > +   panel = of_graph_get_remote_port_parent(endpoint);
> > > > > > > > +   if (!panel)
> > > > > > > > +           return -ENODEV;
> > > > > > > > +
> > > > > > > > +   err = clcdfb_of_get_backlight(&fb->dev->dev, fb->panel);
> > > > > > > > +   if (err)
> > > > > > > > +           return err;
> > > > > > > > +
> > > > > > > > +   err = clcdfb_of_get_mode(&fb->dev->dev, panel, fb->panel);
> > > > > > > > +   if (err)
> > > > > > > > +           return err;
> > > > > > > > +
> > > > > > > > +   err = of_property_read_u32(fb->dev->dev.of_node, "max-memory-bandwidth",
> > > > > > > > +                   &max_bandwidth);
> > > > > > > > +   if (!err) {
> > > > > > > > +           /*
> > > > > > > > +            * max_bandwidth is in bytes per second and pixclock in
> > > > > > > > +            * pico-seconds, so the maximum allowed bits per pixel is
> > > > > > > > +            *   8 * max_bandwidth / (PICOS2KHZ(pixclock) * 1000)
> > > > > > > > +            * Rearrange this calculation to avoid overflow and then ensure
> > > > > > > > +            * result is a valid format.
> > > > > > > > +            */
> > > > > > > > +           bpp = max_bandwidth / (1000 / 8)
> > > > > > > > +                   / PICOS2KHZ(fb->panel->mode.pixclock);
> > > > > > > > +           bpp = rounddown_pow_of_two(bpp);
> > > > > > > > +           if (bpp > 32)
> > > > > > > > +                   bpp = 32;
> > > > > > > > +   } else
> > > > > > > > +           bpp = 32;
> > > > > > > > +   fb->panel->bpp = bpp;
> > > > > > > > +
> > > > > > > > +#ifdef CONFIG_CPU_BIG_ENDIAN
> > > > > > > > +   fb->panel->cntl |= CNTL_BEBO;
> > > > > > > > +#endif
> > > > > > > > +   fb->panel->width = -1;
> > > > > > > > +   fb->panel->height = -1;
> > > > > > > > +
> > > > > > > > +   if (of_property_read_u32_array(endpoint,
> > > > > > > > +                   "arm,pl11x,tft-r0g0b0-pads",
> > > > > > > > +                   tft_r0b0g0, ARRAY_SIZE(tft_r0b0g0)) != 0)
> > > > > > > > +           return -ENOENT;
> > > > > > > > +
> > > > > > > > +   return clcdfb_of_init_tft_panel(fb, tft_r0b0g0[0],
> > > > > > > > +                                   tft_r0b0g0[1],  tft_r0b0g0[2]);
> > > > > > > > +}
> > > > > > > > +
> > > > > > > > +static int clcdfb_of_vram_setup(struct clcd_fb *fb)
> > > > > > > > +{
> > > > > > > > +   int err;
> > > > > > > > +   struct device_node *memory;
> > > > > > > > +   u64 size;
> > > > > > > > +
> > > > > > > > +   err = clcdfb_of_init_display(fb);
> > > > > > > > +   if (err)
> > > > > > > > +           return err;
> > > > > > > > +
> > > > > > > > +   memory = of_parse_phandle(fb->dev->dev.of_node, "memory-region", 0);
> > > > > > > > +   if (!memory)
> > > > > > > > +           return -ENODEV;
> > > > > > > > +
> > > > > > > > +   fb->fb.screen_base = of_iomap(memory, 0);
> > > > > > > > +   if (!fb->fb.screen_base)
> > > > > > > > +           return -ENOMEM;
> > > > > > > > +
> > > > > > > > +   fb->fb.fix.smem_start = of_translate_address(memory,
> > > > > > > > +                   of_get_address(memory, 0, &size, NULL));
> > > > > > > > +   fb->fb.fix.smem_len = size;
> > > > > > > > +
> > > > > > > > +   return 0;
> > > > > > > > +}
> > > > > > > > +
> > > > > > > > +static int clcdfb_of_vram_mmap(struct clcd_fb *fb, struct vm_area_struct *vma)
> > > > > > > > +{
> > > > > > > > +   unsigned long off, user_size, kernel_size;
> > > > > > > > +
> > > > > > > > +
> > > > > > > > +   off = vma->vm_pgoff << PAGE_SHIFT;
> > > > > > > > +   user_size = vma->vm_end - vma->vm_start;
> > > > > > > > +   kernel_size = fb->fb.fix.smem_len;
> > > > > > > > +
> > > > > > > > +   if (off >= kernel_size || user_size > (kernel_size - off))
> > > > > > > > +           return -ENXIO;
> > > > > > > > +
> > > > > > > > +   return remap_pfn_range(vma, vma->vm_start,
> > > > > > > > +                   __phys_to_pfn(fb->fb.fix.smem_start) + vma->vm_pgoff,
> > > > > > > > +                   user_size,
> > > > > > > > +                   pgprot_writecombine(vma->vm_page_prot));
> > > > > > > > +}
> > > > > > > > +
> > > > > > > > +static void clcdfb_of_vram_remove(struct clcd_fb *fb)
> > > > > > > > +{
> > > > > > > > +   iounmap(fb->fb.screen_base);
> > > > > > > > +}
> > > > > > > > +
> > > > > > > > +static int clcdfb_of_dma_setup(struct clcd_fb *fb)
> > > > > > > > +{
> > > > > > > > +   unsigned long framesize;
> > > > > > > > +   dma_addr_t dma;
> > > > > > > > +   int err;
> > > > > > > > +
> > > > > > > > +   err = clcdfb_of_init_display(fb);
> > > > > > > > +   if (err)
> > > > > > > > +           return err;
> > > > > > > > +
> > > > > > > > +   framesize = PAGE_ALIGN(fb->panel->mode.xres * fb->panel->mode.yres *
> > > > > > > > +                   fb->panel->bpp / 8);
> > > > > > > > +   fb->fb.screen_base = dma_alloc_coherent(&fb->dev->dev, framesize,
> > > > > > > > +                   &dma, GFP_KERNEL);
> > > > > > > > +   if (!fb->fb.screen_base)
> > > > > > > > +           return -ENOMEM;
> > > > > > > > +
> > > > > > > > +   fb->fb.fix.smem_start = dma;
> > > > > > > > +   fb->fb.fix.smem_len = framesize;
> > > > > > > > +
> > > > > > > > +   return 0;
> > > > > > > > +}
> > > > > > > > +
> > > > > > > > +static int clcdfb_of_dma_mmap(struct clcd_fb *fb, struct vm_area_struct *vma)
> > > > > > > > +{
> > > > > > > > +   return dma_mmap_wc(&fb->dev->dev, vma, fb->fb.screen_base,
> > > > > > > > +                      fb->fb.fix.smem_start, fb->fb.fix.smem_len);
> > > > > > > > +}
> > > > > > > > +
> > > > > > > > +static void clcdfb_of_dma_remove(struct clcd_fb *fb)
> > > > > > > > +{
> > > > > > > > +   dma_free_coherent(&fb->dev->dev, fb->fb.fix.smem_len,
> > > > > > > > +                   fb->fb.screen_base, fb->fb.fix.smem_start);
> > > > > > > > +}
> > > > > > > > +
> > > > > > > > +static struct clcd_board *clcdfb_of_get_board(struct amba_device *dev)
> > > > > > > > +{
> > > > > > > > +   struct clcd_board *board = devm_kzalloc(&dev->dev, sizeof(*board),
> > > > > > > > +                   GFP_KERNEL);
> > > > > > > > +   struct device_node *node = dev->dev.of_node;
> > > > > > > > +
> > > > > > > > +   if (!board)
> > > > > > > > +           return NULL;
> > > > > > > > +
> > > > > > > > +   board->name = of_node_full_name(node);
> > > > > > > > +   board->caps = CLCD_CAP_ALL;
> > > > > > > > +   board->check = clcdfb_check;
> > > > > > > > +   board->decode = clcdfb_decode;
> > > > > > > > +   if (of_find_property(node, "memory-region", NULL)) {
> > > > > > > > +           board->setup = clcdfb_of_vram_setup;
> > > > > > > > +           board->mmap = clcdfb_of_vram_mmap;
> > > > > > > > +           board->remove = clcdfb_of_vram_remove;
> > > > > > > > +   } else {
> > > > > > > > +           board->setup = clcdfb_of_dma_setup;
> > > > > > > > +           board->mmap = clcdfb_of_dma_mmap;
> > > > > > > > +           board->remove = clcdfb_of_dma_remove;
> > > > > > > > +   }
> > > > > > > > +
> > > > > > > > +   return board;
> > > > > > > > +}
> > > > > > > > +#else
> > > > > > > > +static struct clcd_board *clcdfb_of_get_board(struct amba_device *dev)
> > > > > > > > +{
> > > > > > > > +   return NULL;
> > > > > > > > +}
> > > > > > > > +#endif
> > > > > > > > +
> > > > > > > > +static int clcdfb_probe(struct amba_device *dev, const struct amba_id *id)
> > > > > > > > +{
> > > > > > > > +   struct clcd_board *board = dev_get_platdata(&dev->dev);
> > > > > > > > +   struct clcd_fb *fb;
> > > > > > > > +   int ret;
> > > > > > > > +
> > > > > > > > +   if (!board)
> > > > > > > > +           board = clcdfb_of_get_board(dev);
> > > > > > > > +
> > > > > > > > +   if (!board)
> > > > > > > > +           return -EINVAL;
> > > > > > > > +
> > > > > > > > +   ret = dma_set_mask_and_coherent(&dev->dev, DMA_BIT_MASK(32));
> > > > > > > > +   if (ret)
> > > > > > > > +           goto out;
> > > > > > > > +
> > > > > > > > +   ret = amba_request_regions(dev, NULL);
> > > > > > > > +   if (ret) {
> > > > > > > > +           printk(KERN_ERR "CLCD: unable to reserve regs region\n");
> > > > > > > > +           goto out;
> > > > > > > > +   }
> > > > > > > > +
> > > > > > > > +   fb = kzalloc(sizeof(*fb), GFP_KERNEL);
> > > > > > > > +   if (!fb) {
> > > > > > > > +           ret = -ENOMEM;
> > > > > > > > +           goto free_region;
> > > > > > > > +   }
> > > > > > > > +
> > > > > > > > +   fb->dev = dev;
> > > > > > > > +   fb->board = board;
> > > > > > > > +
> > > > > > > > +   dev_info(&fb->dev->dev, "PL%03x designer %02x rev%u at 0x%08llx\n",
> > > > > > > > +           amba_part(dev), amba_manf(dev), amba_rev(dev),
> > > > > > > > +           (unsigned long long)dev->res.start);
> > > > > > > > +
> > > > > > > > +   ret = fb->board->setup(fb);
> > > > > > > > +   if (ret)
> > > > > > > > +           goto free_fb;
> > > > > > > > +
> > > > > > > > +   ret = clcdfb_register(fb);
> > > > > > > > +   if (ret == 0) {
> > > > > > > > +           amba_set_drvdata(dev, fb);
> > > > > > > > +           goto out;
> > > > > > > > +   }
> > > > > > > > +
> > > > > > > > +   fb->board->remove(fb);
> > > > > > > > + free_fb:
> > > > > > > > +   kfree(fb);
> > > > > > > > + free_region:
> > > > > > > > +   amba_release_regions(dev);
> > > > > > > > + out:
> > > > > > > > +   return ret;
> > > > > > > > +}
> > > > > > > > +
> > > > > > > > +static int clcdfb_remove(struct amba_device *dev)
> > > > > > > > +{
> > > > > > > > +   struct clcd_fb *fb = amba_get_drvdata(dev);
> > > > > > > > +
> > > > > > > > +   clcdfb_disable(fb);
> > > > > > > > +   unregister_framebuffer(&fb->fb);
> > > > > > > > +   if (fb->fb.cmap.len)
> > > > > > > > +           fb_dealloc_cmap(&fb->fb.cmap);
> > > > > > > > +   iounmap(fb->regs);
> > > > > > > > +   clk_unprepare(fb->clk);
> > > > > > > > +   clk_put(fb->clk);
> > > > > > > > +
> > > > > > > > +   fb->board->remove(fb);
> > > > > > > > +
> > > > > > > > +   kfree(fb);
> > > > > > > > +
> > > > > > > > +   amba_release_regions(dev);
> > > > > > > > +
> > > > > > > > +   return 0;
> > > > > > > > +}
> > > > > > > > +
> > > > > > > > +static const struct amba_id clcdfb_id_table[] = {
> > > > > > > > +   {
> > > > > > > > +           .id     = 0x00041110,
> > > > > > > > +           .mask   = 0x000ffffe,
> > > > > > > > +   },
> > > > > > > > +   { 0, 0 },
> > > > > > > > +};
> > > > > > > > +
> > > > > > > > +MODULE_DEVICE_TABLE(amba, clcdfb_id_table);
> > > > > > > > +
> > > > > > > > +static struct amba_driver clcd_driver = {
> > > > > > > > +   .drv            = {
> > > > > > > > +           .name   = "clcd-pl11x",
> > > > > > > > +   },
> > > > > > > > +   .probe          = clcdfb_probe,
> > > > > > > > +   .remove         = clcdfb_remove,
> > > > > > > > +   .id_table       = clcdfb_id_table,
> > > > > > > > +};
> > > > > > > > +
> > > > > > > > +static int __init amba_clcdfb_init(void)
> > > > > > > > +{
> > > > > > > > +   if (fb_get_options("ambafb", NULL))
> > > > > > > > +           return -ENODEV;
> > > > > > > > +
> > > > > > > > +   return amba_driver_register(&clcd_driver);
> > > > > > > > +}
> > > > > > > > +
> > > > > > > > +module_init(amba_clcdfb_init);
> > > > > > > > +
> > > > > > > > +static void __exit amba_clcdfb_exit(void)
> > > > > > > > +{
> > > > > > > > +   amba_driver_unregister(&clcd_driver);
> > > > > > > > +}
> > > > > > > > +
> > > > > > > > +module_exit(amba_clcdfb_exit);
> > > > > > > > +
> > > > > > > > +MODULE_DESCRIPTION("ARM PrimeCell PL110 CLCD core driver");
> > > > > > > > +MODULE_LICENSE("GPL");
> > > > > > > > diff --git a/include/linux/amba/clcd-regs.h b/include/linux/amba/clcd-regs.h
> > > > > > > > new file mode 100644
> > > > > > > > index 000000000000..421b0fa90d6a
> > > > > > > > --- /dev/null
> > > > > > > > +++ b/include/linux/amba/clcd-regs.h
> > > > > > > > @@ -0,0 +1,87 @@
> > > > > > > > +/*
> > > > > > > > + * David A Rusling
> > > > > > > > + *
> > > > > > > > + * Copyright (C) 2001 ARM Limited
> > > > > > > > + *
> > > > > > > > + * This file is subject to the terms and conditions of the GNU General Public
> > > > > > > > + * License.  See the file COPYING in the main directory of this archive
> > > > > > > > + * for more details.
> > > > > > > > + */
> > > > > > > > +
> > > > > > > > +#ifndef AMBA_CLCD_REGS_H
> > > > > > > > +#define AMBA_CLCD_REGS_H
> > > > > > > > +
> > > > > > > > +/*
> > > > > > > > + * CLCD Controller Internal Register addresses
> > > > > > > > + */
> > > > > > > > +#define CLCD_TIM0          0x00000000
> > > > > > > > +#define CLCD_TIM1          0x00000004
> > > > > > > > +#define CLCD_TIM2          0x00000008
> > > > > > > > +#define CLCD_TIM3          0x0000000c
> > > > > > > > +#define CLCD_UBAS          0x00000010
> > > > > > > > +#define CLCD_LBAS          0x00000014
> > > > > > > > +
> > > > > > > > +#define CLCD_PL110_IENB            0x00000018
> > > > > > > > +#define CLCD_PL110_CNTL            0x0000001c
> > > > > > > > +#define CLCD_PL110_STAT            0x00000020
> > > > > > > > +#define CLCD_PL110_INTR    0x00000024
> > > > > > > > +#define CLCD_PL110_UCUR            0x00000028
> > > > > > > > +#define CLCD_PL110_LCUR            0x0000002C
> > > > > > > > +
> > > > > > > > +#define CLCD_PL111_CNTL            0x00000018
> > > > > > > > +#define CLCD_PL111_IENB            0x0000001c
> > > > > > > > +#define CLCD_PL111_RIS             0x00000020
> > > > > > > > +#define CLCD_PL111_MIS             0x00000024
> > > > > > > > +#define CLCD_PL111_ICR             0x00000028
> > > > > > > > +#define CLCD_PL111_UCUR            0x0000002c
> > > > > > > > +#define CLCD_PL111_LCUR            0x00000030
> > > > > > > > +
> > > > > > > > +#define CLCD_PALL          0x00000200
> > > > > > > > +#define CLCD_PALETTE               0x00000200
> > > > > > > > +
> > > > > > > > +#define TIM2_PCD_LO_MASK   GENMASK(4, 0)
> > > > > > > > +#define TIM2_PCD_LO_BITS   5
> > > > > > > > +#define TIM2_CLKSEL                (1 << 5)
> > > > > > > > +#define TIM2_ACB_MASK              GENMASK(10, 6)
> > > > > > > > +#define TIM2_IVS           (1 << 11)
> > > > > > > > +#define TIM2_IHS           (1 << 12)
> > > > > > > > +#define TIM2_IPC           (1 << 13)
> > > > > > > > +#define TIM2_IOE           (1 << 14)
> > > > > > > > +#define TIM2_BCD           (1 << 26)
> > > > > > > > +#define TIM2_PCD_HI_MASK   GENMASK(31, 27)
> > > > > > > > +#define TIM2_PCD_HI_BITS   5
> > > > > > > > +#define TIM2_PCD_HI_SHIFT  27
> > > > > > > > +
> > > > > > > > +#define CNTL_LCDEN         (1 << 0)
> > > > > > > > +#define CNTL_LCDBPP1               (0 << 1)
> > > > > > > > +#define CNTL_LCDBPP2               (1 << 1)
> > > > > > > > +#define CNTL_LCDBPP4               (2 << 1)
> > > > > > > > +#define CNTL_LCDBPP8               (3 << 1)
> > > > > > > > +#define CNTL_LCDBPP16              (4 << 1)
> > > > > > > > +#define CNTL_LCDBPP16_565  (6 << 1)
> > > > > > > > +#define CNTL_LCDBPP16_444  (7 << 1)
> > > > > > > > +#define CNTL_LCDBPP24              (5 << 1)
> > > > > > > > +#define CNTL_LCDBW         (1 << 4)
> > > > > > > > +#define CNTL_LCDTFT                (1 << 5)
> > > > > > > > +#define CNTL_LCDMONO8              (1 << 6)
> > > > > > > > +#define CNTL_LCDDUAL               (1 << 7)
> > > > > > > > +#define CNTL_BGR           (1 << 8)
> > > > > > > > +#define CNTL_BEBO          (1 << 9)
> > > > > > > > +#define CNTL_BEPO          (1 << 10)
> > > > > > > > +#define CNTL_LCDPWR                (1 << 11)
> > > > > > > > +#define CNTL_LCDVCOMP(x)   ((x) << 12)
> > > > > > > > +#define CNTL_LDMAFIFOTIME  (1 << 15)
> > > > > > > > +#define CNTL_WATERMARK             (1 << 16)
> > > > > > > > +
> > > > > > > > +/* ST Microelectronics variant bits */
> > > > > > > > +#define CNTL_ST_1XBPP_444  0x0
> > > > > > > > +#define CNTL_ST_1XBPP_5551 (1 << 17)
> > > > > > > > +#define CNTL_ST_1XBPP_565  (1 << 18)
> > > > > > > > +#define CNTL_ST_CDWID_12   0x0
> > > > > > > > +#define CNTL_ST_CDWID_16   (1 << 19)
> > > > > > > > +#define CNTL_ST_CDWID_18   (1 << 20)
> > > > > > > > +#define CNTL_ST_CDWID_24   ((1 << 19)|(1 << 20))
> > > > > > > > +#define CNTL_ST_CEAEN              (1 << 21)
> > > > > > > > +#define CNTL_ST_LCDBPP24_PACKED    (6 << 1)
> > > > > > > > +
> > > > > > > > +#endif /* AMBA_CLCD_REGS_H */
> > > > > > > > diff --git a/include/linux/amba/clcd.h b/include/linux/amba/clcd.h
> > > > > > > > new file mode 100644
> > > > > > > > index 000000000000..b6e0cbeaf533
> > > > > > > > --- /dev/null
> > > > > > > > +++ b/include/linux/amba/clcd.h
> > > > > > > > @@ -0,0 +1,290 @@
> > > > > > > > +/*
> > > > > > > > + * linux/include/asm-arm/hardware/amba_clcd.h -- Integrator LCD panel.
> > > > > > > > + *
> > > > > > > > + * David A Rusling
> > > > > > > > + *
> > > > > > > > + * Copyright (C) 2001 ARM Limited
> > > > > > > > + *
> > > > > > > > + * This file is subject to the terms and conditions of the GNU General Public
> > > > > > > > + * License.  See the file COPYING in the main directory of this archive
> > > > > > > > + * for more details.
> > > > > > > > + */
> > > > > > > > +#include <linux/fb.h>
> > > > > > > > +#include <linux/amba/clcd-regs.h>
> > > > > > > > +
> > > > > > > > +enum {
> > > > > > > > +   /* individual formats */
> > > > > > > > +   CLCD_CAP_RGB444         = (1 << 0),
> > > > > > > > +   CLCD_CAP_RGB5551        = (1 << 1),
> > > > > > > > +   CLCD_CAP_RGB565         = (1 << 2),
> > > > > > > > +   CLCD_CAP_RGB888         = (1 << 3),
> > > > > > > > +   CLCD_CAP_BGR444         = (1 << 4),
> > > > > > > > +   CLCD_CAP_BGR5551        = (1 << 5),
> > > > > > > > +   CLCD_CAP_BGR565         = (1 << 6),
> > > > > > > > +   CLCD_CAP_BGR888         = (1 << 7),
> > > > > > > > +
> > > > > > > > +   /* connection layouts */
> > > > > > > > +   CLCD_CAP_444            = CLCD_CAP_RGB444 | CLCD_CAP_BGR444,
> > > > > > > > +   CLCD_CAP_5551           = CLCD_CAP_RGB5551 | CLCD_CAP_BGR5551,
> > > > > > > > +   CLCD_CAP_565            = CLCD_CAP_RGB565 | CLCD_CAP_BGR565,
> > > > > > > > +   CLCD_CAP_888            = CLCD_CAP_RGB888 | CLCD_CAP_BGR888,
> > > > > > > > +
> > > > > > > > +   /* red/blue ordering */
> > > > > > > > +   CLCD_CAP_RGB            = CLCD_CAP_RGB444 | CLCD_CAP_RGB5551 |
> > > > > > > > +                             CLCD_CAP_RGB565 | CLCD_CAP_RGB888,
> > > > > > > > +   CLCD_CAP_BGR            = CLCD_CAP_BGR444 | CLCD_CAP_BGR5551 |
> > > > > > > > +                             CLCD_CAP_BGR565 | CLCD_CAP_BGR888,
> > > > > > > > +
> > > > > > > > +   CLCD_CAP_ALL            = CLCD_CAP_BGR | CLCD_CAP_RGB,
> > > > > > > > +};
> > > > > > > > +
> > > > > > > > +struct backlight_device;
> > > > > > > > +
> > > > > > > > +struct clcd_panel {
> > > > > > > > +   struct fb_videomode     mode;
> > > > > > > > +   signed short            width;  /* width in mm */
> > > > > > > > +   signed short            height; /* height in mm */
> > > > > > > > +   u32                     tim2;
> > > > > > > > +   u32                     tim3;
> > > > > > > > +   u32                     cntl;
> > > > > > > > +   u32                     caps;
> > > > > > > > +   unsigned int            bpp:8,
> > > > > > > > +                           fixedtimings:1,
> > > > > > > > +                           grayscale:1;
> > > > > > > > +   unsigned int            connector;
> > > > > > > > +   struct backlight_device *backlight;
> > > > > > > > +   /*
> > > > > > > > +    * If the B/R lines are switched between the CLCD
> > > > > > > > +    * and the panel we need to know this and not try to
> > > > > > > > +    * compensate with the BGR bit in the control register.
> > > > > > > > +    */
> > > > > > > > +   bool                    bgr_connection;
> > > > > > > > +};
> > > > > > > > +
> > > > > > > > +struct clcd_regs {
> > > > > > > > +   u32                     tim0;
> > > > > > > > +   u32                     tim1;
> > > > > > > > +   u32                     tim2;
> > > > > > > > +   u32                     tim3;
> > > > > > > > +   u32                     cntl;
> > > > > > > > +   unsigned long           pixclock;
> > > > > > > > +};
> > > > > > > > +
> > > > > > > > +struct clcd_fb;
> > > > > > > > +
> > > > > > > > +/*
> > > > > > > > + * the board-type specific routines
> > > > > > > > + */
> > > > > > > > +struct clcd_board {
> > > > > > > > +   const char *name;
> > > > > > > > +
> > > > > > > > +   /*
> > > > > > > > +    * Optional.  Hardware capability flags.
> > > > > > > > +    */
> > > > > > > > +   u32     caps;
> > > > > > > > +
> > > > > > > > +   /*
> > > > > > > > +    * Optional.  Check whether the var structure is acceptable
> > > > > > > > +    * for this display.
> > > > > > > > +    */
> > > > > > > > +   int     (*check)(struct clcd_fb *fb, struct fb_var_screeninfo *var);
> > > > > > > > +
> > > > > > > > +   /*
> > > > > > > > +    * Compulsory.  Decode fb->fb.var into regs->*.  In the case of
> > > > > > > > +    * fixed timing, set regs->* to the register values required.
> > > > > > > > +    */
> > > > > > > > +   void    (*decode)(struct clcd_fb *fb, struct clcd_regs *regs);
> > > > > > > > +
> > > > > > > > +   /*
> > > > > > > > +    * Optional.  Disable any extra display hardware.
> > > > > > > > +    */
> > > > > > > > +   void    (*disable)(struct clcd_fb *);
> > > > > > > > +
> > > > > > > > +   /*
> > > > > > > > +    * Optional.  Enable any extra display hardware.
> > > > > > > > +    */
> > > > > > > > +   void    (*enable)(struct clcd_fb *);
> > > > > > > > +
> > > > > > > > +   /*
> > > > > > > > +    * Setup platform specific parts of CLCD driver
> > > > > > > > +    */
> > > > > > > > +   int     (*setup)(struct clcd_fb *);
> > > > > > > > +
> > > > > > > > +   /*
> > > > > > > > +    * mmap the framebuffer memory
> > > > > > > > +    */
> > > > > > > > +   int     (*mmap)(struct clcd_fb *, struct vm_area_struct *);
> > > > > > > > +
> > > > > > > > +   /*
> > > > > > > > +    * Remove platform specific parts of CLCD driver
> > > > > > > > +    */
> > > > > > > > +   void    (*remove)(struct clcd_fb *);
> > > > > > > > +};
> > > > > > > > +
> > > > > > > > +struct amba_device;
> > > > > > > > +struct clk;
> > > > > > > > +
> > > > > > > > +/* this data structure describes each frame buffer device we find */
> > > > > > > > +struct clcd_fb {
> > > > > > > > +   struct fb_info          fb;
> > > > > > > > +   struct amba_device      *dev;
> > > > > > > > +   struct clk              *clk;
> > > > > > > > +   struct clcd_panel       *panel;
> > > > > > > > +   struct clcd_board       *board;
> > > > > > > > +   void                    *board_data;
> > > > > > > > +   void __iomem            *regs;
> > > > > > > > +   u16                     off_ienb;
> > > > > > > > +   u16                     off_cntl;
> > > > > > > > +   u32                     clcd_cntl;
> > > > > > > > +   u32                     cmap[16];
> > > > > > > > +   bool                    clk_enabled;
> > > > > > > > +};
> > > > > > > > +
> > > > > > > > +static inline void clcdfb_decode(struct clcd_fb *fb, struct clcd_regs *regs)
> > > > > > > > +{
> > > > > > > > +   struct fb_var_screeninfo *var = &fb->fb.var;
> > > > > > > > +   u32 val, cpl;
> > > > > > > > +
> > > > > > > > +   /*
> > > > > > > > +    * Program the CLCD controller registers and start the CLCD
> > > > > > > > +    */
> > > > > > > > +   val = ((var->xres / 16) - 1) << 2;
> > > > > > > > +   val |= (var->hsync_len - 1) << 8;
> > > > > > > > +   val |= (var->right_margin - 1) << 16;
> > > > > > > > +   val |= (var->left_margin - 1) << 24;
> > > > > > > > +   regs->tim0 = val;
> > > > > > > > +
> > > > > > > > +   val = var->yres;
> > > > > > > > +   if (fb->panel->cntl & CNTL_LCDDUAL)
> > > > > > > > +           val /= 2;
> > > > > > > > +   val -= 1;
> > > > > > > > +   val |= (var->vsync_len - 1) << 10;
> > > > > > > > +   val |= var->lower_margin << 16;
> > > > > > > > +   val |= var->upper_margin << 24;
> > > > > > > > +   regs->tim1 = val;
> > > > > > > > +
> > > > > > > > +   val = fb->panel->tim2;
> > > > > > > > +   val |= var->sync & FB_SYNC_HOR_HIGH_ACT  ? 0 : TIM2_IHS;
> > > > > > > > +   val |= var->sync & FB_SYNC_VERT_HIGH_ACT ? 0 : TIM2_IVS;
> > > > > > > > +
> > > > > > > > +   cpl = var->xres_virtual;
> > > > > > > > +   if (fb->panel->cntl & CNTL_LCDTFT)        /* TFT */
> > > > > > > > +           /* / 1 */;
> > > > > > > > +   else if (!var->grayscale)                 /* STN color */
> > > > > > > > +           cpl = cpl * 8 / 3;
> > > > > > > > +   else if (fb->panel->cntl & CNTL_LCDMONO8) /* STN monochrome, 8bit */
> > > > > > > > +           cpl /= 8;
> > > > > > > > +   else                                      /* STN monochrome, 4bit */
> > > > > > > > +           cpl /= 4;
> > > > > > > > +
> > > > > > > > +   regs->tim2 = val | ((cpl - 1) << 16);
> > > > > > > > +
> > > > > > > > +   regs->tim3 = fb->panel->tim3;
> > > > > > > > +
> > > > > > > > +   val = fb->panel->cntl;
> > > > > > > > +   if (var->grayscale)
> > > > > > > > +           val |= CNTL_LCDBW;
> > > > > > > > +
> > > > > > > > +   if (fb->panel->caps && fb->board->caps && var->bits_per_pixel >= 16) {
> > > > > > > > +           /*
> > > > > > > > +            * if board and panel supply capabilities, we can support
> > > > > > > > +            * changing BGR/RGB depending on supplied parameters. Here
> > > > > > > > +            * we switch to what the framebuffer is providing if need
> > > > > > > > +            * be, so if the framebuffer is BGR but the display connection
> > > > > > > > +            * is RGB (first case) we switch it around. Vice versa mutatis
> > > > > > > > +            * mutandis if the framebuffer is RGB but the display connection
> > > > > > > > +            * is BGR, we flip it around.
> > > > > > > > +            */
> > > > > > > > +           if (var->red.offset == 0)
> > > > > > > > +                   val &= ~CNTL_BGR;
> > > > > > > > +           else
> > > > > > > > +                   val |= CNTL_BGR;
> > > > > > > > +           if (fb->panel->bgr_connection)
> > > > > > > > +                   val ^= CNTL_BGR;
> > > > > > > > +   }
> > > > > > > > +
> > > > > > > > +   switch (var->bits_per_pixel) {
> > > > > > > > +   case 1:
> > > > > > > > +           val |= CNTL_LCDBPP1;
> > > > > > > > +           break;
> > > > > > > > +   case 2:
> > > > > > > > +           val |= CNTL_LCDBPP2;
> > > > > > > > +           break;
> > > > > > > > +   case 4:
> > > > > > > > +           val |= CNTL_LCDBPP4;
> > > > > > > > +           break;
> > > > > > > > +   case 8:
> > > > > > > > +           val |= CNTL_LCDBPP8;
> > > > > > > > +           break;
> > > > > > > > +   case 16:
> > > > > > > > +           /*
> > > > > > > > +            * PL110 cannot choose between 5551 and 565 modes in its
> > > > > > > > +            * control register.  It is possible to use 565 with
> > > > > > > > +            * custom external wiring.
> > > > > > > > +            */
> > > > > > > > +           if (amba_part(fb->dev) == 0x110 ||
> > > > > > > > +               var->green.length == 5)
> > > > > > > > +                   val |= CNTL_LCDBPP16;
> > > > > > > > +           else if (var->green.length == 6)
> > > > > > > > +                   val |= CNTL_LCDBPP16_565;
> > > > > > > > +           else
> > > > > > > > +                   val |= CNTL_LCDBPP16_444;
> > > > > > > > +           break;
> > > > > > > > +   case 32:
> > > > > > > > +           val |= CNTL_LCDBPP24;
> > > > > > > > +           break;
> > > > > > > > +   }
> > > > > > > > +
> > > > > > > > +   regs->cntl = val;
> > > > > > > > +   regs->pixclock = var->pixclock;
> > > > > > > > +}
> > > > > > > > +
> > > > > > > > +static inline int clcdfb_check(struct clcd_fb *fb, struct fb_var_screeninfo *var)
> > > > > > > > +{
> > > > > > > > +   var->xres_virtual = var->xres = (var->xres + 15) & ~15;
> > > > > > > > +   var->yres_virtual = var->yres = (var->yres + 1) & ~1;
> > > > > > > > +
> > > > > > > > +#define CHECK(e,l,h) (var->e < l || var->e > h)
> > > > > > > > +   if (CHECK(right_margin, (5+1), 256) ||  /* back porch */
> > > > > > > > +       CHECK(left_margin, (5+1), 256) ||   /* front porch */
> > > > > > > > +       CHECK(hsync_len, (5+1), 256) ||
> > > > > > > > +       var->xres > 4096 ||
> > > > > > > > +       var->lower_margin > 255 ||          /* back porch */
> > > > > > > > +       var->upper_margin > 255 ||          /* front porch */
> > > > > > > > +       var->vsync_len > 32 ||
> > > > > > > > +       var->yres > 1024)
> > > > > > > > +           return -EINVAL;
> > > > > > > > +#undef CHECK
> > > > > > > > +
> > > > > > > > +   /* single panel mode: PCD = max(PCD, 1) */
> > > > > > > > +   /* dual panel mode: PCD = max(PCD, 5) */
> > > > > > > > +
> > > > > > > > +   /*
> > > > > > > > +    * You can't change the grayscale setting, and
> > > > > > > > +    * we can only do non-interlaced video.
> > > > > > > > +    */
> > > > > > > > +   if (var->grayscale != fb->fb.var.grayscale ||
> > > > > > > > +       (var->vmode & FB_VMODE_MASK) != FB_VMODE_NONINTERLACED)
> > > > > > > > +           return -EINVAL;
> > > > > > > > +
> > > > > > > > +#define CHECK(e) (var->e != fb->fb.var.e)
> > > > > > > > +   if (fb->panel->fixedtimings &&
> > > > > > > > +       (CHECK(xres)                ||
> > > > > > > > +        CHECK(yres)                ||
> > > > > > > > +        CHECK(bits_per_pixel)      ||
> > > > > > > > +        CHECK(pixclock)            ||
> > > > > > > > +        CHECK(left_margin)         ||
> > > > > > > > +        CHECK(right_margin)        ||
> > > > > > > > +        CHECK(upper_margin)        ||
> > > > > > > > +        CHECK(lower_margin)        ||
> > > > > > > > +        CHECK(hsync_len)           ||
> > > > > > > > +        CHECK(vsync_len)           ||
> > > > > > > > +        CHECK(sync)))
> > > > > > > > +           return -EINVAL;
> > > > > > > > +#undef CHECK
> > > > > > > > +
> > > > > > > > +   var->nonstd = 0;
> > > > > > > > +   var->accel_flags = 0;
> > > > > > > > +
> > > > > > > > +   return 0;
> > > > > > > > +}
> > > > > > > >
> > > > > > >
> > > > > > > _______________________________________________
> > > > > > > dri-devel mailing list
> > > > > > > dri-devel at lists.freedesktop.org
> > > > > > > https://lists.freedesktop.org/mailman/listinfo/dri-devel
> > > > > >
> > > > > > --
> > > > > > Daniel Vetter
> > > > > > Software Engineer, Intel Corporation
> > > > > > http://blog.ffwll.ch
> > > >
> > > > --
> > > > Daniel Vetter
> > > > Software Engineer, Intel Corporation
> > > > http://blog.ffwll.ch
> > >
> > > --
> > > Daniel Vetter
> > > Software Engineer, Intel Corporation
> > > http://blog.ffwll.ch
>
>
>
> --
> Daniel Vetter
> Software Engineer, Intel Corporation
> http://blog.ffwll.ch



More information about the linux-arm-kernel mailing list