All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH V2] OMAP: DSS: Add NEC NL8048HL11-01B display panel
       [not found] <E0D41E29EB0DAC4E9F3FF173962E9E94026B7E8AB0@dbde02.ent.ti.com>
@ 2010-02-11  8:21 ` Y, Kishore
  2010-02-11  8:44   ` Hiremath, Vaibhav
  0 siblings, 1 reply; 4+ messages in thread
From: Y, Kishore @ 2010-02-11  8:21 UTC (permalink / raw)
  To: Tomi Valkeinen
  Cc: Hiremath, Vaibhav, Aguirre, Sergio, Christensen, Mikkel, linux-omap

From: Erik Gilling <konkers@android.com>

NEC WVGA LCD NL8048HL11-01B support has been added.

Signed-off-by: Mukund Mittal <mmittal@ti.com>
Signed-off-by: Kishore Y <kishore.y@ti.com>
---
Changes from version 1:
	incorporated cosmetic comments.

 drivers/video/omap2/displays/Kconfig               |    6 +
 drivers/video/omap2/displays/Makefile              |    1 +
 .../omap2/displays/panel-nec-NL8048HL11-01B.c      |  284 ++++++++++++++++++++
 3 files changed, 291 insertions(+), 0 deletions(-)
 create mode 100644 drivers/video/omap2/displays/panel-nec-NL8048HL11-01B.c

diff --git a/drivers/video/omap2/displays/Kconfig b/drivers/video/omap2/displays/Kconfig
index dfb57ee..010ed48 100644
--- a/drivers/video/omap2/displays/Kconfig
+++ b/drivers/video/omap2/displays/Kconfig
@@ -37,4 +37,10 @@ config PANEL_TPO_TD043MTEA1
         help
           LCD Panel used in OMAP3 Pandora
 
+config PANEL_NEC_NL8048HL11_01B
+        tristate "NEC NL8048HL11-01B Panel"
+        depends on OMAP2_DSS
+        help
+          LCD Panel from NEC.
+
 endmenu
diff --git a/drivers/video/omap2/displays/Makefile b/drivers/video/omap2/displays/Makefile
index e2bb321..9aace29 100644
--- a/drivers/video/omap2/displays/Makefile
+++ b/drivers/video/omap2/displays/Makefile
@@ -5,3 +5,4 @@ obj-$(CONFIG_PANEL_SHARP_LQ043T1DG01) += panel-sharp-lq043t1dg01.o
 obj-$(CONFIG_PANEL_TAAL) += panel-taal.o
 obj-$(CONFIG_PANEL_TOPPOLY_TDO35S) += panel-toppoly-tdo35s.o
 obj-$(CONFIG_PANEL_TPO_TD043MTEA1) += panel-tpo-td043mtea1.o
+obj-$(CONFIG_PANEL_NEC_NL8048HL11_01B) += panel-nec-NL8048HL11-01B.o
diff --git a/drivers/video/omap2/displays/panel-nec-NL8048HL11-01B.c b/drivers/video/omap2/displays/panel-nec-NL8048HL11-01B.c
new file mode 100644
index 0000000..4d7e8ab
--- /dev/null
+++ b/drivers/video/omap2/displays/panel-nec-NL8048HL11-01B.c
@@ -0,0 +1,284 @@
+/*
+ * NEC panel support
+ *
+ * This program is free software; you can redistribute it and/or modify it
+ * under the terms of the GNU General Public License version 2 as published by
+ * the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it will be useful, but WITHOUT
+ * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
+ * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License for
+ * more details.
+ *
+ * You should have received a copy of the GNU General Public License along with
+ * this program.  If not, see <http://www.gnu.org/licenses/>.
+ */
+
+#include <linux/module.h>
+#include <linux/delay.h>
+#include <linux/platform_device.h>
+#include <linux/spi/spi.h>
+#include <asm/mach-types.h>
+#include <plat/gpio.h>
+#include <plat/mux.h>
+#include <plat/control.h>
+#include <plat/display.h>
+
+#define LCD_XRES		800
+#define LCD_YRES		480
+/* NEC PIX Clock Ratings
+ * MINi:21.8MHz TYP:23.8MHz MAX:25.7MHz
+ */
+#define LCD_PIXEL_CLOCK		23800
+
+/* NEC NL8048HL11-01B  Manual
+ * defines HFB, HSW, HBP, VFP, VSW, VBP as shown below
+ */
+
+static struct omap_video_timings nec_8048_panel_timings = {
+	/* 800 x 480 @ 60 Hz  Reduced blanking VESA CVT 0.31M3-R */
+	.x_res          = LCD_XRES,
+	.y_res          = LCD_YRES,
+	.pixel_clock    = LCD_PIXEL_CLOCK,
+	.hfp            = 6,
+	.hsw            = 1,
+	.hbp            = 4,
+	.vfp            = 3,
+	.vsw            = 1,
+	.vbp            = 4,
+};
+
+static int nec_8048_panel_probe(struct omap_dss_device *dssdev)
+{
+	dssdev->panel.config = OMAP_DSS_LCD_TFT | OMAP_DSS_LCD_IVS |
+				OMAP_DSS_LCD_IHS | OMAP_DSS_LCD_RF |
+				OMAP_DSS_LCD_ONOFF;
+	dssdev->panel.timings = nec_8048_panel_timings;
+	dssdev->panel.recommended_bpp = 16;
+
+	return 0;
+}
+
+static void nec_8048_panel_remove(struct omap_dss_device *dssdev)
+{
+}
+
+static int nec_8048_panel_enable(struct omap_dss_device *dssdev)
+{
+	int r = 0;
+
+	/* Delay recommended by panel DATASHEET */
+	mdelay(4);
+	if (dssdev->platform_enable)
+		r = dssdev->platform_enable(dssdev);
+
+	return r;
+}
+
+static void nec_8048_panel_disable(struct omap_dss_device *dssdev)
+{
+	if (dssdev->platform_disable)
+		dssdev->platform_disable(dssdev);
+	/* Delay recommended by panel DATASHEET */
+	mdelay(4);
+}
+
+static int nec_8048_panel_suspend(struct omap_dss_device *dssdev)
+{
+	nec_8048_panel_disable(dssdev);
+	return 0;
+}
+
+static int nec_8048_panel_resume(struct omap_dss_device *dssdev)
+{
+	return nec_8048_panel_enable(dssdev);
+}
+
+static struct omap_dss_driver nec_8048_driver = {
+	.probe          = nec_8048_panel_probe,
+	.remove         = nec_8048_panel_remove,
+	.enable         = nec_8048_panel_enable,
+	.disable        = nec_8048_panel_disable,
+	.suspend        = nec_8048_panel_suspend,
+	.resume         = nec_8048_panel_resume,
+
+	.driver		= {
+		.name	= "NEC_8048_panel",
+		.owner 	= THIS_MODULE,
+	},
+};
+
+static int
+spi_send(struct spi_device *spi, unsigned char reg_addr, unsigned char reg_data)
+{
+	int ret = 0;
+	unsigned int cmd = 0;
+	unsigned int data = 0;
+
+	cmd = 0x0000 | reg_addr; /* register address write */
+	data = 0x0100 | reg_data ; /* register data write */
+	data = (cmd << 16) | data;
+	ret = spi_write(spi, (unsigned char *)&data, 4);
+	if (ret) {
+		printk(KERN_ERR "error in spi_write %x\n", data);
+		return ret;
+	}
+
+	/* Delay, part of init seqence recommended by panel DATASHEET */
+	udelay(10);
+	return 0;
+}
+
+static int init_nec_8048_wvga_lcd(struct spi_device *spi)
+{
+	/* Initialization Sequence */
+	/* spi_send(spi, REG, VAL) */
+	spi_send(spi, 3, 0x01);
+	spi_send(spi, 0, 0x00);
+	spi_send(spi, 1, 0x01);    /* R1 = 0x01 (normal), 0x03 (reversed) */
+	spi_send(spi, 4, 0x00);
+	spi_send(spi, 5, 0x14);
+	spi_send(spi, 6, 0x24);
+	spi_send(spi, 16, 0xD7);
+	spi_send(spi, 17, 0x00);
+	spi_send(spi, 18, 0x00);
+	spi_send(spi, 19, 0x55);
+	spi_send(spi, 20, 0x01);
+	spi_send(spi, 21, 0x70);
+	spi_send(spi, 22, 0x1E);
+	spi_send(spi, 23, 0x25);
+	spi_send(spi, 24, 0x25);
+	spi_send(spi, 25, 0x02);
+	spi_send(spi, 26, 0x02);
+	spi_send(spi, 27, 0xA0);
+	spi_send(spi, 32, 0x2F);
+	spi_send(spi, 33, 0x0F);
+	spi_send(spi, 34, 0x0F);
+	spi_send(spi, 35, 0x0F);
+	spi_send(spi, 36, 0x0F);
+	spi_send(spi, 37, 0x0F);
+	spi_send(spi, 38, 0x0F);
+	spi_send(spi, 39, 0x00);
+	spi_send(spi, 40, 0x02);
+	spi_send(spi, 41, 0x02);
+	spi_send(spi, 42, 0x02);
+	spi_send(spi, 43, 0x0F);
+	spi_send(spi, 44, 0x0F);
+	spi_send(spi, 45, 0x0F);
+	spi_send(spi, 46, 0x0F);
+	spi_send(spi, 47, 0x0F);
+	spi_send(spi, 48, 0x0F);
+	spi_send(spi, 49, 0x0F);
+	spi_send(spi, 50, 0x00);
+	spi_send(spi, 51, 0x02);
+	spi_send(spi, 52, 0x02);
+	spi_send(spi, 53, 0x02);
+	spi_send(spi, 80, 0x0C);
+	spi_send(spi, 83, 0x42);
+	spi_send(spi, 84, 0x42);
+	spi_send(spi, 85, 0x41);
+	spi_send(spi, 86, 0x14);
+	spi_send(spi, 89, 0x88);
+	spi_send(spi, 90, 0x01);
+	spi_send(spi, 91, 0x00);
+	spi_send(spi, 92, 0x02);
+	spi_send(spi, 93, 0x0C);
+	spi_send(spi, 94, 0x1C);
+	spi_send(spi, 95, 0x27);
+	spi_send(spi, 98, 0x49);
+	spi_send(spi, 99, 0x27);
+	spi_send(spi, 102, 0x76);
+	spi_send(spi, 103, 0x27);
+	spi_send(spi, 112, 0x01);
+	spi_send(spi, 113, 0x0E);
+	spi_send(spi, 114, 0x02);
+	spi_send(spi, 115, 0x0C);
+	spi_send(spi, 118, 0x0C);
+	spi_send(spi, 121, 0x30); /* R121 = 0x30 (normal), 0x10 (reversed) */
+	spi_send(spi, 130, 0x00);
+	spi_send(spi, 131, 0x00);
+	spi_send(spi, 132, 0xFC);
+	spi_send(spi, 134, 0x00);
+	spi_send(spi, 136, 0x00);
+	spi_send(spi, 138, 0x00);
+	spi_send(spi, 139, 0x00);
+	spi_send(spi, 140, 0x00);
+	spi_send(spi, 141, 0xFC);
+	spi_send(spi, 143, 0x00);
+	spi_send(spi, 145, 0x00);
+	spi_send(spi, 147, 0x00);
+	spi_send(spi, 148, 0x00);
+	spi_send(spi, 149, 0x00);
+	spi_send(spi, 150, 0xFC);
+	spi_send(spi, 152, 0x00);
+	spi_send(spi, 154, 0x00);
+	spi_send(spi, 156, 0x00);
+	spi_send(spi, 157, 0x00);
+	udelay(20);
+	spi_send(spi, 2, 0x00);
+	return 0;
+}
+
+static int nec_8048_spi_probe(struct spi_device *spi)
+{
+	spi->mode = SPI_MODE_0;
+	spi->bits_per_word = 32;
+	spi_setup(spi);
+
+	init_nec_8048_wvga_lcd(spi);
+
+	omap_dss_register_driver(&nec_8048_driver);
+	return 0;
+}
+
+static int nec_8048_spi_remove(struct spi_device *spi)
+{
+	omap_dss_unregister_driver(&nec_8048_driver);
+
+	return 0;
+}
+
+static int nec_8048_spi_suspend(struct spi_device *spi, pm_message_t mesg)
+{
+	spi_send(spi, 2, 0x01);
+	mdelay(40);
+
+	return 0;
+}
+
+static int nec_8048_spi_resume(struct spi_device *spi)
+{
+	/* reinitialize the panel */
+	spi_setup(spi);
+	spi_send(spi, 2, 0x00);
+	init_nec_8048_wvga_lcd(spi);
+
+	return 0;
+}
+
+static struct spi_driver nec_8048_spi_driver = {
+	.probe           = nec_8048_spi_probe,
+	.remove	= __devexit_p(nec_8048_spi_remove),
+	.suspend         = nec_8048_spi_suspend,
+	.resume          = nec_8048_spi_resume,
+	.driver         = {
+		.name   = "nec_8048_spi",
+		.bus    = &spi_bus_type,
+		.owner  = THIS_MODULE,
+	},
+};
+
+static int __init nec_8048_lcd_init(void)
+{
+	return spi_register_driver(&nec_8048_spi_driver);
+}
+
+static void __exit nec_8048_lcd_exit(void)
+{
+	return spi_unregister_driver(&nec_8048_spi_driver);
+}
+
+module_init(nec_8048_lcd_init);
+module_exit(nec_8048_lcd_exit);
+MODULE_LICENSE("GPL");
+
-- 
1.5.6.3

