[PATCH v3 1/2] deactivate 802.11b

[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 at chrisi01.de>
> > 
> > Hiermit wird 802.11b auf den Knoten deaktiviert
> > 
> > Signed-off-by: Christian Dresel <fff at chrisi01.de>
> > Signed-off-by: Tim Niemeyer <tim at 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
> > 
> 
> 
-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 488 bytes
Beschreibung: This is a digitally signed message part
URL         : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20170820/28a3d38a/attachment.sig>