[v3,1/2] deactivate 802.11b

Submitted by Tim Niemeyer on Aug. 19, 2017, 12:34 p.m.

Details

Message ID 1503146082-10812-1-git-send-email-tim@tn-x.org
State Accepted
Headers show

Commit Message

Tim Niemeyer Aug. 19, 2017, 12:34 p.m. 
From: Christian Dresel <fff@chrisi01.de>

Hiermit wird 802.11b auf den Knoten deaktiviert

Signed-off-by: Christian Dresel <fff@chrisi01.de>
Signed-off-by: Tim Niemeyer <tim@tn-x.org>
---

Changes in v3:
- Change mode only for 11g hardware

 src/packages/fff/fff-wireless/Makefile                         | 2 +-
 src/packages/fff/fff-wireless/files/lib/functions/fff/wireless | 8 +++++++-
 2 files changed, 8 insertions(+), 2 deletions(-)

Patch hide | download patch | download mbox 

diff --git a/src/packages/fff/fff-wireless/Makefile b/src/packages/fff/fff-wireless/Makefile
index 12456e5..3ab6c81 100644
--- a/src/packages/fff/fff-wireless/Makefile
+++ b/src/packages/fff/fff-wireless/Makefile
@@ -1,7 +1,7 @@ 
 include $(TOPDIR)/rules.mk
 
 PKG_NAME:=fff-wireless
-PKG_VERSION:=0.0.1
+PKG_VERSION:=0.0.2
 PKG_RELEASE:=1
 
 PKG_BUILD_DIR:=$(BUILD_DIR)/fff-wireless
