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

Reply via email to