As per [1] and [2], ADV7535/7533 supports only 2-, 3-, or 4-lane. Drop
unsupported 1-lane.

[1] 
https://www.analog.com/media/en/technical-documentation/data-sheets/ADV7535.pdf
[2] 
https://www.analog.com/media/en/technical-documentation/data-sheets/ADV7533.pdf

Fixes: 1e4d58cd7f88 ("drm/bridge: adv7533: Create a MIPI DSI device")
Reported-by: Hien Huynh <hien.huynh...@renesas.com>
Cc: sta...@vger.kernel.org
Reviewed-by: Laurent Pinchart <laurent.pinchart+rene...@ideasonboard.com>
Reviewed-by: Adam Ford <aford...@gmail.com>
Signed-off-by: Biju Das <biju.das...@bp.renesas.com>
---
Changes in v7:
 - No change.
Changes in v6:
 - Added Rb tag from Adam.
Changes in v5:
 - No change.
Changes in v4:
 - Added link to ADV7533 data sheet.
 - Collected tags
Changes in v3:
 - Updated commit header and description
 - Updated fixes tag
 - Dropped single lane support
Changes in v2:
 - Added the tag "Cc: sta...@vger.kernel.org" in the sign-off area.
 - Dropped Archit Taneja invalid Mail address
---
 drivers/gpu/drm/bridge/adv7511/adv7533.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/bridge/adv7511/adv7533.c 
b/drivers/gpu/drm/bridge/adv7511/adv7533.c
index 5f195e91b3e6..122ad91e8a32 100644
--- a/drivers/gpu/drm/bridge/adv7511/adv7533.c
+++ b/drivers/gpu/drm/bridge/adv7511/adv7533.c
@@ -172,7 +172,7 @@ int adv7533_parse_dt(struct device_node *np, struct adv7511 
*adv)
 
        of_property_read_u32(np, "adi,dsi-lanes", &num_lanes);
 
-       if (num_lanes < 1 || num_lanes > 4)
+       if (num_lanes < 2 || num_lanes > 4)
                return -EINVAL;
 
        adv->num_dsi_lanes = num_lanes;
-- 
2.43.0

Reply via email to