Re: [PATCH v2 0/5] memory: renesas-rpc-if: Trivial fixes

From: Geert Uytterhoeven <geert@linux-m68k.org>
To: "Lad, Prabhakar" <prabhakar.csengg@gmail.com>
Cc: Krzysztof Kozlowski <krzk@kernel.org>,
	Sergei Shtylyov <sergei.shtylyov@gmail.com>,
	Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>,
	Philipp Zabel <p.zabel@pengutronix.de>,
	Jiri Kosina <trivial@kernel.org>, Mark Brown <broonie@kernel.org>,
	Linux-Renesas <linux-renesas-soc@vger.kernel.org>,
	Pavel Machek <pavel@denx.de>,
	Geert Uytterhoeven <geert+renesas@glider.be>,
	LKML <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v2 0/5] memory: renesas-rpc-if: Trivial fixes
Date: Thu, 3 Dec 2020 11:52:26 +0100	[thread overview]
Message-ID: <CAMuHMdUHujqMnCKH14GM1SzN+3_E4j9u1R8uDuM3YmVjcd_MTw@mail.gmail.com> (raw)
In-Reply-To: <CA+V-a8vjTQv7wrdJFe6TS3saUE=Sj6ty0JSz0VZUd=TyDVfp4Q@mail.gmail.com>

Hi Prabhakar,

On Thu, Dec 3, 2020 at 11:42 AM Lad, Prabhakar
<prabhakar.csengg@gmail.com> wrote:
> On Thu, Nov 26, 2020 at 7:11 PM Lad Prabhakar
> <prabhakar.mahadev-lad.rj@bp.renesas.com> wrote:
> > This patch series fixes trivial issues in RPC-IF driver.
> >
> > Changes for v2:
> > * Balanced PM in rpcif_disable_rpm
> > * Fixed typo in patch 4/5
> > * Dropped C++ style fixes patch
> > * Included RB tags from Sergei
> >
> > Cheers,
> > Prabhakar
> >
> > Lad Prabhakar (5):
> >   memory: renesas-rpc-if: Return correct value to the caller of
> >     rpcif_manual_xfer()
> >   memory: renesas-rpc-if: Fix unbalanced pm_runtime_enable in
> >     rpcif_{enable,disable}_rpm
> >   memory: renesas-rpc-if: Fix a reference leak in rpcif_probe()
> >   memory: renesas-rpc-if: Make rpcif_enable/disable_rpm() as static
> >     inline
> >   memory: renesas-rpc-if: Export symbols as GPL
> >
> As these are fixes to the existing driver will these be part of v5.10 release ?

IIUIC, only the first one[*] is a fix for an issue that could happen during
normal operation?

[*] -EPROBE_DEFER would be eaten, causing no reprobe to happen.

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds