usb: typec: Return void in typec_partner_set_pd_revision
authorKyle Tso <kyletso@google.com>
Tue, 2 Feb 2021 09:55:12 +0000 (17:55 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 2 Feb 2021 18:42:30 +0000 (19:42 +0100)
typec_partner_set_pd_revision doesn't need any return value.

Fixes: 29b01295a829 ("usb: typec: Add typec_partner_set_pd_revision")
Reviewed-by: Heikki Krogerus <heikki.krogerus@linux.intel.com>
Reviewed-by: Benson Leung <bleung@chromium.org>
Signed-off-by: Kyle Tso <kyletso@google.com>
Link: https://lore.kernel.org/r/20210202095512.761214-1-kyletso@google.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/usb/typec/class.c
include/linux/usb/typec.h

index b6ceab3..a7d1bc8 100644 (file)
@@ -755,15 +755,11 @@ EXPORT_SYMBOL_GPL(typec_partner_set_identity);
  *
  * This routine is used to report that the PD revision of the port partner has
  * become available.
- *
- * Returns 0 on success or negative error number on failure.
  */
-int typec_partner_set_pd_revision(struct typec_partner *partner, u16 pd_revision)
+void typec_partner_set_pd_revision(struct typec_partner *partner, u16 pd_revision)
 {
-       int ret;
-
        if (partner->pd_revision == pd_revision)
-               return 0;
+               return;
 
        partner->pd_revision = pd_revision;
        sysfs_notify(&partner->dev.kobj, NULL, "usb_power_delivery_revision");
@@ -773,8 +769,6 @@ int typec_partner_set_pd_revision(struct typec_partner *partner, u16 pd_revision
                             "supports_usb_power_delivery");
        }
        kobject_uevent(&partner->dev.kobj, KOBJ_CHANGE);
-
-       return 0;
 }
 EXPORT_SYMBOL_GPL(typec_partner_set_pd_revision);
 
index 4946eca..a94df82 100644 (file)
@@ -126,7 +126,7 @@ struct typec_altmode_desc {
        enum typec_port_data    roles;
 };
 
-int typec_partner_set_pd_revision(struct typec_partner *partner, u16 pd_revision);
+void typec_partner_set_pd_revision(struct typec_partner *partner, u16 pd_revision);
 int typec_partner_set_num_altmodes(struct typec_partner *partner, int num_altmodes);
 struct typec_altmode
 *typec_partner_register_altmode(struct typec_partner *partner,