diff mbox series

[1/1] package/wf111: Fix manufacturer name and url

Message ID 20180129082819.13870-1-corjon.j@ecagroup.com
State Accepted
Commit cd431297e384f11a2ec285c4ec14a49a7f89fe3a
Headers show
Series [1/1] package/wf111: Fix manufacturer name and url | expand

Commit Message

Julien Corjon Jan. 29, 2018, 8:28 a.m. UTC
Signed-off-by: Julien Corjon <corjon.j@ecagroup.com>
---
 package/wf111/Config.in | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

Comments

Peter Korsgaard Jan. 29, 2018, 1:40 p.m. UTC | #1
>>>>> "Julien" == Julien Corjon <corjon.j@ecagroup.com> writes:

 > Signed-off-by: Julien Corjon <corjon.j@ecagroup.com>

Committed, thanks.
Yann E. MORIN Feb. 3, 2018, 3:40 p.m. UTC | #2
Julien, All,

On 2018-01-29 09:28 +0100, Julien Corjon spake thusly:
> Signed-off-by: Julien Corjon <corjon.j@ecagroup.com>
> ---
>  package/wf111/Config.in | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/package/wf111/Config.in b/package/wf111/Config.in
> index 80109f1b88..081bd960cb 100644
> --- a/package/wf111/Config.in
> +++ b/package/wf111/Config.in
> @@ -6,7 +6,7 @@ config BR2_PACKAGE_WF111
>  	# dynamically linked against the glibc.
>  	depends on BR2_TOOLCHAIN_USES_GLIBC
>  	help
> -	  BlueGiga WF111 WiFi driver and utilities.
> +	  Silicon Labs WF111 WiFi driver and utilities.
>  
>  	  Warning: CONFIG_WIRELESS_EXT and CONFIG_WEXT_PRIV must be
>  	  selected in the Linux kernel configuration. These are blind
> @@ -17,16 +17,16 @@ config BR2_PACKAGE_WF111
>  	    configuration entry that selects them, for example.
>  	  - By enabling another random WiFi driver that select them.
>  
> -	  http://www.bluegiga.com/en-US/products/wifi-modules/wf111-wifi-module/
> +	  https://www.silabs.com/products/wireless/wi-fi/wf111-bluegiga-module
>  
>  if BR2_PACKAGE_WF111
>  
>  config BR2_PACKAGE_WF111_TARBALL_PATH
>  	string "Local tarball location"
>  	help
> -	  The WF111 tarball can be retrieved on the BlueGiga website
> -	  after registration. This option specifies the path where the
> -	  tarball is locally saved.
> +	  The WF111 tarball can be retrieved on the Silicon Labs
> +	  website after registration. This option specifies the path
> +	  where the tarball is locally saved.

As I just replied to your previous patch (but you were dropped from Cc
for whatever reason), this is no longer true: one can very well do the
download without registration nowdays: try to download the URLs from
command line:

    wget 'https://www.silabs.com/documents/login/software/wf111-linux-driver-5-2-2-r3-armv5te.tar.gz'

and it magically works now. So we can drop the special local tarball
path now. Care to send a patch?

Regards,
Yann E. MORIN.

>  endif
>  
> -- 
> 2.14.1
> 
> _______________________________________________
> buildroot mailing list
> buildroot@busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
Yann E. MORIN Feb. 3, 2018, 5:07 p.m. UTC | #3
Julien, all,

On 2018-02-03 16:40 +0100, Yann E. MORIN spake thusly:
> On 2018-01-29 09:28 +0100, Julien Corjon spake thusly:
> > Signed-off-by: Julien Corjon <corjon.j@ecagroup.com>
[--SNIP--]
> >  config BR2_PACKAGE_WF111_TARBALL_PATH
> >  	string "Local tarball location"
> >  	help
> > -	  The WF111 tarball can be retrieved on the BlueGiga website
> > -	  after registration. This option specifies the path where the
> > -	  tarball is locally saved.
> > +	  The WF111 tarball can be retrieved on the Silicon Labs
> > +	  website after registration. This option specifies the path
> > +	  where the tarball is locally saved.
> 
> As I just replied to your previous patch (but you were dropped from Cc
> for whatever reason), this is no longer true: one can very well do the
> download without registration nowdays: try to download the URLs from
> command line:
> 
>     wget 'https://www.silabs.com/documents/login/software/wf111-linux-driver-5-2-2-r3-armv5te.tar.gz'
> 
> and it magically works now. So we can drop the special local tarball
> path now. Care to send a patch?

I got bored not being at FOSDEM, so I did it:
    https://patchwork.ozlabs.org/patch/868928/

And I also fixed the armv5 case:
    https://patchwork.ozlabs.org/patch/868927/

Regards,
Yann E. MORIN.
Peter Korsgaard Feb. 3, 2018, 7:17 p.m. UTC | #4
>>>>> "Yann" == Yann E MORIN <yann.morin.1998@free.fr> writes:

Hi,

 >> path now. Care to send a patch?

 > I got bored not being at FOSDEM, so I did it:
 >     https://patchwork.ozlabs.org/patch/868928/

 > And I also fixed the armv5 case:
 >     https://patchwork.ozlabs.org/patch/868927/

Heh ;)
diff mbox series

Patch

diff --git a/package/wf111/Config.in b/package/wf111/Config.in
index 80109f1b88..081bd960cb 100644
--- a/package/wf111/Config.in
+++ b/package/wf111/Config.in
@@ -6,7 +6,7 @@  config BR2_PACKAGE_WF111
 	# dynamically linked against the glibc.
 	depends on BR2_TOOLCHAIN_USES_GLIBC
 	help
-	  BlueGiga WF111 WiFi driver and utilities.
+	  Silicon Labs WF111 WiFi driver and utilities.
 
 	  Warning: CONFIG_WIRELESS_EXT and CONFIG_WEXT_PRIV must be
 	  selected in the Linux kernel configuration. These are blind
@@ -17,16 +17,16 @@  config BR2_PACKAGE_WF111
 	    configuration entry that selects them, for example.
 	  - By enabling another random WiFi driver that select them.
 
-	  http://www.bluegiga.com/en-US/products/wifi-modules/wf111-wifi-module/
+	  https://www.silabs.com/products/wireless/wi-fi/wf111-bluegiga-module
 
 if BR2_PACKAGE_WF111
 
 config BR2_PACKAGE_WF111_TARBALL_PATH
 	string "Local tarball location"
 	help
-	  The WF111 tarball can be retrieved on the BlueGiga website
-	  after registration. This option specifies the path where the
-	  tarball is locally saved.
+	  The WF111 tarball can be retrieved on the Silicon Labs
+	  website after registration. This option specifies the path
+	  where the tarball is locally saved.
 
 endif