Hi Heikki, > From: Heikki Krogerus, Sent: Monday, May 28, 2018 10:05 PM > > On Fri, May 25, 2018 at 04:12:56PM +0900, Yoshihiro Shimoda wrote: > > @@ -2573,6 +2635,12 @@ static void renesas_usb3_init_ram(struct renesas_usb3 *usb3, struct device *dev, > > EXTCON_NONE, > > }; > > > > +static struct usb_role_switch_desc renesas_usb3_role_switch_desc = { > > You can constify that. Thank you for the review! I'll modify this in v6 patch. Best regards, Yoshihiro Shimoda > > + .set = renesas_usb3_role_switch_set, > > + .get = renesas_usb3_role_switch_get, > > + .allow_userspace_control = true, > > +}; > > + > > static int renesas_usb3_probe(struct platform_device *pdev) > > { > > struct renesas_usb3 *usb3; > > Thanks, > > -- > heikki -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html