Re: [PATCH RESEND] i2c: at91: Use dev_err_probe() instead of dev_err()

From: Andi Shyti
Date: Tue Aug 29 2023 - 09:01:46 EST


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@xxxxxxxxxxxx>
>
> Reviewed-by: Andi Shyti <andi.shyti@xxxxxxxxxx>

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