Skip to contentSkip to conversation

S5PV210: Add MIPI-DSI Driver.

PATCH New In-Ki Dae 2 participants 2 messages Jul 5, 2010 Framebuffer layer

lore
Share & tools
lore

In-Ki Dae

author
#0 lore raw

PATCH 2/2 S5PV210: Add MIPI-DSI Driver.

Hi, Guennadi,

You mean is that it uses include/video/mipi_display.h file instead of plat/dsim.h? I cann't find mipi_display.h in mainline kernel.

otherwise, it creates generic parts of my dsim.h header file as mipi_display.h and use that header file?

Please, give me more detailed.

Thank you.

------- Original Message -------
Sender : Guennadi Liakhovetski<[email protected]>
Date : 2010-07-05 04:33 (GMT+09:00)
Title : Re: [PATCH v1 2/2] S5PV210: Add MIPI-DSI Driver.

Quoted 36 lines · InKi Dae

this patch addes MIPI-DSI Driver.

to use this driver, some structures below should be added to machine specific file.

struct dsim_config
- define clock info, data lane count and video mode info for MIPI-DSI Controller.

struct dsim_lcd_config
- define interface mode, channel ID, Pixel format and so on.

struct s5p_platform_dsim
- define callbacks for initializing D-PHY, MIPI reset and trigger releated interfaces of s3c-fb.c file.

struct mipi_ddi_platform_data
- define following callbacks.
- a function for transferring and receiving command data to mipi based lcd panel.
- a function for getting framedone status of mipi-dsi controller.
- a function for clearing framedone interrupt of mipi-dsi controller.
- a function for checking i80 framedone status of display controller.(fimd)
- a function for triggering to display controller.(fimd)

Signed-off-by InKi Dae <inki.dae at samsung.com <mailto:[email protected]>>
Signed-off-by Kyungmin Park <[email protected]

<mailto:[email protected]>>
---

arch/arm/plat-samsung/include/plat/dsim.h new file +0 -0
new file mode 100644
index 0000000..49d8946
@@ -0,0 +1,493 @@

[snip]

Quoted 10 lines
arch/arm/plat-samsung/include/plat/dsim.h new file +10 -0
+enum dsim_fifo_state {
+	DSIM_RX_DATA_FULL = (1 << 25),
+	DSIM_RX_DATA_EMPTY = (1 << 24),
+	SFR_HEADER_FULL = (1 << 23),
+	SFR_HEADER_EMPTY = (1 << 22),
+	SFR_PAYLOAD_FULL = (1 << 21),
+	SFR_PAYLOAD_EMPTY = (1 << 20),
+	I80_HEADER_FULL = (1 << 19),
+	I80_HEADER_EMPTY = (1 << 18),
+	I80_PALOAD_FULL = (1 << 17),

arch/arm/plat-samsung/include/plat/dsim.h

Quoted 10 lines
arch/arm/plat-samsung/include/plat/dsim.h new file +10 -0
+	I80_PALOAD_EMPTY = (1 << 16),
+	SUB_DISP_HEADER_FULL = (1 << 15),
+	SUB_DISP_HEADER_EMPTY = (1 << 14),
+	SUB_DISP_PAYLOAD_FULL = (1 << 13),
+	SUB_DISP_PAYLOAD_EMPTY = (1 << 12),
+	MAIN_DISP_HEADER_FULL = (1 << 11),
+	MAIN_DISP_HEADER_EMPTY = (1 << 10),
+	MAIN_DISP_PAYLOAD_FULL = (1 << 9),
+	MAIN_DISP_PAYLOAD_EMPTY = (1 << 8),
+};

Please use include/video/mipi_display.h for these transaction types and, possibly, for other generic MIPI DSI defines.

Thanks
Guennadi
---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
http://www.open-technology.de/

#1 lore raw

PATCH 2/2 S5PV210: Add MIPI-DSI Driver.

