Skip to content

Commit 2ac5ba4

Browse files
committed
memory: tegra-mc: Drop tegra_mc_setup_latency_allowance() return value
tegra_mc_setup_latency_allowance() only succeeds, thus its return value can be dropped making code a bit simpler. Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com> Acked-by: Thierry Reding <treding@nvidia.com> Link: https://patch.msgid.link/20260226-memory-simplify-v1-2-ccb94f378628@oss.qualcomm.com Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
1 parent 8a39b1d commit 2ac5ba4

1 file changed

Lines changed: 2 additions & 8 deletions

File tree

  • drivers/memory/tegra

drivers/memory/tegra/mc.c

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -386,7 +386,7 @@ EXPORT_SYMBOL_GPL(tegra_mc_get_emem_device_count);
386386
defined(CONFIG_ARCH_TEGRA_124_SOC) || \
387387
defined(CONFIG_ARCH_TEGRA_132_SOC) || \
388388
defined(CONFIG_ARCH_TEGRA_210_SOC)
389-
static int tegra_mc_setup_latency_allowance(struct tegra_mc *mc)
389+
static void tegra_mc_setup_latency_allowance(struct tegra_mc *mc)
390390
{
391391
unsigned long long tick;
392392
unsigned int i;
@@ -414,8 +414,6 @@ static int tegra_mc_setup_latency_allowance(struct tegra_mc *mc)
414414

415415
/* latch new values */
416416
mc_writel(mc, MC_TIMING_UPDATE, MC_TIMING_CONTROL);
417-
418-
return 0;
419417
}
420418

421419
static int load_one_timing(struct tegra_mc *mc,
@@ -517,11 +515,7 @@ int tegra30_mc_probe(struct tegra_mc *mc)
517515
/* ensure that debug features are disabled */
518516
mc_writel(mc, 0x00000000, MC_TIMING_CONTROL_DBG);
519517

520-
err = tegra_mc_setup_latency_allowance(mc);
521-
if (err < 0) {
522-
dev_err(mc->dev, "failed to setup latency allowance: %d\n", err);
523-
return err;
524-
}
518+
tegra_mc_setup_latency_allowance(mc);
525519

526520
err = tegra_mc_setup_timings(mc);
527521
if (err < 0) {

0 commit comments

Comments
 (0)