On Wed, Oct 17, 2012 at 4:53 PM, Bing Zhao <bzhao@xxxxxxxxxxx> wrote: > Hi Fengguang, > >> > This commit is the first patch (1/5) of "add scan flags support" series. >> > It adds a placeholder for nl80211_scan_flags. 3/5 and 4/5 add actual flags to it. >> >> But that placeholder is not a valid C construct and breaks kernel build .. >> >> > So, you have to get the whole patchset to compile. >> >> When preparing/organizing patches, every step of the patchset should >> be able to build and boot fine. That will make it friendly to bisect. > > That's a good point. Thanks! Hmm, the original patch series was build-tested for this reason; sounds like there's a difference in compiler switches. But regardless I defer to Bing. > > > Hi Johannes, > > Do you want me to respin this series or you will do it? > > Thanks, > Bing > -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html