* [PATCH] drm/amd/powerplay: warn on smu interface version mismatch
@ 2019-06-26 6:24 Yuan, Xiaojie
[not found] ` <20190626062402.18551-1-xiaojie.yuan-5C7GfCeVMHo@public.gmane.org>
0 siblings, 1 reply; 5+ messages in thread
From: Yuan, Xiaojie @ 2019-06-26 6:24 UTC (permalink / raw)
To: amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW; +Cc: Wang, Kevin(Yang), Yuan, Xiaojie
Signed-off-by: Xiaojie Yuan <xiaojie.yuan@amd.com>
---
drivers/gpu/drm/amd/powerplay/smu_v11_0.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/amd/powerplay/smu_v11_0.c b/drivers/gpu/drm/amd/powerplay/smu_v11_0.c
index c3f48fae6f32..339d063e24ff 100644
--- a/drivers/gpu/drm/amd/powerplay/smu_v11_0.c
+++ b/drivers/gpu/drm/amd/powerplay/smu_v11_0.c
@@ -272,8 +272,7 @@ static int smu_v11_0_check_fw_version(struct smu_context *smu)
"smu fw version = 0x%08x (%d.%d.%d)\n",
smu->smc_if_version, if_version,
smu_version, smu_major, smu_minor, smu_debug);
- pr_err("SMU driver if version not matched\n");
- ret = -EINVAL;
+ pr_warn("SMU driver if version not matched\n");
}
return ret;
--
2.20.1
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] drm/amd/powerplay: warn on smu interface version mismatch
[not found] ` <20190626062402.18551-1-xiaojie.yuan-5C7GfCeVMHo@public.gmane.org>
@ 2019-06-26 6:34 ` Yuan, Xiaojie
[not found] ` <BN8PR12MB36021167B2EBF7F48CE93CCA89E20-h6+T2+wrnx1lGJ5xs4l7kQdYzm3356FpvxpqHgZTriW3zl9H0oFU5g@public.gmane.org>
0 siblings, 1 reply; 5+ messages in thread
From: Yuan, Xiaojie @ 2019-06-26 6:34 UTC (permalink / raw)
To: amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW; +Cc: Wang, Kevin(Yang)
Current SMU IF version check is too strict, driver with old smu11_driver_if.h sometimes works fine with new SMU firmware. We prefer to see a warning instead a error for debug purposes.
BR,
Xiaojie
________________________________________
From: Yuan, Xiaojie
Sent: Wednesday, June 26, 2019 2:24:19 PM
To: amd-gfx@lists.freedesktop.org
Cc: Wang, Kevin(Yang); Yuan, Xiaojie
Subject: [PATCH] drm/amd/powerplay: warn on smu interface version mismatch
Signed-off-by: Xiaojie Yuan <xiaojie.yuan@amd.com>
---
drivers/gpu/drm/amd/powerplay/smu_v11_0.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/amd/powerplay/smu_v11_0.c b/drivers/gpu/drm/amd/powerplay/smu_v11_0.c
index c3f48fae6f32..339d063e24ff 100644
--- a/drivers/gpu/drm/amd/powerplay/smu_v11_0.c
+++ b/drivers/gpu/drm/amd/powerplay/smu_v11_0.c
@@ -272,8 +272,7 @@ static int smu_v11_0_check_fw_version(struct smu_context *smu)
"smu fw version = 0x%08x (%d.%d.%d)\n",
smu->smc_if_version, if_version,
smu_version, smu_major, smu_minor, smu_debug);
- pr_err("SMU driver if version not matched\n");
- ret = -EINVAL;
+ pr_warn("SMU driver if version not matched\n");
}
return ret;
--
2.20.1
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] drm/amd/powerplay: warn on smu interface version mismatch
[not found] ` <BN8PR12MB36021167B2EBF7F48CE93CCA89E20-h6+T2+wrnx1lGJ5xs4l7kQdYzm3356FpvxpqHgZTriW3zl9H0oFU5g@public.gmane.org>
@ 2019-06-26 6:37 ` Kevin Wang
2019-06-27 1:51 ` Quan, Evan
1 sibling, 0 replies; 5+ messages in thread
From: Kevin Wang @ 2019-06-26 6:37 UTC (permalink / raw)
To: Yuan, Xiaojie, amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW; +Cc: Wang, Kevin(Yang)
please add this message in patch commit. after
Reviewed-by: Kevin Wang <kevin1.wang@amd.com>
Best Regards,
Kevin
On 6/26/19 2:34 PM, Yuan, Xiaojie wrote:
> Current SMU IF version check is too strict, driver with old smu11_driver_if.h sometimes works fine with new SMU firmware. We prefer to see a warning instead a error for debug purposes.
>
> BR,
> Xiaojie
>
> ________________________________________
> From: Yuan, Xiaojie
> Sent: Wednesday, June 26, 2019 2:24:19 PM
> To: amd-gfx@lists.freedesktop.org
> Cc: Wang, Kevin(Yang); Yuan, Xiaojie
> Subject: [PATCH] drm/amd/powerplay: warn on smu interface version mismatch
>
> Signed-off-by: Xiaojie Yuan <xiaojie.yuan@amd.com>
> ---
> drivers/gpu/drm/amd/powerplay/smu_v11_0.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/powerplay/smu_v11_0.c b/drivers/gpu/drm/amd/powerplay/smu_v11_0.c
> index c3f48fae6f32..339d063e24ff 100644
> --- a/drivers/gpu/drm/amd/powerplay/smu_v11_0.c
> +++ b/drivers/gpu/drm/amd/powerplay/smu_v11_0.c
> @@ -272,8 +272,7 @@ static int smu_v11_0_check_fw_version(struct smu_context *smu)
> "smu fw version = 0x%08x (%d.%d.%d)\n",
> smu->smc_if_version, if_version,
> smu_version, smu_major, smu_minor, smu_debug);
> - pr_err("SMU driver if version not matched\n");
> - ret = -EINVAL;
> + pr_warn("SMU driver if version not matched\n");
> }
>
> return ret;
> --
> 2.20.1
>
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
^ permalink raw reply [flat|nested] 5+ messages in thread
* RE: [PATCH] drm/amd/powerplay: warn on smu interface version mismatch
[not found] ` <BN8PR12MB36021167B2EBF7F48CE93CCA89E20-h6+T2+wrnx1lGJ5xs4l7kQdYzm3356FpvxpqHgZTriW3zl9H0oFU5g@public.gmane.org>
2019-06-26 6:37 ` Kevin Wang
@ 2019-06-27 1:51 ` Quan, Evan
[not found] ` <MN2PR12MB3344BE7B57FC531F96DEDB44E4FD0-rweVpJHSKToDMgCC8P//OwdYzm3356FpvxpqHgZTriW3zl9H0oFU5g@public.gmane.org>
1 sibling, 1 reply; 5+ messages in thread
From: Quan, Evan @ 2019-06-27 1:51 UTC (permalink / raw)
To: Yuan, Xiaojie, amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW; +Cc: Wang, Kevin(Yang)
I do not think this is a good idea.
As there is still some cases that version mismatch will cause unexpected issues. And they will be hard to debug.
If this is for debug purpose only, I would suggest to keep this in your custom branch only.
Regards,
Evan
> -----Original Message-----
> From: amd-gfx <amd-gfx-bounces@lists.freedesktop.org> On Behalf Of
> Yuan, Xiaojie
> Sent: Wednesday, June 26, 2019 2:34 PM
> To: amd-gfx@lists.freedesktop.org
> Cc: Wang, Kevin(Yang) <Kevin1.Wang@amd.com>
> Subject: Re: [PATCH] drm/amd/powerplay: warn on smu interface version
> mismatch
>
> Current SMU IF version check is too strict, driver with old smu11_driver_if.h
> sometimes works fine with new SMU firmware. We prefer to see a warning
> instead a error for debug purposes.
>
> BR,
> Xiaojie
>
> ________________________________________
> From: Yuan, Xiaojie
> Sent: Wednesday, June 26, 2019 2:24:19 PM
> To: amd-gfx@lists.freedesktop.org
> Cc: Wang, Kevin(Yang); Yuan, Xiaojie
> Subject: [PATCH] drm/amd/powerplay: warn on smu interface version
> mismatch
>
> Signed-off-by: Xiaojie Yuan <xiaojie.yuan@amd.com>
> ---
> drivers/gpu/drm/amd/powerplay/smu_v11_0.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/powerplay/smu_v11_0.c
> b/drivers/gpu/drm/amd/powerplay/smu_v11_0.c
> index c3f48fae6f32..339d063e24ff 100644
> --- a/drivers/gpu/drm/amd/powerplay/smu_v11_0.c
> +++ b/drivers/gpu/drm/amd/powerplay/smu_v11_0.c
> @@ -272,8 +272,7 @@ static int smu_v11_0_check_fw_version(struct
> smu_context *smu)
> "smu fw version = 0x%08x (%d.%d.%d)\n",
> smu->smc_if_version, if_version,
> smu_version, smu_major, smu_minor, smu_debug);
> - pr_err("SMU driver if version not matched\n");
> - ret = -EINVAL;
> + pr_warn("SMU driver if version not matched\n");
> }
>
> return ret;
> --
> 2.20.1
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] drm/amd/powerplay: warn on smu interface version mismatch
[not found] ` <MN2PR12MB3344BE7B57FC531F96DEDB44E4FD0-rweVpJHSKToDMgCC8P//OwdYzm3356FpvxpqHgZTriW3zl9H0oFU5g@public.gmane.org>
@ 2019-06-27 8:09 ` Yuan, Xiaojie
0 siblings, 0 replies; 5+ messages in thread
From: Yuan, Xiaojie @ 2019-06-27 8:09 UTC (permalink / raw)
To: Quan, Evan, amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW; +Cc: Wang, Kevin(Yang)
Thanks Evan, got your point. Then I'll keep the patch only in our bring up branch.
BR,
Xiaojie
________________________________________
From: Quan, Evan
Sent: Thursday, June 27, 2019 9:51:22 AM
To: Yuan, Xiaojie; amd-gfx@lists.freedesktop.org
Cc: Wang, Kevin(Yang)
Subject: RE: [PATCH] drm/amd/powerplay: warn on smu interface version mismatch
I do not think this is a good idea.
As there is still some cases that version mismatch will cause unexpected issues. And they will be hard to debug.
If this is for debug purpose only, I would suggest to keep this in your custom branch only.
Regards,
Evan
> -----Original Message-----
> From: amd-gfx <amd-gfx-bounces@lists.freedesktop.org> On Behalf Of
> Yuan, Xiaojie
> Sent: Wednesday, June 26, 2019 2:34 PM
> To: amd-gfx@lists.freedesktop.org
> Cc: Wang, Kevin(Yang) <Kevin1.Wang@amd.com>
> Subject: Re: [PATCH] drm/amd/powerplay: warn on smu interface version
> mismatch
>
> Current SMU IF version check is too strict, driver with old smu11_driver_if.h
> sometimes works fine with new SMU firmware. We prefer to see a warning
> instead a error for debug purposes.
>
> BR,
> Xiaojie
>
> ________________________________________
> From: Yuan, Xiaojie
> Sent: Wednesday, June 26, 2019 2:24:19 PM
> To: amd-gfx@lists.freedesktop.org
> Cc: Wang, Kevin(Yang); Yuan, Xiaojie
> Subject: [PATCH] drm/amd/powerplay: warn on smu interface version
> mismatch
>
> Signed-off-by: Xiaojie Yuan <xiaojie.yuan@amd.com>
> ---
> drivers/gpu/drm/amd/powerplay/smu_v11_0.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/powerplay/smu_v11_0.c
> b/drivers/gpu/drm/amd/powerplay/smu_v11_0.c
> index c3f48fae6f32..339d063e24ff 100644
> --- a/drivers/gpu/drm/amd/powerplay/smu_v11_0.c
> +++ b/drivers/gpu/drm/amd/powerplay/smu_v11_0.c
> @@ -272,8 +272,7 @@ static int smu_v11_0_check_fw_version(struct
> smu_context *smu)
> "smu fw version = 0x%08x (%d.%d.%d)\n",
> smu->smc_if_version, if_version,
> smu_version, smu_major, smu_minor, smu_debug);
> - pr_err("SMU driver if version not matched\n");
> - ret = -EINVAL;
> + pr_warn("SMU driver if version not matched\n");
> }
>
> return ret;
> --
> 2.20.1
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2019-06-27 8:09 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-06-26 6:24 [PATCH] drm/amd/powerplay: warn on smu interface version mismatch Yuan, Xiaojie
[not found] ` <20190626062402.18551-1-xiaojie.yuan-5C7GfCeVMHo@public.gmane.org>
2019-06-26 6:34 ` Yuan, Xiaojie
[not found] ` <BN8PR12MB36021167B2EBF7F48CE93CCA89E20-h6+T2+wrnx1lGJ5xs4l7kQdYzm3356FpvxpqHgZTriW3zl9H0oFU5g@public.gmane.org>
2019-06-26 6:37 ` Kevin Wang
2019-06-27 1:51 ` Quan, Evan
[not found] ` <MN2PR12MB3344BE7B57FC531F96DEDB44E4FD0-rweVpJHSKToDMgCC8P//OwdYzm3356FpvxpqHgZTriW3zl9H0oFU5g@public.gmane.org>
2019-06-27 8:09 ` Yuan, Xiaojie
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.