2013-06-05 16:16:16

by Sujith Manoharan

[permalink] [raw]
Subject: Re: [PATCH 2/7] ath9k: Enable WoW only for AR9462

Mohammed Shafi wrote:
> i remember i tested wow with AR9462, AR9485, AR9280, AR9287, though quirky
> with AR9002 families. Does it really makes sense to remove them ? Its bit
> painful to find the bugs in HAL code when we do it later, thoughts ?

WoW has to be tested quite a bit for it to be fully operational and it's largely
affected by the platform/chip quirks. Our internal test cycles cover various aspects
of WoW to make sure it doesn't affect normal usage and we never covered
the AR9002 family. Keeping untested, possibly incomplete code is not really useful...

Stanislaw raised a concern about a potential RH customer - if that happens, the patch
can be reverted (though, AR9462 would be a much better choice).