[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20230829130004.tx45hldmp7balxr6@intel.intel>
Date: Tue, 29 Aug 2023 15:00:04 +0200
From: Andi Shyti <andi.shyti@...nel.org>
To: Yann Sionneau <yann@...nneau.net>
Cc: Codrin Ciubotariu <codrin.ciubotariu@...rochip.com>,
Nicolas Ferre <nicolas.ferre@...rochip.com>,
Alexandre Belloni <alexandre.belloni@...tlin.com>,
Claudiu Beznea <claudiu.beznea@...on.dev>,
linux-i2c@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH RESEND] i2c: at91: Use dev_err_probe() instead of
dev_err()
BTW...
> > Change
> > if (IS_ERR(x)) { dev_err(...); return PTR_ERR(x); }
> > into
> > return dev_err_probe()
> >
> > Also, return the correct error instead of hardcoding -ENODEV
> > This change has also the advantage of handling the -EPROBE_DEFER situation.
> >
> > Signed-off-by: Yann Sionneau <yann@...nneau.net>
>
> Reviewed-by: Andi Shyti <andi.shyti@...nel.org>
for the next time, please, if you make changes to the patch,
starting from the commit title, commit message, patch body, you
need to increment the versioning and note the change in the
changelog.
Use resend only if the patch is taken and sent as it is without
any change, anywhere.
This should have been [PATCH v2].
Andi
Powered by blists - more mailing lists