On Wed, 1 Aug 2012, Lan Tianyu wrote: > On 2012/7/31 22:39, Alan Stern wrote: > > On Tue, 31 Jul 2012, Lan Tianyu wrote: > > > >> How about checking RESET_MORPHS before doing reset_resume, set reset_resume > >> to 0 and do resume when RESET_MORPHS is set. > > > > No, that won't work. When we do a reset-resume it is because we _know_ > > that a regular resume will fail. > persist is default to be enabled. When user space set a device's > avoid_reset_quirk, they also should set persist to 0. Since these > devices can be reset and regular resume can work. Right? It doesn't really matter. Yes, regular resume can work. And reset-resume will fail, regardless of the persist setting. > > I think the best course is to leave things the way they are. Just add > > an explanation to persist.txt that the Persist mechanism is likely to > > fail if the avoid_reset_quirk attribute is set. > > If we leave things the way they are, do we still need previous patch > "Take attribute avoid_reset_quirk out of usb device's attribute group"? No, we don't. And it doesn't need to do anything special for hubs. Alan Stern -- 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