Messages in this thread | | | Date | Wed, 18 Nov 2015 11:21:39 -0800 | From | Florian Fainelli <> | Subject | Re: [PATCH v2 06/27] brcm80211: move under broadcom vendor directory |
| |
On 18/11/15 11:19, Hauke Mehrtens wrote: > On 11/18/2015 03:45 PM, Kalle Valo wrote: >> Part of reorganising wireless drivers directory and Kconfig. Note that I had to >> edit Makefiles from subdirectories to use the new location. >> >> Signed-off-by: Kalle Valo <kvalo@codeaurora.org> >> --- > > I would prefer to remove the brcm80211 directory in this process and create: > drivers/net/wireless/broadcom/brcmfmac > drivers/net/wireless/broadcom/brcmsmac > drivers/net/wireless/broadcom/brcmutil > drivers/net/wireless/broadcom/include > > This way we have one directory less.
Would not that make keeping track of the previous and future history harder for people contributing to these drivers? I could imagine that for Arend and other Broadcom engineers, dealing with a simple level move would be manageable, but having to account for a different directory hierarchy could be a pain.
What is the impact on compat-wireless after/before these changes by the way? -- Florian
| |