From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 16036C433EF for ; Thu, 28 Apr 2022 15:58:48 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1349722AbiD1QCB (ORCPT ); Thu, 28 Apr 2022 12:02:01 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:57684 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1349716AbiD1QB7 (ORCPT ); Thu, 28 Apr 2022 12:01:59 -0400 Received: from relay9-d.mail.gandi.net (relay9-d.mail.gandi.net [217.70.183.199]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 99D8EADD52; Thu, 28 Apr 2022 08:58:43 -0700 (PDT) Received: (Authenticated sender: miquel.raynal@bootlin.com) by mail.gandi.net (Postfix) with ESMTPSA id 9A137FF80F; Thu, 28 Apr 2022 15:58:39 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1651161522; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=RoOUgUB+tVmwGRMBe7ve0/QR//eapSfROEriVtLKfnY=; b=iMBWrk2Bal++CuTl9wJbEE4OYunUyCfZM2BV3W2EMYdCWMN9QD0BrPaPm593/8xVtYFeez Onb7Wz/W2ONA2Yn4p1qjPBLTQJic6t7TOiwgoksNc1q82m8hI4sv2xSAEjAp9g4F1Fb8oD OWrbbtl1TO3lrmmyihJvH5ZRLSODZHlSO8MUcgNxXys0H48oAqsxqeaWchD7qIUaUF3fmT TCZVPawv/dENM0WdaJ19MmL8BcSRkgX32gNwu19FIALZIf0frwayYEJV8N/77LWbaOnvjZ bC3in8udpsSH8vK7wvqHcVyLSOtqvGBZnd4mCVCGccsD3aJlWpZTWcrP9Xq0ww== Date: Thu, 28 Apr 2022 17:58:38 +0200 From: Miquel Raynal To: Alexander Aring , Stefan Schmidt , linux-wpan@vger.kernel.org Cc: "David S. Miller" , Jakub Kicinski , netdev@vger.kernel.org, Xue Liu , Marcel Holtmann , Harry Morris , David Girault , Romuald Despres , Frederic Blain , Nicolas Schodet , Thomas Petazzoni Subject: Re: [PATCH wpan-next v3 3/4] net: mac802154: Set durations automatically Message-ID: <20220428175838.08bb7717@xps13> In-Reply-To: <20220201180629.93410-4-miquel.raynal@bootlin.com> References: <20220201180629.93410-1-miquel.raynal@bootlin.com> <20220201180629.93410-4-miquel.raynal@bootlin.com> Organization: Bootlin X-Mailer: Claws Mail 3.17.7 (GTK+ 2.24.32; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org Hi Stefan, miquel.raynal@bootlin.com wrote on Tue, 1 Feb 2022 19:06:28 +0100: > As depicted in the IEEE 802.15.4 specification, modulation/bands are > tight to a number of page/channels so we can for most of them derive the > durations automatically. >=20 > The two locations that must call this new helper to set the variou > symbol durations are: > - when manually requesting a channel change though the netlink interface > - at PHY creation, once the device driver has set the default > page/channel >=20 > If an information is missing, the symbol duration is not touched, a > debug message is eventually printed. This keeps the compatibility with > the unconverted drivers for which it was too complicated for me to find > their precise information. If they initially provided a symbol duration, > it would be kept. If they don't, the symbol duration value is left > untouched. >=20 > Once the symbol duration derived, the lifs and sifs durations are > updated as well. >=20 > Signed-off-by: Miquel Raynal > --- > include/net/cfg802154.h | 2 ++ > net/mac802154/cfg.c | 1 + > net/mac802154/main.c | 46 +++++++++++++++++++++++++++++++++++++++++ > 3 files changed, 49 insertions(+) >=20 > diff --git a/include/net/cfg802154.h b/include/net/cfg802154.h > index 8a4b6a50452f..49b4bcc24032 100644 > --- a/include/net/cfg802154.h > +++ b/include/net/cfg802154.h > @@ -405,4 +405,6 @@ static inline const char *wpan_phy_name(struct wpan_p= hy *phy) > return dev_name(&phy->dev); > } > =20 > +void ieee802154_configure_durations(struct wpan_phy *phy); > + > #endif /* __NET_CFG802154_H */ > diff --git a/net/mac802154/cfg.c b/net/mac802154/cfg.c > index fbeebe3bc31d..1e4a9f74ed43 100644 > --- a/net/mac802154/cfg.c > +++ b/net/mac802154/cfg.c > @@ -118,6 +118,7 @@ ieee802154_set_channel(struct wpan_phy *wpan_phy, u8 = page, u8 channel) > if (!ret) { > wpan_phy->current_page =3D page; > wpan_phy->current_channel =3D channel; > + ieee802154_configure_durations(wpan_phy); > } > =20 > return ret; > diff --git a/net/mac802154/main.c b/net/mac802154/main.c > index 53153367f9d0..5546ef86e231 100644 > --- a/net/mac802154/main.c > +++ b/net/mac802154/main.c > @@ -113,6 +113,50 @@ ieee802154_alloc_hw(size_t priv_data_len, const stru= ct ieee802154_ops *ops) > } > EXPORT_SYMBOL(ieee802154_alloc_hw); > =20 > +void ieee802154_configure_durations(struct wpan_phy *phy) > +{ > + u32 duration =3D 0; > + > + switch (phy->current_page) { > + case 0: > + if (BIT(phy->current_page) & 0x1) I am very sorry to spot this only now but this is wrong.=20 all the conditions from here and below should be: if (BIT(phy->current_channel & )) The masks look good, the durations as well, but the conditions are wrong. > + /* 868 MHz BPSK 802.15.4-2003: 20 ksym/s */ > + duration =3D 50 * NSEC_PER_USEC; > + else if (phy->current_page & 0x7FE) Ditto > + /* 915 MHz BPSK 802.15.4-2003: 40 ksym/s */ > + duration =3D 25 * NSEC_PER_USEC; > + else if (phy->current_page & 0x7FFF800) Ditto > + /* 2400 MHz O-QPSK 802.15.4-2006: 62.5 ksym/s */ > + duration =3D 16 * NSEC_PER_USEC; > + break; > + case 2: > + if (BIT(phy->current_page) & 0x1) Ditto > + /* 868 MHz O-QPSK 802.15.4-2006: 25 ksym/s */ > + duration =3D 40 * NSEC_PER_USEC; > + else if (phy->current_page & 0x7FE) Ditto > + /* 915 MHz O-QPSK 802.15.4-2006: 62.5 ksym/s */ > + duration =3D 16 * NSEC_PER_USEC; > + break; > + case 3: > + if (BIT(phy->current_page) & 0x3FFF) Ditto > + /* 2.4 GHz CSS 802.15.4a-2007: 1/6 Msym/s */ > + duration =3D 6 * NSEC_PER_USEC; > + break; I see it's "only" in wpan-next (781830c800dd "net: mac802154: Set durations automatically") and was not yet pulled in the net-next tree so please let me know what you prefer: I can either provide a proper patch to fit it (without upstream Fixes reference), or you can just apply this diff below and push -f the branch. Let me know what you prefer. Again, sorry to only see this now. Thanks, Miqu=C3=A8l --- commit 4122765e5f982ed8f0ccea5fd813ef4d53d20a90 (HEAD) Author: Miquel Raynal Date: Thu Apr 28 17:57:59 2022 +0200 fixup! net: mac802154: Set durations automatically diff --git a/net/mac802154/main.c b/net/mac802154/main.c index 5546ef86e231..bbbdac6ee028 100644 --- a/net/mac802154/main.c +++ b/net/mac802154/main.c @@ -119,26 +119,26 @@ void ieee802154_configure_durations(struct wpan_phy *= phy) =20 switch (phy->current_page) { case 0: - if (BIT(phy->current_page) & 0x1) + if (BIT(phy->current_channel) & 0x1) /* 868 MHz BPSK 802.15.4-2003: 20 ksym/s */ duration =3D 50 * NSEC_PER_USEC; - else if (phy->current_page & 0x7FE) + else if (phy->current_channel & 0x7FE) /* 915 MHz BPSK 802.15.4-2003: 40 ksym/s */ duration =3D 25 * NSEC_PER_USEC; - else if (phy->current_page & 0x7FFF800) + else if (phy->current_channel & 0x7FFF800) /* 2400 MHz O-QPSK 802.15.4-2006: 62.5 ksym/s */ duration =3D 16 * NSEC_PER_USEC; break; case 2: - if (BIT(phy->current_page) & 0x1) + if (BIT(phy->current_channel) & 0x1) /* 868 MHz O-QPSK 802.15.4-2006: 25 ksym/s */ duration =3D 40 * NSEC_PER_USEC; - else if (phy->current_page & 0x7FE) + else if (phy->current_channel & 0x7FE) /* 915 MHz O-QPSK 802.15.4-2006: 62.5 ksym/s */ duration =3D 16 * NSEC_PER_USEC; break; case 3: - if (BIT(phy->current_page) & 0x3FFF) + if (BIT(phy->current_channel) & 0x3FFF) /* 2.4 GHz CSS 802.15.4a-2007: 1/6 Msym/s */ duration =3D 6 * NSEC_PER_USEC; break;