Re: [PATCH 3/3] usb: Change persist_enabled when attribute avoid_reset_quirk is modified

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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


[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux