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 016E6C77B7A for ; Tue, 30 May 2023 13:32:42 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id: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=yD2Q6Ip/qxc1UXmap2t3rjOls8wmPBnbcgQnagcJbR8=; b=m1CP5A1wN2Nhr5 yDDmqIqiLyKu2cjT3nOnfg6EdAPlwpvpGS38EJLnJFkkueXiJY/NQYniJ+vJ16vljV9JWqw1Efpbe IUnadaxcpFcFKSn1ZeQ2lnrMRpdpfx5Nn221XhD7vqEFWDyT5S9HHX/lQVNpUQ2oTm1MYMQWdmgAp AGVXhjVTOq6KCqna72xc9yGvbMM702EJzj9QBUP+6X5InG4De0zuhAUl6PZsFkeIeHMRcMBBw4zOQ xOtBgmx/oPJAQ6ZEV94C/YewIV6JNXaorxvudk0isJ3P5SmSZxF1rgcEuKNh1sBwRB7XckEbEZe7f bgfJ+gQX+zYlXehDyu7g==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1q3zSW-00E3JW-0W; Tue, 30 May 2023 13:32:20 +0000 Received: from fllv0016.ext.ti.com ([198.47.19.142]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1q3zST-00E3Ir-1Y for linux-arm-kernel@lists.infradead.org; Tue, 30 May 2023 13:32:19 +0000 Received: from fllv0035.itg.ti.com ([10.64.41.0]) by fllv0016.ext.ti.com (8.15.2/8.15.2) with ESMTP id 34UDW1CU010504; Tue, 30 May 2023 08:32:01 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ti.com; s=ti-com-17Q1; t=1685453521; bh=TVuo4D7cz4/tEVfuFzYLxXwg0NBeCVTRyOotng1WRcs=; h=Date:Subject:To:CC:References:From:In-Reply-To; b=SaIiBja+wsle1bsWMjGbHRz9yQalDYwsCqXj/8iXi4GH26M13G6np3C79uhB1z7fH 8aKOJoZxnNtpJDtnqFE6X2Yh6VYvIXFc0Q0TItAVyAsfAthDA+mcVT1VoqULm+vpWk 11Bl7CsVkx129Z3NkzeaOrW8BAe2bzXoysczXnig= Received: from DLEE105.ent.ti.com (dlee105.ent.ti.com [157.170.170.35]) by fllv0035.itg.ti.com (8.15.2/8.15.2) with ESMTPS id 34UDW17I063353 (version=TLSv1.2 cipher=AES256-GCM-SHA384 bits=256 verify=FAIL); Tue, 30 May 2023 08:32:01 -0500 Received: from DLEE108.ent.ti.com (157.170.170.38) by DLEE105.ent.ti.com (157.170.170.35) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2507.23; Tue, 30 May 2023 08:32:01 -0500 Received: from fllv0040.itg.ti.com (10.64.41.20) by DLEE108.ent.ti.com (157.170.170.38) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2507.23 via Frontend Transport; Tue, 30 May 2023 08:32:01 -0500 Received: from [10.249.142.56] (ileaxei01-snat2.itg.ti.com [10.180.69.6]) by fllv0040.itg.ti.com (8.15.2/8.15.2) with ESMTP id 34UDVvu7012242; Tue, 30 May 2023 08:31:58 -0500 Message-ID: Date: Tue, 30 May 2023 19:01:57 +0530 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101 Thunderbird/102.10.0 Subject: Re: [PATCH v1 3/5] arm64: dts: ti: add verdin am62 To: Nishanth Menon , Francesco Dolcini CC: Francesco Dolcini , Tero Kristo , Rob Herring , Krzysztof Kozlowski , Conor Dooley , , , References: <20230524143631.42471-1-francesco@dolcini.it> <20230524143631.42471-4-francesco@dolcini.it> <20230530121044.sjhv452b4hs4lyiy@flyer> Content-Language: en-US From: "Raghavendra, Vignesh" In-Reply-To: <20230530121044.sjhv452b4hs4lyiy@flyer> X-EXCLAIMER-MD-CONFIG: e1e8a2fd-e40a-4ac6-ac9b-f7e9cc9ee180 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230530_063217_657686_1FA02199 X-CRM114-Status: GOOD ( 17.46 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi On 5/30/2023 5:40 PM, Nishanth Menon wrote: > On 16:36-20230524, Francesco Dolcini wrote: > [...] >> diff --git a/arch/arm64/boot/dts/ti/k3-am62-verdin-dev.dtsi b/arch/arm64/boot/dts/ti/k3-am62-verdin-dev.dtsi >> new file mode 100644 >> index 000000000000..e138b1c8ed14 >> --- /dev/null >> +++ b/arch/arm64/boot/dts/ti/k3-am62-verdin-dev.dtsi >> @@ -0,0 +1,233 @@ >> +// SPDX-License-Identifier: GPL-2.0-or-later OR MIT >> +/* >> + * Copyright 2023 Toradex > > Please also add appropriate product links to dts/dtsi > >> + */ >> + >> +/ { >> + sound_card: sound-card { >> + compatible = "simple-audio-card"; >> + simple-audio-card,bitclock-master = <&dailink_master>; >> + simple-audio-card,format = "i2s"; >> + simple-audio-card,frame-master = <&dailink_master>; >> + simple-audio-card,name = "verdin-nau8822"; >> + simple-audio-card,routing = >> + "Headphones", "LHP", >> + "Headphones", "RHP", >> + "Speaker", "LSPK", >> + "Speaker", "RSPK", >> + "Line Out", "AUXOUT1", >> + "Line Out", "AUXOUT2", >> + "LAUX", "Line In", >> + "RAUX", "Line In", >> + "LMICP", "Mic In", >> + "RMICP", "Mic In"; >> + simple-audio-card,widgets = >> + "Headphones", "Headphones", >> + "Line Out", "Line Out", >> + "Speaker", "Speaker", >> + "Microphone", "Mic In", >> + "Line", "Line In"; >> + >> + dailink_master: simple-audio-card,codec { >> + clocks = <&k3_clks 157 10>; >> + sound-dai = <&nau8822_1a>; >> + }; >> + >> + simple-audio-card,cpu { >> + sound-dai = <&mcasp0>; >> + }; >> + }; >> +}; >> + >> +/* Verdin ETHs */ >> +&cpsw3g { >> + pinctrl-names = "default"; >> + pinctrl-0 = <&pinctrl_rgmii1 &pinctrl_rgmii2>; > > here and elsewhere: > pinctrl-0 = <&pinctrl_rgmii1>, <&pinctrl_rgmii2>; > > >> + status = "okay"; >> +}; > [...] >> + /* EEPROM */ >> + eeprom@57 { >> + compatible = "st,24c02", "atmel,24c02"; > > checkpatch warns: DT compatible string "st,24c02" appears un-documented > Checkpatch now seems outdated in favor of dtbs_check. DT schemas use regex for compatibles and thus simple grep in Doucmentation/devicetree-bindings/ doesn't help. Eg: In this case: Documentation/devicetree/bindings/eeprom/at24.yaml has the regex to cover the compatibles >> + reg = <0x57>; >> + pagesize = <16>; >> + }; >> +}; >> + >> +/* Verdin I2C_2_DSI */ >> +&main_i2c2 { >> + status = "okay"; > > Here and few other dtsis: > you should set status along with pinmux. > > [...] >> + >> +/* Verdin UART_2 */ >> +&wkup_uart0 { >> + /* FIXME: WKUP UART0 is used by DM firmware */ >> + status = "reserved"; > > If you do configure this in R5 SPL, you'd want to add the pinmux as > well. > >> +}; >> diff --git a/arch/arm64/boot/dts/ti/k3-am62-verdin-wifi.dtsi b/arch/arm64/boot/dts/ti/k3-am62-verdin-wifi.dtsi >> new file mode 100644 >> index 000000000000..289db1666fc0 >> --- /dev/null >> +++ b/arch/arm64/boot/dts/ti/k3-am62-verdin-wifi.dtsi >> @@ -0,0 +1,36 @@ >> +// SPDX-License-Identifier: GPL-2.0-or-later OR MIT >> +/* >> + * Copyright 2023 Toradex >> + */ >> + >> +/ { >> + wifi_pwrseq: wifi-pwrseq { >> + compatible = "mmc-pwrseq-simple"; >> + pinctrl-names = "default"; >> + pinctrl-0 = <&pinctrl_wifi_en>; >> + reset-gpios = <&main_gpio0 22 GPIO_ACTIVE_LOW>; >> + }; >> +}; >> + >> + > > Drop extra EoLs. > > [...] > >> diff --git a/arch/arm64/boot/dts/ti/k3-am62-verdin.dtsi b/arch/arm64/boot/dts/ti/k3-am62-verdin.dtsi >> new file mode 100644 >> index 000000000000..2e7cb607df45 >> --- /dev/null >> +++ b/arch/arm64/boot/dts/ti/k3-am62-verdin.dtsi >> @@ -0,0 +1,1400 @@ >> +// SPDX-License-Identifier: GPL-2.0-or-later OR MIT > > Assuming you really intent this instead of purely GPL-2.0 > > [...] > >> +/* MDIO, shared by Verdin ETH_1 (On-module PHY) and Verdin ETH_2_RGMII */ >> +&cpsw3g_mdio { >> + assigned-clocks = <&k3_clks 157 20>; >> + assigned-clock-parents = <&k3_clks 157 22>; >> + assigned-clock-rates = <25000000>; >> + pinctrl-names = "default"; >> + pinctrl-0 = <&pinctrl_eth_clock &pinctrl_mdio>; >> + status = "disabled"; >> + >> + cpsw3g_phy0: ethernet-phy@0 { >> + compatible = "ethernet-phy-id2000.a231"; > > Check binding - we don't include any compatibles that dont have yaml > conversion done (pinctrl is the only exception). Same here ;) Documentation/devicetree/bindings/net/ethernet-phy.yaml covers it > > [...] > >> +/* TODO: Verdin DSI_1 / TIDSS */ > Drop the TODO. > > [...] > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel