| Message ID | 20231107112713.21399-1-amhetre@nvidia.com |
|---|---|
| State | Handled Elsewhere |
| Headers | show |
| Series | [V2,1/2] memory: tegra: Add SID override programming for MC clients | expand |
On Tue, 07 Nov 2023 16:57:12 +0530, Ashish Mhetre wrote: > For some devices the bootloader/firmware may set up the device in > bypass. Memory clients like display needs kernel to program SID after > resume because bootloader/firmware programs the SID of display device to > bypass. In order to make sure that kernel IOMMU mappings for these > devices work after resume, add SID override programming support for all > memory clients on memory controller resume. > > [...] Applied, thanks! [1/2] memory: tegra: Add SID override programming for MC clients https://git.kernel.org/krzk/linux-mem-ctrl/c/fe3b082a6eb8b1526ed7397c849d6b2a6baeb6a1 [2/2] memory: tegra: Skip SID programming if SID registers aren't set https://git.kernel.org/krzk/linux-mem-ctrl/c/0d6c918011ce4764ed277de4726a468b7ffe5fed Best regards,
diff --git a/drivers/memory/tegra/tegra186.c b/drivers/memory/tegra/tegra186.c index 533f85a4b2bd..8203f0db1350 100644 --- a/drivers/memory/tegra/tegra186.c +++ b/drivers/memory/tegra/tegra186.c @@ -136,9 +136,23 @@ static int tegra186_mc_probe_device(struct tegra_mc *mc, struct device *dev) return 0; } +static int tegra186_mc_resume(struct tegra_mc *mc) +{ + unsigned int i; + + for (i = 0; i < mc->soc->num_clients; i++) { + const struct tegra_mc_client *client = &mc->soc->clients[i]; + + tegra186_mc_client_sid_override(mc, client, client->sid); + } + + return 0; +} + const struct tegra_mc_ops tegra186_mc_ops = { .probe = tegra186_mc_probe, .remove = tegra186_mc_remove, + .resume = tegra186_mc_resume, .probe_device = tegra186_mc_probe_device, .handle_irq = tegra30_mc_handle_irq, }; diff --git a/include/soc/tegra/mc.h b/include/soc/tegra/mc.h index 71ae37d3bedd..af1d73a7f0cd 100644 --- a/include/soc/tegra/mc.h +++ b/include/soc/tegra/mc.h @@ -162,6 +162,7 @@ struct tegra_mc_ops { */ int (*probe)(struct tegra_mc *mc); void (*remove)(struct tegra_mc *mc); + int (*resume)(struct tegra_mc *mc); irqreturn_t (*handle_irq)(int irq, void *data); int (*probe_device)(struct tegra_mc *mc, struct device *dev); };
For some devices the bootloader/firmware may set up the device in bypass. Memory clients like display needs kernel to program SID after resume because bootloader/firmware programs the SID of display device to bypass. In order to make sure that kernel IOMMU mappings for these devices work after resume, add SID override programming support for all memory clients on memory controller resume. This partially reverts 'commit ef86b2c2807f ("memory: tegra: Remove clients SID override programming")' Signed-off-by: Ashish Mhetre <amhetre@nvidia.com> --- Changes in v2: - Fixed build issues reported by Kernel test robot from v1 drivers/memory/tegra/tegra186.c | 14 ++++++++++++++ include/soc/tegra/mc.h | 1 + 2 files changed, 15 insertions(+)