From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 62C6ECCF9FE for ; Mon, 3 Nov 2025 08:56:30 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: Content-Type:In-Reply-To:From:References:Cc:To:Subject:MIME-Version:Date: Message-ID:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=58f5QKwX2Ejxemztq5REJDukN0YTDxePOufXYt3Rb78=; b=kzr6Nj+WPBRsIQnKe/Z+HTD6Zb 837mO63GLJq2V6rjTNM7l1wTRNxoxfxLLbHxpvkfqS+zgBqXKI4pghr6quxBPAANTsf936uXBpTMM BR0SqKqw5r0wJ1/8YlFklTiSWH5qq+QvD+cKq/ytF8fMi7/eS9Cev8/uujtla+g/WO5k+tvPaUKiz g+UCmp5gnKcLT/gRRuvG7RDsLfDM3YrN/jvsdI+f7d6oI+6lY8NHDCJ3k+1Bvc2B5hH2gG4b1tHLa bJ9jMKIjmkDgf6qvgvRJZBqPA33kK7ZBn/2Lsi9opxvh9Dur4CcxfCyBvkt0mm5idLV9TWg2xkjwV tBwXHfcw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1vFqMW-00000009TNQ-0zSl; Mon, 03 Nov 2025 08:56:28 +0000 Received: from bali.collaboradmins.com ([2a01:4f8:201:9162::2]) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1vFqMT-00000009TMt-20S9; Mon, 03 Nov 2025 08:56:26 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1762160182; bh=lCXRosTn4Mj/QrN4xjJ+JLcI6xF053GP6NbQY85J2pM=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=oTTeiws+XDqPYCSAEBmgaVpzZ/YfXq9FkPZ0Ey/u6YtOJFo2pQ8gPQkabyV+BBR12 fz6LC96S19XknHxdj2DUI0F8l1O1a0C+GYaQNbF3yBHbiuQGgFIjZzjOEQ0ptxuOBx B5GezsczBTGxMtcz57f7Wg3IQ/dcNXDvkDztQ+Npaqq2vNkVI/Dk3efXhRjprTm6hJ zyFSnD5NUR9X3y7HZ6beHMKygzQMpPftJV1ubpxa/1x1i7ePTI13uaLOVvSx9Rfu9F a543B8xvlujZowXMyWLBxiRQ34JLWQsacceKV0/NMhP71UlICEL3/tdHrujknqESGg YMOzXRu/boqpw== Received: from [192.168.1.100] (2-237-20-237.ip236.fastwebnet.it [2.237.20.237]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) (Authenticated sender: kholk11) by bali.collaboradmins.com (Postfix) with ESMTPSA id 8B4BE17E1091; Mon, 3 Nov 2025 09:56:21 +0100 (CET) Message-ID: <781e754e-6603-4ae3-9340-24403a6d8137@collabora.com> Date: Mon, 3 Nov 2025 09:56:21 +0100 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v2 00/15] arm64: dts: mediatek: Add Openwrt One AP functionality To: Sjoerd Simons , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Matthias Brugger , Ryder Lee , Jianjun Wang , Bjorn Helgaas , Lorenzo Pieralisi , =?UTF-8?Q?Krzysztof_Wilczy=C5=84ski?= , Manivannan Sadhasivam , Chunfeng Yun , Vinod Koul , Kishon Vijay Abraham I , Lee Jones , Andrew Lunn , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Lorenzo Bianconi , Felix Fietkau Cc: kernel@collabora.com, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, linux-pci@vger.kernel.org, linux-phy@lists.infradead.org, netdev@vger.kernel.org, Daniel Golle , Bryan Hinton , Conor Dooley References: <20251101-openwrt-one-network-v2-0-2a162b9eea91@collabora.com> From: AngeloGioacchino Del Regno Content-Language: en-US In-Reply-To: <20251101-openwrt-one-network-v2-0-2a162b9eea91@collabora.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20251103_005625_725185_81A70C00 X-CRM114-Status: GOOD ( 26.60 ) X-BeenThere: linux-mediatek@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "Linux-mediatek" Errors-To: linux-mediatek-bounces+linux-mediatek=archiver.kernel.org@lists.infradead.org Il 01/11/25 14:32, Sjoerd Simons ha scritto: > Significant changes in V2: > * https://lore.kernel.org/lkml/20251016-openwrt-one-network-v1-0-de259719b6f2@collabora.com/ > * Only introduce labels in mt7981b.dtsi when required > * Switch Airoha EN8811H phy irq to level rather then edge triggered > * Move uart0 pinctrl from board dts to soc dtsi > * Only overwrite constraints with non-default values in MT7981 bindings > * Make SPI NOR nvmem cell labels more meaningfull > * Seperate fixing and disable-by-default for the mt7981 in seperate > patches > > This series add various peripherals to the Openwrt One, to make it > actually useful an access point: > > * Pcie express (tested with nvme storage) > * Wired network interfaces > * Wireless network interfaces (2.4g, 5ghz wifi) > * Status leds > * SPI NOR for factory data > > Unsurprisingly the series is a mix of dt binding updates, extensions of > the mt7981b and the openwrt one dtb. All driver support required is > already available. > > Sadly during testing i've found various quirks requiring kernel > arguments. Documenting those here both as note to self and making it > easier for others to test :) > > * fw_devlink=permissive: the nvmem fixed-layout doesn't create a layout > device, so doesn't trigger fw_devlink This should really be fixed in fw_devlink I believe. It's not the first device that uses nvmem fixed-layout and will not be the last one. > * clk_ignore_unused: Needed when building CONFIG_NET_MEDIATEK_SOC as a > module. If the ethernet related clocks (gp1/gp2) get disabled the > mac ends up in a weird state causing it not to function correctly. I'm sure that this can be resolved by adding resets. > * pcie_aspm: ASPM is forced to enabled in 6.18-rc1, unfortunately > enabling ASPM L1.1 ends up triggering unrecoverable AERs. That must be resolved in the PCIe driver - either it must disable L1.1 support or needs some fixes around. Still, I think if you add resets to the PCIe node you should at least get the MAC recovered at PM resume time (but being this a router, I really don't think that this would matter - still, for the sake of completion...) Overall, this series is good and I'm fine with picking all of the changes: even if there are some needed quirks, those aren't freezing the boot process and the worst thing that could ever happen is that in some conditions some devices will simply not probe. I guess that the resets, etc, can be added later as a fix - but at least we can get those devices at least partially up and running . Can anyone pick the bindings please, so that I can pick everything else? Cheers, Angelo > > Patches are against the mediatek trees for-next branch > > Signed-off-by: Sjoerd Simons > --- > Sjoerd Simons (15): > arm64: dts: mediatek: mt7981b: Configure UART0 pinmux > arm64: dts: mediatek: mt7981b: Add reserved memory for TF-A > dt-bindings: mfd: syscon: Add mt7981-topmisc > dt-bindings: PCI: mediatek-gen3: Add MT7981 PCIe compatible > dt-bindings: phy: mediatek,tphy: Add support for MT7981 > arm64: dts: mediatek: mt7981b: Add PCIe and USB support > arm64: dts: mediatek: mt7981b-openwrt-one: Enable PCIe and USB > dt-bindings: net: mediatek,net: Correct bindings for MT7981 > arm64: dts: mediatek: mt7981b: Add Ethernet and WiFi offload support > arm64: dts: mediatek: mt7981b-openwrt-one: Enable SPI NOR > arm64: dts: mediatek: mt7981b-openwrt-one: Enable Ethernet > arm64: dts: mediatek: mt7981b: Disable wifi by default > arm64: dts: mediatek: mt7981b: Add wifi memory region > arm64: dts: mediatek: mt7981b-openwrt-one: Enable wifi > arm64: dts: mediatek: mt7981b-openwrt-one: Enable software leds > > Documentation/devicetree/bindings/mfd/syscon.yaml | 1 + > .../devicetree/bindings/net/mediatek,net.yaml | 13 +- > .../bindings/pci/mediatek-pcie-gen3.yaml | 1 + > .../devicetree/bindings/phy/mediatek,tphy.yaml | 1 + > .../boot/dts/mediatek/mt7981b-openwrt-one.dts | 263 +++++++++++++++++++++ > arch/arm64/boot/dts/mediatek/mt7981b.dtsi | 247 ++++++++++++++++++- > 6 files changed, 519 insertions(+), 7 deletions(-) > --- > base-commit: 860a0efbb95de468b17c86ed5cf8d90ee4bc5d7b > change-id: 20251016-openwrt-one-network-40bc9ac1b25c > > Best regards,