[PATCH v6 05/14] drm/exynos: dsi: add TE interrupt handler to support LCD I80 interface

2014-07-22 Thread YoungJun Cho
Hi Varka, On 07/22/2014 08:14 PM, Varka Bhadram wrote: > On 07/22/2014 04:40 PM, YoungJun Cho wrote: >> Hi Varka, >> >> This irq handler should be registered in attach() and unregistered in >> detach(). >> >> The devm_* APIs are released(freed) in remove(), right? >> >> Logically the panel could b

[PATCH v6 05/14] drm/exynos: dsi: add TE interrupt handler to support LCD I80 interface

2014-07-22 Thread YoungJun Cho
Hi Varka, This irq handler should be registered in attach() and unregistered in detach(). The devm_* APIs are released(freed) in remove(), right? Logically the panel could be attached and detached several times after dsi is probed and not removed. So I don't use devm_* APIs. Thank you. Best r

[PATCH v6 05/14] drm/exynos: dsi: add TE interrupt handler to support LCD I80 interface

2014-07-22 Thread YoungJun Cho
This is a temporary solution and should be made by more generic way. To support LCD I80 interface, the DSI host should register TE interrupt handler from the TE GPIO of attached panel. So the panel generates a tearing effect synchronization signal then the DSI host calls the CRTC device manager to

[PATCH v6 05/14] drm/exynos: dsi: add TE interrupt handler to support LCD I80 interface

2014-07-22 Thread YoungJun Cho
Hi Andrzej, On 07/22/2014 07:12 PM, Andrzej Hajda wrote: > On 07/22/2014 03:23 AM, Inki Dae wrote: >> On 2014? 07? 21? 23:01, Andrzej Hajda wrote: >>> On 07/17/2014 11:01 AM, YoungJun Cho wrote: To support LCD I80 interface, the DSI host should register TE interrupt handler from the TE G

[PATCH v6 05/14] drm/exynos: dsi: add TE interrupt handler to support LCD I80 interface

2014-07-22 Thread Varka Bhadram
On 07/22/2014 05:23 PM, YoungJun Cho wrote: > Hi Varka, > > On 07/22/2014 08:14 PM, Varka Bhadram wrote: >> On 07/22/2014 04:40 PM, YoungJun Cho wrote: >>> Hi Varka, >>> >>> This irq handler should be registered in attach() and unregistered in >>> detach(). >>> >>> The devm_* APIs are released(free

[PATCH v6 05/14] drm/exynos: dsi: add TE interrupt handler to support LCD I80 interface

2014-07-22 Thread Varka Bhadram
On 07/22/2014 04:40 PM, YoungJun Cho wrote: > Hi Varka, > > This irq handler should be registered in attach() and unregistered in > detach(). > > The devm_* APIs are released(freed) in remove(), right? > > Logically the panel could be attached and detached several times after > dsi is probed and

[PATCH v6 05/14] drm/exynos: dsi: add TE interrupt handler to support LCD I80 interface

2014-07-22 Thread Varka Bhadram
On 07/22/2014 04:19 PM, YoungJun Cho wrote: (...) > + ret = gpio_request_one(dsi->te_gpio, GPIOF_IN, "te_gpio"); devm_* APIs..? > + if (ret) { > + dev_err(dsi->dev, "gpio request failed with %d\n", ret); > + goto out; > + } > + > + /* > + * This TE G

[PATCH v6 05/14] drm/exynos: dsi: add TE interrupt handler to support LCD I80 interface

2014-07-22 Thread Andrzej Hajda
On 07/22/2014 03:23 AM, Inki Dae wrote: > On 2014? 07? 21? 23:01, Andrzej Hajda wrote: >> On 07/17/2014 11:01 AM, YoungJun Cho wrote: >>> To support LCD I80 interface, the DSI host should register >>> TE interrupt handler from the TE GPIO of attached panel. >>> So the panel generates a tearing effe

[PATCH v6 05/14] drm/exynos: dsi: add TE interrupt handler to support LCD I80 interface

2014-07-22 Thread YoungJun Cho
Hi, On 07/22/2014 10:23 AM, Inki Dae wrote: > On 2014? 07? 21? 23:01, Andrzej Hajda wrote: >> On 07/17/2014 11:01 AM, YoungJun Cho wrote: >>> To support LCD I80 interface, the DSI host should register >>> TE interrupt handler from the TE GPIO of attached panel. >>> So the panel generates a tearing

[PATCH v6 05/14] drm/exynos: dsi: add TE interrupt handler to support LCD I80 interface

2014-07-22 Thread Inki Dae
On 2014? 07? 21? 23:01, Andrzej Hajda wrote: > On 07/17/2014 11:01 AM, YoungJun Cho wrote: >> To support LCD I80 interface, the DSI host should register >> TE interrupt handler from the TE GPIO of attached panel. >> So the panel generates a tearing effect synchronization signal >> then the DSI host

[PATCH v6 05/14] drm/exynos: dsi: add TE interrupt handler to support LCD I80 interface

2014-07-21 Thread Andrzej Hajda
On 07/17/2014 11:01 AM, YoungJun Cho wrote: > To support LCD I80 interface, the DSI host should register > TE interrupt handler from the TE GPIO of attached panel. > So the panel generates a tearing effect synchronization signal > then the DSI host calls the CRTC device manager to trigger > to tran

[PATCH v6 05/14] drm/exynos: dsi: add TE interrupt handler to support LCD I80 interface

2014-07-17 Thread YoungJun Cho
To support LCD I80 interface, the DSI host should register TE interrupt handler from the TE GPIO of attached panel. So the panel generates a tearing effect synchronization signal then the DSI host calls the CRTC device manager to trigger to transfer video image. Signed-off-by: YoungJun Cho Acked-