Subject: Re: linux-next: build warning after merge of the hid tree

From: Stephen Rothwell <[email-masked]>
Date: Wed, 5 Apr 2023 11:48:07 +1000

Hi Bastien,

On Tue, 04 Apr 2023 11:18:35 +0200 Bastien Nocera <[email-masked]> wrote:
>
> Before I send patches, does this look correct to you?
> 
> diff --git a/drivers/usb/core/message.c b/drivers/usb/core/message.c
> index 7930dca84616..7c7f88ca4f62 100644
> --- a/drivers/usb/core/message.c
> +++ b/drivers/usb/core/message.c
> @@ -1926,7 +1926,7 @@ static void __usb_wireless_status_intf(struct work_struct *ws)
> 1926⋮1926│ 
> 1927⋮1927│ /**
> 1928⋮1928│  * usb_set_wireless_status - sets the wireless_status struct member
> 1929⋮    │- * @dev: the device to modify
>     ⋮1929│+ * @iface: the interface to modify
> 1930⋮1930│  * @status: the new wireless status
> 1931⋮1931│  *
> 1932⋮1932│  * Set the wireless_status struct member to the new value, and emit

That looks good.

> diff --git a/include/linux/usb.h b/include/linux/usb.h
> index a48eeec62a66..3d523ebf0c09 100644
> --- a/include/linux/usb.h
> +++ b/include/linux/usb.h
> @@ -261,7 +261,8 @@ struct usb_interface {
>  261⋮ 261│      unsigned needs_binding:1;       /* needs delayed unbind/rebind */
>  262⋮ 262│      unsigned resetting_device:1;    /* true: bandwidth alloc after reset */
>  263⋮ 263│      unsigned authorized:1;          /* used for interface authorization */
>  264⋮    │-     enum usb_wireless_status wireless_status;
>     ⋮ 264│+     enum usb_wireless_status wireless_status; /* whether the emitter is
>     ⋮ 265│+                                                * connected to this receiver */
>  265⋮ 266│      struct work_struct wireless_status_work;
>  266⋮ 267│ 
>  267⋮ 268│      struct device dev;              /* interface specific device info */
> 

This is not right, though.  The warning was about
"wireless_status_work" not "wireless_status", so I think you need a
description of wireless_status_work in the block comment above the
struct definition.

-- 
Cheers,
Stephen Rothwell