omap4-fb: add driver
Subsystems (3)
PATCH 1/1 omap4-fb: add driver
This patch adds omap4 display controller support.
--- arch/arm/mach-omap/Makefile | 1 + arch/arm/mach-omap/include/mach/omap4-fb.h | 52 +++ arch/arm/mach-omap/omap4_fb.c | 29 ++ drivers/video/Kconfig | 8 + drivers/video/Makefile | 1 + drivers/video/omap4.c | 525 ++++++++++++++++++++++++++++ drivers/video/omap4.h | 408 +++++++++++++++++++++ 7 files changed, 1024 insertions(+) create mode 100644 arch/arm/mach-omap/include/mach/omap4-fb.h create mode 100644 arch/arm/mach-omap/omap4_fb.c create mode 100644 drivers/video/omap4.c create mode 100644 drivers/video/omap4.h
arch/arm/mach-omap/Makefile +1 -0
@@ -28,6 +28,7 @@ obj-$(CONFIG_OMAP3_CLOCK_CONFIG) += omap3_clock.o pbl-$(CONFIG_OMAP3_CLOCK_CONFIG) += omap3_clock.o obj-$(CONFIG_OMAP_GPMC) += gpmc.o devices-gpmc-nand.o obj-$(CONFIG_SHELL_NONE) += xload.o +obj-$(CONFIG_DRIVER_VIDEO_OMAP4) += omap4_fb.o obj-$(CONFIG_I2C_TWL6030) += omap4_twl6030_mmc.o obj-$(CONFIG_OMAP4_USBBOOT) += omap4_rom_usb.o obj-y += gpio.o
arch/arm/mach-omap/include/mach/omap4-fb.h new file +52 -0
52 lines not on this page · raw message
arch/arm/mach-omap/omap4_fb.c new file +29 -0
@@ -0,0 +1,29 @@ +#include <driver.h> +#include <common.h> +#include <linux/ioport.h> +#include <mach/omap4-fb.h> + +static struct resource omap4_fb_resources[] = { + [OMAP4_FB_RESOURCE_DSS ] = { + .start =0x48040000 , + .end =0x48040000 +512 -1 , + .flags =IORESOURCE_MEM |IORESOURCE_MEM_32BIT , + .name ="DSS" , + }, + + [OMAP4_FB_RESOURCE_DISPC ] = { + .start =0x48041000 , + .end =0x48041000 +3072 -1 , + .flags =IORESOURCE_MEM |IORESOURCE_MEM_32BIT , + .name ="DISPC" , + }, +}; + +struct device_d *omap4_add_display (void *pdata) +{ +return add_generic_device_res ("omap4_fb" ,-1 , + omap4_fb_resources, +ARRAY_SIZE (omap4_fb_resources), + pdata); +} +EXPORT_SYMBOL (omap4_add_display);
drivers/video/Kconfig +8 -0
@@ -45,6 +45,14 @@ config DRIVER_VIDEO_S3C24XX help Add support for the S3C244x LCD controller. +config DRIVER_VIDEO_OMAP4 + bool "OMAP4 framebuffer driver" + depends on ARCH_OMAP4 + help + Add support for the OMAP4 Display Controller. + DSI is unsupported, only DISPC parallel mode on LCD2 + is supported. + if DRIVER_VIDEO_S3C24XX config DRIVER_VIDEO_S3C_VERBOSE
drivers/video/Makefile +1 -0
@@ -8,3 +8,4 @@ obj-$(CONFIG_DRIVER_VIDEO_IMX_IPU) += imx-ipu-fb.o obj-$(CONFIG_DRIVER_VIDEO_S3C24XX) += s3c24xx.o obj-$(CONFIG_DRIVER_VIDEO_PXA) += pxa.o obj-$(CONFIG_DRIVER_VIDEO_SDL) += sdl.o +obj-$(CONFIG_DRIVER_VIDEO_OMAP4) += omap4.o
drivers/video/omap4.c new file +525 -0
525 lines not on this page · raw message
drivers/video/omap4.h new file +408 -0
408 lines not on this page · raw message
Signature · 9 lines
1.7.10.4 _______________________________________________ barebox mailing list [email protected] http://lists.infradead.org/mailman/listinfo/barebox
Hi Christoph,
mostly ok, some comments inline.
drivers/video/Kconfig config DRIVER_VIDEO_S3C24XX · line 49
Quoted 10 lines · Christoph Fritz
drivers/video/Kconfig +2
-0@@ -45,6 +45,14 @@ config DRIVER_VIDEO_S3C24XX help Add support for the S3C244x LCD controller. +config DRIVER_VIDEO_OMAP4 + bool "OMAP4 framebuffer driver"
\t please
Quoted 6 lines
drivers/video/Kconfig +6
-0+ depends on ARCH_OMAP4 + help + Add support for the OMAP4 Display Controller. + DSI is unsupported, only DISPC parallel mode on LCD2 + is supported. +
drivers/video/Kconfig config DRIVER_VIDEO_S3C24XX
Quoted 10 lines
drivers/video/Kconfig +9
-0if DRIVER_VIDEO_S3C24XX + +struct omap4fb_colors { + struct fb_bitfield red; + struct fb_bitfield green; + struct fb_bitfield blue; + struct fb_bitfield transp; +}; + +static struct omap4fb_colors const omap4FB_COLORS[] = {
lower case variable names please.
Quoted 22 lines
drivers/video/Kconfig +22
-0+ [0] = { + .red = { .length = 0, .offset = 0 }, + }, + [1] = { + .blue = { .length = 8, .offset = 0 }, + .green = { .length = 8, .offset = 8 }, + .red = { .length = 8, .offset = 16 }, + }, + [2] = { + .blue = { .length = 8, .offset = 0 }, + .green = { .length = 8, .offset = 8 }, + .red = { .length = 8, .offset = 16 }, + .transp = { .length = 8, .offset = 24 }, + }, +}; + +static void omap4fb_fill_bootenv(struct omap4fb_device *fbi, + size_t screen_size) +{ + char buf[sizeof("bootsplash=0x12345678+12345678910")]; + int rc; +
drivers/video/Kconfig
Quoted 10 lines
drivers/video/Kconfig +10
-0+ snprintf(buf, sizeof buf, "bootsplash=0x%08lx+%zu", + (unsigned long)fbi->info.screen_base, + screen_size); + + rc = dev_set_param(fbi->dev, "bootargs", buf); + if (rc < 0) { + dev_err(fbi->dev, "failed to set bootargs '%s': %d\n", + buf, rc); + } +}
Where is this used in the kernel? You should probably rather use the simple fb driver Steven Warren suggested.
drivers/video/Kconfig
Quoted 9 lines
drivers/video/Kconfig +9
-0+ + rc = register_framebuffer(info); + if (rc < 0) { + dev_err(dev, "failed to register framebuffer: %d\n", rc); + goto out; + } + + rc = 0; + dev_info(dev, "registered omap4 framebuffer\n");
The context is already in dev_info, so you can just do a
dev_info(dev, "registered\n");
Quoted 156 lines
drivers/video/Kconfig +156
-0+/* TRM: 10.2.7.3 Display Controller Registers */ +struct omap4_regs_dispc { + uint32_t const revision; + OMAP4_RESERVED(0x04, 0x10); + uint32_t sysconfig; + uint32_t const sysstatus; + uint32_t irqstatus; + uint32_t irqenable; + OMAP4_RESERVED(0x20, 0x40); + uint32_t control1; + uint32_t config1; + OMAP4_RESERVED(0x48, 0x4C); + + uint32_t default_color[2]; + uint32_t trans_color[2]; + + uint32_t const line_status; + uint32_t line_number; + uint32_t timing_h1; + uint32_t timing_v1; + uint32_t pol_freq1; + uint32_t divisor1; + uint32_t global_alpha; + uint32_t size_tv; + uint32_t size_lcd1; + + struct { + uint32_t ba[2]; + uint32_t position; + uint32_t size; + OMAP4_RESERVED(0x90, 0xA0); + uint32_t attributes; + uint32_t buf_threshold; + uint32_t const buf_size_status; + uint32_t row_inc; + uint32_t pixel_inc; + OMAP4_RESERVED(0xB4, 0xB8); + uint32_t table_ba; + } gfx; + + struct omap4_regs_dispc_vid vid1; + OMAP4_RESERVED(0x144, 0x14C); + struct omap4_regs_dispc_vid vid2; + + uint32_t data1_cycle[3]; + + uint32_t vid1_fir_coef_v[8]; + uint32_t vid2_fir_coef_v[8]; + uint32_t cpr1_coef_r; + uint32_t cpr1_coef_g; + uint32_t cpr1_coef_b; + uint32_t gfx_preload; + uint32_t vid_preload[2]; + uint32_t control2; + OMAP4_RESERVED(0x23C, 0x300); + + struct { + uint32_t accu[2]; + uint32_t ba[2]; + struct { + uint32_t h; + uint32_t hv; + } fir_coef[8]; + uint32_t fir_coef_v[8]; + uint32_t attributes; + uint32_t conv_coef0; + uint32_t conv_coef1; + uint32_t conv_coef2; + uint32_t conv_coef3; + uint32_t conv_coef4; + uint32_t const buf_size_status; + uint32_t buf_threshold; + uint32_t fir; + uint32_t picture_size; + uint32_t pixel_inc; + uint32_t position; + uint32_t preload; + uint32_t row_inc; + uint32_t size; + } vid3; + + uint32_t default_color2; + uint32_t trans_color2; + uint32_t cpr2_coef_b; + uint32_t cpr2_coef_g; + uint32_t cpr2_coef_r; + uint32_t data2_cycle[3]; + uint32_t size_lcd2; + OMAP4_RESERVED(0x3D0, 0x400); + uint32_t timing_h2; + uint32_t timing_v2; + uint32_t pol_freq2; + uint32_t divisor2; + OMAP4_RESERVED(0x410, 0x500); + + struct { + uint32_t accu[2]; + uint32_t ba[2]; + struct { + uint32_t h; + uint32_t hv; + } fir_coef[8]; + uint32_t fir_coef_v[8]; + uint32_t attributes; + uint32_t conv_coef0; + uint32_t conv_coef1; + uint32_t conv_coef2; + uint32_t conv_coef3; + uint32_t conv_coef4; + uint32_t const buf_size_status; + uint32_t buf_threshold; + uint32_t fir; + uint32_t picture_size; + uint32_t pixel_inc; + OMAP4_RESERVED(0x59C, 0x5A4); + uint32_t row_inc; + uint32_t size; + } wb; + + OMAP4_RESERVED(0x5AC, 0x600); + uint32_t vid1_ba_uv[2]; + uint32_t vid2_ba_uv[2]; + uint32_t vid3_ba_uv[2]; + uint32_t wb_ba_uv[2]; + uint32_t config2; + uint32_t vid1_attributes2; + uint32_t vid2_attributes2; + uint32_t vid3_attributes2; + uint32_t gamma_table0; + uint32_t gamma_table1; + uint32_t gamma_table2; + uint32_t vid1_fir2; + uint32_t vid1_accu2[2]; + struct { + uint32_t h2; + uint32_t hv2; + } vid1_fir_coef[8]; + uint32_t vid1_fir_coef_v2[8]; + uint32_t vid2_fir2; + uint32_t vid2_accu2[2]; + struct { + uint32_t h2; + uint32_t hv2; + } vid2_fir_coef[8]; + uint32_t vid2_fir_coef_v2[8]; + OMAP4_RESERVED(0x714, 0x724); + uint32_t vid3_fir2; + uint32_t vid3_accu2[2]; + struct { + uint32_t h2; + uint32_t hv2; + } vid3_fir_coef[8]; + uint32_t vid3_fir_coef_v2[8]; + uint32_t wb_fir2; + uint32_t wb_accu2[2]; + OMAP4_RESERVED(0x79C, 0x7A0);
drivers/video/Kconfig
Quoted 10 lines
drivers/video/Kconfig +10
-0+ struct { + uint32_t h2; + uint32_t hv2; + } wb_fir_coef[8]; + uint32_t wb_fir_coef_v2[8]; + uint32_t global_buffer; + uint32_t divisor; + OMAP4_RESERVED(0x808, 0x810); + uint32_t wb_attributes2; +};
This looks awful. I wonder who ever had the idea that placing structs over registers is a good idea.
Sascha
Signature · 9 lines
Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ barebox mailing list [email protected] http://lists.infradead.org/mailman/listinfo/barebox
PATCH v2 1/1 omap4-fb: add driver
This patch adds omap4 display controller support.
--- arch/arm/mach-omap/Makefile | 1 + arch/arm/mach-omap/include/mach/omap4-fb.h | 52 +++ arch/arm/mach-omap/omap4_fb.c | 29 ++ drivers/video/Kconfig | 8 + drivers/video/Makefile | 1 + drivers/video/omap4.c | 507 ++++++++++++++++++++++++++++ drivers/video/omap4.h | 408 ++++++++++++++++++++++ 7 files changed, 1006 insertions(+) create mode 100644 arch/arm/mach-omap/include/mach/omap4-fb.h create mode 100644 arch/arm/mach-omap/omap4_fb.c create mode 100644 drivers/video/omap4.c create mode 100644 drivers/video/omap4.h
arch/arm/mach-omap/Makefile +1 -0
@@ -28,6 +28,7 @@ obj-$(CONFIG_OMAP3_CLOCK_CONFIG) += omap3_clock.o pbl-$(CONFIG_OMAP3_CLOCK_CONFIG) += omap3_clock.o obj-$(CONFIG_OMAP_GPMC) += gpmc.o devices-gpmc-nand.o obj-$(CONFIG_SHELL_NONE) += xload.o +obj-$(CONFIG_DRIVER_VIDEO_OMAP4) += omap4_fb.o obj-$(CONFIG_I2C_TWL6030) += omap4_twl6030_mmc.o obj-$(CONFIG_OMAP4_USBBOOT) += omap4_rom_usb.o obj-y += gpio.o
arch/arm/mach-omap/include/mach/omap4-fb.h new file +52 -0
52 lines not on this page · raw message
arch/arm/mach-omap/omap4_fb.c new file +29 -0
@@ -0,0 +1,29 @@ +#include <driver.h> +#include <common.h> +#include <linux/ioport.h> +#include <mach/omap4-fb.h> + +static struct resource omap4_fb_resources[] = { + [OMAP4_FB_RESOURCE_DSS ] = { + .start =0x48040000 , + .end =0x48040000 +512 -1 , + .flags =IORESOURCE_MEM |IORESOURCE_MEM_32BIT , + .name ="DSS" , + }, + + [OMAP4_FB_RESOURCE_DISPC ] = { + .start =0x48041000 , + .end =0x48041000 +3072 -1 , + .flags =IORESOURCE_MEM |IORESOURCE_MEM_32BIT , + .name ="DISPC" , + }, +}; + +struct device_d *omap4_add_display (void *pdata) +{ +return add_generic_device_res ("omap4_fb" ,-1 , + omap4_fb_resources, +ARRAY_SIZE (omap4_fb_resources), + pdata); +} +EXPORT_SYMBOL (omap4_add_display);
drivers/video/Kconfig +8 -0
@@ -45,6 +45,14 @@ config DRIVER_VIDEO_S3C24XX help Add support for the S3C244x LCD controller. +config DRIVER_VIDEO_OMAP4 + bool "OMAP4 framebuffer driver" + depends on ARCH_OMAP4 + help + Add support for the OMAP4 Display Controller. + DSI is unsupported, only DISPC parallel mode on LCD2 + is supported. + if DRIVER_VIDEO_S3C24XX config DRIVER_VIDEO_S3C_VERBOSE
drivers/video/Makefile +1 -0
@@ -8,3 +8,4 @@ obj-$(CONFIG_DRIVER_VIDEO_IMX_IPU) += imx-ipu-fb.o obj-$(CONFIG_DRIVER_VIDEO_S3C24XX) += s3c24xx.o obj-$(CONFIG_DRIVER_VIDEO_PXA) += pxa.o obj-$(CONFIG_DRIVER_VIDEO_SDL) += sdl.o +obj-$(CONFIG_DRIVER_VIDEO_OMAP4) += omap4.o
drivers/video/omap4.c new file +507 -0
507 lines not on this page · raw message
drivers/video/omap4.h new file +408 -0
408 lines not on this page · raw message
Signature · 8 lines
1.7.10.4 _______________________________________________ barebox mailing list [email protected] http://lists.infradead.org/mailman/listinfo/barebox
Quoted 196 lines · Christoph Fritz
This patch adds omap4 display controller support.
Signed-off-by Christoph Fritz <[email protected]>--- arch/arm/mach-omap/Makefile | 1 + arch/arm/mach-omap/include/mach/omap4-fb.h | 52 +++ arch/arm/mach-omap/omap4_fb.c | 29 ++ drivers/video/Kconfig | 8 + drivers/video/Makefile | 1 + drivers/video/omap4.c | 507 ++++++++++++++++++++++++++++ drivers/video/omap4.h | 408 ++++++++++++++++++++++ 7 files changed, 1006 insertions(+) create mode 100644 arch/arm/mach-omap/include/mach/omap4-fb.h create mode 100644 arch/arm/mach-omap/omap4_fb.c create mode 100644 drivers/video/omap4.c create mode 100644 drivers/video/omap4.harch/arm/mach-omap/Makefile +1
-0@@ -28,6 +28,7 @@ obj-$(CONFIG_OMAP3_CLOCK_CONFIG) += omap3_clock.o pbl-$(CONFIG_OMAP3_CLOCK_CONFIG) += omap3_clock.o obj-$(CONFIG_OMAP_GPMC) += gpmc.o devices-gpmc-nand.o obj-$(CONFIG_SHELL_NONE) += xload.o +obj-$(CONFIG_DRIVER_VIDEO_OMAP4) += omap4_fb.o obj-$(CONFIG_I2C_TWL6030) += omap4_twl6030_mmc.o obj-$(CONFIG_OMAP4_USBBOOT) += omap4_rom_usb.o obj-y += gpio.oarch/arm/mach-omap/include/mach/omap4-fb.h new file +52
-0@@ -0,0 +1,52 @@ +#ifndef H_BAREBOX_ARCH_ARM_MACH_OMAP_MACH_FB4_H +#define H_BAREBOX_ARCH_ARM_MACH_OMAP_MACH_FB4_H + +#include <fb.h> + +/* device resource index */ +enum { +OMAP4_FB_RESOURCE_DSS , +OMAP4_FB_RESOURCE_DISPC , +}; + +#define OMAP_DSS_LCD_TFT (1u << 0) +#define OMAP_DSS_LCD_IVS (1u << 1) +#define OMAP_DSS_LCD_IHS (1u << 2) +#define OMAP_DSS_LCD_IPC (1u << 3) +#define OMAP_DSS_LCD_IEO (1u << 4) +#define OMAP_DSS_LCD_RF (1u << 5) +#define OMAP_DSS_LCD_ONOFF (1u << 6) + +#define OMAP_DSS_LCD_DATALINES (_l) ((_l) << 10) +#define OMAP_DSS_LCD_DATALINES_msk OMAP_DSS_LCD_DATALINES(3u) +#define OMAP_DSS_LCD_DATALINES_12 OMAP_DSS_LCD_DATALINES(0u) +#define OMAP_DSS_LCD_DATALINES_16 OMAP_DSS_LCD_DATALINES(1u) +#define OMAP_DSS_LCD_DATALINES_18 OMAP_DSS_LCD_DATALINES(2u) +#define OMAP_DSS_LCD_DATALINES_24 OMAP_DSS_LCD_DATALINES(3u) + +struct omap4fb_display { +struct fb_videomode mode; + +unsigned long config; + +unsigned int power_on_delay; +unsigned int power_off_delay; +}; + +struct omap4fb_platform_data { +struct omap4fb_display const *displays; +size_t num_displays; + +unsigned int dss_clk_hz; + +unsigned int bpp; + +struct resource const *screen; + +void (*enable)(int p); +}; + +struct device_d ; +struct device_d *omap4_add_display (void *pdata); + +#endif /* H_BAREBOX_ARCH_ARM_MACH_OMAP_MACH_FB4_H */ arch/arm/mach-omap/omap4_fb.c new file +29
-0@@ -0,0 +1,29 @@ +#include <driver.h> +#include <common.h> +#include <linux/ioport.h> +#include <mach/omap4-fb.h> + +static struct resource omap4_fb_resources[] = { + [OMAP4_FB_RESOURCE_DSS ] = { + .start =0x48040000 , + .end =0x48040000 +512 -1 , + .flags =IORESOURCE_MEM |IORESOURCE_MEM_32BIT , + .name ="DSS" , + }, + + [OMAP4_FB_RESOURCE_DISPC ] = { + .start =0x48041000 , + .end =0x48041000 +3072 -1 , + .flags =IORESOURCE_MEM |IORESOURCE_MEM_32BIT , + .name ="DISPC" , + }, +}; + +struct device_d *omap4_add_display (void *pdata) +{ +return add_generic_device_res ("omap4_fb" ,-1 , + omap4_fb_resources, +ARRAY_SIZE (omap4_fb_resources), + pdata); +} +EXPORT_SYMBOL (omap4_add_display);drivers/video/Kconfig +8
-0@@ -45,6 +45,14 @@ config DRIVER_VIDEO_S3C24XX help Add support for the S3C244x LCD controller. +config DRIVER_VIDEO_OMAP4 + bool "OMAP4 framebuffer driver" + depends on ARCH_OMAP4 + help + Add support for the OMAP4 Display Controller. + DSI is unsupported, only DISPC parallel mode on LCD2 + is supported. + if DRIVER_VIDEO_S3C24XX config DRIVER_VIDEO_S3C_VERBOSEdrivers/video/Makefile +1
-0@@ -8,3 +8,4 @@ obj-$(CONFIG_DRIVER_VIDEO_IMX_IPU) += imx-ipu-fb.o obj-$(CONFIG_DRIVER_VIDEO_S3C24XX) += s3c24xx.o obj-$(CONFIG_DRIVER_VIDEO_PXA) += pxa.o obj-$(CONFIG_DRIVER_VIDEO_SDL) += sdl.o +obj-$(CONFIG_DRIVER_VIDEO_OMAP4) += omap4.odrivers/video/omap4.c new file +40
-0@@ -0,0 +1,507 @@ +/* +* TI Omap4 Frame Buffer device driver +* +* Copyright (C) 2013 Christoph Fritz <[email protected]> +* Based on work by Enrico Scholz, sponsored by Phytec +* +* See file CREDITS for list of people who contributed to this +* project. +* +* This program is free software; you can redistribute it and/or +* modify it under the terms of the GNU General Public License as +* published by the Free Software Foundation; either version 2 of +* the License, or (at your option) any later version. +* +* 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 <driver.h> +#include <fb.h> +#include <errno.h> +#include <xfuncs.h> +#include <init.h> +#include <stdio.h> +#include <io.h> +#include <common.h> +#include <malloc.h> +#include <common.h> +#include <clock.h> + +#include <mach/omap4-silicon.h> +#include <mach/omap4-fb.h> + +#include "omap4.h" + +struct omap4fb_device { +struct fb_info info; +struct device_d *dev;
drivers/video/omap4.c line 50
Quoted 10 lines
drivers/video/omap4.c new file +10
-0+ +struct omap4fb_display const *cur_display; + +struct omap4fb_display const *displays; +size_t num_displays; + +struct { +struct omap4_regs_dss __iomem *dss; +struct omap4_regs_dispc __iomem *dispc; + } regs;
please drop those non-sense regs struct
Quoted 21 lines
drivers/video/omap4.c new file +21
-0+ +struct { +void __iomem *addr; +size_t size; + } prealloc_screen; + +struct { +uint32_t dispc_control; +uint32_t dispc_pol_freq; + } shadow; + +struct { +unsigned int dss_clk_hz; +unsigned int lckd; +unsigned int pckd; + } divisor; + +void (*enable_fn)(int ); + +struct fb_videomode video_modes[]; +};
drivers/video/omap4.c line 81
Quoted 10 lines
drivers/video/omap4.c new file +10
-0+ +static inline void fb_write (uint32_t v,void __iomem *addr) +{ +__raw_writel (v, addr); +} + +static inline uint32_t fb_read (void const __iomem *addr) +{ +return __raw_readl (addr); +}
what is the dif?
Quoted 8 lines
drivers/video/omap4.c new file +8
-0+ +static void omap4fb_enable (struct fb_info *info) +{ +struct omap4fb_device *fbi = + container_of(info, struct omap4fb_device, info); +struct omap4_regs_dispc __iomem *dispc = fbi->regs.dispc; + +dev_info (fbi->dev,"%s\n" , __func__);
drivers/video/omap4.c line 99
Quoted 10 lines
drivers/video/omap4.c new file +10
-0+ +if (!fbi->cur_display) { +dev_err (fbi->dev,"no valid mode set\n" ); +return ; + } + +if (fbi->enable_fn) + fbi->enable_fn (1 ); + +udelay (fbi->cur_display->power_on_delay *1000u );
mdelay
Quoted 325 lines
drivers/video/omap4.c new file +325
-0+ +fb_write (fb_read (&dispc->control2) | +DSS_DISPC_CONTROL_LCDENABLE | +1 *DSS_DISPC_CONTROL_LCDENABLESIGNAL , + &dispc->control2); + +fb_write (fb_read (&dispc->vid1.attributes) | + DSS_DISPC_VIDn_ATTRIBUTES_VIDENABLE, + &dispc->vid1.attributes); + +fb_write (fb_read (&dispc->control2) | +DSS_DISPC_CONTROL_GOLCD , + &dispc->control2); +} + +static void omap4fb_disable (struct fb_info *info) +{ +struct omap4fb_device *fbi = + container_of(info, struct omap4fb_device, info); +struct omap4_regs_dispc __iomem *dispc = fbi->regs.dispc; + +dev_info (fbi->dev,"%s\n" , __func__); + +if (!fbi->cur_display) { +dev_err (fbi->dev,"no valid mode set\n" ); +return ; + } + +fb_write (fb_read (&dispc->control2) & + ~(DSS_DISPC_CONTROL_LCDENABLE | +DSS_DISPC_CONTROL_LCDENABLESIGNAL ), + &dispc->control2); + +fb_write (fb_read (&dispc->vid1.attributes) & + ~(DSS_DISPC_VIDn_ATTRIBUTES_VIDENABLE), + &dispc->vid1.attributes); + +if (fbi->prealloc_screen.addr ==NULL ) +/* free frame buffer; but only when screen is not +* preallocated */ +free (info->screen_base); + + info->screen_base =NULL ; + +udelay (fbi->cur_display->power_off_delay *1000u ); + +if (fbi->enable_fn) + fbi->enable_fn (0 ); +} + +static void omap4fb_calc_divisor (struct omap4fb_device *fbi, +struct fb_videomode const *mode) +{ +unsigned int l, k, t, b; + + b =UINT_MAX ; +for (l =1 ; l <256 ; l++) { +for (k =1 ; k <256 ; k++) { + t =abs (mode->pixclock *100 - + (fbi->divisor.dss_clk_hz / l / k)); +if (t <= b) { + b = t; + fbi->divisor.lckd = l; + fbi->divisor.pckd = k; + } + } + } +} + +static unsigned int omap4fb_calc_format (struct fb_info const *info) +{ +switch (info->bits_per_pixel) { +case 24 :return 9 ; +case 32 :return 0x8 ;/* xRGB24-8888 (32-bit container) */ +default : +printf ("%s: unsupported bpp %d\n" , __func__, + info->bits_per_pixel); +return 0 ; + } +} + +struct omap4fb_colors { +struct fb_bitfield red; +struct fb_bitfield green; +struct fb_bitfield blue; +struct fb_bitfield transp; +}; + +static struct omap4fb_colors const omap4fb_col[] = { + [0 ] = { + .red = { .length =0 , .offset =0 }, + }, + [1 ] = { + .blue = { .length =8 , .offset =0 }, + .green = { .length =8 , .offset =8 }, + .red = { .length =8 , .offset =16 }, + }, + [2 ] = { + .blue = { .length =8 , .offset =0 }, + .green = { .length =8 , .offset =8 }, + .red = { .length =8 , .offset =16 }, + .transp = { .length =8 , .offset =24 }, + }, +}; + +static void omap4fb_fill_shadow (struct omap4fb_device *fbi, +struct omap4fb_display const *display) +{ + fbi->shadow.dispc_control =0 ; + fbi->shadow.dispc_pol_freq =0 ; + + fbi->shadow.dispc_control |=DSS_DISPC_CONTROL_STNTFT ; + +switch (display->config & OMAP_DSS_LCD_DATALINES_msk) { +case OMAP_DSS_LCD_DATALINES_12 : + fbi->shadow.dispc_control |=DSS_DISPC_CONTROL_TFTDATALINES_12 ; +break ; +case OMAP_DSS_LCD_DATALINES_16 : + fbi->shadow.dispc_control |=DSS_DISPC_CONTROL_TFTDATALINES_16 ; +break ; +case OMAP_DSS_LCD_DATALINES_18 : + fbi->shadow.dispc_control |=DSS_DISPC_CONTROL_TFTDATALINES_18 ; +break ; +case OMAP_DSS_LCD_DATALINES_24 : + fbi->shadow.dispc_control |=DSS_DISPC_CONTROL_TFTDATALINES_24 ; +break ; + } + +if (display->config &OMAP_DSS_LCD_IPC ) + fbi->shadow.dispc_pol_freq |=DSS_DISPC_POL_FREQ_IPC ; + +if (display->config &OMAP_DSS_LCD_IVS ) + fbi->shadow.dispc_pol_freq |=DSS_DISPC_POL_FREQ_IVS ; + +if (display->config &OMAP_DSS_LCD_IHS ) + fbi->shadow.dispc_pol_freq |=DSS_DISPC_POL_FREQ_IHS ; + +if (display->config &OMAP_DSS_LCD_IEO ) + fbi->shadow.dispc_pol_freq |=DSS_DISPC_POL_FREQ_IEO ; + +if (display->config &OMAP_DSS_LCD_RF ) + fbi->shadow.dispc_pol_freq |=DSS_DISPC_POL_FREQ_RF ; + +if (display->config &OMAP_DSS_LCD_ONOFF ) + fbi->shadow.dispc_pol_freq |=DSS_DISPC_POL_FREQ_ONOFF ; +} + +static int omap4fb_activate_var (struct fb_info *info) +{ +struct omap4fb_device *fbi = + container_of(info, struct omap4fb_device, info); +struct omap4_regs_dispc __iomem *dispc = fbi->regs.dispc; +struct fb_videomode const *mode = info->mode; +size_t size = mode->xres * mode->yres * (info->bits_per_pixel /8 ); +int rc; +unsigned int fmt =omap4fb_calc_format (info); +struct omap4fb_colors const *cols; +size_t i; +struct omap4fb_display const *new_display =NULL ; + +for (i =0 ; i < fbi->num_displays && new_display ==NULL ; ++i) { +if (strcmp (mode->name, fbi->displays[i].mode.name) ==0 ) + new_display = &fbi->displays[i]; + } + +if (WARN_ON (!new_display)) { +dev_err (fbi->dev,"no matching display found for this mode '%s'\n" , + mode->name); + rc = -ENXIO ; + goto out; + } + +if (fbi->prealloc_screen.addr ==NULL ) { +/* case 1: no preallocated screen */ +free (info->screen_base); + info->screen_base =memalign (0x100 , size); + }else if (fbi->prealloc_screen.size < size) { +/* case 2: preallocated screen, but too small */ +dev_err (fbi->dev, +"allocated framebuffer too small (%zu < %zu)\n" , + fbi->prealloc_screen.size, size); + rc = -ENOMEM ; + goto out; + }else { +/* case 3: preallocated screen */ + info->screen_base = fbi->prealloc_screen.addr; + } + +omap4fb_fill_shadow (fbi, new_display); + +omap4fb_calc_divisor (fbi, mode); + +switch (info->bits_per_pixel) { +case 24 : + cols = &omap4fb_col[1 ]; +break ; +case 32 : + cols = &omap4fb_col[2 ]; +break ; +default : + cols = &omap4fb_col[0 ]; + } + + info->red = cols->red; + info->green = cols->green; + info->blue = cols->blue; + info->transp = cols->transp; + +fb_write (fbi->shadow.dispc_control, + &dispc->control2); + +fb_write (fbi->shadow.dispc_pol_freq, + &dispc->pol_freq2); + +fb_write (DSS_DISPC_TIMING_H_HSW (mode->hsync_len -1 ) | +DSS_DISPC_TIMING_H_HFP (mode->right_margin -1 ) | +DSS_DISPC_TIMING_H_HBP (mode->left_margin -1 ), + &dispc->timing_h2); + +fb_write (DSS_DISPC_TIMING_V_VSW (mode->vsync_len -1 ) | +DSS_DISPC_TIMING_V_VFP (mode->lower_margin) | +DSS_DISPC_TIMING_V_VBP (mode->upper_margin), + &dispc->timing_v2); + +fb_write (DSS_DISPC_DIVISOR_ENABLE |DSS_DISPC_DIVISOR_LCD (1 ), + &dispc->divisor); + +fb_write (DSS_DISPC_DIVISOR2_LCD (fbi->divisor.lckd) | +DSS_DISPC_DIVISOR2_PCD (fbi->divisor.pckd), + &dispc->divisor2); + +fb_write (DSS_DISPC_SIZE_LCD_PPL (mode->xres -1 ) | +DSS_DISPC_SIZE_LCD_LPP (mode->yres -1 ), + &dispc->size_lcd2); + +fb_write (0x0000FF00 , &dispc->default_color2); + +/* we use VID1 */ +fb_write ((uintptr_t )info->screen_base, &dispc->vid1.ba[0 ]); +fb_write ((uintptr_t )info->screen_base, &dispc->vid1.ba[1 ]); + +fb_write (DSS_DISPC_VIDn_POSITION_VIDPOSX (0 ) | +DSS_DISPC_VIDn_POSITION_VIDPOSY (0 ), + &dispc->vid1.position); +fb_write (DSS_DISPC_VIDn_SIZE_VIDSIZEX (mode->xres -1 ) | +DSS_DISPC_VIDn_SIZE_VIDSIZEY (mode->yres -1 ), + &dispc->vid1.size); +fb_write (DSS_DISPC_VIDn_PICTURE_SIZE_VIDORGSIZEX (mode->xres -1 ) | +DSS_DISPC_VIDn_PICTURE_SIZE_VIDORGSIZEY (mode->yres -1 ), + &dispc->vid1.picture_size); +fb_write (1 , &dispc->vid1.row_inc); +fb_write (1 , &dispc->vid1.pixel_inc); + +fb_write (0xfff , &dispc->vid_preload[0 ]); + +fb_write (DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT (fmt) | + DSS_DISPC_VIDn_ATTRIBUTES_VIDBURSTSIZE_8x128 | + DSS_DISPC_VIDn_ATTRIBUTES_ZORDERENABLE | + DSS_DISPC_VIDn_ATTRIBUTES_CHANNELOUT2_SECONDARY_LCD, + &dispc->vid1.attributes); + +while (fb_read (&dispc->control2) &DSS_DISPC_CONTROL_GOLCD ) + ;/* noop */ + +fb_write (fb_read (&dispc->control2) | +DSS_DISPC_CONTROL_GOLCD , + &dispc->control2); + + fbi->cur_display = new_display; + info->xres = mode->xres; + info->yres = mode->yres; + + rc =0 ; + +out: +return rc; +} + +static void omap4fb_reset (struct omap4fb_device const *fbi) +{ +struct omap4_regs_dispc __iomem *dispc = fbi->regs.dispc; +struct omap4_regs_dss __iomem *dss = fbi->regs.dss; + +uint32_t v =fb_read (&dispc->control2); + +/* step 1: stop the LCD controller */ +if (v &DSS_DISPC_CONTROL_LCDENABLE ) { +fb_write (v & ~DSS_DISPC_CONTROL_LCDENABLE , + &dispc->control2); + +fb_write (DSS_DISPC_IRQSTATUS_FRAMEDONE2 , &dispc->irqstatus); + +while ((fb_read (&dispc->irqstatus) & +DSS_DISPC_IRQSTATUS_FRAMEDONE ) ==0 ) + ;/* noop */ + } + +/* step 2: wait for reset done status */ +while (!(fb_read (&dss->sysstatus) &DSS_DSS_SYSSTATUS_RESETDONE )) + ;/* noop */ + +/* DSS_CTL: set to reset value */ +fb_write (0 , &dss->ctrl); +} + +static struct fb_ops omap4fb_ops = { + .fb_enable = omap4fb_enable, + .fb_disable = omap4fb_disable, + .fb_activate_var = omap4fb_activate_var, +}; + +static int omap4fb_probe (struct device_d *dev) +{ +struct omap4fb_platform_data const *pdata = dev->platform_data; +struct omap4fb_device *fbi; +struct fb_info *info; +struct omap4_regs_dispc __iomem *dispc; +struct omap4_regs_dss __iomem *dss; +int rc; +size_t i; + +if (!pdata) +return -ENODEV ; + + fbi = xzalloc(sizeof *fbi +
drivers/video/omap4.c line 434
Quoted 10 lines
drivers/video/omap4.c new file +10
-0+ pdata->num_displays * sizeof fbi->video_modes[0]); + info = &fbi->info; + + fbi->dev = dev; + +/* CM_DSS_CLKSTCTRL (TRM: 935) trigger SW_WKUP */ +__raw_writel (0x2 ,0x4a009100 );/* TODO: move this to clockmanagement */ + + fbi->regs.dss =dev_request_mem_region (dev,OMAP4_FB_RESOURCE_DSS ); + fbi->regs.dispc =dev_request_mem_region (dev,OMAP4_FB_RESOURCE_DISPC );
use named dev_request
Quoted 15 lines
drivers/video/omap4.c new file +15
-0+ +if (!fbi->regs.dss || !fbi->regs.dispc) { +dev_err (dev,"Insufficient register description\n" ); + rc = -EINVAL ; + goto out; + } + + dispc = fbi->regs.dispc; + dss = fbi->regs.dss; +dev_info (dev,"HW-Revision 0x%04x 0x%04x\n" , +fb_read (&dispc->revision), +fb_read (&dss->revision)); + + + if (!pdata->dss_clk_hz | !pdata->displays | !pdata->num_displays |
drivers/video/omap4.c line 459
Quoted 10 lines
drivers/video/omap4.c new file +10
-0+ !pdata->bpp) { +dev_err (dev,"Insufficient omap4fb_platform_data\n" ); + rc = -EINVAL ; + goto out; + } + + fbi->enable_fn = pdata->enable; + fbi->displays = pdata->displays; + fbi->num_displays = pdata->num_displays; + fbi->divisor.dss_clk_hz = pdata->dss_clk_hz;
why via pdata?
Quoted 10 lines
drivers/video/omap4.c new file +10
-0+ +for (i =0 ; i < pdata->num_displays; ++i) + fbi->video_modes[i] = pdata->displays[i].mode; + + info->mode_list = fbi->video_modes; + info->num_modes = pdata->num_displays; + + info->priv = fbi; + info->fbops = &omap4fb_ops; + info->bits_per_pixel = pdata->bpp;
drivers/video/omap4.c line 479
Quoted 10 lines
drivers/video/omap4.c new file +10
-0+ +if (pdata->screen) { + fbi->prealloc_screen.addr = + (void __iomem *)pdata->screen->start; + fbi->prealloc_screen.size =resource_size (pdata->screen); + } + +omap4fb_reset (fbi); + +dev_add_param (fbi->dev,"bootargs" ,NULL ,NULL ,0 );
??
Quoted 451 lines
drivers/video/omap4.c new file +28
-0+ + rc =register_framebuffer (info); +if (rc <0 ) { +dev_err (dev,"failed to register framebuffer: %d\n" , rc); + goto out; + } + + rc =0 ; +dev_info (dev,"registered\n" ); + +out: +if (rc <0 ) +free (fbi); + +return rc; +} + +static struct driver_d omap4fb_driver = { + .name ="omap4_fb" , + .probe = omap4fb_probe, +}; + +static int omap4fb_init (void ) +{ +return platform_driver_register (&omap4fb_driver); +} + +device_initcall (omap4fb_init);drivers/video/omap4.h new file +408
-0@@ -0,0 +1,408 @@ +/* +* TI Omap4 Frame Buffer device driver +* +* Copyright (C) 2013 Christoph Fritz <[email protected]> +* +* See file CREDITS for list of people who contributed to this +* project. +* +* This program is free software; you can redistribute it and/or +* modify it under the terms of the GNU General Public License as +* published by the Free Software Foundation; either version 2 of +* the License, or (at your option) any later version. +* +* 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 H_BAREBOX_DRIVER_VIDEO_OMAP4_REGS_H +#define H_BAREBOX_DRIVER_VIDEO_OMAP4_REGS_H + +#include <types.h> +#include <common.h> + +#define OMAP4_RESERVED (_f, _t) \ + uint32_t const reserved_ ## _f ## _ ## _t[((_t) - (_f)) / 4] + + +/* TRM: 10.1.3.2 DSS Registers */ +struct omap4_regs_dss { +uint32_t revision; + OMAP4_RESERVED(0x4, 0x14); +uint32_t const sysstatus; + OMAP4_RESERVED(0x18, 0x40); +uint32_t ctrl; + OMAP4_RESERVED(0x44, 0x5C); +uint32_t const status; +}; + + +struct omap4_regs_dispc_vid { +uint32_t ba[2 ]; +uint32_t position; +uint32_t size; +uint32_t attributes; +uint32_t buf_threshold; +uint32_t const buf_size_status; +uint32_t row_inc; +uint32_t pixel_inc; +uint32_t fir; +uint32_t picture_size; +uint32_t accu[2 ]; +struct { +uint32_t h; +uint32_t hv; + } fir_coef[8 ]; +uint32_t conv_coef0; +uint32_t conv_coef1; +uint32_t conv_coef2; +uint32_t conv_coef3; +uint32_t conv_coef4; +}; + +/* TRM: 10.2.7.3 Display Controller Registers */ +struct omap4_regs_dispc { +uint32_t const revision; + OMAP4_RESERVED(0x04, 0x10); +uint32_t sysconfig; +uint32_t const sysstatus; +uint32_t irqstatus; +uint32_t irqenable; + OMAP4_RESERVED(0x20, 0x40); +uint32_t control1; +uint32_t config1; + OMAP4_RESERVED(0x48, 0x4C); + +uint32_t default_color[2 ]; +uint32_t trans_color[2 ]; + +uint32_t const line_status; +uint32_t line_number; +uint32_t timing_h1; +uint32_t timing_v1; +uint32_t pol_freq1; +uint32_t divisor1; +uint32_t global_alpha; +uint32_t size_tv; +uint32_t size_lcd1; + +struct { +uint32_t ba[2 ]; +uint32_t position; +uint32_t size; + OMAP4_RESERVED(0x90, 0xA0); +uint32_t attributes; +uint32_t buf_threshold; +uint32_t const buf_size_status; +uint32_t row_inc; +uint32_t pixel_inc; + OMAP4_RESERVED(0xB4, 0xB8); +uint32_t table_ba; + } gfx; + +struct omap4_regs_dispc_vid vid1; + OMAP4_RESERVED(0x144, 0x14C); +struct omap4_regs_dispc_vid vid2; + +uint32_t data1_cycle[3 ]; + +uint32_t vid1_fir_coef_v[8 ]; +uint32_t vid2_fir_coef_v[8 ]; +uint32_t cpr1_coef_r; +uint32_t cpr1_coef_g; +uint32_t cpr1_coef_b; +uint32_t gfx_preload; +uint32_t vid_preload[2 ]; +uint32_t control2; + OMAP4_RESERVED(0x23C, 0x300); + +struct { +uint32_t accu[2 ]; +uint32_t ba[2 ]; +struct { +uint32_t h; +uint32_t hv; + } fir_coef[8 ]; +uint32_t fir_coef_v[8 ]; +uint32_t attributes; +uint32_t conv_coef0; +uint32_t conv_coef1; +uint32_t conv_coef2; +uint32_t conv_coef3; +uint32_t conv_coef4; +uint32_t const buf_size_status; +uint32_t buf_threshold; +uint32_t fir; +uint32_t picture_size; +uint32_t pixel_inc; +uint32_t position; +uint32_t preload; +uint32_t row_inc; +uint32_t size; + } vid3; + +uint32_t default_color2; +uint32_t trans_color2; +uint32_t cpr2_coef_b; +uint32_t cpr2_coef_g; +uint32_t cpr2_coef_r; +uint32_t data2_cycle[3 ]; +uint32_t size_lcd2; + OMAP4_RESERVED(0x3D0, 0x400); +uint32_t timing_h2; +uint32_t timing_v2; +uint32_t pol_freq2; +uint32_t divisor2; + OMAP4_RESERVED(0x410, 0x500); + +struct { +uint32_t accu[2 ]; +uint32_t ba[2 ]; +struct { +uint32_t h; +uint32_t hv; + } fir_coef[8 ]; +uint32_t fir_coef_v[8 ]; +uint32_t attributes; +uint32_t conv_coef0; +uint32_t conv_coef1; +uint32_t conv_coef2; +uint32_t conv_coef3; +uint32_t conv_coef4; +uint32_t const buf_size_status; +uint32_t buf_threshold; +uint32_t fir; +uint32_t picture_size; +uint32_t pixel_inc; + OMAP4_RESERVED(0x59C, 0x5A4); +uint32_t row_inc; +uint32_t size; + } wb; + + OMAP4_RESERVED(0x5AC, 0x600); +uint32_t vid1_ba_uv[2 ]; +uint32_t vid2_ba_uv[2 ]; +uint32_t vid3_ba_uv[2 ]; +uint32_t wb_ba_uv[2 ]; +uint32_t config2; +uint32_t vid1_attributes2; +uint32_t vid2_attributes2; +uint32_t vid3_attributes2; +uint32_t gamma_table0; +uint32_t gamma_table1; +uint32_t gamma_table2; +uint32_t vid1_fir2; +uint32_t vid1_accu2[2 ]; +struct { +uint32_t h2; +uint32_t hv2; + } vid1_fir_coef[8 ]; +uint32_t vid1_fir_coef_v2[8 ]; +uint32_t vid2_fir2; +uint32_t vid2_accu2[2 ]; +struct { +uint32_t h2; +uint32_t hv2; + } vid2_fir_coef[8 ]; +uint32_t vid2_fir_coef_v2[8 ]; + OMAP4_RESERVED(0x714, 0x724); +uint32_t vid3_fir2; +uint32_t vid3_accu2[2 ]; +struct { +uint32_t h2; +uint32_t hv2; + } vid3_fir_coef[8 ]; +uint32_t vid3_fir_coef_v2[8 ]; +uint32_t wb_fir2; +uint32_t wb_accu2[2 ]; + OMAP4_RESERVED(0x79C, 0x7A0); +struct { +uint32_t h2; +uint32_t hv2; + } wb_fir_coef[8 ]; +uint32_t wb_fir_coef_v2[8 ]; +uint32_t global_buffer; +uint32_t divisor; + OMAP4_RESERVED(0x808, 0x810); +uint32_t wb_attributes2; +}; + +#define DSS_DISPC_VIDn_POSITION_VIDPOSX (_x) ((_x) << 0) +#define DSS_DISPC_VIDn_POSITION_VIDPOSY (_y) ((_y) << 16) + +#define DSS_DISPC_VIDn_PICTURE_SIZE_VIDORGSIZEX (_x) ((_x) << 0) +#define DSS_DISPC_VIDn_PICTURE_SIZE_VIDORGSIZEY (_y) ((_y) << 16) + +#define DSS_DISPC_VIDn_SIZE_VIDSIZEX (_x) ((_x) << 0) +#define DSS_DISPC_VIDn_SIZE_VIDSIZEY (_y) ((_y) << 16) + +#define DSS_DISPC_SIZE_LCD_PPL (_x) ((_x) << 0) +#define DSS_DISPC_SIZE_LCD_LPP (_y) ((_y) << 16) + +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDENABLE (1u << 0) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT (_fmt) ((_fmt) << 1) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT_RGB12 \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT(4u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT_ARGB16 \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT(5u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT_RGB16 \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT(6u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT_ARGB16o \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT(7u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT_xRGB24u \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT(8u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT_RGB24p \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT(9u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT_YUV2 \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT(10u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT_UYVY \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT(11u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT_ARGB32 \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT(12u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT_RGBA32 \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT(13u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT_xRGB32 \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT(14u) + +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDRESIZEENABLE (_f) ((_f) << 5) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDRESIZEENABLE_NONE \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDRESIZEENABLE(0u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDRESIZEENABLE_H \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDRESIZEENABLE(1u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDRESIZEENABLE_V \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDRESIZEENABLE(2u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDRESIZEENABLE_HV \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDRESIZEENABLE(3u) + +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDCOLORCONVENABLE (1u << 9) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDREPLICATIONENABLE (1u << 10) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDFULLRANGE (1u << 11) + +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDROTATION (_r) ((_r) << 12) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDROTATION_0 \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDROTATION(0u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDROTATION_90 \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDROTATION(1u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDROTATION_180 \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDROTATION(2u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDROTATION_270 \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDROTATION(3u) + +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDBURSTSIZE (_b) ((_b) << 14) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDBURSTSIZE_2x128 \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDBURSTSIZE(0u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDBURSTSIZE_4x128 \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDBURSTSIZE(1u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDBURSTSIZE_8x128 \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDBURSTSIZE(2u) + +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDCHANNELOUT (1u << 16) +#define DSS_DISPC_VIDn_ATTRIBUTES_SELFREFRESHAUTO (1u << 17) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDFIFOPRELOAD (1u << 19) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDVERTICALTAPS (1u << 21) +#define DSS_DISPC_VIDn_ATTRIBUTES_DOUBLESTRIDE (1u << 22) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDARBITRATION (1u << 23) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDSELFREFRESH (1u << 24) +#define DSS_DISPC_VIDn_ATTRIBUTES_ZORDERENABLE (1u << 25) + +#define DSS_DISPC_VIDn_ATTRIBUTES_CHANNELOUT2 (_b) ((_b) << 30) +#define DSS_DISPC_VIDn_ATTRIBUTES_CHANNELOUT2_PRIMARY_LCD \ + DSS_DISPC_VIDn_ATTRIBUTES_CHANNELOUT2(0u) +#define DSS_DISPC_VIDn_ATTRIBUTES_CHANNELOUT2_SECONDARY_LCD \ + DSS_DISPC_VIDn_ATTRIBUTES_CHANNELOUT2(1u) +#define DSS_DISPC_VIDn_ATTRIBUTES_CHANNELOUT2_WRITEBACK_MEM \ + DSS_DISPC_VIDn_ATTRIBUTES_CHANNELOUT2(3u) + +#define DSS_DISPC_CONFIG_PIXELGATED (1u << 0) +#define DSS_DISPC_CONFIG_PIXELDATAGATED (1u << 4) +#define DSS_DISPC_CONFIG_PIXELCLOCKGATED (1u << 5) +#define DSS_DISPC_CONFIG_HSYNCGATED (1u << 6) +#define DSS_DISPC_CONFIG_VSYNCGATED (1u << 7) +#define DSS_DISPC_CONFIG_ACBIASGATED (1u << 8) +#define DSS_DISPC_CONFIG_FUNCGATED (1u << 9) +#define DSS_DISPC_CONFIG_TCKLCDENABLE (1u << 10) +#define DSS_DISPC_CONFIG_TCKLCDSELECTION (1u << 11) +#define DSS_DISPC_CONFIG_CPR (1u << 15) +#define DSS_DISPC_CONFIG_FIFOHANDCHECK (1u << 16) +#define DSS_DISPC_CONFIG_OUTPUTMODE (1u << 22) +#define DSS_DISPC_CONFIG_FIDFIRST (1u << 23) +#define DSS_DISPC_CONFIG_COLORCONV (1u << 24) +#define DSS_DISPC_CONFIG_FULLRANGE (1u << 25) + +#define DSS_DISPC_CONTROL_LCDENABLE (1u << 0) +#define DSS_DISPC_CONTROL_TVENABLE (1u << 1) +#define DSS_DISPC_CONTROL_MONOCOLOR (1u << 2) +#define DSS_DISPC_CONTROL_STNTFT (1u << 3) +#define DSS_DISPC_CONTROL_M8B (1u << 4) +#define DSS_DISPC_CONTROL_GOLCD (1u << 5) +#define DSS_DISPC_CONTROL_GOTV (1u << 6) +#define DSS_DISPC_CONTROL_STDITHERENABLE (1u << 7) + +#define DSS_DISPC_CONTROL_TFTDATALINES (_l) ((_l) << 8) +#define DSS_DISPC_CONTROL_TFTDATALINES_12 \ + DSS_DISPC_CONTROL_TFTDATALINES(0u) +#define DSS_DISPC_CONTROL_TFTDATALINES_16 \ + DSS_DISPC_CONTROL_TFTDATALINES(1u) +#define DSS_DISPC_CONTROL_TFTDATALINES_18 \ + DSS_DISPC_CONTROL_TFTDATALINES(2u) +#define DSS_DISPC_CONTROL_TFTDATALINES_24 \ + DSS_DISPC_CONTROL_TFTDATALINES(3u) + +#define DSS_DISPC_CONTROL_STALLMODE (1u << 11) +#define DSS_DISPC_CONTROL_OVERLAYOPTIMIZATION (1u << 12) +#define DSS_DISPC_CONTROL_GPIN0 (1u << 13)/* ro */ +#define DSS_DISPC_CONTROL_GPIN1 (1u << 14)/* ro */ +#define DSS_DISPC_CONTROL_GPOUT0 (1u << 15) +#define DSS_DISPC_CONTROL_GPOUT1 (1u << 16) +#define DSS_DISPC_CONTROL_HT (_ht) ((_ht) << 17) +#define DSS_DISPC_CONTROL_TDMENABLE (1u << 20) +#define DSS_DISPC_CONTROL_TDMPARALLELMODE (_pm) ((_pm) << 21) +#define DSS_DISPC_CONTROL_TDMCYCLEFORMAT (_cf) ((_cf) << 23) +#define DSS_DISPC_CONTROL_TDMUNUSEDBITS (_ub) ((_ub) << 25) +#define DSS_DISPC_CONTROL_PCKFREEENABLE (1u << 27) +#define DSS_DISPC_CONTROL_LCDENABLESIGNAL (1u << 28) +#define DSS_DISPC_CONTROL_LCDENABLEPOL (1u << 29) +#define DSS_DISPC_CONTROL_SPATIALTEMPD (_df) ((_df) << 30) + +#define DSS_DISPC_POL_FREQ_IVS (1u << 12) +#define DSS_DISPC_POL_FREQ_IHS (1u << 13) +#define DSS_DISPC_POL_FREQ_IPC (1u << 14) +#define DSS_DISPC_POL_FREQ_IEO (1u << 15) +#define DSS_DISPC_POL_FREQ_RF (1u << 16) +#define DSS_DISPC_POL_FREQ_ONOFF (1u << 17) + +#define DSS_DISPC_TIMING_H_HSW (_hsw) ((_hsw) << 0) +#define DSS_DISPC_TIMING_H_HFP (_hfp) ((_hfp) << 8) +#define DSS_DISPC_TIMING_H_HBP (_hbp) ((_hbp) << 20) + +#define DSS_DISPC_TIMING_V_VSW (_vsw) ((_vsw) << 0) +#define DSS_DISPC_TIMING_V_VFP (_vfp) ((_vfp) << 8) +#define DSS_DISPC_TIMING_V_VBP (_vbp) ((_vbp) << 20) + +#define DSS_DISPC_DIVISOR_ENABLE (1u << 0) +#define DSS_DISPC_DIVISOR_LCD (_lcd) ((_lcd) << 16) + +#define DSS_DISPC_DIVISOR2_PCD (_pcd) ((_pcd) << 0) +#define DSS_DISPC_DIVISOR2_LCD (_lcd) ((_lcd) << 16) + +#define DSS_DISPC_IRQSTATUS_FRAMEDONE (1u << 0) +#define DSS_DISPC_IRQSTATUS_FRAMEDONE2 (1u << 22) + +#define DSS_DSS_SYSCONFIG_SOFTRESET (1u << 1) +#define DSS_DSS_SYSSTATUS_RESETDONE (1u << 0) + +#define DSS_DSS_CONTROL_DISPC_CLK_SWITCH (1u << 0) +#define DSS_DSS_CONTROL_DSI_CLK_SWITCH (1u << 1) + +#define CM_FCLKEN_DSS_EN_DSS1 (1u << 0) +#define CM_FCLKEN_DSS_EN_DSS2 (1u << 1) +#define CM_FCLKEN_DSS_EN_TV (1u << 2) + +#define CM_ICLKEN_DSS_EN_DSS (1u << 0) + + +#undef OMAP4_RESERVED + +#endif /* H_BAREBOX_DRIVER_VIDEO_OMAP4_REGS_H */ Signature · 8 lines
1.7.10.4 _______________________________________________ barebox mailing list [email protected] http://lists.infradead.org/mailman/listinfo/barebox
_______________________________________________
barebox mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/barebox
Quoted 14 lines · Jean-Christophe PLAGNIOL-VILLARD
drivers/video/omap4.c line 81
Quoted 12 lines · Christoph Fritz
This patch adds omap4 display controller support.
drivers/video/omap4.c new file +10
-0+ +static inline void fb_write (uint32_t v,void __iomem *addr) +{ +__raw_writel (v, addr); +} + +static inline uint32_t fb_read (void const __iomem *addr) +{ +return __raw_readl (addr); +}what is the dif?
Maybe Christoph understands what you mean here. I don't
Quoted 4 lines
drivers/video/omap4.c line 434
Quoted 3 lines
drivers/video/omap4.c new file +3
-0+ + fbi->regs.dss =dev_request_mem_region (dev,OMAP4_FB_RESOURCE_DSS ); + fbi->regs.dispc =dev_request_mem_region (dev,OMAP4_FB_RESOURCE_DISPC );use named
dev_request
We don't have such a function.
Could you please trim the context to sensible limits when reviewing patches? It's hard to find your comments when you keep the complete patch.
Sascha
Signature · 9 lines
Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ barebox mailing list [email protected] http://lists.infradead.org/mailman/listinfo/barebox
Quoted 16 lines · Sascha Hauer
Quoted 14 lines · Jean-Christophe PLAGNIOL-VILLARD
drivers/video/omap4.c line 81
Quoted 12 lines · Christoph Fritz
This patch adds omap4 display controller support.
drivers/video/omap4.c new file +10
-0+ +static inline void fb_write (uint32_t v,void __iomem *addr) +{ +__raw_writel (v, addr); +} + +static inline uint32_t fb_read (void const __iomem *addr) +{ +return __raw_readl (addr); +}what is the dif?
Quoted 1 line
Maybe Christoph understands what you mean here. I don't
those 2 inline as useless
Quoted 7 lines
Quoted 5 lines
drivers/video/omap4.c line 434
Quoted 4 lines
drivers/video/omap4.c new file +3
-0+ + fbi->regs.dss =dev_request_mem_region (dev,OMAP4_FB_RESOURCE_DSS ); + fbi->regs.dispc =dev_request_mem_region (dev,OMAP4_FB_RESOURCE_DISPC );use named
dev_requestWe don't have such a function.
yes we do
dev_request_mem_region_by_name
Quoted 12 lines
Could you please trim the context to sensible limits when reviewing patches? It's hard to find your comments when you keep the complete patch.
Sascha
Signature · 4 lines
Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
_______________________________________________
barebox mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/barebox
Hi Jean-Christophe
Quoted 17 lines · Jean-Christophe PLAGNIOL-VILLARD
Quoted 16 lines · Sascha Hauer
Quoted 14 lines · Jean-Christophe PLAGNIOL-VILLARD
drivers/video/omap4.c line 81
Quoted 12 lines · Christoph Fritz
This patch adds omap4 display controller support.
drivers/video/omap4.c new file +10
-0+ +static inline void fb_write (uint32_t v,void __iomem *addr) +{ +__raw_writel (v, addr); +} + +static inline uint32_t fb_read (void const __iomem *addr) +{ +return __raw_readl (addr); +}what is the dif?
Quoted 2 lines
Quoted 1 line
Maybe Christoph understands what you mean here. I don't
those 2 inline as useless
You want me to purge the "inline" because gcc is smart enough to use inlining on its own?
Or are you noting that using a function here is overkill for only one call inside?
I use functions here because it makes debugging more easy and if you have a weird hardware config, here you can easily adapt the read and write functions.
Quoted 9 lines
Quoted 6 lines
Quoted 4 lines
drivers/video/omap4.c line 434
Quoted 3 lines
drivers/video/omap4.c new file +3
-0+ + fbi->regs.dss =dev_request_mem_region (dev,OMAP4_FB_RESOURCE_DSS ); + fbi->regs.dispc =dev_request_mem_region (dev,OMAP4_FB_RESOURCE_DISPC );use named
dev_requestWe don't have such a function.
yes we do
dev_request_mem_region_by_name
Ok
Thanks
-- Christoph
_______________________________________________
barebox mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/barebox
*ping*
Quoted 40 lines · Christoph Fritz
Hi Jean-Christophe
Quoted 17 lines · Jean-Christophe PLAGNIOL-VILLARD
Quoted 16 lines · Sascha Hauer
Quoted 14 lines · Jean-Christophe PLAGNIOL-VILLARD
drivers/video/omap4.c line 81
Quoted 12 lines · Christoph Fritz
This patch adds omap4 display controller support.
drivers/video/omap4.c new file +10
-0+ +static inline void fb_write (uint32_t v,void __iomem *addr) +{ +__raw_writel (v, addr); +} + +static inline uint32_t fb_read (void const __iomem *addr) +{ +return __raw_readl (addr); +}what is the dif?
Quoted 2 lines
Quoted 1 line
Maybe Christoph understands what you mean here. I don't
those 2 inline as useless
You want me to purge the "inline" because gcc is smart enough to use inlining on its own?
Or are you noting that using a function here is overkill for only one call inside?
I use functions here because it makes debugging more easy and if you have a weird hardware config, here you can easily adapt the read and write functions.
Quoted 6 lines
Quoted 6 lines
Quoted 4 lines
drivers/video/omap4.c line 434
Quoted 3 lines
drivers/video/omap4.c new file +3
-0+ + fbi->regs.dss =dev_request_mem_region (dev,OMAP4_FB_RESOURCE_DSS ); + fbi->regs.dispc =dev_request_mem_region (dev,OMAP4_FB_RESOURCE_DISPC );use named
dev_requestWe don't have such a function.
Quoted 9 lines
Quoted 3 lines
yes we do
dev_request_mem_region_by_nameOk
Thanks
-- Christoph
_______________________________________________
barebox mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/barebox
This patch adds omap4 display controller support.
---
changes since v2:
- use dev_request_mem_region_by_name()
--- arch/arm/mach-omap/Makefile | 1 + arch/arm/mach-omap/include/mach/omap4-fb.h | 46 +++ arch/arm/mach-omap/omap4_fb.c | 27 ++ drivers/video/Kconfig | 8 + drivers/video/Makefile | 1 + drivers/video/omap4.c | 507 ++++++++++++++++++++++++++++ drivers/video/omap4.h | 408 ++++++++++++++++++++++ 7 files changed, 998 insertions(+) create mode 100644 arch/arm/mach-omap/include/mach/omap4-fb.h create mode 100644 arch/arm/mach-omap/omap4_fb.c create mode 100644 drivers/video/omap4.c create mode 100644 drivers/video/omap4.h
arch/arm/mach-omap/Makefile +1 -0
@@ -28,6 +28,7 @@ obj-$(CONFIG_OMAP3_CLOCK_CONFIG) += omap3_clock.o pbl-$(CONFIG_OMAP3_CLOCK_CONFIG) += omap3_clock.o obj-$(CONFIG_OMAP_GPMC) += gpmc.o devices-gpmc-nand.o obj-$(CONFIG_SHELL_NONE) += xload.o +obj-$(CONFIG_DRIVER_VIDEO_OMAP4) += omap4_fb.o obj-$(CONFIG_I2C_TWL6030) += omap4_twl6030_mmc.o obj-$(CONFIG_OMAP4_USBBOOT) += omap4_rom_usb.o obj-y += gpio.o
arch/arm/mach-omap/include/mach/omap4-fb.h new file +46 -0
46 lines not on this page · raw message
arch/arm/mach-omap/omap4_fb.c new file +27 -0
@@ -0,0 +1,27 @@ +#include <driver.h> +#include <common.h> +#include <linux/ioport.h> +#include <mach/omap4-fb.h> + +static struct resource omap4_fb_resources[] = { + { + .name ="omap4_dss" , + .start =0x48040000 , + .end =0x48040000 +512 -1 , + .flags =IORESOURCE_MEM |IORESOURCE_MEM_32BIT , + }, { + .name ="omap4_dispc" , + .start =0x48041000 , + .end =0x48041000 +3072 -1 , + .flags =IORESOURCE_MEM |IORESOURCE_MEM_32BIT , + }, +}; + +struct device_d *omap4_add_display (void *pdata) +{ +return add_generic_device_res ("omap4_fb" ,-1 , + omap4_fb_resources, +ARRAY_SIZE (omap4_fb_resources), + pdata); +} +EXPORT_SYMBOL (omap4_add_display);
drivers/video/Kconfig +8 -0
@@ -45,6 +45,14 @@ config DRIVER_VIDEO_S3C24XX help Add support for the S3C244x LCD controller. +config DRIVER_VIDEO_OMAP4 + bool "OMAP4 framebuffer driver" + depends on ARCH_OMAP4 + help + Add support for the OMAP4 Display Controller. + DSI is unsupported, only DISPC parallel mode on LCD2 + is supported. + if DRIVER_VIDEO_S3C24XX config DRIVER_VIDEO_S3C_VERBOSE
drivers/video/Makefile +1 -0
@@ -8,3 +8,4 @@ obj-$(CONFIG_DRIVER_VIDEO_IMX_IPU) += imx-ipu-fb.o obj-$(CONFIG_DRIVER_VIDEO_S3C24XX) += s3c24xx.o obj-$(CONFIG_DRIVER_VIDEO_PXA) += pxa.o obj-$(CONFIG_DRIVER_VIDEO_SDL) += sdl.o +obj-$(CONFIG_DRIVER_VIDEO_OMAP4) += omap4.o
drivers/video/omap4.c new file +507 -0
507 lines not on this page · raw message
drivers/video/omap4.h new file +408 -0
408 lines not on this page · raw message
Signature · 9 lines
1.7.10.4 _______________________________________________ barebox mailing list [email protected] http://lists.infradead.org/mailman/listinfo/barebox
Quoted 7 lines · Christoph Fritz
This patch adds omap4 display controller support.
Signed-off-by Christoph Fritz <[email protected]>---
changes since v2:
- usedev_request_mem_region_by_name()
---
Quoted 8 lines
arch/arm/mach-omap/Makefile | 1 + arch/arm/mach-omap/include/mach/omap4-fb.h | 46 +++ arch/arm/mach-omap/omap4_fb.c | 27 ++ drivers/video/Kconfig | 8 + drivers/video/Makefile | 1 + drivers/video/omap4.c | 507 ++++++++++++++++++++++++++++ drivers/video/omap4.h | 408 ++++++++++++++++++++++ 7 files changed, 998 insertions(+)
In linux we does not use the struct for regs and I'll *never* accept this so please drop me this. This is a huge nightmare to maintain and hugely easy to move registers by touching the struct
really DROP IT
Best Regards,
J.
Quoted 1,062 lines
create mode 100644
arch/arm/mach-omap/include/mach/omap4-fb.hcreate mode 100644arch/arm/mach-omap/omap4_fb.ccreate mode 100644drivers/video/omap4.c
create mode 100644drivers/video/omap4.harch/arm/mach-omap/Makefile +0
-0@@ -28,6 +28,7 @@ obj-$(CONFIG_OMAP3_CLOCK_CONFIG) += omap3_clock.opbl-$(CONFIG_OMAP3_CLOCK_CONFIG) += omap3_clock.o obj-$(CONFIG_OMAP_GPMC) += gpmc.o devices-gpmc-nand.o obj-$(CONFIG_SHELL_NONE) += xload.o +obj-$(CONFIG_DRIVER_VIDEO_OMAP4) += omap4_fb.o obj-$(CONFIG_I2C_TWL6030) += omap4_twl6030_mmc.o obj-$(CONFIG_OMAP4_USBBOOT) += omap4_rom_usb.o obj-y += gpio.o
arch/arm/mach-omap/include/mach/omap4-fb.h new file +46
-0@@ -0,0 +1,46 @@ +#ifndef H_BAREBOX_ARCH_ARM_MACH_OMAP_MACH_FB4_H +#define H_BAREBOX_ARCH_ARM_MACH_OMAP_MACH_FB4_H + +#include <fb.h> + +#define OMAP_DSS_LCD_TFT (1u << 0) +#define OMAP_DSS_LCD_IVS (1u << 1) +#define OMAP_DSS_LCD_IHS (1u << 2) +#define OMAP_DSS_LCD_IPC (1u << 3) +#define OMAP_DSS_LCD_IEO (1u << 4) +#define OMAP_DSS_LCD_RF (1u << 5) +#define OMAP_DSS_LCD_ONOFF (1u << 6) + +#define OMAP_DSS_LCD_DATALINES (_l) ((_l) << 10) +#define OMAP_DSS_LCD_DATALINES_msk OMAP_DSS_LCD_DATALINES(3u) +#define OMAP_DSS_LCD_DATALINES_12 OMAP_DSS_LCD_DATALINES(0u) +#define OMAP_DSS_LCD_DATALINES_16 OMAP_DSS_LCD_DATALINES(1u) +#define OMAP_DSS_LCD_DATALINES_18 OMAP_DSS_LCD_DATALINES(2u) +#define OMAP_DSS_LCD_DATALINES_24 OMAP_DSS_LCD_DATALINES(3u) + +struct omap4fb_display { +struct fb_videomode mode; + +unsigned long config; + +unsigned int power_on_delay; +unsigned int power_off_delay; +}; + +struct omap4fb_platform_data { +struct omap4fb_display const *displays; +size_t num_displays; + +unsigned int dss_clk_hz; + +unsigned int bpp; + +struct resource const *screen; + +void (*enable)(int p); +}; + +struct device_d ; +struct device_d *omap4_add_display (void *pdata); + +#endif /* H_BAREBOX_ARCH_ARM_MACH_OMAP_MACH_FB4_H */ arch/arm/mach-omap/omap4_fb.c new file +27
-0@@ -0,0 +1,27 @@ +#include <driver.h> +#include <common.h> +#include <linux/ioport.h> +#include <mach/omap4-fb.h> + +static struct resource omap4_fb_resources[] = { + { + .name ="omap4_dss" , + .start =0x48040000 , + .end =0x48040000 +512 -1 , + .flags =IORESOURCE_MEM |IORESOURCE_MEM_32BIT , + }, { + .name ="omap4_dispc" , + .start =0x48041000 , + .end =0x48041000 +3072 -1 , + .flags =IORESOURCE_MEM |IORESOURCE_MEM_32BIT , + }, +}; + +struct device_d *omap4_add_display (void *pdata) +{ +return add_generic_device_res ("omap4_fb" ,-1 , + omap4_fb_resources, +ARRAY_SIZE (omap4_fb_resources), + pdata); +} +EXPORT_SYMBOL (omap4_add_display);drivers/video/Kconfig +8
-0@@ -45,6 +45,14 @@ config DRIVER_VIDEO_S3C24XX help Add support for the S3C244x LCD controller. +config DRIVER_VIDEO_OMAP4 + bool "OMAP4 framebuffer driver" + depends on ARCH_OMAP4 + help + Add support for the OMAP4 Display Controller. + DSI is unsupported, only DISPC parallel mode on LCD2 + is supported. +if
DRIVER_VIDEO_S3C24XXconfig
DRIVER_VIDEO_S3C_VERBOSEdrivers/video/Makefile +0
-0@@ -8,3 +8,4 @@ obj-$(CONFIG_DRIVER_VIDEO_IMX_IPU) += imx-ipu-fb.oobj-$(CONFIG_DRIVER_VIDEO_S3C24XX) += s3c24xx.o
obj-$(CONFIG_DRIVER_VIDEO_PXA) += pxa.o
obj-$(CONFIG_DRIVER_VIDEO_SDL) += sdl.odrivers/video/Makefile +1
-0+obj-$(CONFIG_DRIVER_VIDEO_OMAP4) += omap4.odrivers/video/omap4.c new file +507
-0@@ -0,0 +1,507 @@ +/* +* TI Omap4 Frame Buffer device driver +* +* Copyright (C) 2013 Christoph Fritz <[email protected]> +* Based on work by Enrico Scholz, sponsored by Phytec +* +* See file CREDITS for list of people who contributed to this +* project. +* +* This program is free software; you can redistribute it and/or +* modify it under the terms of the GNU General Public License as +* published by the Free Software Foundation; either version 2 of +* the License, or (at your option) any later version. +* +* 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 <driver.h> +#include <fb.h> +#include <errno.h> +#include <xfuncs.h> +#include <init.h> +#include <stdio.h> +#include <io.h> +#include <common.h> +#include <malloc.h> +#include <common.h> +#include <clock.h> + +#include <mach/omap4-silicon.h> +#include <mach/omap4-fb.h> + +#include "omap4.h" + +struct omap4fb_device { +struct fb_info info; +struct device_d *dev; + +struct omap4fb_display const *cur_display; + +struct omap4fb_display const *displays; +size_t num_displays; + +struct { +struct omap4_regs_dss __iomem *dss; +struct omap4_regs_dispc __iomem *dispc; + } regs; + +struct { +void __iomem *addr; +size_t size; + } prealloc_screen; + +struct { +uint32_t dispc_control; +uint32_t dispc_pol_freq; + } shadow; + +struct { +unsigned int dss_clk_hz; +unsigned int lckd; +unsigned int pckd; + } divisor; + +void (*enable_fn)(int ); + +struct fb_videomode video_modes[]; +}; + +static inline void fb_write (uint32_t v,void __iomem *addr) +{ +__raw_writel (v, addr); +} + +static inline uint32_t fb_read (void const __iomem *addr) +{ +return __raw_readl (addr); +} + +static void omap4fb_enable (struct fb_info *info) +{ +struct omap4fb_device *fbi = + container_of(info, struct omap4fb_device, info); +struct omap4_regs_dispc __iomem *dispc = fbi->regs.dispc; + +dev_info (fbi->dev,"%s\n" , __func__); + +if (!fbi->cur_display) { +dev_err (fbi->dev,"no valid mode set\n" ); +return ; + } + +if (fbi->enable_fn) + fbi->enable_fn (1 ); + +udelay (fbi->cur_display->power_on_delay *1000u ); + +fb_write (fb_read (&dispc->control2) | +DSS_DISPC_CONTROL_LCDENABLE | +1 *DSS_DISPC_CONTROL_LCDENABLESIGNAL , + &dispc->control2); + +fb_write (fb_read (&dispc->vid1.attributes) | + DSS_DISPC_VIDn_ATTRIBUTES_VIDENABLE, + &dispc->vid1.attributes); + +fb_write (fb_read (&dispc->control2) | +DSS_DISPC_CONTROL_GOLCD , + &dispc->control2); +} + +static void omap4fb_disable (struct fb_info *info) +{ +struct omap4fb_device *fbi = + container_of(info, struct omap4fb_device, info); +struct omap4_regs_dispc __iomem *dispc = fbi->regs.dispc; + +dev_info (fbi->dev,"%s\n" , __func__); + +if (!fbi->cur_display) { +dev_err (fbi->dev,"no valid mode set\n" ); +return ; + } + +fb_write (fb_read (&dispc->control2) & + ~(DSS_DISPC_CONTROL_LCDENABLE | +DSS_DISPC_CONTROL_LCDENABLESIGNAL ), + &dispc->control2); + +fb_write (fb_read (&dispc->vid1.attributes) & + ~(DSS_DISPC_VIDn_ATTRIBUTES_VIDENABLE), + &dispc->vid1.attributes); + +if (fbi->prealloc_screen.addr ==NULL ) +/* free frame buffer; but only when screen is not +* preallocated */ +free (info->screen_base); + + info->screen_base =NULL ; + +udelay (fbi->cur_display->power_off_delay *1000u ); + +if (fbi->enable_fn) + fbi->enable_fn (0 ); +} + +static void omap4fb_calc_divisor (struct omap4fb_device *fbi, +struct fb_videomode const *mode) +{ +unsigned int l, k, t, b; + + b =UINT_MAX ; +for (l =1 ; l <256 ; l++) { +for (k =1 ; k <256 ; k++) { + t =abs (mode->pixclock *100 - + (fbi->divisor.dss_clk_hz / l / k)); +if (t <= b) { + b = t; + fbi->divisor.lckd = l; + fbi->divisor.pckd = k; + } + } + } +} + +static unsigned int omap4fb_calc_format (struct fb_info const *info) +{ +switch (info->bits_per_pixel) { +case 24 :return 9 ; +case 32 :return 0x8 ;/* xRGB24-8888 (32-bit container) */ +default : +printf ("%s: unsupported bpp %d\n" , __func__, + info->bits_per_pixel); +return 0 ; + } +} + +struct omap4fb_colors { +struct fb_bitfield red; +struct fb_bitfield green; +struct fb_bitfield blue; +struct fb_bitfield transp; +}; + +static struct omap4fb_colors const omap4fb_col[] = { + [0 ] = { + .red = { .length =0 , .offset =0 }, + }, + [1 ] = { + .blue = { .length =8 , .offset =0 }, + .green = { .length =8 , .offset =8 }, + .red = { .length =8 , .offset =16 }, + }, + [2 ] = { + .blue = { .length =8 , .offset =0 }, + .green = { .length =8 , .offset =8 }, + .red = { .length =8 , .offset =16 }, + .transp = { .length =8 , .offset =24 }, + }, +}; + +static void omap4fb_fill_shadow (struct omap4fb_device *fbi, +struct omap4fb_display const *display) +{ + fbi->shadow.dispc_control =0 ; + fbi->shadow.dispc_pol_freq =0 ; + + fbi->shadow.dispc_control |=DSS_DISPC_CONTROL_STNTFT ; + +switch (display->config & OMAP_DSS_LCD_DATALINES_msk) { +case OMAP_DSS_LCD_DATALINES_12 : + fbi->shadow.dispc_control |=DSS_DISPC_CONTROL_TFTDATALINES_12 ; +break ; +case OMAP_DSS_LCD_DATALINES_16 : + fbi->shadow.dispc_control |=DSS_DISPC_CONTROL_TFTDATALINES_16 ; +break ; +case OMAP_DSS_LCD_DATALINES_18 : + fbi->shadow.dispc_control |=DSS_DISPC_CONTROL_TFTDATALINES_18 ; +break ; +case OMAP_DSS_LCD_DATALINES_24 : + fbi->shadow.dispc_control |=DSS_DISPC_CONTROL_TFTDATALINES_24 ; +break ; + } + +if (display->config &OMAP_DSS_LCD_IPC ) + fbi->shadow.dispc_pol_freq |=DSS_DISPC_POL_FREQ_IPC ; + +if (display->config &OMAP_DSS_LCD_IVS ) + fbi->shadow.dispc_pol_freq |=DSS_DISPC_POL_FREQ_IVS ; + +if (display->config &OMAP_DSS_LCD_IHS ) + fbi->shadow.dispc_pol_freq |=DSS_DISPC_POL_FREQ_IHS ; + +if (display->config &OMAP_DSS_LCD_IEO ) + fbi->shadow.dispc_pol_freq |=DSS_DISPC_POL_FREQ_IEO ; + +if (display->config &OMAP_DSS_LCD_RF ) + fbi->shadow.dispc_pol_freq |=DSS_DISPC_POL_FREQ_RF ; + +if (display->config &OMAP_DSS_LCD_ONOFF ) + fbi->shadow.dispc_pol_freq |=DSS_DISPC_POL_FREQ_ONOFF ; +} + +static int omap4fb_activate_var (struct fb_info *info) +{ +struct omap4fb_device *fbi = + container_of(info, struct omap4fb_device, info); +struct omap4_regs_dispc __iomem *dispc = fbi->regs.dispc; +struct fb_videomode const *mode = info->mode; +size_t size = mode->xres * mode->yres * (info->bits_per_pixel /8 ); +int rc; +unsigned int fmt =omap4fb_calc_format (info); +struct omap4fb_colors const *cols; +size_t i; +struct omap4fb_display const *new_display =NULL ; + +for (i =0 ; i < fbi->num_displays && new_display ==NULL ; ++i) { +if (strcmp (mode->name, fbi->displays[i].mode.name) ==0 ) + new_display = &fbi->displays[i]; + } + +if (WARN_ON (!new_display)) { +dev_err (fbi->dev,"no matching display found for this mode '%s'\n" , + mode->name); + rc = -ENXIO ; + goto out; + } + +if (fbi->prealloc_screen.addr ==NULL ) { +/* case 1: no preallocated screen */ +free (info->screen_base); + info->screen_base =memalign (0x100 , size); + }else if (fbi->prealloc_screen.size < size) { +/* case 2: preallocated screen, but too small */ +dev_err (fbi->dev, +"allocated framebuffer too small (%zu < %zu)\n" , + fbi->prealloc_screen.size, size); + rc = -ENOMEM ; + goto out; + }else { +/* case 3: preallocated screen */ + info->screen_base = fbi->prealloc_screen.addr; + } + +omap4fb_fill_shadow (fbi, new_display); + +omap4fb_calc_divisor (fbi, mode); + +switch (info->bits_per_pixel) { +case 24 : + cols = &omap4fb_col[1 ]; +break ; +case 32 : + cols = &omap4fb_col[2 ]; +break ; +default : + cols = &omap4fb_col[0 ]; + } + + info->red = cols->red; + info->green = cols->green; + info->blue = cols->blue; + info->transp = cols->transp; + +fb_write (fbi->shadow.dispc_control, + &dispc->control2); + +fb_write (fbi->shadow.dispc_pol_freq, + &dispc->pol_freq2); + +fb_write (DSS_DISPC_TIMING_H_HSW (mode->hsync_len -1 ) | +DSS_DISPC_TIMING_H_HFP (mode->right_margin -1 ) | +DSS_DISPC_TIMING_H_HBP (mode->left_margin -1 ), + &dispc->timing_h2); + +fb_write (DSS_DISPC_TIMING_V_VSW (mode->vsync_len -1 ) | +DSS_DISPC_TIMING_V_VFP (mode->lower_margin) | +DSS_DISPC_TIMING_V_VBP (mode->upper_margin), + &dispc->timing_v2); + +fb_write (DSS_DISPC_DIVISOR_ENABLE |DSS_DISPC_DIVISOR_LCD (1 ), + &dispc->divisor); + +fb_write (DSS_DISPC_DIVISOR2_LCD (fbi->divisor.lckd) | +DSS_DISPC_DIVISOR2_PCD (fbi->divisor.pckd), + &dispc->divisor2); + +fb_write (DSS_DISPC_SIZE_LCD_PPL (mode->xres -1 ) | +DSS_DISPC_SIZE_LCD_LPP (mode->yres -1 ), + &dispc->size_lcd2); + +fb_write (0x0000FF00 , &dispc->default_color2); + +/* we use VID1 */ +fb_write ((uintptr_t )info->screen_base, &dispc->vid1.ba[0 ]); +fb_write ((uintptr_t )info->screen_base, &dispc->vid1.ba[1 ]); + +fb_write (DSS_DISPC_VIDn_POSITION_VIDPOSX (0 ) | +DSS_DISPC_VIDn_POSITION_VIDPOSY (0 ), + &dispc->vid1.position); +fb_write (DSS_DISPC_VIDn_SIZE_VIDSIZEX (mode->xres -1 ) | +DSS_DISPC_VIDn_SIZE_VIDSIZEY (mode->yres -1 ), + &dispc->vid1.size); +fb_write (DSS_DISPC_VIDn_PICTURE_SIZE_VIDORGSIZEX (mode->xres -1 ) | +DSS_DISPC_VIDn_PICTURE_SIZE_VIDORGSIZEY (mode->yres -1 ), + &dispc->vid1.picture_size); +fb_write (1 , &dispc->vid1.row_inc); +fb_write (1 , &dispc->vid1.pixel_inc); + +fb_write (0xfff , &dispc->vid_preload[0 ]); + +fb_write (DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT (fmt) | + DSS_DISPC_VIDn_ATTRIBUTES_VIDBURSTSIZE_8x128 | + DSS_DISPC_VIDn_ATTRIBUTES_ZORDERENABLE | + DSS_DISPC_VIDn_ATTRIBUTES_CHANNELOUT2_SECONDARY_LCD, + &dispc->vid1.attributes); + +while (fb_read (&dispc->control2) &DSS_DISPC_CONTROL_GOLCD ) + ;/* noop */ + +fb_write (fb_read (&dispc->control2) | +DSS_DISPC_CONTROL_GOLCD , + &dispc->control2); + + fbi->cur_display = new_display; + info->xres = mode->xres; + info->yres = mode->yres; + + rc =0 ; + +out: +return rc; +} + +static void omap4fb_reset (struct omap4fb_device const *fbi) +{ +struct omap4_regs_dispc __iomem *dispc = fbi->regs.dispc; +struct omap4_regs_dss __iomem *dss = fbi->regs.dss; + +uint32_t v =fb_read (&dispc->control2); + +/* step 1: stop the LCD controller */ +if (v &DSS_DISPC_CONTROL_LCDENABLE ) { +fb_write (v & ~DSS_DISPC_CONTROL_LCDENABLE , + &dispc->control2); + +fb_write (DSS_DISPC_IRQSTATUS_FRAMEDONE2 , &dispc->irqstatus); + +while ((fb_read (&dispc->irqstatus) & +DSS_DISPC_IRQSTATUS_FRAMEDONE ) ==0 ) + ;/* noop */ + } + +/* step 2: wait for reset done status */ +while (!(fb_read (&dss->sysstatus) &DSS_DSS_SYSSTATUS_RESETDONE )) + ;/* noop */ + +/* DSS_CTL: set to reset value */ +fb_write (0 , &dss->ctrl); +} + +static struct fb_ops omap4fb_ops = { + .fb_enable = omap4fb_enable, + .fb_disable = omap4fb_disable, + .fb_activate_var = omap4fb_activate_var, +}; + +static int omap4fb_probe (struct device_d *dev) +{ +struct omap4fb_platform_data const *pdata = dev->platform_data; +struct omap4fb_device *fbi; +struct fb_info *info; +struct omap4_regs_dispc __iomem *dispc; +struct omap4_regs_dss __iomem *dss; +int rc; +size_t i; + +if (!pdata) +return -ENODEV ; + + fbi =xzalloc (sizeof *fbi + + pdata->num_displays *sizeof fbi->video_modes[0 ]); + info = &fbi->info; + + fbi->dev = dev; + +/* CM_DSS_CLKSTCTRL (TRM: 935) trigger SW_WKUP */ +__raw_writel (0x2 ,0x4a009100 );/* TODO: move this to clockmanagement */ + + fbi->regs.dss =dev_request_mem_region_by_name (dev,"omap4_dss" ); + fbi->regs.dispc =dev_request_mem_region_by_name (dev,"omap4_dispc" ); + +if (!fbi->regs.dss || !fbi->regs.dispc) { +dev_err (dev,"Insufficient register description\n" ); + rc = -EINVAL ; + goto out; + } + + dispc = fbi->regs.dispc; + dss = fbi->regs.dss; +dev_info (dev,"HW-Revision 0x%04x 0x%04x\n" , +fb_read (&dispc->revision), +fb_read (&dss->revision)); + + +if (!pdata->dss_clk_hz | !pdata->displays | !pdata->num_displays | + !pdata->bpp) { +dev_err (dev,"Insufficient omap4fb_platform_data\n" ); + rc = -EINVAL ; + goto out; + } + + fbi->enable_fn = pdata->enable; + fbi->displays = pdata->displays; + fbi->num_displays = pdata->num_displays; + fbi->divisor.dss_clk_hz = pdata->dss_clk_hz; + +for (i =0 ; i < pdata->num_displays; ++i) + fbi->video_modes[i] = pdata->displays[i].mode; + + info->mode_list = fbi->video_modes; + info->num_modes = pdata->num_displays; + + info->priv = fbi; + info->fbops = &omap4fb_ops; + info->bits_per_pixel = pdata->bpp; + +if (pdata->screen) { + fbi->prealloc_screen.addr = + (void __iomem *)pdata->screen->start; + fbi->prealloc_screen.size =resource_size (pdata->screen); + } + +omap4fb_reset (fbi); + +dev_add_param (fbi->dev,"bootargs" ,NULL ,NULL ,0 ); + + rc =register_framebuffer (info); +if (rc <0 ) { +dev_err (dev,"failed to register framebuffer: %d\n" , rc); + goto out; + } + + rc =0 ; +dev_info (dev,"registered\n" ); + +out: +if (rc <0 ) +free (fbi); + +return rc; +} + +static struct driver_d omap4fb_driver = { + .name ="omap4_fb" , + .probe = omap4fb_probe, +}; + +static int omap4fb_init (void ) +{ +return platform_driver_register (&omap4fb_driver); +} + +device_initcall (omap4fb_init);drivers/video/omap4.h new file +408
-0@@ -0,0 +1,408 @@ +/* +* TI Omap4 Frame Buffer device driver +* +* Copyright (C) 2013 Christoph Fritz <[email protected]> +* +* See file CREDITS for list of people who contributed to this +* project. +* +* This program is free software; you can redistribute it and/or +* modify it under the terms of the GNU General Public License as +* published by the Free Software Foundation; either version 2 of +* the License, or (at your option) any later version. +* +* 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 H_BAREBOX_DRIVER_VIDEO_OMAP4_REGS_H +#define H_BAREBOX_DRIVER_VIDEO_OMAP4_REGS_H + +#include <types.h> +#include <common.h> + +#define OMAP4_RESERVED (_f, _t) \ + uint32_t const reserved_ ## _f ## _ ## _t[((_t) - (_f)) / 4] + + +/* TRM: 10.1.3.2 DSS Registers */ +struct omap4_regs_dss { +uint32_t revision; + OMAP4_RESERVED(0x4, 0x14); +uint32_t const sysstatus; + OMAP4_RESERVED(0x18, 0x40); +uint32_t ctrl; + OMAP4_RESERVED(0x44, 0x5C); +uint32_t const status; +}; + + +struct omap4_regs_dispc_vid { +uint32_t ba[2 ]; +uint32_t position; +uint32_t size; +uint32_t attributes; +uint32_t buf_threshold; +uint32_t const buf_size_status; +uint32_t row_inc; +uint32_t pixel_inc; +uint32_t fir; +uint32_t picture_size; +uint32_t accu[2 ]; +struct { +uint32_t h; +uint32_t hv; + } fir_coef[8 ]; +uint32_t conv_coef0; +uint32_t conv_coef1; +uint32_t conv_coef2; +uint32_t conv_coef3; +uint32_t conv_coef4; +}; + +/* TRM: 10.2.7.3 Display Controller Registers */ +struct omap4_regs_dispc { +uint32_t const revision; + OMAP4_RESERVED(0x04, 0x10); +uint32_t sysconfig; +uint32_t const sysstatus; +uint32_t irqstatus; +uint32_t irqenable; + OMAP4_RESERVED(0x20, 0x40); +uint32_t control1; +uint32_t config1; + OMAP4_RESERVED(0x48, 0x4C); + +uint32_t default_color[2 ]; +uint32_t trans_color[2 ]; + +uint32_t const line_status; +uint32_t line_number; +uint32_t timing_h1; +uint32_t timing_v1; +uint32_t pol_freq1; +uint32_t divisor1; +uint32_t global_alpha; +uint32_t size_tv; +uint32_t size_lcd1; + +struct { +uint32_t ba[2 ]; +uint32_t position; +uint32_t size; + OMAP4_RESERVED(0x90, 0xA0); +uint32_t attributes; +uint32_t buf_threshold; +uint32_t const buf_size_status; +uint32_t row_inc; +uint32_t pixel_inc; + OMAP4_RESERVED(0xB4, 0xB8); +uint32_t table_ba; + } gfx; + +struct omap4_regs_dispc_vid vid1; + OMAP4_RESERVED(0x144, 0x14C); +struct omap4_regs_dispc_vid vid2; + +uint32_t data1_cycle[3 ]; + +uint32_t vid1_fir_coef_v[8 ]; +uint32_t vid2_fir_coef_v[8 ]; +uint32_t cpr1_coef_r; +uint32_t cpr1_coef_g; +uint32_t cpr1_coef_b; +uint32_t gfx_preload; +uint32_t vid_preload[2 ]; +uint32_t control2; + OMAP4_RESERVED(0x23C, 0x300); + +struct { +uint32_t accu[2 ]; +uint32_t ba[2 ]; +struct { +uint32_t h; +uint32_t hv; + } fir_coef[8 ]; +uint32_t fir_coef_v[8 ]; +uint32_t attributes; +uint32_t conv_coef0; +uint32_t conv_coef1; +uint32_t conv_coef2; +uint32_t conv_coef3; +uint32_t conv_coef4; +uint32_t const buf_size_status; +uint32_t buf_threshold; +uint32_t fir; +uint32_t picture_size; +uint32_t pixel_inc; +uint32_t position; +uint32_t preload; +uint32_t row_inc; +uint32_t size; + } vid3; + +uint32_t default_color2; +uint32_t trans_color2; +uint32_t cpr2_coef_b; +uint32_t cpr2_coef_g; +uint32_t cpr2_coef_r; +uint32_t data2_cycle[3 ]; +uint32_t size_lcd2; + OMAP4_RESERVED(0x3D0, 0x400); +uint32_t timing_h2; +uint32_t timing_v2; +uint32_t pol_freq2; +uint32_t divisor2; + OMAP4_RESERVED(0x410, 0x500); + +struct { +uint32_t accu[2 ]; +uint32_t ba[2 ]; +struct { +uint32_t h; +uint32_t hv; + } fir_coef[8 ]; +uint32_t fir_coef_v[8 ]; +uint32_t attributes; +uint32_t conv_coef0; +uint32_t conv_coef1; +uint32_t conv_coef2; +uint32_t conv_coef3; +uint32_t conv_coef4; +uint32_t const buf_size_status; +uint32_t buf_threshold; +uint32_t fir; +uint32_t picture_size; +uint32_t pixel_inc; + OMAP4_RESERVED(0x59C, 0x5A4); +uint32_t row_inc; +uint32_t size; + } wb; + + OMAP4_RESERVED(0x5AC, 0x600); +uint32_t vid1_ba_uv[2 ]; +uint32_t vid2_ba_uv[2 ]; +uint32_t vid3_ba_uv[2 ]; +uint32_t wb_ba_uv[2 ]; +uint32_t config2; +uint32_t vid1_attributes2; +uint32_t vid2_attributes2; +uint32_t vid3_attributes2; +uint32_t gamma_table0; +uint32_t gamma_table1; +uint32_t gamma_table2; +uint32_t vid1_fir2; +uint32_t vid1_accu2[2 ]; +struct { +uint32_t h2; +uint32_t hv2; + } vid1_fir_coef[8 ]; +uint32_t vid1_fir_coef_v2[8 ]; +uint32_t vid2_fir2; +uint32_t vid2_accu2[2 ]; +struct { +uint32_t h2; +uint32_t hv2; + } vid2_fir_coef[8 ]; +uint32_t vid2_fir_coef_v2[8 ]; + OMAP4_RESERVED(0x714, 0x724); +uint32_t vid3_fir2; +uint32_t vid3_accu2[2 ]; +struct { +uint32_t h2; +uint32_t hv2; + } vid3_fir_coef[8 ]; +uint32_t vid3_fir_coef_v2[8 ]; +uint32_t wb_fir2; +uint32_t wb_accu2[2 ]; + OMAP4_RESERVED(0x79C, 0x7A0); +struct { +uint32_t h2; +uint32_t hv2; + } wb_fir_coef[8 ]; +uint32_t wb_fir_coef_v2[8 ]; +uint32_t global_buffer; +uint32_t divisor; + OMAP4_RESERVED(0x808, 0x810); +uint32_t wb_attributes2; +}; + +#define DSS_DISPC_VIDn_POSITION_VIDPOSX (_x) ((_x) << 0) +#define DSS_DISPC_VIDn_POSITION_VIDPOSY (_y) ((_y) << 16) + +#define DSS_DISPC_VIDn_PICTURE_SIZE_VIDORGSIZEX (_x) ((_x) << 0) +#define DSS_DISPC_VIDn_PICTURE_SIZE_VIDORGSIZEY (_y) ((_y) << 16) + +#define DSS_DISPC_VIDn_SIZE_VIDSIZEX (_x) ((_x) << 0) +#define DSS_DISPC_VIDn_SIZE_VIDSIZEY (_y) ((_y) << 16) + +#define DSS_DISPC_SIZE_LCD_PPL (_x) ((_x) << 0) +#define DSS_DISPC_SIZE_LCD_LPP (_y) ((_y) << 16) + +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDENABLE (1u << 0) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT (_fmt) ((_fmt) << 1) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT_RGB12 \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT(4u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT_ARGB16 \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT(5u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT_RGB16 \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT(6u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT_ARGB16o \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT(7u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT_xRGB24u \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT(8u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT_RGB24p \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT(9u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT_YUV2 \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT(10u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT_UYVY \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT(11u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT_ARGB32 \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT(12u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT_RGBA32 \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT(13u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT_xRGB32 \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT(14u) + +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDRESIZEENABLE (_f) ((_f) << 5) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDRESIZEENABLE_NONE \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDRESIZEENABLE(0u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDRESIZEENABLE_H \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDRESIZEENABLE(1u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDRESIZEENABLE_V \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDRESIZEENABLE(2u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDRESIZEENABLE_HV \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDRESIZEENABLE(3u) + +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDCOLORCONVENABLE (1u << 9) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDREPLICATIONENABLE (1u << 10) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDFULLRANGE (1u << 11) + +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDROTATION (_r) ((_r) << 12) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDROTATION_0 \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDROTATION(0u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDROTATION_90 \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDROTATION(1u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDROTATION_180 \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDROTATION(2u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDROTATION_270 \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDROTATION(3u) + +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDBURSTSIZE (_b) ((_b) << 14) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDBURSTSIZE_2x128 \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDBURSTSIZE(0u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDBURSTSIZE_4x128 \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDBURSTSIZE(1u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDBURSTSIZE_8x128 \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDBURSTSIZE(2u) + +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDCHANNELOUT (1u << 16) +#define DSS_DISPC_VIDn_ATTRIBUTES_SELFREFRESHAUTO (1u << 17) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDFIFOPRELOAD (1u << 19) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDVERTICALTAPS (1u << 21) +#define DSS_DISPC_VIDn_ATTRIBUTES_DOUBLESTRIDE (1u << 22) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDARBITRATION (1u << 23) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDSELFREFRESH (1u << 24) +#define DSS_DISPC_VIDn_ATTRIBUTES_ZORDERENABLE (1u << 25) + +#define DSS_DISPC_VIDn_ATTRIBUTES_CHANNELOUT2 (_b) ((_b) << 30) +#define DSS_DISPC_VIDn_ATTRIBUTES_CHANNELOUT2_PRIMARY_LCD \ + DSS_DISPC_VIDn_ATTRIBUTES_CHANNELOUT2(0u) +#define DSS_DISPC_VIDn_ATTRIBUTES_CHANNELOUT2_SECONDARY_LCD \ + DSS_DISPC_VIDn_ATTRIBUTES_CHANNELOUT2(1u) +#define DSS_DISPC_VIDn_ATTRIBUTES_CHANNELOUT2_WRITEBACK_MEM \ + DSS_DISPC_VIDn_ATTRIBUTES_CHANNELOUT2(3u) + +#define DSS_DISPC_CONFIG_PIXELGATED (1u << 0) +#define DSS_DISPC_CONFIG_PIXELDATAGATED (1u << 4) +#define DSS_DISPC_CONFIG_PIXELCLOCKGATED (1u << 5) +#define DSS_DISPC_CONFIG_HSYNCGATED (1u << 6) +#define DSS_DISPC_CONFIG_VSYNCGATED (1u << 7) +#define DSS_DISPC_CONFIG_ACBIASGATED (1u << 8) +#define DSS_DISPC_CONFIG_FUNCGATED (1u << 9) +#define DSS_DISPC_CONFIG_TCKLCDENABLE (1u << 10) +#define DSS_DISPC_CONFIG_TCKLCDSELECTION (1u << 11) +#define DSS_DISPC_CONFIG_CPR (1u << 15) +#define DSS_DISPC_CONFIG_FIFOHANDCHECK (1u << 16) +#define DSS_DISPC_CONFIG_OUTPUTMODE (1u << 22) +#define DSS_DISPC_CONFIG_FIDFIRST (1u << 23) +#define DSS_DISPC_CONFIG_COLORCONV (1u << 24) +#define DSS_DISPC_CONFIG_FULLRANGE (1u << 25) + +#define DSS_DISPC_CONTROL_LCDENABLE (1u << 0) +#define DSS_DISPC_CONTROL_TVENABLE (1u << 1) +#define DSS_DISPC_CONTROL_MONOCOLOR (1u << 2) +#define DSS_DISPC_CONTROL_STNTFT (1u << 3) +#define DSS_DISPC_CONTROL_M8B (1u << 4) +#define DSS_DISPC_CONTROL_GOLCD (1u << 5) +#define DSS_DISPC_CONTROL_GOTV (1u << 6) +#define DSS_DISPC_CONTROL_STDITHERENABLE (1u << 7) + +#define DSS_DISPC_CONTROL_TFTDATALINES (_l) ((_l) << 8) +#define DSS_DISPC_CONTROL_TFTDATALINES_12 \ + DSS_DISPC_CONTROL_TFTDATALINES(0u) +#define DSS_DISPC_CONTROL_TFTDATALINES_16 \ + DSS_DISPC_CONTROL_TFTDATALINES(1u) +#define DSS_DISPC_CONTROL_TFTDATALINES_18 \ + DSS_DISPC_CONTROL_TFTDATALINES(2u) +#define DSS_DISPC_CONTROL_TFTDATALINES_24 \ + DSS_DISPC_CONTROL_TFTDATALINES(3u) + +#define DSS_DISPC_CONTROL_STALLMODE (1u << 11) +#define DSS_DISPC_CONTROL_OVERLAYOPTIMIZATION (1u << 12) +#define DSS_DISPC_CONTROL_GPIN0 (1u << 13)/* ro */ +#define DSS_DISPC_CONTROL_GPIN1 (1u << 14)/* ro */ +#define DSS_DISPC_CONTROL_GPOUT0 (1u << 15) +#define DSS_DISPC_CONTROL_GPOUT1 (1u << 16) +#define DSS_DISPC_CONTROL_HT (_ht) ((_ht) << 17) +#define DSS_DISPC_CONTROL_TDMENABLE (1u << 20) +#define DSS_DISPC_CONTROL_TDMPARALLELMODE (_pm) ((_pm) << 21) +#define DSS_DISPC_CONTROL_TDMCYCLEFORMAT (_cf) ((_cf) << 23) +#define DSS_DISPC_CONTROL_TDMUNUSEDBITS (_ub) ((_ub) << 25) +#define DSS_DISPC_CONTROL_PCKFREEENABLE (1u << 27) +#define DSS_DISPC_CONTROL_LCDENABLESIGNAL (1u << 28) +#define DSS_DISPC_CONTROL_LCDENABLEPOL (1u << 29) +#define DSS_DISPC_CONTROL_SPATIALTEMPD (_df) ((_df) << 30) + +#define DSS_DISPC_POL_FREQ_IVS (1u << 12) +#define DSS_DISPC_POL_FREQ_IHS (1u << 13) +#define DSS_DISPC_POL_FREQ_IPC (1u << 14) +#define DSS_DISPC_POL_FREQ_IEO (1u << 15) +#define DSS_DISPC_POL_FREQ_RF (1u << 16) +#define DSS_DISPC_POL_FREQ_ONOFF (1u << 17) + +#define DSS_DISPC_TIMING_H_HSW (_hsw) ((_hsw) << 0) +#define DSS_DISPC_TIMING_H_HFP (_hfp) ((_hfp) << 8) +#define DSS_DISPC_TIMING_H_HBP (_hbp) ((_hbp) << 20) + +#define DSS_DISPC_TIMING_V_VSW (_vsw) ((_vsw) << 0) +#define DSS_DISPC_TIMING_V_VFP (_vfp) ((_vfp) << 8) +#define DSS_DISPC_TIMING_V_VBP (_vbp) ((_vbp) << 20) + +#define DSS_DISPC_DIVISOR_ENABLE (1u << 0) +#define DSS_DISPC_DIVISOR_LCD (_lcd) ((_lcd) << 16) + +#define DSS_DISPC_DIVISOR2_PCD (_pcd) ((_pcd) << 0) +#define DSS_DISPC_DIVISOR2_LCD (_lcd) ((_lcd) << 16) + +#define DSS_DISPC_IRQSTATUS_FRAMEDONE (1u << 0) +#define DSS_DISPC_IRQSTATUS_FRAMEDONE2 (1u << 22) + +#define DSS_DSS_SYSCONFIG_SOFTRESET (1u << 1) +#define DSS_DSS_SYSSTATUS_RESETDONE (1u << 0) + +#define DSS_DSS_CONTROL_DISPC_CLK_SWITCH (1u << 0) +#define DSS_DSS_CONTROL_DSI_CLK_SWITCH (1u << 1) + +#define CM_FCLKEN_DSS_EN_DSS1 (1u << 0) +#define CM_FCLKEN_DSS_EN_DSS2 (1u << 1) +#define CM_FCLKEN_DSS_EN_TV (1u << 2) + +#define CM_ICLKEN_DSS_EN_DSS (1u << 0) + + +#undef OMAP4_RESERVED + +#endif /* H_BAREBOX_DRIVER_VIDEO_OMAP4_REGS_H */ Signature · 1 line
1.7.10.4
_______________________________________________
barebox mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/barebox
PATCH v4 1/1 omap4-fb: add driver
This patch adds omap4 display controller support.
---
changes since v2:
- use dev_request_mem_region_by_name()
changes since v3:
- remove register struct
- use uncached screen_base
--- arch/arm/mach-omap/Makefile | 1 + arch/arm/mach-omap/include/mach/omap4-fb.h | 46 +++ arch/arm/mach-omap/omap4_fb.c | 27 ++ drivers/video/Kconfig | 8 + drivers/video/Makefile | 1 + drivers/video/omap4.c | 505 ++++++++++++++++++++++++++++ drivers/video/omap4.h | 179 ++++++++++ 7 files changed, 767 insertions(+) create mode 100644 arch/arm/mach-omap/include/mach/omap4-fb.h create mode 100644 arch/arm/mach-omap/omap4_fb.c create mode 100644 drivers/video/omap4.c create mode 100644 drivers/video/omap4.h
arch/arm/mach-omap/Makefile +1 -0
@@ -28,6 +28,7 @@ obj-$(CONFIG_OMAP3_CLOCK_CONFIG) += omap3_clock.o pbl-$(CONFIG_OMAP3_CLOCK_CONFIG) += omap3_clock.o obj-$(CONFIG_OMAP_GPMC) += gpmc.o devices-gpmc-nand.o obj-$(CONFIG_SHELL_NONE) += xload.o +obj-$(CONFIG_DRIVER_VIDEO_OMAP4) += omap4_fb.o obj-$(CONFIG_I2C_TWL6030) += omap4_twl6030_mmc.o obj-$(CONFIG_OMAP4_USBBOOT) += omap4_rom_usb.o obj-y += gpio.o
arch/arm/mach-omap/include/mach/omap4-fb.h new file +46 -0
46 lines not on this page · raw message
arch/arm/mach-omap/omap4_fb.c new file +27 -0
@@ -0,0 +1,27 @@ +#include <driver.h> +#include <common.h> +#include <linux/ioport.h> +#include <mach/omap4-fb.h> + +static struct resource omap4_fb_resources[] = { + { + .name ="omap4_dss" , + .start =0x48040000 , + .end =0x48040000 +512 -1 , + .flags =IORESOURCE_MEM |IORESOURCE_MEM_32BIT , + }, { + .name ="omap4_dispc" , + .start =0x48041000 , + .end =0x48041000 +3072 -1 , + .flags =IORESOURCE_MEM |IORESOURCE_MEM_32BIT , + }, +}; + +struct device_d *omap4_add_display (void *pdata) +{ +return add_generic_device_res ("omap4_fb" ,-1 , + omap4_fb_resources, +ARRAY_SIZE (omap4_fb_resources), + pdata); +} +EXPORT_SYMBOL (omap4_add_display);
drivers/video/Kconfig +8 -0
@@ -45,6 +45,14 @@ config DRIVER_VIDEO_S3C24XX help Add support for the S3C244x LCD controller. +config DRIVER_VIDEO_OMAP4 + bool "OMAP4 framebuffer driver" + depends on ARCH_OMAP4 + help + Add support for the OMAP4 Display Controller. + DSI is unsupported, only DISPC parallel mode on LCD2 + is supported. + if DRIVER_VIDEO_S3C24XX config DRIVER_VIDEO_S3C_VERBOSE
drivers/video/Makefile +1 -0
@@ -8,3 +8,4 @@ obj-$(CONFIG_DRIVER_VIDEO_IMX_IPU) += imx-ipu-fb.o obj-$(CONFIG_DRIVER_VIDEO_S3C24XX) += s3c24xx.o obj-$(CONFIG_DRIVER_VIDEO_PXA) += pxa.o obj-$(CONFIG_DRIVER_VIDEO_SDL) += sdl.o +obj-$(CONFIG_DRIVER_VIDEO_OMAP4) += omap4.o
drivers/video/omap4.c new file +505 -0
505 lines not on this page · raw message
drivers/video/omap4.h new file +179 -0
179 lines not on this page · raw message
Signature · 9 lines
1.7.10.4 _______________________________________________ barebox mailing list [email protected] http://lists.infradead.org/mailman/listinfo/barebox
Hi,
Quoted 41 lines · Christoph Fritz
This patch adds omap4 display controller support.
Signed-off-by Christoph Fritz <[email protected]>---
changes since v2:
- usedev_request_mem_region_by_name()
changes since v3:
- remove register struct
- use uncachedscreen_base--- arch/arm/mach-omap/Makefile | 1 + arch/arm/mach-omap/include/mach/omap4-fb.h | 46 +++ arch/arm/mach-omap/omap4_fb.c | 27 ++ drivers/video/Kconfig | 8 + drivers/video/Makefile | 1 + drivers/video/omap4.c | 505 ++++++++++++++++++++++++++++ drivers/video/omap4.h | 179 ++++++++++ 7 files changed, 767 insertions(+) create mode 100644 arch/arm/mach-omap/include/mach/omap4-fb.h create mode 100644 arch/arm/mach-omap/omap4_fb.c create mode 100644 drivers/video/omap4.c create mode 100644 drivers/video/omap4.harch/arm/mach-omap/Makefile +1
-0@@ -28,6 +28,7 @@ obj-$(CONFIG_OMAP3_CLOCK_CONFIG) += omap3_clock.o pbl-$(CONFIG_OMAP3_CLOCK_CONFIG) += omap3_clock.o obj-$(CONFIG_OMAP_GPMC) += gpmc.o devices-gpmc-nand.o obj-$(CONFIG_SHELL_NONE) += xload.o +obj-$(CONFIG_DRIVER_VIDEO_OMAP4) += omap4_fb.o obj-$(CONFIG_I2C_TWL6030) += omap4_twl6030_mmc.o obj-$(CONFIG_OMAP4_USBBOOT) += omap4_rom_usb.o obj-y += gpio.oarch/arm/mach-omap/include/mach/omap4-fb.h new file +0
-0@@ -0,0 +1,46 @@
...
arch/arm/mach-omap/include/mach/omap4-fb.h
Quoted 12 lines
arch/arm/mach-omap/include/mach/omap4-fb.h new file +12
-0+fb_write (1 , fbi->dispc,O4_DISPC_VID1_PIXEL_INC ); + +fb_write (0xfff , fbi->dispc,O4_DISPC_VID1_PRELOAD ); + +fb_write (DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT (fmt) | + DSS_DISPC_VIDn_ATTRIBUTES_VIDBURSTSIZE_8x128 | + DSS_DISPC_VIDn_ATTRIBUTES_ZORDERENABLE | + DSS_DISPC_VIDn_ATTRIBUTES_CHANNELOUT2_SECONDARY_LCD, + fbi->dispc,O4_DISPC_VID1_ATTRIBUTES ); + +while (fb_read (fbi->dispc,O4_DISPC_CONTROL2 ) &DSS_DISPC_CONTROL_GOLCD ) + ;/* noop */
For busy waiting on statusbits I would prefer a mechanism like this:
start = get_time_ns();
while (condition) {
...
if (is_timeout(start, waiting_time)) {
dev_err(dev, "foo"
return -ETIMEOUT;
}
mdelay(interval);...
}
I mean it avoid deadlocks in barebox and it's better to start linux without a display instead of a deadlock in barebox.
On the other hand you need a waiting_time which works with all displays together. Don't know if there any hardware dependency.
}
Quoted 14 lines
arch/arm/mach-omap/include/mach/omap4-fb.h new file +14
-0+ +fb_write (fb_read (fbi->dispc,O4_DISPC_CONTROL2 ) | +DSS_DISPC_CONTROL_GOLCD , + fbi->dispc,O4_DISPC_CONTROL2 ); + + fbi->cur_display = new_display; + info->xres = mode->xres; + info->yres = mode->yres; + + rc =0 ; + +out: +return rc; +}
...
arch/arm/mach-omap/include/mach/omap4-fb.h
Quoted 5 lines
arch/arm/mach-omap/include/mach/omap4-fb.h new file +5
-0+ +omap4fb_reset (fbi); + +dev_add_param (fbi->dev,"bootargs" ,NULL ,NULL ,0 ); +
Jean-Christophe PLAGNIOL-VILLARD said why we need this here in the last mails. I don't saw any answer to this and asking myself... what do this function?
Regards
Alex
_______________________________________________
barebox mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/barebox
Am Freitag, den 14.06.2013, 08:29 +0200 schrieb Alexander Aring:
Quoted 65 lines
Hi,
Quoted 41 lines · Christoph Fritz
This patch adds omap4 display controller support.
Signed-off-by Christoph Fritz <[email protected]>---
changes since v2:
- usedev_request_mem_region_by_name()
changes since v3:
- remove register struct
- use uncachedscreen_base--- arch/arm/mach-omap/Makefile | 1 + arch/arm/mach-omap/include/mach/omap4-fb.h | 46 +++ arch/arm/mach-omap/omap4_fb.c | 27 ++ drivers/video/Kconfig | 8 + drivers/video/Makefile | 1 + drivers/video/omap4.c | 505 ++++++++++++++++++++++++++++ drivers/video/omap4.h | 179 ++++++++++ 7 files changed, 767 insertions(+) create mode 100644 arch/arm/mach-omap/include/mach/omap4-fb.h create mode 100644 arch/arm/mach-omap/omap4_fb.c create mode 100644 drivers/video/omap4.c create mode 100644 drivers/video/omap4.harch/arm/mach-omap/Makefile +1
-0@@ -28,6 +28,7 @@ obj-$(CONFIG_OMAP3_CLOCK_CONFIG) += omap3_clock.o pbl-$(CONFIG_OMAP3_CLOCK_CONFIG) += omap3_clock.o obj-$(CONFIG_OMAP_GPMC) += gpmc.o devices-gpmc-nand.o obj-$(CONFIG_SHELL_NONE) += xload.o +obj-$(CONFIG_DRIVER_VIDEO_OMAP4) += omap4_fb.o obj-$(CONFIG_I2C_TWL6030) += omap4_twl6030_mmc.o obj-$(CONFIG_OMAP4_USBBOOT) += omap4_rom_usb.o obj-y += gpio.oarch/arm/mach-omap/include/mach/omap4-fb.h new file +0
-0@@ -0,0 +1,46 @@...
arch/arm/mach-omap/include/mach/omap4-fb.h
Quoted 12 lines
arch/arm/mach-omap/include/mach/omap4-fb.h new file +12
-0+fb_write (1 , fbi->dispc,O4_DISPC_VID1_PIXEL_INC ); + +fb_write (0xfff , fbi->dispc,O4_DISPC_VID1_PRELOAD ); + +fb_write (DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT (fmt) | + DSS_DISPC_VIDn_ATTRIBUTES_VIDBURSTSIZE_8x128 | + DSS_DISPC_VIDn_ATTRIBUTES_ZORDERENABLE | + DSS_DISPC_VIDn_ATTRIBUTES_CHANNELOUT2_SECONDARY_LCD, + fbi->dispc,O4_DISPC_VID1_ATTRIBUTES ); + +while (fb_read (fbi->dispc,O4_DISPC_CONTROL2 ) &DSS_DISPC_CONTROL_GOLCD ) + ;/* noop */ For busy waiting on statusbits I would prefer a mechanism like this:
start =
get_time_ns();
while (condition) {
...
Quoted 7 lines
if (is_timeout(start, waiting_time)) { dev_err(dev, "foo" return -ETIMEOUT; } mdelay(interval);...
}
There is also a convenience wrapper: wait_on_timeout(timeout, condition)
Jan
Quoted 33 lines
I mean it avoid deadlocks in barebox and it's better to start linux without a display instead of a deadlock in barebox.
On the other hand you need awaiting_timewhich works with all displays together. Don't know if there any hardware dependency.}
Quoted 14 lines
arch/arm/mach-omap/include/mach/omap4-fb.h new file +14
-0+ +fb_write (fb_read (fbi->dispc,O4_DISPC_CONTROL2 ) | +DSS_DISPC_CONTROL_GOLCD , + fbi->dispc,O4_DISPC_CONTROL2 ); + + fbi->cur_display = new_display; + info->xres = mode->xres; + info->yres = mode->yres; + + rc =0 ; + +out: +return rc; +}...
arch/arm/mach-omap/include/mach/omap4-fb.h
Quoted 5 lines
arch/arm/mach-omap/include/mach/omap4-fb.h new file +5
-0+ +omap4fb_reset (fbi); + +dev_add_param (fbi->dev,"bootargs" ,NULL ,NULL ,0 ); +Jean-Christophe PLAGNIOL-VILLARD said why we need this here in the last mails. I don't saw any answer to this and asking myself... what do this function?
Quoted 7 lines
Regards
Alex_______________________________________________
barebox mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/barebox
_______________________________________________
barebox mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/barebox
This patch adds omap4 display controller support.
---
changes since v2:
- use dev_request_mem_region_by_name()
changes since v3:
- remove register struct
- use uncached screen_base
changes since v4:
- remove useless dev_add_param()
- use wait_on_timeout() instead while-deadlock
--- arch/arm/mach-omap/Makefile | 1 + arch/arm/mach-omap/include/mach/omap4-fb.h | 46 +++ arch/arm/mach-omap/omap4_fb.c | 27 ++ drivers/video/Kconfig | 8 + drivers/video/Makefile | 1 + drivers/video/omap4.c | 527 ++++++++++++++++++++++++++++ drivers/video/omap4.h | 181 ++++++++++ 7 files changed, 791 insertions(+) create mode 100644 arch/arm/mach-omap/include/mach/omap4-fb.h create mode 100644 arch/arm/mach-omap/omap4_fb.c create mode 100644 drivers/video/omap4.c create mode 100644 drivers/video/omap4.h
arch/arm/mach-omap/Makefile +1 -0
@@ -28,6 +28,7 @@ obj-$(CONFIG_OMAP3_CLOCK_CONFIG) += omap3_clock.o pbl-$(CONFIG_OMAP3_CLOCK_CONFIG) += omap3_clock.o obj-$(CONFIG_OMAP_GPMC) += gpmc.o devices-gpmc-nand.o obj-$(CONFIG_SHELL_NONE) += xload.o +obj-$(CONFIG_DRIVER_VIDEO_OMAP4) += omap4_fb.o obj-$(CONFIG_I2C_TWL6030) += omap4_twl6030_mmc.o obj-$(CONFIG_OMAP4_USBBOOT) += omap4_rom_usb.o obj-y += gpio.o
arch/arm/mach-omap/include/mach/omap4-fb.h new file +46 -0
46 lines not on this page · raw message
arch/arm/mach-omap/omap4_fb.c new file +27 -0
@@ -0,0 +1,27 @@ +#include <driver.h> +#include <common.h> +#include <linux/ioport.h> +#include <mach/omap4-fb.h> + +static struct resource omap4_fb_resources[] = { + { + .name ="omap4_dss" , + .start =0x48040000 , + .end =0x48040000 +512 -1 , + .flags =IORESOURCE_MEM |IORESOURCE_MEM_32BIT , + }, { + .name ="omap4_dispc" , + .start =0x48041000 , + .end =0x48041000 +3072 -1 , + .flags =IORESOURCE_MEM |IORESOURCE_MEM_32BIT , + }, +}; + +struct device_d *omap4_add_display (void *pdata) +{ +return add_generic_device_res ("omap4_fb" ,-1 , + omap4_fb_resources, +ARRAY_SIZE (omap4_fb_resources), + pdata); +} +EXPORT_SYMBOL (omap4_add_display);
drivers/video/Kconfig +8 -0
@@ -45,6 +45,14 @@ config DRIVER_VIDEO_S3C24XX help Add support for the S3C244x LCD controller. +config DRIVER_VIDEO_OMAP4 + bool "OMAP4 framebuffer driver" + depends on ARCH_OMAP4 + help + Add support for the OMAP4 Display Controller. + DSI is unsupported, only DISPC parallel mode on LCD2 + is supported. + if DRIVER_VIDEO_S3C24XX config DRIVER_VIDEO_S3C_VERBOSE
drivers/video/Makefile +1 -0
@@ -8,3 +8,4 @@ obj-$(CONFIG_DRIVER_VIDEO_IMX_IPU) += imx-ipu-fb.o obj-$(CONFIG_DRIVER_VIDEO_S3C24XX) += s3c24xx.o obj-$(CONFIG_DRIVER_VIDEO_PXA) += pxa.o obj-$(CONFIG_DRIVER_VIDEO_SDL) += sdl.o +obj-$(CONFIG_DRIVER_VIDEO_OMAP4) += omap4.o
drivers/video/omap4.c new file +527 -0
527 lines not on this page · raw message
drivers/video/omap4.h new file +181 -0
181 lines not on this page · raw message
Signature · 9 lines
1.7.10.4 _______________________________________________ barebox mailing list [email protected] http://lists.infradead.org/mailman/listinfo/barebox
Quoted 12 lines · Christoph Fritz
This patch adds omap4 display controller support.
Signed-off-by Christoph Fritz <[email protected]>---
changes since v2:
- usedev_request_mem_region_by_name()
changes since v3:
- remove register struct
- use uncachedscreen_base
changes since v4:
- remove uselessdev_add_param()
- usewait_on_timeout()instead while-deadlock
Quoted 4 lines
--- arch/arm/mach-omap/Makefile | 1 + arch/arm/mach-omap/include/mach/omap4-fb.h | 46 +++ arch/arm/mach-omap/omap4_fb.c | 27 ++
Sasha will device but for me this is 2 patches
1 for the driver
1 for the mach-omap
Quoted 209 lines
drivers/video/Kconfig | 8 + drivers/video/Makefile | 1 + drivers/video/omap4.c | 527 ++++++++++++++++++++++++++++ drivers/video/omap4.h | 181 ++++++++++ 7 files changed, 791 insertions(+) create mode 100644 arch/arm/mach-omap/include/mach/omap4-fb.h create mode 100644 arch/arm/mach-omap/omap4_fb.c create mode 100644 drivers/video/omap4.c create mode 100644 drivers/video/omap4.harch/arm/mach-omap/Makefile +1
-0@@ -28,6 +28,7 @@ obj-$(CONFIG_OMAP3_CLOCK_CONFIG) += omap3_clock.o pbl-$(CONFIG_OMAP3_CLOCK_CONFIG) += omap3_clock.o obj-$(CONFIG_OMAP_GPMC) += gpmc.o devices-gpmc-nand.o obj-$(CONFIG_SHELL_NONE) += xload.o +obj-$(CONFIG_DRIVER_VIDEO_OMAP4) += omap4_fb.o obj-$(CONFIG_I2C_TWL6030) += omap4_twl6030_mmc.o obj-$(CONFIG_OMAP4_USBBOOT) += omap4_rom_usb.o obj-y += gpio.oarch/arm/mach-omap/include/mach/omap4-fb.h new file +46
-0@@ -0,0 +1,46 @@ +#ifndef H_BAREBOX_ARCH_ARM_MACH_OMAP_MACH_FB4_H +#define H_BAREBOX_ARCH_ARM_MACH_OMAP_MACH_FB4_H + +#include <fb.h> + +#define OMAP_DSS_LCD_TFT (1u << 0) +#define OMAP_DSS_LCD_IVS (1u << 1) +#define OMAP_DSS_LCD_IHS (1u << 2) +#define OMAP_DSS_LCD_IPC (1u << 3) +#define OMAP_DSS_LCD_IEO (1u << 4) +#define OMAP_DSS_LCD_RF (1u << 5) +#define OMAP_DSS_LCD_ONOFF (1u << 6) + +#define OMAP_DSS_LCD_DATALINES (_l) ((_l) << 10) +#define OMAP_DSS_LCD_DATALINES_msk OMAP_DSS_LCD_DATALINES(3u) +#define OMAP_DSS_LCD_DATALINES_12 OMAP_DSS_LCD_DATALINES(0u) +#define OMAP_DSS_LCD_DATALINES_16 OMAP_DSS_LCD_DATALINES(1u) +#define OMAP_DSS_LCD_DATALINES_18 OMAP_DSS_LCD_DATALINES(2u) +#define OMAP_DSS_LCD_DATALINES_24 OMAP_DSS_LCD_DATALINES(3u) + +struct omap4fb_display { +struct fb_videomode mode; + +unsigned long config; + +unsigned int power_on_delay; +unsigned int power_off_delay; +}; + +struct omap4fb_platform_data { +struct omap4fb_display const *displays; +size_t num_displays; + +unsigned int dss_clk_hz; + +unsigned int bpp; + +struct resource const *screen; + +void (*enable)(int p); +}; + +struct device_d ; +struct device_d *omap4_add_display (void *pdata); + +#endif /* H_BAREBOX_ARCH_ARM_MACH_OMAP_MACH_FB4_H */ arch/arm/mach-omap/omap4_fb.c new file +27
-0@@ -0,0 +1,27 @@ +#include <driver.h> +#include <common.h> +#include <linux/ioport.h> +#include <mach/omap4-fb.h> + +static struct resource omap4_fb_resources[] = { + { + .name ="omap4_dss" , + .start =0x48040000 , + .end =0x48040000 +512 -1 , + .flags =IORESOURCE_MEM |IORESOURCE_MEM_32BIT , + }, { + .name ="omap4_dispc" , + .start =0x48041000 , + .end =0x48041000 +3072 -1 , + .flags =IORESOURCE_MEM |IORESOURCE_MEM_32BIT , + }, +}; + +struct device_d *omap4_add_display (void *pdata) +{ +return add_generic_device_res ("omap4_fb" ,-1 , + omap4_fb_resources, +ARRAY_SIZE (omap4_fb_resources), + pdata); +} +EXPORT_SYMBOL (omap4_add_display);drivers/video/Kconfig +8
-0@@ -45,6 +45,14 @@ config DRIVER_VIDEO_S3C24XX help Add support for the S3C244x LCD controller. +config DRIVER_VIDEO_OMAP4 + bool "OMAP4 framebuffer driver" + depends on ARCH_OMAP4 + help + Add support for the OMAP4 Display Controller. + DSI is unsupported, only DISPC parallel mode on LCD2 + is supported. + if DRIVER_VIDEO_S3C24XX config DRIVER_VIDEO_S3C_VERBOSEdrivers/video/Makefile +1
-0@@ -8,3 +8,4 @@ obj-$(CONFIG_DRIVER_VIDEO_IMX_IPU) += imx-ipu-fb.o obj-$(CONFIG_DRIVER_VIDEO_S3C24XX) += s3c24xx.o obj-$(CONFIG_DRIVER_VIDEO_PXA) += pxa.o obj-$(CONFIG_DRIVER_VIDEO_SDL) += sdl.o +obj-$(CONFIG_DRIVER_VIDEO_OMAP4) += omap4.odrivers/video/omap4.c new file +68
-0@@ -0,0 +1,527 @@ +/* +* TI Omap4 Frame Buffer device driver +* +* Copyright (C) 2013 Christoph Fritz <[email protected]> +* Based on work by Enrico Scholz, sponsored by Phytec +* +* See file CREDITS for list of people who contributed to this +* project. +* +* This program is free software; you can redistribute it and/or +* modify it under the terms of the GNU General Public License as +* published by the Free Software Foundation; either version 2 of +* the License, or (at your option) any later version. +* +* 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 <driver.h> +#include <fb.h> +#include <errno.h> +#include <xfuncs.h> +#include <init.h> +#include <stdio.h> +#include <io.h> +#include <common.h> +#include <malloc.h> +#include <common.h> +#include <clock.h> + +#include <mach/omap4-silicon.h> +#include <mach/omap4-fb.h> + +#include <asm/mmu.h> + +#include "omap4.h" + +struct omap4fb_device { +struct fb_info info; +struct device_d *dev; + +struct omap4fb_display const *cur_display; + +struct omap4fb_display const *displays; +size_t num_displays; + +void __iomem *dss; +void __iomem *dispc; + +struct { +void __iomem *addr; +size_t size; + } prealloc_screen; + +struct { +uint32_t dispc_control; +uint32_t dispc_pol_freq; + } shadow; + +struct { +unsigned int dss_clk_hz; +unsigned int lckd; +unsigned int pckd; + } divisor; +size_t dma_size; +void (*enable_fn)(int );
drivers/video/omap4.c line 78
Quoted 10 lines
drivers/video/omap4.c new file +10
-0+ +struct fb_videomode video_modes[]; +}; + +static void omap4fb_enable (struct fb_info *info) +{ +struct omap4fb_device *fbi = + container_of(info, struct omap4fb_device, info); + +dev_info (fbi->dev,"%s\n" , __func__);
dev_dbg
no need verbose for nornal operation
Quoted 21 lines
drivers/video/omap4.c new file +21
-0+ +if (!fbi->cur_display) { +dev_err (fbi->dev,"no valid mode set\n" ); +return ; + } + +if (fbi->enable_fn) + fbi->enable_fn (1 ); + +udelay (fbi->cur_display->power_on_delay *1000u ); + +fb_write (fb_read (fbi->dispc,O4_DISPC_CONTROL2 ) | +DSS_DISPC_CONTROL_LCDENABLE | +DSS_DISPC_CONTROL_LCDENABLESIGNAL , + fbi->dispc,O4_DISPC_CONTROL2 ); + +fb_write (fb_read (fbi->dispc,O4_DISPC_VID1_ATTRIBUTES ) | + DSS_DISPC_VIDn_ATTRIBUTES_VIDENABLE, + fbi->dispc,O4_DISPC_VID1_ATTRIBUTES ); + + fb_write(fb_read(fbi->dispc, O4_DISPC_CONTROL2) |
drivers/video/omap4.c line 109
Quoted 10 lines
drivers/video/omap4.c new file +10
-0+DSS_DISPC_CONTROL_GOLCD , + fbi->dispc, O4_DISPC_CONTROL2); +} + +static void omap4fb_disable (struct fb_info *info) +{ +struct omap4fb_device *fbi = + container_of(info, struct omap4fb_device, info); + +dev_info (fbi->dev,"%s\n" , __func__);
ditto
Quoted 43 lines
drivers/video/omap4.c new file +43
-0+ +if (!fbi->cur_display) { +dev_err (fbi->dev,"no valid mode set\n" ); +return ; + } + +fb_write (fb_read (fbi->dispc,O4_DISPC_CONTROL2 ) & + ~(DSS_DISPC_CONTROL_LCDENABLE | +DSS_DISPC_CONTROL_LCDENABLESIGNAL ), + fbi->dispc,O4_DISPC_CONTROL2 ); + +fb_write (fb_read (fbi->dispc,O4_DISPC_VID1_ATTRIBUTES ) & + ~(DSS_DISPC_VIDn_ATTRIBUTES_VIDENABLE), + fbi->dispc,O4_DISPC_VID1_ATTRIBUTES ); + +if (fbi->prealloc_screen.addr ==NULL ) { +/* free frame buffer; but only when screen is not +* preallocated */ +if (info->screen_base) +dma_free_coherent (info->screen_base, fbi->dma_size); + } + + info->screen_base =NULL ; + +udelay (fbi->cur_display->power_off_delay *1000u ); + +if (fbi->enable_fn) + fbi->enable_fn (0 ); +} + +static void omap4fb_calc_divisor (struct omap4fb_device *fbi, +struct fb_videomode const *mode) +{ +unsigned int l, k, t, b; + + b =UINT_MAX ; +for (l =1 ; l <256 ; l++) { +for (k =1 ; k <256 ; k++) { + t =abs (mode->pixclock *100 - + (fbi->divisor.dss_clk_hz / l / k)); +if (t <= b) { + b = t; + fbi->divisor.lckd = l;
drivers/video/omap4.c line 162
Quoted 10 lines
drivers/video/omap4.c new file +10
-0+ fbi->divisor.pckd = k; + } + } + } +} + +static unsigned int omap4fb_calc_format (struct fb_info const *info) +{ +switch (info->bits_per_pixel) { +case 24 :return 9 ;
case x:
return
drivers/video/omap4.c line 165
Quoted 3 lines
drivers/video/omap4.c new file +3
-0+case 32 :return 0x8 ;/* xRGB24-8888 (32-bit container) */ +default : + printf("%s: unsupported bpp %d\n", __func__,
dev_err
Quoted 77 lines
drivers/video/omap4.c new file +77
-0+ info->bits_per_pixel); +return 0 ; + } +} + +struct omap4fb_colors { +struct fb_bitfield red; +struct fb_bitfield green; +struct fb_bitfield blue; +struct fb_bitfield transp; +}; + +static struct omap4fb_colors const omap4fb_col[] = { + [0 ] = { + .red = { .length =0 , .offset =0 }, + }, + [1 ] = { + .blue = { .length =8 , .offset =0 }, + .green = { .length =8 , .offset =8 }, + .red = { .length =8 , .offset =16 }, + }, + [2 ] = { + .blue = { .length =8 , .offset =0 }, + .green = { .length =8 , .offset =8 }, + .red = { .length =8 , .offset =16 }, + .transp = { .length =8 , .offset =24 }, + }, +}; + +static void omap4fb_fill_shadow (struct omap4fb_device *fbi, +struct omap4fb_display const *display) +{ + fbi->shadow.dispc_control =0 ; + fbi->shadow.dispc_pol_freq =0 ; + + fbi->shadow.dispc_control |=DSS_DISPC_CONTROL_STNTFT ; + +switch (display->config & OMAP_DSS_LCD_DATALINES_msk) { +case OMAP_DSS_LCD_DATALINES_12 : + fbi->shadow.dispc_control |=DSS_DISPC_CONTROL_TFTDATALINES_12 ; +break ; +case OMAP_DSS_LCD_DATALINES_16 : + fbi->shadow.dispc_control |=DSS_DISPC_CONTROL_TFTDATALINES_16 ; +break ; +case OMAP_DSS_LCD_DATALINES_18 : + fbi->shadow.dispc_control |=DSS_DISPC_CONTROL_TFTDATALINES_18 ; +break ; +case OMAP_DSS_LCD_DATALINES_24 : + fbi->shadow.dispc_control |=DSS_DISPC_CONTROL_TFTDATALINES_24 ; +break ; + } + +if (display->config &OMAP_DSS_LCD_IPC ) + fbi->shadow.dispc_pol_freq |=DSS_DISPC_POL_FREQ_IPC ; + +if (display->config &OMAP_DSS_LCD_IVS ) + fbi->shadow.dispc_pol_freq |=DSS_DISPC_POL_FREQ_IVS ; + +if (display->config &OMAP_DSS_LCD_IHS ) + fbi->shadow.dispc_pol_freq |=DSS_DISPC_POL_FREQ_IHS ; + +if (display->config &OMAP_DSS_LCD_IEO ) + fbi->shadow.dispc_pol_freq |=DSS_DISPC_POL_FREQ_IEO ; + +if (display->config &OMAP_DSS_LCD_RF ) + fbi->shadow.dispc_pol_freq |=DSS_DISPC_POL_FREQ_RF ; + +if (display->config &OMAP_DSS_LCD_ONOFF ) + fbi->shadow.dispc_pol_freq |=DSS_DISPC_POL_FREQ_ONOFF ; +} + +static int omap4fb_activate_var (struct fb_info *info) +{ +struct omap4fb_device *fbi = + container_of(info, struct omap4fb_device, info); +struct fb_videomode const *mode = info->mode; +size_t size = mode->xres * mode->yres * (info->bits_per_pixel /8 );
drivers/video/omap4.c line 252
Quoted 10 lines
drivers/video/omap4.c new file +10
-0+int rc; +unsigned int fmt =omap4fb_calc_format (info); +struct omap4fb_colors const *cols; +size_t i; +struct omap4fb_display const *new_display =NULL ; + +for (i =0 ; i < fbi->num_displays && new_display ==NULL ; ++i) { +if (strcmp (mode->name, fbi->displays[i].mode.name) ==0 ) + new_display = &fbi->displays[i]; + }
do this search in a function and return
Quoted 5 lines
drivers/video/omap4.c new file +5
-0+ +if (WARN_ON (!new_display)) { +dev_err (fbi->dev,"no display found for this mode '%s'\n" , + mode->name); + rc = -ENXIO ;
drivers/video/omap4.c line 267
Quoted 10 lines
drivers/video/omap4.c new file +10
-0+ goto out; + } + +/*Free old screen buf*/ +if (!fbi->prealloc_screen.addr && info->screen_base) +dma_free_coherent (info->screen_base, fbi->dma_size); + + fbi->dma_size =PAGE_ALIGN (size); + +if (fbi->prealloc_screen.addr ==NULL ) {
consistant !xxx
Quoted 180 lines
drivers/video/omap4.c new file +180
-0+/* case 1: no preallocated screen */ + info->screen_base =dma_alloc_coherent (size); + }else if (fbi->prealloc_screen.size < fbi->dma_size) { +/* case 2: preallocated screen, but too small */ +dev_err (fbi->dev, +"allocated framebuffer too small (%zu < %zu)\n" , + fbi->prealloc_screen.size, fbi->dma_size); + rc = -ENOMEM ; + goto out; + }else { +/* case 3: preallocated screen */ + info->screen_base = fbi->prealloc_screen.addr; + } + +omap4fb_fill_shadow (fbi, new_display); + +omap4fb_calc_divisor (fbi, mode); + +switch (info->bits_per_pixel) { +case 24 : + cols = &omap4fb_col[1 ]; +break ; +case 32 : + cols = &omap4fb_col[2 ]; +break ; +default : + cols = &omap4fb_col[0 ]; + } + + info->red = cols->red; + info->green = cols->green; + info->blue = cols->blue; + info->transp = cols->transp; + +fb_write (fbi->shadow.dispc_control, + fbi->dispc,O4_DISPC_CONTROL2 ); + +fb_write (fbi->shadow.dispc_pol_freq, + fbi->dispc,O4_DISPC_POL_FREQ2 ); + +fb_write (DSS_DISPC_TIMING_H_HSW (mode->hsync_len -1 ) | +DSS_DISPC_TIMING_H_HFP (mode->right_margin -1 ) | +DSS_DISPC_TIMING_H_HBP (mode->left_margin -1 ), + fbi->dispc,O4_DISPC_TIMING_H2 ); + +fb_write (DSS_DISPC_TIMING_V_VSW (mode->vsync_len -1 ) | +DSS_DISPC_TIMING_V_VFP (mode->lower_margin) | +DSS_DISPC_TIMING_V_VBP (mode->upper_margin), + fbi->dispc,O4_DISPC_TIMING_V2 ); + +fb_write (DSS_DISPC_DIVISOR_ENABLE |DSS_DISPC_DIVISOR_LCD (1 ), + fbi->dispc,O4_DISPC_DIVISOR ); + +fb_write (DSS_DISPC_DIVISOR2_LCD (fbi->divisor.lckd) | +DSS_DISPC_DIVISOR2_PCD (fbi->divisor.pckd), + fbi->dispc,O4_DISPC_DIVISOR2 ); + +fb_write (DSS_DISPC_SIZE_LCD_PPL (mode->xres -1 ) | +DSS_DISPC_SIZE_LCD_LPP (mode->yres -1 ), + fbi->dispc,O4_DISPC_SIZE_LCD2 ); + +fb_write (0x0000ff00 , fbi->dispc,O4_DISPC_DEFAULT_COLOR2 ); + +/* we use VID1 */ +fb_write ((uintptr_t )info->screen_base, fbi->dispc,O4_DISPC_VID1_BA0 ); +fb_write ((uintptr_t )info->screen_base, fbi->dispc,O4_DISPC_VID1_BA1 ); + +fb_write (DSS_DISPC_VIDn_POSITION_VIDPOSX (0 ) | +DSS_DISPC_VIDn_POSITION_VIDPOSY (0 ), + fbi->dispc,O4_DISPC_VID1_POSITION ); +fb_write (DSS_DISPC_VIDn_SIZE_VIDSIZEX (mode->xres -1 ) | +DSS_DISPC_VIDn_SIZE_VIDSIZEY (mode->yres -1 ), + fbi->dispc,O4_DISPC_VID1_SIZE ); +fb_write (DSS_DISPC_VIDn_PICTURE_SIZE_VIDORGSIZEX (mode->xres -1 ) | +DSS_DISPC_VIDn_PICTURE_SIZE_VIDORGSIZEY (mode->yres -1 ), + fbi->dispc,O4_DISPC_VID1_PICTURE_SIZE ); +fb_write (1 , fbi->dispc,O4_DISPC_VID1_ROW_INC ); +fb_write (1 , fbi->dispc,O4_DISPC_VID1_PIXEL_INC ); + +fb_write (0xfff , fbi->dispc,O4_DISPC_VID1_PRELOAD ); + +fb_write (DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT (fmt) | + DSS_DISPC_VIDn_ATTRIBUTES_VIDBURSTSIZE_8x128 | + DSS_DISPC_VIDn_ATTRIBUTES_ZORDERENABLE | + DSS_DISPC_VIDn_ATTRIBUTES_CHANNELOUT2_SECONDARY_LCD, + fbi->dispc,O4_DISPC_VID1_ATTRIBUTES ); + + rc =wait_on_timeout (O4_TIMEOUT , + !(fb_read (fbi->dispc,O4_DISPC_CONTROL2 ) & +DSS_DISPC_CONTROL_GOLCD )); + +if (rc) { +dev_err (fbi->dev,"timeout: dispc golcd\n" ); + goto out; + } + +fb_write (fb_read (fbi->dispc,O4_DISPC_CONTROL2 ) | +DSS_DISPC_CONTROL_GOLCD , + fbi->dispc,O4_DISPC_CONTROL2 ); + + fbi->cur_display = new_display; + info->xres = mode->xres; + info->yres = mode->yres; + + rc =0 ; + +out: +return rc; +} + +static int omap4fb_reset (struct omap4fb_device const *fbi) +{ +uint32_t v =fb_read (fbi->dispc,O4_DISPC_CONTROL2 ); +int rc; + +/* step 1: stop the LCD controller */ +if (v &DSS_DISPC_CONTROL_LCDENABLE ) { +fb_write (v & ~DSS_DISPC_CONTROL_LCDENABLE , + fbi->dispc,O4_DISPC_CONTROL2 ); + +fb_write (DSS_DISPC_IRQSTATUS_FRAMEDONE2 , fbi->dispc, +O4_DISPC_IRQSTATUS ); + + + rc =wait_on_timeout (O4_TIMEOUT , + ((fb_read (fbi->dispc,O4_DISPC_IRQSTATUS ) & +DSS_DISPC_IRQSTATUS_FRAMEDONE ) !=0 )); + +if (rc) { +dev_err (fbi->dev,"timeout: irqstatus framedone\n" ); +return -ETIMEDOUT ; + } + } + +/* step 2: wait for reset done status */ + rc =wait_on_timeout (O4_TIMEOUT , + (fb_read (fbi->dss,O4_DSS_SYSSTATUS ) & +DSS_DSS_SYSSTATUS_RESETDONE )); + +if (rc) { +dev_err (fbi->dev,"timeout: sysstatus resetdone\n" ); +return -ETIMEDOUT ; + } + +/* DSS_CTL: set to reset value */ +fb_write (0 , fbi->dss,O4_DSS_CTRL ); + +return 0 ; +} + +static struct fb_ops omap4fb_ops = { + .fb_enable = omap4fb_enable, + .fb_disable = omap4fb_disable, + .fb_activate_var = omap4fb_activate_var, +}; + +static int omap4fb_probe (struct device_d *dev) +{ +struct omap4fb_platform_data const *pdata = dev->platform_data; +struct omap4fb_device *fbi; +struct fb_info *info; +int rc; +size_t i; + +if (!pdata) +return -ENODEV ; + + fbi =xzalloc (sizeof *fbi + + pdata->num_displays *sizeof fbi->video_modes[0 ]); + info = &fbi->info; + + fbi->dev = dev; + +/* CM_DSS_CLKSTCTRL (TRM: 935) trigger SW_WKUP */ +__raw_writel (0x2 ,0x4a009100 );/* TODO: move this to clockmanagement */ + + fbi->dss =dev_request_mem_region_by_name (dev,"omap4_dss" ); + fbi->dispc =dev_request_mem_region_by_name (dev,"omap4_dispc" ); + +if (!fbi->dss || !fbi->dispc) {
drivers/video/omap4.c line 457
Quoted 10 lines
drivers/video/omap4.c new file +10
-0+dev_err (dev,"Insufficient register description\n" ); + rc = -EINVAL ; + goto out; + } + +dev_info (dev,"HW-Revision 0x%04x 0x%04x\n" , +fb_read (fbi->dss,O4_DISPC_REVISION ), +fb_read (fbi->dss,O4_DSS_REVISION )); + +
drop one line
Quoted 95 lines
drivers/video/omap4.c new file +70
-0+if (!pdata->dss_clk_hz | !pdata->displays | !pdata->num_displays | + !pdata->bpp) { +dev_err (dev,"Insufficient omap4fb_platform_data\n" ); + rc = -EINVAL ; + goto out; + } + + fbi->enable_fn = pdata->enable; + fbi->displays = pdata->displays; + fbi->num_displays = pdata->num_displays; + fbi->divisor.dss_clk_hz = pdata->dss_clk_hz; + +for (i =0 ; i < pdata->num_displays; ++i) + fbi->video_modes[i] = pdata->displays[i].mode; + + info->mode_list = fbi->video_modes; + info->num_modes = pdata->num_displays; + + info->priv = fbi; + info->fbops = &omap4fb_ops; + info->bits_per_pixel = pdata->bpp; + +if (pdata->screen) { +if (!IS_ALIGNED (pdata->screen->start,PAGE_SIZE ) || + !IS_ALIGNED (resource_size (pdata->screen),PAGE_SIZE )) { +dev_err (dev,"screen resource not aligned\n" ); + rc = -EINVAL ; + goto out; + } + fbi->prealloc_screen.addr = + (void __iomem *)pdata->screen->start; + fbi->prealloc_screen.size =resource_size (pdata->screen); +remap_range (fbi->prealloc_screen.addr, + fbi->prealloc_screen.size, +mmu_get_pte_uncached_flags ()); + } + + rc =omap4fb_reset (fbi); +if (rc <0 ) { +dev_err (dev,"failed to reset: %d\n" , rc); + goto out; + } + + rc =register_framebuffer (info); +if (rc <0 ) { +dev_err (dev,"failed to register framebuffer: %d\n" , rc); + goto out; + } + + rc =0 ; +dev_info (dev,"registered\n" ); + +out: +if (rc <0 ) +free (fbi); + +return rc; +} + +static struct driver_d omap4fb_driver = { + .name ="omap4_fb" , + .probe = omap4fb_probe, +}; + +static int omap4fb_init (void ) +{ +return platform_driver_register (&omap4fb_driver); +} + +device_initcall (omap4fb_init);drivers/video/omap4.h new file +19
-0@@ -0,0 +1,181 @@ +/* +* TI Omap4 Frame Buffer device driver +* +* Copyright (C) 2013 Christoph Fritz <[email protected]> +* +* See file CREDITS for list of people who contributed to this +* project. +* +* This program is free software; you can redistribute it and/or +* modify it under the terms of the GNU General Public License as +* published by the Free Software Foundation; either version 2 of +* the License, or (at your option) any later version. +* +* 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. +*/ +
drivers/video/omap4.h line 29
Quoted 10 lines
drivers/video/omap4.h new file +10
-0+#ifndef H_BAREBOX_DRIVER_VIDEO_OMAP4_REGS_H +#define H_BAREBOX_DRIVER_VIDEO_OMAP4_REGS_H + +#include <types.h> +#include <common.h> + +#define O4_TIMEOUT (128 * USECOND) + +#define fb_read (io, reg) __raw_readl((io)+(reg)) +#define fb_write (val, io, reg) __raw_writel((val), (io)+(reg))
fb_read is too much generic
better to use 2 macro for both resources and pass the fbi struct
Quoted 157 lines
drivers/video/omap4.h new file +152
-0+ +/* TRM: 10.1.3.2 DSS Registers */ +#define O4_DSS_REVISION 0x0 +#define O4_DSS_SYSSTATUS 0x14 +#define O4_DSS_CTRL 0x40 +#define O4_DSS_STATUS 0x5c + +/* TRM: 10.2.7.3 Display Controller Registers */ +#define O4_DISPC_REVISION 0x0 +#define O4_DISPC_IRQSTATUS 0x18 +#define O4_DISPC_CONTROL2 0x238 +#define O4_DISPC_VID1_ATTRIBUTES 0xcc +#define O4_DISPC_TIMING_H2 0x400 +#define O4_DISPC_TIMING_V2 0x404 +#define O4_DISPC_POL_FREQ2 0x408 +#define O4_DISPC_DIVISOR2 0x40c +#define O4_DISPC_DEFAULT_COLOR2 0x3ac +#define O4_DISPC_SIZE_LCD2 0x3cc +#define O4_DISPC_DIVISOR 0x804 +#define O4_DISPC_VID1_BA0 0xbc +#define O4_DISPC_VID1_BA1 0xc0 +#define O4_DISPC_VID1_POSITION 0xc4 +#define O4_DISPC_VID1_SIZE 0xc8 +#define O4_DISPC_VID1_PICTURE_SIZE 0xe4 +#define O4_DISPC_VID1_ROW_INC 0xd8 +#define O4_DISPC_VID1_PIXEL_INC 0xdc +#define O4_DISPC_VID1_PRELOAD 0x230 + +#define DSS_DISPC_VIDn_POSITION_VIDPOSX (_x) ((_x) << 0) +#define DSS_DISPC_VIDn_POSITION_VIDPOSY (_y) ((_y) << 16) + +#define DSS_DISPC_VIDn_PICTURE_SIZE_VIDORGSIZEX (_x) ((_x) << 0) +#define DSS_DISPC_VIDn_PICTURE_SIZE_VIDORGSIZEY (_y) ((_y) << 16) + +#define DSS_DISPC_VIDn_SIZE_VIDSIZEX (_x) ((_x) << 0) +#define DSS_DISPC_VIDn_SIZE_VIDSIZEY (_y) ((_y) << 16) + +#define DSS_DISPC_SIZE_LCD_PPL (_x) ((_x) << 0) +#define DSS_DISPC_SIZE_LCD_LPP (_y) ((_y) << 16) + +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDENABLE (1u << 0) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT (_fmt) ((_fmt) << 1) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT_RGB12 \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT(4u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT_ARGB16 \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT(5u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT_RGB16 \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT(6u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT_ARGB16o \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT(7u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT_xRGB24u \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT(8u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT_RGB24p \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT(9u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT_YUV2 \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT(10u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT_UYVY \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT(11u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT_ARGB32 \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT(12u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT_RGBA32 \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT(13u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT_xRGB32 \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDFORMAT(14u) + +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDBURSTSIZE (_b) ((_b) << 14) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDBURSTSIZE_2x128 \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDBURSTSIZE(0u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDBURSTSIZE_4x128 \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDBURSTSIZE(1u) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDBURSTSIZE_8x128 \ + DSS_DISPC_VIDn_ATTRIBUTES_VIDBURSTSIZE(2u) + +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDCHANNELOUT (1u << 16) +#define DSS_DISPC_VIDn_ATTRIBUTES_SELFREFRESHAUTO (1u << 17) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDFIFOPRELOAD (1u << 19) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDVERTICALTAPS (1u << 21) +#define DSS_DISPC_VIDn_ATTRIBUTES_DOUBLESTRIDE (1u << 22) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDARBITRATION (1u << 23) +#define DSS_DISPC_VIDn_ATTRIBUTES_VIDSELFREFRESH (1u << 24) +#define DSS_DISPC_VIDn_ATTRIBUTES_ZORDERENABLE (1u << 25) + +#define DSS_DISPC_VIDn_ATTRIBUTES_CHANNELOUT2 (_b) ((_b) << 30) +#define DSS_DISPC_VIDn_ATTRIBUTES_CHANNELOUT2_PRIMARY_LCD \ + DSS_DISPC_VIDn_ATTRIBUTES_CHANNELOUT2(0u) +#define DSS_DISPC_VIDn_ATTRIBUTES_CHANNELOUT2_SECONDARY_LCD \ + DSS_DISPC_VIDn_ATTRIBUTES_CHANNELOUT2(1u) +#define DSS_DISPC_VIDn_ATTRIBUTES_CHANNELOUT2_WRITEBACK_MEM \ + DSS_DISPC_VIDn_ATTRIBUTES_CHANNELOUT2(3u) + +#define DSS_DISPC_CONTROL_LCDENABLE (1u << 0) +#define DSS_DISPC_CONTROL_TVENABLE (1u << 1) +#define DSS_DISPC_CONTROL_MONOCOLOR (1u << 2) +#define DSS_DISPC_CONTROL_STNTFT (1u << 3) +#define DSS_DISPC_CONTROL_M8B (1u << 4) +#define DSS_DISPC_CONTROL_GOLCD (1u << 5) +#define DSS_DISPC_CONTROL_GOTV (1u << 6) +#define DSS_DISPC_CONTROL_STDITHERENABLE (1u << 7) + +#define DSS_DISPC_CONTROL_TFTDATALINES (_l) ((_l) << 8) +#define DSS_DISPC_CONTROL_TFTDATALINES_12 \ + DSS_DISPC_CONTROL_TFTDATALINES(0u) +#define DSS_DISPC_CONTROL_TFTDATALINES_16 \ + DSS_DISPC_CONTROL_TFTDATALINES(1u) +#define DSS_DISPC_CONTROL_TFTDATALINES_18 \ + DSS_DISPC_CONTROL_TFTDATALINES(2u) +#define DSS_DISPC_CONTROL_TFTDATALINES_24 \ + DSS_DISPC_CONTROL_TFTDATALINES(3u) + +#define DSS_DISPC_CONTROL_STALLMODE (1u << 11) +#define DSS_DISPC_CONTROL_OVERLAYOPTIMIZATION (1u << 12) +#define DSS_DISPC_CONTROL_GPIN0 (1u << 13)/* ro */ +#define DSS_DISPC_CONTROL_GPIN1 (1u << 14)/* ro */ +#define DSS_DISPC_CONTROL_GPOUT0 (1u << 15) +#define DSS_DISPC_CONTROL_GPOUT1 (1u << 16) +#define DSS_DISPC_CONTROL_HT (_ht) ((_ht) << 17) +#define DSS_DISPC_CONTROL_TDMENABLE (1u << 20) +#define DSS_DISPC_CONTROL_TDMPARALLELMODE (_pm) ((_pm) << 21) +#define DSS_DISPC_CONTROL_TDMCYCLEFORMAT (_cf) ((_cf) << 23) +#define DSS_DISPC_CONTROL_TDMUNUSEDBITS (_ub) ((_ub) << 25) +#define DSS_DISPC_CONTROL_PCKFREEENABLE (1u << 27) +#define DSS_DISPC_CONTROL_LCDENABLESIGNAL (1u << 28) +#define DSS_DISPC_CONTROL_LCDENABLEPOL (1u << 29) +#define DSS_DISPC_CONTROL_SPATIALTEMPD (_df) ((_df) << 30) + +#define DSS_DISPC_POL_FREQ_IVS (1u << 12) +#define DSS_DISPC_POL_FREQ_IHS (1u << 13) +#define DSS_DISPC_POL_FREQ_IPC (1u << 14) +#define DSS_DISPC_POL_FREQ_IEO (1u << 15) +#define DSS_DISPC_POL_FREQ_RF (1u << 16) +#define DSS_DISPC_POL_FREQ_ONOFF (1u << 17) + +#define DSS_DISPC_TIMING_H_HSW (_hsw) ((_hsw) << 0) +#define DSS_DISPC_TIMING_H_HFP (_hfp) ((_hfp) << 8) +#define DSS_DISPC_TIMING_H_HBP (_hbp) ((_hbp) << 20) + +#define DSS_DISPC_TIMING_V_VSW (_vsw) ((_vsw) << 0) +#define DSS_DISPC_TIMING_V_VFP (_vfp) ((_vfp) << 8) +#define DSS_DISPC_TIMING_V_VBP (_vbp) ((_vbp) << 20) + +#define DSS_DISPC_DIVISOR_ENABLE (1u << 0) +#define DSS_DISPC_DIVISOR_LCD (_lcd) ((_lcd) << 16) + +#define DSS_DISPC_DIVISOR2_PCD (_pcd) ((_pcd) << 0) +#define DSS_DISPC_DIVISOR2_LCD (_lcd) ((_lcd) << 16) + +#define DSS_DISPC_IRQSTATUS_FRAMEDONE (1u << 0) +#define DSS_DISPC_IRQSTATUS_FRAMEDONE2 (1u << 22) + +#define DSS_DSS_SYSSTATUS_RESETDONE (1u << 0) + +#endif /* H_BAREBOX_DRIVER_VIDEO_O4_REGS_H */ Signature · 1 line
1.7.10.4
_______________________________________________
barebox mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/barebox
PATCH v6 1/1 omap4-fb: add driver
This patch adds omap4 display controller support.
---
changes since v2:
- use dev_request_mem_region_by_name()
changes since v3:
- remove register struct
- use uncached screen_base
changes since v4:
- remove useless dev_add_param()
- use wait_on_timeout() instead while-deadlock
changes since v5:
- use dev_dbg() and dev_err(), be less verbose
- fix coding-style issues
- add omap4fb_find_display_by_name()
- add additional read/write macros
--- arch/arm/mach-omap/Makefile | 1 + arch/arm/mach-omap/include/mach/omap4-fb.h | 46 +++ arch/arm/mach-omap/omap4_fb.c | 27 ++ drivers/video/Kconfig | 8 + drivers/video/Makefile | 1 + drivers/video/omap4.c | 526 ++++++++++++++++++++++++++++ drivers/video/omap4.h | 187 ++++++++++ 7 files changed, 796 insertions(+) create mode 100644 arch/arm/mach-omap/include/mach/omap4-fb.h create mode 100644 arch/arm/mach-omap/omap4_fb.c create mode 100644 drivers/video/omap4.c create mode 100644 drivers/video/omap4.h
arch/arm/mach-omap/Makefile +1 -0
@@ -28,6 +28,7 @@ obj-$(CONFIG_OMAP3_CLOCK_CONFIG) += omap3_clock.o pbl-$(CONFIG_OMAP3_CLOCK_CONFIG) += omap3_clock.o obj-$(CONFIG_OMAP_GPMC) += gpmc.o devices-gpmc-nand.o obj-$(CONFIG_SHELL_NONE) += xload.o +obj-$(CONFIG_DRIVER_VIDEO_OMAP4) += omap4_fb.o obj-$(CONFIG_I2C_TWL6030) += omap4_twl6030_mmc.o obj-$(CONFIG_OMAP4_USBBOOT) += omap4_rom_usb.o obj-y += gpio.o
arch/arm/mach-omap/include/mach/omap4-fb.h new file +46 -0
46 lines not on this page · raw message
arch/arm/mach-omap/omap4_fb.c new file +27 -0
@@ -0,0 +1,27 @@ +#include <driver.h> +#include <common.h> +#include <linux/ioport.h> +#include <mach/omap4-fb.h> + +static struct resource omap4_fb_resources[] = { + { + .name ="omap4_dss" , + .start =0x48040000 , + .end =0x48040000 +512 -1 , + .flags =IORESOURCE_MEM |IORESOURCE_MEM_32BIT , + }, { + .name ="omap4_dispc" , + .start =0x48041000 , + .end =0x48041000 +3072 -1 , + .flags =IORESOURCE_MEM |IORESOURCE_MEM_32BIT , + }, +}; + +struct device_d *omap4_add_display (void *pdata) +{ +return add_generic_device_res ("omap4_fb" ,-1 , + omap4_fb_resources, +ARRAY_SIZE (omap4_fb_resources), + pdata); +} +EXPORT_SYMBOL (omap4_add_display);
drivers/video/Kconfig +8 -0
@@ -45,6 +45,14 @@ config DRIVER_VIDEO_S3C24XX help Add support for the S3C244x LCD controller. +config DRIVER_VIDEO_OMAP4 + bool "OMAP4 framebuffer driver" + depends on ARCH_OMAP4 + help + Add support for the OMAP4 Display Controller. + DSI is unsupported, only DISPC parallel mode on LCD2 + is supported. + if DRIVER_VIDEO_S3C24XX config DRIVER_VIDEO_S3C_VERBOSE
drivers/video/Makefile +1 -0
@@ -8,3 +8,4 @@ obj-$(CONFIG_DRIVER_VIDEO_IMX_IPU) += imx-ipu-fb.o obj-$(CONFIG_DRIVER_VIDEO_S3C24XX) += s3c24xx.o obj-$(CONFIG_DRIVER_VIDEO_PXA) += pxa.o obj-$(CONFIG_DRIVER_VIDEO_SDL) += sdl.o +obj-$(CONFIG_DRIVER_VIDEO_OMAP4) += omap4.o
drivers/video/omap4.c new file +526 -0
526 lines not on this page · raw message
drivers/video/omap4.h new file +187 -0
187 lines not on this page · raw message
Signature · 8 lines
1.7.10.4 _______________________________________________ barebox mailing list [email protected] http://lists.infradead.org/mailman/listinfo/barebox
Hi Christoph,
Some small comments, then we shoulf be good to go.
arch/arm/mach-omap/Makefile obj-$(CONFIG_OMAP3_CLOCK_CONFIG) += omap3_clock.o · line 31
Quoted 10 lines · Christoph Fritz
This patch adds omap4 display controller support.
index 94e42c6..e70ddbd 100644
arch/arm/mach-omap/Makefile +1
-0@@ -28,6 +28,7 @@ obj-$(CONFIG_OMAP3_CLOCK_CONFIG) += omap3_clock.o pbl-$(CONFIG_OMAP3_CLOCK_CONFIG) += omap3_clock.o obj-$(CONFIG_OMAP_GPMC) += gpmc.o devices-gpmc-nand.o obj-$(CONFIG_SHELL_NONE) += xload.o +obj-$(CONFIG_DRIVER_VIDEO_OMAP4) += omap4_fb.o
Please compile this in unconditionally. Otherwise board using this have to add a dependency on CONFIG_DRIVER_VIDEO_OMAP4 aswell.
arch/arm/mach-omap/Makefile obj-$(CONFIG_OMAP3_CLOCK_CONFIG) += omap3_clock.o · line 32
Quoted 1 line
arch/arm/mach-omap/Makefile +1
-0+struct device_d *omap4_add_display(void *pdata)
Let this function take a struct omap4fb_platform_data *. This makes it more clear how to call it.
arch/arm/mach-omap/Makefile obj-$(CONFIG_OMAP3_CLOCK_CONFIG) += omap3_clock.o
Quoted 4 lines
arch/arm/mach-omap/Makefile +4
-0+static void omap4fb_enable(struct fb_info *info) +{ + struct omap4fb_device *fbi = + container_of(info, struct omap4fb_device, info);
You could add a static inline to_omap4fb function. It's less error prone and the above would fit into a single line.
Sascha
Signature · 9 lines
Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ barebox mailing list [email protected] http://lists.infradead.org/mailman/listinfo/barebox
Quoted 7 lines · Sascha Hauer
Hi Christoph,
Some small comments, then we shoulf be good to go.
Quoted 2 lines · Christoph Fritz
This patch adds omap4 display controller support.
Quoted 11 lines
arch/arm/mach-omap/Makefile obj-$(CONFIG_OMAP3_CLOCK_CONFIG) += omap3_clock.o · line 31
Quoted 8 lines
index 94e42c6..e70ddbd 100644
arch/arm/mach-omap/Makefile +1
-0@@ -28,6 +28,7 @@ obj-$(CONFIG_OMAP3_CLOCK_CONFIG) += omap3_clock.o pbl-$(CONFIG_OMAP3_CLOCK_CONFIG) += omap3_clock.o obj-$(CONFIG_OMAP_GPMC) += gpmc.o devices-gpmc-nand.o obj-$(CONFIG_SHELL_NONE) += xload.o +obj-$(CONFIG_DRIVER_VIDEO_OMAP4) += omap4_fb.oPlease compile this in unconditionally. Otherwise board using this have to add a dependency on
CONFIG_DRIVER_VIDEO_OMAP4aswell.
if you have an inline for the fonction no need
as this will do the trick too
the mach-omap and the driver should be splittted and IIRC the omap fb is not only for omap 4
I add Tomi to this patch from TI
Quoted 5 lines
arch/arm/mach-omap/Makefile obj-$(CONFIG_OMAP3_CLOCK_CONFIG) += omap3_clock.o · line 32
Quoted 2 lines
arch/arm/mach-omap/Makefile +1
-0+struct device_d *omap4_add_display(void *pdata)Let this function take a struct
omap4fb_platform_data*. This makes it more clear how to call it.
and check the prototype by gcc
Quoted 9 lines
arch/arm/mach-omap/Makefile obj-$(CONFIG_OMAP3_CLOCK_CONFIG) += omap3_clock.o
Quoted 5 lines
arch/arm/mach-omap/Makefile +4
-0+static void omap4fb_enable(struct fb_info *info) +{ + struct omap4fb_device *fbi = + container_of(info, struct omap4fb_device, info);You could add a static inline
to_omap4fbfunction. It's less error prone and the above would fit into a single line.
Quoted 7 lines
Sascha
Signature · 4 lines
Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
_______________________________________________
barebox mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/barebox
PATCH v7 1/1 omap4-fb: add driver
In reply to Sascha HauerThis patch adds omap4 display controller support.
---
changes since v2:
- use dev_request_mem_region_by_name()
changes since v3:
- remove register struct
- use uncached screen_base
changes since v4:
- remove useless dev_add_param()
- use wait_on_timeout() instead while-deadlock
changes since v5:
- use dev_dbg() and dev_err(), be less verbose
- fix coding-style issues
- add omap4fb_find_display_by_name()
- add additional read/write macros
changes since v6:
- add inline function to_omap4fb()
- use 'struct omap4fb_platform_data *' as argument of omap4_add_display()
- remove platform data check from omap4fb_probe()
- compile omap4_fb.o unconditionally of CONFIG_DRIVER_VIDEO_OMAP4
--- arch/arm/mach-omap/Makefile | 1 + arch/arm/mach-omap/include/mach/omap4-fb.h | 47 +++ arch/arm/mach-omap/omap4_fb.c | 27 ++ drivers/video/Kconfig | 8 + drivers/video/Makefile | 1 + drivers/video/omap4.c | 524 ++++++++++++++++++++++++++++ drivers/video/omap4.h | 187 ++++++++++ 7 files changed, 795 insertions(+) create mode 100644 arch/arm/mach-omap/include/mach/omap4-fb.h create mode 100644 arch/arm/mach-omap/omap4_fb.c create mode 100644 drivers/video/omap4.c create mode 100644 drivers/video/omap4.h
arch/arm/mach-omap/Makefile +1 -0
@@ -23,6 +23,7 @@ obj-$(CONFIG_ARCH_OMAP3) += omap3_core.o omap3_generic.o auxcr.o pbl-$(CONFIG_ARCH_OMAP3) += omap3_core.o omap3_generic.o auxcr.o obj-$(CONFIG_ARCH_OMAP4) += omap4_generic.o omap4_clock.o pbl-$(CONFIG_ARCH_OMAP4) += omap4_generic.o omap4_clock.o +obj-$(CONFIG_ARCH_OMAP4) += omap4_fb.o obj-$(CONFIG_ARCH_AM33XX) += am33xx_generic.o am33xx_clock.o am33xx_mux.o obj-$(CONFIG_OMAP3_CLOCK_CONFIG) += omap3_clock.o pbl-$(CONFIG_OMAP3_CLOCK_CONFIG) += omap3_clock.o
arch/arm/mach-omap/include/mach/omap4-fb.h new file +47 -0
47 lines not on this page · raw message
arch/arm/mach-omap/omap4_fb.c new file +27 -0
@@ -0,0 +1,27 @@ +#include <driver.h> +#include <common.h> +#include <linux/ioport.h> +#include <mach/omap4-fb.h> + +static struct resource omap4_fb_resources[] = { + { + .name ="omap4_dss" , + .start =0x48040000 , + .end =0x48040000 +512 -1 , + .flags =IORESOURCE_MEM |IORESOURCE_MEM_32BIT , + }, { + .name ="omap4_dispc" , + .start =0x48041000 , + .end =0x48041000 +3072 -1 , + .flags =IORESOURCE_MEM |IORESOURCE_MEM_32BIT , + }, +}; + +struct device_d *omap4_add_display (struct omap4fb_platform_data *o4_pdata) +{ +return add_generic_device_res ("omap4_fb" ,-1 , + omap4_fb_resources, +ARRAY_SIZE (omap4_fb_resources), + o4_pdata); +} +EXPORT_SYMBOL (omap4_add_display);
drivers/video/Kconfig +8 -0
@@ -45,6 +45,14 @@ config DRIVER_VIDEO_S3C24XX help Add support for the S3C244x LCD controller. +config DRIVER_VIDEO_OMAP4 + bool "OMAP4 framebuffer driver" + depends on ARCH_OMAP4 + help + Add support for the OMAP4 Display Controller. + DSI is unsupported, only DISPC parallel mode on LCD2 + is supported. + if DRIVER_VIDEO_S3C24XX config DRIVER_VIDEO_S3C_VERBOSE
drivers/video/Makefile +1 -0
@@ -8,3 +8,4 @@ obj-$(CONFIG_DRIVER_VIDEO_IMX_IPU) += imx-ipu-fb.o obj-$(CONFIG_DRIVER_VIDEO_S3C24XX) += s3c24xx.o obj-$(CONFIG_DRIVER_VIDEO_PXA) += pxa.o obj-$(CONFIG_DRIVER_VIDEO_SDL) += sdl.o +obj-$(CONFIG_DRIVER_VIDEO_OMAP4) += omap4.o
drivers/video/omap4.c new file +524 -0
524 lines not on this page · raw message
drivers/video/omap4.h new file +187 -0
187 lines not on this page · raw message
Signature · 9 lines
1.7.10.4 _______________________________________________ barebox mailing list [email protected] http://lists.infradead.org/mailman/listinfo/barebox
Hi,
only a very small nitpick.
Quoted 56 lines · Christoph Fritz
This patch adds omap4 display controller support.
Signed-off-by Christoph Fritz <[email protected]>---
changes since v2:
- usedev_request_mem_region_by_name()
changes since v3:
- remove register struct
- use uncachedscreen_base
changes since v4:
- remove uselessdev_add_param()
- usewait_on_timeout()instead while-deadlock
changes since v5:
- usedev_dbg()anddev_err(), be less verbose
- fix coding-style issues
- addomap4fb_find_display_by_name()
- add additional read/write macros--- arch/arm/mach-omap/Makefile | 1 + arch/arm/mach-omap/include/mach/omap4-fb.h | 46 +++ arch/arm/mach-omap/omap4_fb.c | 27 ++ drivers/video/Kconfig | 8 + drivers/video/Makefile | 1 + drivers/video/omap4.c | 526 ++++++++++++++++++++++++++++ drivers/video/omap4.h | 187 ++++++++++ 7 files changed, 796 insertions(+) create mode 100644 arch/arm/mach-omap/include/mach/omap4-fb.h create mode 100644 arch/arm/mach-omap/omap4_fb.c create mode 100644 drivers/video/omap4.c create mode 100644 drivers/video/omap4.harch/arm/mach-omap/Makefile +1
-0@@ -28,6 +28,7 @@ obj-$(CONFIG_OMAP3_CLOCK_CONFIG) += omap3_clock.o pbl-$(CONFIG_OMAP3_CLOCK_CONFIG) += omap3_clock.o obj-$(CONFIG_OMAP_GPMC) += gpmc.o devices-gpmc-nand.o obj-$(CONFIG_SHELL_NONE) += xload.o +obj-$(CONFIG_DRIVER_VIDEO_OMAP4) += omap4_fb.o obj-$(CONFIG_I2C_TWL6030) += omap4_twl6030_mmc.o obj-$(CONFIG_OMAP4_USBBOOT) += omap4_rom_usb.o obj-y += gpio.oarch/arm/mach-omap/include/mach/omap4-fb.h new file +7
-0@@ -0,0 +1,46 @@ +#ifndef H_BAREBOX_ARCH_ARM_MACH_OMAP_MACH_FB4_H +#define H_BAREBOX_ARCH_ARM_MACH_OMAP_MACH_FB4_H + +#include <fb.h> + +#define OMAP_DSS_LCD_TFT (1u << 0) +#define OMAP_DSS_LCD_IVS (1u << 1)
...
arch/arm/mach-omap/include/mach/omap4-fb.h
Quoted 4 lines
arch/arm/mach-omap/include/mach/omap4-fb.h new file +4
-0+ +if (!pdata) +return -ENODEV ; +
In my opinion this is a false errno for this case. But better is to drop this check completely. It's only a check that a programmer doesn't set a pdata which is very unlikely.
I think there was another patches on this list to remove something like this.
Regards
Alex
_______________________________________________
barebox mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/barebox
PATCH v6 1/1 omap4-fb: add driver
In reply to Jean-Christophe PLAGNIOL-VILLARDThis patch adds omap4 display controller support.
---
changes since v2:
- use dev_request_mem_region_by_name()
changes since v3:
- remove register struct
- use uncached screen_base
changes since v4:
- remove useless dev_add_param()
- use wait_on_timeout() instead while-deadlock
changes since v5:
- use dev_dbg() and dev_err(), be less verbose
- fix coding-style issues
- add omap4fb_find_display_by_name()
- add additional read/write macros
--- arch/arm/mach-omap/Makefile | 1 + arch/arm/mach-omap/include/mach/omap4-fb.h | 46 +++ arch/arm/mach-omap/omap4_fb.c | 27 ++ drivers/video/Kconfig | 8 + drivers/video/Makefile | 1 + drivers/video/omap4.c | 526 ++++++++++++++++++++++++++++ drivers/video/omap4.h | 187 ++++++++++ 7 files changed, 796 insertions(+) create mode 100644 arch/arm/mach-omap/include/mach/omap4-fb.h create mode 100644 arch/arm/mach-omap/omap4_fb.c create mode 100644 drivers/video/omap4.c create mode 100644 drivers/video/omap4.h
arch/arm/mach-omap/Makefile +1 -0
@@ -28,6 +28,7 @@ obj-$(CONFIG_OMAP3_CLOCK_CONFIG) += omap3_clock.o pbl-$(CONFIG_OMAP3_CLOCK_CONFIG) += omap3_clock.o obj-$(CONFIG_OMAP_GPMC) += gpmc.o devices-gpmc-nand.o obj-$(CONFIG_SHELL_NONE) += xload.o +obj-$(CONFIG_DRIVER_VIDEO_OMAP4) += omap4_fb.o obj-$(CONFIG_I2C_TWL6030) += omap4_twl6030_mmc.o obj-$(CONFIG_OMAP4_USBBOOT) += omap4_rom_usb.o obj-y += gpio.o
arch/arm/mach-omap/include/mach/omap4-fb.h new file +46 -0
46 lines not on this page · raw message
arch/arm/mach-omap/omap4_fb.c new file +27 -0
@@ -0,0 +1,27 @@ +#include <driver.h> +#include <common.h> +#include <linux/ioport.h> +#include <mach/omap4-fb.h> + +static struct resource omap4_fb_resources[] = { + { + .name ="omap4_dss" , + .start =0x48040000 , + .end =0x48040000 +512 -1 , + .flags =IORESOURCE_MEM |IORESOURCE_MEM_32BIT , + }, { + .name ="omap4_dispc" , + .start =0x48041000 , + .end =0x48041000 +3072 -1 , + .flags =IORESOURCE_MEM |IORESOURCE_MEM_32BIT , + }, +}; + +struct device_d *omap4_add_display (void *pdata) +{ +return add_generic_device_res ("omap4_fb" ,-1 , + omap4_fb_resources, +ARRAY_SIZE (omap4_fb_resources), + pdata); +} +EXPORT_SYMBOL (omap4_add_display);
drivers/video/Kconfig +8 -0
@@ -45,6 +45,14 @@ config DRIVER_VIDEO_S3C24XX help Add support for the S3C244x LCD controller. +config DRIVER_VIDEO_OMAP4 + bool "OMAP4 framebuffer driver" + depends on ARCH_OMAP4 + help + Add support for the OMAP4 Display Controller. + DSI is unsupported, only DISPC parallel mode on LCD2 + is supported. + if DRIVER_VIDEO_S3C24XX config DRIVER_VIDEO_S3C_VERBOSE
drivers/video/Makefile +1 -0
@@ -8,3 +8,4 @@ obj-$(CONFIG_DRIVER_VIDEO_IMX_IPU) += imx-ipu-fb.o obj-$(CONFIG_DRIVER_VIDEO_S3C24XX) += s3c24xx.o obj-$(CONFIG_DRIVER_VIDEO_PXA) += pxa.o obj-$(CONFIG_DRIVER_VIDEO_SDL) += sdl.o +obj-$(CONFIG_DRIVER_VIDEO_OMAP4) += omap4.o
drivers/video/omap4.c new file +526 -0
526 lines not on this page · raw message
drivers/video/omap4.h new file +187 -0
187 lines not on this page · raw message
Signature · 8 lines
1.7.10.4 _______________________________________________ barebox mailing list [email protected] http://lists.infradead.org/mailman/listinfo/barebox
Hi,
Quoted 1 line · Christoph Fritz
This patch adds omap4 display controller support.
I haven't been following the discussions about this patch, so maybe these have been covered:
OMAP2/3/4/5 all have very similar DSS. Why did you go for omap_4_-fb driver, not omap-fb driver?
drivers/video/Kconfig config DRIVER_VIDEO_S3C24XX · line 55
Quoted 8 lines
drivers/video/Kconfig +8
-0+config DRIVER_VIDEO_OMAP4 + bool "OMAP4 framebuffer driver" + depends on ARCH_OMAP4 + help + Add support for the OMAP4 Display Controller. + DSI is unsupported, only DISPC parallel mode on LCD2 + is supported. +
Also HDMI, RFBI and VENC seem to be unsupported. The DISPC parallel mode you mention is MIPI DPI.
Supporting only the DPI output is "easy", do you have plans to support the other outputs and external encoders used in many boards? That may require a more generic approach, with multiple display "entities" in the pipeline. Not that the first version should support that, but I just wanted to mention it so that you don't design the driver in too non-generic manner.
Tomi
_______________________________________________
barebox mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/barebox
Hi Tomi
Quoted 8 lines · Tomi Valkeinen
Quoted 1 line · Christoph Fritz
This patch adds omap4 display controller support.
I haven't been following the discussions about this patch, so maybe these have been covered:
OMAP2/3/4/5 all have very similar DSS. Why did you go for omap_4_-fb driver, not omap-fb driver?
I'm currently only having an omap4 board here. But I could rename the driver to the more generic name "omap-fb" and explain in the KCONFIG that it currently only covers omap4 devices.
So we could go from there up to all other OMAP devices. Okay?
Quoted 12 lines
drivers/video/Kconfig config DRIVER_VIDEO_S3C24XX · line 55
Quoted 8 lines
drivers/video/Kconfig +8
-0+config DRIVER_VIDEO_OMAP4 + bool "OMAP4 framebuffer driver" + depends on ARCH_OMAP4 + help + Add support for the OMAP4 Display Controller. + DSI is unsupported, only DISPC parallel mode on LCD2 + is supported. +Also HDMI, RFBI and VENC seem to be unsupported. The DISPC parallel mode you mention is MIPI DPI.
Quoted 2 lines
Supporting only the DPI output is "easy", do you have plans to support the other outputs and external encoders used in many boards?
No, at present I have no intention to do so.
Quoted 5 lines
That may
require a more generic approach, with multiple display "entities" in the pipeline. Not that the first version should support that, but I just wanted to mention it so that you don't design the driver in too non-generic manner.
Thanks
-- Christoph
_______________________________________________
barebox mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/barebox
Quoted 9 lines · Christoph Fritz
Hi Tomi
Quoted 6 lines · Tomi Valkeinen
Quoted 1 line · Christoph Fritz
This patch adds omap4 display controller support.
I haven't been following the discussions about this patch, so maybe these have been covered:
Quoted 8 lines
Quoted 2 lines
OMAP2/3/4/5 all have very similar DSS. Why did you go for omap_4_-fb driver, not omap-fb driver?
I'm currently only having an omap4 board here. But I could rename the driver to the more generic name "omap-fb" and explain in the KCONFIG that it currently only covers omap4 devices.
So we could go from there up to all other OMAP devices. Okay?
That sounds good to me.
Tomi
_______________________________________________
barebox mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/barebox