* [PATCH] usb: typec: Document priority and mode_selection fields in struct typec_altmode
@ 2026-03-11 16:33 Aldo Conte
2026-03-12 9:12 ` Heikki Krogerus
0 siblings, 1 reply; 2+ messages in thread
From: Aldo Conte @ 2026-03-11 16:33 UTC (permalink / raw)
To: heikki.krogerus, gregkh; +Cc: linux-usb, linux-kernel
The fields 'priority' and 'mode_selection' in struct typec_altmode are
missing from the kernel-doc comment, which results in warnings when
building the documentation with 'make htmldocs'.
WARNING: ./include/linux/usb/typec_altmode.h:44 struct member 'priority' not described in 'typec_altmode'
WARNING: ./include/linux/usb/typec_altmode.h:44 struct member 'mode_selection' not described in 'typec_altmode'
Document both fields to keep the kernel-doc comment aligned with the
structure definition.
Signed-off-by: Aldo Conte <aldocontelk@gmail.com>
---
include/linux/usb/typec_altmode.h | 3 +++
1 file changed, 3 insertions(+)
diff --git a/include/linux/usb/typec_altmode.h b/include/linux/usb/typec_altmode.h
index 0513d333b797..b90cc5cfff8d 100644
--- a/include/linux/usb/typec_altmode.h
+++ b/include/linux/usb/typec_altmode.h
@@ -26,6 +26,9 @@ struct typec_altmode_ops;
* @mode: Index of the Mode
* @vdo: VDO returned by Discover Modes USB PD command
* @active: Tells has the mode been entered or not
+ * @priority: Priority used by the automatic alternate mode selection process
+ * @mode_selection: Whether entry to this alternate mode is managed by the
+ * automatic alternate mode selection process or by the specific driver
* @desc: Optional human readable description of the mode
* @ops: Operations vector from the driver
* @cable_ops: Cable operations vector from the driver.
--
2.53.0
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] usb: typec: Document priority and mode_selection fields in struct typec_altmode
2026-03-11 16:33 [PATCH] usb: typec: Document priority and mode_selection fields in struct typec_altmode Aldo Conte
@ 2026-03-12 9:12 ` Heikki Krogerus
0 siblings, 0 replies; 2+ messages in thread
From: Heikki Krogerus @ 2026-03-12 9:12 UTC (permalink / raw)
To: Aldo Conte; +Cc: gregkh, linux-usb, linux-kernel
Wed, Mar 11, 2026 at 05:33:20PM +0100, Aldo Conte kirjoitti:
> The fields 'priority' and 'mode_selection' in struct typec_altmode are
> missing from the kernel-doc comment, which results in warnings when
> building the documentation with 'make htmldocs'.
>
> WARNING: ./include/linux/usb/typec_altmode.h:44 struct member 'priority' not described in 'typec_altmode'
> WARNING: ./include/linux/usb/typec_altmode.h:44 struct member 'mode_selection' not described in 'typec_altmode'
>
> Document both fields to keep the kernel-doc comment aligned with the
> structure definition.
>
> Signed-off-by: Aldo Conte <aldocontelk@gmail.com>
Reviewed-by: Heikki Krogerus <heikki.krogerus@linux.intel.com>
> ---
> include/linux/usb/typec_altmode.h | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/include/linux/usb/typec_altmode.h b/include/linux/usb/typec_altmode.h
> index 0513d333b797..b90cc5cfff8d 100644
> --- a/include/linux/usb/typec_altmode.h
> +++ b/include/linux/usb/typec_altmode.h
> @@ -26,6 +26,9 @@ struct typec_altmode_ops;
> * @mode: Index of the Mode
> * @vdo: VDO returned by Discover Modes USB PD command
> * @active: Tells has the mode been entered or not
> + * @priority: Priority used by the automatic alternate mode selection process
> + * @mode_selection: Whether entry to this alternate mode is managed by the
> + * automatic alternate mode selection process or by the specific driver
> * @desc: Optional human readable description of the mode
> * @ops: Operations vector from the driver
> * @cable_ops: Cable operations vector from the driver.
> --
> 2.53.0
--
heikki
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2026-03-12 9:13 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-03-11 16:33 [PATCH] usb: typec: Document priority and mode_selection fields in struct typec_altmode Aldo Conte
2026-03-12 9:12 ` Heikki Krogerus
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox