[PATCH] patches: fix path in name_assign_type patch

[PATCH] patches: fix path in name_assign_type patch

[Arend van Spriel]

Patch for brcmfmac need update path as it now resides under
the broadcom vendor directory.

Signed-off-by: Arend van Spriel <arend@broadcom.com>
---
 .../network/0055-name_assign_type/brcmfmac.patch                      | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/patches/collateral-evolutions/network/0055-name_assign_type/brcmfmac.patch b/patches/collateral-evolutions/network/0055-name_assign_type/brcmfmac.patch
index c68e9d0..bcbac24 100644
--- a/patches/collateral-evolutions/network/0055-name_assign_type/brcmfmac.patch
+++ b/patches/collateral-evolutions/network/0055-name_assign_type/brcmfmac.patch
@@ -1,5 +1,5 @@
---- a/drivers/net/wireless/brcm80211/brcmfmac/p2p.c
-+++ b/drivers/net/wireless/brcm80211/brcmfmac/p2p.c
+--- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c
++++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c
 @@ -2204,7 +2204,9 @@ struct wireless_dev *brcmf_p2p_add_vif(s
  	}
  
-- 
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe backports" in

Re: [PATCH] patches: fix path in name_assign_type patch

[Hauke Mehrtens]

On 11/30/2015 12:53 PM, Arend van Spriel wrote:
> Patch for brcmfmac need update path as it now resides under
> the broadcom vendor directory.
> 
> Signed-off-by: Arend van Spriel <arend@broadcom.com>
> ---
>  .../network/0055-name_assign_type/brcmfmac.patch                      | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/patches/collateral-evolutions/network/0055-name_assign_type/brcmfmac.patch b/patches/collateral-evolutions/network/0055-name_assign_type/brcmfmac.patch
> index c68e9d0..bcbac24 100644
> --- a/patches/collateral-evolutions/network/0055-name_assign_type/brcmfmac.patch
> +++ b/patches/collateral-evolutions/network/0055-name_assign_type/brcmfmac.patch
> @@ -1,5 +1,5 @@
> ---- a/drivers/net/wireless/brcm80211/brcmfmac/p2p.c
> -+++ b/drivers/net/wireless/brcm80211/brcmfmac/p2p.c
> +--- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c
> ++++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c
>  @@ -2204,7 +2204,9 @@ struct wireless_dev *brcmf_p2p_add_vif(s
>   	}
>   
> 
I will do this for all patches at once, but thanks.

Is it ok to replace the b43, b43legacy and brcm80211 by this single line?
drivers/net/wireless/broadcom/

Hauke
--
To unsubscribe from this list: send the line "unsubscribe backports" in

Re: [PATCH] patches: fix path in name_assign_type patch

[Johannes Berg]

> Is it ok to replace the b43, b43legacy and brcm80211 by this single
> line?
> drivers/net/wireless/broadcom/
> 

You mean in a single patch? I generally considered each file (or at
least driver) separately so you could restrict copy-list to it.

johannes
--
To unsubscribe from this list: send the line "unsubscribe backports" in

Re: [PATCH] patches: fix path in name_assign_type patch

[Arend van Spriel]

On 12/06/2015 03:37 PM, Johannes Berg wrote:
>
>> Is it ok to replace the b43, b43legacy and brcm80211 by this single
>> line?
>> drivers/net/wireless/broadcom/
>>
>
> You mean in a single patch? I generally considered each file (or at
> least driver) separately so you could restrict copy-list to it.

There is option to do:

drivers/net/wireless/broadcom/Kconfig
drivers/net/wireless/broadcom/Makefile
drivers/net/wireless/broadcom/b43
drivers/net/wireless/broadcom/b43legacy
drivers/net/wireless/broadcom/brcm80211

Or (as Hauke suggests):

drivers/net/wireless/broadcom/

To restrict the copy-list the first option would be preferred.

Regards,
Arend

> johannes
> --
> To unsubscribe from this list: send the line "unsubscribe backports" in
>

--
To unsubscribe from this list: send the line "unsubscribe backports" in

Re: [PATCH] patches: fix path in name_assign_type patch

[Johannes Berg]

On Mon, 2015-12-07 at 10:00 +0100, Arend van Spriel wrote:
> 
> > > drivers/net/wireless/broadcom/

> drivers/net/wireless/broadcom/Kconfig
> drivers/net/wireless/broadcom/Makefile
> drivers/net/wireless/broadcom/b43
> drivers/net/wireless/broadcom/b43legacy
> drivers/net/wireless/broadcom/brcm80211
> 
> Or (as Hauke suggests):
> 
> drivers/net/wireless/broadcom/
> 
> To restrict the copy-list the first option would be preferred.
> 

Oh, you meant for the copy-list?

I thought upstream backports just shipped a single copy-list line
"ddrivers/net/wireless/"?

johannes
--
To unsubscribe from this list: send the line "unsubscribe backports" in

Re: [PATCH] patches: fix path in name_assign_type patch

[Arend van Spriel]

