Hi Fabio, On Wed, Jun 24, 2020 at 1:55 PM Fabio Estevam <feste...@gmail.com> wrote: > > We could avoid checking for if (!IS_ERR(gpr)) two times in a row if > the following patch is applied before yours: > https://pastebin.com/raw/Ftkq2VVp > > What do you think?
I think this is a good idea. I was never fully comfortable with checking IS_ERR(gpr) twice, but I know from experience that minimal patches are more likely to be accepted. Shawn / Andy, what do you think? Should I re-spin this as a patch set, with Fabio's patch in front?
