From: Johannes Berg <johannes@sipsolutions.net>
To: Arik Nemtsov <arik@wizery.com>
Cc: "linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>,
"Luis R. Rodriguez" <mcgrof@suse.com>,
Jonathan Doron <jond@wizery.com>
Subject: Re: [PATCH v8 2/4] cfg80211: allow wiphy specific regdomain management
Date: Mon, 15 Dec 2014 19:41:07 +0100 [thread overview]
Message-ID: <1418668867.5731.8.camel@sipsolutions.net> (raw)
In-Reply-To: <CA+XVXffHzjVg_KiaFz+sdQPsV+2iTmPGGoLHE7K9dhs7mYDhWg@mail.gmail.com> (sfid-20141215_181305_579984_21253B72)
On Mon, 2014-12-15 at 19:12 +0200, Arik Nemtsov wrote:
> On Fri, Dec 12, 2014 at 2:39 PM, Johannes Berg
> <johannes@sipsolutions.net> wrote:
> > On Wed, 2014-12-03 at 18:08 +0200, Arik Nemtsov wrote:
> >
> >> +void nl80211_send_reg_change_event(struct regulatory_request *request)
> >> +{
> >> + nl80211_common_reg_change_event(NL80211_CMD_REG_CHANGE, request);
> >> +}
> >> +
> >> +void nl80211_send_wiphy_reg_change_event(struct regulatory_request *request)
> >> +{
> >> + nl80211_common_reg_change_event(NL80211_CMD_WIPHY_REG_CHANGE, request);
> >> +}
> >
> > These are trivial wrappers, I'd make them inlines.
>
> You mean inline here or in the header header? I'll do it here..
Inline here makes no sense since it's declared in the header - it'd have
to be in the header.
johannes
next prev parent reply other threads:[~2014-12-15 18:41 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-12-03 16:08 [PATCH v8 1/4] cfg80211: allow usermode to query wiphy specific regdom Arik Nemtsov
2014-12-03 16:08 ` [PATCH v8 2/4] cfg80211: allow wiphy specific regdomain management Arik Nemtsov
2014-12-12 12:39 ` Johannes Berg
2014-12-15 17:12 ` Arik Nemtsov
2014-12-15 18:41 ` Johannes Berg [this message]
2014-12-15 19:28 ` Arik Nemtsov
2014-12-03 16:08 ` [PATCH v8 3/4] cfg80211: return private regdom for self-managed devices Arik Nemtsov
2014-12-03 16:08 ` [PATCH v8 4/4] cfg80211: correctly check ad-hoc channels Arik Nemtsov
2014-12-12 12:40 ` Johannes Berg
2014-12-12 12:37 ` [PATCH v8 1/4] cfg80211: allow usermode to query wiphy specific regdom Johannes Berg
2014-12-15 17:07 ` Arik Nemtsov
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=1418668867.5731.8.camel@sipsolutions.net \
--to=johannes@sipsolutions.net \
--cc=arik@wizery.com \
--cc=jond@wizery.com \
--cc=linux-wireless@vger.kernel.org \
--cc=mcgrof@suse.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).