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 8F6BEC4332F for ; Thu, 1 Dec 2022 22:58:38 +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:Message-ID:Date:To:Cc:From:Subject: References:In-Reply-To:MIME-Version:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=+dI/eWRdrzRuOqLSp+YXIgxrTtYmrp0R2RAveVb1H1o=; b=nzp0YC+m4JazhF s73YHvV1cEzeyGuLCTpazlrAXhONy2qrUaYtWhAkoQ/tA6fiUArH/V83a6bPbug5EYo394JyAoi1W UGl84IFOnaF01gsgDs5ald0OfOVMQFaR1oRT6+XPtPB/opX0Z5rPkwtCcJfHA6kA0HIQZtdqvnONG TU02DLx6oFhZidRvJKE1nsVzspHyl/YEFfIu9r1NCw/MPH3uvNWBGrmiEBB6ZZDn2a9dqcVu14jSi 1MuJyKmHQG2nmrPMg4CMumlFu0nGLxjMyZ/QK28HoSoS/4XnUPSVt5fLKhy0P6IGFQhuDPgUJjYr9 m+YYyHZi6k2iTDCeaZfA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1p0sUq-00BSSE-Rw; Thu, 01 Dec 2022 22:57:36 +0000 Received: from perceval.ideasonboard.com ([213.167.242.64]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1p0r1H-00B8B3-8H for linux-arm-kernel@lists.infradead.org; Thu, 01 Dec 2022 21:23:01 +0000 Received: from pendragon.ideasonboard.com (cpc89244-aztw30-2-0-cust3082.18-1.cable.virginm.net [86.31.172.11]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id 54AF433F; Thu, 1 Dec 2022 22:22:50 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1669929770; bh=pDWhxoN/Md3mKSG+3r96ylw1SbXPZl82x+f8fxaD2kI=; h=In-Reply-To:References:Subject:From:Cc:To:Date:From; b=iBIb/BPfYA3reSgOomq5YQfEfcUG7LiCjZKDtAZJp6v8TsSuXqlm+/zu+ZhLIs0o0 xlfgZZn2eDrTZaImIDxPuqtVRcmt7C+XUp+5CXlZM1v+N71mr7WgzgkivUDLRqR1HV xTPBZtYe3ZR7fXoEDNnMcwiJokD4BYG37z1qjIDA= MIME-Version: 1.0 In-Reply-To: <23e61494-5567-5701-3a90-3b8105b4c944@pengutronix.de> References: <20221017151050.2321919-1-dan.scally@ideasonboard.com> <20221017151050.2321919-4-dan.scally@ideasonboard.com> <23e61494-5567-5701-3a90-3b8105b4c944@pengutronix.de> Subject: Re: [PATCH v4 3/3] arm64: dts: Add device tree for the Debix Model A Board From: Kieran Bingham Cc: s.hauer@pengutronix.de, debix-tech@polyhex.net, linux-imx@nxp.com, kernel@pengutronix.de, festevam@gmail.com, laurent.pinchart@ideasonboard.com To: Ahmad Fatoum , Daniel Scally , devicetree@vger.kernel.org, krzysztof.kozlowski@linaro.org, leoyang.li@nxp.com, linux-arm-kernel@lists.infradead.org, marcel.ziswiler@toradex.com, robh@kernel.org, shawnguo@kernel.org Date: Thu, 01 Dec 2022 21:22:47 +0000 Message-ID: <166992976772.1079859.10476322048669145302@Monstersaurus> User-Agent: alot/0.10 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20221201_132259_490722_AF2F2E2C X-CRM114-Status: GOOD ( 26.32 ) 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 Quoting Ahmad Fatoum (2022-12-01 17:10:20) > Hello Daniel, > > On 17.10.22 17:10, Daniel Scally wrote: > > Add a device tree file describing the Debix Model A board from > > Polyhex Technology Co. > > Thanks for your patch. Some minor comments below. > > > Changes in v3 (Laurent): > > > > - Added IOB copyright notice > > - Removed the eth node for the connector that's on the separate I/O > > board > > I'd have left the FEC node in and described the PHY, but left the FEC disabled. > Only the magnetics are on the expansion board, while the PHY is on the > base board. > > > +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) > > +/* > > + * Copyright 2019 NXP > > + * Copyright 2022 Ideas on Board Oy > > + */ > > + > > +/dts-v1/; > > + > > +#include > > +#include > > +#include > > + > > +#include "imx8mp.dtsi" > > + > > +/ { > > + model = "Polyhex Debix Model A i.MX8MPlus board"; > > + compatible = "polyhex,imx8mp-debix-model-a", "fsl,imx8mp"; > > I see that Model A and Model B share the same SoC and PCB. Could you > add polyhex,imx8mp-debix as a second compatible? That way, bootloader > may match against that compatible when they support both. > You'll need to adjust the binding accordingly. Polyhex also make a SOM with the brand DEBIX. - https://debix.io/hardware/debix-som-a-io-board.html But perhaps it will be fine, as that will be "polyhex,imx8mp-debix-som" (and perhaps they'll do an A/B variant too?) -- Kieran > > > + > > + chosen { > > + stdout-path = &uart2; > > + }; > > + > > + gpio-leds { > > + compatible = "gpio-leds"; > > + pinctrl-names = "default"; > > + pinctrl-0 = <&pinctrl_gpio_led>; > > + > > + status-led { > > + function = LED_FUNCTION_POWER; > > + color = ; > > + gpios = <&gpio3 16 GPIO_ACTIVE_HIGH>; > > + default-state = "on"; > > + }; > > + }; > > + > > + reg_usdhc2_vmmc: regulator-usdhc2 { > > + compatible = "regulator-fixed"; > > + pinctrl-names = "default"; > > + pinctrl-0 = <&pinctrl_reg_usdhc2_vmmc>; > > + regulator-name = "VSD_3V3"; > > + regulator-min-microvolt = <3300000>; > > + regulator-max-microvolt = <3300000>; > > + gpio = <&gpio2 19 GPIO_ACTIVE_HIGH>; > > + enable-active-high; > > + }; > > +}; > > + > > +&A53_0 { > > + cpu-supply = <&buck2>; > > +}; > > + > > +&A53_1 { > > + cpu-supply = <&buck2>; > > +}; > > + > > +&A53_2 { > > + cpu-supply = <&buck2>; > > +}; > > + > > +&A53_3 { > > + cpu-supply = <&buck2>; > > +}; > > + > > +&eqos { > > + pinctrl-names = "default"; > > + pinctrl-0 = <&pinctrl_eqos>; > > + phy-connection-type = "rgmii-id"; > > + phy-handle = <ðphy0>; > > + status = "okay"; > > + > > + mdio { > > + compatible = "snps,dwmac-mdio"; > > + #address-cells = <1>; > > + #size-cells = <0>; > > + > > + ethphy0: ethernet-phy@0 { > > Could you append a /* RTL8211E */ comment here? This can be very useful for others > who need to bring up the same chip in the future. > > > + compatible = "ethernet-phy-ieee802.3-c22"; > > + reg = <0>; > > Is the PHY really at address 0 or does it just answer at this address > because it's the broadcast address? > > > > +&iomuxc { > > + pinctrl-names = "default"; > > + pinctrl-0 = <&pinctrl_hog>; > > + > > + pinctrl_hog: hoggrp { > > + fsl,pins = < > > + MX8MP_IOMUXC_HDMI_DDC_SCL__HDMIMIX_HDMI_SCL 0x400001c3 > > + MX8MP_IOMUXC_HDMI_DDC_SDA__HDMIMIX_HDMI_SDA 0x400001c3 > > + MX8MP_IOMUXC_HDMI_HPD__HDMIMIX_HDMI_HPD 0x40000019 > > + MX8MP_IOMUXC_HDMI_CEC__HDMIMIX_HDMI_CEC 0x40000019 > > Why do you hog these? > > > + pinctrl_usb1_vbus: usb1grp { > > This is unused. > > > + pinctrl_usdhc2: usdhc2grp { > > + fsl,pins = < > > + MX8MP_IOMUXC_SD2_CLK__USDHC2_CLK 0x190 > > + MX8MP_IOMUXC_SD2_CMD__USDHC2_CMD 0x1d0 > > + MX8MP_IOMUXC_SD2_DATA0__USDHC2_DATA0 0x1d0 > > + MX8MP_IOMUXC_SD2_DATA1__USDHC2_DATA1 0x1d0 > > + MX8MP_IOMUXC_SD2_DATA2__USDHC2_DATA2 0x1d0 > > + MX8MP_IOMUXC_SD2_DATA3__USDHC2_DATA3 0x1d0 > > + MX8MP_IOMUXC_GPIO1_IO04__USDHC2_VSELECT 0xc1 > > Just to make sure this doesn't fry SD-Cards by mistake: VSELECT is indeed > connected to a 1.8V/3.3V switch powering vqmmc? > > > +/* SD Card */ > > +&usdhc2 { > > + assigned-clocks = <&clk IMX8MP_CLK_USDHC2>; > > + assigned-clock-rates = <400000000>; > > I wonder why this is necessary. Do you see a difference > in /sys/kernel/debug/mmcX/ios between having this and leaving > it out? > > > + status = "okay"; > > +}; > > + > > +/* eMMc */ > > eMMC > > > +&usdhc3 { > > + assigned-clocks = <&clk IMX8MP_CLK_USDHC3>; > > + assigned-clock-rates = <400000000>; > > + pinctrl-names = "default", "state_100mhz", "state_200mhz"; > > + pinctrl-0 = <&pinctrl_usdhc3>; > > + pinctrl-1 = <&pinctrl_usdhc3_100mhz>; > > + pinctrl-2 = <&pinctrl_usdhc3_200mhz>; > > + bus-width = <8>; > > + non-removable; > > + status = "okay"; > > +}; > > + > > +&wdog1 { > > + pinctrl-names = "default"; > > + pinctrl-0 = <&pinctrl_wdog>; > > + fsl,ext-reset-output; > > + status = "okay"; > > +}; > > > Cheers, > Ahmad > > -- > Pengutronix e.K. | | > Steuerwalder Str. 21 | http://www.pengutronix.de/ | > 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | > Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel