linux-kernel - Re: [RESEND PATCH v11 02/11] usb: typec: Export typec bus and typec altmode device type

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <aTGPpInIEzFhftf4@kuha>
Date: Thu, 4 Dec 2025 15:41:56 +0200
From: Heikki Krogerus <heikki.krogerus@...ux.intel.com>
To: Chaoyi Chen <kernel@...kyi.com>
Cc: Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
	Dmitry Baryshkov <dmitry.baryshkov@....qualcomm.com>,
	Peter Chen <hzpeterchen@...il.com>,
	Luca Ceresoli <luca.ceresoli@...tlin.com>,
	Rob Herring <robh@...nel.org>,
	Krzysztof Kozlowski <krzk+dt@...nel.org>,
	Conor Dooley <conor+dt@...nel.org>, Vinod Koul <vkoul@...nel.org>,
	Kishon Vijay Abraham I <kishon@...nel.org>,
	Heiko Stuebner <heiko@...ech.de>, Sandy Huang <hjc@...k-chips.com>,
	Andy Yan <andy.yan@...k-chips.com>,
	Yubing Zhang <yubing.zhang@...k-chips.com>,
	Frank Wang <frank.wang@...k-chips.com>,
	Andrzej Hajda <andrzej.hajda@...el.com>,
	Neil Armstrong <neil.armstrong@...aro.org>,
	Robert Foss <rfoss@...nel.org>,
	Laurent Pinchart <Laurent.pinchart@...asonboard.com>,
	Jonas Karlman <jonas@...boo.se>,
	Jernej Skrabec <jernej.skrabec@...il.com>,
	Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
	Maxime Ripard <mripard@...nel.org>,
	Thomas Zimmermann <tzimmermann@...e.de>,
	David Airlie <airlied@...il.com>, Simona Vetter <simona@...ll.ch>,
	Amit Sunil Dhamne <amitsd@...gle.com>,
	Chaoyi Chen <chaoyi.chen@...k-chips.com>,
	Dragan Simic <dsimic@...jaro.org>, Johan Jonker <jbx6244@...il.com>,
	Diederik de Haas <didi.debian@...ow.org>,
	Peter Robinson <pbrobinson@...il.com>, linux-usb@...r.kernel.org,
	devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
	linux-phy@...ts.infradead.org, linux-arm-kernel@...ts.infradead.org,
	linux-rockchip@...ts.infradead.org, dri-devel@...ts.freedesktop.org
Subject: Re: [RESEND PATCH v11 02/11] usb: typec: Export typec bus and typec
 altmode device type

Thu, Dec 04, 2025 at 02:31:00PM +0800, Chaoyi Chen kirjoitti:
> From: Chaoyi Chen <chaoyi.chen@...k-chips.com>
> 
> The DRM may want to register a notifier on the typec bus to know when
> a typec altmode device is added, and distinguish between different
> types of typec altmode device.
> 
> Export these things.
> 
> Signed-off-by: Chaoyi Chen <chaoyi.chen@...k-chips.com>

Reviewed-by: Heikki Krogerus <heikki.krogerus@...ux.intel.com>

