Re: [PATCH v3 1/4] dev_printk: add new dev_err_probe() helpers
From: Nuno Sá
Date: Wed Jun 26 2024 - 10:57:22 EST
On Mon, 2024-06-17 at 20:41 +0100, Jonathan Cameron wrote:
> On Sat, 8 Jun 2024 19:07:48 +0100
> Jonathan Cameron <jic23@xxxxxxxxxx> wrote:
>
> > On Thu, 6 Jun 2024 09:22:37 +0200
> > Nuno Sa <nuno.sa@xxxxxxxxxx> wrote:
> >
> > > This is similar to dev_err_probe() but for cases where an ERR_PTR() or
> > > ERR_CAST() is to be returned simplifying patterns like:
> > >
> > > dev_err_probe(dev, ret, ...);
> > > return ERR_PTR(ret)
> > > or
> > > dev_err_probe(dev, PTR_ERR(ptr), ...);
> > > return ERR_CAST(ptr)
> > >
> > > Signed-off-by: Nuno Sa <nuno.sa@xxxxxxxxxx>
> >
> > I'm convinced this is worth doing but would like inputs from others
> > before I pick this series up.
>
> Andi and Andy,
>
> You both commented on earlier versions. Do you think this is a good
> change set?
>
> I've +CC a few more based on a quick look at the original
> dev_err_probe() series. Whilst this isn't adding a bunch of new stuff
> around deferred probing (like that series did), maybe some of those
> reviewers will give opinions here?
>
Hi,
I there something else needed from my side? Would be nice to have some
feedback...
- Nuno Sá