diff mbox series

[01/13] mailbox: altera: convert to devm_platform_ioremap_resource

Message ID 20191228183538.26189-1-tiny.windzz@gmail.com (mailing list archive)
State New, archived
Headers show
Series [01/13] mailbox: altera: convert to devm_platform_ioremap_resource | expand

Commit Message

Yangtao Li Dec. 28, 2019, 6:35 p.m. UTC
Use devm_platform_ioremap_resource() to simplify code.

Signed-off-by: Yangtao Li <tiny.windzz@gmail.com>
---
 drivers/mailbox/mailbox-altera.c | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

Comments

Tan, Ley Foon Dec. 30, 2019, 1:50 a.m. UTC | #1
> -----Original Message-----
> From: Yangtao Li <tiny.windzz@gmail.com>
> Sent: Sunday, December 29, 2019 2:35 AM
> To: jassisinghbrar@gmail.com; nsaenzjulienne@suse.de;
> f.fainelli@gmail.com; rjui@broadcom.com; sbranden@broadcom.com; bcm-
> kernel-feedback-list@broadcom.com; lftan@altera.com;
> matthias.bgg@gmail.com; agross@kernel.org; bjorn.andersson@linaro.org;
> mcoquelin.stm32@gmail.com; alexandre.torgue@st.com;
> thierry.reding@gmail.com; jonathanh@nvidia.com; linux-
> kernel@vger.kernel.org; linux-rpi-kernel@lists.infradead.org; linux-arm-
> kernel@lists.infradead.org; nios2-dev@lists.rocketboards.org; linux-
> mediatek@lists.infradead.org; linux-arm-msm@vger.kernel.org; linux-
> stm32@st-md-mailman.stormreply.com; linux-tegra@vger.kernel.org
> Cc: Yangtao Li <tiny.windzz@gmail.com>
> Subject: [PATCH 01/13] mailbox: altera: convert to
> devm_platform_ioremap_resource
> 
> Use devm_platform_ioremap_resource() to simplify code.
> 
> Signed-off-by: Yangtao Li <tiny.windzz@gmail.com>

Reviewed-by: Ley Foon Tan <ley.foon.tan@intel.com>

> ---
>  drivers/mailbox/mailbox-altera.c | 5 +----
>  1 file changed, 1 insertion(+), 4 deletions(-)
> 
> diff --git a/drivers/mailbox/mailbox-altera.c b/drivers/mailbox/mailbox-
> altera.c
> index 75282666fb06..afb320e9d69c 100644
> --- a/drivers/mailbox/mailbox-altera.c
> +++ b/drivers/mailbox/mailbox-altera.c
> @@ -285,7 +285,6 @@ static const struct mbox_chan_ops altera_mbox_ops
> = {  static int altera_mbox_probe(struct platform_device *pdev)  {
>         struct altera_mbox *mbox;
> -       struct resource *regs;
>         struct mbox_chan *chans;
>         int ret;
> 
> @@ -299,9 +298,7 @@ static int altera_mbox_probe(struct platform_device
> *pdev)
>         if (!chans)
>                 return -ENOMEM;
> 
> -       regs = platform_get_resource(pdev, IORESOURCE_MEM, 0);
> -
> -       mbox->mbox_base = devm_ioremap_resource(&pdev->dev, regs);
> +       mbox->mbox_base = devm_platform_ioremap_resource(pdev, 0);
>         if (IS_ERR(mbox->mbox_base))
>                 return PTR_ERR(mbox->mbox_base);
> 
> --
> 2.17.1
> 
> 
> ________________________________
> 
> Confidentiality Notice.
> This message may contain information that is confidential or otherwise
> protected from disclosure. If you are not the intended recipient, you are
> hereby notified that any use, disclosure, dissemination, distribution, or
> copying of this message, or any attachments, is strictly prohibited. If you
> have received this message in error, please advise the sender by reply e-mail,
> and delete the message and any attachments. Thank you.
diff mbox series

Patch

diff --git a/drivers/mailbox/mailbox-altera.c b/drivers/mailbox/mailbox-altera.c
index 75282666fb06..afb320e9d69c 100644
--- a/drivers/mailbox/mailbox-altera.c
+++ b/drivers/mailbox/mailbox-altera.c
@@ -285,7 +285,6 @@  static const struct mbox_chan_ops altera_mbox_ops = {
 static int altera_mbox_probe(struct platform_device *pdev)
 {
 	struct altera_mbox *mbox;
-	struct resource	*regs;
 	struct mbox_chan *chans;
 	int ret;
 
@@ -299,9 +298,7 @@  static int altera_mbox_probe(struct platform_device *pdev)
 	if (!chans)
 		return -ENOMEM;
 
-	regs = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-
-	mbox->mbox_base = devm_ioremap_resource(&pdev->dev, regs);
+	mbox->mbox_base = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(mbox->mbox_base))
 		return PTR_ERR(mbox->mbox_base);