Skip to content

Commit 5373438

Browse files
andredkrzk
authored andcommitted
firmware: exynos-acpm: silence EPROBE_DEFER error on boot
This driver emits error messages when client drivers are trying to get an interface handle to this driver here before this driver has completed _probe(). Given this driver returns -EPROBE_DEFER in that case, this is not an error and shouldn't be emitted to the log, similar to how dev_err_probe() behaves, so just remove them. This change also allows us to simplify the logic around releasing of the acpm_np handle. Fixes: a88927b ("firmware: add Exynos ACPM protocol driver") Signed-off-by: André Draszik <[email protected]> Link: https://lore.kernel.org/r/[email protected] Signed-off-by: Krzysztof Kozlowski <[email protected]>
1 parent 67af3cd commit 5373438

File tree

1 file changed

+2
-9
lines changed

1 file changed

+2
-9
lines changed

drivers/firmware/samsung/exynos-acpm.c

Lines changed: 2 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -680,24 +680,17 @@ static const struct acpm_handle *acpm_get_by_phandle(struct device *dev,
680680
return ERR_PTR(-ENODEV);
681681

682682
pdev = of_find_device_by_node(acpm_np);
683-
if (!pdev) {
684-
dev_err(dev, "Cannot find device node %s\n", acpm_np->name);
685-
of_node_put(acpm_np);
686-
return ERR_PTR(-EPROBE_DEFER);
687-
}
688-
689683
of_node_put(acpm_np);
684+
if (!pdev)
685+
return ERR_PTR(-EPROBE_DEFER);
690686

691687
acpm = platform_get_drvdata(pdev);
692688
if (!acpm) {
693-
dev_err(dev, "Cannot get drvdata from %s\n",
694-
dev_name(&pdev->dev));
695689
platform_device_put(pdev);
696690
return ERR_PTR(-EPROBE_DEFER);
697691
}
698692

699693
if (!try_module_get(pdev->dev.driver->owner)) {
700-
dev_err(dev, "Cannot get module reference.\n");
701694
platform_device_put(pdev);
702695
return ERR_PTR(-EPROBE_DEFER);
703696
}

0 commit comments

Comments
 (0)