On Thu, 19 May 2022 at 09:57, Lucas Stach <l.st...@pengutronix.de> wrote:
>
> Am Donnerstag, dem 19.05.2022 um 01:38 +0200 schrieb Marek Vasut:
> > Handle empty data-lanes = < >; property, which translates to
> > dsi_lanes = 0 as invalid.
> >
> > Fixes: ceb515ba29ba6 ("drm/bridge: ti-sn65dsi83: Add TI SN65DSI83 and
> > SN65DSI84 driver")
> > Signed-off-by: Marek Vasut <ma...@denx.de>
> > Cc: Jonas Karlman <jo...@kwiboo.se>
> > Cc: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
> > Cc: Lucas Stach <l.st...@pengutronix.de>
> > Cc: Marek Vasut <ma...@denx.de>
> > Cc: Maxime Ripard <max...@cerno.tech>
> > Cc: Neil Armstrong <narmstr...@baylibre.com>
> > Cc: Robert Foss <robert.f...@linaro.org>
> > Cc: Sam Ravnborg <s...@ravnborg.org>
>
> Reviewed-by: Lucas Stach <l.st...@pengutronix.de>
>
> > ---
> > drivers/gpu/drm/bridge/ti-sn65dsi83.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/gpu/drm/bridge/ti-sn65dsi83.c
> > b/drivers/gpu/drm/bridge/ti-sn65dsi83.c
> > index d64d4385188dd..dc65f424e7f3c 100644
> > --- a/drivers/gpu/drm/bridge/ti-sn65dsi83.c
> > +++ b/drivers/gpu/drm/bridge/ti-sn65dsi83.c
> > @@ -585,7 +585,7 @@ static int sn65dsi83_parse_dt(struct sn65dsi83 *ctx,
> > enum sn65dsi83_model model)
> > ctx->host_node = of_graph_get_remote_port_parent(endpoint);
> > of_node_put(endpoint);
> >
> > - if (ctx->dsi_lanes < 0 || ctx->dsi_lanes > 4) {
> > + if (ctx->dsi_lanes <= 0 || ctx->dsi_lanes > 4) {
> > ret = -EINVAL;
> > goto err_put_node;
> > }
>
>
Applied to drm-misc-next.