Regards,
Kishore Y

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

* RE: [PATCH V2] OMAP: DSS: Add NEC NL8048HL11-01B display panel
  2010-02-11  8:21 ` [PATCH V2] OMAP: DSS: Add NEC NL8048HL11-01B display panel Y, Kishore
@ 2010-02-11  8:44   ` Hiremath, Vaibhav
  2010-02-11  8:52     ` Y, Kishore
  0 siblings, 1 reply; 4+ messages in thread
From: Hiremath, Vaibhav @ 2010-02-11  8:44 UTC (permalink / raw)
  To: Y, Kishore, Tomi Valkeinen
  Cc: Aguirre, Sergio, Christensen, Mikkel, linux-omap

> -----Original Message-----
> From: Y, Kishore
> Sent: Thursday, February 11, 2010 1:51 PM
> To: Tomi Valkeinen
> Cc: Hiremath, Vaibhav; Aguirre, Sergio; Christensen, Mikkel; linux-
> omap@vger.kernel.org
> Subject: [PATCH V2] OMAP: DSS: Add NEC NL8048HL11-01B display panel
> 
> From: Erik Gilling <konkers@android.com>
> 
> NEC WVGA LCD NL8048HL11-01B support has been added.
> 
> Signed-off-by: Mukund Mittal <mmittal@ti.com>
> Signed-off-by: Kishore Y <kishore.y@ti.com>
> ---
> Changes from version 1:
> 	incorporated cosmetic comments.
> 
>  drivers/video/omap2/displays/Kconfig               |    6 +
>  drivers/video/omap2/displays/Makefile              |    1 +
>  .../omap2/displays/panel-nec-NL8048HL11-01B.c      |  284
> ++++++++++++++++++++
>  3 files changed, 291 insertions(+), 0 deletions(-)
>  create mode 100644 drivers/video/omap2/displays/panel-nec-
> NL8048HL11-01B.c
> 
> diff --git a/drivers/video/omap2/displays/Kconfig
> b/drivers/video/omap2/displays/Kconfig
> index dfb57ee..010ed48 100644
> --- a/drivers/video/omap2/displays/Kconfig
> +++ b/drivers/video/omap2/displays/Kconfig
> @@ -37,4 +37,10 @@ config PANEL_TPO_TD043MTEA1
>          help
>            LCD Panel used in OMAP3 Pandora
> 
> +config PANEL_NEC_NL8048HL11_01B
> +        tristate "NEC NL8048HL11-01B Panel"
> +        depends on OMAP2_DSS
> +        help
> +          LCD Panel from NEC.
> +
>  endmenu
> diff --git a/drivers/video/omap2/displays/Makefile
> b/drivers/video/omap2/displays/Makefile
> index e2bb321..9aace29 100644
> --- a/drivers/video/omap2/displays/Makefile
> +++ b/drivers/video/omap2/displays/Makefile
> @@ -5,3 +5,4 @@ obj-$(CONFIG_PANEL_SHARP_LQ043T1DG01) += panel-
> sharp-lq043t1dg01.o
>  obj-$(CONFIG_PANEL_TAAL) += panel-taal.o
>  obj-$(CONFIG_PANEL_TOPPOLY_TDO35S) += panel-toppoly-tdo35s.o
>  obj-$(CONFIG_PANEL_TPO_TD043MTEA1) += panel-tpo-td043mtea1.o
> +obj-$(CONFIG_PANEL_NEC_NL8048HL11_01B) += panel-nec-NL8048HL11-
> 01B.o
[Hiremath, Vaibhav] I think you have submitted patch from your working directory and not linux-master. If I understand correctly we do not have these panels supported in linux-omap. Please rebase it with linux-omap and re-send.

One small change mentioned below -

> diff --git a/drivers/video/omap2/displays/panel-nec-NL8048HL11-01B.c
> b/drivers/video/omap2/displays/panel-nec-NL8048HL11-01B.c
> new file mode 100644
> index 0000000..4d7e8ab
> --- /dev/null
> +++ b/drivers/video/omap2/displays/panel-nec-NL8048HL11-01B.c
> @@ -0,0 +1,284 @@
> +/*
> + * NEC panel support
> + *
[Hiremath, Vaibhav] copy-right is missing.

> + * This program is free software; you can redistribute it and/or
> modify it
> + * under the terms of the GNU General Public License version 2 as
> published by
> + * the Free Software Foundation.
> + *
> + * This program is distributed in the hope that it will be useful,
> but WITHOUT
> + * ANY WARRANTY; without even the implied warranty of
> MERCHANTABILITY or
> + * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public
> License for
> + * more details.
> + *
> + * You should have received a copy of the GNU General Public
> License along with
> + * this program.  If not, see <http://www.gnu.org/licenses/>.
> + */
> +
> +#include <linux/module.h>
> +#include <linux/delay.h>
> +#include <linux/platform_device.h>
> +#include <linux/spi/spi.h>
[Hiremath, Vaibhav] you may want to add extra blank line here
> +#include <asm/mach-types.h>

[Hiremath, Vaibhav] ditto.

> +#include <plat/gpio.h>
> +#include <plat/mux.h>
> +#include <plat/control.h>
> +#include <plat/display.h>
> +
> +#define LCD_XRES		800
> +#define LCD_YRES		480
> +/* NEC PIX Clock Ratings
> + * MINi:21.8MHz TYP:23.8MHz MAX:25.7MHz
> + */
> +#define LCD_PIXEL_CLOCK		23800
> +
> +/* NEC NL8048HL11-01B  Manual
> + * defines HFB, HSW, HBP, VFP, VSW, VBP as shown below
> + */
> +
> +static struct omap_video_timings nec_8048_panel_timings = {
> +	/* 800 x 480 @ 60 Hz  Reduced blanking VESA CVT 0.31M3-R */
> +	.x_res          = LCD_XRES,
> +	.y_res          = LCD_YRES,
> +	.pixel_clock    = LCD_PIXEL_CLOCK,
> +	.hfp            = 6,
> +	.hsw            = 1,
> +	.hbp            = 4,
> +	.vfp            = 3,
> +	.vsw            = 1,
> +	.vbp            = 4,
> +};
> +
> +static int nec_8048_panel_probe(struct omap_dss_device *dssdev)
> +{
> +	dssdev->panel.config = OMAP_DSS_LCD_TFT | OMAP_DSS_LCD_IVS |
> +				OMAP_DSS_LCD_IHS | OMAP_DSS_LCD_RF |
> +				OMAP_DSS_LCD_ONOFF;
> +	dssdev->panel.timings = nec_8048_panel_timings;
> +	dssdev->panel.recommended_bpp = 16;
> +
> +	return 0;
> +}
> +
> +static void nec_8048_panel_remove(struct omap_dss_device *dssdev)
> +{
> +}
> +
> +static int nec_8048_panel_enable(struct omap_dss_device *dssdev)
> +{
> +	int r = 0;
> +
> +	/* Delay recommended by panel DATASHEET */
> +	mdelay(4);
> +	if (dssdev->platform_enable)
> +		r = dssdev->platform_enable(dssdev);
> +
> +	return r;
> +}
> +
> +static void nec_8048_panel_disable(struct omap_dss_device *dssdev)
> +{
> +	if (dssdev->platform_disable)
> +		dssdev->platform_disable(dssdev);
> +	/* Delay recommended by panel DATASHEET */
> +	mdelay(4);
> +}
> +
> +static int nec_8048_panel_suspend(struct omap_dss_device *dssdev)
> +{
> +	nec_8048_panel_disable(dssdev);
> +	return 0;
> +}
> +
> +static int nec_8048_panel_resume(struct omap_dss_device *dssdev)
> +{
> +	return nec_8048_panel_enable(dssdev);
> +}
> +
> +static struct omap_dss_driver nec_8048_driver = {
> +	.probe          = nec_8048_panel_probe,
> +	.remove         = nec_8048_panel_remove,
> +	.enable         = nec_8048_panel_enable,
> +	.disable        = nec_8048_panel_disable,
> +	.suspend        = nec_8048_panel_suspend,
> +	.resume         = nec_8048_panel_resume,
> +
> +	.driver		= {
> +		.name	= "NEC_8048_panel",
> +		.owner 	= THIS_MODULE,
> +	},
> +};
> +
> +static int
> +spi_send(struct spi_device *spi, unsigned char reg_addr, unsigned
> char reg_data)
> +{
> +	int ret = 0;
> +	unsigned int cmd = 0;
> +	unsigned int data = 0;
> +
> +	cmd = 0x0000 | reg_addr; /* register address write */
> +	data = 0x0100 | reg_data ; /* register data write */
> +	data = (cmd << 16) | data;
> +	ret = spi_write(spi, (unsigned char *)&data, 4);
> +	if (ret) {
> +		printk(KERN_ERR "error in spi_write %x\n", data);
> +		return ret;
> +	}
> +
> +	/* Delay, part of init seqence recommended by panel DATASHEET
> */
> +	udelay(10);
> +	return 0;
> +}
> +
> +static int init_nec_8048_wvga_lcd(struct spi_device *spi)
> +{
> +	/* Initialization Sequence */
> +	/* spi_send(spi, REG, VAL) */
> +	spi_send(spi, 3, 0x01);
> +	spi_send(spi, 0, 0x00);
> +	spi_send(spi, 1, 0x01);    /* R1 = 0x01 (normal), 0x03
> (reversed) */
> +	spi_send(spi, 4, 0x00);
> +	spi_send(spi, 5, 0x14);
> +	spi_send(spi, 6, 0x24);
> +	spi_send(spi, 16, 0xD7);
> +	spi_send(spi, 17, 0x00);
> +	spi_send(spi, 18, 0x00);
> +	spi_send(spi, 19, 0x55);
> +	spi_send(spi, 20, 0x01);
> +	spi_send(spi, 21, 0x70);
> +	spi_send(spi, 22, 0x1E);
> +	spi_send(spi, 23, 0x25);
> +	spi_send(spi, 24, 0x25);
> +	spi_send(spi, 25, 0x02);
> +	spi_send(spi, 26, 0x02);
> +	spi_send(spi, 27, 0xA0);
> +	spi_send(spi, 32, 0x2F);
> +	spi_send(spi, 33, 0x0F);
> +	spi_send(spi, 34, 0x0F);
> +	spi_send(spi, 35, 0x0F);
> +	spi_send(spi, 36, 0x0F);
> +	spi_send(spi, 37, 0x0F);
> +	spi_send(spi, 38, 0x0F);
> +	spi_send(spi, 39, 0x00);
> +	spi_send(spi, 40, 0x02);
> +	spi_send(spi, 41, 0x02);
> +	spi_send(spi, 42, 0x02);
> +	spi_send(spi, 43, 0x0F);
> +	spi_send(spi, 44, 0x0F);
> +	spi_send(spi, 45, 0x0F);
> +	spi_send(spi, 46, 0x0F);
> +	spi_send(spi, 47, 0x0F);
> +	spi_send(spi, 48, 0x0F);
> +	spi_send(spi, 49, 0x0F);
> +	spi_send(spi, 50, 0x00);
> +	spi_send(spi, 51, 0x02);
> +	spi_send(spi, 52, 0x02);
> +	spi_send(spi, 53, 0x02);
> +	spi_send(spi, 80, 0x0C);
> +	spi_send(spi, 83, 0x42);
> +	spi_send(spi, 84, 0x42);
> +	spi_send(spi, 85, 0x41);
> +	spi_send(spi, 86, 0x14);
> +	spi_send(spi, 89, 0x88);
> +	spi_send(spi, 90, 0x01);
> +	spi_send(spi, 91, 0x00);
> +	spi_send(spi, 92, 0x02);
> +	spi_send(spi, 93, 0x0C);
> +	spi_send(spi, 94, 0x1C);
> +	spi_send(spi, 95, 0x27);
> +	spi_send(spi, 98, 0x49);
> +	spi_send(spi, 99, 0x27);
> +	spi_send(spi, 102, 0x76);
> +	spi_send(spi, 103, 0x27);
> +	spi_send(spi, 112, 0x01);
> +	spi_send(spi, 113, 0x0E);
> +	spi_send(spi, 114, 0x02);
> +	spi_send(spi, 115, 0x0C);
> +	spi_send(spi, 118, 0x0C);
> +	spi_send(spi, 121, 0x30); /* R121 = 0x30 (normal), 0x10
> (reversed) */
> +	spi_send(spi, 130, 0x00);
> +	spi_send(spi, 131, 0x00);
> +	spi_send(spi, 132, 0xFC);
> +	spi_send(spi, 134, 0x00);
> +	spi_send(spi, 136, 0x00);
> +	spi_send(spi, 138, 0x00);
> +	spi_send(spi, 139, 0x00);
> +	spi_send(spi, 140, 0x00);
> +	spi_send(spi, 141, 0xFC);
> +	spi_send(spi, 143, 0x00);
> +	spi_send(spi, 145, 0x00);
> +	spi_send(spi, 147, 0x00);
> +	spi_send(spi, 148, 0x00);
> +	spi_send(spi, 149, 0x00);
> +	spi_send(spi, 150, 0xFC);
> +	spi_send(spi, 152, 0x00);
> +	spi_send(spi, 154, 0x00);
> +	spi_send(spi, 156, 0x00);
> +	spi_send(spi, 157, 0x00);
> +	udelay(20);
> +	spi_send(spi, 2, 0x00);
> +	return 0;
> +}
> +
> +static int nec_8048_spi_probe(struct spi_device *spi)
> +{
> +	spi->mode = SPI_MODE_0;
> +	spi->bits_per_word = 32;
> +	spi_setup(spi);
> +
> +	init_nec_8048_wvga_lcd(spi);
> +
> +	omap_dss_register_driver(&nec_8048_driver);
[Hiremath, Vaibhav] Should be something

return omap_dss_register_driver(&nec_8048_driver);

Thanks,
Vaibhav

> +	return 0;
> +}
> +
> +static int nec_8048_spi_remove(struct spi_device *spi)
> +{
> +	omap_dss_unregister_driver(&nec_8048_driver);
> +
> +	return 0;
> +}
> +
> +static int nec_8048_spi_suspend(struct spi_device *spi,
> pm_message_t mesg)
> +{
> +	spi_send(spi, 2, 0x01);
> +	mdelay(40);
> +
> +	return 0;
> +}
> +
> +static int nec_8048_spi_resume(struct spi_device *spi)
> +{
> +	/* reinitialize the panel */
> +	spi_setup(spi);
> +	spi_send(spi, 2, 0x00);
> +	init_nec_8048_wvga_lcd(spi);
> +
> +	return 0;
> +}
> +
> +static struct spi_driver nec_8048_spi_driver = {
> +	.probe           = nec_8048_spi_probe,
> +	.remove	= __devexit_p(nec_8048_spi_remove),
> +	.suspend         = nec_8048_spi_suspend,
> +	.resume          = nec_8048_spi_resume,
> +	.driver         = {
> +		.name   = "nec_8048_spi",
> +		.bus    = &spi_bus_type,
> +		.owner  = THIS_MODULE,
> +	},
> +};
> +
> +static int __init nec_8048_lcd_init(void)
> +{
> +	return spi_register_driver(&nec_8048_spi_driver);
> +}
> +
> +static void __exit nec_8048_lcd_exit(void)
> +{
> +	return spi_unregister_driver(&nec_8048_spi_driver);
> +}
> +
> +module_init(nec_8048_lcd_init);
> +module_exit(nec_8048_lcd_exit);
> +MODULE_LICENSE("GPL");
> +
> --
> 1.5.6.3
> 
> Regards,
> Kishore Y

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

* RE: [PATCH V2] OMAP: DSS: Add NEC NL8048HL11-01B display panel
  2010-02-11  8:44   ` Hiremath, Vaibhav
