Message ID | 20221031114341.10377-1-jirislaby@kernel.org |
---|---|
State | New |
Headers |
Return-Path: <linux-kernel-owner@vger.kernel.org> Delivered-To: ouuuleilei@gmail.com Received: by 2002:a5d:6687:0:0:0:0:0 with SMTP id l7csp2261949wru; Mon, 31 Oct 2022 04:45:47 -0700 (PDT) X-Google-Smtp-Source: AMsMyM4PUWbacBqaWFZHK2iO91a2f+ZqNwZXlbVvR6BQaHpm5PCWajKfGH8QvuDTJOs/VvUhi1Oc X-Received: by 2002:a05:6402:298e:b0:451:129e:1b35 with SMTP id eq14-20020a056402298e00b00451129e1b35mr13007680edb.79.1667216747431; Mon, 31 Oct 2022 04:45:47 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1667216747; cv=none; d=google.com; s=arc-20160816; b=b1UClMGfhslt+qnJ5QjvgbhpN9UzpJ5UvdMMtdrJFo2KBz7GjmV3ysm5bLj7wAKpxc l2i9mDmJPVOx5pHO7EZsom1dsLN4HfE4y6oleT02DUuEvCPX0dLrMX5eBaWJoZp+RG5W BvKiQMD1bMWN/9NZt24Dq0dZwl1VdU4KKkCkc12t3zrgWl2/Z19dkvu0t4vpHhnZaWfF jklEoKMzR2MDFc3xuSIllnjpC8gIYPPGdU44cFvLO/sScIBYBpBWrjB7TNQp7VQbqrrc JJPoAcE5dWDMdrOu8j92UDHGlQgGnvuAcrl804OpBwcfQBqRRgBk4CsfkMXAiOH1YTvF 6aqw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from:dkim-signature; bh=FyR55Y7VqviQBY3RJFuseMd0/2jCIdqvXkpKiuzP5H4=; b=sXcS9aIQjNVSUoPBDD+iHnvRnN96JpPCpoPSK7WKO/4dLSx6YjfemJVPC+Omo4zcqo NmiBJAugMoi8X+Bcm4Z59n7Aui7bP8T1IuxQWEbLqTYtpPOhkUxMjsZ9QxffaJwsGI3r e0SoRx4vNK0cc0pHSuJ88M6guQMI+Rm4SrhkyXY/t4aOssvgjyk6SBttFCNVVMrvJn7s 5Q9mH0ROMyO+loP+5zkY5JK0a9/+PuusuM4cIMQuzo+uMmcbJu42jWaM3kWhAg/ayvWu KqPzf0MX4weAZ9xRYbf1S6qQ61qNEBp0kJdFuH0lWgvt1dJzHQJ0lAWIM0il5vtfYjHw t6fA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=TapGG1cj; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id o7-20020a056402038700b004587e08bc14si5951095edv.508.2022.10.31.04.45.23; Mon, 31 Oct 2022 04:45:47 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=TapGG1cj; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230497AbiJaLoQ (ORCPT <rfc822;kartikey406@gmail.com> + 99 others); Mon, 31 Oct 2022 07:44:16 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:41678 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230491AbiJaLny (ORCPT <rfc822;linux-kernel@vger.kernel.org>); Mon, 31 Oct 2022 07:43:54 -0400 Received: from ams.source.kernel.org (ams.source.kernel.org [IPv6:2604:1380:4601:e00::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id D961EE0F1; Mon, 31 Oct 2022 04:43:47 -0700 (PDT) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ams.source.kernel.org (Postfix) with ESMTPS id 9251DB815DB; Mon, 31 Oct 2022 11:43:46 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id E6DD8C433C1; Mon, 31 Oct 2022 11:43:42 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1667216625; bh=7CgMWQKsHtTPXkZqm7iC2oNhmPIigVqj/vJZBx2o4xg=; h=From:To:Cc:Subject:Date:From; b=TapGG1cj/MNNBx3Qa2teCF20AMx1UBkTPTwClP7Q/KpQFgnt4NM+sbJ3FL2VaGB26 t5nMsH/23wFoUzUAFu92sc83IHmkcmwrHEq1wpAzudldQD6tL+A+gYWAoQHRRAEAuu 4MGxdDKnRp1SkCh9bflpkd2ix0K5qWVr/04Aj1nsoxWGdfRErg1QeDfxYlXnE4IC6l WhR+n7CahsIZARz57uccJaeGqN6Y7c6vEq+2A204LUFS5MAzK2O7avnVbTtnos8Onk 9w61rP4t6wavtcmvmPIPfU0r957YQ0QPMeXyRbNFE7v9GUZ0ZaXwGBqFcJyxDFj3rp iSo1C0Tof/1jg== From: "Jiri Slaby (SUSE)" <jirislaby@kernel.org> To: kvalo@kernel.org Cc: linux-kernel@vger.kernel.org, "Jiri Slaby (SUSE)" <jirislaby@kernel.org>, Martin Liska <mliska@suse.cz>, "David S. Miller" <davem@davemloft.net>, Eric Dumazet <edumazet@google.com>, Jakub Kicinski <kuba@kernel.org>, Paolo Abeni <pabeni@redhat.com>, ath11k@lists.infradead.org, linux-wireless@vger.kernel.org, netdev@vger.kernel.org Subject: [PATCH] ath11k (gcc13): synchronize ath11k_mac_he_gi_to_nl80211_he_gi()'s return type Date: Mon, 31 Oct 2022 12:43:41 +0100 Message-Id: <20221031114341.10377-1-jirislaby@kernel.org> X-Mailer: git-send-email 2.38.1 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=-8.1 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_HI, SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: <linux-kernel.vger.kernel.org> X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1748203468124924302?= X-GMAIL-MSGID: =?utf-8?q?1748203468124924302?= |
Series |
ath11k (gcc13): synchronize ath11k_mac_he_gi_to_nl80211_he_gi()'s return type
|
|
Commit Message
Jiri Slaby
Oct. 31, 2022, 11:43 a.m. UTC
ath11k_mac_he_gi_to_nl80211_he_gi() generates a valid warning with gcc-13:
drivers/net/wireless/ath/ath11k/mac.c:321:20: error: conflicting types for 'ath11k_mac_he_gi_to_nl80211_he_gi' due to enum/integer mismatch; have 'enum nl80211_he_gi(u8)'
drivers/net/wireless/ath/ath11k/mac.h:166:5: note: previous declaration of 'ath11k_mac_he_gi_to_nl80211_he_gi' with type 'u32(u8)'
I.e. the type of the return value ath11k_mac_he_gi_to_nl80211_he_gi() in
the declaration is u32, while the definition spells enum nl80211_he_gi.
Synchronize them to the latter.
Cc: Martin Liska <mliska@suse.cz>
Cc: Kalle Valo <kvalo@kernel.org>
Cc: "David S. Miller" <davem@davemloft.net>
Cc: Eric Dumazet <edumazet@google.com>
Cc: Jakub Kicinski <kuba@kernel.org>
Cc: Paolo Abeni <pabeni@redhat.com>
Cc: ath11k@lists.infradead.org
Cc: linux-wireless@vger.kernel.org
Cc: netdev@vger.kernel.org
Signed-off-by: Jiri Slaby (SUSE) <jirislaby@kernel.org>
---
drivers/net/wireless/ath/ath11k/mac.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Comments
On 10/31/2022 4:43 AM, Jiri Slaby (SUSE) wrote: > ath11k_mac_he_gi_to_nl80211_he_gi() generates a valid warning with gcc-13: > drivers/net/wireless/ath/ath11k/mac.c:321:20: error: conflicting types for 'ath11k_mac_he_gi_to_nl80211_he_gi' due to enum/integer mismatch; have 'enum nl80211_he_gi(u8)' > drivers/net/wireless/ath/ath11k/mac.h:166:5: note: previous declaration of 'ath11k_mac_he_gi_to_nl80211_he_gi' with type 'u32(u8)' > > I.e. the type of the return value ath11k_mac_he_gi_to_nl80211_he_gi() in > the declaration is u32, while the definition spells enum nl80211_he_gi. > Synchronize them to the latter. > > Cc: Martin Liska <mliska@suse.cz> > Cc: Kalle Valo <kvalo@kernel.org> > Cc: "David S. Miller" <davem@davemloft.net> > Cc: Eric Dumazet <edumazet@google.com> > Cc: Jakub Kicinski <kuba@kernel.org> > Cc: Paolo Abeni <pabeni@redhat.com> > Cc: ath11k@lists.infradead.org > Cc: linux-wireless@vger.kernel.org > Cc: netdev@vger.kernel.org > Signed-off-by: Jiri Slaby (SUSE) <jirislaby@kernel.org> Suggest the subject should be wifi: ath11k: synchronize ath11k_mac_he_gi_to_nl80211_he_gi()'s return type The reference to gcc in the description should be sufficient. Kalle can update this when he merges Reviewed-by: Jeff Johnson <quic_jjohnson@quicinc.com> > --- > drivers/net/wireless/ath/ath11k/mac.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/net/wireless/ath/ath11k/mac.h b/drivers/net/wireless/ath/ath11k/mac.h > index 2a0d3afb0c99..0231783ad754 100644 > --- a/drivers/net/wireless/ath/ath11k/mac.h > +++ b/drivers/net/wireless/ath/ath11k/mac.h > @@ -163,7 +163,7 @@ void ath11k_mac_drain_tx(struct ath11k *ar); > void ath11k_mac_peer_cleanup_all(struct ath11k *ar); > int ath11k_mac_tx_mgmt_pending_free(int buf_id, void *skb, void *ctx); > u8 ath11k_mac_bw_to_mac80211_bw(u8 bw); > -u32 ath11k_mac_he_gi_to_nl80211_he_gi(u8 sgi); > +enum nl80211_he_gi ath11k_mac_he_gi_to_nl80211_he_gi(u8 sgi); > enum nl80211_he_ru_alloc ath11k_mac_phy_he_ru_to_nl80211_he_ru_alloc(u16 ru_phy); > enum nl80211_he_ru_alloc ath11k_mac_he_ru_tones_to_nl80211_he_ru_alloc(u16 ru_tones); > enum ath11k_supported_bw ath11k_mac_mac80211_bw_to_ath11k_bw(enum rate_info_bw bw);
On 31. 10. 22, 22:16, Jeff Johnson wrote: > On 10/31/2022 4:43 AM, Jiri Slaby (SUSE) wrote: >> ath11k_mac_he_gi_to_nl80211_he_gi() generates a valid warning with >> gcc-13: >> drivers/net/wireless/ath/ath11k/mac.c:321:20: error: conflicting >> types for 'ath11k_mac_he_gi_to_nl80211_he_gi' due to enum/integer >> mismatch; have 'enum nl80211_he_gi(u8)' >> drivers/net/wireless/ath/ath11k/mac.h:166:5: note: previous >> declaration of 'ath11k_mac_he_gi_to_nl80211_he_gi' with type 'u32(u8)' >> >> I.e. the type of the return value ath11k_mac_he_gi_to_nl80211_he_gi() in >> the declaration is u32, while the definition spells enum nl80211_he_gi. >> Synchronize them to the latter. >> >> Cc: Martin Liska <mliska@suse.cz> >> Cc: Kalle Valo <kvalo@kernel.org> >> Cc: "David S. Miller" <davem@davemloft.net> >> Cc: Eric Dumazet <edumazet@google.com> >> Cc: Jakub Kicinski <kuba@kernel.org> >> Cc: Paolo Abeni <pabeni@redhat.com> >> Cc: ath11k@lists.infradead.org >> Cc: linux-wireless@vger.kernel.org >> Cc: netdev@vger.kernel.org >> Signed-off-by: Jiri Slaby (SUSE) <jirislaby@kernel.org> > > Suggest the subject should be > wifi: ath11k: synchronize ath11k_mac_he_gi_to_nl80211_he_gi()'s return type FWIW I copied from: $ git log --format=%s drivers/net/wireless/ath/ath11k/mac.h ath11k: Handle keepalive during WoWLAN suspend and resume ath11k: reduce the wait time of 11d scan and hw scan while add interface ath11k: Add basic WoW functionalities ath11k: add support for hardware rfkill for QCA6390 ath11k: report tx bitrate for iw wlan station dump ath11k: add 11d scan offload support ath11k: fix read fail for htt_stats and htt_peer_stats for single pdev ath11k: add support for BSS color change ath11k: add support for 80P80 and 160 MHz bandwidth ath11k: Add support for STA to handle beacon miss ath11k: add support to configure spatial reuse parameter set ath11k: remove "ath11k_mac_get_ar_vdev_stop_status" references ath11k: Perform per-msdu rx processing ath11k: fix incorrect peer stats counters update ath11k: Move mac80211 hw allocation before wmi_init command ath11k: fix missed bw conversion in tx completion ath11k: driver for Qualcomm IEEE 802.11ax devices > The reference to gcc in the description should be sufficient. > > Kalle can update this when he merges OK, thanks. > Reviewed-by: Jeff Johnson <quic_jjohnson@quicinc.com> > >> --- >> drivers/net/wireless/ath/ath11k/mac.h | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/net/wireless/ath/ath11k/mac.h >> b/drivers/net/wireless/ath/ath11k/mac.h >> index 2a0d3afb0c99..0231783ad754 100644 >> --- a/drivers/net/wireless/ath/ath11k/mac.h >> +++ b/drivers/net/wireless/ath/ath11k/mac.h >> @@ -163,7 +163,7 @@ void ath11k_mac_drain_tx(struct ath11k *ar); >> void ath11k_mac_peer_cleanup_all(struct ath11k *ar); >> int ath11k_mac_tx_mgmt_pending_free(int buf_id, void *skb, void *ctx); >> u8 ath11k_mac_bw_to_mac80211_bw(u8 bw); >> -u32 ath11k_mac_he_gi_to_nl80211_he_gi(u8 sgi); >> +enum nl80211_he_gi ath11k_mac_he_gi_to_nl80211_he_gi(u8 sgi); >> enum nl80211_he_ru_alloc >> ath11k_mac_phy_he_ru_to_nl80211_he_ru_alloc(u16 ru_phy); >> enum nl80211_he_ru_alloc >> ath11k_mac_he_ru_tones_to_nl80211_he_ru_alloc(u16 ru_tones); >> enum ath11k_supported_bw ath11k_mac_mac80211_bw_to_ath11k_bw(enum >> rate_info_bw bw); >
Jiri Slaby <jirislaby@kernel.org> writes: > On 31. 10. 22, 22:16, Jeff Johnson wrote: > >> On 10/31/2022 4:43 AM, Jiri Slaby (SUSE) wrote: >>> ath11k_mac_he_gi_to_nl80211_he_gi() generates a valid warning with >>> gcc-13: >>> drivers/net/wireless/ath/ath11k/mac.c:321:20: error: conflicting >>> types for 'ath11k_mac_he_gi_to_nl80211_he_gi' due to enum/integer >>> mismatch; have 'enum nl80211_he_gi(u8)' >>> drivers/net/wireless/ath/ath11k/mac.h:166:5: note: previous >>> declaration of 'ath11k_mac_he_gi_to_nl80211_he_gi' with type >>> 'u32(u8)' >>> >>> I.e. the type of the return value ath11k_mac_he_gi_to_nl80211_he_gi() in >>> the declaration is u32, while the definition spells enum nl80211_he_gi. >>> Synchronize them to the latter. >>> >>> Cc: Martin Liska <mliska@suse.cz> >>> Cc: Kalle Valo <kvalo@kernel.org> >>> Cc: "David S. Miller" <davem@davemloft.net> >>> Cc: Eric Dumazet <edumazet@google.com> >>> Cc: Jakub Kicinski <kuba@kernel.org> >>> Cc: Paolo Abeni <pabeni@redhat.com> >>> Cc: ath11k@lists.infradead.org >>> Cc: linux-wireless@vger.kernel.org >>> Cc: netdev@vger.kernel.org >>> Signed-off-by: Jiri Slaby (SUSE) <jirislaby@kernel.org> >> >> Suggest the subject should be >> wifi: ath11k: synchronize ath11k_mac_he_gi_to_nl80211_he_gi()'s return type > > FWIW I copied from: > $ git log --format=%s drivers/net/wireless/ath/ath11k/mac.h > ath11k: Handle keepalive during WoWLAN suspend and resume > ath11k: reduce the wait time of 11d scan and hw scan while add interface > ath11k: Add basic WoW functionalities > ath11k: add support for hardware rfkill for QCA6390 > ath11k: report tx bitrate for iw wlan station dump > ath11k: add 11d scan offload support > ath11k: fix read fail for htt_stats and htt_peer_stats for single pdev > ath11k: add support for BSS color change > ath11k: add support for 80P80 and 160 MHz bandwidth > ath11k: Add support for STA to handle beacon miss > ath11k: add support to configure spatial reuse parameter set > ath11k: remove "ath11k_mac_get_ar_vdev_stop_status" references > ath11k: Perform per-msdu rx processing > ath11k: fix incorrect peer stats counters update > ath11k: Move mac80211 hw allocation before wmi_init command > ath11k: fix missed bw conversion in tx completion > ath11k: driver for Qualcomm IEEE 802.11ax devices Yeah, using "wifi:" is a new prefix we started using with wireless patches this year.
On 11/1/22 01:45, Kalle Valo wrote: > Jiri Slaby <jirislaby@kernel.org> writes: > >> On 31. 10. 22, 22:16, Jeff Johnson wrote: >> >>> On 10/31/2022 4:43 AM, Jiri Slaby (SUSE) wrote: >>> Suggest the subject should be >>> wifi: ath11k: synchronize ath11k_mac_he_gi_to_nl80211_he_gi()'s return type >> >> FWIW I copied from: >> $ git log --format=%s drivers/net/wireless/ath/ath11k/mac.h >> ath11k: Handle keepalive during WoWLAN suspend and resume >> ath11k: reduce the wait time of 11d scan and hw scan while add interface >> ath11k: Add basic WoW functionalities >> ath11k: add support for hardware rfkill for QCA6390 >> ath11k: report tx bitrate for iw wlan station dump >> ath11k: add 11d scan offload support >> ath11k: fix read fail for htt_stats and htt_peer_stats for single pdev >> ath11k: add support for BSS color change >> ath11k: add support for 80P80 and 160 MHz bandwidth >> ath11k: Add support for STA to handle beacon miss >> ath11k: add support to configure spatial reuse parameter set >> ath11k: remove "ath11k_mac_get_ar_vdev_stop_status" references >> ath11k: Perform per-msdu rx processing >> ath11k: fix incorrect peer stats counters update >> ath11k: Move mac80211 hw allocation before wmi_init command >> ath11k: fix missed bw conversion in tx completion >> ath11k: driver for Qualcomm IEEE 802.11ax devices > > Yeah, using "wifi:" is a new prefix we started using with wireless > patches this year. > It would be nice if that was documented somewhere...
Randy Dunlap <rdunlap@infradead.org> writes: > On 11/1/22 01:45, Kalle Valo wrote: >> Jiri Slaby <jirislaby@kernel.org> writes: >> >>> On 31. 10. 22, 22:16, Jeff Johnson wrote: >>> >>>> On 10/31/2022 4:43 AM, Jiri Slaby (SUSE) wrote: > >>>> Suggest the subject should be >>>> wifi: ath11k: synchronize ath11k_mac_he_gi_to_nl80211_he_gi()'s return type >>> >>> FWIW I copied from: >>> $ git log --format=%s drivers/net/wireless/ath/ath11k/mac.h >>> ath11k: Handle keepalive during WoWLAN suspend and resume >>> ath11k: reduce the wait time of 11d scan and hw scan while add interface >>> ath11k: Add basic WoW functionalities >>> ath11k: add support for hardware rfkill for QCA6390 >>> ath11k: report tx bitrate for iw wlan station dump >>> ath11k: add 11d scan offload support >>> ath11k: fix read fail for htt_stats and htt_peer_stats for single pdev >>> ath11k: add support for BSS color change >>> ath11k: add support for 80P80 and 160 MHz bandwidth >>> ath11k: Add support for STA to handle beacon miss >>> ath11k: add support to configure spatial reuse parameter set >>> ath11k: remove "ath11k_mac_get_ar_vdev_stop_status" references >>> ath11k: Perform per-msdu rx processing >>> ath11k: fix incorrect peer stats counters update >>> ath11k: Move mac80211 hw allocation before wmi_init command >>> ath11k: fix missed bw conversion in tx completion >>> ath11k: driver for Qualcomm IEEE 802.11ax devices >> >> Yeah, using "wifi:" is a new prefix we started using with wireless >> patches this year. >> > > It would be nice if that was documented somewhere... It is mentioned on our wiki but I doubt anyone reads it :) https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches#subject Do let me know if there are other places which should have this info. I did assume it will take at least a year or two before people get used to the new prefix, but my patchwork script has a check for this and it's trivial to fix the subject before I commit the patch. So hopefully the switch goes smoothly.
Hi-- On 11/1/22 10:19, Kalle Valo wrote: > Randy Dunlap <rdunlap@infradead.org> writes: > >> On 11/1/22 01:45, Kalle Valo wrote: >>> Jiri Slaby <jirislaby@kernel.org> writes: >>> >>>> On 31. 10. 22, 22:16, Jeff Johnson wrote: >>>> >>>>> On 10/31/2022 4:43 AM, Jiri Slaby (SUSE) wrote: >> >>>>> Suggest the subject should be >>>>> wifi: ath11k: synchronize ath11k_mac_he_gi_to_nl80211_he_gi()'s return type >>>> >>>> FWIW I copied from: >>>> $ git log --format=%s drivers/net/wireless/ath/ath11k/mac.h >>>> ath11k: Handle keepalive during WoWLAN suspend and resume >>>> ath11k: reduce the wait time of 11d scan and hw scan while add interface >>>> ath11k: Add basic WoW functionalities >>>> ath11k: add support for hardware rfkill for QCA6390 >>>> ath11k: report tx bitrate for iw wlan station dump >>>> ath11k: add 11d scan offload support >>>> ath11k: fix read fail for htt_stats and htt_peer_stats for single pdev >>>> ath11k: add support for BSS color change >>>> ath11k: add support for 80P80 and 160 MHz bandwidth >>>> ath11k: Add support for STA to handle beacon miss >>>> ath11k: add support to configure spatial reuse parameter set >>>> ath11k: remove "ath11k_mac_get_ar_vdev_stop_status" references >>>> ath11k: Perform per-msdu rx processing >>>> ath11k: fix incorrect peer stats counters update >>>> ath11k: Move mac80211 hw allocation before wmi_init command >>>> ath11k: fix missed bw conversion in tx completion >>>> ath11k: driver for Qualcomm IEEE 802.11ax devices >>> >>> Yeah, using "wifi:" is a new prefix we started using with wireless >>> patches this year. >>> >> >> It would be nice if that was documented somewhere... > > It is mentioned on our wiki but I doubt anyone reads it :) I think that you are correct. ;) > https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches#subject > > Do let me know if there are other places which should have this info. Ideally it would be in the subsystem's profile document as described in the MAINTAINERS file: P: Subsystem Profile document for more details submitting patches to the given subsystem. This is either an in-tree file, or a URI. See Documentation/maintainer/maintainer-entry-profile.rst for details. although that seems to be overkill IMHO just to add a prefix: setting. You could just clone some other maintainer's Profile document and then modify it to anything that you would like to have in it as far as Maintaining and patching are concerned. > I did assume it will take at least a year or two before people get used > to the new prefix, but my patchwork script has a check for this and it's > trivial to fix the subject before I commit the patch. So hopefully the > switch goes smoothly. OK. Thanks.
On 01. 11. 22, 18:19, Kalle Valo wrote: > I did assume it will take at least a year or two before people get used > to the new prefix, but my patchwork script has a check for this and it's > trivial to fix the subject before I commit the patch. So hopefully the > switch goes smoothly. I think so. It will take some turnarounds for you before this starts appearing in git log output for every wireless driver. Then, people will start picking the prefix up from there ;). thanks,
"Jiri Slaby (SUSE)" <jirislaby@kernel.org> wrote: > ath11k_mac_he_gi_to_nl80211_he_gi() generates a valid warning with gcc-13: > drivers/net/wireless/ath/ath11k/mac.c:321:20: error: conflicting types for 'ath11k_mac_he_gi_to_nl80211_he_gi' due to enum/integer mismatch; have 'enum nl80211_he_gi(u8)' > drivers/net/wireless/ath/ath11k/mac.h:166:5: note: previous declaration of 'ath11k_mac_he_gi_to_nl80211_he_gi' with type 'u32(u8)' > > I.e. the type of the return value ath11k_mac_he_gi_to_nl80211_he_gi() in > the declaration is u32, while the definition spells enum nl80211_he_gi. > Synchronize them to the latter. > > Cc: Martin Liska <mliska@suse.cz> > Cc: Kalle Valo <kvalo@kernel.org> > Cc: "David S. Miller" <davem@davemloft.net> > Cc: Eric Dumazet <edumazet@google.com> > Cc: Jakub Kicinski <kuba@kernel.org> > Cc: Paolo Abeni <pabeni@redhat.com> > Cc: ath11k@lists.infradead.org > Cc: linux-wireless@vger.kernel.org > Cc: netdev@vger.kernel.org > Signed-off-by: Jiri Slaby (SUSE) <jirislaby@kernel.org> > Reviewed-by: Jeff Johnson <quic_jjohnson@quicinc.com> > Signed-off-by: Kalle Valo <quic_kvalo@quicinc.com> Patch applied to ath-next branch of ath.git, thanks. dd1c23226945 wifi: ath11k: synchronize ath11k_mac_he_gi_to_nl80211_he_gi()'s return type
Randy Dunlap <rdunlap@infradead.org> writes: >>>> Yeah, using "wifi:" is a new prefix we started using with wireless >>>> patches this year. >>>> >>> >>> It would be nice if that was documented somewhere... >> >> It is mentioned on our wiki but I doubt anyone reads it :) > > I think that you are correct. ;) > >> https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches#subject >> >> Do let me know if there are other places which should have this info. > > Ideally it would be in the subsystem's profile document as described in the > MAINTAINERS file: > > P: Subsystem Profile document for more details submitting > patches to the given subsystem. This is either an in-tree file, > or a URI. See Documentation/maintainer/maintainer-entry-profile.rst > for details. > > although that seems to be overkill IMHO just to add a prefix: setting. > > You could just clone some other maintainer's Profile document and then modify it > to anything that you would like to have in it as far as Maintaining and patching > are concerned. Ah, we should add that doc for wireless. Thanks for the idea, I added that to my todo list.
On 11/2/22 23:20, Kalle Valo wrote: > Randy Dunlap <rdunlap@infradead.org> writes: > >>>>> Yeah, using "wifi:" is a new prefix we started using with wireless >>>>> patches this year. >>>>> >>>> >>>> It would be nice if that was documented somewhere... >>> >>> It is mentioned on our wiki but I doubt anyone reads it :) >> >> I think that you are correct. ;) >> >>> https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches#subject >>> >>> Do let me know if there are other places which should have this info. >> >> Ideally it would be in the subsystem's profile document as described in the >> MAINTAINERS file: >> >> P: Subsystem Profile document for more details submitting >> patches to the given subsystem. This is either an in-tree file, >> or a URI. See Documentation/maintainer/maintainer-entry-profile.rst >> for details. >> >> although that seems to be overkill IMHO just to add a prefix: setting. >> >> You could just clone some other maintainer's Profile document and then modify it >> to anything that you would like to have in it as far as Maintaining and patching >> are concerned. > > Ah, we should add that doc for wireless. Thanks for the idea, I added > that to my todo list. > Thank you. :)
diff --git a/drivers/net/wireless/ath/ath11k/mac.h b/drivers/net/wireless/ath/ath11k/mac.h index 2a0d3afb0c99..0231783ad754 100644 --- a/drivers/net/wireless/ath/ath11k/mac.h +++ b/drivers/net/wireless/ath/ath11k/mac.h @@ -163,7 +163,7 @@ void ath11k_mac_drain_tx(struct ath11k *ar); void ath11k_mac_peer_cleanup_all(struct ath11k *ar); int ath11k_mac_tx_mgmt_pending_free(int buf_id, void *skb, void *ctx); u8 ath11k_mac_bw_to_mac80211_bw(u8 bw); -u32 ath11k_mac_he_gi_to_nl80211_he_gi(u8 sgi); +enum nl80211_he_gi ath11k_mac_he_gi_to_nl80211_he_gi(u8 sgi); enum nl80211_he_ru_alloc ath11k_mac_phy_he_ru_to_nl80211_he_ru_alloc(u16 ru_phy); enum nl80211_he_ru_alloc ath11k_mac_he_ru_tones_to_nl80211_he_ru_alloc(u16 ru_tones); enum ath11k_supported_bw ath11k_mac_mac80211_bw_to_ath11k_bw(enum rate_info_bw bw);