* [PATCH] drm/bridge: ti-sn65dsi83: Handle dsi_lanes == 0 as invalid
@ 2022-05-18 23:38 Marek Vasut
2022-05-19 7:43 ` Andrzej Hajda
2022-05-19 7:57 ` Lucas Stach
0 siblings, 2 replies; 7+ messages in thread
From: Marek Vasut @ 2022-05-18 23:38 UTC (permalink / raw)
To: dri-devel
Cc: Marek Vasut, Laurent Pinchart, Jonas Karlman, Neil Armstrong,
robert.foss, Maxime Ripard, Sam Ravnborg
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 <marex@denx.de>
Cc: Jonas Karlman <jonas@kwiboo.se>
Cc: Laurent Pinchart <Laurent.pinchart@ideasonboard.com>
Cc: Lucas Stach <l.stach@pengutronix.de>
Cc: Marek Vasut <marex@denx.de>
Cc: Maxime Ripard <maxime@cerno.tech>
Cc: Neil Armstrong <narmstrong@baylibre.com>
Cc: Robert Foss <robert.foss@linaro.org>
Cc: Sam Ravnborg <sam@ravnborg.org>
---
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;
}
--
2.35.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH] drm/bridge: ti-sn65dsi83: Handle dsi_lanes == 0 as invalid
2022-05-18 23:38 [PATCH] drm/bridge: ti-sn65dsi83: Handle dsi_lanes == 0 as invalid Marek Vasut
@ 2022-05-19 7:43 ` Andrzej Hajda
2022-05-19 7:57 ` Lucas Stach
1 sibling, 0 replies; 7+ messages in thread
From: Andrzej Hajda @ 2022-05-19 7:43 UTC (permalink / raw)
To: Marek Vasut, dri-devel
Cc: Neil Armstrong, Jonas Karlman, robert.foss, Maxime Ripard,
Sam Ravnborg, Laurent Pinchart
On 19.05.2022 01:38, Marek Vasut wrote:
> 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 <marex@denx.de>
> Cc: Jonas Karlman <jonas@kwiboo.se>
> Cc: Laurent Pinchart <Laurent.pinchart@ideasonboard.com>
> Cc: Lucas Stach <l.stach@pengutronix.de>
> Cc: Marek Vasut <marex@denx.de>
> Cc: Maxime Ripard <maxime@cerno.tech>
> Cc: Neil Armstrong <narmstrong@baylibre.com>
> Cc: Robert Foss <robert.foss@linaro.org>
> Cc: Sam Ravnborg <sam@ravnborg.org>
I wonder if you could not make common helper to get number of lanes with
checks included, quite trivial, but maybe worth to standardize it.
Reviewed-by: Andrzej Hajda <andrzej.hajda@intel.com>
Regards
Andrzej
> ---
> 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;
> }
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] drm/bridge: ti-sn65dsi83: Handle dsi_lanes == 0 as invalid
2022-05-18 23:38 [PATCH] drm/bridge: ti-sn65dsi83: Handle dsi_lanes == 0 as invalid Marek Vasut
2022-05-19 7:43 ` Andrzej Hajda
@ 2022-05-19 7:57 ` Lucas Stach
2022-05-23 9:40 ` Robert Foss
1 sibling, 1 reply; 7+ messages in thread
From: Lucas Stach @ 2022-05-19 7:57 UTC (permalink / raw)
To: Marek Vasut, dri-devel
Cc: Neil Armstrong, Jonas Karlman, robert.foss, Maxime Ripard,
Sam Ravnborg, Laurent Pinchart
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 <marex@denx.de>
> Cc: Jonas Karlman <jonas@kwiboo.se>
> Cc: Laurent Pinchart <Laurent.pinchart@ideasonboard.com>
> Cc: Lucas Stach <l.stach@pengutronix.de>
> Cc: Marek Vasut <marex@denx.de>
> Cc: Maxime Ripard <maxime@cerno.tech>
> Cc: Neil Armstrong <narmstrong@baylibre.com>
> Cc: Robert Foss <robert.foss@linaro.org>
> Cc: Sam Ravnborg <sam@ravnborg.org>
Reviewed-by: Lucas Stach <l.stach@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;
> }
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] drm/bridge: ti-sn65dsi83: Handle dsi_lanes == 0 as invalid
2022-05-19 7:57 ` Lucas Stach
@ 2022-05-23 9:40 ` Robert Foss
2022-05-23 9:58 ` Marek Vasut
0 siblings, 1 reply; 7+ messages in thread
From: Robert Foss @ 2022-05-23 9:40 UTC (permalink / raw)
To: Lucas Stach
Cc: Marek Vasut, Neil Armstrong, Jonas Karlman, dri-devel,
Maxime Ripard, Sam Ravnborg, Laurent Pinchart
On Thu, 19 May 2022 at 09:57, Lucas Stach <l.stach@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 <marex@denx.de>
> > Cc: Jonas Karlman <jonas@kwiboo.se>
> > Cc: Laurent Pinchart <Laurent.pinchart@ideasonboard.com>
> > Cc: Lucas Stach <l.stach@pengutronix.de>
> > Cc: Marek Vasut <marex@denx.de>
> > Cc: Maxime Ripard <maxime@cerno.tech>
> > Cc: Neil Armstrong <narmstrong@baylibre.com>
> > Cc: Robert Foss <robert.foss@linaro.org>
> > Cc: Sam Ravnborg <sam@ravnborg.org>
>
> Reviewed-by: Lucas Stach <l.stach@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.
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] drm/bridge: ti-sn65dsi83: Handle dsi_lanes == 0 as invalid
2022-05-23 9:40 ` Robert Foss
@ 2022-05-23 9:58 ` Marek Vasut
2022-05-23 10:23 ` Robert Foss
0 siblings, 1 reply; 7+ messages in thread
From: Marek Vasut @ 2022-05-23 9:58 UTC (permalink / raw)
To: Robert Foss, Lucas Stach
Cc: Neil Armstrong, Jonas Karlman, dri-devel, Maxime Ripard,
Sam Ravnborg, Laurent Pinchart
On 5/23/22 11:40, Robert Foss wrote:
> On Thu, 19 May 2022 at 09:57, Lucas Stach <l.stach@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.
[...]
>>> 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.
This was already applied to drm-misc-next-fixes , since this is a fix
for something that was already in release . The conflict will resolve
itself easily though ?
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] drm/bridge: ti-sn65dsi83: Handle dsi_lanes == 0 as invalid
2022-05-23 9:58 ` Marek Vasut
@ 2022-05-23 10:23 ` Robert Foss
2022-05-23 10:54 ` Marek Vasut
0 siblings, 1 reply; 7+ messages in thread
From: Robert Foss @ 2022-05-23 10:23 UTC (permalink / raw)
To: Marek Vasut
Cc: Neil Armstrong, Jonas Karlman, dri-devel, Maxime Ripard,
Sam Ravnborg, Laurent Pinchart
On Mon, 23 May 2022 at 11:58, Marek Vasut <marex@denx.de> wrote:
>
> On 5/23/22 11:40, Robert Foss wrote:
> > On Thu, 19 May 2022 at 09:57, Lucas Stach <l.stach@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.
>
> [...]
>
> >>> 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.
>
> This was already applied to drm-misc-next-fixes , since this is a fix
> for something that was already in release . The conflict will resolve
> itself easily though ?
Ah, I didn't realize. I'm not sure if it will resolve itself or not.
But I think either it is reverted in drm-misc-next now, or we wait for
an issue to crop up. I think either way is acceptable, but waiting
maybe causes less churn. So unless someone has another preference,
let's wait and see if a conflict arises.
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] drm/bridge: ti-sn65dsi83: Handle dsi_lanes == 0 as invalid
2022-05-23 10:23 ` Robert Foss
@ 2022-05-23 10:54 ` Marek Vasut
0 siblings, 0 replies; 7+ messages in thread
From: Marek Vasut @ 2022-05-23 10:54 UTC (permalink / raw)
To: Robert Foss
Cc: Neil Armstrong, Jonas Karlman, dri-devel, Maxime Ripard,
Sam Ravnborg, Laurent Pinchart
On 5/23/22 12:23, Robert Foss wrote:
> On Mon, 23 May 2022 at 11:58, Marek Vasut <marex@denx.de> wrote:
>>
>> On 5/23/22 11:40, Robert Foss wrote:
>>> On Thu, 19 May 2022 at 09:57, Lucas Stach <l.stach@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.
>>
>> [...]
>>
>>>>> 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.
>>
>> This was already applied to drm-misc-next-fixes , since this is a fix
>> for something that was already in release . The conflict will resolve
>> itself easily though ?
>
> Ah, I didn't realize. I'm not sure if it will resolve itself or not.
> But I think either it is reverted in drm-misc-next now, or we wait for
> an issue to crop up. I think either way is acceptable, but waiting
> maybe causes less churn. So unless someone has another preference,
> let's wait and see if a conflict arises.
I agree, we wait.
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2022-05-23 10:54 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-05-18 23:38 [PATCH] drm/bridge: ti-sn65dsi83: Handle dsi_lanes == 0 as invalid Marek Vasut
2022-05-19 7:43 ` Andrzej Hajda
2022-05-19 7:57 ` Lucas Stach
2022-05-23 9:40 ` Robert Foss
2022-05-23 9:58 ` Marek Vasut
2022-05-23 10:23 ` Robert Foss
2022-05-23 10:54 ` Marek Vasut
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.