From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mx1.white.stw.pengutronix.de (mx1.white.stw.pengutronix.de [185.203.200.13]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id C5702580395 for ; Wed, 9 Sep 2026 16:20:32 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=185.203.200.13 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1788970837; cv=none; b=Wsu9vGePJ7EQ+S5MJfMbq6UlS/KAgNvcEA+UK66nMX5wBYmd5vaFKP7z11GNvcrwbJRTdN391XvygesMwqjfJUVWFXH8TS69pyNTbIKteJ2rkJgEKrxRaS0NnjZP+0ET6jDYKR+3b/KIuIl5MU49fRKWm9bUH3RPoaqzx49xY1Y= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1788970837; c=relaxed/simple; bh=UUeVEBaFv+mU0zzEufRC1He4yG7Kd/oISg/1H00kCKI=; h=Message-ID:Date:MIME-Version:From:Subject:To:Cc:References: In-Reply-To:Content-Type; b=Z7w58HXtNB5yDYy++T8hyyzmXkSvEgS+I9JvD0j9xl8o+guUn7vSHBCg7ON44CdKhsBkcCacQhSikzz5sCeoRlmgJDwk0T2ToxXW/tjv38Z0HjIv3PhI+rzlt9UqJSi10Fes01NaN/RILkmLzZKskVPENilvxwEv8rHnpqyQQ1Q= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=pengutronix.de; spf=pass smtp.mailfrom=pengutronix.de; arc=none smtp.client-ip=185.203.200.13 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=pengutronix.de Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=pengutronix.de X-PTX-Original-Recipient: laurentiumihalcea111@gmail.com X-PTX-Original-Recipient: robh@kernel.org X-PTX-Original-Recipient: krzk+dt@kernel.org X-PTX-Original-Recipient: conor+dt@kernel.org X-PTX-Original-Recipient: s.hauer@pengutronix.de X-PTX-Original-Recipient: frank.li@nxp.com X-PTX-Original-Recipient: festevam@gmail.com X-PTX-Original-Recipient: elena.popa@nxp.com X-PTX-Original-Recipient: consmin.stoica@nxp.com X-PTX-Original-Recipient: linux-arm-kernel@lists.infradead.org X-PTX-Original-Recipient: devicetree@vger.kernel.org X-PTX-Original-Recipient: linux-kernel@vger.kernel.org X-PTX-Original-Recipient: kernel@pengutronix.de X-PTX-Original-Recipient: imx@lists.linux.dev Received: from [0.0.0.0] (ptz.office.stw.pengutronix.de [IPv6:2a0a:edc0:0:900:1d::77]) (Authenticated sender: afa@pengutronix.de) by mx1.white.stw.pengutronix.de (Postfix) with ESMTPSA id 4AEF620007C; Wed, 09 Sep 2026 18:20:30 +0200 (CEST) Message-ID: Date: Wed, 9 Sep 2026 18:20:29 +0200 Precedence: bulk X-Mailing-List: imx@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird From: Ahmad Fatoum Subject: Re: [PATCH v4 2/2] arm64: dts: freescale: add MR-NAVQ95 basic board support To: Laurentiu Mihalcea , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Sascha Hauer , Frank Li , Fabio Estevam , elena.popa@nxp.com, consmin.stoica@nxp.com Cc: linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Pengutronix Kernel Team , imx@lists.linux.dev References: <20260909153943.1048-1-laurentiumihalcea111@gmail.com> <20260909153943.1048-3-laurentiumihalcea111@gmail.com> Content-Language: en-US, de-DE, de-BE In-Reply-To: <20260909153943.1048-3-laurentiumihalcea111@gmail.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Hi, On 9/9/26 5:39 PM, Laurentiu Mihalcea wrote: > From: Laurentiu Mihalcea > > Add basic support for NXP's MR-NAVQ95 board. This board is designed for > robotics applications (e.g. drones, rovers, humanoids, etc..) and has the > following hardware feats (list is not exhaustive): > > * based on i.MX95 19x19 SoC > * 64GB eMMC > * up to 16GB of LPDDR5 memory > * 1 x USB 2.0 TypeC > * 1 x USB 3.0 TypeC > * 1 x RJ45 port with RTL8211 PHY > > The support targets revision B of the board. > > Signed-off-by: Laurentiu Mihalcea > --- > + chosen { > + stdout-path = &lpuart1; I would recommend instead "serial0:115200n8" here. This introduces no change for Linux, but for other consumers of the device tree, they will be able to agree on a common baudrate. > + memory@80000000 { > + device_type = "memory"; > + /* actual size filled in by the bootloader */ > + reg = <0x0 0x80000000 0 0x80000000>; If the board has at least 2G always, this is fine. > +&usdhc2 { > + bootph-pre-ram; > + bus-width = <4>; > + cd-gpios = <&gpio3 0 GPIO_ACTIVE_LOW>; > + pinctrl-0 = <&pinctrl_usdhc2>, <&pinctrl_usdhc2_gpio>; > + pinctrl-1 = <&pinctrl_usdhc2>, <&pinctrl_usdhc2_gpio>; > + pinctrl-2 = <&pinctrl_usdhc2>, <&pinctrl_usdhc2_gpio>; > + pinctrl-3 = <&pinctrl_usdhc2>, <&pinctrl_usdhc2_gpio>; > + pinctrl-names = "default", "state_100mhz", "state_200mhz", "sleep"; Looks unexpected to have same pinctrl groups for the higher speeds. Are higher speed modes even possible on this board? If not, I would suggest dropping the extra states. Cheers, Ahmad > + vmmc-supply = <®_usdhc2_vmmc>; > + > + status = "okay"; > +}; -- 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 |