[PATCH 0/3] usb: typec: ucsi: Add support for SET_PDOS command

[PATCH 0/3] usb: typec: ucsi: Add support for SET_PDOS command

[Pooja Katiyar]

This series implements support for UCSI SET_PDOS command. It provides
interface to send message out data structure and update source or sink
capabilities PDOs on a connector over debugfs interface.

Pooja Katiyar (3):
  usb: typec: ucsi: Add support for message out data structure
  usb: typec: ucsi: Enable debugfs for message_out data structure
  usb: typec: ucsi: Add support for SET_PDOS command

 drivers/usb/typec/ucsi/debugfs.c   | 32 ++++++++++++++++++++++++++++++
 drivers/usb/typec/ucsi/ucsi.h      | 17 ++++++++++++++++
 drivers/usb/typec/ucsi/ucsi_acpi.c | 16 +++++++++++++++
 3 files changed, 65 insertions(+)

-- 
2.34.1

[PATCH 1/3] usb: typec: ucsi: Add support for message out data structure

[Pooja Katiyar]

Add support for updating message out data structure for UCSI ACPI
interface for UCSI 2.1 and UCSI 3.0 commands such as Set PDOs and
LPM Firmware Update.

Additionally, introduce a wrapper function to access this operation
via debugfs.

Reviewed-by: Heikki Krogerus <heikki.krogerus@linux.intel.com>
Signed-off-by: Pooja Katiyar <pooja.katiyar@intel.com>
---
 drivers/usb/typec/ucsi/ucsi.h      | 10 ++++++++++
 drivers/usb/typec/ucsi/ucsi_acpi.c | 16 ++++++++++++++++
 2 files changed, 26 insertions(+)

