Cecilia Hu ??:
> Sebastien Roy wrote:
>> On Fri, 2008-12-05 at 12:51 +0800, Cecilia Hu wrote:
>>> The architecture and interface of it are clear enough to be a self
>>> review, so the case is marked as "closed approved automatic".
>>
>> I think that now that drivers affect system-wide administratively
>> visible behavior unrelated to their immediate function such as the
>> system's ability to do suspend/resume and fast-reboot, I think some more
>> details are needed beyond what was originally provided.
>>
>> Does the driver have proper support for DDI_SUSPEND in its detach()
>> routine, and for DDI_RESUME in its attach() routine? Given that it's
>> USB, does it even need to for the system to suspend/resume with such a
>> device?
>>
>> Similar question for quiesce() and fast-reboot.
>>
>> All device driver cases should probably include such details to preempt
>> these kinds of questions.
>
> Good points and I agree. Those parts should be included in further
> driver cases. About this one, urtw, AFAIK:
> . suspend/resume: supports
> . quiesce and fast-reboot: supports
btw: urtw driver will support fast-reboot by register
"ddi_quiesce_not_needed".
> . brussels: supports
>
Thanks,
Michael
