> >On 12/2/2010 7:36 PM, Hao Wu wrote: >> +static const char *state_string(enum usb_otg_state state) >> +{ >> + switch (state) { >> + case OTG_STATE_A_IDLE: >> + return "a_idle"; >> + case OTG_STATE_A_WAIT_VRISE: >> + return "a_wait_vrise"; >> + case OTG_STATE_A_WAIT_BCON: >> + return "a_wait_bcon"; >> + case OTG_STATE_A_HOST: >> + return "a_host"; >> + case OTG_STATE_A_SUSPEND: >> + return "a_suspend"; >> + case OTG_STATE_A_PERIPHERAL: >> + return "a_peripheral"; >> + case OTG_STATE_A_WAIT_VFALL: >> + return "a_wait_vfall"; >> + case OTG_STATE_A_VBUS_ERR: >> + return "a_vbus_err"; >> + case OTG_STATE_B_IDLE: >> + return "b_idle"; >> + case OTG_STATE_B_PERIPHERAL: >> + return "b_peripheral"; >> + case OTG_STATE_B_WAIT_ACON: >> + return "b_wait_acon"; >> + case OTG_STATE_B_HOST: >> + return "b_host"; >> + default: >> + return "UNDEFINED"; >> + } >> +} >If you can move this function to drivers/usb/otg/otg.c, all OTG drivers >can use this to >print the state name. > Hm.. no problem, I have seen same/similar function defined in other otg drivers. Then we can share it after moving this function into a common place. 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