Skip to content

memory: tegra: Skip restricted register access from Guest

Sumit Gupta linux-devicetree linux-tegra lkml 2 versions – In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v3 Latest2 patches 19 repliesIn review
  2. v22 patches 4 repliesSuperseded

Patches in v3 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/2dt-bindings: make sid and broadcast reg optionalDocumentation/devicetree/bindings/memory-controllers/nvidia,tegra186-mc.yaml+49 -4611 replies
2/2memory: tegra: make sid and broadcast regions optionaldrivers/memory/tegra/mc.c drivers/memory/tegra/mc.h drivers/memory/tegra/tegra186.c+33 -238 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v3 yet.

Read the v3 thread

Cover letter [Patch v3 0/2] memory: tegra: Skip restricted register access from Guest

MC SID and Broadbast channel register access is restricted for Guest VM. But the Tegra MC driver is considering both these regions as mandatory and causes error on access. Change that to consider both the regions as optional in SoC's from Tegra186 onwards. Then skip access to the restricted registers from Guest if its region is not present in Guest DT and hence not mapped.
Previously, the solution in [1] was not accepted. Now, handled the problem with this alternate solution.

---
v2[2] -> v3:
- fix the yaml error.
- add null check for ch_regs in mc_ch_readl() causing error in old SoC's
- remove unused populate label reported by kernel test robot.
- fix existing compile warning about length of name in tegra186_mc_probe.

v1[1] -> v2:
- consider broadcast channel registers also as restricted along with sid
- make sid and broadcast regions optional and access if present in dt.
- update the yaml file.

Sumit Gupta (2):
dt-bindings: make sid and broadcast reg optional
memory: tegra: make sid and broadcast regions optional

 .../nvidia,tegra186-mc.yaml                   | 95 ++++++++++---------
 drivers/memory/tegra/mc.c                     |  9 +-
 drivers/memory/tegra/mc.h                     | 22 +++--
 drivers/memory/tegra/tegra186.c               | 25 ++---
 4 files changed, 82 insertions(+), 69 deletions(-)

[1]https://lore.kernel.org/lkml/[email protected]/ [2]https://lore.kernel.org/lkml/[email protected]/

Signature · 1 line
2.17.1