Linux MIPS Architecture development
 help / color / mirror / Atom feed
From: Florian Fainelli <florian.fainelli@broadcom.com>
To: Alessio Ferri <alessio.ferri@mythread.it>,
	Johannes Berg <johannes@sipsolutions.net>
Cc: "Rob Herring" <robh@kernel.org>,
	"Krzysztof Kozlowski" <krzk+dt@kernel.org>,
	"Conor Dooley" <conor+dt@kernel.org>,
	"Rafał Miłecki" <zajec5@gmail.com>,
	"Philipp Zabel" <p.zabel@pengutronix.de>,
	"Broadcom internal kernel review list"
	<bcm-kernel-feedback-list@broadcom.com>,
	"Thomas Bogendoerfer" <tsbogend@alpha.franken.de>,
	linux-wireless@vger.kernel.org, devicetree@vger.kernel.org,
	linux-kernel@vger.kernel.org, linux-mips@vger.kernel.org
Subject: Re: [PATCH v2 0/5] bcma: support the BCM6362 integrated WLAN backplane
Date: Tue, 13 Oct 2026 09:25:30 -0700	[thread overview]
Message-ID: <6ceddef2-09af-49ba-87c8-4c39e34d36ac@broadcom.com> (raw)
In-Reply-To: <20261013181600.58ad8dc3@DELL-MOBILE03.ad.smart.it>

On 10/13/26 09:16, Alessio Ferri wrote:
> On Tue, 13 Oct 2026 14:56:30 +0200
> Alessio Ferri <alessio.ferri@mythread.it> wrote:
> 
>> On Tue, 13 Oct 2026 14:29:37 +0200
>> Johannes Berg <johannes@sipsolutions.net> wrote:
>>
>>> On Mon, 2026-10-12 at 00:45 +0200, Alessio Ferri wrote:
>>>>
>>>> My router is running daily with it, using openwrt kernel 6.12.
>>>
>>> [...]
>>>    
>>>> This was drafted with the help of an LLM and cross built with gcc
>>>> mips cross compiler and checked with the dev version of sparse.
>>>> This time i also ran dt-check-style and it didn't flag anything.
>>>>    
>>>
>>> So ... are you running it or not? I'm confused.
>>>    
>>
>> I don't see your source of confusion: i am running it with kernel 6.12
>> and it works, but last i checked the current kernel is not version
>> 6.12... so i had to rebuild it for the current kernel and i did it
>> with help from an LLM. And also i ran the checks for the dt part,
>> OpenWRT build system couldn't care less that i didn't even register
>> the bindings, but i know bindings maintainer care about it.
>>
>>> Either way, I have no idea what to do with it - I guess Rafał if you
>>> want this picked up then let folks know? But the last patch should
>>> probably go through some MIPS trees anyway, so the others might as
>>> well go there too, I don't care (and don't really _want_ to be
>>> responsible for this obscure corner just because some wifi drivers
>>> use it).
>>>
>>> johannes
>>
>> The original code is Broadcom's. You can also ask Broadcom directly if
>> this is correct/sane for them.
> 
> I must say that i was pretty vague in the last statement:
> By "code" i really meant "C headers", and by "ask Broadcom directly" i
> meant "ping the Broadcom devs already included in the cc if they do not
> respond by themselves in a few days".

As you should know by now, the folks working on this never cared for 
upstream in any way shape or form, so using bcma and Device Tree, and 
combining both might be a tad too much to ask to review.

That said, I think you approach makes sense, but Jonas Gorski who is not 
copied on this patch series, but is now aware of it, should really be 
reviewing this.
-- 
Florian

  reply	other threads:[~2026-10-13 16:25 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-10-11 22:45 [PATCH v2 0/5] bcma: support the BCM6362 integrated WLAN backplane Alessio Ferri
2026-10-11 22:45 ` [PATCH v2 1/5] dt-bindings: bus: add Broadcom BCM6362 WLAN AXI backplane Alessio Ferri
2026-10-12 19:13   ` Conor Dooley
2026-10-13 16:09     ` Alessio Ferri
2026-10-13 16:42       ` Conor Dooley
2026-10-11 22:45 ` [PATCH v2 2/5] bcma: scan: support big-endian backplanes Alessio Ferri
2026-10-11 22:45 ` [PATCH v2 3/5] bcma: scan: accept wrapper-less cores on wrapper-less backplanes Alessio Ferri
2026-10-11 22:45 ` [PATCH v2 4/5] bcma: add host driver for the BCM6362 integrated WLAN backplane Alessio Ferri
2026-10-11 22:45 ` [PATCH v2 5/5] MIPS: dts: bcm6362: add the " Alessio Ferri
2026-10-13 12:29 ` [PATCH v2 0/5] bcma: support the BCM6362 " Johannes Berg
2026-10-13 12:56   ` Alessio Ferri
2026-10-13 16:16     ` Alessio Ferri
2026-10-13 16:25       ` Florian Fainelli [this message]
2026-10-13 17:26         ` 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=6ceddef2-09af-49ba-87c8-4c39e34d36ac@broadcom.com \
    --to=florian.fainelli@broadcom.com \
    --cc=alessio.ferri@mythread.it \
    --cc=bcm-kernel-feedback-list@broadcom.com \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=johannes@sipsolutions.net \
    --cc=krzk+dt@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mips@vger.kernel.org \
    --cc=linux-wireless@vger.kernel.org \
    --cc=p.zabel@pengutronix.de \
    --cc=robh@kernel.org \
    --cc=tsbogend@alpha.franken.de \
    --cc=zajec5@gmail.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