> ---
> 
> Changes in v11:
> - Also export typec bus.
> - Add more detailed comments.
> 
> (no changes since v9)
> 
>  drivers/usb/typec/bus.c           |  1 +
>  drivers/usb/typec/bus.h           | 10 ----------
>  drivers/usb/typec/class.c         |  3 +++
>  include/linux/usb/typec.h         |  3 +++
>  include/linux/usb/typec_altmode.h |  8 ++++++++
>  5 files changed, 15 insertions(+), 10 deletions(-)
> 
> diff --git a/drivers/usb/typec/bus.c b/drivers/usb/typec/bus.c
> index a33da7f458a5..e84b134a3381 100644
> --- a/drivers/usb/typec/bus.c
> +++ b/drivers/usb/typec/bus.c
> @@ -569,3 +569,4 @@ const struct bus_type typec_bus = {
>  	.probe = typec_probe,
>  	.remove = typec_remove,
>  };
> +EXPORT_SYMBOL_GPL(typec_bus);
> diff --git a/drivers/usb/typec/bus.h b/drivers/usb/typec/bus.h
> index b58e131450d1..7df5deb1dd3a 100644
> --- a/drivers/usb/typec/bus.h
> +++ b/drivers/usb/typec/bus.h
> @@ -5,7 +5,6 @@
>  
>  #include <linux/usb/typec_altmode.h>
>  
> -struct bus_type;
>  struct typec_mux;
>  struct typec_retimer;
>  
> @@ -28,13 +27,4 @@ struct altmode {
>  
>  #define to_altmode(d) container_of(d, struct altmode, adev)
>  
> -extern const struct bus_type typec_bus;
> -extern const struct device_type typec_port_altmode_dev_type;
> -extern const struct device_type typec_plug_altmode_dev_type;
> -extern const struct device_type typec_partner_altmode_dev_type;
> -
> -#define is_typec_port_altmode(dev) ((dev)->type == &typec_port_altmode_dev_type)
> -#define is_typec_plug_altmode(dev) ((dev)->type == &typec_plug_altmode_dev_type)
> -#define is_typec_partner_altmode(dev) ((dev)->type == &typec_partner_altmode_dev_type)
> -
>  #endif /* __USB_TYPEC_ALTMODE_H__ */
> diff --git a/drivers/usb/typec/class.c b/drivers/usb/typec/class.c
> index d6b88317f8a4..c4ff4310ff58 100644
> --- a/drivers/usb/typec/class.c
> +++ b/drivers/usb/typec/class.c
> @@ -537,18 +537,21 @@ const struct device_type typec_port_altmode_dev_type = {
>  	.groups = typec_altmode_groups,
>  	.release = typec_altmode_release,
>  };
> +EXPORT_SYMBOL_GPL(typec_port_altmode_dev_type);
>  
>  const struct device_type typec_plug_altmode_dev_type = {
>  	.name = "typec_plug_alternate_mode",
>  	.groups = typec_altmode_groups,
>  	.release = typec_altmode_release,
>  };
> +EXPORT_SYMBOL_GPL(typec_plug_altmode_dev_type);
>  
>  const struct device_type typec_partner_altmode_dev_type = {
>  	.name = "typec_partner_alternate_mode",
>  	.groups = typec_altmode_groups,
>  	.release = typec_altmode_release,
>  };
> +EXPORT_SYMBOL_GPL(typec_partner_altmode_dev_type);
>  
>  static struct typec_altmode *
>  typec_register_altmode(struct device *parent,
> diff --git a/include/linux/usb/typec.h b/include/linux/usb/typec.h
> index 309251572e2e..c6fd46902fce 100644
> --- a/include/linux/usb/typec.h
> +++ b/include/linux/usb/typec.h
> @@ -20,12 +20,15 @@ struct typec_port;
>  struct typec_altmode_ops;
>  struct typec_cable_ops;
>  
> +struct bus_type;
>  struct fwnode_handle;
>  struct device;
>  
>  struct usb_power_delivery;
>  struct usb_power_delivery_desc;
>  
> +extern const struct bus_type typec_bus;
> +
>  enum typec_port_type {
>  	TYPEC_PORT_SRC,
>  	TYPEC_PORT_SNK,
> diff --git a/include/linux/usb/typec_altmode.h b/include/linux/usb/typec_altmode.h
> index f7db3bd4c90e..9197a4637a93 100644
> --- a/include/linux/usb/typec_altmode.h
> +++ b/include/linux/usb/typec_altmode.h
> @@ -9,6 +9,14 @@
>  
>  #define MODE_DISCOVERY_MAX	6
>  
> +extern const struct device_type typec_port_altmode_dev_type;
> +extern const struct device_type typec_plug_altmode_dev_type;
> +extern const struct device_type typec_partner_altmode_dev_type;
> +
> +#define is_typec_port_altmode(dev) ((dev)->type == &typec_port_altmode_dev_type)
> +#define is_typec_plug_altmode(dev) ((dev)->type == &typec_plug_altmode_dev_type)
> +#define is_typec_partner_altmode(dev) ((dev)->type == &typec_partner_altmode_dev_type)
> +
>  struct typec_altmode_ops;
>  
>  /**
> -- 
> 2.51.1

-- 
heikki