On Friday, March 17, 2017 2:40:50 PM CET Zefir Kurtisi wrote:
> On 03/16/2017 04:13 PM, Simon Wunderlich wrote:
> > @@ -126,10 +142,14 @@ int ath9k_cmn_init_channels_rates(struct ath_common
> > *common)>
> > {
> >
> > struct ath_hw *ah = (struct ath_hw *)common->ah;
> > void *channels;
> >
> > + int num_5ghz_chan = ARRAY_SIZE(ath9k_5ghz_chantable);
> > +
> > + if (!IS_ENABLED(CONFIG_ATH9K_LICENSED_CHAN))
> > + num_5ghz_chan -= ATH9K_NUM_LICENSED_CHANNELS;
>
> These two lines seem wrong to me, since the extra channels are only added to
> the list if CONFIG_ATH9K_LICENSED_CHAN is defined. If it is not, this cuts
> off the last 7 regular channels, no?
Oh, I think you are right!
I'll revise this part and re-send. Seems I can just get rid of those two lines
...
Cheers,
Simon
signature.asc
Description: This is a digitally signed message part.
