Pls ignore this, sorry On 2019/7/16 21:25, YueHaibing wrote: > It seems we should use 'range' instead of 'priv->range' > in lbtf_geo_init(), because 'range' is the corret one > related to current regioncode. > > Reported-by: Hulk Robot <hulkci@xxxxxxxxxx> > Fixes: 691cdb49388b ("libertas_tf: command helper functions for libertas_tf") > Signed-off-by: YueHaibing <yuehaibing@xxxxxxxxxx> > --- > drivers/net/wireless/marvell/libertas_tf/cmd.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/net/wireless/marvell/libertas_tf/cmd.c b/drivers/net/wireless/marvell/libertas_tf/cmd.c > index 1eacca0..a333172 100644 > --- a/drivers/net/wireless/marvell/libertas_tf/cmd.c > +++ b/drivers/net/wireless/marvell/libertas_tf/cmd.c > @@ -65,7 +65,7 @@ static void lbtf_geo_init(struct lbtf_private *priv) > break; > } > > - for (ch = priv->range.start; ch < priv->range.end; ch++) > + for (ch = range.start; ch < range.end; ch++) > priv->channels[CHAN_TO_IDX(ch)].flags = 0; > } > >