platform/chrome: cros_typec_switch: Use PTR_ERR_OR_ZERO() to simplify
authorStephen Boyd <swboyd@chromium.org>
Tue, 30 Aug 2022 22:58:30 +0000 (15:58 -0700)
committerTzung-Bi Shih <tzungbi@kernel.org>
Thu, 1 Sep 2022 02:15:34 +0000 (10:15 +0800)
Use the standard error pointer macro to shorten the code and simplify.

Cc: Prashant Malani <pmalani@chromium.org>
Cc: Tzung-Bi Shih <tzungbi@kernel.org>
Signed-off-by: Stephen Boyd <swboyd@chromium.org>
Acked-by: Prashant Malani <pmalani@chromium.org>
Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
Link: https://lore.kernel.org/r/20220830225831.2362403-4-swboyd@chromium.org
drivers/platform/chrome/cros_typec_switch.c

index 3381d84..09ad0d2 100644 (file)
@@ -185,10 +185,8 @@ static int cros_typec_register_mode_switch(struct cros_typec_port *port,
        };
 
        port->mode_switch = typec_mux_register(port->sdata->dev, &mode_switch_desc);
-       if (IS_ERR(port->mode_switch))
-               return PTR_ERR(port->mode_switch);
 
-       return 0;
+       return PTR_ERR_OR_ZERO(port->mode_switch);
 }
 
 static int cros_typec_register_retimer(struct cros_typec_port *port, struct fwnode_handle *fwnode)
@@ -201,10 +199,8 @@ static int cros_typec_register_retimer(struct cros_typec_port *port, struct fwno
        };
 
        port->retimer = typec_retimer_register(port->sdata->dev, &retimer_desc);
-       if (IS_ERR(port->retimer))
-               return PTR_ERR(port->retimer);
 
-       return 0;
+       return PTR_ERR_OR_ZERO(port->retimer);
 }
 
 static int cros_typec_register_switches(struct cros_typec_switch_data *sdata)