Re: [PATCH 002/002] USB: serial: sierra driver autosuspend support

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

 



On Thu, 2009-07-09 at 23:36 -0700, Oliver Neukum wrote:
> Am Freitag, 10. Juli 2009 02:56:02 schrieb Elina Pasheva:
> > +module_param(support_autopm, bool, S_IRUGO | S_IWUSR);
> > +MODULE_PARM_DESC(support_autopm, "Device auto suspend support");
> 
> You implemented it correctly, but I cannot understand why it is needed
> at all. If you don't want autosuspend don't enable it the generic way.
> Why a module parameter?
> 
> 	Regards
> 		Oliver
> 
> 

Hi Oliver,

Thank you for your feedback.
In principal we do want sierra driver to support autopm but some of our
devices do not support auto suspend. Having a module parameter like
"support_autopm" is an easier way to control this instead of asking the
customer to recompile and re-install the driver.

Regards,
Elina

--
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