@ 2010-02-11  8:52     ` Y, Kishore
  2010-02-11  9:11       ` Hiremath, Vaibhav
  0 siblings, 1 reply; 4+ messages in thread
From: Y, Kishore @ 2010-02-11  8:52 UTC (permalink / raw)
  To: Hiremath, Vaibhav, Tomi Valkeinen
  Cc: Aguirre, Sergio, Christensen, Mikkel, linux-omap

> -----Original Message-----
> From: Hiremath, Vaibhav
> Sent: Thursday, February 11, 2010 2:14 PM
> To: Y, Kishore; Tomi Valkeinen
> Cc: Aguirre, Sergio; Christensen, Mikkel; linux-omap@vger.kernel.org
> Subject: RE: [PATCH V2] OMAP: DSS: Add NEC NL8048HL11-01B display panel
> 
> > -----Original Message-----
> > From: Y, Kishore
> > Sent: Thursday, February 11, 2010 1:51 PM
> > To: Tomi Valkeinen
> > Cc: Hiremath, Vaibhav; Aguirre, Sergio; Christensen, Mikkel; linux-
> > omap@vger.kernel.org
> > Subject: [PATCH V2] OMAP: DSS: Add NEC NL8048HL11-01B display panel
> >
> > From: Erik Gilling <konkers@android.com>
> >
> > NEC WVGA LCD NL8048HL11-01B support has been added.
> >
> > Signed-off-by: Mukund Mittal <mmittal@ti.com>
> > Signed-off-by: Kishore Y <kishore.y@ti.com>
> > ---
> > Changes from version 1:
> > 	incorporated cosmetic comments.
> >
> >  drivers/video/omap2/displays/Kconfig               |    6 +
> >  drivers/video/omap2/displays/Makefile              |    1 +
> >  .../omap2/displays/panel-nec-NL8048HL11-01B.c      |  284
> > ++++++++++++++++++++
> >  3 files changed, 291 insertions(+), 0 deletions(-)
> >  create mode 100644 drivers/video/omap2/displays/panel-nec-
> > NL8048HL11-01B.c
> >
> > diff --git a/drivers/video/omap2/displays/Kconfig
> > b/drivers/video/omap2/displays/Kconfig
> > index dfb57ee..010ed48 100644
> > --- a/drivers/video/omap2/displays/Kconfig
> > +++ b/drivers/video/omap2/displays/Kconfig
> > @@ -37,4 +37,10 @@ config PANEL_TPO_TD043MTEA1
> >          help
> >            LCD Panel used in OMAP3 Pandora
> >
> > +config PANEL_NEC_NL8048HL11_01B
> > +        tristate "NEC NL8048HL11-01B Panel"
> > +        depends on OMAP2_DSS
> > +        help
> > +          LCD Panel from NEC.
> > +
> >  endmenu
> > diff --git a/drivers/video/omap2/displays/Makefile
> > b/drivers/video/omap2/displays/Makefile
> > index e2bb321..9aace29 100644
> > --- a/drivers/video/omap2/displays/Makefile
> > +++ b/drivers/video/omap2/displays/Makefile
> > @@ -5,3 +5,4 @@ obj-$(CONFIG_PANEL_SHARP_LQ043T1DG01) += panel-
> > sharp-lq043t1dg01.o
> >  obj-$(CONFIG_PANEL_TAAL) += panel-taal.o
> >  obj-$(CONFIG_PANEL_TOPPOLY_TDO35S) += panel-toppoly-tdo35s.o
> >  obj-$(CONFIG_PANEL_TPO_TD043MTEA1) += panel-tpo-td043mtea1.o
> > +obj-$(CONFIG_PANEL_NEC_NL8048HL11_01B) += panel-nec-NL8048HL11-
> > 01B.o
> [Hiremath, Vaibhav] I think you have submitted patch from your working
> directory and not linux-master. If I understand correctly we do not have
> these panels supported in linux-omap. Please rebase it with linux-omap and
> re-send.

