>> >> +enum usb_otg_events { >> >> + /* according to OTG Spec */ >> >> + USB_OTG_CONNECT, /* a_conn/b_conn */ >> >> + USB_OTG_DISCON, >> >> + USB_OTG_HOST_SUSP, /* bus request */ >> >> + USB_OTG_HOST_RESU, >> >> + USB_OTG_DEV_SUSP, /* bus suspend/resume event */ >> >> + USB_OTG_DEV_RESU, >> > >> > Why use strange contractions like that and not just call it SUSPEND/RESUME? >> >> It seems a little long if no contractions. Do you see if it is fine to keep them here? > >Do we somehow now have a limit on lengths of enumerated types? Please >spell it all out. Ok, will fix it in next version patch submission. Thanks Hao -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html