From: Kalle Valo <kvalo@kernel.org>
To: Gilad Itzkovitch <gilad.itzkovitch@morsemicro.com>
Cc: johannes@sipsolutions.net, linux-wireless@vger.kernel.org,
quic_jjohnson@quicinc.com,
Kieran Frewen <kieran.frewen@morsemicro.com>,
kernel test robot <lkp@intel.com>
Subject: Re: [PATCH v5 1/2] wifi: cfg80211: Add short_beacon_tail/head/period
Date: Wed, 19 Apr 2023 14:37:01 +0300 [thread overview]
Message-ID: <87ttxcqf1e.fsf@kernel.org> (raw)
In-Reply-To: <20230417012151.2512303-1-gilad.itzkovitch@virscient.com> (Gilad Itzkovitch's message of "Mon, 17 Apr 2023 13:21:50 +1200")
Gilad Itzkovitch <gilad.itzkovitch@morsemicro.com> writes:
> From: Kieran Frewen <kieran.frewen@morsemicro.com>
>
> Support variables to handle short beacon period and adding a
> separate tail/head for them. Also, add short beacon period,
> head and tail attributes for user configuration.
>
> Reported-by: kernel test robot <lkp@intel.com>
> Link: https://lore.kernel.org/oe-kbuild-all/202304131044.Y2mHsqp7-lkp@intel.com/
> Signed-off-by: Kieran Frewen <kieran.frewen@morsemicro.com>
> Co-developed-by: Gilad Itzkovitch <gilad.itzkovitch@morsemicro.com>
> Signed-off-by: Gilad Itzkovitch <gilad.itzkovitch@morsemicro.com>
>
> v5:
> - initialise s1g beacon variables to false
> v4:
> - squash cfg80211 commits together
> - add protection for short head/tail with S1G band check
> - better validation rule for short_beacon_period
> and align its types
> ---
We (wireless) prefer that the changelog is after the "---" line. Not
sure if Johannes can fix that during commit or not.
--
https://patchwork.kernel.org/project/linux-wireless/list/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
next prev parent reply other threads:[~2023-04-19 11:37 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-04-17 1:21 [PATCH v5 1/2] wifi: cfg80211: Add short_beacon_tail/head/period Gilad Itzkovitch
2023-04-17 1:21 ` [PATCH v5 2/2] wifi: mac80211: S1G beacon/short beacon support Gilad Itzkovitch
2023-04-19 11:37 ` Kalle Valo
2023-04-19 11:37 ` Kalle Valo [this message]
2023-08-08 10:27 ` [PATCH v6 1/2] wifi: cfg80211: Add short_beacon_tail/head/period Bassem Dawood
2023-08-08 10:27 ` [PATCH v6 2/2] wifi: mac80211: S1G beacon/short beacon support Bassem Dawood
2023-08-08 18:41 ` [PATCH v6 1/2] wifi: cfg80211: Add short_beacon_tail/head/period Jeff Johnson
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87ttxcqf1e.fsf@kernel.org \
--to=kvalo@kernel.org \
--cc=gilad.itzkovitch@morsemicro.com \
--cc=johannes@sipsolutions.net \
--cc=kieran.frewen@morsemicro.com \
--cc=linux-wireless@vger.kernel.org \
--cc=lkp@intel.com \
--cc=quic_jjohnson@quicinc.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.