Re: [PATCH] drm: bridge: adv7511: Add missing bridge type

2020-07-23 Thread Vinod Koul
On 24-07-20, 04:30, Laurent Pinchart wrote:
> Hi Vinod,
> 
> Thank you for the patch.
> 
> On Thu, Jul 23, 2020 at 04:15:23PM +0530, Vinod Koul wrote:
> > Add bridge type as DRM_MODE_CONNECTOR_HDMIA
> > 
> > Signed-off-by: Vinod Koul 
> 
> This has already been submitted: 
> https://lore.kernel.org/dri-devel/20200720124228.12552-1-laurentiu.pa...@oss.nxp.com/

OKay lets drop this one then

-- 
~Vinod


Re: [PATCH] drm: bridge: adv7511: Add missing bridge type

2020-07-23 Thread Laurent Pinchart
Hi Vinod,

Thank you for the patch.

On Thu, Jul 23, 2020 at 04:15:23PM +0530, Vinod Koul wrote:
> Add bridge type as DRM_MODE_CONNECTOR_HDMIA
> 
> Signed-off-by: Vinod Koul 

This has already been submitted: 
https://lore.kernel.org/dri-devel/20200720124228.12552-1-laurentiu.pa...@oss.nxp.com/

> ---
> 
> I found this when testing Dragon-board 410c which uses this bridge
> [6.671913] msm 1a0.mdss: [drm:msm_dsi_manager_ext_bridge_init [msm]] 
> *ERROR* drm_bridge_connector_init failed: -22
> [6.678879] msm 1a0.mdss: [drm:msm_dsi_modeset_init [msm]] *ERROR* 
> failed to create dsi connector: -19
> 
>  drivers/gpu/drm/bridge/adv7511/adv7511_drv.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/gpu/drm/bridge/adv7511/adv7511_drv.c 
> b/drivers/gpu/drm/bridge/adv7511/adv7511_drv.c
> index f45cdca9cce5..a0d392c338da 100644
> --- a/drivers/gpu/drm/bridge/adv7511/adv7511_drv.c
> +++ b/drivers/gpu/drm/bridge/adv7511/adv7511_drv.c
> @@ -1283,6 +1283,7 @@ static int adv7511_probe(struct i2c_client *i2c, const 
> struct i2c_device_id *id)
>   adv7511->bridge.ops = DRM_BRIDGE_OP_DETECT | DRM_BRIDGE_OP_EDID
>   | DRM_BRIDGE_OP_HPD;
>   adv7511->bridge.of_node = dev->of_node;
> + adv7511->bridge.type = DRM_MODE_CONNECTOR_HDMIA;
>  
>   drm_bridge_add(>bridge);
>  

-- 
Regards,

Laurent Pinchart


[PATCH] drm: bridge: adv7511: Add missing bridge type

2020-07-23 Thread Vinod Koul
Add bridge type as DRM_MODE_CONNECTOR_HDMIA

Signed-off-by: Vinod Koul 
---

I found this when testing Dragon-board 410c which uses this bridge
[6.671913] msm 1a0.mdss: [drm:msm_dsi_manager_ext_bridge_init [msm]] 
*ERROR* drm_bridge_connector_init failed: -22
[6.678879] msm 1a0.mdss: [drm:msm_dsi_modeset_init [msm]] *ERROR* 
failed to create dsi connector: -19

 drivers/gpu/drm/bridge/adv7511/adv7511_drv.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/gpu/drm/bridge/adv7511/adv7511_drv.c 
b/drivers/gpu/drm/bridge/adv7511/adv7511_drv.c
index f45cdca9cce5..a0d392c338da 100644
--- a/drivers/gpu/drm/bridge/adv7511/adv7511_drv.c
+++ b/drivers/gpu/drm/bridge/adv7511/adv7511_drv.c
@@ -1283,6 +1283,7 @@ static int adv7511_probe(struct i2c_client *i2c, const 
struct i2c_device_id *id)
adv7511->bridge.ops = DRM_BRIDGE_OP_DETECT | DRM_BRIDGE_OP_EDID
| DRM_BRIDGE_OP_HPD;
adv7511->bridge.of_node = dev->of_node;
+   adv7511->bridge.type = DRM_MODE_CONNECTOR_HDMIA;
 
drm_bridge_add(>bridge);
 
-- 
2.26.2