On Fri, May 25, 2012 at 3:31 PM, Arend van Spriel <arend@xxxxxxxxxxxx> wrote: > On 04/16/2012 10:17 PM, Seth Forshee wrote: >> Hi Arnd, > > You missed an 'e' ;-) > >> Here's the latest update to the brcmsmac regulatory rework that I've >> been working on. I've broken it up into a series of smaller patches, >> cleaned things up, and finished what changes I can with the information >> available to me. > > It took a while. There were some discussions and we agreed that your > changes establish a better integration with the regulatory framework. > There are some concerns about the content of the crda database, but that > is another thing outside the scope of your patches. BTW if you would like to address anything please let me know. If you'd like to review things in private too that's fine, just let me know. I should say that the staging area for the next generation regulatory code is being done through the regulatory simulator: git://github.com/mcgrof/regsim.git If you have any early stage idea / changes this is where we should be reviewing / strategizing on them. I should note that this does let you use the code for regulatory anywhere, the kernel, firmware, whatever. Due to synch issues with host / firmware database though my recommendation is obviously to trust the host with the reg data, unless you resolve the synch issues. I suppose one way to deal with synch issues as a temporary stage (before removing firmware reg stuff) is to send the data structure of the regdomain to firmware so it can base prepare itself / not have any issues. >> I've attempted to maintain the same high-level behavior that the >> brcmsmac internal regulatory support currently enforces. > > Appreciated. We can add/change in subsequent patches. I collected review > comments internally including mine. I will provide them in response to > each individual patch. > > General comment: you should run 'checkpatch.pl --strict' script over the > patches. I came across several issues that will probably be flagged by it. Arend, in the future please trim all hunks that are not relevant to your replies, that makes review easier. Luis -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html