I actually tested thin on dss2 branch of LO and not master branch. I would rebase it to master and send the next version taking in other comments.

> 
> One small change mentioned below -
> 
> > diff --git a/drivers/video/omap2/displays/panel-nec-NL8048HL11-01B.c
> > b/drivers/video/omap2/displays/panel-nec-NL8048HL11-01B.c
> > new file mode 100644
> > index 0000000..4d7e8ab
> > --- /dev/null
> > +++ b/drivers/video/omap2/displays/panel-nec-NL8048HL11-01B.c
> > @@ -0,0 +1,284 @@
> > +/*
> > + * NEC panel support
> > + *
> [Hiremath, Vaibhav] copy-right is missing.
I will add it.
> 
> > + * This program is free software; you can redistribute it and/or
> > modify it
> > + * under the terms of the GNU General Public License version 2 as
> > published by
> > + * the Free Software Foundation.
> > + *
> > + * This program is distributed in the hope that it will be useful,
> > but WITHOUT
> > + * ANY WARRANTY; without even the implied warranty of
> > MERCHANTABILITY or
> > + * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public
> > License for
> > + * more details.
> > + *
> > + * You should have received a copy of the GNU General Public
> > License along with
> > + * this program.  If not, see <http://www.gnu.org/licenses/>.
> > + */
> > +
> > +#include <linux/module.h>
> > +#include <linux/delay.h>
> > +#include <linux/platform_device.h>
> > +#include <linux/spi/spi.h>
> [Hiremath, Vaibhav] you may want to add extra blank line here
> > +#include <asm/mach-types.h>
> 
> [Hiremath, Vaibhav] ditto.
> 
> > +#include <plat/gpio.h>
> > +#include <plat/mux.h>
> > +#include <plat/control.h>
> > +#include <plat/display.h>
> > +
> > +#define LCD_XRES		800
> > +#define LCD_YRES		480
> > +/* NEC PIX Clock Ratings
> > + * MINi:21.8MHz TYP:23.8MHz MAX:25.7MHz
> > + */
> > +#define LCD_PIXEL_CLOCK		23800
> > +
> > +/* NEC NL8048HL11-01B  Manual
> > + * defines HFB, HSW, HBP, VFP, VSW, VBP as shown below
> > + */
> > +
> > +static struct omap_video_timings nec_8048_panel_timings = {
> > +	/* 800 x 480 @ 60 Hz  Reduced blanking VESA CVT 0.31M3-R */
> > +	.x_res          = LCD_XRES,
> > +	.y_res          = LCD_YRES,
> > +	.pixel_clock    = LCD_PIXEL_CLOCK,
> > +	.hfp            = 6,
> > +	.hsw            = 1,
> > +	.hbp            = 4,
> > +	.vfp            = 3,
> > +	.vsw            = 1,
> > +	.vbp            = 4,
> > +};
> > +
> > +static int nec_8048_panel_probe(struct omap_dss_device *dssdev)
> > +{
> > +	dssdev->panel.config = OMAP_DSS_LCD_TFT | OMAP_DSS_LCD_IVS |
> > +				OMAP_DSS_LCD_IHS | OMAP_DSS_LCD_RF |
> > +				OMAP_DSS_LCD_ONOFF;
> > +	dssdev->panel.timings = nec_8048_panel_timings;
> > +	dssdev->panel.recommended_bpp = 16;
> > +
> > +	return 0;
> > +}
> > +
> > +static void nec_8048_panel_remove(struct omap_dss_device *dssdev)
> > +{
> > +}
> > +
> > +static int nec_8048_panel_enable(struct omap_dss_device *dssdev)
> > +{
> > +	int r = 0;
> > +
> > +	/* Delay recommended by panel DATASHEET */
> > +	mdelay(4);
> > +	if (dssdev->platform_enable)
> > +		r = dssdev->platform_enable(dssdev);
> > +
> > +	return r;
> > +}
> > +
> > +static void nec_8048_panel_disable(struct omap_dss_device *dssdev)
> > +{
> > +	if (dssdev->platform_disable)
> > +		dssdev->platform_disable(dssdev);
> > +	/* Delay recommended by panel DATASHEET */
> > +	mdelay(4);
> > +}
> > +
> > +static int nec_8048_panel_suspend(struct omap_dss_device *dssdev)
> > +{
> > +	nec_8048_panel_disable(dssdev);
> > +	return 0;
> > +}
> > +
> > +static int nec_8048_panel_resume(struct omap_dss_device *dssdev)
> > +{
> > +	return nec_8048_panel_enable(dssdev);
> > +}
> > +
> > +static struct omap_dss_driver nec_8048_driver = {
> > +	.probe          = nec_8048_panel_probe,
> > +	.remove         = nec_8048_panel_remove,
> > +	.enable         = nec_8048_panel_enable,
> > +	.disable        = nec_8048_panel_disable,
> > +	.suspend        = nec_8048_panel_suspend,
> > +	.resume         = nec_8048_panel_resume,
> > +
> > +	.driver		= {
> > +		.name	= "NEC_8048_panel",
> > +		.owner 	= THIS_MODULE,
> > +	},
> > +};
> > +
> > +static int
> > +spi_send(struct spi_device *spi, unsigned char reg_addr, unsigned
> > char reg_data)
> > +{
> > +	int ret = 0;
> > +	unsigned int cmd = 0;
> > +	unsigned int data = 0;
> > +
> > +	cmd = 0x0000 | reg_addr; /* register address write */
> > +	data = 0x0100 | reg_data ; /* register data write */
> > +	data = (cmd << 16) | data;
> > +	ret = spi_write(spi, (unsigned char *)&data, 4);
> > +	if (ret) {
> > +		printk(KERN_ERR "error in spi_write %x\n", data);
> > +		return ret;
> > +	}
> > +
> > +	/* Delay, part of init seqence recommended by panel DATASHEET
> > */
> > +	udelay(10);
> > +	return 0;
> > +}
> > +
> > +static int init_nec_8048_wvga_lcd(struct spi_device *spi)
> > +{
> > +	/* Initialization Sequence */
> > +	/* spi_send(spi, REG, VAL) */
> > +	spi_send(spi, 3, 0x01);
> > +	spi_send(spi, 0, 0x00);
> > +	spi_send(spi, 1, 0x01);    /* R1 = 0x01 (normal), 0x03
> > (reversed) */
> > +	spi_send(spi, 4, 0x00);
> > +	spi_send(spi, 5, 0x14);
> > +	spi_send(spi, 6, 0x24);
> > +	spi_send(spi, 16, 0xD7);
> > +	spi_send(spi, 17, 0x00);
> > +	spi_send(spi, 18, 0x00);
> > +	spi_send(spi, 19, 0x55);
> > +	spi_send(spi, 20, 0x01);
> > +	spi_send(spi, 21, 0x70);
> > +	spi_send(spi, 22, 0x1E);
> > +	spi_send(spi, 23, 0x25);
> > +	spi_send(spi, 24, 0x25);
> > +	spi_send(spi, 25, 0x02);
> > +	spi_send(spi, 26, 0x02);
> > +	spi_send(spi, 27, 0xA0);
> > +	spi_send(spi, 32, 0x2F);
> > +	spi_send(spi, 33, 0x0F);
> > +	spi_send(spi, 34, 0x0F);
> > +	spi_send(spi, 35, 0x0F);
> > +	spi_send(spi, 36, 0x0F);
> > +	spi_send(spi, 37, 0x0F);
> > +	spi_send(spi, 38, 0x0F);
> > +	spi_send(spi, 39, 0x00);
> > +	spi_send(spi, 40, 0x02);
> > +	spi_send(spi, 41, 0x02);
> > +	spi_send(spi, 42, 0x02);
> > +	spi_send(spi, 43, 0x0F);
> > +	spi_send(spi, 44, 0x0F);
> > +	spi_send(spi, 45, 0x0F);
> > +	spi_send(spi, 46, 0x0F);
> > +	spi_send(spi, 47, 0x0F);
> > +	spi_send(spi, 48, 0x0F);
> > +	spi_send(spi, 49, 0x0F);
> > +	spi_send(spi, 50, 0x00);
> > +	spi_send(spi, 51, 0x02);
> > +	spi_send(spi, 52, 0x02);
> > +	spi_send(spi, 53, 0x02);
> > +	spi_send(spi, 80, 0x0C);
> > +	spi_send(spi, 83, 0x42);
> > +	spi_send(spi, 84, 0x42);
> > +	spi_send(spi, 85, 0x41);
> > +	spi_send(spi, 86, 0x14);
> > +	spi_send(spi, 89, 0x88);
> > +	spi_send(spi, 90, 0x01);
> > +	spi_send(spi, 91, 0x00);
> > +	spi_send(spi, 92, 0x02);
> > +	spi_send(spi, 93, 0x0C);
> > +	spi_send(spi, 94, 0x1C);
> > +	spi_send(spi, 95, 0x27);
> > +	spi_send(spi, 98, 0x49);
> > +	spi_send(spi, 99, 0x27);
> > +	spi_send(spi, 102, 0x76);
> > +	spi_send(spi, 103, 0x27);
> > +	spi_send(spi, 112, 0x01);
> > +	spi_send(spi, 113, 0x0E);
> > +	spi_send(spi, 114, 0x02);
> > +	spi_send(spi, 115, 0x0C);
> > +	spi_send(spi, 118, 0x0C);
> > +	spi_send(spi, 121, 0x30); /* R121 = 0x30 (normal), 0x10
> > (reversed) */
> > +	spi_send(spi, 130, 0x00);
> > +	spi_send(spi, 131, 0x00);
> > +	spi_send(spi, 132, 0xFC);
> > +	spi_send(spi, 134, 0x00);
> > +	spi_send(spi, 136, 0x00);
> > +	spi_send(spi, 138, 0x00);
> > +	spi_send(spi, 139, 0x00);
> > +	spi_send(spi, 140, 0x00);
> > +	spi_send(spi, 141, 0xFC);
> > +	spi_send(spi, 143, 0x00);
> > +	spi_send(spi, 145, 0x00);
> > +	spi_send(spi, 147, 0x00);
> > +	spi_send(spi, 148, 0x00);
> > +	spi_send(spi, 149, 0x00);
> > +	spi_send(spi, 150, 0xFC);
> > +	spi_send(spi, 152, 0x00);
> > +	spi_send(spi, 154, 0x00);
> > +	spi_send(spi, 156, 0x00);
> > +	spi_send(spi, 157, 0x00);
> > +	udelay(20);
> > +	spi_send(spi, 2, 0x00);
> > +	return 0;
> > +}
> > +
> > +static int nec_8048_spi_probe(struct spi_device *spi)
> > +{
> > +	spi->mode = SPI_MODE_0;
> > +	spi->bits_per_word = 32;
> > +	spi_setup(spi);
> > +
> > +	init_nec_8048_wvga_lcd(spi);
> > +
> > +	omap_dss_register_driver(&nec_8048_driver);
> [Hiremath, Vaibhav] Should be something
> 
> return omap_dss_register_driver(&nec_8048_driver);
will change this
> 
> Thanks,
> Vaibhav
> 
> > +	return 0;
> > +}
> > +
> > +static int nec_8048_spi_remove(struct spi_device *spi)
> > +{
> > +	omap_dss_unregister_driver(&nec_8048_driver);
> > +
> > +	return 0;
> > +}
> > +
> > +static int nec_8048_spi_suspend(struct spi_device *spi,
> > pm_message_t mesg)
> > +{
> > +	spi_send(spi, 2, 0x01);
> > +	mdelay(40);
> > +
> > +	return 0;
> > +}
> > +
> > +static int nec_8048_spi_resume(struct spi_device *spi)
> > +{
> > +	/* reinitialize the panel */
> > +	spi_setup(spi);
> > +	spi_send(spi, 2, 0x00);
> > +	init_nec_8048_wvga_lcd(spi);
> > +
> > +	return 0;
> > +}
> > +
> > +static struct spi_driver nec_8048_spi_driver = {
> > +	.probe           = nec_8048_spi_probe,
> > +	.remove	= __devexit_p(nec_8048_spi_remove),
> > +	.suspend         = nec_8048_spi_suspend,
> > +	.resume          = nec_8048_spi_resume,
> > +	.driver         = {
> > +		.name   = "nec_8048_spi",
> > +		.bus    = &spi_bus_type,
> > +		.owner  = THIS_MODULE,
> > +	},
> > +};
> > +
> > +static int __init nec_8048_lcd_init(void)
> > +{
> > +	return spi_register_driver(&nec_8048_spi_driver);
> > +}
> > +
> > +static void __exit nec_8048_lcd_exit(void)
> > +{
> > +	return spi_unregister_driver(&nec_8048_spi_driver);
> > +}
> > +
> > +module_init(nec_8048_lcd_init);
> > +module_exit(nec_8048_lcd_exit);
> > +MODULE_LICENSE("GPL");
> > +
> > --
> > 1.5.6.3
> >
> > Regards,
> > Kishore Y

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

