linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Piotr Oniszczuk <piotr.oniszczuk@gmail.com>
To: Arend van Spriel <arend.vanspriel@broadcom.com>
Cc: linux-wireless <linux-wireless@vger.kernel.org>,
	Ondrej Jirman <megous@megous.com>
Subject: Re: Current mainline brcmfmac reports error on supposedly supported ap6256 chip
Date: Mon, 26 Jul 2021 14:21:17 +0200	[thread overview]
Message-ID: <A76C51D0-C73B-43B0-9AE3-ABB4AED6AB86@gmail.com> (raw)
In-Reply-To: <28fdbe0c-023d-82a3-877b-7b158b109e52@broadcom.com>

Arend,

Neil Armstrong hinted me issue might be related to usage of memcpy() to/from an iomem mapping.
arm64 memcpy update (285133040e6c ("arm64: Import latest memcpy()/memmove() implementation")) triggers a memory abort when dram-access-quirk
is used on the G12A/G12B platforms.

I reverted https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/drivers/mmc/host/meson-gx-mmc.c?h=v5.13.4&id=103a5348c22c3fca8b96c735a9e353b8a0801842

and got nicely working wifi.

So it looks root cause is amlogic specific - not brcmfmac.

Sorry for false report and many many thx for Your head-up!
  
br

> Wiadomość napisana przez Arend van Spriel <arend.vanspriel@broadcom.com> w dniu 26.07.2021, o godz. 11:39:
> 
> On 7/26/2021 11:37 AM, Arend van Spriel wrote:
>> + linux-wireless, + Ondrej
>> Hoi Piotr,
>> Please reply to all. This might interest others with a similar device.
>> On 7/26/2021 11:31 AM, Piotr Oniszczuk wrote:
>>> Arend,
>>> 
>>> pls find dmesg with extra prints you proposed:
>>> 
>>> [  144.634234] brcmfmac: brcmfmac_module_init No platform data available.
>>> [  144.638974] brcmfmac: brcmf_sdio_probe Enter
>>> [  144.643057] brcmfmac: F1 signature read @0x18000000=0x15294345
>>> [  144.648718] brcmfmac: brcmf_chip_recognition found AXI chip: BCM4345/9
>>> [  144.656662] brcmfmac: brcmf_chip_cores_check  [1 ] core 0x800:51 base 0x18000000 wrap 0x18100000
>>> [  144.663690] brcmfmac: brcmf_chip_cores_check  [2 ] core 0x812:54 base 0x18001000 wrap 0x18101000
>>> [  144.672346] brcmfmac: brcmf_chip_cores_check  [3 ] core 0x83e:9  base 0x18002000 wrap 0x18102000
>>> [  144.681031] brcmfmac: brcmf_chip_cores_check  [4 ] core 0x83c:14 base 0x18003000 wrap 0x18103000
>>> [  144.689683] brcmfmac: brcmf_chip_cores_check  [5 ] core 0x829:21 base 0x18004000 wrap 0x18104000
>>> [  144.698365] brcmfmac: brcmf_chip_cores_check  [6 ] core 0x135:0  base 0x00000000 wrap 0x18107000
>>> [  144.707019] brcmfmac: brcmf_chip_cores_check  [7 ] core 0x240:0  base 0x00000000 wrap 0x00000000
>>> [  144.715678] brcmfmac: brcmf_chip_set_passive Enter
>>> [  144.722499] brcmfmac: brcmf_chip_tcm_ramsize: up 0 corecap 0 nab 0 nbb 0
>> I see. That helps. Let me come up with a patch for this.
>> [...]
>>> btw:
>>> to be sure about is my DT correct i done following tests:
>>> 1.boot my distro binary on other board with ap6256 (radxa-rockpi4b). works ok
>>> 2.selective disable wifi pwrseq and clock dt fragments. disabling stops chip detection so indirectly concluding my dt fragments seems to be ok
>> The fact that the wifi device is probed is enough proof. There may be some additional clocks needed, but for now it looks like SDIO bus is properly functioning to access the device.
> 
> ...but feel free to share the DT source so I can have a look.
> 
> Regards,
> Arend


  reply	other threads:[~2021-07-26 12:21 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <CB0AA0D2-A8F3-4217-A3C1-E14D7E9FF11E@gmail.com>
     [not found] ` <17ad94781a0.279b.9b12b7fc0a3841636cfb5e919b41b954@broadcom.com>
     [not found]   ` <6F6333F3-0BDB-48AF-B445-723958ABA8C9@gmail.com>
     [not found]     ` <be6c2b7f-2c85-59fb-3013-d0590c487c51@broadcom.com>
     [not found]       ` <53E7119F-D158-4EF0-940C-D0AA59C23CF6@gmail.com>
2021-07-26  7:54         ` Current mainline brcmfmac reports error on supposedly supported ap6256 chip Arend van Spriel
     [not found]           ` <286E2774-FAA5-47E3-A1FC-FDB09EB37FDC@gmail.com>
2021-07-26  9:37             ` Arend van Spriel
2021-07-26  9:39               ` Arend van Spriel
2021-07-26 12:21                 ` Piotr Oniszczuk [this message]
2021-07-26 12:31                   ` Arend van Spriel
2021-07-26  9:53           ` Piotr Oniszczuk
2021-07-24 13:22 Piotr Oniszczuk

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=A76C51D0-C73B-43B0-9AE3-ABB4AED6AB86@gmail.com \
    --to=piotr.oniszczuk@gmail.com \
    --cc=arend.vanspriel@broadcom.com \
    --cc=linux-wireless@vger.kernel.org \
    --cc=megous@megous.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).