* Re: [PATCH] EDAC/altera: Remove redundant error logging
[not found] <Y6RIfD/Tv66Oj+kd@qemulion>
@ 2023-01-07 20:06 ` Deepak R Varma
0 siblings, 0 replies; only message in thread
From: Deepak R Varma @ 2023-01-07 20:06 UTC (permalink / raw)
To: Dinh Nguyen, Borislav Petkov, Tony Luck, James Morse,
Mauro Carvalho Chehab, Robert Richter, linux-edac, linux-kernel
Cc: Saurabh Singh Sengar, Praveen Kumar, Deepak R Varma
On Thu, Dec 22, 2022 at 05:37:24PM +0530, Deepak R Varma wrote:
> A call to platform_get_irq() already prints an error on failure within
> its own implementation. So printing another error based on its return
> value in the caller is redundant and should be removed. The clean up
> also makes if condition block braces unnecessary. Remove that as well.
>
> Issue identified using platform_get_irq.cocci coccinelle semantic patch.
>
> Signed-off-by: Deepak R Varma <drv@mailo.com>
Hello,
Requesting a review and feedback on this patch proposal?
Thank you,
./drv
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2023-01-07 20:07 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <Y6RIfD/Tv66Oj+kd@qemulion>
2023-01-07 20:06 ` [PATCH] EDAC/altera: Remove redundant error logging Deepak R Varma
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).