RE: [PATCH V3] OMAP: DSS: Add NEC NL8048HL11-01B display panel

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Few minor comments below -

> -----Original Message-----
> From: linux-omap-owner@xxxxxxxxxxxxxxx [mailto:linux-omap-
> owner@xxxxxxxxxxxxxxx] On Behalf Of Y, Kishore
> Sent: Thursday, February 25, 2010 9:08 PM
> To: Tomi Valkeinen
> Cc: linux-omap@xxxxxxxxxxxxxxx
> Subject: [PATCH V3] OMAP: DSS: Add NEC NL8048HL11-01B display panel
> 
> From: Erik Gilling <konkers@xxxxxxxxxxx>
> 
> NEC WVGA LCD NL8048HL11-01B support has been added.
> 
> Signed-off-by: Mukund Mittal <mmittal@xxxxxx>
> Signed-off-by: Kishore Y <kishore.y@xxxxxx>
> Reviewed-by: Gadiyar, Anand <gadiyar@xxxxxx>
> ---
> Changes from version 2:
> 	incorporated cosmetic comments.
> 	included reviewer I missed earlier.
>  drivers/video/omap2/displays/Kconfig               |    6 +
>  drivers/video/omap2/displays/Makefile              |    1 +
>  .../omap2/displays/panel-nec-nl8048hl11-01b.c      |  282
> ++++++++++++++++++++
>  3 files changed, 289 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..5fb4dfe 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 support"
> +        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..03eb9c2 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..ce0560f
> --- /dev/null
> +++ b/drivers/video/omap2/displays/panel-nec-nl8048hl11-01b.c
> @@ -0,0 +1,282 @@
> +/*
> + * NEC panel support
> + *
> + * Copyright (C) 2010 Texas Instruments Inc.
> + *
> + * 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/spi/spi.h>
> +
> +#include <plat/display.h>
> +
> +#define LCD_XRES		800
> +#define LCD_YRES		480
> +/* NEC PIX Clock Ratings
> + * MIN: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;
[Hiremath, Vaibhav] I think we can avoid initialization to 0 here and merge cmd, data into one line.

Thanks,
Vaibhav

> +
> +	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);
> +
> +	return omap_dss_register_driver(&nec_8048_driver);
> +}
> +
> +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
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux