All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH][next] drm/amd/amdgpu: Fix spelling mistake "versoin" -> "version"
@ 2024-11-13  9:53 Colin Ian King
  2024-11-13 10:13 ` Dan Carpenter
  0 siblings, 1 reply; 3+ messages in thread
From: Colin Ian King @ 2024-11-13  9:53 UTC (permalink / raw)
  To: Alex Deucher, Christian König, Xinhui Pan, David Airlie,
	Simona Vetter, amd-gfx, dri-devel
  Cc: kernel-janitors, linux-kernel

There is a spelling mistake in a dev_err message. Fix it.

Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
---
 drivers/gpu/drm/amd/amdgpu/mes_v11_0.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/mes_v11_0.c b/drivers/gpu/drm/amd/amdgpu/mes_v11_0.c
index 9c905b9e9376..feecd099cf86 100644
--- a/drivers/gpu/drm/amd/amdgpu/mes_v11_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/mes_v11_0.c
@@ -646,7 +646,7 @@ static int mes_v11_0_misc_op(struct amdgpu_mes *mes,
 		break;
 	case MES_MISC_OP_CHANGE_CONFIG:
 		if ((mes->adev->mes.sched_version & AMDGPU_MES_VERSION_MASK) < 0x63) {
-			dev_err(mes->adev->dev, "MES FW versoin must be larger than 0x63 to support limit single process feature.\n");
+			dev_err(mes->adev->dev, "MES FW version must be larger than 0x63 to support limit single process feature.\n");
 			return -EINVAL;
 		}
 		misc_pkt.opcode = MESAPI_MISC__CHANGE_CONFIG;
-- 
2.39.5


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH][next] drm/amd/amdgpu: Fix spelling mistake "versoin" -> "version"
  2024-11-13  9:53 [PATCH][next] drm/amd/amdgpu: Fix spelling mistake "versoin" -> "version" Colin Ian King
@ 2024-11-13 10:13 ` Dan Carpenter
  2024-11-13 10:22   ` Colin King (gmail)
  0 siblings, 1 reply; 3+ messages in thread
From: Dan Carpenter @ 2024-11-13 10:13 UTC (permalink / raw)
  To: Colin Ian King
  Cc: Alex Deucher, Christian König, Xinhui Pan, David Airlie,
	Simona Vetter, amd-gfx, dri-devel, kernel-janitors, linux-kernel

On Wed, Nov 13, 2024 at 09:53:49AM +0000, Colin Ian King wrote:
> There is a spelling mistake in a dev_err message. Fix it.
> 
> Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
> ---
>  drivers/gpu/drm/amd/amdgpu/mes_v11_0.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/amd/amdgpu/mes_v11_0.c b/drivers/gpu/drm/amd/amdgpu/mes_v11_0.c
> index 9c905b9e9376..feecd099cf86 100644
> --- a/drivers/gpu/drm/amd/amdgpu/mes_v11_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/mes_v11_0.c
> @@ -646,7 +646,7 @@ static int mes_v11_0_misc_op(struct amdgpu_mes *mes,
>  		break;
>  	case MES_MISC_OP_CHANGE_CONFIG:
>  		if ((mes->adev->mes.sched_version & AMDGPU_MES_VERSION_MASK) < 0x63) {
> -			dev_err(mes->adev->dev, "MES FW versoin must be larger than 0x63 to support limit single process feature.\n");
> +			dev_err(mes->adev->dev, "MES FW version must be larger than 0x63 to support limit single process feature.\n");

What does "to support limit single process feature" mean?

regards,
dan carpenter


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH][next] drm/amd/amdgpu: Fix spelling mistake "versoin" -> "version"
  2024-11-13 10:13 ` Dan Carpenter
@ 2024-11-13 10:22   ` Colin King (gmail)
  0 siblings, 0 replies; 3+ messages in thread
From: Colin King (gmail) @ 2024-11-13 10:22 UTC (permalink / raw)
  To: Dan Carpenter
  Cc: Alex Deucher, Christian König, Xinhui Pan, David Airlie,
	Simona Vetter, amd-gfx, dri-devel, kernel-janitors, linux-kernel

On 13/11/2024 10:13, Dan Carpenter wrote:
> On Wed, Nov 13, 2024 at 09:53:49AM +0000, Colin Ian King wrote:
>> There is a spelling mistake in a dev_err message. Fix it.
>>
>> Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
>> ---
>>   drivers/gpu/drm/amd/amdgpu/mes_v11_0.c | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/gpu/drm/amd/amdgpu/mes_v11_0.c b/drivers/gpu/drm/amd/amdgpu/mes_v11_0.c
>> index 9c905b9e9376..feecd099cf86 100644
>> --- a/drivers/gpu/drm/amd/amdgpu/mes_v11_0.c
>> +++ b/drivers/gpu/drm/amd/amdgpu/mes_v11_0.c
>> @@ -646,7 +646,7 @@ static int mes_v11_0_misc_op(struct amdgpu_mes *mes,
>>   		break;
>>   	case MES_MISC_OP_CHANGE_CONFIG:
>>   		if ((mes->adev->mes.sched_version & AMDGPU_MES_VERSION_MASK) < 0x63) {
>> -			dev_err(mes->adev->dev, "MES FW versoin must be larger than 0x63 to support limit single process feature.\n");
>> +			dev_err(mes->adev->dev, "MES FW version must be larger than 0x63 to support limit single process feature.\n");
> 
> What does "to support limit single process feature" mean?

Looks like I need an automated semantic checker as well as a spelling 
checker.

Colin

> 
> regards,
> dan carpenter
> 


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2024-11-13 14:04 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-11-13  9:53 [PATCH][next] drm/amd/amdgpu: Fix spelling mistake "versoin" -> "version" Colin Ian King
2024-11-13 10:13 ` Dan Carpenter
2024-11-13 10:22   ` Colin King (gmail)

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.