| Message ID | 1483369210-23893-1-git-send-email-stefan@osg.samsung.com (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
Hi Stefan, thanks for the fix! On 01/02/2017 04:00 PM, Stefan Schmidt wrote: > After the addition of the frame_retries callback we could run into cases where > a ATUSB device with an older firmware version would now longer be able to bring > the interface up. > > We keep this functionality disabled now if the minimum firmware version for this > feature is not available. > > Fixes: 5d82288b93db3bc ("ieee802154: atusb: implement .set_frame_retries > ops callback") > Reported-by: Alexander Aring <aar@pengutronix.de> > Signed-off-by: Stefan Schmidt <stefan@osg.samsung.com> > --- Acked-by: Alexander Aring <aar@pengutronix.de> > drivers/net/ieee802154/atusb.c | 14 +++++--------- > 1 file changed, 5 insertions(+), 9 deletions(-) > > diff --git a/drivers/net/ieee802154/atusb.c b/drivers/net/ieee802154/atusb.c > index 63cb679..ac32029 100644 > --- a/drivers/net/ieee802154/atusb.c > +++ b/drivers/net/ieee802154/atusb.c > @@ -562,13 +562,6 @@ static int > atusb_set_frame_retries(struct ieee802154_hw *hw, s8 retries) > { > struct atusb *atusb = hw->priv; > - struct device *dev = &atusb->usb_dev->dev; > - > - if (atusb->fw_ver_maj == 0 && atusb->fw_ver_min < 3) { > - dev_info(dev, "Automatic frame retransmission is only available from " > - "firmware version 0.3. Please update if you want this feature."); > - return -EINVAL; > - } > > return atusb_write_subreg(atusb, SR_MAX_FRAME_RETRIES, retries); > } > @@ -802,8 +795,7 @@ static int atusb_probe(struct usb_interface *interface, > > hw->parent = &usb_dev->dev; > hw->flags = IEEE802154_HW_TX_OMIT_CKSUM | IEEE802154_HW_AFILT | > - IEEE802154_HW_PROMISCUOUS | IEEE802154_HW_CSMA_PARAMS | > - IEEE802154_HW_FRAME_RETRIES; > + IEEE802154_HW_PROMISCUOUS | IEEE802154_HW_CSMA_PARAMS; > > hw->phy->flags = WPAN_PHY_FLAG_TXPOWER | WPAN_PHY_FLAG_CCA_ED_LEVEL | > WPAN_PHY_FLAG_CCA_MODE; > @@ -832,6 +824,10 @@ static int atusb_probe(struct usb_interface *interface, > atusb_get_and_show_build(atusb); > atusb_set_extended_addr(atusb); > > + if (atusb->fw_ver_maj >= 0 && atusb->fw_ver_min >= 3) { > + hw->flags |= IEEE802154_HW_FRAME_RETRIES; > + } > + nitpick, remove brackets? Checkpatch should complain about that, I suppose. - Alex -- To unsubscribe from this list: send the line "unsubscribe linux-wpan" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hello. On 02/01/17 16:39, Alexander Aring wrote: > > Hi Stefan, > > thanks for the fix! > > On 01/02/2017 04:00 PM, Stefan Schmidt wrote: >> After the addition of the frame_retries callback we could run into cases where >> a ATUSB device with an older firmware version would now longer be able to bring >> the interface up. >> >> We keep this functionality disabled now if the minimum firmware version for this >> feature is not available. >> >> Fixes: 5d82288b93db3bc ("ieee802154: atusb: implement .set_frame_retries >> ops callback") >> Reported-by: Alexander Aring <aar@pengutronix.de> >> Signed-off-by: Stefan Schmidt <stefan@osg.samsung.com> >> --- > > Acked-by: Alexander Aring <aar@pengutronix.de> > >> drivers/net/ieee802154/atusb.c | 14 +++++--------- >> 1 file changed, 5 insertions(+), 9 deletions(-) >> >> diff --git a/drivers/net/ieee802154/atusb.c b/drivers/net/ieee802154/atusb.c >> index 63cb679..ac32029 100644 >> --- a/drivers/net/ieee802154/atusb.c >> +++ b/drivers/net/ieee802154/atusb.c >> @@ -562,13 +562,6 @@ static int >> atusb_set_frame_retries(struct ieee802154_hw *hw, s8 retries) >> { >> struct atusb *atusb = hw->priv; >> - struct device *dev = &atusb->usb_dev->dev; >> - >> - if (atusb->fw_ver_maj == 0 && atusb->fw_ver_min < 3) { >> - dev_info(dev, "Automatic frame retransmission is only available from " >> - "firmware version 0.3. Please update if you want this feature."); >> - return -EINVAL; >> - } >> >> return atusb_write_subreg(atusb, SR_MAX_FRAME_RETRIES, retries); >> } >> @@ -802,8 +795,7 @@ static int atusb_probe(struct usb_interface *interface, >> >> hw->parent = &usb_dev->dev; >> hw->flags = IEEE802154_HW_TX_OMIT_CKSUM | IEEE802154_HW_AFILT | >> - IEEE802154_HW_PROMISCUOUS | IEEE802154_HW_CSMA_PARAMS | >> - IEEE802154_HW_FRAME_RETRIES; >> + IEEE802154_HW_PROMISCUOUS | IEEE802154_HW_CSMA_PARAMS; >> >> hw->phy->flags = WPAN_PHY_FLAG_TXPOWER | WPAN_PHY_FLAG_CCA_ED_LEVEL | >> WPAN_PHY_FLAG_CCA_MODE; >> @@ -832,6 +824,10 @@ static int atusb_probe(struct usb_interface *interface, >> atusb_get_and_show_build(atusb); >> atusb_set_extended_addr(atusb); >> >> + if (atusb->fw_ver_maj >= 0 && atusb->fw_ver_min >= 3) { >> + hw->flags |= IEEE802154_HW_FRAME_RETRIES; >> + } >> + > > nitpick, remove brackets? Checkpatch should complain about that, I > suppose. Fair enough. I'm just to used to this. Version 2 with the parenthesis removed and your ack added. regards Stefan Schmidt -- To unsubscribe from this list: send the line "unsubscribe linux-wpan" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/net/ieee802154/atusb.c b/drivers/net/ieee802154/atusb.c index 63cb679..ac32029 100644 --- a/drivers/net/ieee802154/atusb.c +++ b/drivers/net/ieee802154/atusb.c @@ -562,13 +562,6 @@ static int atusb_set_frame_retries(struct ieee802154_hw *hw, s8 retries) { struct atusb *atusb = hw->priv; - struct device *dev = &atusb->usb_dev->dev; - - if (atusb->fw_ver_maj == 0 && atusb->fw_ver_min < 3) { - dev_info(dev, "Automatic frame retransmission is only available from " - "firmware version 0.3. Please update if you want this feature."); - return -EINVAL; - } return atusb_write_subreg(atusb, SR_MAX_FRAME_RETRIES, retries); } @@ -802,8 +795,7 @@ static int atusb_probe(struct usb_interface *interface, hw->parent = &usb_dev->dev; hw->flags = IEEE802154_HW_TX_OMIT_CKSUM | IEEE802154_HW_AFILT | - IEEE802154_HW_PROMISCUOUS | IEEE802154_HW_CSMA_PARAMS | - IEEE802154_HW_FRAME_RETRIES; + IEEE802154_HW_PROMISCUOUS | IEEE802154_HW_CSMA_PARAMS; hw->phy->flags = WPAN_PHY_FLAG_TXPOWER | WPAN_PHY_FLAG_CCA_ED_LEVEL | WPAN_PHY_FLAG_CCA_MODE; @@ -832,6 +824,10 @@ static int atusb_probe(struct usb_interface *interface, atusb_get_and_show_build(atusb); atusb_set_extended_addr(atusb); + if (atusb->fw_ver_maj >= 0 && atusb->fw_ver_min >= 3) { + hw->flags |= IEEE802154_HW_FRAME_RETRIES; + } + ret = atusb_get_and_clear_error(atusb); if (ret) { dev_err(&atusb->usb_dev->dev,
After the addition of the frame_retries callback we could run into cases where a ATUSB device with an older firmware version would now longer be able to bring the interface up. We keep this functionality disabled now if the minimum firmware version for this feature is not available. Fixes: 5d82288b93db3bc ("ieee802154: atusb: implement .set_frame_retries ops callback") Reported-by: Alexander Aring <aar@pengutronix.de> Signed-off-by: Stefan Schmidt <stefan@osg.samsung.com> --- drivers/net/ieee802154/atusb.c | 14 +++++--------- 1 file changed, 5 insertions(+), 9 deletions(-)