Subject: Re: [PATCH -next] net: bcmasp: Clean up redundant dev_err_probe()

From: Simon Horman <[email-masked]>
Date: Thu, 27 Jul 2023 15:32:32 +0200

On Thu, Jul 27, 2023 at 07:55:51PM +0800, Chen Jiahao wrote:
> Refering to platform_get_irq()'s definition, the return value has

nit: Refering -> Referring

> already been checked, error message also been printed via
> dev_err_probe() if ret < 0. Calling dev_err_probe() one more time
> outside platform_get_irq() is obviously redundant.
> 
> Removing dev_err_probe() outside platform_get_irq() to clean up
> above problem.
> 
> Signed-off-by: Chen Jiahao <[email-masked]>

Reviewed-by: Simon Horman <[email-masked]>