]> git.ipfire.org Git - thirdparty/kernel/stable.git/commitdiff
Revert "usb: typec: ucsi: Enable debugfs for message_out data structure"
authorJohan Hovold <johan@kernel.org>
Mon, 22 Dec 2025 15:22:02 +0000 (16:22 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 23 Dec 2025 14:59:01 +0000 (15:59 +0100)
This reverts commit 775fae520e6ae62c393a8daf42dc534f09692f3f.

The new buffer management code that this relies on is broken so revert
for now.

It also looks like the error handling needs some more thought as the
message out size is not reset on errors.

Signed-off-by: Johan Hovold <johan@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Link: https://patch.msgid.link/20251222152204.2846-3-johan@kernel.org
drivers/usb/typec/ucsi/debugfs.c

index 90d11b79d2c074fbd228907a30259625ad133bba..924f9302755344306c6272b85fdc357a71bb390e 100644 (file)
@@ -110,30 +110,6 @@ static int ucsi_vbus_volt_show(struct seq_file *m, void *v)
 }
 DEFINE_SHOW_ATTRIBUTE(ucsi_vbus_volt);
 
-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);
-
-       ucsi->message_out_size = min(count / 2, UCSI_MAX_MESSAGE_OUT_LENGTH);
-       ret = hex2bin(ucsi->message_out, buf, ucsi->message_out_size);
-       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);
@@ -146,8 +122,6 @@ void ucsi_debugfs_register(struct ucsi *ucsi)
        debugfs_create_file("peak_current", 0400, ucsi->debugfs->dentry, ucsi, &ucsi_peak_curr_fops);
        debugfs_create_file("avg_current", 0400, ucsi->debugfs->dentry, ucsi, &ucsi_avg_curr_fops);
        debugfs_create_file("vbus_voltage", 0400, ucsi->debugfs->dentry, ucsi, &ucsi_vbus_volt_fops);
-       debugfs_create_file("message_out", 0200, ucsi->debugfs->dentry, ucsi,
-                           &ucsi_message_out_fops);
 }
 
 void ucsi_debugfs_unregister(struct ucsi *ucsi)