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 9AD4DC5ACB3 for ; Tue, 21 Nov 2023 10:31:47 +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=78SMtTzj3Csk7guC6pNn8i/imOMFF308d7Q1ndRFViM=; b=Fk4cNx/Y7D1W+ILGF/iw5Dxb+i e2IXn7WXg2HRgJ1YFR/Du0B7Pu/FZmIEKgZup/Zg4mirCSVj7o2P0IQKlJX/aBQsMtD+hdEoYF9sR 0WLLVwgIYR1nCjtkZq/Ed+3HVb9h/7GZB5OAJ6VkySd18dFQDNimyrxSImuOev4tj5TILcNRSGm3c q2YYKcVJtKhaZ9WDbefHpq8x4gzrOIHGTr/EVmJbnC6kb5PTzCrWwFlRcYbxMN7c71rSdeSwMLsSc qUCC1xoycqowDImTMW0WLnE3BDSUxc5qI011pN30+7x8u1Oth6IbiOuvuHfu8m7Ow3Sb5M3W/1BCL xCnf0qlA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1r5O2k-00GGX6-0x; Tue, 21 Nov 2023 10:31:46 +0000 Received: from madras.collabora.co.uk ([2a00:1098:0:82:1000:25:2eeb:e5ab]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1r5O2g-00GGWC-2e; Tue, 21 Nov 2023 10:31:44 +0000 Received: from [100.107.97.3] (cola.collaboradmins.com [195.201.22.229]) (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 madras.collabora.co.uk (Postfix) with ESMTPSA id 3E6EF66072F6; Tue, 21 Nov 2023 10:31:40 +0000 (GMT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1700562701; bh=eAVo/wl10HB6quoWDWzqBLbaFO4CsVTt1yPV5+TaoDY=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=mQ20Oz031InsVrpqHi0kgA/AXyHLX5d9bTw4qTZDREQDWHB3Yj3/gLrQxy29fiGme FcFGvgwa6ZT4ufBtA6iOGsPjZa9fRPQ5+V1oM6o8mcRRm8olQ1ksZ4lph6ob9vw2aK bnK1WkUggHazaT+R0AL4i65IGNtGkoBfgsSMe2gKYLMiO8VBnOtp/TgGsaIBH+RJ1J VwNoNlJzFdR13miP1atJTG06P+ckTykG4fzMTrkdXZjoedJ70xc5opI+cyclGdFXAR ZifobcwJMY658DeSDprZOI40CY4kDSKY1j520zia4xp9WJoBHRbZpKFk9p9KUq+sez OtWhDO4UINipQ== Message-ID: <94165a35-4ce9-4624-938f-00545562fee4@collabora.com> Date: Tue, 21 Nov 2023 11:31:37 +0100 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH 3/3] arm64: dts: mediatek: Add Acelink EW-7886CAX Content-Language: en-US To: =?UTF-8?B?UmFmYcWCIE1pxYJlY2tp?= , Matthias Brugger , Rob Herring , Krzysztof Kozlowski , Conor Dooley Cc: =?UTF-8?Q?N=C3=ADcolas_F_=2E_R_=2E_A_=2E_Prado?= , Macpaul Lin , =?UTF-8?Q?Bernhard_Rosenkr=C3=A4nzer?= , Heiko Stuebner , Jernej Skrabec , Chris Morgan , Linus Walleij , Sean Wang , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, =?UTF-8?B?UmFmYcWCIE1pxYJlY2tp?= References: <20231117104315.9718-2-zajec5@gmail.com> <20231117104315.9718-3-zajec5@gmail.com> <0c3267e5-5371-4fd8-a0f6-360ff28c9dda@collabora.com> From: AngeloGioacchino Del Regno In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20231121_023143_127729_228A9FCA X-CRM114-Status: GOOD ( 20.13 ) 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 20/11/23 18:27, Rafał Miłecki ha scritto: > On 20.11.2023 15:17, AngeloGioacchino Del Regno wrote: >> Il 17/11/23 11:43, Rafał Miłecki ha scritto: >>> From: Rafał Miłecki >>> >>> Acelink EW-7886CAX is an MT7986A (AKA Filogic 830) based access point. >>> It has 512 MiB of RAM, one 2.5 Gbps PoE (802.3at) Ethernet port and >>> on-SoC Wi-Fi. >>> >>> Signed-off-by: Rafał Miłecki >>> --- >>>   arch/arm64/boot/dts/mediatek/Makefile         |   1 + >>>   .../mediatek/mt7986a-acelink-ew-7886cax.dts   | 175 ++++++++++++++++++ >>>   2 files changed, 176 insertions(+) >>>   create mode 100644 arch/arm64/boot/dts/mediatek/mt7986a-acelink-ew-7886cax.dts >>> >>> diff --git a/arch/arm64/boot/dts/mediatek/Makefile >>> b/arch/arm64/boot/dts/mediatek/Makefile >>> index e6e7592a3645..9ff2ab6c5e4d 100644 >>> --- a/arch/arm64/boot/dts/mediatek/Makefile >>> +++ b/arch/arm64/boot/dts/mediatek/Makefile >>> @@ -8,6 +8,7 @@ dtb-$(CONFIG_ARCH_MEDIATEK) += mt6797-evb.dtb >>>   dtb-$(CONFIG_ARCH_MEDIATEK) += mt6797-x20-dev.dtb >>>   dtb-$(CONFIG_ARCH_MEDIATEK) += mt7622-rfb1.dtb >>>   dtb-$(CONFIG_ARCH_MEDIATEK) += mt7622-bananapi-bpi-r64.dtb >>> +dtb-$(CONFIG_ARCH_MEDIATEK) += mt7986a-acelink-ew-7886cax.dtb >>>   dtb-$(CONFIG_ARCH_MEDIATEK) += mt7986a-bananapi-bpi-r3.dtb >>>   dtb-$(CONFIG_ARCH_MEDIATEK) += mt7986a-bananapi-bpi-r3-emmc.dtbo >>>   dtb-$(CONFIG_ARCH_MEDIATEK) += mt7986a-bananapi-bpi-r3-nand.dtbo >>> diff --git a/arch/arm64/boot/dts/mediatek/mt7986a-acelink-ew-7886cax.dts >>> b/arch/arm64/boot/dts/mediatek/mt7986a-acelink-ew-7886cax.dts >>> new file mode 100644 >>> index 000000000000..18d19281dfdb >>> --- /dev/null >>> +++ b/arch/arm64/boot/dts/mediatek/mt7986a-acelink-ew-7886cax.dts >>> @@ -0,0 +1,175 @@ >>> +// SPDX-License-Identifier: GPL-2.0-only OR MIT >>> + >>> +/dts-v1/; >>> +#include >>> +#include >>> +#include >>> + >>> +#include "mt7986a.dtsi" >>> + >>> +/ { >>> +    model = "Acelink EW-7886CAX"; >>> +    compatible = "acelink,ew-7886cax", "mediatek,mt7986a"; >>> + >>> +    aliases { >>> +        serial0 = &uart0; >>> +    }; >>> + >>> +    chosen { >>> +        stdout-path = "serial0:115200n8"; >>> +    }; >>> + >>> +    memory@40000000 { >>> +        reg = <0 0x40000000 0 0x20000000>; >>> +        device_type = "memory"; >>> +    }; >>> + >>> +    keys { >>> +        compatible = "gpio-keys"; >>> + >>> +        key-restart { >>> +            label = "Reset"; >>> +            gpios = <&pio 7 GPIO_ACTIVE_LOW>; >>> +            linux,code = ; >>> +        }; >>> +    }; >>> + >>> +    leds { >>> +        compatible = "gpio-leds"; >>> + >>> +        led-0 { >> >> Please, reorder by name >> >>              color =    ... >>              function = ... >>              gpios = ... > > Can you explain why and if there is a place I can find rules to follow > regarding such aspects? I really would like to just be aware of all > rules and don't waste anyone's time for such details. > > FWIW I checked Documentation/devicetree/bindings/*.rst (after few years > I admit) but I couldn't find anything there about properties order. > > If we currently don't have rules I don't really think we should enforce > following per-maintainer preferences. I really don't object your > suggestions but there is simply no way to remember each maintainer's > rules. We simply have too many subsystems and architectures boards. > > What I said is not a maintainer preference but more of a general rule, which is in the process of being documented. Check the patch there, at the time of writing, v2: https://lore.kernel.org/all/20231120084044.23838-1-krzysztof.kozlowski@linaro.org/ Regards, Angelo