| Message ID | 20201126191146.8753-3-prabhakar.mahadev-lad.rj@bp.renesas.com |
|---|---|
| State | Accepted |
| Commit | 61a6d854b9555b420fbfae62ef26baa8b9493b32 |
| Headers | show |
| Series | None | expand |
On 11/26/20 10:11 PM, Lad Prabhakar wrote: > rpcif_enable_rpm calls pm_runtime_enable, so rpcif_disable_rpm needs to > call pm_runtime_disable and not pm_runtime_put_sync. > > Fixes: ca7d8b980b67f ("memory: add Renesas RPC-IF driver") > Reported-by: Reported-by: Geert Uytterhoeven <geert+renesas@glider.be> Reported by reported? :-) > Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> > Cc: stable@vger.kernel.org Reviewed-by: Sergei Shtylyov <sergei.shtylyov@gmail.com> [...] MBR, Sergei
On Thu, Nov 26, 2020 at 07:11:43PM +0000, Lad Prabhakar wrote: > rpcif_enable_rpm calls pm_runtime_enable, so rpcif_disable_rpm needs to > call pm_runtime_disable and not pm_runtime_put_sync. > > Fixes: ca7d8b980b67f ("memory: add Renesas RPC-IF driver") > Reported-by: Reported-by: Geert Uytterhoeven <geert+renesas@glider.be> > Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> > Cc: stable@vger.kernel.org > --- > drivers/memory/renesas-rpc-if.c | 2 +- Thanks, applied with corrected Reported-by. Best regards, Krzysztof
diff --git a/drivers/memory/renesas-rpc-if.c b/drivers/memory/renesas-rpc-if.c index 69f2e2b4cd50..a8d0ba368625 100644 --- a/drivers/memory/renesas-rpc-if.c +++ b/drivers/memory/renesas-rpc-if.c @@ -212,7 +212,7 @@ EXPORT_SYMBOL(rpcif_enable_rpm); void rpcif_disable_rpm(struct rpcif *rpc) { - pm_runtime_put_sync(rpc->dev); + pm_runtime_disable(rpc->dev); } EXPORT_SYMBOL(rpcif_disable_rpm);
rpcif_enable_rpm calls pm_runtime_enable, so rpcif_disable_rpm needs to call pm_runtime_disable and not pm_runtime_put_sync. Fixes: ca7d8b980b67f ("memory: add Renesas RPC-IF driver") Reported-by: Reported-by: Geert Uytterhoeven <geert+renesas@glider.be> Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> Cc: stable@vger.kernel.org --- drivers/memory/renesas-rpc-if.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)