[PATCH v2 2/3] Documentation/fb: add documentation for sm712fb

linux-fbdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed

* [PATCH v2 2/3] Documentation/fb: add documentation for sm712fb
       [not found] <1437192539-14150-1-git-send-email-sudipm.mukherjee@gmail.com>
@ 2015-07-18  4:20 ` Sudip Mukherjee
  2015-07-18  4:20 ` [PATCH v2 3/3] MAINTAINERS: update maintainers list Sudip Mukherjee
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 10+ messages in thread
From: Sudip Mukherjee @ 2015-07-18  4:20 UTC (permalink / raw)
  To: Jean-Christophe Plagniol-Villard, Tomi Valkeinen, Jonathan Corbet,
	Greg Kroah-Hartman
  Cc: linux-kernel, linux-fbdev, linux-doc, devel, Sudip Mukherjee

Create the documentation for SM712. Mention all the supported modes and
how to use.

Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org>
---
 Documentation/fb/sm712fb.txt | 31 +++++++++++++++++++++++++++++++
 1 file changed, 31 insertions(+)
 create mode 100644 Documentation/fb/sm712fb.txt

diff --git a/Documentation/fb/sm712fb.txt b/Documentation/fb/sm712fb.txt
new file mode 100644
index 0000000..c388442
--- /dev/null
+++ b/Documentation/fb/sm712fb.txt
@@ -0,0 +1,31 @@
+What is sm712fb?
+========+
+This is a graphics framebuffer driver for Silicon Motion SM712 based processors.
+
+How to use it?
+=======
+
+Switching modes is done using the video=sm712fb:... boot parameter.
+
+If you want, for example, enable a resolution of 1280x1024x24bpp you should
+pass to the kernel this command line: "video=sm712fb:0x31B".
+
+You should not compile-in vesafb.
+
+Currently supported video modes are:
+
+[Graphic modes]
+
+bpp | 640x480  800x600  1024x768  1280x1024
+----+--------------------------------------------
+  8 | 0x301    0x303    0x305    0x307
+ 16 | 0x311    0x314    0x317    0x31A
+ 24 | 0x312    0x315    0x318    0x31B
+
+Missing Features
+========
+(alias TODO list)
+
+	* 2D acceleratrion
+	* dual-head support
-- 
1.8.1.2


^ permalink raw reply related	[flat|nested] 10+ messages in thread

* [PATCH v2 3/3] MAINTAINERS: update maintainers list
       [not found] <1437192539-14150-1-git-send-email-sudipm.mukherjee@gmail.com>
  2015-07-18  4:20 ` [PATCH v2 2/3] Documentation/fb: add documentation for sm712fb Sudip Mukherjee
@ 2015-07-18  4:20 ` Sudip Mukherjee
  2015-07-27  4:53 ` [PATCH v2 1/3] staging: sm7xxfb: move sm712fb out of staging Sudip Mukherjee
  2015-09-01 13:27 ` Tomi Valkeinen
  3 siblings, 0 replies; 10+ messages in thread
From: Sudip Mukherjee @ 2015-07-18  4:20 UTC (permalink / raw)
  To: Jean-Christophe Plagniol-Villard, Tomi Valkeinen, Jonathan Corbet,
	Greg Kroah-Hartman
  Cc: linux-kernel, linux-fbdev, linux-doc, devel, Sudip Mukherjee

Now since sm712fb has moved out of staging update the maintainers list
accordingly.

Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org>
---
 MAINTAINERS | 17 +++++++++--------
 1 file changed, 9 insertions(+), 8 deletions(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 8133cef..2c77c30 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -9301,6 +9301,15 @@ S:	Maintained
 F:	drivers/media/i2c/ov2659.c
 F:	include/media/ov2659.h
 
+SILICON MOTION SM712 FRAME BUFFER DRIVER
+M:	Sudip Mukherjee <sudipm.mukherjee@gmail.com>
+M:	Teddy Wang <teddy.wang@siliconmotion.com>
+M:	Sudip Mukherjee <sudip@vectorindia.org>
+L:	linux-fbdev@vger.kernel.org
+S:	Maintained
+F:	drivers/video/fbdev/sm712*
+F:	Documentation/fb/sm712fb.txt
+
 SIS 190 ETHERNET DRIVER
 M:	Francois Romieu <romieu@fr.zoreil.com>
 L:	netdev@vger.kernel.org
@@ -9721,14 +9730,6 @@ L:	linux-wireless@vger.kernel.org
 S:	Maintained
 F:	drivers/staging/rtl8723au/
 
-STAGING - SILICON MOTION SM7XX FRAME BUFFER DRIVER
-M:	Sudip Mukherjee <sudipm.mukherjee@gmail.com>
-M:	Teddy Wang <teddy.wang@siliconmotion.com>
-M:	Sudip Mukherjee <sudip@vectorindia.org>
-L:	linux-fbdev@vger.kernel.org
-S:	Maintained
-F:	drivers/staging/sm7xxfb/
-
 STAGING - SILICON MOTION SM750 FRAME BUFFER DRIVER
 M:	Sudip Mukherjee <sudipm.mukherjee@gmail.com>
 M:	Teddy Wang <teddy.wang@siliconmotion.com>
-- 
1.8.1.2


^ permalink raw reply related	[flat|nested] 10+ messages in thread

* Re: [PATCH v2 1/3] staging: sm7xxfb: move sm712fb out of staging
       [not found] <1437192539-14150-1-git-send-email-sudipm.mukherjee@gmail.com>
  2015-07-18  4:20 ` [PATCH v2 2/3] Documentation/fb: add documentation for sm712fb Sudip Mukherjee
  2015-07-18  4:20 ` [PATCH v2 3/3] MAINTAINERS: update maintainers list Sudip Mukherjee
@ 2015-07-27  4:53 ` Sudip Mukherjee
  2015-09-01 13:27 ` Tomi Valkeinen
  3 siblings, 0 replies; 10+ messages in thread
From: Sudip Mukherjee @ 2015-07-27  4:53 UTC (permalink / raw)
  To: Jean-Christophe Plagniol-Villard, Tomi Valkeinen,
	Greg Kroah-Hartman
  Cc: linux-kernel, linux-fbdev, devel

On Sat, Jul 18, 2015 at 09:38:57AM +0530, 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.
Hi Tomi,

A gentle ping

regards
sudip

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH v2 1/3] staging: sm7xxfb: move sm712fb out of staging
       [not found] <1437192539-14150-1-git-send-email-sudipm.mukherjee@gmail.com>
                   ` (2 preceding siblings ...)
  2015-07-27  4:53 ` [PATCH v2 1/3] staging: sm7xxfb: move sm712fb out of staging Sudip Mukherjee
@ 2015-09-01 13:27 ` Tomi Valkeinen
  2015-09-01 13:55   ` Sudip Mukherjee
  3 siblings, 1 reply; 10+ messages in thread
From: Tomi Valkeinen @ 2015-09-01 13:27 UTC (permalink / raw)
  To: Sudip Mukherjee
  Cc: Jean-Christophe Plagniol-Villard, Jonathan Corbet,
	Greg Kroah-Hartman, linux-kernel, linux-fbdev, linux-doc, devel

[-- Attachment #1: Type: text/plain, Size: 32589 bytes --]



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


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH v2 1/3] staging: sm7xxfb: move sm712fb out of staging
  2015-09-01 13:27 ` Tomi Valkeinen
@ 2015-09-01 13:55   ` Sudip Mukherjee
  2015-09-02 11:58     ` Tomi Valkeinen
  0 siblings, 1 reply; 10+ messages in thread
From: Sudip Mukherjee @ 2015-09-01 13:55 UTC (permalink / raw)
  To: Tomi Valkeinen
  Cc: Jean-Christophe Plagniol-Villard, Jonathan Corbet,
	Greg Kroah-Hartman, linux-kernel, linux-fbdev, linux-doc, devel

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

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH v2 1/3] staging: sm7xxfb: move sm712fb out of staging
  2015-09-01 13:55   ` Sudip Mukherjee
@ 2015-09-02 11:58     ` Tomi Valkeinen
  2015-09-02 12:48       ` Sudip Mukherjee
  0 siblings, 1 reply; 10+ messages in thread
From: Tomi Valkeinen @ 2015-09-02 11:58 UTC (permalink / raw)
  To: Sudip Mukherjee
  Cc: Jean-Christophe Plagniol-Villard, Jonathan Corbet,
	Greg Kroah-Hartman, linux-kernel, linux-fbdev, linux-doc, devel

[-- Attachment #1: Type: text/plain, Size: 3049 bytes --]



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


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH v2 1/3] staging: sm7xxfb: move sm712fb out of staging
  2015-09-02 11:58     ` Tomi Valkeinen
@ 2015-09-02 12:48       ` Sudip Mukherjee
  2015-09-24 11:58         ` Tomi Valkeinen
  0 siblings, 1 reply; 10+ messages in thread
From: Sudip Mukherjee @ 2015-09-02 12:48 UTC (permalink / raw)
  To: Tomi Valkeinen
  Cc: Jean-Christophe Plagniol-Villard, Jonathan Corbet,
	Greg Kroah-Hartman, linux-kernel, linux-fbdev, linux-doc, devel

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

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH v2 1/3] staging: sm7xxfb: move sm712fb out of staging
  2015-09-02 12:48       ` Sudip Mukherjee
@ 2015-09-24 11:58         ` Tomi Valkeinen
  2015-09-25 13:54           ` Sudip Mukherjee
  0 siblings, 1 reply; 10+ messages in thread
From: Tomi Valkeinen @ 2015-09-24 11:58 UTC (permalink / raw)
  To: Sudip Mukherjee
  Cc: Jean-Christophe Plagniol-Villard, Jonathan Corbet,
	Greg Kroah-Hartman, linux-kernel, linux-fbdev, linux-doc, devel

