Re: [PATCH v5 3/4] drm/bridge: Introduce LT9611 DSI to HDMI bridge

2020-07-23 Thread Vinod Koul
On 23-07-20, 13:39, Sam Ravnborg wrote: > On Thu, Jul 23, 2020 at 04:11:51PM +0530, Vinod Koul wrote: > > Hi Sam, Laurent, > > > > On 22-07-20, 16:14, Laurent Pinchart wrote: > > > > > +static int lt9611_bridge_attach(struct drm_bridge *bridge, > > > > > + enum

Re: [PATCH v5 3/4] drm/bridge: Introduce LT9611 DSI to HDMI bridge

2020-07-23 Thread Sam Ravnborg
On Thu, Jul 23, 2020 at 04:11:51PM +0530, Vinod Koul wrote: > Hi Sam, Laurent, > > On 22-07-20, 16:14, Laurent Pinchart wrote: > > > > +static int lt9611_bridge_attach(struct drm_bridge *bridge, > > > > + enum drm_bridge_attach_flags flags) > > > > +{ > > > > +

Re: [PATCH v5 3/4] drm/bridge: Introduce LT9611 DSI to HDMI bridge

2020-07-23 Thread Vinod Koul
Hi Sam, Laurent, On 22-07-20, 16:14, Laurent Pinchart wrote: > > > +static int lt9611_bridge_attach(struct drm_bridge *bridge, > > > + enum drm_bridge_attach_flags flags) > > > +{ > > > + struct lt9611 *lt9611 = bridge_to_lt9611(bridge); > > > + int ret; > > > + > > > + if

Re: [PATCH v5 3/4] drm/bridge: Introduce LT9611 DSI to HDMI bridge

2020-07-22 Thread Laurent Pinchart
Hello, On Sun, Jul 19, 2020 at 07:18:06PM +0200, Sam Ravnborg wrote: > Hi Vinod. > > Three trivial points below. > The rest looks good. > > With these fixed you can add: > Reviewed-by: Sam Ravnborg > > Sam > > On Wed, Jul 08, 2020 at 04:05:58PM +0530, Vinod Koul wrote: > > Lontium

Re: [PATCH v5 3/4] drm/bridge: Introduce LT9611 DSI to HDMI bridge

2020-07-19 Thread Vinod Koul
Hi Sam, On 19-07-20, 19:18, Sam Ravnborg wrote: > Hi Vinod. > > Three trivial points below. > The rest looks good. > > With these fixed you can add: > Reviewed-by: Sam Ravnborg Thanks, I will send an update with nits fixed -- ~Vinod

Re: [PATCH v5 3/4] drm/bridge: Introduce LT9611 DSI to HDMI bridge

2020-07-19 Thread Sam Ravnborg
Hi Vinod. Three trivial points below. The rest looks good. With these fixed you can add: Reviewed-by: Sam Ravnborg Sam On Wed, Jul 08, 2020 at 04:05:58PM +0530, Vinod Koul wrote: > Lontium Lt9611 is a DSI to HDMI bridge which supports two DSI ports and > I2S port as an input and HDMI

[PATCH v5 3/4] drm/bridge: Introduce LT9611 DSI to HDMI bridge

2020-07-08 Thread Vinod Koul
Lontium Lt9611 is a DSI to HDMI bridge which supports two DSI ports and I2S port as an input and HDMI port as output Co-developed-by: Bjorn Andersson Signed-off-by: Bjorn Andersson Co-developed-by: Srinivas Kandagatla Signed-off-by: Srinivas Kandagatla Tested-by: John Stultz Signed-off-by: