From: Thierry Reding <thierry.reding@gmail.com>
To: Thierry Reding <thierry.reding@gmail.com>,
Jassi Brar <jassisinghbrar@gmail.com>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Jiri Slaby <jslaby@suse.com>,
Mikko Perttunen <mperttunen@nvidia.com>,
Jon Hunter <jonathanh@nvidia.com>, Timo Alho <talho@nvidia.com>,
Pekka Pessi <ppessi@nvidia.com>,
Mika Liljeberg <mliljeberg@nvidia.com>,
linux-tegra@vger.kernel.org, linux-serial@vger.kernel.org,
devicetree@vger.kernel.org
Subject: [PATCH v3 07/12] mailbox: tegra-hsp: Use device-managed registration API
Date: Wed, 28 Nov 2018 10:54:16 +0100 [thread overview]
Message-ID: <20181128095421.20573-8-thierry.reding@gmail.com> (raw)
In-Reply-To: <20181128095421.20573-1-thierry.reding@gmail.com>
From: Thierry Reding <treding@nvidia.com>
In order to get rid of a lot of cleanup boilerplate code, use the
device-managed registration API.
Signed-off-by: Thierry Reding <treding@nvidia.com>
---
drivers/mailbox/tegra-hsp.c | 30 ++++++------------------------
1 file changed, 6 insertions(+), 24 deletions(-)
diff --git a/drivers/mailbox/tegra-hsp.c b/drivers/mailbox/tegra-hsp.c
index 9628e52e2371..e443f6a2ec4b 100644
--- a/drivers/mailbox/tegra-hsp.c
+++ b/drivers/mailbox/tegra-hsp.c
@@ -721,7 +721,7 @@ static int tegra_hsp_probe(struct platform_device *pdev)
}
}
- err = mbox_controller_register(&hsp->mbox_db);
+ err = devm_mbox_controller_register(&pdev->dev, &hsp->mbox_db);
if (err < 0) {
dev_err(&pdev->dev, "failed to register doorbell mailbox: %d\n",
err);
@@ -745,15 +745,15 @@ static int tegra_hsp_probe(struct platform_device *pdev)
if (err < 0) {
dev_err(&pdev->dev, "failed to add mailboxes: %d\n",
err);
- goto unregister_mbox_db;
+ return err;
}
}
- err = mbox_controller_register(&hsp->mbox_sm);
+ err = devm_mbox_controller_register(&pdev->dev, &hsp->mbox_sm);
if (err < 0) {
dev_err(&pdev->dev, "failed to register shared mailbox: %d\n",
err);
- goto unregister_mbox_db;
+ return err;
}
platform_set_drvdata(pdev, hsp);
@@ -766,33 +766,16 @@ static int tegra_hsp_probe(struct platform_device *pdev)
dev_err(&pdev->dev,
"failed to request doorbell IRQ#%u: %d\n",
hsp->doorbell_irq, err);
- goto unregister_mbox_sm;
+ return err;
}
}
if (hsp->shared_irqs) {
err = tegra_hsp_request_shared_irq(hsp);
if (err < 0)
- goto unregister_mbox_sm;
+ return err;
}
- return 0;
-
-unregister_mbox_sm:
- mbox_controller_unregister(&hsp->mbox_sm);
-unregister_mbox_db:
- mbox_controller_unregister(&hsp->mbox_db);
-
- return err;
-}
-
-static int tegra_hsp_remove(struct platform_device *pdev)
-{
- struct tegra_hsp *hsp = platform_get_drvdata(pdev);
-
- mbox_controller_unregister(&hsp->mbox_sm);
- mbox_controller_unregister(&hsp->mbox_db);
-
return 0;
}
@@ -842,7 +825,6 @@ static struct platform_driver tegra_hsp_driver = {
.pm = &tegra_hsp_pm_ops,
},
.probe = tegra_hsp_probe,
- .remove = tegra_hsp_remove,
};
static int __init tegra_hsp_init(void)
--
2.19.1
next prev parent reply other threads:[~2018-11-28 20:55 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-11-28 9:54 [PATCH v3 00/12] serial: Add Tegra Combined UART driver Thierry Reding
2018-11-28 9:54 ` [PATCH v3 01/12] mailbox: Support blocking transfers in atomic context Thierry Reding
2018-11-28 9:54 ` [PATCH v3 02/12] mailbox: Allow multiple controllers per device Thierry Reding
2018-11-28 9:54 ` [PATCH v3 03/12] dt-bindings: tegra186-hsp: Add shared mailboxes Thierry Reding
2018-11-28 9:54 ` [PATCH v3 04/12] mailbox: tegra-hsp: Add support for " Thierry Reding
2018-11-28 9:54 ` [PATCH v3 05/12] mailbox: tegra-hsp: Add suspend/resume support Thierry Reding
2018-11-28 9:54 ` [PATCH v3 06/12] mailbox: tegra-hsp: use devm_kstrdup_const() Thierry Reding
2018-11-28 9:54 ` Thierry Reding [this message]
2018-11-28 9:54 ` [PATCH v3 08/12] dt-bindings: serial: Add bindings for nvidia,tegra194-tcu Thierry Reding
2018-11-28 9:54 ` [PATCH v3 09/12] serial: Add Tegra Combined UART driver Thierry Reding
2018-11-28 9:54 ` [PATCH v3 10/12] arm64: tegra: Add nodes for TCU on Tegra194 Thierry Reding
2018-11-28 9:54 ` [PATCH v3 11/12] arm64: tegra: Mark TCU as primary serial port on Tegra194 P2888 Thierry Reding
2018-11-28 9:54 ` [PATCH v3 12/12] arm64: defconfig: Enable Tegra TCU Thierry Reding
2018-12-05 16:15 ` [PATCH v3 00/12] serial: Add Tegra Combined UART driver Thierry Reding
2018-12-22 5:13 ` Jassi Brar
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20181128095421.20573-8-thierry.reding@gmail.com \
--to=thierry.reding@gmail.com \
--cc=devicetree@vger.kernel.org \
--cc=gregkh@linuxfoundation.org \
--cc=jassisinghbrar@gmail.com \
--cc=jonathanh@nvidia.com \
--cc=jslaby@suse.com \
--cc=linux-serial@vger.kernel.org \
--cc=linux-tegra@vger.kernel.org \
--cc=mliljeberg@nvidia.com \
--cc=mperttunen@nvidia.com \
--cc=ppessi@nvidia.com \
--cc=talho@nvidia.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.