On 12/07/2015 10:02 AM, Johannes Berg wrote:
> On Mon, 2015-12-07 at 10:00 +0100, Arend van Spriel wrote:
>>
>>>> drivers/net/wireless/broadcom/
>
>> drivers/net/wireless/broadcom/Kconfig
>> drivers/net/wireless/broadcom/Makefile
>> drivers/net/wireless/broadcom/b43
>> drivers/net/wireless/broadcom/b43legacy
>> drivers/net/wireless/broadcom/brcm80211
>>
>> Or (as Hauke suggests):
>>
>> drivers/net/wireless/broadcom/
>>
>> To restrict the copy-list the first option would be preferred.
>>
>
> Oh, you meant for the copy-list?
>
> I thought upstream backports just shipped a single copy-list line
> "ddrivers/net/wireless/"?

Nope. It is:

# 802.11
drivers/net/wireless/Kconfig
drivers/net/wireless/Makefile

It looks for subfolders and modifies the Kconfig and Makefile for the 
ones missing (comment out).

Regards,
Arend

> johannes
>

--
To unsubscribe from this list: send the line "unsubscribe backports" in

Re: [PATCH] patches: fix path in name_assign_type patch

[Hauke Mehrtens]

On 12/07/2015 10:06 AM, Arend van Spriel wrote:
> On 12/07/2015 10:02 AM, Johannes Berg wrote:
>> On Mon, 2015-12-07 at 10:00 +0100, Arend van Spriel wrote:
>>>
>>>>> drivers/net/wireless/broadcom/
>>
>>> drivers/net/wireless/broadcom/Kconfig
>>> drivers/net/wireless/broadcom/Makefile
>>> drivers/net/wireless/broadcom/b43
>>> drivers/net/wireless/broadcom/b43legacy
>>> drivers/net/wireless/broadcom/brcm80211
>>>
>>> Or (as Hauke suggests):
>>>
>>> drivers/net/wireless/broadcom/
>>>
>>> To restrict the copy-list the first option would be preferred.
>>>
>>
>> Oh, you meant for the copy-list?
>>
>> I thought upstream backports just shipped a single copy-list line
>> "ddrivers/net/wireless/"?
> 
> Nope. It is:
> 
> # 802.11
> drivers/net/wireless/Kconfig
> drivers/net/wireless/Makefile
> 
> It looks for subfolders and modifies the Kconfig and Makefile for the
> ones missing (comment out).
> 
> Regards,
> Arend
> 
>> johannes
>>

Arend is right I was asking if you would like:
drivers/net/wireless/broadcom/Kconfig
drivers/net/wireless/broadcom/Makefile
drivers/net/wireless/broadcom/b43
drivers/net/wireless/broadcom/b43legacy
drivers/net/wireless/broadcom/brcm80211

or this:
drivers/net/wireless/broadcom/

or this:
drivers/net/wireless/

for the copy-list file.

This is not just about the Broadcom driver I would do it in the same way
for all wifi driver.

Do both of you replace the copy-list file in your internal versions?

Hauke
--
To unsubscribe from this list: send the line "unsubscribe backports" in

Re: [PATCH] patches: fix path in name_assign_type patch

[Johannes Berg]

On Wed, 2015-12-09 at 23:57 +0100, Hauke Mehrtens wrote:
> 
> Do both of you replace the copy-list file in your internal versions?
> 

I think Arend said he did, and I certainly do that too. I actually just
have my own and run the gentree/git-tracker scripts with the right
commandline parameter.

So I think

drivers/net/wireless/

makes most sense since that requires no maintenance?

johannes
--
To unsubscribe from this list: send the line "unsubscribe backports" in

Re: [PATCH] patches: fix path in name_assign_type patch

[Arend van Spriel]

On 12/05/2015 09:14 PM, Hauke Mehrtens wrote:
> On 11/30/2015 12:53 PM, Arend van Spriel wrote:
>> Patch for brcmfmac need update path as it now resides under
>> the broadcom vendor directory.
>>
>> Signed-off-by: Arend van Spriel <arend@broadcom.com>
>> ---
>>   .../network/0055-name_assign_type/brcmfmac.patch                      | 4 ++--
>>   1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/patches/collateral-evolutions/network/0055-name_assign_type/brcmfmac.patch b/patches/collateral-evolutions/network/0055-name_assign_type/brcmfmac.patch
>> index c68e9d0..bcbac24 100644
>> --- a/patches/collateral-evolutions/network/0055-name_assign_type/brcmfmac.patch
>> +++ b/patches/collateral-evolutions/network/0055-name_assign_type/brcmfmac.patch
>> @@ -1,5 +1,5 @@
>> ---- a/drivers/net/wireless/brcm80211/brcmfmac/p2p.c
>> -+++ b/drivers/net/wireless/brcm80211/brcmfmac/p2p.c
>> +--- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c
>> ++++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c
>>   @@ -2204,7 +2204,9 @@ struct wireless_dev *brcmf_p2p_add_vif(s
>>    	}
>>
>>
> I will do this for all patches at once, but thanks.
>
> Is it ok to replace the b43, b43legacy and brcm80211 by this single line?
> drivers/net/wireless/broadcom/

Fine by me. I have my own copy-list to only deal with brcm80211.

Regards,
Arend

> Hauke
> --
> To unsubscribe from this list: send the line "unsubscribe backports" in
>

--
To unsubscribe from this list: send the line "unsubscribe backports" in