2016-06-28 01:43:41

by David Lin

[permalink] [raw]
Subject: [PATCH v8] Add new mac80211 driver mwlwifi.

PATCH v8 changes since PATCH v7:

- Used scnprintf() to replace sprintf() for debugfs output messages to avoid
overwriting buffer boundary.
- Used mutex to replace spinlock for the protection of firmware command.
- Used NL80211_BAND_XXXX instead of IEEE80211_BAND_XXXX (in order to work with
updated mac80211).
- Used usleep_range() instead of mdelay().
- Modified the code to work with new mac80211 API ampdu_action() and get peer
AMSDU information from parameters of this function instead of peeking ADDBA
related packets.
- Removed BA stream if traffic is not heavy.
- Removed version information.
- Added DFS, WPS, WDS and thermal function.
- Changed length of mac vht_mpdu from 7991 to 3895.

David Lin (1):
Add new mac80211 driver mwlwifi.

MAINTAINERS | 6 +
drivers/net/wireless/marvell/Kconfig | 1 +
drivers/net/wireless/marvell/Makefile | 1 +
drivers/net/wireless/marvell/mwlwifi/Kconfig | 23 +
drivers/net/wireless/marvell/mwlwifi/Makefile | 13 +
drivers/net/wireless/marvell/mwlwifi/debugfs.c | 780 +++++++
drivers/net/wireless/marvell/mwlwifi/debugfs.h | 24 +
drivers/net/wireless/marvell/mwlwifi/dev.h | 502 +++++
drivers/net/wireless/marvell/mwlwifi/fwcmd.c | 2747 +++++++++++++++++++++++
drivers/net/wireless/marvell/mwlwifi/fwcmd.h | 214 ++
drivers/net/wireless/marvell/mwlwifi/fwdl.c | 186 ++
drivers/net/wireless/marvell/mwlwifi/fwdl.h | 25 +
drivers/net/wireless/marvell/mwlwifi/hostcmd.h | 883 ++++++++
drivers/net/wireless/marvell/mwlwifi/isr.c | 172 ++
drivers/net/wireless/marvell/mwlwifi/isr.h | 27 +
drivers/net/wireless/marvell/mwlwifi/mac80211.c | 719 ++++++
drivers/net/wireless/marvell/mwlwifi/main.c | 759 +++++++
drivers/net/wireless/marvell/mwlwifi/rx.c | 513 +++++
drivers/net/wireless/marvell/mwlwifi/rx.h | 25 +
drivers/net/wireless/marvell/mwlwifi/sysadpt.h | 83 +
drivers/net/wireless/marvell/mwlwifi/thermal.c | 182 ++
drivers/net/wireless/marvell/mwlwifi/thermal.h | 40 +
drivers/net/wireless/marvell/mwlwifi/tx.c | 1250 +++++++++++
drivers/net/wireless/marvell/mwlwifi/tx.h | 37 +
24 files changed, 9212 insertions(+)
create mode 100644 drivers/net/wireless/marvell/mwlwifi/Kconfig
create mode 100644 drivers/net/wireless/marvell/mwlwifi/Makefile
create mode 100644 drivers/net/wireless/marvell/mwlwifi/debugfs.c
create mode 100644 drivers/net/wireless/marvell/mwlwifi/debugfs.h
create mode 100644 drivers/net/wireless/marvell/mwlwifi/dev.h
create mode 100644 drivers/net/wireless/marvell/mwlwifi/fwcmd.c
create mode 100644 drivers/net/wireless/marvell/mwlwifi/fwcmd.h
create mode 100644 drivers/net/wireless/marvell/mwlwifi/fwdl.c
create mode 100644 drivers/net/wireless/marvell/mwlwifi/fwdl.h
create mode 100644 drivers/net/wireless/marvell/mwlwifi/hostcmd.h
create mode 100644 drivers/net/wireless/marvell/mwlwifi/isr.c
create mode 100644 drivers/net/wireless/marvell/mwlwifi/isr.h
create mode 100644 drivers/net/wireless/marvell/mwlwifi/mac80211.c
create mode 100644 drivers/net/wireless/marvell/mwlwifi/main.c
create mode 100644 drivers/net/wireless/marvell/mwlwifi/rx.c
create mode 100644 drivers/net/wireless/marvell/mwlwifi/rx.h
create mode 100644 drivers/net/wireless/marvell/mwlwifi/sysadpt.h
create mode 100644 drivers/net/wireless/marvell/mwlwifi/thermal.c
create mode 100644 drivers/net/wireless/marvell/mwlwifi/thermal.h
create mode 100644 drivers/net/wireless/marvell/mwlwifi/tx.c
create mode 100644 drivers/net/wireless/marvell/mwlwifi/tx.h

