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 E5842C3DA42 for ; Wed, 10 Jul 2024 07:18:50 +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-Type: Content-Transfer-Encoding:MIME-Version:References:In-Reply-To:Message-ID:Date :Subject:Cc:To:From:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=bzFdmN4YSMsYqNzg1gKeQE0QeWkldNu+KdO/Z6kO2Ok=; b=MJbPvT4syw5FeTdX4GHaBt4cMm gG76tvpKxK+d1V3oIjDECT3PuM+Ubn8o6P5387vWCFhnK4DhR5E3oUCCD3rlKgRiym9X8YgUBmwee PDnB8UzEN6SZPg0AbO4CCrM02NxztryVDCVJ2htc4P1XiPgJKgiOJ/Y/Eh725iz86t77a+KOoipcq EYQobxvilPvD5deL9wWOw062Q5VLOwTD6ryvyCkPUAmxNS7bq4QmCQpnH23YLxiKomNH6kJuh/mwl SJCcJQviKtcuCdp3grPv7/qYbzsN6t64Z7pCXaWK9A50dsAy8EtT1Pr+RzZIof+xl7j7mcE0y/LWA BHR+845g==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sRRb3-00000009icb-3Fzm; Wed, 10 Jul 2024 07:18:38 +0000 Received: from mx1.tq-group.com ([93.104.207.81]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1sRRal-00000009iXi-2y41 for linux-arm-kernel@lists.infradead.org; Wed, 10 Jul 2024 07:18:22 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=tq-group.com; i=@tq-group.com; q=dns/txt; s=key1; t=1720595899; x=1752131899; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=bzFdmN4YSMsYqNzg1gKeQE0QeWkldNu+KdO/Z6kO2Ok=; b=BXy3zMunpleSj9Y/9lDRCJds7dyUIDwV9wif/fkBqKLE1CuOiZ/gUYZd hnKWEG7kJjlKxN/82WDdrherQnyxJ4htpprKZPWGdYJKNQf5YMOwqOIII H8gb0Xnw6W5ggcZ131GPqJTVG1I3RvG1bzyAn8fLco83Tht5bFn/fPvSE 1AmVO6t0dRY1DydulNosKqKT016btEicFQziXJGDLnJ1+TUtG4lqwRG9B zn21WxsCRuP4Gn/1dg6eNOBNJUSVJ5ukntGjFJHgkDGqu6YXE9TsY2pPp 7QTeNe+d6w3BfJ4FsPrxZJ+ec0BWFLyQn7uPQNIaYQKS4NscQboUP+v5I A==; X-CSE-ConnectionGUID: iBBx2iWjTXqVlKJTpMk04g== X-CSE-MsgGUID: a3vuhXTYTue6VNz0KPa+jA== X-IronPort-AV: E=Sophos;i="6.09,197,1716242400"; d="scan'208";a="37831245" Received: from vmailcow01.tq-net.de ([10.150.86.48]) by mx1.tq-group.com with ESMTP; 10 Jul 2024 09:18:17 +0200 Received: from [127.0.0.1] (localhost [127.0.0.1]) by localhost (Mailerdaemon) with ESMTPSA id D9E26160B1D; Wed, 10 Jul 2024 09:18:10 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ew.tq-group.com; s=dkim; t=1720595893; h=from:subject:date:message-id:to:cc:mime-version:content-type: content-transfer-encoding:in-reply-to:references; bh=bzFdmN4YSMsYqNzg1gKeQE0QeWkldNu+KdO/Z6kO2Ok=; b=T7x6sGhgYzu7QX5HNtiaCZ0KFg4oZ43vL8ZbpFnbdYSyQYxbNHc/IbZ2WLU/qBrYvulVlo j8prnJLSET50sm0dN0JVoe/Dz5uLVlvf2AsRqLoaGqupap+PHHDOpmfc/EUyLiQoSRKN0I M//on1iSYxik7leOemWKsOWxJjo51dF0qno35w7M3f7qqH0oIkCkV4IvrE3Tjem7XSGmqK O5fA2/SwsxX0Fjn3cPpMmXwOi9K3s0+1K1l7OeO0IolxQpIsm7YhWeOYI3QQuIa2RPPrgL yJWugsvPjdnsEp6gNyx6GxfBkxS8BrTFCl8FFmZQ8HEd71hwXS6YzAdDTZOhsA== From: Alexander Stein To: York Sun , Borislav Petkov , Tony Luck , James Morse , Mauro Carvalho Chehab , Robert Richter , Krzysztof Kozlowski , Rob Herring , Conor Dooley , Krzysztof Kozlowski , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , linux-arm-kernel@lists.infradead.org Cc: linux-edac@vger.kernel.org, linux-kernel@vger.kernel.org, Borislav Petkov , devicetree@vger.kernel.org, imx@lists.linux.dev, linux-arm-kernel@lists.infradead.org, Frank Li , Frank Li Subject: Re: [PATCH 4/6] dt-bindings: memory: fsl: Add compatible string nxp, imx9-memory-controller Date: Wed, 10 Jul 2024 09:18:16 +0200 Message-ID: <6068670.lOV4Wx5bFT@steina-w> Organization: TQ-Systems GmbH In-Reply-To: <20240709-imx95_edac-v1-4-3e9c146c1b01@nxp.com> References: <20240709-imx95_edac-v1-0-3e9c146c1b01@nxp.com> <20240709-imx95_edac-v1-4-3e9c146c1b01@nxp.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="iso-8859-1" X-Last-TLS-Session-Version: TLSv1.3 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240710_001820_261483_DF0BA7B7 X-CRM114-Status: GOOD ( 14.26 ) 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 Hi Frank, Am Dienstag, 9. Juli 2024, 22:23:05 CEST schrieb Frank Li: > iMX9 memory controller is similar with other layerscape chips. But some > register layout has a little bit difference, so add new compatible string > 'nxp,imx9-memory-controller' for it. Is this controller the same for all i.MX9 SoC? E.g. i.MX91, i.MX93, i.MX95 and any future variants? Best regards, Alexander > imx9 need two 'reg', one for DDR controller and the other is ECC inject > engine register space. Keep the same restriction for other compatible > string. >=20 > Signed-off-by: Frank Li > --- > .../bindings/memory-controllers/fsl/fsl,ddr.yaml | 31 ++++++++++++++++= +++++- > 1 file changed, 30 insertions(+), 1 deletion(-) >=20 > diff --git a/Documentation/devicetree/bindings/memory-controllers/fsl/fsl= ,ddr.yaml b/Documentation/devicetree/bindings/memory-controllers/fsl/fsl,dd= r.yaml > index 84f778a99546b..e0786153eec73 100644 > --- a/Documentation/devicetree/bindings/memory-controllers/fsl/fsl,ddr.ya= ml > +++ b/Documentation/devicetree/bindings/memory-controllers/fsl/fsl,ddr.ya= ml > @@ -40,6 +40,7 @@ properties: > - fsl,p1021-memory-controller > - fsl,p2020-memory-controller > - fsl,qoriq-memory-controller > + - nxp,imx9-memory-controller > =20 > interrupts: > maxItems: 1 > @@ -51,13 +52,41 @@ properties: > type: boolean > =20 > reg: > - maxItems: 1 > + items: > + - description: Controller register space > + - description: Inject register space > + minItems: 1 > + > + reg-names: > + items: > + - const: ctrl > + - const: inject > + minItems: 1 > =20 > required: > - compatible > - interrupts > - reg > =20 > +allOf: > + - if: > + properties: > + compatible: > + contains: > + enum: > + - nxp,imx9-memory-controller > + then: > + properties: > + reg: > + minItems: 2 > + reg-names: > + minItems: 2 > + else: > + properties: > + reg: > + maxItems: 1 > + reg-names: false > + > additionalProperties: false > =20 > examples: >=20 >=20 =2D-=20 TQ-Systems GmbH | M=FChlstra=DFe 2, Gut Delling | 82229 Seefeld, Germany Amtsgericht M=FCnchen, HRB 105018 Gesch=E4ftsf=FChrer: Detlef Schneider, R=FCdiger Stahl, Stefan Schneider http://www.tq-group.com/