mirror of
git://nv-tegra.nvidia.com/linux-nv-oot.git
synced 2025-12-22 17:25:35 +03:00
drm/tegra: Don't print error on probe defer
The following error messages are sometimes observed on boot ...
tegra-nvjpg 15380000.nvjpg: failed to get icc write handle
tegra-nvdec 15480000.nvdec: failed to get icc write handle
tegra-nvjpg 15540000.nvjpg: failed to get icc write handle
tegra-nvenc 154c0000.nvenc: failed to get icc write handle
tegra-vic 15340000.vic: failed to get icc write handle
tegra-nvjpg 15380000.nvjpg: failed to get icc write handle
The above messages are harmless because the ICC core is returning
-EPROBE_DEFER to indicate that the ICC provider is not available. When
-EPROBE_DEFER is returned the kernel will attempt to probe the device
again and so print an error on -EPROBE_DEFER can be misleading. Fix the
above by using the function dev_err_probe() to print error messages
because this function will only print an error message if the error code
is not -EPROBE_DEFER.
Bug 3436156
Bug 4496044
Change-Id: I47b77e5a0f2bdb817a832daa305246c8803f456b
Signed-off-by: Jon Hunter <jonathanh@nvidia.com>
Reviewed-on: https://git-master.nvidia.com/r/c/linux-nv-oot/+/3075237
(cherry picked from commit 61319aef4d)
Reviewed-on: https://git-master.nvidia.com/r/c/linux-nv-oot/+/3075849
Reviewed-by: Brad Griffis <bgriffis@nvidia.com>
GVS: Gerrit_Virtual_Submit <buildbot_gerritrpt@nvidia.com>
This commit is contained in:
committed by
mobile promotions
parent
76261f8ace
commit
7e85d429e7
@@ -1,6 +1,6 @@
|
||||
// SPDX-License-Identifier: GPL-2.0-only
|
||||
/*
|
||||
* SPDX-FileCopyrightText: Copyright (c) 2015-2023, NVIDIA CORPORATION & AFFILIATES. All Rights Reserved.
|
||||
* SPDX-FileCopyrightText: Copyright (c) 2015-2024, NVIDIA CORPORATION & AFFILIATES. All Rights Reserved.
|
||||
*/
|
||||
|
||||
#include <linux/bitops.h>
|
||||
@@ -804,10 +804,9 @@ static int nvdec_probe(struct platform_device *pdev)
|
||||
}
|
||||
|
||||
nvdec->icc_write = devm_of_icc_get(dev, "write");
|
||||
if (IS_ERR(nvdec->icc_write)) {
|
||||
dev_err(&pdev->dev, "failed to get icc write handle\n");
|
||||
return PTR_ERR(nvdec->icc_write);
|
||||
}
|
||||
if (IS_ERR(nvdec->icc_write))
|
||||
return dev_err_probe(&pdev->dev, PTR_ERR(nvdec->icc_write),
|
||||
"failed to get icc write handle\n");
|
||||
|
||||
platform_set_drvdata(pdev, nvdec);
|
||||
|
||||
|
||||
@@ -1,6 +1,6 @@
|
||||
// SPDX-License-Identifier: GPL-2.0-only
|
||||
/*
|
||||
* SPDX-FileCopyrightText: Copyright (c) 2021-2023, NVIDIA CORPORATION & AFFILIATES. All Rights Reserved.
|
||||
* SPDX-FileCopyrightText: Copyright (c) 2021-2024, NVIDIA CORPORATION & AFFILIATES. All Rights Reserved.
|
||||
*/
|
||||
|
||||
#include <linux/bitops.h>
|
||||
@@ -670,10 +670,9 @@ static int nvenc_probe(struct platform_device *pdev)
|
||||
return err;
|
||||
|
||||
nvenc->icc_write = devm_of_icc_get(dev, "write");
|
||||
if (IS_ERR(nvenc->icc_write)) {
|
||||
dev_err(&pdev->dev, "failed to get icc write handle\n");
|
||||
return PTR_ERR(nvenc->icc_write);
|
||||
}
|
||||
if (IS_ERR(nvenc->icc_write))
|
||||
return dev_err_probe(&pdev->dev, PTR_ERR(nvenc->icc_write),
|
||||
"failed to get icc write handle\n");
|
||||
|
||||
platform_set_drvdata(pdev, nvenc);
|
||||
|
||||
|
||||
@@ -1,6 +1,6 @@
|
||||
// SPDX-License-Identifier: GPL-2.0-only
|
||||
/*
|
||||
* Copyright (c) 2021-2023, NVIDIA CORPORATION & AFFILIATES. All Rights Reserved.
|
||||
* SPDX-FileCopyrightText: Copyright (c) 2021-2024, NVIDIA CORPORATION & AFFILIATES. All Rights Reserved.
|
||||
*/
|
||||
|
||||
#include <linux/bitops.h>
|
||||
@@ -655,10 +655,9 @@ static int nvjpg_probe(struct platform_device *pdev)
|
||||
return err;
|
||||
|
||||
nvjpg->icc_write = devm_of_icc_get(dev, "write");
|
||||
if (IS_ERR(nvjpg->icc_write)) {
|
||||
dev_err(&pdev->dev, "failed to get icc write handle\n");
|
||||
return PTR_ERR(nvjpg->icc_write);
|
||||
}
|
||||
if (IS_ERR(nvjpg->icc_write))
|
||||
return dev_err_probe(&pdev->dev, PTR_ERR(nvjpg->icc_write),
|
||||
"failed to get icc write handle\n");
|
||||
|
||||
platform_set_drvdata(pdev, nvjpg);
|
||||
|
||||
|
||||
@@ -1,6 +1,6 @@
|
||||
// SPDX-License-Identifier: GPL-2.0-only
|
||||
/*
|
||||
* SPDX-FileCopyrightText: Copyright (C) 2015-2023 NVIDIA CORPORATION. All rights reserved.
|
||||
* SPDX-FileCopyrightText: Copyright (C) 2015-2024 NVIDIA CORPORATION. All rights reserved.
|
||||
*/
|
||||
|
||||
#include <linux/bitops.h>
|
||||
@@ -723,10 +723,9 @@ static int vic_probe(struct platform_device *pdev)
|
||||
}
|
||||
|
||||
vic->icc_write = devm_of_icc_get(dev, "write");
|
||||
if (IS_ERR(vic->icc_write)) {
|
||||
dev_err(&pdev->dev, "failed to get icc write handle\n");
|
||||
return PTR_ERR(vic->icc_write);
|
||||
}
|
||||
if (IS_ERR(vic->icc_write))
|
||||
return dev_err_probe(&pdev->dev, PTR_ERR(vic->icc_write),
|
||||
"failed to get icc write handle\n");
|
||||
|
||||
if (!dev->pm_domain) {
|
||||
vic->rst = devm_reset_control_get(dev, "vic");
|
||||
|
||||
Reference in New Issue
Block a user