From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mout01.posteo.de (mout01.posteo.de [185.67.36.65]) (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 C99A922FF38 for ; Thu, 6 Feb 2025 22:30:25 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=185.67.36.65 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1738881029; cv=none; b=NppfYsa1nQKkQWa/5XUasVzHZjNJxgrouC6XofgHRNANJkBtcsUIAzwiUCgxek5wcXNP82E0IVXHwmiD7z7aZVBGGGOXqxxPpFrO7mm2bT1DHTOCmJjuNFVeo6zj3euuYB4SiJwC+35ljl0KlYebz5bnrKzBw1ExzS+jJGnXAog= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1738881029; c=relaxed/simple; bh=u6G1eD2kmbp6domphDX7910Spwzkgf3Yjk2DeO59O7o=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=paNrFsrW4hA7+Qqj2EpOWIE/JMNAJexeW8pP7uv6VmIcEaO6eKewIEm/B6Zfir+wFF417rNka9A4Rt/8q4DQsAcFBxYM+2/+aLiLBDwtAhJFzAJuK5/PoNVoRB37pZCWrcnYPobH2E+6j5bJtrfTCqsz1Vp03M+C2KBdI+lgy6Q= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=posteo.net; spf=pass smtp.mailfrom=posteo.net; dkim=pass (2048-bit key) header.d=posteo.net header.i=@posteo.net header.b=SkLC+loC; arc=none smtp.client-ip=185.67.36.65 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=posteo.net Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=posteo.net Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=posteo.net header.i=@posteo.net header.b="SkLC+loC" Received: from submission (posteo.de [185.67.36.169]) by mout01.posteo.de (Postfix) with ESMTPS id F24DC240027 for ; Thu, 6 Feb 2025 23:30:23 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=posteo.net; s=2017; t=1738881023; bh=u6G1eD2kmbp6domphDX7910Spwzkgf3Yjk2DeO59O7o=; h=Date:From:To:Cc:Subject:Message-ID:MIME-Version:Content-Type: Content-Disposition:Content-Transfer-Encoding:From; b=SkLC+loCqCAxHzvTwGtfUpnyhBnXIHUTV2Pu2Xozi2Vuk2zpEzjMzYHOdg6vYVyU3 pfHZP3GZg8N+2Ps7oJ1fDkLEhQomdJmhEIfjyKEDa0CuJslg+tYKUnmhkJzn75syAJ YNOFj7cCcxVJdNoqRqycNJhLZccDrrvFqEoRczLWH4bHGYcyzxhspWJ6qKp6TdCBf9 EFp0nvh67qbHWKVCNSxkm6lUrJgXAsp9HndNSLza9c7cFwh3AYG41klDKxV+tCTUhA x9GFP3E+z4UVO6RU88VfMxpUr76LmYlAHtsb2b8W0GJxKjPyOVg2szlp3KEVuqfS78 pY8Ud/liEujXw== Received: from customer (localhost [127.0.0.1]) by submission (posteo.de) with ESMTPSA id 4YpsHY0mq5z9rxF; Thu, 6 Feb 2025 23:30:16 +0100 (CET) Date: Thu, 6 Feb 2025 22:30:15 +0000 From: =?utf-8?Q?J=2E_Neusch=C3=A4fer?= To: Krzysztof Kozlowski Cc: =?utf-8?Q?J=2E_Neusch=C3=A4fer?= , devicetree@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, Scott Wood , Madhavan Srinivasan , Michael Ellerman , Nicholas Piggin , Christophe Leroy , Naveen N Rao , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Damien Le Moal , Niklas Cassel , Herbert Xu , "David S. Miller" , Lee Jones , Vinod Koul , Lorenzo Pieralisi , Krzysztof =?utf-8?Q?Wilczy=C5=84ski?= , Manivannan Sadhasivam , Bjorn Helgaas , =?utf-8?Q?J=2E_Neusch=C3=A4fer?= , Wim Van Sebroeck , Guenter Roeck , Mark Brown , Miquel Raynal , Richard Weinberger , Vignesh Raghavendra , linux-kernel@vger.kernel.org, linux-ide@vger.kernel.org, linux-crypto@vger.kernel.org, dmaengine@vger.kernel.org, linux-pci@vger.kernel.org, linux-watchdog@vger.kernel.org, linux-spi@vger.kernel.org, linux-mtd@lists.infradead.org Subject: Re: [PATCH RFC 9/9] dt-bindings: nand: Convert fsl,elbc bindings to YAML Message-ID: References: <20250126-ppcyaml-v1-0-50649f51c3dd@posteo.net> <20250126-ppcyaml-v1-9-50649f51c3dd@posteo.net> <20250127-cuddly-dalmatian-of-saturation-5f1ae2@krzk-bin> Precedence: bulk X-Mailing-List: dmaengine@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20250127-cuddly-dalmatian-of-saturation-5f1ae2@krzk-bin> On Mon, Jan 27, 2025 at 09:37:32AM +0100, Krzysztof Kozlowski wrote: > On Sun, Jan 26, 2025 at 07:59:04PM +0100, J. Neuschäfer wrote: > > Convert the Freescale localbus controller bindings from text form to > > YAML. The list of compatible strings reflects current usage. > > simple-bus and 20 other compatibles you used were not present in the > original binding. Does above "list of compatible strings" mean you just > added them? Ah, I should make this clearer: - by "list of compatible strings" I mean the list as it is after applying the patch - by "current usage" I mean what is found in arch/*/boot/dts > > > > > Changes compared to the txt version: > > - removed the board-control (fsl,mpc8272ads-bcsr) node because it only > > appears in this example and nowhere else > > - added a new example with NAND flash > > > > Remaining issues: > > - The localbus is not really a simple-bus: Unit addresses are not simply > > addresses on a memory bus. Instead, they have a format: The first cell > > is a chip select number, the remaining one or two cells are bus > > addresses. > > > > Signed-off-by: J. Neuschäfer > > --- > > .../devicetree/bindings/mtd/fsl,elbc-fcm-nand.yaml | 61 +++++++++ > > .../bindings/powerpc/fsl/fsl,elbc-gpcm-uio.yaml | 55 ++++++++ > > Please split the conversion from adding new bindings. For example above > file and its compatible fsl,elbc-gpcm-uio was not documented in original > TXT. Fair point, I'll split them. > > ... > > > diff --git a/Documentation/devicetree/bindings/powerpc/fsl/fsl,elbc.yaml b/Documentation/devicetree/bindings/powerpc/fsl/fsl,elbc.yaml > > new file mode 100644 > > index 0000000000000000000000000000000000000000..6bbceb82c77826499abe85879e9189b18d396eea > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/powerpc/fsl/fsl,elbc.yaml > > @@ -0,0 +1,150 @@ > > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > > +%YAML 1.2 > > +--- > > +$id: http://devicetree.org/schemas/powerpc/fsl/fsl,elbc.yaml# > > +$schema: http://devicetree.org/meta-schemas/core.yaml# > > + > > +title: Freescale Enhanced Local Bus Controller > > What sort of bus is it? Memory bus? Then place it with others, see > memory directory. Yes, a memory bus controller with explicit chip-select lines, for various kinds of memory (flash, RAM), and possibly other peripherals. I'll move it to bindings/memory-controllers. > > > + > > +maintainers: > > + - J. Neuschäfer > > + > > +properties: > > + $nodename: > > + pattern: "^localbus@[0-9a-f]+$" > > + > > + compatible: > > + oneOf: > > + - items: > > + - enum: > > + - fsl,mpc8313-elbc > > + - fsl,mpc8315-elbc > > + - fsl,mpc8377-elbc > > + - fsl,mpc8378-elbc > > + - fsl,mpc8379-elbc > > + - fsl,mpc8536-elbc > > + - fsl,mpc8569-elbc > > + - fsl,mpc8572-elbc > > + - fsl,p1020-elbc > > + - fsl,p1021-elbc > > + - fsl,p1023-elbc > > + - fsl,p2020-elbc > > + - fsl,p2041-elbc > > + - fsl,p3041-elbc > > + - fsl,p4080-elbc > > + - fsl,p5020-elbc > > + - fsl,p5040-elbc > > + - const: fsl,elbc > > + - const: simple-bus > > + > > + - items: > > + - const: fsl,mpc8272-localbus > > + - const: fsl,pq2-localbus > > + > > + - items: > > + - enum: > > + - fsl,mpc8247-localbus > > + - fsl,mpc8248-localbus > > + - fsl,mpc8360-localbus > > + - const: fsl,pq2pro-localbus > > + - const: simple-bus > > + > > + - items: > > + - enum: > > + - fsl,mpc8540-localbus > > + - fsl,mpc8544-lbc > > + - fsl,mpc8544-localbus > > + - fsl,mpc8548-lbc > > + - fsl,mpc8548-localbus > > + - fsl,mpc8560-localbus > > + - fsl,mpc8568-localbus > > + - const: fsl,pq3-localbus > > + - const: simple-bus > > + > > + reg: > > + maxItems: 1 > > + > > + interrupts: > > + maxItems: 1 > > + > > + "#address-cells": > > + enum: [2, 3] > > + description: | > > + The first cell is the chipselect number, and the remaining cells are the > > + offset into the chipselect. > > + > > + "#size-cells": > > + enum: [1, 2] > > + description: | > > + Either one or two, depending on how large each chipselect can be. > > + > > + ranges: > > + description: | > > + Each range corresponds to a single chipselect, and covers the entire > > + access window as configured. > > + > > +patternProperties: > > + "^.*@.*$": > > + type: object > > And probably you need ? > > + > > +additionalProperties: false > > + > > +examples: > > + - | > > + localbus@f0010100 { > > + compatible = "fsl,mpc8272-localbus", > > + "fsl,pq2-localbus"; > > + #address-cells = <2>; > > + #size-cells = <1>; > > + reg = <0xf0010100 0x40>; > > compatible, then reg - see DTS coding style. I'll clean up the various style issues in the examples. (These examples are quite old, either from the old version of this binding or from existing dts files.) Best regards, J. Neuschäfer > > > + > > + ranges = <0x0 0x0 0xfe000000 0x02000000 > > + 0x1 0x0 0xf4500000 0x00008000 > > + 0x2 0x0 0xfd810000 0x00010000>; > > + > > + flash@0,0 { > > + compatible = "jedec-flash"; > > + reg = <0x0 0x0 0x2000000>; > > Well, here it is correct > > > + bank-width = <4>; > > + device-width = <1>; > > + }; > > + > > + simple-periph@2,0 { > > + compatible = "fsl,elbc-gpcm-uio"; > > + reg = <0x2 0x0 0x10000>; > > + elbc-gpcm-br = <0xfd810800>; > > + elbc-gpcm-or = <0xffff09f7>; > > + }; > > + }; > > + > > + - | > > + localbus@e0005000 { > > compatible, reg > > > + #address-cells = <2>; > > + #size-cells = <1>; > > + compatible = "fsl,mpc8315-elbc", "fsl,elbc", "simple-bus"; > > + reg = <0xe0005000 0x1000>; > > + interrupts = <77 0x8>; > > + interrupt-parent = <&ipic>; > > + > > + ranges = <0x0 0x0 0xfe000000 0x00800000 > > + 0x1 0x0 0xe0600000 0x00002000 > > + 0x2 0x0 0xf0000000 0x00020000 > > + 0x3 0x0 0xfa000000 0x00008000>; > > + > > + flash@0,0 { > > compatible, reg > > > + #address-cells = <1>; > > + #size-cells = <1>; > > + compatible = "cfi-flash"; > > + reg = <0x0 0x0 0x800000>; > > + bank-width = <2>; > > + device-width = <1>; > > + }; > > + > > + nand@1,0 { > > compatible, reg > > > + #address-cells = <1>; > > + #size-cells = <1>; > > + compatible = "fsl,mpc8315-fcm-nand", > > + "fsl,elbc-fcm-nand"; > > + reg = <0x1 0x0 0x2000>; > > + }; > > Best regards, > Krzysztof >