Message ID | 20210511211549.30571-1-digetx@gmail.com (mailing list archive) |
---|---|
State | Awaiting Upstream |
Delegated to: | Netdev Maintainers |
Headers | show |
Series | [v2,1/2] cfg80211: Add wiphy_info_once() | expand |
Context | Check | Description |
---|---|---|
netdev/cover_letter | success | Link |
netdev/fixes_present | success | Link |
netdev/patch_count | success | Link |
netdev/tree_selection | success | Guessed tree name to be net-next |
netdev/subject_prefix | success | Link |
netdev/cc_maintainers | success | CCed 5 of 5 maintainers |
netdev/source_inline | success | Was 0 now: 0 |
netdev/verify_signedoff | success | Link |
netdev/module_param | success | Was 0 now: 0 |
netdev/build_32bit | success | Errors and warnings before: 397 this patch: 397 |
netdev/kdoc | success | Errors and warnings before: 9 this patch: 9 |
netdev/verify_fixes | success | Link |
netdev/checkpatch | success | total: 0 errors, 0 warnings, 0 checks, 8 lines checked |
netdev/build_allmodconfig_warn | success | Errors and warnings before: 397 this patch: 397 |
netdev/header_inline | success | Link |
12.05.2021 00:15, Dmitry Osipenko пишет: > Add wiphy_info_once() helper that prints info message only once. > > Signed-off-by: Dmitry Osipenko <digetx@gmail.com> > --- > > Changelog: > > v2: - New patch added in v2. > > include/net/cfg80211.h | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/include/net/cfg80211.h b/include/net/cfg80211.h > index 5224f885a99a..3b19e03509b3 100644 > --- a/include/net/cfg80211.h > +++ b/include/net/cfg80211.h > @@ -8154,6 +8154,8 @@ bool cfg80211_iftype_allowed(struct wiphy *wiphy, enum nl80211_iftype iftype, > dev_notice(&(wiphy)->dev, format, ##args) > #define wiphy_info(wiphy, format, args...) \ > dev_info(&(wiphy)->dev, format, ##args) > +#define wiphy_info_once(wiphy, format, args...) \ > + dev_info_once(&(wiphy)->dev, format, ##args) > > #define wiphy_err_ratelimited(wiphy, format, args...) \ > dev_err_ratelimited(&(wiphy)->dev, format, ##args) > Ping? Arend, is this series good to you? I assume Kalle could pick it up if you'll give ack. Thanks in advance.
Dmitry Osipenko <digetx@gmail.com> writes: > 12.05.2021 00:15, Dmitry Osipenko пишет: >> Add wiphy_info_once() helper that prints info message only once. >> >> Signed-off-by: Dmitry Osipenko <digetx@gmail.com> >> --- >> >> Changelog: >> >> v2: - New patch added in v2. >> >> include/net/cfg80211.h | 2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/include/net/cfg80211.h b/include/net/cfg80211.h >> index 5224f885a99a..3b19e03509b3 100644 >> --- a/include/net/cfg80211.h >> +++ b/include/net/cfg80211.h >> @@ -8154,6 +8154,8 @@ bool cfg80211_iftype_allowed(struct wiphy *wiphy, enum nl80211_iftype iftype, >> dev_notice(&(wiphy)->dev, format, ##args) >> #define wiphy_info(wiphy, format, args...) \ >> dev_info(&(wiphy)->dev, format, ##args) >> +#define wiphy_info_once(wiphy, format, args...) \ >> + dev_info_once(&(wiphy)->dev, format, ##args) >> >> #define wiphy_err_ratelimited(wiphy, format, args...) \ >> dev_err_ratelimited(&(wiphy)->dev, format, ##args) >> > > Ping? > > Arend, is this series good to you? I assume Kalle could pick it up if > you'll give ack. Thanks in advance. Normally cfg80211 changes go via Johannes' tree though I guess small changes I could take it via my tree, but then I need an ack from Johannes.
19.06.2021 09:27, Kalle Valo пишет: > Dmitry Osipenko <digetx@gmail.com> writes: > >> 12.05.2021 00:15, Dmitry Osipenko пишет: >>> Add wiphy_info_once() helper that prints info message only once. >>> >>> Signed-off-by: Dmitry Osipenko <digetx@gmail.com> >>> --- >>> >>> Changelog: >>> >>> v2: - New patch added in v2. >>> >>> include/net/cfg80211.h | 2 ++ >>> 1 file changed, 2 insertions(+) >>> >>> diff --git a/include/net/cfg80211.h b/include/net/cfg80211.h >>> index 5224f885a99a..3b19e03509b3 100644 >>> --- a/include/net/cfg80211.h >>> +++ b/include/net/cfg80211.h >>> @@ -8154,6 +8154,8 @@ bool cfg80211_iftype_allowed(struct wiphy *wiphy, enum nl80211_iftype iftype, >>> dev_notice(&(wiphy)->dev, format, ##args) >>> #define wiphy_info(wiphy, format, args...) \ >>> dev_info(&(wiphy)->dev, format, ##args) >>> +#define wiphy_info_once(wiphy, format, args...) \ >>> + dev_info_once(&(wiphy)->dev, format, ##args) >>> >>> #define wiphy_err_ratelimited(wiphy, format, args...) \ >>> dev_err_ratelimited(&(wiphy)->dev, format, ##args) >>> >> >> Ping? >> >> Arend, is this series good to you? I assume Kalle could pick it up if >> you'll give ack. Thanks in advance. > > Normally cfg80211 changes go via Johannes' tree though I guess small > changes I could take it via my tree, but then I need an ack from > Johannes. > Thank you for the clarification. Johannes, are these patches good to you?
On Sat, 2021-06-19 at 14:02 +0300, Dmitry Osipenko wrote: > > Johannes, are these patches good to you? > Yeah, that's fine. I was going to pick it up now but I guess it'll be faster for Kalle to do that, so Acked-by: Johannes Berg <johannes@sipsolutions.net> and I'll assign it to Kalle in patchwork. johannes
Dmitry Osipenko <digetx@gmail.com> wrote: > Add wiphy_info_once() helper that prints info message only once. > > Signed-off-by: Dmitry Osipenko <digetx@gmail.com> > Acked-by: Johannes Berg <johannes@sipsolutions.net> 2 patches applied to wireless-drivers-next.git, thanks. 761025b51c54 cfg80211: Add wiphy_info_once() 78f0a64f66d4 brcmfmac: Silence error messages about unsupported firmware features
diff --git a/include/net/cfg80211.h b/include/net/cfg80211.h index 5224f885a99a..3b19e03509b3 100644 --- a/include/net/cfg80211.h +++ b/include/net/cfg80211.h @@ -8154,6 +8154,8 @@ bool cfg80211_iftype_allowed(struct wiphy *wiphy, enum nl80211_iftype iftype, dev_notice(&(wiphy)->dev, format, ##args) #define wiphy_info(wiphy, format, args...) \ dev_info(&(wiphy)->dev, format, ##args) +#define wiphy_info_once(wiphy, format, args...) \ + dev_info_once(&(wiphy)->dev, format, ##args) #define wiphy_err_ratelimited(wiphy, format, args...) \ dev_err_ratelimited(&(wiphy)->dev, format, ##args)
Add wiphy_info_once() helper that prints info message only once. Signed-off-by: Dmitry Osipenko <digetx@gmail.com> --- Changelog: v2: - New patch added in v2. include/net/cfg80211.h | 2 ++ 1 file changed, 2 insertions(+)