[PATCH fwctl] fwctl: fix repeated device word in log message

Shannon Nelson posted 1 patch 10 months ago
drivers/fwctl/main.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH fwctl] fwctl: fix repeated device word in log message
Posted by Shannon Nelson 10 months ago
Remove the repeated word "device" from a dev_warn() message.

Signed-off-by: Shannon Nelson <shannon.nelson@amd.com>
---
 drivers/fwctl/main.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/fwctl/main.c b/drivers/fwctl/main.c
index cb1ac9c40239..bc6378506296 100644
--- a/drivers/fwctl/main.c
+++ b/drivers/fwctl/main.c
@@ -105,7 +105,7 @@ static int fwctl_cmd_rpc(struct fwctl_ucmd *ucmd)
 		if (!test_and_set_bit(0, &fwctl_tainted)) {
 			dev_warn(
 				&fwctl->dev,
-				"%s(%d): has requested full access to the physical device device",
+				"%s(%d): has requested full access to the physical device",
 				current->comm, task_pid_nr(current));
 			add_taint(TAINT_FWCTL, LOCKDEP_STILL_OK);
 		}
-- 
2.17.1
Re: [PATCH fwctl] fwctl: fix repeated device word in log message
Posted by Jason Gunthorpe 10 months ago
On Tue, Apr 08, 2025 at 03:33:00PM -0700, Shannon Nelson wrote:
> Remove the repeated word "device" from a dev_warn() message.
> 
> Signed-off-by: Shannon Nelson <shannon.nelson@amd.com>
> ---
>  drivers/fwctl/main.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Applied thanks

Jason
Re: [PATCH fwctl] fwctl: fix repeated device word in log message
Posted by Dave Jiang 10 months ago

On 4/8/25 3:33 PM, Shannon Nelson wrote:
> Remove the repeated word "device" from a dev_warn() message.
> 
> Signed-off-by: Shannon Nelson <shannon.nelson@amd.com>
Reviewed-by: Dave Jiang <dave.jiang@intel.com>

> ---
>  drivers/fwctl/main.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/fwctl/main.c b/drivers/fwctl/main.c
> index cb1ac9c40239..bc6378506296 100644
> --- a/drivers/fwctl/main.c
> +++ b/drivers/fwctl/main.c
> @@ -105,7 +105,7 @@ static int fwctl_cmd_rpc(struct fwctl_ucmd *ucmd)
>  		if (!test_and_set_bit(0, &fwctl_tainted)) {
>  			dev_warn(
>  				&fwctl->dev,
> -				"%s(%d): has requested full access to the physical device device",
> +				"%s(%d): has requested full access to the physical device",
>  				current->comm, task_pid_nr(current));
>  			add_taint(TAINT_FWCTL, LOCKDEP_STILL_OK);
>  		}