All of lore.kernel.org
 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.