Re: [PATCH net-next 12/12] arm64: dts: apple: Add WiFi module and antenna properties

From: Kalle Valo <kvalo@kernel.org>
To: "Russell King \(Oracle\)" <linux@armlinux.org.uk>
Cc: Arend van Spriel <aspriel@gmail.com>,
	 Franky Lin <franky.lin@broadcom.com>,
	 Hante Meuleman <hante.meuleman@broadcom.com>,
	 Alyssa Rosenzweig <alyssa@rosenzweig.io>,
	 asahi@lists.linux.dev,  brcm80211-dev-list.pdl@broadcom.com,
	 "David S. Miller" <davem@davemloft.net>,
	 devicetree@vger.kernel.org,  Eric Dumazet <edumazet@google.com>,
	 Hector Martin <marcan@marcan.st>,
	 Jakub Kicinski <kuba@kernel.org>,
	 Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
	 linux-arm-kernel@lists.infradead.org,
	 linux-wireless@vger.kernel.org,  netdev@vger.kernel.org,
	 Paolo Abeni <pabeni@redhat.com>,
	 Rafa__ Mi__ecki <zajec5@gmail.com>,
	 Rob Herring <robh+dt@kernel.org>,
	 SHA-cyfmac-dev-list@infineon.com,
	 Sven Peter <sven@svenpeter.dev>
Subject: Re: [PATCH net-next 12/12] arm64: dts: apple: Add WiFi module and antenna properties
Date: Fri, 16 Sep 2022 10:27:31 +0300	[thread overview]
Message-ID: <87edwbaie4.fsf@kernel.org> (raw)
In-Reply-To: <YyCvkrDgsFLYNZ9t@shell.armlinux.org.uk> (Russell King's message of "Tue, 13 Sep 2022 17:28:02 +0100")

"Russell King (Oracle)" <linux@armlinux.org.uk> writes:

> On Wed, Sep 07, 2022 at 11:16:22AM +0300, Kalle Valo wrote:
>> Russell King (Oracle) <rmk+kernel@armlinux.org.uk> writes:
>> 
>> > From: Hector Martin <marcan@marcan.st>
>> >
>> > Add the new module-instance/antenna-sku properties required to select
>> > WiFi firmwares properly to all board device trees.
>> >
>> > Signed-off-by: Hector Martin <marcan@marcan.st>
>> > Signed-off-by: Russell King (Oracle) <rmk+kernel@armlinux.org.uk>
>> > ---
>> >  arch/arm64/boot/dts/apple/t8103-j274.dts  | 4 ++++
>> >  arch/arm64/boot/dts/apple/t8103-j293.dts  | 4 ++++
>> >  arch/arm64/boot/dts/apple/t8103-j313.dts  | 4 ++++
>> >  arch/arm64/boot/dts/apple/t8103-j456.dts  | 4 ++++
>> >  arch/arm64/boot/dts/apple/t8103-j457.dts  | 4 ++++
>> >  arch/arm64/boot/dts/apple/t8103-jxxx.dtsi | 2 ++
>> >  6 files changed, 22 insertions(+)
>> 
>> Is it ok to take this via wireless-next? Can I get an ack from the
>> maintainers of these files?
>
> I'm not sure who you're expecting to get an ack from.

The problem is that I don't know how .dts files are merged, that's why
I'm extra careful that I'm not breaking the normal flow for them.

> If it's the maintainers of these files, that'll be Hector himself, and
> as he authored the change, there seems to be little point in also
> having an Acked-by from him too.
>
> I just asked Hector on #asahi-dev:
> 17:21 < rmk> also, I think Kalle Valo is waiting on an answer on the arm64 DTS
>              changes for brcmfmac:
> 17:21 < rmk> Is it ok to take this via wireless-next? Can I get an ack from the
> 17:21 < rmk> maintainers of these files?
> 17:21 <@marcan> ah yeah, merging via wireless-next is fine, let me give you an
>                 ack

That sounds good to me, thanks!

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

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