Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1216844 > unrolled thread
| Started by | Tomi Valkeinen <tomi.valkeinen@ti.com> |
|---|---|
| First post | 2015-09-01 15:30 +0200 |
| Last post | 2015-09-02 14:50 +0200 |
| Articles | 4 — 2 participants |
Back to article view | Back to linux.kernel
This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by
below is the oldest one visible, not the original post.
Re: [PATCH v2 1/3] staging: sm7xxfb: move sm712fb out of staging Tomi Valkeinen <tomi.valkeinen@ti.com> - 2015-09-01 15:30 +0200
Re: [PATCH v2 1/3] staging: sm7xxfb: move sm712fb out of staging Sudip Mukherjee <sudipm.mukherjee@gmail.com> - 2015-09-01 16:00 +0200
Re: [PATCH v2 1/3] staging: sm7xxfb: move sm712fb out of staging Tomi Valkeinen <tomi.valkeinen@ti.com> - 2015-09-02 14:00 +0200
Re: [PATCH v2 1/3] staging: sm7xxfb: move sm712fb out of staging Sudip Mukherjee <sudipm.mukherjee@gmail.com> - 2015-09-02 14:50 +0200
| From | Tomi Valkeinen <tomi.valkeinen@ti.com> |
|---|---|
| Date | 2015-09-01 15:30 +0200 |
| Subject | Re: [PATCH v2 1/3] staging: sm7xxfb: move sm712fb out of staging |
| Message-ID | <q3TNp-7Nw-11@gated-at.bofh.it> |
[Multipart message — attachments visible in raw view] — view raw
On 18/07/15 07:08, Sudip Mukherjee wrote:
> Now since all cleanups are done and the code is ready to be merged lets
> move it out of staging into fbdev location.
Have you considered writing a DRM driver for this? I'm not happy at all
adding new fbdev drivers, as the DRM framework is much better,
supported, and continuously improved. With fbdev you end up with things
like module parameters used to define video modes etc, which is just ugly.
Anyway, some comments below.
> diff --git a/drivers/video/fbdev/sm712.h b/drivers/video/fbdev/sm712.h
> new file mode 100644
> index 0000000..aad1cc4
> --- /dev/null
> +++ b/drivers/video/fbdev/sm712.h
> @@ -0,0 +1,116 @@
> +/*
> + * Silicon Motion SM712 frame buffer device
> + *
> + * Copyright (C) 2006 Silicon Motion Technology Corp.
> + * Authors: Ge Wang, gewang@siliconmotion.com
> + * Boyod boyod.yang@siliconmotion.com.cn
> + *
> + * Copyright (C) 2009 Lemote, Inc.
> + * Author: Wu Zhangjin, wuzhangjin@gmail.com
> + *
> + * This file is subject to the terms and conditions of the GNU General Public
> + * License. See the file COPYING in the main directory of this archive for
> + * more details.
> + */
> +
> +#define FB_ACCEL_SMI_LYNX 88
> +
> +#define SCREEN_X_RES 1024
> +#define SCREEN_Y_RES 600
> +#define SCREEN_BPP 16
> +
> +/*Assume SM712 graphics chip has 4MB VRAM */
> +#define SM712_VIDEOMEMORYSIZE 0x00400000
> +/*Assume SM722 graphics chip has 8MB VRAM */
> +#define SM722_VIDEOMEMORYSIZE 0x00800000
> +
> +#define dac_reg (0x3c8)
> +#define dac_val (0x3c9)
> +
> +extern void __iomem *smtc_regbaseaddress;
Uh, what is that?
> +#define smtc_mmiowb(dat, reg) writeb(dat, smtc_regbaseaddress + reg)
> +
> +#define smtc_mmiorb(reg) readb(smtc_regbaseaddress + reg)
> +
> +#define SIZE_SR00_SR04 (0x04 - 0x00 + 1)
> +#define SIZE_SR10_SR24 (0x24 - 0x10 + 1)
> +#define SIZE_SR30_SR75 (0x75 - 0x30 + 1)
> +#define SIZE_SR80_SR93 (0x93 - 0x80 + 1)
> +#define SIZE_SRA0_SRAF (0xAF - 0xA0 + 1)
> +#define SIZE_GR00_GR08 (0x08 - 0x00 + 1)
> +#define SIZE_AR00_AR14 (0x14 - 0x00 + 1)
> +#define SIZE_CR00_CR18 (0x18 - 0x00 + 1)
> +#define SIZE_CR30_CR4D (0x4D - 0x30 + 1)
> +#define SIZE_CR90_CRA7 (0xA7 - 0x90 + 1)
> +
> +static inline void smtc_crtcw(int reg, int val)
> +{
> + smtc_mmiowb(reg, 0x3d4);
> + smtc_mmiowb(val, 0x3d5);
> +}
> +
> +static inline void smtc_grphw(int reg, int val)
> +{
> + smtc_mmiowb(reg, 0x3ce);
> + smtc_mmiowb(val, 0x3cf);
> +}
> +
> +static inline void smtc_attrw(int reg, int val)
> +{
> + smtc_mmiorb(0x3da);
> + smtc_mmiowb(reg, 0x3c0);
> + smtc_mmiorb(0x3c1);
> + smtc_mmiowb(val, 0x3c0);
> +}
> +
> +static inline void smtc_seqw(int reg, int val)
> +{
> + smtc_mmiowb(reg, 0x3c4);
> + smtc_mmiowb(val, 0x3c5);
> +}
> +
> +static inline unsigned int smtc_seqr(int reg)
> +{
> + smtc_mmiowb(reg, 0x3c4);
> + return smtc_mmiorb(0x3c5);
> +}
There's quite a lot of magic numbers there, and the same continues
through the driver. You should use defines to assign symbolic names for
most of the numbers.
> +
> +/* The next structure holds all information relevant for a specific video mode.
> + */
> +
> +struct modeinit {
> + int mmsizex;
> + int mmsizey;
> + int bpp;
> + int hz;
> + unsigned char init_misc;
> + unsigned char init_sr00_sr04[SIZE_SR00_SR04];
> + unsigned char init_sr10_sr24[SIZE_SR10_SR24];
> + unsigned char init_sr30_sr75[SIZE_SR30_SR75];
> + unsigned char init_sr80_sr93[SIZE_SR80_SR93];
> + unsigned char init_sra0_sraf[SIZE_SRA0_SRAF];
> + unsigned char init_gr00_gr08[SIZE_GR00_GR08];
> + unsigned char init_ar00_ar14[SIZE_AR00_AR14];
> + unsigned char init_cr00_cr18[SIZE_CR00_CR18];
> + unsigned char init_cr30_cr4d[SIZE_CR30_CR4D];
> + unsigned char init_cr90_cra7[SIZE_CR90_CRA7];
> +};
> +
> +#ifdef __BIG_ENDIAN
> +#define pal_rgb(r, g, b, val) (((r & 0xf800) >> 8) | \
> + ((g & 0xe000) >> 13) | \
> + ((g & 0x1c00) << 3) | \
> + ((b & 0xf800) >> 3))
> +#define big_addr 0x800000
> +#define mmio_addr 0x00800000
> +#define seqw17() smtc_seqw(0x17, 0x30)
> +#define big_pixel_depth(p, d) {if (p == 24) {p = 32; d = 32; } }
> +#define big_swap(p) ((p & 0xff00ff00 >> 8) | (p & 0x00ff00ff << 8))
> +#else
> +#define pal_rgb(r, g, b, val) val
> +#define big_addr 0
> +#define mmio_addr 0x00c00000
> +#define seqw17() do { } while (0)
> +#define big_pixel_depth(p, d) do { } while (0)
> +#define big_swap(p) p
> +#endif
> diff --git a/drivers/video/fbdev/sm712fb.c b/drivers/video/fbdev/sm712fb.c
> new file mode 100644
> index 0000000..629bfa2
> --- /dev/null
> +++ b/drivers/video/fbdev/sm712fb.c
> @@ -0,0 +1,1653 @@
> +/*
> + * Silicon Motion SM7XX frame buffer device
> + *
> + * Copyright (C) 2006 Silicon Motion Technology Corp.
> + * Authors: Ge Wang, gewang@siliconmotion.com
> + * Boyod boyod.yang@siliconmotion.com.cn
> + *
> + * Copyright (C) 2009 Lemote, Inc.
> + * Author: Wu Zhangjin, wuzhangjin@gmail.com
> + *
> + * Copyright (C) 2011 Igalia, S.L.
> + * Author: Javier M. Mellid <jmunhoz@igalia.com>
> + *
> + * This file is subject to the terms and conditions of the GNU General Public
> + * License. See the file COPYING in the main directory of this archive for
> + * more details.
> + *
> + * Framebuffer driver for Silicon Motion SM710, SM712, SM721 and SM722 chips
> + */
> +
> +#include <linux/io.h>
> +#include <linux/fb.h>
> +#include <linux/pci.h>
> +#include <linux/init.h>
> +#include <linux/slab.h>
> +#include <linux/uaccess.h>
> +#include <linux/module.h>
> +#include <linux/console.h>
> +#include <linux/screen_info.h>
> +
> +#ifdef CONFIG_PM
> +#include <linux/pm.h>
> +#endif
> +
> +#include "sm712.h"
> +
> +/*
> +* Private structure
> +*/
> +struct smtcfb_info {
> + struct pci_dev *pdev;
> + struct fb_info *fb;
> + u16 chip_id;
> + u8 chip_rev_id;
> +
> + void __iomem *lfb; /* linear frame buffer */
> + void __iomem *dp_regs; /* drawing processor control regs */
> + void __iomem *vp_regs; /* video processor control regs */
> + void __iomem *cp_regs; /* capture processor control regs */
> + void __iomem *mmio; /* memory map IO port */
> +
> + u_int width;
> + u_int height;
> + u_int hz;
> +
> + u32 colreg[17];
> +};
> +
> +void __iomem *smtc_regbaseaddress; /* Memory Map IO starting address */
You can't have globals like this in the driver, they must be inside the
per-device data. Just think what happens if someone has two of these
devices.
> +
> +static struct fb_var_screeninfo smtcfb_var = {
> + .xres = 1024,
> + .yres = 600,
> + .xres_virtual = 1024,
> + .yres_virtual = 600,
> + .bits_per_pixel = 16,
> + .red = {16, 8, 0},
> + .green = {8, 8, 0},
> + .blue = {0, 8, 0},
> + .activate = FB_ACTIVATE_NOW,
> + .height = -1,
> + .width = -1,
> + .vmode = FB_VMODE_NONINTERLACED,
> + .nonstd = 0,
> + .accel_flags = FB_ACCELF_TEXT,
> +};
> +
> +static struct fb_fix_screeninfo smtcfb_fix = {
> + .id = "smXXXfb",
> + .type = FB_TYPE_PACKED_PIXELS,
> + .visual = FB_VISUAL_TRUECOLOR,
> + .line_length = 800 * 3,
> + .accel = FB_ACCEL_SMI_LYNX,
> + .type_aux = 0,
> + .xpanstep = 0,
> + .ypanstep = 0,
> + .ywrapstep = 0,
> +};
These should be const.
> +
> +struct vesa_mode {
> + char index[6];
> + u16 lfb_width;
> + u16 lfb_height;
> + u16 lfb_depth;
> +};
> +
> +static const struct vesa_mode vesa_mode_table[] = {
> + {"0x301", 640, 480, 8},
> + {"0x303", 800, 600, 8},
> + {"0x305", 1024, 768, 8},
> + {"0x307", 1280, 1024, 8},
> +
> + {"0x311", 640, 480, 16},
> + {"0x314", 800, 600, 16},
> + {"0x317", 1024, 768, 16},
> + {"0x31A", 1280, 1024, 16},
> +
> + {"0x312", 640, 480, 24},
> + {"0x315", 800, 600, 24},
> + {"0x318", 1024, 768, 24},
> + {"0x31B", 1280, 1024, 24},
> +};
We have "vesa_modes" in include/linux/fb.h. What is the above table for?
> +
> +/**********************************************************************
> + SM712 Mode table.
> + **********************************************************************/
> +static const struct modeinit vgamode[] = {
> + {
> + /* mode#0: 640 x 480 16Bpp 60Hz */
> + 640, 480, 16, 60,
> + /* Init_MISC */
> + 0xE3,
> + { /* Init_SR0_SR4 */
> + 0x03, 0x01, 0x0F, 0x00, 0x0E,
> + },
> + { /* Init_SR10_SR24 */
> + 0xFF, 0xBE, 0xEF, 0xFF, 0x00, 0x0E, 0x17, 0x2C,
> + 0x99, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
> + 0xC4, 0x30, 0x02, 0x01, 0x01,
> + },
> + { /* Init_SR30_SR75 */
> + 0x32, 0x03, 0xA0, 0x09, 0xC0, 0x32, 0x32, 0x32,
> + 0x32, 0x32, 0x32, 0x32, 0x00, 0x00, 0x03, 0xFF,
> + 0x00, 0xFC, 0x00, 0x00, 0x20, 0x18, 0x00, 0xFC,
> + 0x20, 0x0C, 0x44, 0x20, 0x00, 0x32, 0x32, 0x32,
> + 0x04, 0x24, 0x63, 0x4F, 0x52, 0x0B, 0xDF, 0xEA,
> + 0x04, 0x50, 0x19, 0x32, 0x32, 0x00, 0x00, 0x32,
> + 0x01, 0x80, 0x7E, 0x1A, 0x1A, 0x00, 0x00, 0x00,
> + 0x50, 0x03, 0x74, 0x14, 0x07, 0x82, 0x07, 0x04,
> + 0x00, 0x45, 0x30, 0x30, 0x40, 0x30,
> + },
> + { /* Init_SR80_SR93 */
> + 0xFF, 0x07, 0x00, 0x6F, 0x7F, 0x7F, 0xFF, 0x32,
> + 0xF7, 0x00, 0x00, 0x00, 0xEF, 0xFF, 0x32, 0x32,
> + 0x00, 0x00, 0x00, 0x00,
> + },
> + { /* Init_SRA0_SRAF */
> + 0x00, 0xFF, 0xBF, 0xFF, 0xFF, 0xED, 0xED, 0xED,
> + 0x7B, 0xFF, 0xFF, 0xFF, 0xBF, 0xEF, 0xFF, 0xDF,
> + },
> + { /* Init_GR00_GR08 */
> + 0x00, 0x00, 0x00, 0x00, 0x00, 0x40, 0x05, 0x0F,
> + 0xFF,
> + },
> + { /* Init_AR00_AR14 */
> + 0x00, 0x01, 0x02, 0x03, 0x04, 0x05, 0x06, 0x07,
> + 0x08, 0x09, 0x0A, 0x0B, 0x0C, 0x0D, 0x0E, 0x0F,
> + 0x41, 0x00, 0x0F, 0x00, 0x00,
> + },
> + { /* Init_CR00_CR18 */
> + 0x5F, 0x4F, 0x4F, 0x00, 0x53, 0x1F, 0x0B, 0x3E,
> + 0x00, 0x40, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
> + 0xEA, 0x0C, 0xDF, 0x50, 0x40, 0xDF, 0x00, 0xE3,
> + 0xFF,
> + },
> + { /* Init_CR30_CR4D */
> + 0x00, 0x00, 0x00, 0x00, 0x00, 0x55, 0x03, 0x20,
> + 0x00, 0x00, 0x00, 0x40, 0x00, 0xE7, 0xFF, 0xFD,
> + 0x5F, 0x4F, 0x00, 0x54, 0x00, 0x0B, 0xDF, 0x00,
> + 0xEA, 0x0C, 0x2E, 0x00, 0x4F, 0xDF,
> + },
> + { /* Init_CR90_CRA7 */
> + 0x56, 0xDD, 0x5E, 0xEA, 0x87, 0x44, 0x8F, 0x55,
> + 0x0A, 0x8F, 0x55, 0x0A, 0x00, 0x00, 0x18, 0x00,
> + 0x11, 0x10, 0x0B, 0x0A, 0x0A, 0x0A, 0x0A, 0x00,
> + },
> + },
> + {
> + /* mode#1: 640 x 480 24Bpp 60Hz */
> + 640, 480, 24, 60,
> + /* Init_MISC */
> + 0xE3,
> + { /* Init_SR0_SR4 */
> + 0x03, 0x01, 0x0F, 0x00, 0x0E,
> + },
> + { /* Init_SR10_SR24 */
> + 0xFF, 0xBE, 0xEF, 0xFF, 0x00, 0x0E, 0x17, 0x2C,
> + 0x99, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
> + 0xC4, 0x30, 0x02, 0x01, 0x01,
> + },
> + { /* Init_SR30_SR75 */
> + 0x32, 0x03, 0xA0, 0x09, 0xC0, 0x32, 0x32, 0x32,
> + 0x32, 0x32, 0x32, 0x32, 0x00, 0x00, 0x03, 0xFF,
> + 0x00, 0xFC, 0x00, 0x00, 0x20, 0x18, 0x00, 0xFC,
> + 0x20, 0x0C, 0x44, 0x20, 0x00, 0x32, 0x32, 0x32,
> + 0x04, 0x24, 0x63, 0x4F, 0x52, 0x0B, 0xDF, 0xEA,
> + 0x04, 0x50, 0x19, 0x32, 0x32, 0x00, 0x00, 0x32,
> + 0x01, 0x80, 0x7E, 0x1A, 0x1A, 0x00, 0x00, 0x00,
> + 0x50, 0x03, 0x74, 0x14, 0x07, 0x82, 0x07, 0x04,
> + 0x00, 0x45, 0x30, 0x30, 0x40, 0x30,
> + },
> + { /* Init_SR80_SR93 */
> + 0xFF, 0x07, 0x00, 0x6F, 0x7F, 0x7F, 0xFF, 0x32,
> + 0xF7, 0x00, 0x00, 0x00, 0xEF, 0xFF, 0x32, 0x32,
> + 0x00, 0x00, 0x00, 0x00,
> + },
> + { /* Init_SRA0_SRAF */
> + 0x00, 0xFF, 0xBF, 0xFF, 0xFF, 0xED, 0xED, 0xED,
> + 0x7B, 0xFF, 0xFF, 0xFF, 0xBF, 0xEF, 0xFF, 0xDF,
> + },
> + { /* Init_GR00_GR08 */
> + 0x00, 0x00, 0x00, 0x00, 0x00, 0x40, 0x05, 0x0F,
> + 0xFF,
> + },
> + { /* Init_AR00_AR14 */
> + 0x00, 0x01, 0x02, 0x03, 0x04, 0x05, 0x06, 0x07,
> + 0x08, 0x09, 0x0A, 0x0B, 0x0C, 0x0D, 0x0E, 0x0F,
> + 0x41, 0x00, 0x0F, 0x00, 0x00,
> + },
> + { /* Init_CR00_CR18 */
> + 0x5F, 0x4F, 0x4F, 0x00, 0x53, 0x1F, 0x0B, 0x3E,
> + 0x00, 0x40, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
> + 0xEA, 0x0C, 0xDF, 0x50, 0x40, 0xDF, 0x00, 0xE3,
> + 0xFF,
> + },
> + { /* Init_CR30_CR4D */
> + 0x00, 0x00, 0x00, 0x00, 0x00, 0x55, 0x03, 0x20,
> + 0x00, 0x00, 0x00, 0x40, 0x00, 0xE7, 0xFF, 0xFD,
> + 0x5F, 0x4F, 0x00, 0x54, 0x00, 0x0B, 0xDF, 0x00,
> + 0xEA, 0x0C, 0x2E, 0x00, 0x4F, 0xDF,
> + },
> + { /* Init_CR90_CRA7 */
> + 0x56, 0xDD, 0x5E, 0xEA, 0x87, 0x44, 0x8F, 0x55,
> + 0x0A, 0x8F, 0x55, 0x0A, 0x00, 0x00, 0x18, 0x00,
> + 0x11, 0x10, 0x0B, 0x0A, 0x0A, 0x0A, 0x0A, 0x00,
> + },
> + },
> + {
> + /* mode#0: 640 x 480 32Bpp 60Hz */
> + 640, 480, 32, 60,
> + /* Init_MISC */
> + 0xE3,
> + { /* Init_SR0_SR4 */
> + 0x03, 0x01, 0x0F, 0x00, 0x0E,
> + },
> + { /* Init_SR10_SR24 */
> + 0xFF, 0xBE, 0xEF, 0xFF, 0x00, 0x0E, 0x17, 0x2C,
> + 0x99, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
> + 0xC4, 0x30, 0x02, 0x01, 0x01,
> + },
> + { /* Init_SR30_SR75 */
> + 0x32, 0x03, 0xA0, 0x09, 0xC0, 0x32, 0x32, 0x32,
> + 0x32, 0x32, 0x32, 0x32, 0x00, 0x00, 0x03, 0xFF,
> + 0x00, 0xFC, 0x00, 0x00, 0x20, 0x18, 0x00, 0xFC,
> + 0x20, 0x0C, 0x44, 0x20, 0x00, 0x32, 0x32, 0x32,
> + 0x04, 0x24, 0x63, 0x4F, 0x52, 0x0B, 0xDF, 0xEA,
> + 0x04, 0x50, 0x19, 0x32, 0x32, 0x00, 0x00, 0x32,
> + 0x01, 0x80, 0x7E, 0x1A, 0x1A, 0x00, 0x00, 0x00,
> + 0x50, 0x03, 0x74, 0x14, 0x07, 0x82, 0x07, 0x04,
> + 0x00, 0x45, 0x30, 0x30, 0x40, 0x30,
> + },
> + { /* Init_SR80_SR93 */
> + 0xFF, 0x07, 0x00, 0x6F, 0x7F, 0x7F, 0xFF, 0x32,
> + 0xF7, 0x00, 0x00, 0x00, 0xEF, 0xFF, 0x32, 0x32,
> + 0x00, 0x00, 0x00, 0x00,
> + },
> + { /* Init_SRA0_SRAF */
> + 0x00, 0xFF, 0xBF, 0xFF, 0xFF, 0xED, 0xED, 0xED,
> + 0x7B, 0xFF, 0xFF, 0xFF, 0xBF, 0xEF, 0xFF, 0xDF,
> + },
> + { /* Init_GR00_GR08 */
> + 0x00, 0x00, 0x00, 0x00, 0x00, 0x40, 0x05, 0x0F,
> + 0xFF,
> + },
> + { /* Init_AR00_AR14 */
> + 0x00, 0x01, 0x02, 0x03, 0x04, 0x05, 0x06, 0x07,
> + 0x08, 0x09, 0x0A, 0x0B, 0x0C, 0x0D, 0x0E, 0x0F,
> + 0x41, 0x00, 0x0F, 0x00, 0x00,
> + },
> + { /* Init_CR00_CR18 */
> + 0x5F, 0x4F, 0x4F, 0x00, 0x53, 0x1F, 0x0B, 0x3E,
> + 0x00, 0x40, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
> + 0xEA, 0x0C, 0xDF, 0x50, 0x40, 0xDF, 0x00, 0xE3,
> + 0xFF,
> + },
> + { /* Init_CR30_CR4D */
> + 0x00, 0x00, 0x00, 0x00, 0x00, 0x55, 0x03, 0x20,
> + 0x00, 0x00, 0x00, 0x40, 0x00, 0xE7, 0xFF, 0xFD,
> + 0x5F, 0x4F, 0x00, 0x54, 0x00, 0x0B, 0xDF, 0x00,
> + 0xEA, 0x0C, 0x2E, 0x00, 0x4F, 0xDF,
> + },
> + { /* Init_CR90_CRA7 */
> + 0x56, 0xDD, 0x5E, 0xEA, 0x87, 0x44, 0x8F, 0x55,
> + 0x0A, 0x8F, 0x55, 0x0A, 0x00, 0x00, 0x18, 0x00,
> + 0x11, 0x10, 0x0B, 0x0A, 0x0A, 0x0A, 0x0A, 0x00,
> + },
> + },
> +
> + { /* mode#2: 800 x 600 16Bpp 60Hz */
> + 800, 600, 16, 60,
> + /* Init_MISC */
> + 0x2B,
> + { /* Init_SR0_SR4 */
> + 0x03, 0x01, 0x0F, 0x03, 0x0E,
> + },
> + { /* Init_SR10_SR24 */
> + 0xFF, 0xBE, 0xEE, 0xFF, 0x00, 0x0E, 0x17, 0x2C,
> + 0x99, 0x00, 0x01, 0x00, 0x00, 0x00, 0x00, 0x00,
> + 0xC4, 0x30, 0x02, 0x01, 0x01,
> + },
> + { /* Init_SR30_SR75 */
> + 0x34, 0x03, 0x20, 0x09, 0xC0, 0x24, 0x24, 0x24,
> + 0x24, 0x24, 0x24, 0x24, 0x00, 0x00, 0x03, 0xFF,
> + 0x00, 0xFC, 0x00, 0x00, 0x20, 0x38, 0x00, 0xFC,
> + 0x20, 0x0C, 0x44, 0x20, 0x00, 0x24, 0x24, 0x24,
> + 0x04, 0x48, 0x83, 0x63, 0x68, 0x72, 0x57, 0x58,
> + 0x04, 0x55, 0x59, 0x24, 0x24, 0x00, 0x00, 0x24,
> + 0x01, 0x80, 0x7A, 0x1A, 0x1A, 0x00, 0x00, 0x00,
> + 0x50, 0x03, 0x74, 0x14, 0x1C, 0x85, 0x35, 0x13,
> + 0x02, 0x45, 0x30, 0x35, 0x40, 0x20,
> + },
> + { /* Init_SR80_SR93 */
> + 0x00, 0x00, 0x00, 0x6F, 0x7F, 0x7F, 0xFF, 0x24,
> + 0x00, 0x00, 0x00, 0x00, 0xFF, 0xFF, 0x24, 0x24,
> + 0x00, 0x00, 0x00, 0x00,
> + },
> + { /* Init_SRA0_SRAF */
> + 0x00, 0xFF, 0xBF, 0xFF, 0xFF, 0xED, 0xED, 0xED,
> + 0x7B, 0xFF, 0xFF, 0xFF, 0xBF, 0xEF, 0xBF, 0xDF,
> + },
> + { /* Init_GR00_GR08 */
> + 0x00, 0x00, 0x00, 0x00, 0x00, 0x40, 0x05, 0x0F,
> + 0xFF,
> + },
> + { /* Init_AR00_AR14 */
> + 0x00, 0x01, 0x02, 0x03, 0x04, 0x05, 0x06, 0x07,
> + 0x08, 0x09, 0x0A, 0x0B, 0x0C, 0x0D, 0x0E, 0x0F,
> + 0x41, 0x00, 0x0F, 0x00, 0x00,
> + },
> + { /* Init_CR00_CR18 */
> + 0x7F, 0x63, 0x63, 0x00, 0x68, 0x18, 0x72, 0xF0,
> + 0x00, 0x60, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
> + 0x58, 0x0C, 0x57, 0x64, 0x40, 0x57, 0x00, 0xE3,
> + 0xFF,
> + },
> + { /* Init_CR30_CR4D */
> + 0x00, 0x00, 0x00, 0x00, 0x00, 0x33, 0x03, 0x20,
> + 0x00, 0x00, 0x00, 0x40, 0x00, 0xE7, 0xBF, 0xFD,
> + 0x7F, 0x63, 0x00, 0x69, 0x18, 0x72, 0x57, 0x00,
> + 0x58, 0x0C, 0xE0, 0x20, 0x63, 0x57,
> + },
> + { /* Init_CR90_CRA7 */
> + 0x56, 0x4B, 0x5E, 0x55, 0x86, 0x9D, 0x8E, 0xAA,
> + 0xDB, 0x2A, 0xDF, 0x33, 0x00, 0x00, 0x18, 0x00,
> + 0x20, 0x1F, 0x1A, 0x19, 0x0F, 0x0F, 0x0F, 0x00,
> + },
> + },
> + { /* mode#3: 800 x 600 24Bpp 60Hz */
> + 800, 600, 24, 60,
> + 0x2B,
> + { /* Init_SR0_SR4 */
> + 0x03, 0x01, 0x0F, 0x03, 0x0E,
> + },
> + { /* Init_SR10_SR24 */
> + 0xFF, 0xBE, 0xEE, 0xFF, 0x00, 0x0E, 0x17, 0x2C,
> + 0x99, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
> + 0xC4, 0x30, 0x02, 0x01, 0x01,
> + },
> + { /* Init_SR30_SR75 */
> + 0x36, 0x03, 0x20, 0x09, 0xC0, 0x36, 0x36, 0x36,
> + 0x36, 0x36, 0x36, 0x36, 0x00, 0x00, 0x03, 0xFF,
> + 0x00, 0xFC, 0x00, 0x00, 0x20, 0x18, 0x00, 0xFC,
> + 0x20, 0x0C, 0x44, 0x20, 0x00, 0x36, 0x36, 0x36,
> + 0x04, 0x48, 0x83, 0x63, 0x68, 0x72, 0x57, 0x58,
> + 0x04, 0x55, 0x59, 0x36, 0x36, 0x00, 0x00, 0x36,
> + 0x01, 0x80, 0x7E, 0x1A, 0x1A, 0x00, 0x00, 0x00,
> + 0x50, 0x03, 0x74, 0x14, 0x1C, 0x85, 0x35, 0x13,
> + 0x02, 0x45, 0x30, 0x30, 0x40, 0x20,
> + },
> + { /* Init_SR80_SR93 */
> + 0xFF, 0x07, 0x00, 0x6F, 0x7F, 0x7F, 0xFF, 0x36,
> + 0xF7, 0x00, 0x00, 0x00, 0xEF, 0xFF, 0x36, 0x36,
> + 0x00, 0x00, 0x00, 0x00,
> + },
> + { /* Init_SRA0_SRAF */
> + 0x00, 0xFF, 0xBF, 0xFF, 0xFF, 0xED, 0xED, 0xED,
> + 0x7B, 0xFF, 0xFF, 0xFF, 0xBF, 0xEF, 0xBF, 0xDF,
> + },
> + { /* Init_GR00_GR08 */
> + 0x00, 0x00, 0x00, 0x00, 0x00, 0x40, 0x05, 0x0F,
> + 0xFF,
> + },
> + { /* Init_AR00_AR14 */
> + 0x00, 0x01, 0x02, 0x03, 0x04, 0x05, 0x06, 0x07,
> + 0x08, 0x09, 0x0A, 0x0B, 0x0C, 0x0D, 0x0E, 0x0F,
> + 0x41, 0x00, 0x0F, 0x00, 0x00,
> + },
> + { /* Init_CR00_CR18 */
> + 0x7F, 0x63, 0x63, 0x00, 0x68, 0x18, 0x72, 0xF0,
> + 0x00, 0x60, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
> + 0x58, 0x0C, 0x57, 0x64, 0x40, 0x57, 0x00, 0xE3,
> + 0xFF,
> + },
> + { /* Init_CR30_CR4D */
> + 0x00, 0x00, 0x00, 0x00, 0x00, 0x33, 0x03, 0x20,
> + 0x00, 0x00, 0x00, 0x40, 0x00, 0xE7, 0xBF, 0xFD,
> + 0x7F, 0x63, 0x00, 0x69, 0x18, 0x72, 0x57, 0x00,
> + 0x58, 0x0C, 0xE0, 0x20, 0x63, 0x57,
> + },
> + { /* Init_CR90_CRA7 */
> + 0x56, 0x4B, 0x5E, 0x55, 0x86, 0x9D, 0x8E, 0xAA,
> + 0xDB, 0x2A, 0xDF, 0x33, 0x00, 0x00, 0x18, 0x00,
> + 0x20, 0x1F, 0x1A, 0x19, 0x0F, 0x0F, 0x0F, 0x00,
> + },
> + },
> + { /* mode#7: 800 x 600 32Bpp 60Hz */
> + 800, 600, 32, 60,
> + /* Init_MISC */
> + 0x2B,
> + { /* Init_SR0_SR4 */
> + 0x03, 0x01, 0x0F, 0x03, 0x0E,
> + },
> + { /* Init_SR10_SR24 */
> + 0xFF, 0xBE, 0xEE, 0xFF, 0x00, 0x0E, 0x17, 0x2C,
> + 0x99, 0x00, 0x01, 0x00, 0x00, 0x00, 0x00, 0x00,
> + 0xC4, 0x30, 0x02, 0x01, 0x01,
> + },
> + { /* Init_SR30_SR75 */
> + 0x34, 0x03, 0x20, 0x09, 0xC0, 0x24, 0x24, 0x24,
> + 0x24, 0x24, 0x24, 0x24, 0x00, 0x00, 0x03, 0xFF,
> + 0x00, 0xFC, 0x00, 0x00, 0x20, 0x38, 0x00, 0xFC,
> + 0x20, 0x0C, 0x44, 0x20, 0x00, 0x24, 0x24, 0x24,
> + 0x04, 0x48, 0x83, 0x63, 0x68, 0x72, 0x57, 0x58,
> + 0x04, 0x55, 0x59, 0x24, 0x24, 0x00, 0x00, 0x24,
> + 0x01, 0x80, 0x7A, 0x1A, 0x1A, 0x00, 0x00, 0x00,
> + 0x50, 0x03, 0x74, 0x14, 0x1C, 0x85, 0x35, 0x13,
> + 0x02, 0x45, 0x30, 0x35, 0x40, 0x20,
> + },
> + { /* Init_SR80_SR93 */
> + 0x00, 0x00, 0x00, 0x6F, 0x7F, 0x7F, 0xFF, 0x24,
> + 0x00, 0x00, 0x00, 0x00, 0xFF, 0xFF, 0x24, 0x24,
> + 0x00, 0x00, 0x00, 0x00,
> + },
> + { /* Init_SRA0_SRAF */
> + 0x00, 0xFF, 0xBF, 0xFF, 0xFF, 0xED, 0xED, 0xED,
> + 0x7B, 0xFF, 0xFF, 0xFF, 0xBF, 0xEF, 0xBF, 0xDF,
> + },
> + { /* Init_GR00_GR08 */
> + 0x00, 0x00, 0x00, 0x00, 0x00, 0x40, 0x05, 0x0F,
> + 0xFF,
> + },
> + { /* Init_AR00_AR14 */
> + 0x00, 0x01, 0x02, 0x03, 0x04, 0x05, 0x06, 0x07,
> + 0x08, 0x09, 0x0A, 0x0B, 0x0C, 0x0D, 0x0E, 0x0F,
> + 0x41, 0x00, 0x0F, 0x00, 0x00,
> + },
> + { /* Init_CR00_CR18 */
> + 0x7F, 0x63, 0x63, 0x00, 0x68, 0x18, 0x72, 0xF0,
> + 0x00, 0x60, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
> + 0x58, 0x0C, 0x57, 0x64, 0x40, 0x57, 0x00, 0xE3,
> + 0xFF,
> + },
> + { /* Init_CR30_CR4D */
> + 0x00, 0x00, 0x00, 0x00, 0x00, 0x33, 0x03, 0x20,
> + 0x00, 0x00, 0x00, 0x40, 0x00, 0xE7, 0xBF, 0xFD,
> + 0x7F, 0x63, 0x00, 0x69, 0x18, 0x72, 0x57, 0x00,
> + 0x58, 0x0C, 0xE0, 0x20, 0x63, 0x57,
> + },
> + { /* Init_CR90_CRA7 */
> + 0x56, 0x4B, 0x5E, 0x55, 0x86, 0x9D, 0x8E, 0xAA,
> + 0xDB, 0x2A, 0xDF, 0x33, 0x00, 0x00, 0x18, 0x00,
> + 0x20, 0x1F, 0x1A, 0x19, 0x0F, 0x0F, 0x0F, 0x00,
> + },
> + },
> + /* We use 1024x768 table to light 1024x600 panel for lemote */
> + { /* mode#4: 1024 x 600 16Bpp 60Hz */
> + 1024, 600, 16, 60,
> + /* Init_MISC */
> + 0xEB,
> + { /* Init_SR0_SR4 */
> + 0x03, 0x01, 0x0F, 0x00, 0x0E,
> + },
> + { /* Init_SR10_SR24 */
> + 0xC8, 0x40, 0x14, 0x60, 0x00, 0x0A, 0x17, 0x20,
> + 0x51, 0x00, 0x01, 0x00, 0x00, 0x00, 0x00, 0x00,
> + 0xC4, 0x30, 0x02, 0x00, 0x01,
> + },
> + { /* Init_SR30_SR75 */
> + 0x22, 0x03, 0x24, 0x09, 0xC0, 0x22, 0x22, 0x22,
> + 0x22, 0x22, 0x22, 0x22, 0x00, 0x00, 0x03, 0xFF,
> + 0x00, 0xFC, 0x00, 0x00, 0x20, 0x18, 0x00, 0xFC,
> + 0x20, 0x0C, 0x44, 0x20, 0x00, 0x22, 0x22, 0x22,
> + 0x06, 0x68, 0xA7, 0x7F, 0x83, 0x24, 0xFF, 0x03,
> + 0x00, 0x60, 0x59, 0x22, 0x22, 0x00, 0x00, 0x22,
> + 0x01, 0x80, 0x7A, 0x1A, 0x1A, 0x00, 0x00, 0x00,
> + 0x50, 0x03, 0x16, 0x02, 0x0D, 0x82, 0x09, 0x02,
> + 0x04, 0x45, 0x3F, 0x30, 0x40, 0x20,
> + },
> + { /* Init_SR80_SR93 */
> + 0xFF, 0x07, 0x00, 0xFF, 0xFF, 0xFF, 0xFF, 0x3A,
> + 0xF7, 0x00, 0x00, 0x00, 0xFF, 0xFF, 0x3A, 0x3A,
> + 0x00, 0x00, 0x00, 0x00,
> + },
> + { /* Init_SRA0_SRAF */
> + 0x00, 0xFB, 0x9F, 0x01, 0x00, 0xED, 0xED, 0xED,
> + 0x7B, 0xFB, 0xFF, 0xFF, 0x97, 0xEF, 0xBF, 0xDF,
> + },
> + { /* Init_GR00_GR08 */
> + 0x00, 0x00, 0x00, 0x00, 0x00, 0x40, 0x05, 0x0F,
> + 0xFF,
> + },
> + { /* Init_AR00_AR14 */
> + 0x00, 0x01, 0x02, 0x03, 0x04, 0x05, 0x06, 0x07,
> + 0x08, 0x09, 0x0A, 0x0B, 0x0C, 0x0D, 0x0E, 0x0F,
> + 0x41, 0x00, 0x0F, 0x00, 0x00,
> + },
> + { /* Init_CR00_CR18 */
> + 0xA3, 0x7F, 0x7F, 0x00, 0x85, 0x16, 0x24, 0xF5,
> + 0x00, 0x60, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
> + 0x03, 0x09, 0xFF, 0x80, 0x40, 0xFF, 0x00, 0xE3,
> + 0xFF,
> + },
> + { /* Init_CR30_CR4D */
> + 0x00, 0x00, 0x00, 0x00, 0x00, 0x80, 0x02, 0x20,
> + 0x00, 0x00, 0x00, 0x40, 0x00, 0xFF, 0xBF, 0xFF,
> + 0xA3, 0x7F, 0x00, 0x82, 0x0b, 0x6f, 0x57, 0x00,
> + 0x5c, 0x0f, 0xE0, 0xe0, 0x7F, 0x57,
> + },
> + { /* Init_CR90_CRA7 */
> + 0x55, 0xD9, 0x5D, 0xE1, 0x86, 0x1B, 0x8E, 0x26,
> + 0xDA, 0x8D, 0xDE, 0x94, 0x00, 0x00, 0x18, 0x00,
> + 0x03, 0x03, 0x03, 0x03, 0x03, 0x03, 0x15, 0x03,
> + },
> + },
> + { /* mode#5: 1024 x 768 24Bpp 60Hz */
> + 1024, 768, 24, 60,
> + /* Init_MISC */
> + 0xEB,
> + { /* Init_SR0_SR4 */
> + 0x03, 0x01, 0x0F, 0x03, 0x0E,
> + },
> + { /* Init_SR10_SR24 */
> + 0xF3, 0xB6, 0xC0, 0xDD, 0x00, 0x0E, 0x17, 0x2C,
> + 0x99, 0x02, 0x01, 0x00, 0x00, 0x00, 0x00, 0x00,
> + 0xC4, 0x30, 0x02, 0x01, 0x01,
> + },
> + { /* Init_SR30_SR75 */
> + 0x38, 0x03, 0x20, 0x09, 0xC0, 0x3A, 0x3A, 0x3A,
> + 0x3A, 0x3A, 0x3A, 0x3A, 0x00, 0x00, 0x03, 0xFF,
> + 0x00, 0xFC, 0x00, 0x00, 0x20, 0x18, 0x00, 0xFC,
> + 0x20, 0x0C, 0x44, 0x20, 0x00, 0x00, 0x00, 0x3A,
> + 0x06, 0x68, 0xA7, 0x7F, 0x83, 0x24, 0xFF, 0x03,
> + 0x00, 0x60, 0x59, 0x3A, 0x3A, 0x00, 0x00, 0x3A,
> + 0x01, 0x80, 0x7E, 0x1A, 0x1A, 0x00, 0x00, 0x00,
> + 0x50, 0x03, 0x74, 0x14, 0x3B, 0x0D, 0x09, 0x02,
> + 0x04, 0x45, 0x30, 0x30, 0x40, 0x20,
> + },
> + { /* Init_SR80_SR93 */
> + 0xFF, 0x07, 0x00, 0xFF, 0xFF, 0xFF, 0xFF, 0x3A,
> + 0xF7, 0x00, 0x00, 0x00, 0xFF, 0xFF, 0x3A, 0x3A,
> + 0x00, 0x00, 0x00, 0x00,
> + },
> + { /* Init_SRA0_SRAF */
> + 0x00, 0xFB, 0x9F, 0x01, 0x00, 0xED, 0xED, 0xED,
> + 0x7B, 0xFB, 0xFF, 0xFF, 0x97, 0xEF, 0xBF, 0xDF,
> + },
> + { /* Init_GR00_GR08 */
> + 0x00, 0x00, 0x00, 0x00, 0x00, 0x40, 0x05, 0x0F,
> + 0xFF,
> + },
> + { /* Init_AR00_AR14 */
> + 0x00, 0x01, 0x02, 0x03, 0x04, 0x05, 0x06, 0x07,
> + 0x08, 0x09, 0x0A, 0x0B, 0x0C, 0x0D, 0x0E, 0x0F,
> + 0x41, 0x00, 0x0F, 0x00, 0x00,
> + },
> + { /* Init_CR00_CR18 */
> + 0xA3, 0x7F, 0x7F, 0x00, 0x85, 0x16, 0x24, 0xF5,
> + 0x00, 0x60, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
> + 0x03, 0x09, 0xFF, 0x80, 0x40, 0xFF, 0x00, 0xE3,
> + 0xFF,
> + },
> + { /* Init_CR30_CR4D */
> + 0x00, 0x00, 0x00, 0x00, 0x00, 0x80, 0x02, 0x20,
> + 0x00, 0x00, 0x00, 0x40, 0x00, 0xFF, 0xBF, 0xFF,
> + 0xA3, 0x7F, 0x00, 0x86, 0x15, 0x24, 0xFF, 0x00,
> + 0x01, 0x07, 0xE5, 0x20, 0x7F, 0xFF,
> + },
> + { /* Init_CR90_CRA7 */
> + 0x55, 0xD9, 0x5D, 0xE1, 0x86, 0x1B, 0x8E, 0x26,
> + 0xDA, 0x8D, 0xDE, 0x94, 0x00, 0x00, 0x18, 0x00,
> + 0x03, 0x03, 0x03, 0x03, 0x03, 0x03, 0x15, 0x03,
> + },
> + },
> + { /* mode#4: 1024 x 768 32Bpp 60Hz */
> + 1024, 768, 32, 60,
> + /* Init_MISC */
> + 0xEB,
> + { /* Init_SR0_SR4 */
> + 0x03, 0x01, 0x0F, 0x03, 0x0E,
> + },
> + { /* Init_SR10_SR24 */
> + 0xF3, 0xB6, 0xC0, 0xDD, 0x00, 0x0E, 0x17, 0x2C,
> + 0x99, 0x02, 0x01, 0x00, 0x00, 0x00, 0x00, 0x00,
> + 0xC4, 0x32, 0x02, 0x01, 0x01,
> + },
> + { /* Init_SR30_SR75 */
> + 0x38, 0x03, 0x20, 0x09, 0xC0, 0x3A, 0x3A, 0x3A,
> + 0x3A, 0x3A, 0x3A, 0x3A, 0x00, 0x00, 0x03, 0xFF,
> + 0x00, 0xFC, 0x00, 0x00, 0x20, 0x18, 0x00, 0xFC,
> + 0x20, 0x0C, 0x44, 0x20, 0x00, 0x00, 0x00, 0x3A,
> + 0x06, 0x68, 0xA7, 0x7F, 0x83, 0x24, 0xFF, 0x03,
> + 0x00, 0x60, 0x59, 0x3A, 0x3A, 0x00, 0x00, 0x3A,
> + 0x01, 0x80, 0x7E, 0x1A, 0x1A, 0x00, 0x00, 0x00,
> + 0x50, 0x03, 0x74, 0x14, 0x3B, 0x0D, 0x09, 0x02,
> + 0x04, 0x45, 0x30, 0x30, 0x40, 0x20,
> + },
> + { /* Init_SR80_SR93 */
> + 0xFF, 0x07, 0x00, 0xFF, 0xFF, 0xFF, 0xFF, 0x3A,
> + 0xF7, 0x00, 0x00, 0x00, 0xFF, 0xFF, 0x3A, 0x3A,
> + 0x00, 0x00, 0x00, 0x00,
> + },
> + { /* Init_SRA0_SRAF */
> + 0x00, 0xFB, 0x9F, 0x01, 0x00, 0xED, 0xED, 0xED,
> + 0x7B, 0xFB, 0xFF, 0xFF, 0x97, 0xEF, 0xBF, 0xDF,
> + },
> + { /* Init_GR00_GR08 */
> + 0x00, 0x00, 0x00, 0x00, 0x00, 0x40, 0x05, 0x0F,
> + 0xFF,
> + },
> + { /* Init_AR00_AR14 */
> + 0x00, 0x01, 0x02, 0x03, 0x04, 0x05, 0x06, 0x07,
> + 0x08, 0x09, 0x0A, 0x0B, 0x0C, 0x0D, 0x0E, 0x0F,
> + 0x41, 0x00, 0x0F, 0x00, 0x00,
> + },
> + { /* Init_CR00_CR18 */
> + 0xA3, 0x7F, 0x7F, 0x00, 0x85, 0x16, 0x24, 0xF5,
> + 0x00, 0x60, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
> + 0x03, 0x09, 0xFF, 0x80, 0x40, 0xFF, 0x00, 0xE3,
> + 0xFF,
> + },
> + { /* Init_CR30_CR4D */
> + 0x00, 0x00, 0x00, 0x00, 0x00, 0x80, 0x02, 0x20,
> + 0x00, 0x00, 0x00, 0x40, 0x00, 0xFF, 0xBF, 0xFF,
> + 0xA3, 0x7F, 0x00, 0x86, 0x15, 0x24, 0xFF, 0x00,
> + 0x01, 0x07, 0xE5, 0x20, 0x7F, 0xFF,
> + },
> + { /* Init_CR90_CRA7 */
> + 0x55, 0xD9, 0x5D, 0xE1, 0x86, 0x1B, 0x8E, 0x26,
> + 0xDA, 0x8D, 0xDE, 0x94, 0x00, 0x00, 0x18, 0x00,
> + 0x03, 0x03, 0x03, 0x03, 0x03, 0x03, 0x15, 0x03,
> + },
> + },
> + { /* mode#6: 320 x 240 16Bpp 60Hz */
> + 320, 240, 16, 60,
> + /* Init_MISC */
> + 0xEB,
> + { /* Init_SR0_SR4 */
> + 0x03, 0x01, 0x0F, 0x03, 0x0E,
> + },
> + { /* Init_SR10_SR24 */
> + 0xF3, 0xB6, 0xC0, 0xDD, 0x00, 0x0E, 0x17, 0x2C,
> + 0x99, 0x02, 0x01, 0x00, 0x00, 0x00, 0x00, 0x00,
> + 0xC4, 0x32, 0x02, 0x01, 0x01,
> + },
> + { /* Init_SR30_SR75 */
> + 0x38, 0x03, 0x20, 0x09, 0xC0, 0x3A, 0x3A, 0x3A,
> + 0x3A, 0x3A, 0x3A, 0x3A, 0x00, 0x00, 0x03, 0xFF,
> + 0x00, 0xFC, 0x00, 0x00, 0x20, 0x18, 0x00, 0xFC,
> + 0x20, 0x0C, 0x44, 0x20, 0x00, 0x00, 0x00, 0x3A,
> + 0x06, 0x68, 0xA7, 0x7F, 0x83, 0x24, 0xFF, 0x03,
> + 0x00, 0x60, 0x59, 0x3A, 0x3A, 0x00, 0x00, 0x3A,
> + 0x01, 0x80, 0x7E, 0x1A, 0x1A, 0x00, 0x00, 0x00,
> + 0x50, 0x03, 0x74, 0x14, 0x08, 0x43, 0x08, 0x43,
> + 0x04, 0x45, 0x30, 0x30, 0x40, 0x20,
> + },
> + { /* Init_SR80_SR93 */
> + 0xFF, 0x07, 0x00, 0xFF, 0xFF, 0xFF, 0xFF, 0x3A,
> + 0xF7, 0x00, 0x00, 0x00, 0xFF, 0xFF, 0x3A, 0x3A,
> + 0x00, 0x00, 0x00, 0x00,
> + },
> + { /* Init_SRA0_SRAF */
> + 0x00, 0xFB, 0x9F, 0x01, 0x00, 0xED, 0xED, 0xED,
> + 0x7B, 0xFB, 0xFF, 0xFF, 0x97, 0xEF, 0xBF, 0xDF,
> + },
> + { /* Init_GR00_GR08 */
> + 0x00, 0x00, 0x00, 0x00, 0x00, 0x40, 0x05, 0x0F,
> + 0xFF,
> + },
> + { /* Init_AR00_AR14 */
> + 0x00, 0x01, 0x02, 0x03, 0x04, 0x05, 0x06, 0x07,
> + 0x08, 0x09, 0x0A, 0x0B, 0x0C, 0x0D, 0x0E, 0x0F,
> + 0x41, 0x00, 0x0F, 0x00, 0x00,
> + },
> + { /* Init_CR00_CR18 */
> + 0xA3, 0x7F, 0x7F, 0x00, 0x85, 0x16, 0x24, 0xF5,
> + 0x00, 0x60, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
> + 0x03, 0x09, 0xFF, 0x80, 0x40, 0xFF, 0x00, 0xE3,
> + 0xFF,
> + },
> + { /* Init_CR30_CR4D */
> + 0x00, 0x00, 0x00, 0x00, 0x00, 0x80, 0x02, 0x20,
> + 0x00, 0x00, 0x30, 0x40, 0x00, 0xFF, 0xBF, 0xFF,
> + 0x2E, 0x27, 0x00, 0x2b, 0x0c, 0x0F, 0xEF, 0x00,
> + 0xFe, 0x0f, 0x01, 0xC0, 0x27, 0xEF,
> + },
> + { /* Init_CR90_CRA7 */
> + 0x55, 0xD9, 0x5D, 0xE1, 0x86, 0x1B, 0x8E, 0x26,
> + 0xDA, 0x8D, 0xDE, 0x94, 0x00, 0x00, 0x18, 0x00,
> + 0x03, 0x03, 0x03, 0x03, 0x03, 0x03, 0x15, 0x03,
> + },
> + },
> +
> + { /* mode#8: 320 x 240 32Bpp 60Hz */
> + 320, 240, 32, 60,
> + /* Init_MISC */
> + 0xEB,
> + { /* Init_SR0_SR4 */
> + 0x03, 0x01, 0x0F, 0x03, 0x0E,
> + },
> + { /* Init_SR10_SR24 */
> + 0xF3, 0xB6, 0xC0, 0xDD, 0x00, 0x0E, 0x17, 0x2C,
> + 0x99, 0x02, 0x01, 0x00, 0x00, 0x00, 0x00, 0x00,
> + 0xC4, 0x32, 0x02, 0x01, 0x01,
> + },
> + { /* Init_SR30_SR75 */
> + 0x38, 0x03, 0x20, 0x09, 0xC0, 0x3A, 0x3A, 0x3A,
> + 0x3A, 0x3A, 0x3A, 0x3A, 0x00, 0x00, 0x03, 0xFF,
> + 0x00, 0xFC, 0x00, 0x00, 0x20, 0x18, 0x00, 0xFC,
> + 0x20, 0x0C, 0x44, 0x20, 0x00, 0x00, 0x00, 0x3A,
> + 0x06, 0x68, 0xA7, 0x7F, 0x83, 0x24, 0xFF, 0x03,
> + 0x00, 0x60, 0x59, 0x3A, 0x3A, 0x00, 0x00, 0x3A,
> + 0x01, 0x80, 0x7E, 0x1A, 0x1A, 0x00, 0x00, 0x00,
> + 0x50, 0x03, 0x74, 0x14, 0x08, 0x43, 0x08, 0x43,
> + 0x04, 0x45, 0x30, 0x30, 0x40, 0x20,
> + },
> + { /* Init_SR80_SR93 */
> + 0xFF, 0x07, 0x00, 0xFF, 0xFF, 0xFF, 0xFF, 0x3A,
> + 0xF7, 0x00, 0x00, 0x00, 0xFF, 0xFF, 0x3A, 0x3A,
> + 0x00, 0x00, 0x00, 0x00,
> + },
> + { /* Init_SRA0_SRAF */
> + 0x00, 0xFB, 0x9F, 0x01, 0x00, 0xED, 0xED, 0xED,
> + 0x7B, 0xFB, 0xFF, 0xFF, 0x97, 0xEF, 0xBF, 0xDF,
> + },
> + { /* Init_GR00_GR08 */
> + 0x00, 0x00, 0x00, 0x00, 0x00, 0x40, 0x05, 0x0F,
> + 0xFF,
> + },
> + { /* Init_AR00_AR14 */
> + 0x00, 0x01, 0x02, 0x03, 0x04, 0x05, 0x06, 0x07,
> + 0x08, 0x09, 0x0A, 0x0B, 0x0C, 0x0D, 0x0E, 0x0F,
> + 0x41, 0x00, 0x0F, 0x00, 0x00,
> + },
> + { /* Init_CR00_CR18 */
> + 0xA3, 0x7F, 0x7F, 0x00, 0x85, 0x16, 0x24, 0xF5,
> + 0x00, 0x60, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
> + 0x03, 0x09, 0xFF, 0x80, 0x40, 0xFF, 0x00, 0xE3,
> + 0xFF,
> + },
> + { /* Init_CR30_CR4D */
> + 0x00, 0x00, 0x00, 0x00, 0x00, 0x80, 0x02, 0x20,
> + 0x00, 0x00, 0x30, 0x40, 0x00, 0xFF, 0xBF, 0xFF,
> + 0x2E, 0x27, 0x00, 0x2b, 0x0c, 0x0F, 0xEF, 0x00,
> + 0xFe, 0x0f, 0x01, 0xC0, 0x27, 0xEF,
> + },
> + { /* Init_CR90_CRA7 */
> + 0x55, 0xD9, 0x5D, 0xE1, 0x86, 0x1B, 0x8E, 0x26,
> + 0xDA, 0x8D, 0xDE, 0x94, 0x00, 0x00, 0x18, 0x00,
> + 0x03, 0x03, 0x03, 0x03, 0x03, 0x03, 0x15, 0x03,
> + },
> + },
> +};
What are these tables above for?
Tomi
[toc] | [next] | [standalone]
| From | Sudip Mukherjee <sudipm.mukherjee@gmail.com> |
|---|---|
| Date | 2015-09-01 16:00 +0200 |
| Message-ID | <q3Ugq-8lN-15@gated-at.bofh.it> |
| In reply to | #1216844 |
On Tue, Sep 01, 2015 at 04:27:24PM +0300, Tomi Valkeinen wrote:
>
>
> On 18/07/15 07:08, Sudip Mukherjee wrote:
> > Now since all cleanups are done and the code is ready to be merged lets
> > move it out of staging into fbdev location.
>
> Have you considered writing a DRM driver for this? I'm not happy at all
> adding new fbdev drivers, as the DRM framework is much better,
> supported, and continuously improved. With fbdev you end up with things
> like module parameters used to define video modes etc, which is just ugly.
Yes, I am working on a DRM driver, but since these are all voluntary
work it is taking time. And Greg has already merged it.
>
> Anyway, some comments below.
Some replies inline and remaining I will fix and send patches to you.
>
<snip>
> > +
> > +extern void __iomem *smtc_regbaseaddress;
>
> Uh, what is that?
I guess all of us missed seeing it. :(
As you said in another comments smtc_regbaseaddress will be included in
the per-device data and this will be removed.
>
<snip>
> > +static inline unsigned int smtc_seqr(int reg)
> > +{
> > + smtc_mmiowb(reg, 0x3c4);
> > + return smtc_mmiorb(0x3c5);
> > +}
>
> There's quite a lot of magic numbers there, and the same continues
> through the driver. You should use defines to assign symbolic names for
> most of the numbers.
will do.
>
<snip>
> > +
> > +void __iomem *smtc_regbaseaddress; /* Memory Map IO starting address */
>
> You can't have globals like this in the driver, they must be inside the
> per-device data. Just think what happens if someone has two of these
> devices.
will do.
>
<snip>
> > +
> > +static struct fb_fix_screeninfo smtcfb_fix = {
> > + .id = "smXXXfb",
> > + .type = FB_TYPE_PACKED_PIXELS,
> > + .visual = FB_VISUAL_TRUECOLOR,
> > + .line_length = 800 * 3,
> > + .accel = FB_ACCEL_SMI_LYNX,
> > + .type_aux = 0,
> > + .xpanstep = 0,
> > + .ypanstep = 0,
> > + .ywrapstep = 0,
> > +};
>
> These should be const.
ok.
>
<snip>
> > +static const struct vesa_mode vesa_mode_table[] = {
> > + {"0x301", 640, 480, 8},
> > + {"0x303", 800, 600, 8},
> > + {"0x305", 1024, 768, 8},
> > + {"0x307", 1280, 1024, 8},
> > +
> > + {"0x311", 640, 480, 16},
> > + {"0x314", 800, 600, 16},
> > + {"0x317", 1024, 768, 16},
> > + {"0x31A", 1280, 1024, 16},
> > +
> > + {"0x312", 640, 480, 24},
> > + {"0x315", 800, 600, 24},
> > + {"0x318", 1024, 768, 24},
> > + {"0x31B", 1280, 1024, 24},
> > +};
>
> We have "vesa_modes" in include/linux/fb.h. What is the above table for?
The resolutions that are supported along with the kernel boot parameter
to point to the resolution to boot with.
>
> > +
> > +/**********************************************************************
> > + SM712 Mode table.
> > + **********************************************************************/
> > +static const struct modeinit vgamode[] = {
> > + {
<snip>
> > + { /* Init_CR90_CRA7 */
> > + 0x55, 0xD9, 0x5D, 0xE1, 0x86, 0x1B, 0x8E, 0x26,
> > + 0xDA, 0x8D, 0xDE, 0x94, 0x00, 0x00, 0x18, 0x00,
> > + 0x03, 0x03, 0x03, 0x03, 0x03, 0x03, 0x15, 0x03,
> > + },
> > + },
> > +};
>
> What are these tables above for?
Different register settings based on the display resolution. Do you want
me to do anything with these vgamode table and the vesa_mode_table?
regards
sudip
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [next] | [standalone]
| From | Tomi Valkeinen <tomi.valkeinen@ti.com> |
|---|---|
| Date | 2015-09-02 14:00 +0200 |
| Message-ID | <q4eRQ-4a6-7@gated-at.bofh.it> |
| In reply to | #1216856 |
[Multipart message — attachments visible in raw view] — view raw
On 01/09/15 16:55, Sudip Mukherjee wrote:
> On Tue, Sep 01, 2015 at 04:27:24PM +0300, Tomi Valkeinen wrote:
>>
>>
>> On 18/07/15 07:08, Sudip Mukherjee wrote:
>>> Now since all cleanups are done and the code is ready to be merged lets
>>> move it out of staging into fbdev location.
>>
>> Have you considered writing a DRM driver for this? I'm not happy at all
>> adding new fbdev drivers, as the DRM framework is much better,
>> supported, and continuously improved. With fbdev you end up with things
>> like module parameters used to define video modes etc, which is just ugly.
> Yes, I am working on a DRM driver, but since these are all voluntary
> work it is taking time. And Greg has already merged it.
>>
>> Anyway, some comments below.
> Some replies inline and remaining I will fix and send patches to you.
Wouldn't the time be better spent on the DRM driver?
This driver will be obsolete immediately when there's a DRM driver for
this device, and then it'll be yet another obsoleted fbdev driver we
need to maintain.
>>> +static const struct vesa_mode vesa_mode_table[] = {
>>> + {"0x301", 640, 480, 8},
>>> + {"0x303", 800, 600, 8},
>>> + {"0x305", 1024, 768, 8},
>>> + {"0x307", 1280, 1024, 8},
>>> +
>>> + {"0x311", 640, 480, 16},
>>> + {"0x314", 800, 600, 16},
>>> + {"0x317", 1024, 768, 16},
>>> + {"0x31A", 1280, 1024, 16},
>>> +
>>> + {"0x312", 640, 480, 24},
>>> + {"0x315", 800, 600, 24},
>>> + {"0x318", 1024, 768, 24},
>>> + {"0x31B", 1280, 1024, 24},
>>> +};
>>
>> We have "vesa_modes" in include/linux/fb.h. What is the above table for?
> The resolutions that are supported along with the kernel boot parameter
> to point to the resolution to boot with.
Why does the user need to give such hex values? Why not modes according
to Documentation/fb/modedb.txt?
>>> +
>>> +/**********************************************************************
>>> + SM712 Mode table.
>>> + **********************************************************************/
>>> +static const struct modeinit vgamode[] = {
>>> + {
> <snip>
>>> + { /* Init_CR90_CRA7 */
>>> + 0x55, 0xD9, 0x5D, 0xE1, 0x86, 0x1B, 0x8E, 0x26,
>>> + 0xDA, 0x8D, 0xDE, 0x94, 0x00, 0x00, 0x18, 0x00,
>>> + 0x03, 0x03, 0x03, 0x03, 0x03, 0x03, 0x15, 0x03,
>>> + },
>>> + },
>>> +};
>>
>> What are these tables above for?
> Different register settings based on the display resolution. Do you want
> me to do anything with these vgamode table and the vesa_mode_table?
The vgamode table looks quite horrible. It's unmaintainable, and with a
quick look it seems to have lots of repetition. Large blocks of the data
for different modes are the same.
I don't know what the register there are, but I'd imagine you could
write generic functions like, say, "set_timings", which takes normal
linux videomode struct and writes those settings to the registers.
There should be no such bulk-write register tables in a proper driver,
except in some very special cases.
Tomi
[toc] | [prev] | [next] | [standalone]
| From | Sudip Mukherjee <sudipm.mukherjee@gmail.com> |
|---|---|
| Date | 2015-09-02 14:50 +0200 |
| Message-ID | <q4fEe-5kp-3@gated-at.bofh.it> |
| In reply to | #1217544 |
On Wed, Sep 02, 2015 at 02:58:19PM +0300, Tomi Valkeinen wrote: > On 01/09/15 16:55, Sudip Mukherjee wrote: > > On Tue, Sep 01, 2015 at 04:27:24PM +0300, Tomi Valkeinen wrote: > >> On 18/07/15 07:08, Sudip Mukherjee wrote: > >>> Now since all cleanups are done and the code is ready to be merged lets > >>> move it out of staging into fbdev location. <snip> > > Some replies inline and remaining I will fix and send patches to you. > > Wouldn't the time be better spent on the DRM driver? > > This driver will be obsolete immediately when there's a DRM driver for > this device, and then it'll be yet another obsoleted fbdev driver we > need to maintain. Now I am getting confused. :( Since this has already been merged I guess we need to maintain it now. So then should I fix the things you pointed out or should i instead give more priority to the DRM driver and fix these things later? And, just to inform you, there are two more fbdev drivers in staging, staging/sm750fb and staging/fbtft. And the ultimate goal of any driver in staging is to move out staging into the main part of the kernel. And I expect sm750fb to be ready for moving before 4.6 merge window. If you don't want any more fbdev drivers to be added then maybe you can have a talk with Greg about this. He is already in the cc. regards sudip -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web