Re: [PATCH v2 06/27] brcm80211: move under broadcom vendor directory

3 messages, 3 authors, 2015-11-21 · open the first message on its own page

Re: [PATCH v2 06/27] brcm80211: move under broadcom vendor directory

From: Kalle Valo <hidden>
Date: 2015-11-19 07:49:08

Hauke Mehrtens [off-list ref] writes:
On 11/18/2015 03:45 PM, Kalle Valo wrote:
quoted
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 <redacted>
---
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.
I think this could be done separately. This patchset is big enough
already, I would not like to make it anymore complicated.

And I actually like the brcm80211 directory, I would not mind keeping it
still.

-- 
Kalle Valo

Re: [PATCH v2 06/27] brcm80211: move under broadcom vendor directory

From: Arend van Spriel <hidden>
Date: 2015-11-20 21:53:51

On 11/19/2015 08:48 AM, Kalle Valo wrote:
Hauke Mehrtens [off-list ref] writes:
quoted
On 11/18/2015 03:45 PM, Kalle Valo wrote:
quoted
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 <redacted>
---
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.
I think this could be done separately. This patchset is big enough
already, I would not like to make it anymore complicated.

And I actually like the brcm80211 directory, I would not mind keeping it
still.
I prefer to keep it as brcmsmac and brcmfmac rely on brcmutil module so 
I want to keep them together under brcm80211.

So does this patch go in before or after the patches I submitted before 
the merge window. I hope after :-p

Regards,
Arend

Re: [PATCH v2 06/27] brcm80211: move under broadcom vendor directory

From: Hauke Mehrtens <hauke@hauke-m.de>
Date: 2015-11-21 13:46:18

On 11/20/2015 10:53 PM, Arend van Spriel wrote:
On 11/19/2015 08:48 AM, Kalle Valo wrote:
quoted
Hauke Mehrtens [off-list ref] writes:
quoted
On 11/18/2015 03:45 PM, Kalle Valo wrote:
quoted
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 <redacted>
---
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.
I think this could be done separately. This patchset is big enough
already, I would not like to make it anymore complicated.

And I actually like the brcm80211 directory, I would not mind keeping it
still.
I prefer to keep it as brcmsmac and brcmfmac rely on brcmutil module so
I want to keep them together under brcm80211.

So does this patch go in before or after the patches I submitted before
the merge window. I hope after :-p
Ok, then leave it like Kalle proposed. backports should work with both
versions.

Hauke
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help