On 28 December 2016 at 22:07, Arend van Spriel <arend.vanspriel@xxxxxxxxxxxx> wrote: > On 28-12-16 16:59, Rafał Miłecki wrote: >> From: Rafał Miłecki <rafal@xxxxxxxxxx> >> >> They allow specifying hardware limitations of supported channels. This >> may be useful for specifying single band devices or devices that support >> only some part of the whole band. >> E.g. some tri-band routers have separated radios for lower and higher >> part of 5 GHz band. >> >> Signed-off-by: Rafał Miłecki <rafal@xxxxxxxxxx> >> --- >> net/wireless/reg.c | 34 ++++++++++++++++++++++++++++++++++ >> 1 file changed, 34 insertions(+) >> >> diff --git a/net/wireless/reg.c b/net/wireless/reg.c >> index 5dbac37..35ba5c7 100644 >> --- a/net/wireless/reg.c >> +++ b/net/wireless/reg.c >> @@ -1123,6 +1123,26 @@ const char *reg_initiator_name(enum nl80211_reg_initiator initiator) >> } >> EXPORT_SYMBOL(reg_initiator_name); >> >> +static bool reg_center_freq_of_valid(struct wiphy *wiphy, >> + struct ieee80211_channel *chan) >> +{ >> + struct device_node *np = wiphy_dev(wiphy)->of_node; >> + u32 val; >> + >> + if (!np) >> + return true; >> + >> + if (!of_property_read_u32(np, "ieee80211-min-center-freq", &val) && >> + chan->center_freq < KHZ_TO_MHZ(val)) >> + return false; >> + >> + if (!of_property_read_u32(np, "ieee80211-max-center-freq", &val) && >> + chan->center_freq > KHZ_TO_MHZ(val)) >> + return false; > > I suspect these functions rely on CONFIG_OF. So might not build for > !CONFIG_OF. I compiled it with # CONFIG_OF is not set Can you test it and provide a non-working config if you see a compilation error, please? -- Rafał -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html