mbox series

[v2,0/5] rtw88: minor fixes from suggestions during review

Message ID 1556884415-23474-1-git-send-email-yhchuang@realtek.com (mailing list archive)
Headers show
Series rtw88: minor fixes from suggestions during review | expand

Message

Tony Chuang May 3, 2019, 11:53 a.m. UTC
From: Yan-Hsuan Chuang <yhchuang@realtek.com>

The series fix some small problems for rtw88, most of the suggestions
are from the review process.


v1 -> v2

 - modify description for LPS, ", turn off" -> ", to turn off"
 - drop patch "rtw88: mac: remove dangerous while (1)",
   should re-write the power sequence parsing code to make sense of avoiding
   infinite loop
 - unify Makefile license to Dual GPL/BSD


Yan-Hsuan Chuang (5):
  rtw88: add license for Makefile
  rtw88: pci: use ieee80211_ac_numbers instead of 0-3
  rtw88: pci: check if queue mapping exceeds size of ac_to_hwq
  rtw88: fix unassigned rssi_level in rtw_sta_info
  rtw88: more descriptions about LPS

 drivers/net/wireless/realtek/rtw88/Makefile |  2 ++
 drivers/net/wireless/realtek/rtw88/main.c   |  2 +-
 drivers/net/wireless/realtek/rtw88/pci.c    | 10 ++++++----
 drivers/net/wireless/realtek/rtw88/phy.c    |  4 ++--
 4 files changed, 11 insertions(+), 7 deletions(-)

Comments

Kalle Valo May 3, 2019, 12:04 p.m. UTC | #1
<yhchuang@realtek.com> writes:

> From: Yan-Hsuan Chuang <yhchuang@realtek.com>
>
> The series fix some small problems for rtw88, most of the suggestions
> are from the review process.
>
>
> v1 -> v2
>
>  - modify description for LPS, ", turn off" -> ", to turn off"
>  - drop patch "rtw88: mac: remove dangerous while (1)",
>    should re-write the power sequence parsing code to make sense of avoiding
>    infinite loop
>  - unify Makefile license to Dual GPL/BSD
>
>
> Yan-Hsuan Chuang (5):
>   rtw88: add license for Makefile
>   rtw88: pci: use ieee80211_ac_numbers instead of 0-3
>   rtw88: pci: check if queue mapping exceeds size of ac_to_hwq
>   rtw88: fix unassigned rssi_level in rtw_sta_info
>   rtw88: more descriptions about LPS

I was just in the next few minutes about to tag the last -next pull for
5.2. I'll apply patch 1 now so that we have consistent licenses for 5.2
but the rest have to wait for 5.3.
Stanislaw Gruszka May 6, 2019, 8:40 a.m. UTC | #2
On Fri, May 03, 2019 at 03:04:58PM +0300, Kalle Valo wrote:
> <yhchuang@realtek.com> writes:
> 
> > From: Yan-Hsuan Chuang <yhchuang@realtek.com>
> >
> > The series fix some small problems for rtw88, most of the suggestions
> > are from the review process.
> >
> >
> > v1 -> v2
> >
> >  - modify description for LPS, ", turn off" -> ", to turn off"
> >  - drop patch "rtw88: mac: remove dangerous while (1)",
> >    should re-write the power sequence parsing code to make sense of avoiding
> >    infinite loop
> >  - unify Makefile license to Dual GPL/BSD
> >
> >
> > Yan-Hsuan Chuang (5):
> >   rtw88: add license for Makefile
> >   rtw88: pci: use ieee80211_ac_numbers instead of 0-3
> >   rtw88: pci: check if queue mapping exceeds size of ac_to_hwq
> >   rtw88: fix unassigned rssi_level in rtw_sta_info
> >   rtw88: more descriptions about LPS
> 
> I was just in the next few minutes about to tag the last -next pull for
> 5.2. I'll apply patch 1 now so that we have consistent licenses for 5.2
> but the rest have to wait for 5.3.

I think '[PATCH v2 4/5] rtw88: fix unassigned rssi_level in rtw_sta_inf'
should go to 5.2 .

Stanislaw