| From 1293fccc9e892712d910ec96079d3717307f1d2d Mon Sep 17 00:00:00 2001 |
| From: Miquel Raynal <miquel.raynal@bootlin.com> |
| Date: Tue, 25 Jan 2022 13:14:21 +0100 |
| Subject: net: ieee802154: hwsim: Ensure proper channel selection at probe time |
| |
| From: Miquel Raynal <miquel.raynal@bootlin.com> |
| |
| commit 1293fccc9e892712d910ec96079d3717307f1d2d upstream. |
| |
| Drivers are expected to set the PHY current_channel and current_page |
| according to their default state. The hwsim driver is advertising being |
| configured on channel 13 by default but that is not reflected in its own |
| internal pib structure. In order to ensure that this driver consider the |
| current channel as being 13 internally, we at least need to set the |
| pib->channel field to 13. |
| |
| Fixes: f25da51fdc38 ("ieee802154: hwsim: add replacement for fakelb") |
| Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com> |
| [stefan@datenfreihafen.org: fixed assigment from page to channel] |
| Acked-by: Alexander Aring <aahringo@redhat.com> |
| Link: https://lore.kernel.org/r/20220125121426.848337-2-miquel.raynal@bootlin.com |
| Signed-off-by: Stefan Schmidt <stefan@datenfreihafen.org> |
| Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> |
| --- |
| drivers/net/ieee802154/mac802154_hwsim.c | 1 + |
| 1 file changed, 1 insertion(+) |
| |
| --- a/drivers/net/ieee802154/mac802154_hwsim.c |
| +++ b/drivers/net/ieee802154/mac802154_hwsim.c |
| @@ -786,6 +786,7 @@ static int hwsim_add_one(struct genl_inf |
| goto err_pib; |
| } |
| |
| + pib->channel = 13; |
| rcu_assign_pointer(phy->pib, pib); |
| phy->idx = idx; |
| INIT_LIST_HEAD(&phy->edges); |