Quoted 4 lines · In-Ki Dae

Hi, Guennadi,

You mean is that it uses include/video/mipi_display.h file instead of plat/dsim.h? I cann't find mipi_display.h in mainline kernel.

Sorry for not mentioning straight away - it is in linux-next:

http://git.kernel.org/?p=linux/kernel/git/next/linux-next.git;a=blob;f=include/video/mipi_display.h;h=ddcc8ca7316b51695bd827d9370d9027173e3576;hb=HEAD

Thanks
Guennadi

Quoted 77 lines

otherwise, it creates generic parts of my dsim.h header file as mipi_display.h and use that header file?

Please, give me more detailed.

Thank you.

------- Original Message -------
Sender : Guennadi Liakhovetski<[email protected]>
Date : 2010-07-05 04:33 (GMT+09:00)
Title : Re: [PATCH v1 2/2] S5PV210: Add MIPI-DSI Driver.

Quoted 36 lines · InKi Dae

this patch addes MIPI-DSI Driver.

to use this driver, some structures below should be added to machine specific file.

struct dsim_config
- define clock info, data lane count and video mode info for MIPI-DSI Controller.

struct dsim_lcd_config
- define interface mode, channel ID, Pixel format and so on.

struct s5p_platform_dsim
- define callbacks for initializing D-PHY, MIPI reset and trigger releated interfaces of s3c-fb.c file.

struct mipi_ddi_platform_data
- define following callbacks.
- a function for transferring and receiving command data to mipi based lcd panel.
- a function for getting framedone status of mipi-dsi controller.
- a function for clearing framedone interrupt of mipi-dsi controller.
- a function for checking i80 framedone status of display controller.(fimd)
- a function for triggering to display controller.(fimd)

Signed-off-by InKi Dae <inki.dae at samsung.com <mailto:[email protected]>>
Signed-off-by Kyungmin Park <[email protected]

<mailto:[email protected]>>
---

arch/arm/plat-samsung/include/plat/dsim.h new file +0 -0
new file mode 100644
index 0000000..49d8946
@@ -0,0 +1,493 @@

[snip]

Quoted 10 lines
arch/arm/plat-samsung/include/plat/dsim.h new file +10 -0
+enum dsim_fifo_state {
+	DSIM_RX_DATA_FULL = (1 << 25),
+	DSIM_RX_DATA_EMPTY = (1 << 24),
+	SFR_HEADER_FULL = (1 << 23),
+	SFR_HEADER_EMPTY = (1 << 22),
+	SFR_PAYLOAD_FULL = (1 << 21),
+	SFR_PAYLOAD_EMPTY = (1 << 20),
+	I80_HEADER_FULL = (1 << 19),
+	I80_HEADER_EMPTY = (1 << 18),
+	I80_PALOAD_FULL = (1 << 17),

arch/arm/plat-samsung/include/plat/dsim.h

Quoted 10 lines
arch/arm/plat-samsung/include/plat/dsim.h new file +10 -0
+	I80_PALOAD_EMPTY = (1 << 16),
+	SUB_DISP_HEADER_FULL = (1 << 15),
+	SUB_DISP_HEADER_EMPTY = (1 << 14),
+	SUB_DISP_PAYLOAD_FULL = (1 << 13),
+	SUB_DISP_PAYLOAD_EMPTY = (1 << 12),
+	MAIN_DISP_HEADER_FULL = (1 << 11),
+	MAIN_DISP_HEADER_EMPTY = (1 << 10),
+	MAIN_DISP_PAYLOAD_FULL = (1 << 9),
+	MAIN_DISP_PAYLOAD_EMPTY = (1 << 8),
+};

Please use include/video/mipi_display.h for these transaction types and, possibly, for other generic MIPI DSI defines.

Quoted 10 lines

Thanks
Guennadi
---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
http://www.open-technology.de/

---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
http://www.open-technology.de/