On Tuesday 24 January 2012 05:17 PM, Kalle Valo wrote:
Hi Raja, On 01/20/2012 03:35 PM, rmani@xxxxxxxxxxxxxxxx wrote:This series of patch enables the user to specify the suspend mode via module parameter (suspend_mode) while doing insmod of the driver. To select Cut Power mode: insmod ath6kl_sdio.ko suspend_mode = 1 To select Deep Sleep mode: insmod ath6kl_sdio.ko suspend_mode = 2 To select WOW suspend mode: insmod ath6kl_sdio.ko suspend_mode = 3 Existing module param 'suspend_cutpower' variable is renamed to 'suspend_mode' for this purspose. Additionally, If the user doesn't provide any WOW patterns, the default WOW patterns will be configured while going to WOW suspend mode. V2 changes: * Existing automatic suspend mode selection logic is retained. * Suspend mode retry logic is improved without using loop. * Small patches are folded with appropriate patches as per Kalle's comments. * New wmi function is added to process set host sleep mode cmd processed event.Thanks, this is much better now. I reviewed the patches and found only minor issues.
Thanks for your review. I'll take care your comments in V3.
Kalle
-- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
