Re: [PATCH 3/5] drm/bridge: Add PTN3460 bridge driver

2013-10-10 Thread Mark Brown
On Thu, Oct 10, 2013 at 01:18:05PM +0900, Inki Dae wrote: I still think the pin could be replaced with a regulator. But lvds-bridge node has powerdown-gpio property - it say this board will use gpio pin - specific to board. So it seems no problem. No, don't model things that aren't

RE: [PATCH 3/5] drm/bridge: Add PTN3460 bridge driver

2013-10-10 Thread Inki Dae
mailing list'; linux-arm- ker...@lists.infradead.org Subject: Re: [PATCH 3/5] drm/bridge: Add PTN3460 bridge driver On Thu, Oct 10, 2013 at 01:18:05PM +0900, Inki Dae wrote: I still think the pin could be replaced with a regulator. But lvds-bridge node has powerdown-gpio property

Re: [PATCH 3/5] drm/bridge: Add PTN3460 bridge driver

2013-10-10 Thread Mark Brown
On Thu, Oct 10, 2013 at 08:40:38PM +0900, Inki Dae wrote: That is what I mentioned. Some boards _could control_ the actual regulator for lvds-bridge, and that would be depended on how HW engineer designs the board. For the driver this should be totally transparent - it should just control

Re: [PATCH 3/5] drm/bridge: Add PTN3460 bridge driver

2013-10-09 Thread Mark Brown
On Fri, Oct 04, 2013 at 11:05:48AM +0900, Inki Dae wrote: 2013/10/4 Olof Johansson o...@lixom.net: If PD_N is LOW, then the device is in Deep power-down completely, even if supply rail is ON; for the device to be able to operate, the PD_N pin must be HIGH. I still think the pin could be

RE: [PATCH 3/5] drm/bridge: Add PTN3460 bridge driver

2013-10-09 Thread Inki Dae
mailing list; linux-arm- ker...@lists.infradead.org Subject: Re: [PATCH 3/5] drm/bridge: Add PTN3460 bridge driver On Fri, Oct 04, 2013 at 11:05:48AM +0900, Inki Dae wrote: 2013/10/4 Olof Johansson o...@lixom.net: If PD_N is LOW, then the device is in Deep power-down completely, even

Re: [PATCH 3/5] drm/bridge: Add PTN3460 bridge driver

2013-10-03 Thread Inki Dae
Hi, thank you for your contribution and the below is my short comments, 2013/10/2 Sean Paul seanp...@chromium.org: This patch adds a drm_bridge driver for the PTN3460 DisplayPort to LVDS bridge chip. Signed-off-by: Sean Paul seanp...@chromium.org ---

Re: [PATCH 3/5] drm/bridge: Add PTN3460 bridge driver

2013-10-03 Thread Sean Paul
On Thu, Oct 3, 2013 at 9:55 AM, Inki Dae inki@samsung.com wrote: Hi, thank you for your contribution and the below is my short comments, 2013/10/2 Sean Paul seanp...@chromium.org: This patch adds a drm_bridge driver for the PTN3460 DisplayPort to LVDS bridge chip. Signed-off-by: Sean

Re: [PATCH 3/5] drm/bridge: Add PTN3460 bridge driver

2013-10-03 Thread Inki Dae
2013/10/3 Sean Paul seanp...@chromium.org: On Thu, Oct 3, 2013 at 9:55 AM, Inki Dae inki@samsung.com wrote: Hi, thank you for your contribution and the below is my short comments, 2013/10/2 Sean Paul seanp...@chromium.org: This patch adds a drm_bridge driver for the PTN3460 DisplayPort to

Re: [PATCH 3/5] drm/bridge: Add PTN3460 bridge driver

2013-10-03 Thread Olof Johansson
On Thu, Oct 3, 2013 at 10:39 AM, Inki Dae inki@samsung.com wrote: 2013/10/3 Sean Paul seanp...@chromium.org: On Thu, Oct 3, 2013 at 9:55 AM, Inki Dae inki@samsung.com wrote: Can a regulator be used instead of gpio in other board case? No, not to my knowledge. Hm.. plz check it out

Re: [PATCH 3/5] drm/bridge: Add PTN3460 bridge driver

2013-10-03 Thread Sean Paul
On Thu, Oct 3, 2013 at 1:39 PM, Inki Dae inki@samsung.com wrote: 2013/10/3 Sean Paul seanp...@chromium.org: On Thu, Oct 3, 2013 at 9:55 AM, Inki Dae inki@samsung.com wrote: Hi, thank you for your contribution and the below is my short comments, 2013/10/2 Sean Paul

Re: [PATCH 3/5] drm/bridge: Add PTN3460 bridge driver

2013-10-03 Thread Inki Dae
2013/10/4 Sean Paul seanp...@chromium.org: On Thu, Oct 3, 2013 at 1:39 PM, Inki Dae inki@samsung.com wrote: 2013/10/3 Sean Paul seanp...@chromium.org: On Thu, Oct 3, 2013 at 9:55 AM, Inki Dae inki@samsung.com wrote: Hi, thank you for your contribution and the below is my short

Re: [PATCH 3/5] drm/bridge: Add PTN3460 bridge driver

2013-10-03 Thread Sean Paul
On Thu, Oct 3, 2013 at 2:23 PM, Inki Dae inki@samsung.com wrote: 2013/10/4 Sean Paul seanp...@chromium.org: On Thu, Oct 3, 2013 at 1:39 PM, Inki Dae inki@samsung.com wrote: 2013/10/3 Sean Paul seanp...@chromium.org: On Thu, Oct 3, 2013 at 9:55 AM, Inki Dae inki@samsung.com wrote:

Re: [PATCH 3/5] drm/bridge: Add PTN3460 bridge driver

2013-10-03 Thread Inki Dae
2013/10/4 Sean Paul seanp...@chromium.org: On Thu, Oct 3, 2013 at 2:23 PM, Inki Dae inki@samsung.com wrote: 2013/10/4 Sean Paul seanp...@chromium.org: On Thu, Oct 3, 2013 at 1:39 PM, Inki Dae inki@samsung.com wrote: 2013/10/3 Sean Paul seanp...@chromium.org: On Thu, Oct 3, 2013 at

Re: [PATCH 3/5] drm/bridge: Add PTN3460 bridge driver

2013-10-03 Thread Inki Dae
2013/10/4 Olof Johansson o...@lixom.net: On Thu, Oct 3, 2013 at 10:39 AM, Inki Dae inki@samsung.com wrote: 2013/10/3 Sean Paul seanp...@chromium.org: On Thu, Oct 3, 2013 at 9:55 AM, Inki Dae inki@samsung.com wrote: Can a regulator be used instead of gpio in other board case? No, not

Re: [PATCH 3/5] drm/bridge: Add PTN3460 bridge driver

2013-10-02 Thread Olof Johansson
Hi, On Tue, Oct 1, 2013 at 4:40 PM, Sean Paul seanp...@chromium.org wrote: This patch adds a drm_bridge driver for the PTN3460 DisplayPort to LVDS bridge chip. Signed-off-by: Sean Paul seanp...@chromium.org --- [...] +Example: + ptn3460-bridge@20 { Nit: Name is usually generic