* RE: [PATCH V2] OMAP: DSS: Add NEC NL8048HL11-01B display panel
  2010-02-11  8:52     ` Y, Kishore
@ 2010-02-11  9:11       ` Hiremath, Vaibhav
  0 siblings, 0 replies; 4+ messages in thread
From: Hiremath, Vaibhav @ 2010-02-11  9:11 UTC (permalink / raw)
  To: Y, Kishore, Tomi Valkeinen
  Cc: Aguirre, Sergio, Christensen, Mikkel, linux-omap


> -----Original Message-----
> From: Y, Kishore
> Sent: Thursday, February 11, 2010 2:23 PM
> To: Hiremath, Vaibhav; Tomi Valkeinen
> Cc: Aguirre, Sergio; Christensen, Mikkel; linux-omap@vger.kernel.org
> Subject: RE: [PATCH V2] OMAP: DSS: Add NEC NL8048HL11-01B display
> panel
> 
> > -----Original Message-----
> > From: Hiremath, Vaibhav
> > Sent: Thursday, February 11, 2010 2:14 PM
> > To: Y, Kishore; Tomi Valkeinen
> > Cc: Aguirre, Sergio; Christensen, Mikkel; linux-
> omap@vger.kernel.org
> > Subject: RE: [PATCH V2] OMAP: DSS: Add NEC NL8048HL11-01B display
> panel
> >
> > > -----Original Message-----
> > > From: Y, Kishore
> > > Sent: Thursday, February 11, 2010 1:51 PM
> > > To: Tomi Valkeinen
> > > Cc: Hiremath, Vaibhav; Aguirre, Sergio; Christensen, Mikkel;
> linux-
> > > omap@vger.kernel.org
> > > Subject: [PATCH V2] OMAP: DSS: Add NEC NL8048HL11-01B display
> panel
> > >
> > > From: Erik Gilling <konkers@android.com>
> > >
> > > NEC WVGA LCD NL8048HL11-01B support has been added.
> > >
> > > Signed-off-by: Mukund Mittal <mmittal@ti.com>
> > > Signed-off-by: Kishore Y <kishore.y@ti.com>
> > > ---
> > > Changes from version 1:
> > > 	incorporated cosmetic comments.
> > >
> > >  drivers/video/omap2/displays/Kconfig               |    6 +
> > >  drivers/video/omap2/displays/Makefile              |    1 +
> > >  .../omap2/displays/panel-nec-NL8048HL11-01B.c      |  284
> > > ++++++++++++++++++++
> > >  3 files changed, 291 insertions(+), 0 deletions(-)
> > >  create mode 100644 drivers/video/omap2/displays/panel-nec-
> > > NL8048HL11-01B.c
> > >
> > > diff --git a/drivers/video/omap2/displays/Kconfig
> > > b/drivers/video/omap2/displays/Kconfig
> > > index dfb57ee..010ed48 100644
> > > --- a/drivers/video/omap2/displays/Kconfig
> > > +++ b/drivers/video/omap2/displays/Kconfig
> > > @@ -37,4 +37,10 @@ config PANEL_TPO_TD043MTEA1
> > >          help
> > >            LCD Panel used in OMAP3 Pandora
> > >
> > > +config PANEL_NEC_NL8048HL11_01B
> > > +        tristate "NEC NL8048HL11-01B Panel"
> > > +        depends on OMAP2_DSS
> > > +        help
> > > +          LCD Panel from NEC.
> > > +
> > >  endmenu
> > > diff --git a/drivers/video/omap2/displays/Makefile
> > > b/drivers/video/omap2/displays/Makefile
> > > index e2bb321..9aace29 100644
> > > --- a/drivers/video/omap2/displays/Makefile
> > > +++ b/drivers/video/omap2/displays/Makefile
> > > @@ -5,3 +5,4 @@ obj-$(CONFIG_PANEL_SHARP_LQ043T1DG01) += panel-
> > > sharp-lq043t1dg01.o
> > >  obj-$(CONFIG_PANEL_TAAL) += panel-taal.o
> > >  obj-$(CONFIG_PANEL_TOPPOLY_TDO35S) += panel-toppoly-tdo35s.o
> > >  obj-$(CONFIG_PANEL_TPO_TD043MTEA1) += panel-tpo-td043mtea1.o
> > > +obj-$(CONFIG_PANEL_NEC_NL8048HL11_01B) += panel-nec-NL8048HL11-
> > > 01B.o
> > [Hiremath, Vaibhav] I think you have submitted patch from your
> working
> > directory and not linux-master. If I understand correctly we do
> not have
> > these panels supported in linux-omap. Please rebase it with linux-
> omap and
> > re-send.
> 
> I actually tested thin on dss2 branch of LO and not master branch. I
> would rebase it to master and send the next version taking in other
> comments.
> 
[Hiremath, Vaibhav] In that case you are following right method. No need to rebase it to master, since Tomi gives pull request.


