diff mbox series

drm/msm/adreno: Do not print error on "qcom, gpu-pwrlevels" absence

Message ID 20190905111648.11559-1-festevam@gmail.com (mailing list archive)
State New, archived
Headers show
Series drm/msm/adreno: Do not print error on "qcom, gpu-pwrlevels" absence | expand

Commit Message

Fabio Estevam Sept. 5, 2019, 11:16 a.m. UTC
Booting the adreno driver on a imx53 board leads to the following
error message:

adreno 30000000.gpu: [drm:adreno_gpu_init] *ERROR* Could not find the GPU powerlevels

As the "qcom,gpu-pwrlevels" property is optional and never present on
i.MX5, turn the message into debug level instead.

Signed-off-by: Fabio Estevam <festevam@gmail.com>
---
 drivers/gpu/drm/msm/adreno/adreno_gpu.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Jordan Crouse Sept. 5, 2019, 4:38 p.m. UTC | #1
On Thu, Sep 05, 2019 at 08:16:48AM -0300, Fabio Estevam wrote:
> Booting the adreno driver on a imx53 board leads to the following
> error message:
> 
> adreno 30000000.gpu: [drm:adreno_gpu_init] *ERROR* Could not find the GPU powerlevels
> 
> As the "qcom,gpu-pwrlevels" property is optional and never present on
> i.MX5, turn the message into debug level instead.

Sounds legit to me. This is mainly for compatibility with downstream device tree
files and the folks doing DRM on Android should know what they are doing.

Reviewed-by: Jordan Crouse <jcrouse@codeaurora.org>

> Signed-off-by: Fabio Estevam <festevam@gmail.com>
> ---
>  drivers/gpu/drm/msm/adreno/adreno_gpu.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/msm/adreno/adreno_gpu.c b/drivers/gpu/drm/msm/adreno/adreno_gpu.c
> index 048c8be426f3..73c79f1614c1 100644
> --- a/drivers/gpu/drm/msm/adreno/adreno_gpu.c
> +++ b/drivers/gpu/drm/msm/adreno/adreno_gpu.c
> @@ -825,7 +825,7 @@ static int adreno_get_legacy_pwrlevels(struct device *dev)
>  
>  	node = of_get_compatible_child(dev->of_node, "qcom,gpu-pwrlevels");
>  	if (!node) {
> -		DRM_DEV_ERROR(dev, "Could not find the GPU powerlevels\n");
> +		DRM_DEV_DEBUG(dev, "Could not find the GPU powerlevels\n");
>  		return -ENXIO;
>  	}
>  
> -- 
> 2.17.1
>
diff mbox series

Patch

diff --git a/drivers/gpu/drm/msm/adreno/adreno_gpu.c b/drivers/gpu/drm/msm/adreno/adreno_gpu.c
index 048c8be426f3..73c79f1614c1 100644
--- a/drivers/gpu/drm/msm/adreno/adreno_gpu.c
+++ b/drivers/gpu/drm/msm/adreno/adreno_gpu.c
@@ -825,7 +825,7 @@  static int adreno_get_legacy_pwrlevels(struct device *dev)
 
 	node = of_get_compatible_child(dev->of_node, "qcom,gpu-pwrlevels");
 	if (!node) {
-		DRM_DEV_ERROR(dev, "Could not find the GPU powerlevels\n");
+		DRM_DEV_DEBUG(dev, "Could not find the GPU powerlevels\n");
 		return -ENXIO;
 	}