> +static void wiphy_freq_limits_apply(struct wiphy *wiphy) [...] > + if (!wiphy_freq_limits_valid_chan(wiphy, > chan)) { > + pr_debug("Disabling freq %d MHz as > it's out of OF limits\n", > + chan->center_freq); > + chan->flags |= > IEEE80211_CHAN_DISABLED; I think you didn't address the problem in the best way now. The problem with the channel sharing was the way you're applying the limits - at runtime. This is now OK since the new function shouldn't be called when the channel structs are shared, but hooking it all into the regulatory code is now no longer needed. What you can do now, when reading the OF data, is actually apply it to the channel flags immediately. If done *before* wiphy_register(), these flags will be preserved forever, so you no longer need any hooks in regulatory code at all - you can just set the original channel flags according to the OF data. I think this greatly simplifies the flow, since you can also remove wiphy->freq_limits (and n_freq_limits) completely, since now the only effect of the function would be to modify the channel list, and later regulatory updates would always preserve the flags. johannes -- 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