From: Johannes Berg <johannes@sipsolutions.net>
To: "Grumbach, Emmanuel" <emmanuel.grumbach@intel.com>,
"luca@coelho.fi" <luca@coelho.fi>
Cc: "linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>,
"Coelho, Luciano" <luciano.coelho@intel.com>
Subject: Re: [PATCH 06/10] cfg80211: honor NL80211_RRF_NO_HT40{MINUS,PLUS}
Date: Wed, 06 Sep 2017 09:29:41 +0200 [thread overview]
Message-ID: <1504682981.12380.28.camel@sipsolutions.net> (raw)
In-Reply-To: <1504630175.23076.5.camel@intel.com>
On Tue, 2017-09-05 at 16:49 +0000, Grumbach, Emmanuel wrote:
> On Tue, 2017-09-05 at 16:30 +0200, Johannes Berg wrote:
> > On Sat, 2017-08-05 at 11:44 +0300, Luca Coelho wrote:
> >
> > > + regd = get_wiphy_regdom(wiphy);
> > > + if (regd) {
> > > + const struct ieee80211_reg_rule *reg_rule =
> > > + freq_reg_info_regd(MHZ_TO_KHZ(channel-
> > > > center_freq),
> > >
> > > + regd,
> > > MHZ_TO_KHZ(20));
> > >
> >
> > This could return an error, how can you be sure it doesn't?
> >
>
> Hm... so I guess I could check that it didn't return any error and if
> it did, then flags = 0?
>
> Something like this? (on top of this patch):
> [snip]
yeah that's obviously the easy thing to do - I just wasn't sure that
you didn't have a reason to believe it could never be an ERR_PTR :)
johannes
next prev parent reply other threads:[~2017-09-06 7:29 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-08-05 8:44 [PATCH 00/10] mac80211 patches from our internal tree 2017-08-05 Luca Coelho
2017-08-05 8:44 ` [PATCH 01/10] mac80211: add MESH IE in the correct order Luca Coelho
2017-08-05 8:44 ` [PATCH 02/10] mac80211: shorten debug prints using ht_dbg() to avoid warning Luca Coelho
2017-08-05 8:44 ` [PATCH 03/10] mac80211: extend ieee80211_ie_split to support EXTENSION Luca Coelho
2017-08-05 8:44 ` [PATCH 04/10] mac80211: simplify and clarify IE splitting Luca Coelho
2017-08-05 8:44 ` [PATCH 05/10] ieee80211: Add WFA TPC report element OUI type Luca Coelho
2017-09-06 10:29 ` Johannes Berg
2017-09-29 11:21 ` [PATCH v2] " Luca Coelho
2017-08-05 8:44 ` [PATCH 06/10] cfg80211: honor NL80211_RRF_NO_HT40{MINUS,PLUS} Luca Coelho
2017-09-05 14:30 ` Johannes Berg
2017-09-05 16:49 ` Grumbach, Emmanuel
2017-09-06 7:29 ` Johannes Berg [this message]
2017-09-06 7:51 ` Grumbach, Emmanuel
2017-09-06 10:45 ` [PATCH v2] " Emmanuel Grumbach
2017-08-05 8:44 ` [PATCH 07/10] mac80211: add api to start ba session timer expired flow Luca Coelho
2017-08-05 23:14 ` kbuild test robot
2017-08-06 7:15 ` [PATCH v2] " Luca Coelho
2017-08-05 8:44 ` [PATCH 08/10] mac80211: recalculate some sta parameters after insertion Luca Coelho
2017-08-05 8:44 ` [PATCH 09/10] mac80211_hwsim: Add command to control rx status RSSI Luca Coelho
2017-09-06 10:33 ` Johannes Berg
2017-09-29 11:24 ` Luca Coelho
2017-08-05 8:44 ` [PATCH 10/10] nl80211: return error for invalid center_freq in 40 MHz Luca Coelho
2017-08-07 9:06 ` [PATCH 00/10] mac80211 patches from our internal tree 2017-08-05 Kalle Valo
2017-08-07 9:31 ` Luca Coelho
2017-08-07 13:05 ` Kalle Valo
2017-09-05 14:29 ` Johannes Berg
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=1504682981.12380.28.camel@sipsolutions.net \
--to=johannes@sipsolutions.net \
--cc=emmanuel.grumbach@intel.com \
--cc=linux-wireless@vger.kernel.org \
--cc=luca@coelho.fi \
--cc=luciano.coelho@intel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).