--
1.9.3



2016-10-04 17:36:32

by Kalle Valo

[permalink] [raw]
Subject: Re: [PATCH v8] Add new mac80211 driver mwlwifi.

David Lin <[email protected]> writes:

> PATCH v8 changes since PATCH v7:
>
> - Used scnprintf() to replace sprintf() for debugfs output messages to avoid
> overwriting buffer boundary.
> - Used mutex to replace spinlock for the protection of firmware command.
> - Used NL80211_BAND_XXXX instead of IEEE80211_BAND_XXXX (in order to work with
> updated mac80211).
> - Used usleep_range() instead of mdelay().
> - Modified the code to work with new mac80211 API ampdu_action() and get peer
> AMSDU information from parameters of this function instead of peeking ADDBA
> related packets.
> - Removed BA stream if traffic is not heavy.
> - Removed version information.
> - Added DFS, WPS, WDS and thermal function.
> - Changed length of mac vht_mpdu from 7991 to 3895.

I did a diff against v7 and you are not really making this easy:

drivers/net/wireless/marvell/mwlwifi/Makefile | 3 +-
drivers/net/wireless/marvell/mwlwifi/debugfs.c | 591 ++++++++++++++----
drivers/net/wireless/marvell/mwlwifi/debugfs.h | 8 +-
drivers/net/wireless/marvell/mwlwifi/dev.h | 85 ++-
drivers/net/wireless/marvell/mwlwifi/fwcmd.c | 750 ++++++++++++++++-------
drivers/net/wireless/marvell/mwlwifi/fwcmd.h | 24 +-
drivers/net/wireless/marvell/mwlwifi/fwdl.c | 43 +-
drivers/net/wireless/marvell/mwlwifi/fwdl.h | 8 +-
drivers/net/wireless/marvell/mwlwifi/hostcmd.h | 109 +++-
drivers/net/wireless/marvell/mwlwifi/isr.c | 90 ++-
drivers/net/wireless/marvell/mwlwifi/isr.h | 9 +-
drivers/net/wireless/marvell/mwlwifi/mac80211.c | 74 ++-
drivers/net/wireless/marvell/mwlwifi/main.c | 270 ++++----
drivers/net/wireless/marvell/mwlwifi/rx.c | 149 +----
drivers/net/wireless/marvell/mwlwifi/rx.h | 8 +-
drivers/net/wireless/marvell/mwlwifi/sysadpt.h | 18 +-
drivers/net/wireless/marvell/mwlwifi/thermal.c | 182 ++++++
drivers/net/wireless/marvell/mwlwifi/thermal.h | 40 ++
drivers/net/wireless/marvell/mwlwifi/tx.c | 145 +++--
drivers/net/wireless/marvell/mwlwifi/tx.h | 9 +-
20 files changed, 1799 insertions(+), 816 deletions(-)

You do understand that making so drastic changes during the driver
review is very counterproductive and demotivating for the reviewers?
Before I start looking this again I want to have an assurance that you
have frozen the driver and are not adding new features. I do not want
waste my time again just going circles.

You can continue adding new features but do that as separate patches and
submit them after the driver is applied to wireless-drivers-next.

--
Kalle Valo