[-- Attachment #1: Type: text/plain, Size: 1125 bytes --]

On 02/09/15 15:48, Sudip Mukherjee wrote:

> Now I am getting confused. :(
> Since this has already been merged I guess we need to maintain it now.

Oh, ok. I thought it was still in staging. I haven't been able to follow
the list properly lately...

Well, in theory we could still revert it, as it's only -rc2.

> 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?

Is there a public datasheet for the HW?

The driver as it is now is quite horrible. Huge tables of raw data,
written directly to the registers. Lots of magic numbers all around.
Even with a datasheet, presuming it's public and it contains all those
values, the driver is rather unmaintainable.

All those need to be sorted out for the DRM driver anyway, so maybe it's
better to clean up the fb driver first.

> 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

Thanks for pointing this out, I need to check those and think how to
proceed.

 Tomi

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH v2 1/3] staging: sm7xxfb: move sm712fb out of staging
  2015-09-24 11:58         ` Tomi Valkeinen
@ 2015-09-25 13:54           ` Sudip Mukherjee
  2015-09-27  4:42             ` Mike Rapoport
  0 siblings, 1 reply; 10+ messages in thread
From: Sudip Mukherjee @ 2015-09-25 13:54 UTC (permalink / raw)
  To: Tomi Valkeinen
  Cc: Jean-Christophe Plagniol-Villard, Jonathan Corbet,
	Greg Kroah-Hartman, linux-kernel, linux-fbdev, linux-doc, devel

On Thu, Sep 24, 2015 at 02:58:31PM +0300, Tomi Valkeinen wrote:
> 
> On 02/09/15 15:48, Sudip Mukherjee wrote:
> 
<snip>
> 
> Is there a public datasheet for the HW?
I could not find one in Silicon Motion website. I will try to find else
I can talk to them and keep it in my github. And can send you also if
you want.
> 
> The driver as it is now is quite horrible. Huge tables of raw data,
> written directly to the registers. Lots of magic numbers all around.
> Even with a datasheet, presuming it's public and it contains all those
> values, the driver is rather unmaintainable.
> 
> All those need to be sorted out for the DRM driver anyway, so maybe it's
> better to clean up the fb driver first.
Ok. I never looked at them to see how they can be made better. I will
see.

regards
sudip

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH v2 1/3] staging: sm7xxfb: move sm712fb out of staging
  2015-09-25 13:54           ` Sudip Mukherjee
@ 2015-09-27  4:42             ` Mike Rapoport
  0 siblings, 0 replies; 10+ messages in thread
From: Mike Rapoport @ 2015-09-27  4:42 UTC (permalink / raw)
  To: Sudip Mukherjee
  Cc: Tomi Valkeinen, devel, linux-fbdev, linux-doc, Greg Kroah-Hartman,
	Jonathan Corbet, linux-kernel, Jean-Christophe Plagniol-Villard

Hi Sudip,

On Fri, Sep 25, 2015 at 07:12:06PM +0530, Sudip Mukherjee wrote:
> On Thu, Sep 24, 2015 at 02:58:31PM +0300, Tomi Valkeinen wrote:
> > 
> > On 02/09/15 15:48, Sudip Mukherjee wrote:
> > 
> <snip>
> > 
> > Is there a public datasheet for the HW?
> I could not find one in Silicon Motion website. I will try to find else
> I can talk to them and keep it in my github. And can send you also if
> you want.

I'd appreciate sm750 datasheet, if you have one.

> > 
> > The driver as it is now is quite horrible. Huge tables of raw data,
> > written directly to the registers. Lots of magic numbers all around.
> > Even with a datasheet, presuming it's public and it contains all those
> > values, the driver is rather unmaintainable.
> > 
> > All those need to be sorted out for the DRM driver anyway, so maybe it's
> > better to clean up the fb driver first.
> Ok. I never looked at them to see how they can be made better. I will
> see.
> 
> regards
> sudip

--
Sincerely yours,
Mike.

^ permalink raw reply	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2015-09-27  4:42 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <1437192539-14150-1-git-send-email-sudipm.mukherjee@gmail.com>
2015-07-18  4:20 ` [PATCH v2 2/3] Documentation/fb: add documentation for sm712fb Sudip Mukherjee
2015-07-18  4:20 ` [PATCH v2 3/3] MAINTAINERS: update maintainers list Sudip Mukherjee
2015-07-27  4:53 ` [PATCH v2 1/3] staging: sm7xxfb: move sm712fb out of staging Sudip Mukherjee
2015-09-01 13:27 ` Tomi Valkeinen
2015-09-01 13:55   ` Sudip Mukherjee
2015-09-02 11:58     ` Tomi Valkeinen
2015-09-02 12:48       ` Sudip Mukherjee
2015-09-24 11:58         ` Tomi Valkeinen
2015-09-25 13:54           ` Sudip Mukherjee
2015-09-27  4:42             ` Mike Rapoport

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).