memory: tegra: Fix KCONFIG variables for Tegra186 and Tegra194
Subsystems (2)
PATCH 1/1 memory: tegra: Fix KCONFIG variables for Tegra186 and Tegra194
Commit a127e690b051 ("memory: tegra: Add support for the Tegra194 memory controller") and commit 4e04b88633ae ("memory: tegra: Only include support for enabled SoCs") incorrectly added the KCONFIG variables CONFIG_ARCH_TEGRA186_SOC and CONFIG_ARCH_TEGRA194_SOC to the Tegra EMC driver. These KCONFIG variables do not exist and prevent the EMC driver from being probed on Tegra186 and Tegra194. These KCONFIG variable names are simply missing one underscore and so fix this by adding the necessary underscore to the variable names.
--- drivers/memory/tegra/tegra186-emc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
drivers/memory/tegra/tegra186-emc.c +2 -2
@@ -268,10 +268,10 @@ static int tegra186_emc_remove(struct platform_device *pdev) }static const struct of_device_id tegra186_emc_of_match[] = { -#if defined(CONFIG_ARCH_TEGRA186_SOC) +#if defined(CONFIG_ARCH_TEGRA_186_SOC) { .compatible ="nvidia,tegra186-emc" }, #endif -#if defined(CONFIG_ARCH_TEGRA194_SOC) +#if defined(CONFIG_ARCH_TEGRA_194_SOC) { .compatible ="nvidia,tegra194-emc" }, #endif {/* sentinel */ }
Signature · 1 line
2.17.1
A message that is not in the archive (sent off-list or not yet received): 20200712103347.25070-1-jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org
Quoted 9 lines · Jon Hunter
Commit
a127e690b051("memory: tegra: Add support for the Tegra194 memory controller") and commit4e04b88633ae("memory: tegra: Only include support for enabled SoCs") incorrectly added the KCONFIG variablesCONFIG_ARCH_TEGRA186_SOCandCONFIG_ARCH_TEGRA194_SOCto the Tegra EMC driver. These KCONFIG variables do not exist and prevent the EMC driver from being probed on Tegra186 and Tegra194. These KCONFIG variable names are simply missing one underscore and so fix this by adding the necessary underscore to the variable names.
Quoted 4 lines
Signed-off-by Jon Hunter <[email protected]>--- drivers/memory/tegra/tegra186-emc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
I wonder why we didn't catch that earlier. I assume the EMC frequency scaling tests just don't run on these SoC generations with existing kernels, then?
Oddly enough, I also vaguely recall testing EMC frequency scaling on both SoC generations, so I wonder where things went wrong. Anyway, this is obviously correct, so I've applied it.
Thanks,
Thierry