diff --git a/src/packages/fff/fff-wireless/files/lib/functions/fff/wireless b/src/packages/fff/fff-wireless/files/lib/functions/fff/wireless
index 59c8ce2..a083c13 100644
--- a/src/packages/fff/fff-wireless/files/lib/functions/fff/wireless
+++ b/src/packages/fff/fff-wireless/files/lib/functions/fff/wireless
@@ -37,8 +37,14 @@  wifiAddPhy() {
 		set wireless.${radio}.hwmode='${hwmode}'
 		set wireless.${radio}.htmode='HT20'
 		set wireless.${radio}.country='DE'
-		commit wireless
 	__EOF__
+	if [ "$hwmode" = "11g" ]; then
+		uci batch <<-__EOF__
+			set wireless.${radio}.supported_rates='6000 9000 12000 18000 24000 36000 48000 54000'
+			set wireless.${radio}.basic_rate='6000 9000 18000 36000 54000'
+		__EOF__
+	fi;
+	uci commit wireless
 
 	echo "${radio}"
 	return 0

Comments

Christian Dresel Aug. 20, 2017, 9:55 a.m. 
hi

danke fürs überarbeiten, Kleinigkeit Inline

On 19.08.2017 14:34, Tim Niemeyer wrote:
> From: Christian Dresel <fff@chrisi01.de>
> 
> Hiermit wird 802.11b auf den Knoten deaktiviert
> 
> Signed-off-by: Christian Dresel <fff@chrisi01.de>
> Signed-off-by: Tim Niemeyer <tim@tn-x.org>
> ---
> 
> Changes in v3:
> - Change mode only for 11g hardware
> 
>  src/packages/fff/fff-wireless/Makefile                         | 2 +-
>  src/packages/fff/fff-wireless/files/lib/functions/fff/wireless | 8 +++++++-
>  2 files changed, 8 insertions(+), 2 deletions(-)
> 
> diff --git a/src/packages/fff/fff-wireless/Makefile b/src/packages/fff/fff-wireless/Makefile
> index 12456e5..3ab6c81 100644
> --- a/src/packages/fff/fff-wireless/Makefile
> +++ b/src/packages/fff/fff-wireless/Makefile
> @@ -1,7 +1,7 @@
>  include $(TOPDIR)/rules.mk
>  
>  PKG_NAME:=fff-wireless
> -PKG_VERSION:=0.0.1
> +PKG_VERSION:=0.0.2
>  PKG_RELEASE:=1
>  
>  PKG_BUILD_DIR:=$(BUILD_DIR)/fff-wireless
> diff --git a/src/packages/fff/fff-wireless/files/lib/functions/fff/wireless b/src/packages/fff/fff-wireless/files/lib/functions/fff/wireless
> index 59c8ce2..a083c13 100644
> --- a/src/packages/fff/fff-wireless/files/lib/functions/fff/wireless
> +++ b/src/packages/fff/fff-wireless/files/lib/functions/fff/wireless
> @@ -37,8 +37,14 @@ wifiAddPhy() {
>  		set wireless.${radio}.hwmode='${hwmode}'
>  		set wireless.${radio}.htmode='HT20'
>  		set wireless.${radio}.country='DE'
> -		commit wireless
>  	__EOF__
> +	if [ "$hwmode" = "11g" ]; then
> +		uci batch <<-__EOF__
> +			set wireless.${radio}.supported_rates='6000 9000 12000 18000 24000 36000 48000 54000'
> +			set wireless.${radio}.basic_rate='6000 9000 18000 36000 54000'
> +		__EOF__
> +	fi;

der ; hat mich etwas verwundert scheint aber laut Shellchecker sowohl
mit als auch ohne ok zu sein. Da wir in der ganzen file aber nirgens den
; nach einem fi gesetzt haben, würde ich vorschlagen wir lassen ihn weg
(außer es ist aus irgendeinen Grund den ich gerade übersehe nötig, dann
bitte ich um Erklärung).

mfg

Christian

> +	uci commit wireless
>  
>  	echo "${radio}"
>  	return 0
>
Tim Niemeyer Aug. 20, 2017, 12:32 p.m. 
Am Sonntag, den 20.08.2017, 11:55 +0200 schrieb Christian Dresel:
> hi
> 
> danke fürs überarbeiten, Kleinigkeit Inline
> 
> On 19.08.2017 14:34, Tim Niemeyer wrote:
> > From: Christian Dresel <fff@chrisi01.de>
> > 
> > Hiermit wird 802.11b auf den Knoten deaktiviert
> > 
> > Signed-off-by: Christian Dresel <fff@chrisi01.de>
> > Signed-off-by: Tim Niemeyer <tim@tn-x.org>
> > ---
> > 
> > Changes in v3:
> > - Change mode only for 11g hardware
> > 
> >  src/packages/fff/fff-wireless/Makefile                         | 2
> > +-
> >  src/packages/fff/fff-wireless/files/lib/functions/fff/wireless | 8
> > +++++++-
> >  2 files changed, 8 insertions(+), 2 deletions(-)
> > 
> > diff --git a/src/packages/fff/fff-wireless/Makefile
> > b/src/packages/fff/fff-wireless/Makefile
> > index 12456e5..3ab6c81 100644
> > --- a/src/packages/fff/fff-wireless/Makefile
> > +++ b/src/packages/fff/fff-wireless/Makefile
> > @@ -1,7 +1,7 @@
> >  include $(TOPDIR)/rules.mk
> >  
> >  PKG_NAME:=fff-wireless
> > -PKG_VERSION:=0.0.1
> > +PKG_VERSION:=0.0.2
> >  PKG_RELEASE:=1
> >  
> >  PKG_BUILD_DIR:=$(BUILD_DIR)/fff-wireless
> > diff --git a/src/packages/fff/fff-
> > wireless/files/lib/functions/fff/wireless b/src/packages/fff/fff-
> > wireless/files/lib/functions/fff/wireless
> > index 59c8ce2..a083c13 100644
> > --- a/src/packages/fff/fff-
> > wireless/files/lib/functions/fff/wireless
> > +++ b/src/packages/fff/fff-
> > wireless/files/lib/functions/fff/wireless
> > @@ -37,8 +37,14 @@ wifiAddPhy() {
> >  		set wireless.${radio}.hwmode='${hwmode}'
> >  		set wireless.${radio}.htmode='HT20'
> >  		set wireless.${radio}.country='DE'
> > -		commit wireless
> >  	__EOF__
> > +	if [ "$hwmode" = "11g" ]; then
> > +		uci batch <<-__EOF__
> > +			set
> > wireless.${radio}.supported_rates='6000 9000 12000 18000 24000
> > 36000 48000 54000'
> > +			set wireless.${radio}.basic_rate='6000
> > 9000 18000 36000 54000'
> > +		__EOF__
> > +	fi;
> 
> der ; hat mich etwas verwundert scheint aber laut Shellchecker sowohl
> mit als auch ohne ok zu sein. Da wir in der ganzen file aber nirgens
> den
> ; nach einem fi gesetzt haben, würde ich vorschlagen wir lassen ihn
> weg
> (außer es ist aus irgendeinen Grund den ich gerade übersehe nötig,
> dann
> bitte ich um Erklärung).
Ne, good catch. Ist scheinbar so reingerutscht.

Wäre gut, wenn das beim applien entfernt wird.

Tim

> 
> mfg
> 
> Christian
> 
> > +	uci commit wireless
> >  
> >  	echo "${radio}"
> >  	return 0
> > 
> 
>
Tim Niemeyer Sept. 17, 2017, 2:20 p.m. 
Hi

Mir unten stehender Änderung applied.

Tim

Am Sonntag, den 20.08.2017, 14:32 +0200 schrieb Tim Niemeyer:
> Am Sonntag, den 20.08.2017, 11:55 +0200 schrieb Christian Dresel:
> > hi
> > 
> > danke fürs überarbeiten, Kleinigkeit Inline
> > 
> > On 19.08.2017 14:34, Tim Niemeyer wrote:
> > > From: Christian Dresel <fff@chrisi01.de>
> > > 
> > > Hiermit wird 802.11b auf den Knoten deaktiviert
> > > 
> > > Signed-off-by: Christian Dresel <fff@chrisi01.de>
> > > Signed-off-by: Tim Niemeyer <tim@tn-x.org>
> > > ---
> > > 
> > > Changes in v3:
> > > - Change mode only for 11g hardware
> > > 
> > >  src/packages/fff/fff-wireless/Makefile                         | 
> > > 2
> > > +-
> > >  src/packages/fff/fff-wireless/files/lib/functions/fff/wireless |
> > > 8
> > > +++++++-
> > >  2 files changed, 8 insertions(+), 2 deletions(-)
> > > 
> > > diff --git a/src/packages/fff/fff-wireless/Makefile
> > > b/src/packages/fff/fff-wireless/Makefile
> > > index 12456e5..3ab6c81 100644
> > > --- a/src/packages/fff/fff-wireless/Makefile
> > > +++ b/src/packages/fff/fff-wireless/Makefile
> > > @@ -1,7 +1,7 @@
> > >  include $(TOPDIR)/rules.mk
> > >  
> > >  PKG_NAME:=fff-wireless
> > > -PKG_VERSION:=0.0.1
> > > +PKG_VERSION:=0.0.2
> > >  PKG_RELEASE:=1
> > >  
> > >  PKG_BUILD_DIR:=$(BUILD_DIR)/fff-wireless
> > > diff --git a/src/packages/fff/fff-
> > > wireless/files/lib/functions/fff/wireless b/src/packages/fff/fff-
> > > wireless/files/lib/functions/fff/wireless
> > > index 59c8ce2..a083c13 100644
> > > --- a/src/packages/fff/fff-
> > > wireless/files/lib/functions/fff/wireless
> > > +++ b/src/packages/fff/fff-
> > > wireless/files/lib/functions/fff/wireless
> > > @@ -37,8 +37,14 @@ wifiAddPhy() {
> > >  		set wireless.${radio}.hwmode='${hwmode}'
> > >  		set wireless.${radio}.htmode='HT20'
> > >  		set wireless.${radio}.country='DE'
> > > -		commit wireless
> > >  	__EOF__
> > > +	if [ "$hwmode" = "11g" ]; then
> > > +		uci batch <<-__EOF__
> > > +			set
> > > wireless.${radio}.supported_rates='6000 9000 12000 18000 24000
> > > 36000 48000 54000'
> > > +			set wireless.${radio}.basic_rate='6000
> > > 9000 18000 36000 54000'
> > > +		__EOF__
> > > +	fi;
> > 
> > der ; hat mich etwas verwundert scheint aber laut Shellchecker
> > sowohl
> > mit als auch ohne ok zu sein. Da wir in der ganzen file aber
> > nirgens
> > den
> > ; nach einem fi gesetzt haben, würde ich vorschlagen wir lassen ihn
> > weg
> > (außer es ist aus irgendeinen Grund den ich gerade übersehe nötig,
> > dann
> > bitte ich um Erklärung).
> 
> Ne, good catch. Ist scheinbar so reingerutscht.
> 
> Wäre gut, wenn das beim applien entfernt wird.
> 
> Tim
> 
> > 
> > mfg
> > 
> > Christian
> > 
> > > +	uci commit wireless
> > >  
> > >  	echo "${radio}"
> > >  	return 0
> > >