| Message ID | 20240412162510.29483-2-robimarko@gmail.com (mailing list archive) |
|---|---|
| State | Awaiting Upstream |
| Delegated to: | Netdev Maintainers |
| Headers | show |
| Series | [1/2] dt-bindings: net: wireless: ath11k: add ieee80211-freq-limit property | expand |
| Context | Check | Description |
|---|---|---|
| netdev/tree_selection | success | Not a local patch |
On 4/12/24 6:24 PM, Robert Marko wrote: > The common DT property can be used to limit the available channels > but ath11k has to manually call wiphy_read_of_freq_limits(). > > Signed-off-by: Robert Marko <robimarko@gmail.com> I've seen this before. https://patchwork.kernel.org/project/linux-wireless/patch/ed266944c721de8dbf0fe35f387a3a71b2c84037.1686486468.git.chunkeey@gmail.com/ (dt-binding too. it has/had an ack) https://patchwork.kernel.org/project/linux-wireless/patch/fc606d2550d047a53b4289235dd3c0fe23d5daac.1686486468.git.chunkeey@gmail.com/ sooo.... this is awkward. > --- > drivers/net/wireless/ath/ath11k/mac.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/net/wireless/ath/ath11k/mac.c b/drivers/net/wireless/ath/ath11k/mac.c > index c32be587000d..59bde128d351 100644 > --- a/drivers/net/wireless/ath/ath11k/mac.c > +++ b/drivers/net/wireless/ath/ath11k/mac.c > @@ -10124,6 +10124,7 @@ static int __ath11k_mac_register(struct ath11k *ar) > if (ret) > goto err; > > + wiphy_read_of_freq_limits(ar->hw->wiphy); > ath11k_mac_setup_ht_vht_cap(ar, cap, &ht_cap); > ath11k_mac_setup_he_cap(ar, cap); >
On Fri, 12 Apr 2024 at 21:52, Christian Lamparter <christian.lamparter@isd.uni-stuttgart.de> wrote: > > On 4/12/24 6:24 PM, Robert Marko wrote: > > The common DT property can be used to limit the available channels > > but ath11k has to manually call wiphy_read_of_freq_limits(). > > > > Signed-off-by: Robert Marko <robimarko@gmail.com> > > I've seen this before. > > https://patchwork.kernel.org/project/linux-wireless/patch/ed266944c721de8dbf0fe35f387a3a71b2c84037.1686486468.git.chunkeey@gmail.com/ > > (dt-binding too. it has/had an ack) > https://patchwork.kernel.org/project/linux-wireless/patch/fc606d2550d047a53b4289235dd3c0fe23d5daac.1686486468.git.chunkeey@gmail.com/ > > sooo.... this is awkward. Definitively, I completely missed your patch set and we needed this in OpenWrt. You were, first so your patch set should be applied. Regards, Robert > > > --- > > drivers/net/wireless/ath/ath11k/mac.c | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/drivers/net/wireless/ath/ath11k/mac.c b/drivers/net/wireless/ath/ath11k/mac.c > > index c32be587000d..59bde128d351 100644 > > --- a/drivers/net/wireless/ath/ath11k/mac.c > > +++ b/drivers/net/wireless/ath/ath11k/mac.c > > @@ -10124,6 +10124,7 @@ static int __ath11k_mac_register(struct ath11k *ar) > > if (ret) > > goto err; > > > > + wiphy_read_of_freq_limits(ar->hw->wiphy); > > ath11k_mac_setup_ht_vht_cap(ar, cap, &ht_cap); > > ath11k_mac_setup_he_cap(ar, cap); > > >
On 4/12/2024 12:52 PM, Christian Lamparter wrote: > On 4/12/24 6:24 PM, Robert Marko wrote: >> The common DT property can be used to limit the available channels >> but ath11k has to manually call wiphy_read_of_freq_limits(). >> >> Signed-off-by: Robert Marko <robimarko@gmail.com> > > I've seen this before. > > https://patchwork.kernel.org/project/linux-wireless/patch/ed266944c721de8dbf0fe35f387a3a71b2c84037.1686486468.git.chunkeey@gmail.com/ > > (dt-binding too. it has/had an ack) > https://patchwork.kernel.org/project/linux-wireless/patch/fc606d2550d047a53b4289235dd3c0fe23d5daac.1686486468.git.chunkeey@gmail.com/ > > sooo.... this is awkward. Patchwork indicates Changes Requested Any idea what changes Kalle is looking for?
Jeff Johnson <quic_jjohnson@quicinc.com> writes: > On 4/12/2024 12:52 PM, Christian Lamparter wrote: >> On 4/12/24 6:24 PM, Robert Marko wrote: >>> The common DT property can be used to limit the available channels >>> but ath11k has to manually call wiphy_read_of_freq_limits(). >>> >>> Signed-off-by: Robert Marko <robimarko@gmail.com> >> >> I've seen this before. >> >> https://patchwork.kernel.org/project/linux-wireless/patch/ed266944c721de8dbf0fe35f387a3a71b2c84037.1686486468.git.chunkeey@gmail.com/ >> >> (dt-binding too. it has/had an ack) >> https://patchwork.kernel.org/project/linux-wireless/patch/fc606d2550d047a53b4289235dd3c0fe23d5daac.1686486468.git.chunkeey@gmail.com/ >> >> sooo.... this is awkward. > > Patchwork indicates Changes Requested > Any idea what changes Kalle is looking for? I can't remember anymore but most likely I assumed based on Krzysztof's comments there will be v3 and missed that Conor already had acked it. Sorry about that, I set Christian's patches to New state now so that they are back in queue. And I'll drop Robert patches. Does this sound ok to everyone?
On Mon, 15 Apr 2024 at 18:06, Kalle Valo <kvalo@kernel.org> wrote: > > Jeff Johnson <quic_jjohnson@quicinc.com> writes: > > > On 4/12/2024 12:52 PM, Christian Lamparter wrote: > >> On 4/12/24 6:24 PM, Robert Marko wrote: > >>> The common DT property can be used to limit the available channels > >>> but ath11k has to manually call wiphy_read_of_freq_limits(). > >>> > >>> Signed-off-by: Robert Marko <robimarko@gmail.com> > >> > >> I've seen this before. > >> > >> https://patchwork.kernel.org/project/linux-wireless/patch/ed266944c721de8dbf0fe35f387a3a71b2c84037.1686486468.git.chunkeey@gmail.com/ > >> > >> (dt-binding too. it has/had an ack) > >> https://patchwork.kernel.org/project/linux-wireless/patch/fc606d2550d047a53b4289235dd3c0fe23d5daac.1686486468.git.chunkeey@gmail.com/ > >> > >> sooo.... this is awkward. > > > > Patchwork indicates Changes Requested > > Any idea what changes Kalle is looking for? > > I can't remember anymore but most likely I assumed based on Krzysztof's > comments there will be v3 and missed that Conor already had acked it. > Sorry about that, I set Christian's patches to New state now so that > they are back in queue. And I'll drop Robert patches. Does this sound ok > to everyone? Sounds good to me, thanks for looking into it. Regards, Robert > > -- > https://patchwork.kernel.org/project/linux-wireless/list/ > > https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
diff --git a/drivers/net/wireless/ath/ath11k/mac.c b/drivers/net/wireless/ath/ath11k/mac.c index c32be587000d..59bde128d351 100644 --- a/drivers/net/wireless/ath/ath11k/mac.c +++ b/drivers/net/wireless/ath/ath11k/mac.c @@ -10124,6 +10124,7 @@ static int __ath11k_mac_register(struct ath11k *ar) if (ret) goto err; + wiphy_read_of_freq_limits(ar->hw->wiphy); ath11k_mac_setup_ht_vht_cap(ar, cap, &ht_cap); ath11k_mac_setup_he_cap(ar, cap);
The common DT property can be used to limit the available channels but ath11k has to manually call wiphy_read_of_freq_limits(). Signed-off-by: Robert Marko <robimarko@gmail.com> --- drivers/net/wireless/ath/ath11k/mac.c | 1 + 1 file changed, 1 insertion(+)