All of lore.kernel.org
 help / color / mirror / Atom feed
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


  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 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.