Am Donnerstag, dem 19.05.2022 um 01:36 +0200 schrieb Marek Vasut: > Handle empty data-lanes = < >; property, which translates to > dsi_lanes = 0 as invalid. > I'm having a hard time imagining a situation where on would add a empty data-lanes property to the DT. But then 0 DSI lanes is clearly a broken configuration, so:
Reviewed-by: Lucas Stach <[email protected]> > Fixes: bbfd3190b6562 ("drm/bridge: tc358767: Add DSI-to-DPI mode support") > Signed-off-by: Marek Vasut <[email protected]> > Cc: Jonas Karlman <[email protected]> > Cc: Laurent Pinchart <[email protected]> > Cc: Lucas Stach <[email protected]> > Cc: Marek Vasut <[email protected]> > Cc: Maxime Ripard <[email protected]> > Cc: Neil Armstrong <[email protected]> > Cc: Robert Foss <[email protected]> > Cc: Sam Ravnborg <[email protected]> > --- > drivers/gpu/drm/bridge/tc358767.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/bridge/tc358767.c > b/drivers/gpu/drm/bridge/tc358767.c > index 7366968dd7b21..84e6b5aa8dd1f 100644 > --- a/drivers/gpu/drm/bridge/tc358767.c > +++ b/drivers/gpu/drm/bridge/tc358767.c > @@ -1902,7 +1902,7 @@ static int tc_mipi_dsi_host_attach(struct tc_data *tc) > of_node_put(host_node); > of_node_put(endpoint); > > - if (dsi_lanes < 0 || dsi_lanes > 4) > + if (dsi_lanes <= 0 || dsi_lanes > 4) > return -EINVAL; > > if (!host)
