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 D0BF3C79F9F for ; Thu, 10 Sep 2026 15:24:04 +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=Kq6vuj6/rgekH+qZ17iqP5wXEZl9Suonhm2pFqXIlS8=; b=QQ5fuxd3dRNX8ADcqfXfH+Jaiv HPPFr4V9IHTL3IYlx87c9UXyTwX0IejaBKS2cybz9oLJtD9IBHaE2VAGl5dn+ehKthkGemz0EYa7r ZcShD9zqttkN4yEiU3tO/KosuEf17SXwM/+HX5d7SGNENnPCmxUfKVMVigfjdvJAkdgxuGdHZ7QKn wYEUSubZIFO57DXDFYLxH0eIfdvB8AjW1cqeGaWHN+JO3SeimIrSieeYllsuncN+SHnJZUZ8bTd3+ 8OY9MHwn3Id4jXA4pe7evMNmzwKzr6E80fpzcDJT32wlfqiG9GBq36gU9xSH+w64X8V16OPF51lxK 3TztRZ7A==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1x4gd4-0000000EmYz-09Bd; Thu, 10 Sep 2026 15:23:58 +0000 Received: from mx1.white.stw.pengutronix.de ([2a0a:edc0:0:b01:1d::107]) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1x4gd1-0000000EmXz-0qBv for linux-arm-kernel@lists.infradead.org; Thu, 10 Sep 2026 15:23:57 +0000 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 697642020A4; Thu, 10 Sep 2026 17:23:50 +0200 (CEST) Message-ID: <818e023c-8a88-4736-9577-355e97da4466@pengutronix.de> Date: Thu, 10 Sep 2026 17:23:50 +0200 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird 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> From: Ahmad Fatoum Content-Language: en-US, de-DE, de-BE In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.9.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20260910_082355_397831_41A04E29 X-CRM114-Status: GOOD ( 27.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: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hello, On 9/10/26 5:08 PM, Laurentiu Mihalcea wrote: > > Hi Ahmad, > > On 9/9/2026 9:20 AM, Ahmad Fatoum wrote: >> On 9/9/26 5:39 PM, Laurentiu Mihalcea wrote: >>> From: Laurentiu Mihalcea >>> + 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. > > the board only ships with 16GB LPDDR5 so yes, it'll always have at least 2GB. > While the current size is fine, maybe it would be better to change it to 16GB? > Will have to update the commit message as well since "up to 16GB of LPDDR5 memory" > might be a bit misleading. > > my other option was to leave the size set to 0x0 like some QCOM DTSIs seem to do. > Not sure what the preferred way of handling this is? IMO, the DT should not claim a memory size that's less than what's available across variants. So either 2G, 16G or just dropping it is fine by me. No need to change this on my account. >>> +&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. > > The pin configuration is chosen to meet the timing requirements of the 200MHz > state. Because of this, I would expect the configuration to be appropriate for > lower frequencies as well. > > The intention is to use the same configuration as MX95-19x19-EVK since, from what > I've been told, the trace lengths are similar to those of the EVK. Ok. If you can actually attain higher rates this way, then nothing needs to be changed. > Thanks for taking the time to review this! :) 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 |