Hi Marek,
On Sun, Jun 12, 2022 at 12:29:47PM +0200, Marek Vasut wrote:
> Convert driver to use this new helper to standardize
> OF "data-lanes" parsing.
>
> Reviewed-by: Andrzej Hajda <andrzej.ha...@intel.com>
> Signed-off-by: Marek Vasut <ma...@denx.de>
> Cc: Andrzej Hajda <andrzej.ha...@intel.com>
> Cc: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
> Cc: Lucas Stach <l.st...@pengutronix.de>
> Cc: Maxime Ripard <max...@cerno.tech>
> Cc: Robert Foss <robert.f...@linaro.org>
> Cc: Sam Ravnborg <s...@ravnborg.org>
> To: dri-devel@lists.freedesktop.org
Looks fine.
I wondered why the _ep variant could not be used, but endpoint is
required later and there is no need to look it up twice.
Acked-by: Sam Ravnborg <s...@ravnborg.org>
Sam
> ---
> V2: - Rename drm_of_get_data_lanes{,_ep} to drm_of_get_data_lanes_count{,_ep}
> - Add RB from Andrzej
> V3: - Rebase on latest next
> ---
> 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 8bf99b32776e2..b27c0d7c451ad 100644
> --- a/drivers/gpu/drm/bridge/ti-sn65dsi83.c
> +++ b/drivers/gpu/drm/bridge/ti-sn65dsi83.c
> @@ -620,7 +620,7 @@ static int sn65dsi83_host_attach(struct sn65dsi83 *ctx)
> int dsi_lanes, ret;
>
> endpoint = of_graph_get_endpoint_by_regs(dev->of_node, 0, -1);
> - dsi_lanes = of_property_count_u32_elems(endpoint, "data-lanes");
> + dsi_lanes = drm_of_get_data_lanes_count(endpoint, 1, 4);
> host_node = of_graph_get_remote_port_parent(endpoint);
> host = of_find_mipi_dsi_host_by_node(host_node);
> of_node_put(host_node);
> --
> 2.35.1
