memory: tegra186-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. Reviewed-by: Jon Hunter <jonathanh@nvidia.com> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
This commit is contained in:
@@ -273,10 +273,8 @@ static int tegra_emc_interconnect_init(struct tegra186_emc *emc)
|
|||||||
|
|
||||||
/* create External Memory Controller node */
|
/* create External Memory Controller node */
|
||||||
node = icc_node_create(TEGRA_ICC_EMC);
|
node = icc_node_create(TEGRA_ICC_EMC);
|
||||||
if (IS_ERR(node)) {
|
if (IS_ERR(node))
|
||||||
err = PTR_ERR(node);
|
return PTR_ERR(node);
|
||||||
goto err_msg;
|
|
||||||
}
|
|
||||||
|
|
||||||
node->name = "External Memory Controller";
|
node->name = "External Memory Controller";
|
||||||
icc_node_add(node, &emc->provider);
|
icc_node_add(node, &emc->provider);
|
||||||
@@ -304,7 +302,6 @@ static int tegra_emc_interconnect_init(struct tegra186_emc *emc)
|
|||||||
|
|
||||||
remove_nodes:
|
remove_nodes:
|
||||||
icc_nodes_remove(&emc->provider);
|
icc_nodes_remove(&emc->provider);
|
||||||
err_msg:
|
|
||||||
dev_err(emc->dev, "failed to initialize ICC: %d\n", err);
|
dev_err(emc->dev, "failed to initialize ICC: %d\n", err);
|
||||||
|
|
||||||
return err;
|
return err;
|
||||||
|
|||||||
Reference in New Issue
Block a user