| Message ID | 20190716132534.11256-1-yuehaibing@huawei.com (mailing list archive) |
|---|---|
| State | Superseded |
| Delegated to: | Kalle Valo |
| Headers | show |
| Series | libertas_tf: Use correct channel range in lbtf_geo_init | expand |
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@huawei.com> > Fixes: 691cdb49388b ("libertas_tf: command helper functions for libertas_tf") > Signed-off-by: YueHaibing <yuehaibing@huawei.com> > --- > 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; > } > >
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; }
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@huawei.com> Fixes: 691cdb49388b ("libertas_tf: command helper functions for libertas_tf") Signed-off-by: YueHaibing <yuehaibing@huawei.com> --- drivers/net/wireless/marvell/libertas_tf/cmd.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)