diff --git a/drivers/usb/typec/ucsi/ucsi.h b/drivers/usb/typec/ucsi/ucsi.h
index 9c5278a0c5d4..14e3e362c54d 100644
--- a/drivers/usb/typec/ucsi/ucsi.h
+++ b/drivers/usb/typec/ucsi/ucsi.h
@@ -79,6 +79,7 @@ struct ucsi_operations {
 	int (*read_cci)(struct ucsi *ucsi, u32 *cci);
 	int (*poll_cci)(struct ucsi *ucsi, u32 *cci);
 	int (*read_message_in)(struct ucsi *ucsi, void *val, size_t val_len);
+	int (*write_message_out)(struct ucsi *ucsi, void *data, size_t data_len);
 	int (*sync_control)(struct ucsi *ucsi, u64 command, u32 *cci,
 			    void *data, size_t size);
 	int (*async_control)(struct ucsi *ucsi, u64 command);
@@ -532,6 +533,15 @@ struct ucsi_connector {
 int ucsi_send_command(struct ucsi *ucsi, u64 command,
 		      void *retval, size_t size);
 
+static inline int ucsi_send_message_out(struct ucsi *ucsi,
+					void *data, size_t size)
+{
+	if (!ucsi->ops->write_message_out)
+		return -EOPNOTSUPP;
+
+	return ucsi->ops->write_message_out(ucsi, data, size);
+}
+
 void ucsi_altmode_update_active(struct ucsi_connector *con);
 int ucsi_resume(struct ucsi *ucsi);
 
diff --git a/drivers/usb/typec/ucsi/ucsi_acpi.c b/drivers/usb/typec/ucsi/ucsi_acpi.c
index 6b92f296e985..01d85e079d08 100644
--- a/drivers/usb/typec/ucsi/ucsi_acpi.c
+++ b/drivers/usb/typec/ucsi/ucsi_acpi.c
@@ -86,6 +86,21 @@ static int ucsi_acpi_read_message_in(struct ucsi *ucsi, void *val, size_t val_le
 	return 0;
 }
 
+static int ucsi_acpi_write_message_out(struct ucsi *ucsi, void *data, size_t data_len)
+{
+	struct ucsi_acpi *ua = ucsi_get_drvdata(ucsi);
+
+	if (!data || !data_len)
+		return -EINVAL;
+
+	if (ucsi->version <= UCSI_VERSION_1_2)
+		memcpy(ua->base + UCSI_MESSAGE_OUT, data, data_len);
+	else
+		memcpy(ua->base + UCSIv2_MESSAGE_OUT, data, data_len);
+
+	return ucsi_acpi_dsm(ua, UCSI_DSM_FUNC_WRITE);
+}
+
 static int ucsi_acpi_async_control(struct ucsi *ucsi, u64 command)
 {
 	struct ucsi_acpi *ua = ucsi_get_drvdata(ucsi);
@@ -101,6 +116,7 @@ static const struct ucsi_operations ucsi_acpi_ops = {
 	.read_cci = ucsi_acpi_read_cci,
 	.poll_cci = ucsi_acpi_poll_cci,
 	.read_message_in = ucsi_acpi_read_message_in,
+	.write_message_out = ucsi_acpi_write_message_out,
 	.sync_control = ucsi_sync_control_common,
 	.async_control = ucsi_acpi_async_control
 };
-- 
2.34.1

[PATCH 2/3] usb: typec: ucsi: Enable debugfs for message_out data structure

[Pooja Katiyar]

Add debugfs entry for writing message_out data structure to handle
UCSI 2.1 and 3.0 commands through debugfs interface.

Users writing to the message_out debugfs file should ensure the input
data adheres to the following format:
1. Input must be a non-empty valid hexadecimal string.
2. Input length of hexadecimal string must not exceed 256 bytes of
   length to be in alignment with the message out data structure size
   as per the UCSI specification v2.1.
3. If the input string length is odd, then user needs to prepend a
   '0' to the first character for proper hex conversion.

Below are examples of valid hex strings. Note that these values are
just examples. The exact values depend on specific command use case.

	#echo 1A2B3C4D > message_out
	#echo 01234567 > message_out

Reviewed-by: Heikki Krogerus <heikki.krogerus@linux.intel.com>
Signed-off-by: Pooja Katiyar <pooja.katiyar@intel.com>
---
 drivers/usb/typec/ucsi/debugfs.c | 26 ++++++++++++++++++++++++++
 drivers/usb/typec/ucsi/ucsi.h    |  3 +++
 2 files changed, 29 insertions(+)

diff --git a/drivers/usb/typec/ucsi/debugfs.c b/drivers/usb/typec/ucsi/debugfs.c
index eae2b18a2d8a..004587ce3427 100644
--- a/drivers/usb/typec/ucsi/debugfs.c
+++ b/drivers/usb/typec/ucsi/debugfs.c
@@ -76,6 +76,30 @@ static int ucsi_resp_show(struct seq_file *s, void *not_used)
 }
 DEFINE_SHOW_ATTRIBUTE(ucsi_resp);
 
+static ssize_t ucsi_message_out_write(struct file *file,
+				      const char __user *data, size_t count, loff_t *ppos)
+{
+	struct ucsi *ucsi = file->private_data;
+	int ret;
+
+	char *buf __free(kfree) = memdup_user_nul(data, count);
+	if (IS_ERR(buf))
+		return PTR_ERR(buf);
+
+	ret = hex2bin(ucsi->debugfs->message_out, buf,
+		      min(count / 2, sizeof(ucsi->debugfs->message_out)));
+	if (ret)
+		return ret;
+
+	return count;
+}
+
+static const struct file_operations ucsi_message_out_fops = {
+	.open = simple_open,
+	.write = ucsi_message_out_write,
+	.llseek = generic_file_llseek,
+};
+
 void ucsi_debugfs_register(struct ucsi *ucsi)
 {
 	ucsi->debugfs = kzalloc(sizeof(*ucsi->debugfs), GFP_KERNEL);
@@ -85,6 +109,8 @@ void ucsi_debugfs_register(struct ucsi *ucsi)
 	ucsi->debugfs->dentry = debugfs_create_dir(dev_name(ucsi->dev), ucsi_debugfs_root);
 	debugfs_create_file("command", 0200, ucsi->debugfs->dentry, ucsi, &ucsi_cmd_fops);
 	debugfs_create_file("response", 0400, ucsi->debugfs->dentry, ucsi, &ucsi_resp_fops);
+	debugfs_create_file("message_out", 0200, ucsi->debugfs->dentry, ucsi,
+			    &ucsi_message_out_fops);
 }
 
 void ucsi_debugfs_unregister(struct ucsi *ucsi)
diff --git a/drivers/usb/typec/ucsi/ucsi.h b/drivers/usb/typec/ucsi/ucsi.h
index 14e3e362c54d..e7453b055ff7 100644
--- a/drivers/usb/typec/ucsi/ucsi.h
+++ b/drivers/usb/typec/ucsi/ucsi.h
@@ -429,6 +429,8 @@ struct ucsi_bitfield {
 
 /* -------------------------------------------------------------------------- */
 
+#define MESSAGE_OUT_MAX_LEN 256
+
 struct ucsi_debugfs_entry {
 	u64 command;
 	struct ucsi_data {
@@ -436,6 +438,7 @@ struct ucsi_debugfs_entry {
 		u64 high;
 	} response;
 	u32 status;
+	u8 message_out[MESSAGE_OUT_MAX_LEN];
 	struct dentry *dentry;
 };
 
-- 
2.34.1

Re: [PATCH 2/3] usb: typec: ucsi: Enable debugfs for message_out data structure

[Andy Shevchenko]

On Thu, Jun 05, 2025 at 06:38:14PM -0700, Pooja Katiyar wrote:
> Add debugfs entry for writing message_out data structure to handle
> UCSI 2.1 and 3.0 commands through debugfs interface.
> 
> Users writing to the message_out debugfs file should ensure the input
> data adheres to the following format:
> 1. Input must be a non-empty valid hexadecimal string.
> 2. Input length of hexadecimal string must not exceed 256 bytes of
>    length to be in alignment with the message out data structure size
>    as per the UCSI specification v2.1.
> 3. If the input string length is odd, then user needs to prepend a
>    '0' to the first character for proper hex conversion.
> 
> Below are examples of valid hex strings. Note that these values are
> just examples. The exact values depend on specific command use case.
> 
> 	#echo 1A2B3C4D > message_out
> 	#echo 01234567 > message_out

Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>


-- 
With Best Regards,
Andy Shevchenko

[PATCH 3/3] usb: typec: ucsi: Add support for SET_PDOS command

[Pooja Katiyar]

Add support for UCSI SET_PDOS command as per UCSI specification v2.1 and
above to debugfs.

Reviewed-by: Heikki Krogerus <heikki.krogerus@linux.intel.com>
Signed-off-by: Pooja Katiyar <pooja.katiyar@intel.com>
---
 drivers/usb/typec/ucsi/debugfs.c | 6 ++++++
 drivers/usb/typec/ucsi/ucsi.h    | 4 ++++
 2 files changed, 10 insertions(+)

diff --git a/drivers/usb/typec/ucsi/debugfs.c b/drivers/usb/typec/ucsi/debugfs.c
index 004587ce3427..1c8003121d6e 100644
--- a/drivers/usb/typec/ucsi/debugfs.c
+++ b/drivers/usb/typec/ucsi/debugfs.c
@@ -36,6 +36,12 @@ static int ucsi_cmd(void *data, u64 val)
 	case UCSI_SET_NEW_CAM:
 		ret = ucsi_send_command(ucsi, val, NULL, 0);
 		break;
+	case UCSI_SET_PDOS:
+		ret = ucsi_send_message_out(ucsi, ucsi->debugfs->message_out,
+					    UCSI_COMMAND_DATA_LEN(val));
+		if (!ret)
+			ret = ucsi_send_command(ucsi, val, NULL, 0);
+		break;
 	case UCSI_GET_CAPABILITY:
 	case UCSI_GET_CONNECTOR_CAPABILITY:
 	case UCSI_GET_ALTERNATE_MODES:
diff --git a/drivers/usb/typec/ucsi/ucsi.h b/drivers/usb/typec/ucsi/ucsi.h
index e7453b055ff7..50341c40d80a 100644
--- a/drivers/usb/typec/ucsi/ucsi.h
+++ b/drivers/usb/typec/ucsi/ucsi.h
@@ -130,6 +130,7 @@ void ucsi_connector_change(struct ucsi *ucsi, u8 num);
 #define UCSI_GET_CAM_CS			0x18
 #define UCSI_SET_SINK_PATH			0x1c
 #define UCSI_GET_LPM_PPM_INFO			0x22
+#define UCSI_SET_PDOS				0x1d
 
 #define UCSI_CONNECTOR_NUMBER(_num_)		((u64)(_num_) << 16)
 #define UCSI_COMMAND(_cmd_)			((_cmd_) & 0xff)
@@ -204,6 +205,9 @@ void ucsi_connector_change(struct ucsi *ucsi, u8 num);
 #define   UCSI_GET_PD_MESSAGE_TYPE_IDENTITY	4
 #define   UCSI_GET_PD_MESSAGE_TYPE_REVISION	5
 
+/* Data length bits */
+#define UCSI_COMMAND_DATA_LEN(_cmd_)		(((_cmd_) >> 8) & GENMASK(7, 0))
+
 /* -------------------------------------------------------------------------- */
 
 /* Error information returned by PPM in response to GET_ERROR_STATUS command. */
-- 
2.34.1

Re: [PATCH 3/3] usb: typec: ucsi: Add support for SET_PDOS command

[Dmitry Baryshkov]

On Thu, Jun 05, 2025 at 06:38:15PM -0700, Pooja Katiyar wrote:
> Add support for UCSI SET_PDOS command as per UCSI specification v2.1 and
> above to debugfs.
> 
> Reviewed-by: Heikki Krogerus <heikki.krogerus@linux.intel.com>
> Signed-off-by: Pooja Katiyar <pooja.katiyar@intel.com>
> ---
>  drivers/usb/typec/ucsi/debugfs.c | 6 ++++++
>  drivers/usb/typec/ucsi/ucsi.h    | 4 ++++
>  2 files changed, 10 insertions(+)
> 
> diff --git a/drivers/usb/typec/ucsi/debugfs.c b/drivers/usb/typec/ucsi/debugfs.c
> index 004587ce3427..1c8003121d6e 100644
> --- a/drivers/usb/typec/ucsi/debugfs.c
> +++ b/drivers/usb/typec/ucsi/debugfs.c
> @@ -36,6 +36,12 @@ static int ucsi_cmd(void *data, u64 val)
>  	case UCSI_SET_NEW_CAM:
>  		ret = ucsi_send_command(ucsi, val, NULL, 0);
>  		break;
> +	case UCSI_SET_PDOS:
> +		ret = ucsi_send_message_out(ucsi, ucsi->debugfs->message_out,
> +					    UCSI_COMMAND_DATA_LEN(val));

I think this goes against current UCSI command support code. I'd suggest
you to:

- extend .sync_control to also include message_out fields
- return -EINVAL if the write_message_out() callback is not set
- make ucsi_send_command() accept message_out data.

> +		if (!ret)
> +			ret = ucsi_send_command(ucsi, val, NULL, 0);
> +		break;
>  	case UCSI_GET_CAPABILITY:
>  	case UCSI_GET_CONNECTOR_CAPABILITY:
>  	case UCSI_GET_ALTERNATE_MODES:

-- 
With best wishes
Dmitry

Re: [PATCH 3/3] usb: typec: ucsi: Add support for SET_PDOS command

[Heikki Krogerus]

On Fri, Jun 06, 2025 at 06:24:06AM +0300, Dmitry Baryshkov wrote:
> On Thu, Jun 05, 2025 at 06:38:15PM -0700, Pooja Katiyar wrote:
> > Add support for UCSI SET_PDOS command as per UCSI specification v2.1 and
> > above to debugfs.
> > 
> > Reviewed-by: Heikki Krogerus <heikki.krogerus@linux.intel.com>
> > Signed-off-by: Pooja Katiyar <pooja.katiyar@intel.com>
> > ---
> >  drivers/usb/typec/ucsi/debugfs.c | 6 ++++++
> >  drivers/usb/typec/ucsi/ucsi.h    | 4 ++++
> >  2 files changed, 10 insertions(+)
> > 
> > diff --git a/drivers/usb/typec/ucsi/debugfs.c b/drivers/usb/typec/ucsi/debugfs.c
> > index 004587ce3427..1c8003121d6e 100644
> > --- a/drivers/usb/typec/ucsi/debugfs.c
> > +++ b/drivers/usb/typec/ucsi/debugfs.c
> > @@ -36,6 +36,12 @@ static int ucsi_cmd(void *data, u64 val)
> >  	case UCSI_SET_NEW_CAM:
> >  		ret = ucsi_send_command(ucsi, val, NULL, 0);
> >  		break;
> > +	case UCSI_SET_PDOS:
> > +		ret = ucsi_send_message_out(ucsi, ucsi->debugfs->message_out,
> > +					    UCSI_COMMAND_DATA_LEN(val));
> 
> I think this goes against current UCSI command support code. I'd suggest
> you to:
> 
> - extend .sync_control to also include message_out fields
> - return -EINVAL if the write_message_out() callback is not set
> - make ucsi_send_command() accept message_out data.

This makes sense to me.

thanks,

-- 
heikki

[PATCH 0/3] usb: typec: ucsi: Add support for SET_PDOS command

[Pooja Katiyar]

This series implements support for UCSI SET PDOS command. It provides
interface to send message out data and update source or sink
capabilities PDOs on a connector over debugfs interface.

Pooja Katiyar (3):
  usb: typec: ucsi: Add support for message_out data structure
  usb: typec: ucsi: Enable debugfs for message_out data structure
  usb: typec: ucsi: Add support for SET_PDOS command

 drivers/usb/typec/ucsi/cros_ec_ucsi.c   |  6 ++-
 drivers/usb/typec/ucsi/debugfs.c        | 31 +++++++++++++++
 drivers/usb/typec/ucsi/ucsi.c           | 52 ++++++++++++++++++++-----
 drivers/usb/typec/ucsi/ucsi.h           | 20 +++++++++-
 drivers/usb/typec/ucsi/ucsi_acpi.c      | 22 ++++++++++-
 drivers/usb/typec/ucsi/ucsi_ccg.c       |  6 ++-
 drivers/usb/typec/ucsi/ucsi_yoga_c630.c |  6 ++-
 7 files changed, 123 insertions(+), 20 deletions(-)

-- 
2.43.0

Re: [PATCH 0/3] usb: typec: ucsi: Add support for SET_PDOS command

[Johan Hovold]

On Wed, Mar 18, 2026 at 03:25:28PM -0700, Pooja Katiyar wrote:
> This series implements support for UCSI SET PDOS command. It provides
> interface to send message out data and update source or sink
> capabilities PDOs on a connector over debugfs interface.
> 
> Pooja Katiyar (3):
>   usb: typec: ucsi: Add support for message_out data structure
>   usb: typec: ucsi: Enable debugfs for message_out data structure
>   usb: typec: ucsi: Add support for SET_PDOS command

This is really v6 after v5 was reverted:

	https://lore.kernel.org/all/cover.1761773881.git.pooja.katiyar@intel.com/
	https://lore.kernel.org/all/20251222152204.2846-1-johan@kernel.org/

What changed in v6?

Johan

RE: [PATCH 0/3] usb: typec: ucsi: Add support for SET_PDOS command

[Katiyar, Pooja]

Hi Johan,

On Thu, Mar 19, 2026 at 12:40:00AM +0000, Johan Hovold wrote:
> On Wed, Mar 18, 2026 at 03:25:28PM -0700, Pooja Katiyar wrote:
> > This series implements support for UCSI SET PDOS command. It provides
> > interface to send message out data and update source or sink
> > capabilities PDOs on a connector over debugfs interface.
> >
> > Pooja Katiyar (3):
> >   usb: typec: ucsi: Add support for message_out data structure
> >   usb: typec: ucsi: Enable debugfs for message_out data structure
> >   usb: typec: ucsi: Add support for SET_PDOS command
> 
> This is really v6 after v5 was reverted:
> 
> 	https://lore.kernel.org/all/cover.1761773881.git.pooja.katiyar@intel.co
> m/
> 	https://lore.kernel.org/all/20251222152204.2846-1-johan@kernel.org/
> 
> What changed in v6?

Changes in this series compared to v5:

- Removed message_in and message_out related fields from global ucsi data
  structure and replaced with function parameter approach.
- Added message_out parameters to existing function parameters, keeping the
  flow similar to message_in data handling.
- Modified ucsi_sync_control_common() signature to accept msg_out and
  msg_out_size.
- Added write_message_out() to ucsi_operations struct.
- Added ucsi_write_message_out_command() function to handle commands
  which need to send message_out data to PPM.
- Updated platform interfaces and other function callers with new function
  signatures.

This eliminates the concurrency issues that earlier version introduced due
to shared data structure between multiple threads.

Thanks,
Pooja

Re: [PATCH 0/3] usb: typec: ucsi: Add support for SET_PDOS command

[Greg KH]

On Wed, Mar 18, 2026 at 03:25:28PM -0700, Pooja Katiyar wrote:
> This series implements support for UCSI SET PDOS command. It provides
> interface to send message out data and update source or sink
> capabilities PDOs on a connector over debugfs interface.
> 
> Pooja Katiyar (3):
>   usb: typec: ucsi: Add support for message_out data structure
>   usb: typec: ucsi: Enable debugfs for message_out data structure
>   usb: typec: ucsi: Add support for SET_PDOS command
> 
>  drivers/usb/typec/ucsi/cros_ec_ucsi.c   |  6 ++-
>  drivers/usb/typec/ucsi/debugfs.c        | 31 +++++++++++++++
>  drivers/usb/typec/ucsi/ucsi.c           | 52 ++++++++++++++++++++-----
>  drivers/usb/typec/ucsi/ucsi.h           | 20 +++++++++-
>  drivers/usb/typec/ucsi/ucsi_acpi.c      | 22 ++++++++++-
>  drivers/usb/typec/ucsi/ucsi_ccg.c       |  6 ++-
>  drivers/usb/typec/ucsi/ucsi_yoga_c630.c |  6 ++-
>  7 files changed, 123 insertions(+), 20 deletions(-)
> 
> -- 
> 2.43.0
> 
> 

Hi,

This is the friendly patch-bot of Greg Kroah-Hartman.  You have sent him
a patch that has triggered this response.  He used to manually respond
to these common problems, but in order to save his sanity (he kept
writing the same thing over and over, yet to different people), I was
created.  Hopefully you will not take offence and will fix the problem
in your patch and resubmit it so that it can be accepted into the Linux
kernel tree.

You are receiving this message because of the following common error(s)
as indicated below:

- This looks like a new version of a previously submitted patch, but you
  did not list below the --- line any changes from the previous version.
  Please read the section entitled "The canonical patch format" in the
  kernel file, Documentation/process/submitting-patches.rst for what
  needs to be done here to properly describe this.

If you wish to discuss this problem further, or you have questions about
how to resolve this issue, please feel free to respond to this email and
Greg will reply once he has dug out from the pending patches received
from other developers.

thanks,

greg k-h's patch email bot