diff options
| author | Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com> | 2026-05-04 22:00:05 +0300 |
|---|---|---|
| committer | Krzysztof Kozlowski <krzk@kernel.org> | 2026-05-14 19:35:30 +0300 |
| commit | 67de69ef2a780efd71b222652a048ee6aaf45e42 (patch) | |
| tree | 917b69115a1292a39e75c454f6bca0ed5a9231af | |
| parent | 4eda0dac7a2807c0bb628a1931b4f67d945c3758 (diff) | |
| download | linux-67de69ef2a780efd71b222652a048ee6aaf45e42.tar.xz | |
memory: tegra114-emc: Do not print error on icc_node_create() failure
icc_node_create() is alloc-like function, so no need to print error
messages on its failure. Dropping one label makes the code a bit
simpler.
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
Link: https://patch.msgid.link/20260504190004.747509-3-krzysztof.kozlowski@oss.qualcomm.com
| -rw-r--r-- | drivers/memory/tegra/tegra114-emc.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/drivers/memory/tegra/tegra114-emc.c b/drivers/memory/tegra/tegra114-emc.c index fc3bf1a28630..559ce47694a5 100644 --- a/drivers/memory/tegra/tegra114-emc.c +++ b/drivers/memory/tegra/tegra114-emc.c @@ -1204,10 +1204,8 @@ static int tegra114_emc_interconnect_init(struct tegra_emc *emc) /* create External Memory Controller node */ node = icc_node_create(TEGRA_ICC_EMC); - if (IS_ERR(node)) { - err = PTR_ERR(node); - goto err_msg; - } + if (IS_ERR(node)) + return PTR_ERR(node); node->name = "External Memory Controller"; icc_node_add(node, &emc->provider); @@ -1235,7 +1233,6 @@ static int tegra114_emc_interconnect_init(struct tegra_emc *emc) remove_nodes: icc_nodes_remove(&emc->provider); -err_msg: dev_err(emc->dev, "failed to initialize ICC: %d\n", err); return err; |