Skip to content

firmware: tegra: add checks for BPMP error return code

Timo Alho linux-clk linux-tegra lkml 1 version Applied

Status from this message: Applied, thanks.

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest4 patches 15 repliesApplied

Patches in v1 Applied

#Subject and filesSizeRepliesApplied
1/4firmware: tegra: propagate error code to callerdrivers/firmware/tegra/bpmp.c include/soc/tegra/bpmp.h+17 -62 replies
2/4clk: tegra: check BPMP response return codedrivers/clk/tegra/clk-bpmp.c+10 -57 replies
3/4reset: tegra: check BPMP response return codedrivers/reset/tegra/reset-bpmp.c+8 -14 replies
4/4soc/tegra: bpmp: check BPMP response return codedrivers/soc/tegra/powergate-bpmp.c+13 -22 replies

Reviews trailers per patch

All 4 patches have a Reviewed-by or Acked-by.

PatchJon HunterPhilipp ZabelStephen Boyd
1/4A-b
2/4A-b A-b
3/4A-b A-b
4/4A-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes. Scroll the table sideways for all 3 reviewers.

Read the v1 thread

Cover letter [PATCH 0/4] firmware: tegra: add checks for BPMP error return code

Hi Thierry,

There is a bug in the BPMP driver as error code in response message is not being checked.

Patch 1 adds the error code as part of tegra_bpmp_message struct and propagates that code to callers

Patches 2 through 4 add code to client drivers to check the error code appropriately

BR,
Timo

Timo Alho (4):
firmware: tegra: propagate error code to caller
clk: tegra: check BPMP response return code
reset: tegra: check BPMP response return code soc/tegra: bpmp: check BPMP response return code

 drivers/clk/tegra/clk-bpmp.c       | 15 ++++++++++-----
 drivers/firmware/tegra/bpmp.c      | 22 ++++++++++++++++------
 drivers/reset/tegra/reset-bpmp.c   |  9 ++++++++-
 drivers/soc/tegra/powergate-bpmp.c | 15 +++++++++++++--
 include/soc/tegra/bpmp.h           |  1 +
 5 files changed, 48 insertions(+), 14 deletions(-)
Signature · 1 line
2.7.4