diff mbox series

crypto: amlogic - Remove redundant assignment and error messages

Message ID 20240911121751.1900-1-tangbin@cmss.chinamobile.com
State New
Headers show
Series crypto: amlogic - Remove redundant assignment and error messages | expand

Commit Message

Tang Bin Sept. 11, 2024, 12:17 p.m. UTC
In the function meson_crypto_probe, devm_platform_ioremap_resource()
have already contains error message, so remove the
redundant assignment and error messages.

Signed-off-by: Tang Bin <tangbin@cmss.chinamobile.com>
---
 drivers/crypto/amlogic/amlogic-gxl-core.c | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

Comments

Herbert Xu Oct. 5, 2024, 5:29 a.m. UTC | #1
Tang Bin <tangbin@cmss.chinamobile.com> wrote:
> In the function meson_crypto_probe, devm_platform_ioremap_resource()
> have already contains error message, so remove the
> redundant assignment and error messages.
> 
> Signed-off-by: Tang Bin <tangbin@cmss.chinamobile.com>
> ---
> drivers/crypto/amlogic/amlogic-gxl-core.c | 8 +++-----
> 1 file changed, 3 insertions(+), 5 deletions(-)

Patch applied.  Thanks.
diff mbox series

Patch

diff --git a/drivers/crypto/amlogic/amlogic-gxl-core.c b/drivers/crypto/amlogic/amlogic-gxl-core.c
index f54ab0d0b..6fe248620 100644
--- a/drivers/crypto/amlogic/amlogic-gxl-core.c
+++ b/drivers/crypto/amlogic/amlogic-gxl-core.c
@@ -240,11 +240,9 @@  static int meson_crypto_probe(struct platform_device *pdev)
 	platform_set_drvdata(pdev, mc);
 
 	mc->base = devm_platform_ioremap_resource(pdev, 0);
-	if (IS_ERR(mc->base)) {
-		err = PTR_ERR(mc->base);
-		dev_err(&pdev->dev, "Cannot request MMIO err=%d\n", err);
-		return err;
-	}
+	if (IS_ERR(mc->base))
+		return PTR_ERR(mc->base);
+
 	mc->busclk = devm_clk_get(&pdev->dev, "blkmv");
 	if (IS_ERR(mc->busclk)) {
 		err = PTR_ERR(mc->busclk);