* Bin Liu <b-liu@xxxxxx> [160513 12:58]: > On Wed, May 11, 2016 at 06:33:04PM -0700, Tony Lindgren wrote: > > --- a/drivers/usb/musb/omap2430.c > > +++ b/drivers/usb/musb/omap2430.c > > @@ -49,6 +49,9 @@ struct omap2430_glue { > > enum musb_vbus_id_status status; > > struct work_struct omap_musb_mailbox_work; > > struct device *control_otghs; > > + bool cable_connected; > > + bool enabled; > > + bool powered; > > This variable is only used within omap2430_set_power(), so it can be > local within that function to save a few bytes. Well it needs to be static as we keep things powered only if glue is enabled and cable is connected. I'd rather not add any more static stuff that's not specific to the glue instance.. Got any better ideas? BTW, we still have also at least _glue remaining in the 2430 glue. Regards, Tony -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html