linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Kalle Valo <kvalo@kernel.org>
To: Ping-Ke Shih <pkshih@realtek.com>
Cc: "linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>,
	"mweissenbach@ignaz.org" <mweissenbach@ignaz.org>
Subject: Re: [PATCH] wifi: rtw89: pci: early chips only enable 36-bit DMA on specific PCI hosts
Date: Sat, 28 Sep 2024 14:19:45 +0300	[thread overview]
Message-ID: <87a5fs81pq.fsf@kernel.org> (raw)
In-Reply-To: <4f8451a8dfe043d08a208e52f8754283@realtek.com> (Ping-Ke Shih's message of "Fri, 27 Sep 2024 01:34:14 +0000")

Ping-Ke Shih <pkshih@realtek.com> writes:

> Ping-Ke Shih <pkshih@realtek.com> wrote:
>> 
>> The early chips including RTL8852A, RTL8851B, RTL8852B and RTL8852BT have
>> interoperability problems of 36-bit DMA with some PCI hosts. Rollback
>> to 32-bit DMA by default, and only enable 36-bit DMA for tested platforms.
>> 
>> Since all Intel platforms we have can work correctly, add the vendor ID to
>> white list. Otherwise, list vendor/device ID of bridge we have tested.
>> 
>> Fixes: 1fd4b3fe52ef ("wifi: rtw89: pci: support 36-bit PCI DMA address")
>> Reported-by: Marcel Weißenbach <mweissenbach@ignaz.org>
>> Closes:
>> https://lore.kernel.org/linux-wireless/20240918073237.Horde.VLueh0_KaiDw-9asEEcdM84@ignaz.org/T/#m07c5
>> 694df1acb173a42e1a0bab7ac22bd231a2b8
>> Signed-off-by: Ping-Ke Shih <pkshih@realtek.com>
>
> Could you please queue this patch to wireless tree? Because RTL8852BE becomes
> unusable for some users. I assigned this patch to you in patchwork, if you
> don't think so and have other suggestions, please let me know. Thanks. 
>
> The user has reported this patch works as expected [1], so add his tested-by:
> Tested-by: Marcel Weißenbach <mweissenbach@ignaz.org>
>
> Also, I think this should Cc stable, so
> Cc: stable@vger.kernel.org

Ok, I'll add all that.

-- 
https://patchwork.kernel.org/project/linux-wireless/list/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches

  reply	other threads:[~2024-09-28 11:19 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-09-24  2:16 [PATCH] wifi: rtw89: pci: early chips only enable 36-bit DMA on specific PCI hosts Ping-Ke Shih
2024-09-27  1:34 ` Ping-Ke Shih
2024-09-28 11:19   ` Kalle Valo [this message]
2024-10-07 12:25     ` Marcel Weißenbach
2024-10-11 12:43       ` Kalle Valo
2024-10-17 14:23 ` Kalle Valo

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87a5fs81pq.fsf@kernel.org \
    --to=kvalo@kernel.org \
    --cc=linux-wireless@vger.kernel.org \
    --cc=mweissenbach@ignaz.org \
    --cc=pkshih@realtek.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).