Re: [PATCH -resend] drm/msm: use dev_fwnode()

[Jiri Slaby <jirislaby@kernel.org>]

Hmm, I got: "Dmitry Baryshkov no longer works for Linaro.". "To:" field 
fixed using @kernel.org address (was in Cc).

On 23. 07. 25, 7:55, Jiri Slaby (SUSE) wrote:
> irq_domain_create_simple() takes fwnode as the first argument. It can be
> extracted from the struct device using dev_fwnode() helper instead of
> using of_node with of_fwnode_handle().
> 
> So use the dev_fwnode() helper.
> 
> Signed-off-by: Jiri Slaby (SUSE) <jirislaby@kernel.org>
> Cc: Rob Clark <robdclark@gmail.com>
> Cc: Abhinav Kumar <quic_abhinavk@quicinc.com>
> Cc: Dmitry Baryshkov <lumag@kernel.org>
> Cc: Sean Paul <sean@poorly.run>
> Cc: Marijn Suijten <marijn.suijten@somainline.org>
> Cc: David Airlie <airlied@gmail.com>
> Cc: Simona Vetter <simona@ffwll.ch>
> 
> ---
> 
> Cc: linux-arm-msm@vger.kernel.org
> Cc: dri-devel@lists.freedesktop.org
> Cc: freedreno@lists.freedesktop.org
> ---
>   drivers/gpu/drm/msm/msm_mdss.c | 3 +--
>   1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/drivers/gpu/drm/msm/msm_mdss.c b/drivers/gpu/drm/msm/msm_mdss.c
> index 1f5fe7811e01..ec7d83831b83 100644
> --- a/drivers/gpu/drm/msm/msm_mdss.c
> +++ b/drivers/gpu/drm/msm/msm_mdss.c
> @@ -154,8 +154,7 @@ static int _msm_mdss_irq_domain_add(struct msm_mdss *msm_mdss)
>   
>   	dev = msm_mdss->dev;
>   
> -	domain = irq_domain_create_linear(of_fwnode_handle(dev->of_node), 32,
> -			&msm_mdss_irqdomain_ops, msm_mdss);
> +	domain = irq_domain_create_linear(dev_fwnode(dev), 32, &msm_mdss_irqdomain_ops, msm_mdss);
>   	if (!domain) {
>   		dev_err(dev, "failed to add irq_domain\n");
>   		return -EINVAL;


-- 
js
suse labs