Re: [PATCH v2 3/3] remoteporc: ingenic: Simplify with devm_platform_ioremap_resource_byname()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



You still have missed several instances.  I am dropping this set.

On Fri, Sep 06, 2024 at 07:34:05PM +0800, Zhang Zekun wrote:
> platform_get_resource_byname() and devm_ioremap_resource() can be
> replaced by devm_platform_ioremap_resource_byname(), which can
> simplify the code logic a bit, No functional change here.
> 
> Signed-off-by: Zhang Zekun <zhangzekun11@xxxxxxxxxx>
> ---
>  drivers/remoteproc/ingenic_rproc.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/drivers/remoteproc/ingenic_rproc.c b/drivers/remoteproc/ingenic_rproc.c
> index 9902cce28692..1b78d8ddeacf 100644
> --- a/drivers/remoteproc/ingenic_rproc.c
> +++ b/drivers/remoteproc/ingenic_rproc.c
> @@ -183,8 +183,7 @@ static int ingenic_rproc_probe(struct platform_device *pdev)
>  	vpu->dev = &pdev->dev;
>  	platform_set_drvdata(pdev, vpu);
>  
> -	mem = platform_get_resource_byname(pdev, IORESOURCE_MEM, "aux");
> -	vpu->aux_base = devm_ioremap_resource(dev, mem);
> +	vpu->aux_base = devm_platform_ioremap_resource_byname(pdev, "aux");
>  	if (IS_ERR(vpu->aux_base)) {
>  		dev_err(dev, "Failed to ioremap\n");
>  		return PTR_ERR(vpu->aux_base);
> -- 
> 2.17.1
> 




[Index of Archives]     [Linux Sound]     [ALSA Users]     [ALSA Devel]     [Linux Audio Users]     [Linux Media]     [Kernel]     [Photo Sharing]     [Gimp]     [Yosemite News]     [Linux Media]

  Powered by Linux