From: Alessio Ferri <alessio.ferri@mythread.it>
To: "andreas.wendleder" <andreas.wendleder@proton.me>
Cc: "Michael Büsch" <m@bues.ch>,
"b43-dev@lists.infradead.org" <b43-dev@lists.infradead.org>,
"linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>
Subject: Re: BCM4360 AC-PHY support for b43 — reverse-engineered, seeking guidance on a clean path upstream
Date: Tue, 29 Sep 2026 10:58:59 +0200 [thread overview]
Message-ID: <20260929105859.364ea71b@DELL-MOBILE03.ad.smart.it> (raw)
In-Reply-To: <d6tVX7HBAW4Qnebx5Enj79edWG_LVmnRT0bjhcfv9bb1h0cewrTEDHK-VVoSy0NZcMzv3FB8OVEyTFNgA7rs_zaplEfiLXdwcCqhYDeonMo=@proton.me>
On Tue, 29 Sep 2026 08:27:53 +0000
"andreas.wendleder" <andreas.wendleder@proton.me> wrote:
> > Least, but not last, the wl order of macro operation and b43 order
> > are different in multiple places, how did you reconcile the two?
>
> Good sign: we're core_rev 42 too, so our disassembly and yours should
> be directly comparable.
>
I didn't mass disassemble, i just hooked the io accessors, both because
it is faster to learn what the driver is doing and less problematic if
someone ask. Also the trace can be shared, the disassembly does not.
> femctrl/srom: not touched, no handling anywhere in our port. One fixed
> test board, so we've never hit a different value. No data point to
> offer - if you find out what it gates, let me know.
So you harcoded your srom?
>
> wl order vs b43 order: we captured wl's real register-access sequence
> live (bpftrace kprobes on its own osl_readl/writel/delay, wl left
> loaded and bound throughout - never touch its binding while probes
> attach) and diffed it against our call order, case by case.
>
> Sharpest example:
> b43's generic switch_channel calls the per-channel tune, then
> unconditionally re-applies a captured POR snapshot afterward -
> captured during wl's first association, on channel 1. So channel 6
> got tuned correctly, then immediately overwritten back to channel 1's
> values. wl's real order runs the POR replay once at attach, not after
> every switch. Cost us a real mistune bug before the trace diff caught
> it. Smaller example, same category: our TX-cal port was missing one
> PHY register write wl issues between the gain-table override and
> starting the test tone - doesn't crash, just makes the measurement
> never differentiate.
>
But which driver version did you look at? The 6.30 hybrid?
next prev parent reply other threads:[~2026-09-29 8:59 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <rOzY3WnW4q5L1osv7iagU81dLGldzZEsu3EqHxmH4d7Z4vxubBYvRNT0vsrbr8n-CUSEYyXxGFE3h9PHLHgePjm3_rMK_B_0m5loTpyxX-M=@proton.me>
2026-09-28 13:42 ` AW: BCM4360 AC-PHY support for b43 — reverse-engineered, seeking guidance on a clean path upstream andreas.wendleder
2026-09-28 18:03 ` Michael Büsch
2026-09-28 21:49 ` AW: " andreas.wendleder
2026-09-29 6:14 ` Michael Büsch
2026-09-29 6:32 ` AW: " andreas.wendleder
2026-09-29 7:34 ` Alessio Ferri
2026-09-29 7:49 ` AW: " andreas.wendleder
2026-09-29 8:05 ` Alessio Ferri
2026-09-29 8:27 ` AW: " andreas.wendleder
2026-09-29 8:58 ` Alessio Ferri [this message]
[not found] ` <oYmDL00YinKfSdWHxkjmqPRGHWmeA_0SCR6tpafsgCvGyqYAEKuCt0kg7w_C47be5hm7IwFsxRnulMhvPFJEXQNDaDPgZNzMIQ0vfRKnOLw=@proton.me>
[not found] ` <20260929113109.2c561844@DELL-MOBILE03.ad.smart.it>
2026-09-29 14:45 ` andreas.wendleder
2026-09-29 20:13 ` Alessio Ferri
2026-10-01 1:31 ` AW: " andreas.wendleder
2026-10-01 6:02 ` Alessio Ferri
2026-10-01 12:06 ` AW: " andreas.wendleder
2026-10-01 17:12 ` Alessio Ferri
2026-10-01 17:37 ` Alessio Ferri
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=20260929105859.364ea71b@DELL-MOBILE03.ad.smart.it \
--to=alessio.ferri@mythread.it \
--cc=andreas.wendleder@proton.me \
--cc=b43-dev@lists.infradead.org \
--cc=linux-wireless@vger.kernel.org \
--cc=m@bues.ch \
/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