> >
> > One small change mentioned below -
> >
> > > diff --git a/drivers/video/omap2/displays/panel-nec-NL8048HL11-
> 01B.c
> > > b/drivers/video/omap2/displays/panel-nec-NL8048HL11-01B.c
> > > new file mode 100644
> > > index 0000000..4d7e8ab
> > > --- /dev/null
> > > +++ b/drivers/video/omap2/displays/panel-nec-NL8048HL11-01B.c
> > > @@ -0,0 +1,284 @@
> > > +/*
> > > + * NEC panel support
> > > + *
> > [Hiremath, Vaibhav] copy-right is missing.
> I will add it.
> >
> > > + * This program is free software; you can redistribute it
> and/or
> > > modify it
> > > + * under the terms of the GNU General Public License version 2
> as
> > > published by
> > > + * the Free Software Foundation.
> > > + *
> > > + * This program is distributed in the hope that it will be
> useful,
> > > but WITHOUT
> > > + * ANY WARRANTY; without even the implied warranty of
> > > MERCHANTABILITY or
> > > + * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General
> Public
> > > License for
> > > + * more details.
> > > + *
> > > + * You should have received a copy of the GNU General Public
> > > License along with
> > > + * this program.  If not, see <http://www.gnu.org/licenses/>.
> > > + */
> > > +
> > > +#include <linux/module.h>
> > > +#include <linux/delay.h>
> > > +#include <linux/platform_device.h>
> > > +#include <linux/spi/spi.h>
> > [Hiremath, Vaibhav] you may want to add extra blank line here
> > > +#include <asm/mach-types.h>
> >
> > [Hiremath, Vaibhav] ditto.
> >
> > > +#include <plat/gpio.h>
> > > +#include <plat/mux.h>
> > > +#include <plat/control.h>
> > > +#include <plat/display.h>
> > > +
> > > +#define LCD_XRES		800
> > > +#define LCD_YRES		480
> > > +/* NEC PIX Clock Ratings
> > > + * MINi:21.8MHz TYP:23.8MHz MAX:25.7MHz
> > > + */
> > > +#define LCD_PIXEL_CLOCK		23800
> > > +
> > > +/* NEC NL8048HL11-01B  Manual
> > > + * defines HFB, HSW, HBP, VFP, VSW, VBP as shown below
> > > + */
> > > +
> > > +static struct omap_video_timings nec_8048_panel_timings = {
> > > +	/* 800 x 480 @ 60 Hz  Reduced blanking VESA CVT 0.31M3-R */
> > > +	.x_res          = LCD_XRES,
> > > +	.y_res          = LCD_YRES,
> > > +	.pixel_clock    = LCD_PIXEL_CLOCK,
> > > +	.hfp            = 6,
> > > +	.hsw            = 1,
> > > +	.hbp            = 4,
> > > +	.vfp            = 3,
> > > +	.vsw            = 1,
> > > +	.vbp            = 4,
> > > +};
> > > +
> > > +static int nec_8048_panel_probe(struct omap_dss_device *dssdev)
> > > +{
> > > +	dssdev->panel.config = OMAP_DSS_LCD_TFT | OMAP_DSS_LCD_IVS |
> > > +				OMAP_DSS_LCD_IHS | OMAP_DSS_LCD_RF |
> > > +				OMAP_DSS_LCD_ONOFF;
> > > +	dssdev->panel.timings = nec_8048_panel_timings;
> > > +	dssdev->panel.recommended_bpp = 16;
> > > +
> > > +	return 0;
> > > +}
> > > +
> > > +static void nec_8048_panel_remove(struct omap_dss_device
> *dssdev)
> > > +{
> > > +}
> > > +
> > > +static int nec_8048_panel_enable(struct omap_dss_device
> *dssdev)
> > > +{
> > > +	int r = 0;
> > > +
> > > +	/* Delay recommended by panel DATASHEET */
> > > +	mdelay(4);
> > > +	if (dssdev->platform_enable)
> > > +		r = dssdev->platform_enable(dssdev);
> > > +
> > > +	return r;
> > > +}
> > > +
> > > +static void nec_8048_panel_disable(struct omap_dss_device
> *dssdev)
> > > +{
> > > +	if (dssdev->platform_disable)
> > > +		dssdev->platform_disable(dssdev);
> > > +	/* Delay recommended by panel DATASHEET */
> > > +	mdelay(4);
> > > +}
> > > +
> > > +static int nec_8048_panel_suspend(struct omap_dss_device
> *dssdev)
> > > +{
> > > +	nec_8048_panel_disable(dssdev);
> > > +	return 0;
> > > +}
> > > +
> > > +static int nec_8048_panel_resume(struct omap_dss_device
> *dssdev)
> > > +{
> > > +	return nec_8048_panel_enable(dssdev);
> > > +}
> > > +
> > > +static struct omap_dss_driver nec_8048_driver = {
> > > +	.probe          = nec_8048_panel_probe,
> > > +	.remove         = nec_8048_panel_remove,
> > > +	.enable         = nec_8048_panel_enable,
> > > +	.disable        = nec_8048_panel_disable,
> > > +	.suspend        = nec_8048_panel_suspend,
> > > +	.resume         = nec_8048_panel_resume,
> > > +
> > > +	.driver		= {
> > > +		.name	= "NEC_8048_panel",
> > > +		.owner 	= THIS_MODULE,
> > > +	},
> > > +};
> > > +
> > > +static int
> > > +spi_send(struct spi_device *spi, unsigned char reg_addr,
> unsigned
> > > char reg_data)
> > > +{
> > > +	int ret = 0;
> > > +	unsigned int cmd = 0;
> > > +	unsigned int data = 0;
> > > +
> > > +	cmd = 0x0000 | reg_addr; /* register address write */
> > > +	data = 0x0100 | reg_data ; /* register data write */
> > > +	data = (cmd << 16) | data;
> > > +	ret = spi_write(spi, (unsigned char *)&data, 4);
> > > +	if (ret) {
> > > +		printk(KERN_ERR "error in spi_write %x\n", data);
> > > +		return ret;
> > > +	}
> > > +
> > > +	/* Delay, part of init seqence recommended by panel DATASHEET
> > > */
> > > +	udelay(10);
> > > +	return 0;
> > > +}
> > > +
> > > +static int init_nec_8048_wvga_lcd(struct spi_device *spi)
> > > +{
> > > +	/* Initialization Sequence */
> > > +	/* spi_send(spi, REG, VAL) */
> > > +	spi_send(spi, 3, 0x01);
> > > +	spi_send(spi, 0, 0x00);
> > > +	spi_send(spi, 1, 0x01);    /* R1 = 0x01 (normal), 0x03
> > > (reversed) */
> > > +	spi_send(spi, 4, 0x00);
> > > +	spi_send(spi, 5, 0x14);
> > > +	spi_send(spi, 6, 0x24);
> > > +	spi_send(spi, 16, 0xD7);
> > > +	spi_send(spi, 17, 0x00);
> > > +	spi_send(spi, 18, 0x00);
> > > +	spi_send(spi, 19, 0x55);
> > > +	spi_send(spi, 20, 0x01);
> > > +	spi_send(spi, 21, 0x70);
> > > +	spi_send(spi, 22, 0x1E);
> > > +	spi_send(spi, 23, 0x25);
> > > +	spi_send(spi, 24, 0x25);
> > > +	spi_send(spi, 25, 0x02);
> > > +	spi_send(spi, 26, 0x02);
> > > +	spi_send(spi, 27, 0xA0);
> > > +	spi_send(spi, 32, 0x2F);
> > > +	spi_send(spi, 33, 0x0F);
> > > +	spi_send(spi, 34, 0x0F);
> > > +	spi_send(spi, 35, 0x0F);
> > > +	spi_send(spi, 36, 0x0F);
> > > +	spi_send(spi, 37, 0x0F);
> > > +	spi_send(spi, 38, 0x0F);
> > > +	spi_send(spi, 39, 0x00);
> > > +	spi_send(spi, 40, 0x02);
> > > +	spi_send(spi, 41, 0x02);
> > > +	spi_send(spi, 42, 0x02);
> > > +	spi_send(spi, 43, 0x0F);
> > > +	spi_send(spi, 44, 0x0F);
> > > +	spi_send(spi, 45, 0x0F);
> > > +	spi_send(spi, 46, 0x0F);
> > > +	spi_send(spi, 47, 0x0F);
> > > +	spi_send(spi, 48, 0x0F);
> > > +	spi_send(spi, 49, 0x0F);
> > > +	spi_send(spi, 50, 0x00);
> > > +	spi_send(spi, 51, 0x02);
> > > +	spi_send(spi, 52, 0x02);
> > > +	spi_send(spi, 53, 0x02);
> > > +	spi_send(spi, 80, 0x0C);
> > > +	spi_send(spi, 83, 0x42);
> > > +	spi_send(spi, 84, 0x42);
> > > +	spi_send(spi, 85, 0x41);
> > > +	spi_send(spi, 86, 0x14);
> > > +	spi_send(spi, 89, 0x88);
> > > +	spi_send(spi, 90, 0x01);
> > > +	spi_send(spi, 91, 0x00);
> > > +	spi_send(spi, 92, 0x02);
> > > +	spi_send(spi, 93, 0x0C);
> > > +	spi_send(spi, 94, 0x1C);
> > > +	spi_send(spi, 95, 0x27);
> > > +	spi_send(spi, 98, 0x49);
> > > +	spi_send(spi, 99, 0x27);
> > > +	spi_send(spi, 102, 0x76);
> > > +	spi_send(spi, 103, 0x27);
> > > +	spi_send(spi, 112, 0x01);
> > > +	spi_send(spi, 113, 0x0E);
> > > +	spi_send(spi, 114, 0x02);
> > > +	spi_send(spi, 115, 0x0C);
> > > +	spi_send(spi, 118, 0x0C);
> > > +	spi_send(spi, 121, 0x30); /* R121 = 0x30 (normal), 0x10
> > > (reversed) */
> > > +	spi_send(spi, 130, 0x00);
> > > +	spi_send(spi, 131, 0x00);
> > > +	spi_send(spi, 132, 0xFC);
> > > +	spi_send(spi, 134, 0x00);
> > > +	spi_send(spi, 136, 0x00);
> > > +	spi_send(spi, 138, 0x00);
> > > +	spi_send(spi, 139, 0x00);
> > > +	spi_send(spi, 140, 0x00);
> > > +	spi_send(spi, 141, 0xFC);
> > > +	spi_send(spi, 143, 0x00);
> > > +	spi_send(spi, 145, 0x00);
> > > +	spi_send(spi, 147, 0x00);
> > > +	spi_send(spi, 148, 0x00);
> > > +	spi_send(spi, 149, 0x00);
> > > +	spi_send(spi, 150, 0xFC);
> > > +	spi_send(spi, 152, 0x00);
> > > +	spi_send(spi, 154, 0x00);
> > > +	spi_send(spi, 156, 0x00);
> > > +	spi_send(spi, 157, 0x00);
> > > +	udelay(20);
> > > +	spi_send(spi, 2, 0x00);
> > > +	return 0;
> > > +}
> > > +
> > > +static int nec_8048_spi_probe(struct spi_device *spi)
> > > +{
> > > +	spi->mode = SPI_MODE_0;
> > > +	spi->bits_per_word = 32;
> > > +	spi_setup(spi);
> > > +
> > > +	init_nec_8048_wvga_lcd(spi);
> > > +
> > > +	omap_dss_register_driver(&nec_8048_driver);
> > [Hiremath, Vaibhav] Should be something
> >
> > return omap_dss_register_driver(&nec_8048_driver);
> will change this
> >
> > Thanks,
> > Vaibhav
> >
> > > +	return 0;
> > > +}
> > > +
> > > +static int nec_8048_spi_remove(struct spi_device *spi)
> > > +{
> > > +	omap_dss_unregister_driver(&nec_8048_driver);
> > > +
> > > +	return 0;
> > > +}
> > > +
> > > +static int nec_8048_spi_suspend(struct spi_device *spi,
> > > pm_message_t mesg)
> > > +{
> > > +	spi_send(spi, 2, 0x01);
> > > +	mdelay(40);
> > > +
> > > +	return 0;
> > > +}
> > > +
> > > +static int nec_8048_spi_resume(struct spi_device *spi)
> > > +{
> > > +	/* reinitialize the panel */
> > > +	spi_setup(spi);
> > > +	spi_send(spi, 2, 0x00);
> > > +	init_nec_8048_wvga_lcd(spi);
> > > +
> > > +	return 0;
> > > +}
> > > +
> > > +static struct spi_driver nec_8048_spi_driver = {
> > > +	.probe           = nec_8048_spi_probe,
> > > +	.remove	= __devexit_p(nec_8048_spi_remove),
> > > +	.suspend         = nec_8048_spi_suspend,
> > > +	.resume          = nec_8048_spi_resume,
> > > +	.driver         = {
> > > +		.name   = "nec_8048_spi",
> > > +		.bus    = &spi_bus_type,
> > > +		.owner  = THIS_MODULE,
> > > +	},
> > > +};
> > > +
> > > +static int __init nec_8048_lcd_init(void)
> > > +{
> > > +	return spi_register_driver(&nec_8048_spi_driver);
> > > +}
> > > +
> > > +static void __exit nec_8048_lcd_exit(void)
> > > +{
> > > +	return spi_unregister_driver(&nec_8048_spi_driver);
> > > +}
> > > +
> > > +module_init(nec_8048_lcd_init);
> > > +module_exit(nec_8048_lcd_exit);
> > > +MODULE_LICENSE("GPL");
> > > +
> > > --
> > > 1.5.6.3
> > >
> > > Regards,
> > > Kishore Y

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

end of thread, other threads:[~2010-02-11  9:11 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <E0D41E29EB0DAC4E9F3FF173962E9E94026B7E8AB0@dbde02.ent.ti.com>
2010-02-11  8:21 ` [PATCH V2] OMAP: DSS: Add NEC NL8048HL11-01B display panel Y, Kishore
2010-02-11  8:44   ` Hiremath, Vaibhav
2010-02-11  8:52     ` Y, Kishore
2010-02-11  9:11       ` Hiremath, Vaibhav

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.