Hi Kalle,

thank you for reviewing our patch. I am ok with applying this patch without
"ANDROID:" prefix. Do you want me to send an updated patch?

Regards,
Roman.

On Wed, Apr 11, 2018 at 9:23 PM Kalle Valo <kv...@codeaurora.org> wrote:

> r...@google.com writes:

> > From: Bjoern Johansson <bjoe...@google.com>
> >
> > Indicate support for power state changes and handle them by calling an
> > empty function.
> > The important part is "ieee80211_hw_set(hw, SUPPORTS_PS);" at the
> > bottom of the diff. Without this upper layers in the kernel will return
an
> > error code when trying to set the power state because the driver doesn't
> > indicate power state support. This in turn causes VTS
> > (Android Vendor Test Suite) failures because the WiFi HAL can't enable
> > power saving mode. The remaining code is just there to deal with the
> > incoming state change request.
> >
> > Signed-off-by: Bjoern Johansson <bjoe...@google.com>
> > Signed-off-by: Lingfeng Yang <l...@google.com>
> > Signed-off-by: Roman Kiryanov <r...@google.com>

> Why the odd "ANDROID:" prefix?

> --
> Kalle Valo

Reply via email to