| Message ID | 20240911121751.1900-1-tangbin@cmss.chinamobile.com |
|---|---|
| State | Accepted |
| Commit | f74032baac84a93f5d3027ba00e000630f13d620 |
| Headers | show |
| Series | crypto: amlogic - Remove redundant assignment and error messages | expand |
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 --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);
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(-)