Received: by 2002:a05:6a10:206:0:0:0:0 with SMTP id 6csp1521124pxj; Fri, 4 Jun 2021 17:27:41 -0700 (PDT) X-Google-Smtp-Source: ABdhPJy+eyagUtI+kFM+LAjtBPWnSwl6XZUUtcpyGnsuBbhHFxL5hGwXF722Waefghit3t9l3H9B X-Received: by 2002:a05:6402:101a:: with SMTP id c26mr7742429edu.19.1622852861674; Fri, 04 Jun 2021 17:27:41 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1622852861; cv=none; d=google.com; s=arc-20160816; b=vkVi9SMCoTEWuCeB0EO7ZtFBxXghQ9jz8ZU5ukuVPN44BDx7efZNSbHvLt8WJXBNrR km1Rurnc7E8Emxn9sSoCEMJt1dNvdzH1PDtfVkQ6OWWVyP10GD8MUrjQy3t4Ya0qAPVh 5xSQwof7Ael28xQiZGplX7m45uteBIohRBHgEe3aPsYXNRgHMmnzgnuTWFc5URCILqMw ZzTmQjvcu5CATuJ/BsVRyBpAEhB8RjW1fh9pvqb7Pn5I/f3KpZ/ymf9RVZdZY3eimhcs pPLjw6+2JYHLKTjnPvBKxw+yhLhiVTzU5k3gfjm4CkalxdzMaRVWKBIQKxgx8gZckkFU eRgQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:cc:to:subject :message-id:date:from:in-reply-to:references:mime-version :dkim-signature; bh=sUFuRMnJeUbsvj2QDdwiuwSD5F69c4BG1mDs2axqiTs=; b=0JIwppai1ioJ2uhJB2fbtMoIqgDzBAON03Yx5cWl+Vh5025m4LU5os/e0972J6NFZa 3B1tW9xAtAtAogUr+sCJOlP1DrZXbSYWyMzqEEfXoHwTLF/YNhX+tH5EiOPfvbPe/H3p YIS0OhGgfUHsCUo2inlQELTw5uqUWdZaJZYqbuhlix0/DWoqblYDbvymptzOG9s6pxmM GIs9NpwK2YtkxHOZCMA+q51lLsrnJVa12EBm+IxNzpTUPy/IF03ToFEwDu2Td/ua6nWw ktJJjJPoisqV66ys4IyyXR0AgYQZtHu8+EBdNQsqtWs9yG9j9dK0mpJxlPSDR1zSVkjh dK3A== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@nigauri-org.20150623.gappssmtp.com header.s=20150623 header.b=zM5TBVEX; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id z3si5970432edx.85.2021.06.04.17.27.18; Fri, 04 Jun 2021 17:27:41 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@nigauri-org.20150623.gappssmtp.com header.s=20150623 header.b=zM5TBVEX; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229906AbhFEA16 (ORCPT + 99 others); Fri, 4 Jun 2021 20:27:58 -0400 Received: from mail-ua1-f51.google.com ([209.85.222.51]:44782 "EHLO mail-ua1-f51.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229847AbhFEA15 (ORCPT ); Fri, 4 Jun 2021 20:27:57 -0400 Received: by mail-ua1-f51.google.com with SMTP id 68so6207817uao.11 for ; Fri, 04 Jun 2021 17:25:59 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nigauri-org.20150623.gappssmtp.com; s=20150623; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc:content-transfer-encoding; bh=sUFuRMnJeUbsvj2QDdwiuwSD5F69c4BG1mDs2axqiTs=; b=zM5TBVEXzzTUIEFhS1qRdHVmP1tXYlBOmQd4/VjMnJ3DmdqFFU6bkbLyfrNxxnyWa9 blKUy6wA4j4eLX6D4rq1IoWM59ghqmVMwl0PuYuzcZRjV+Q4Rjp84cUyclR0WyP4zOzG vsolpHhgM0Xcwpw48NzYsTtSoedzBg7VxM7QZipwYmptd21HTQy4ujjenVI1qzBiCaep 46wgeIFWFIed9H1Mej2xxgNiZVUdtORElQoVwE5W2SHA8EKBMNh1aLhXGxm/N3ShFfzk c4CE/7gCakEjXeOM9aOcaOSO2Rx9LnqHKnQByJsVJfV9PXN+OTkq5V8kNAvBaboI25PF 2FNA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc:content-transfer-encoding; bh=sUFuRMnJeUbsvj2QDdwiuwSD5F69c4BG1mDs2axqiTs=; b=gIdzZvq/ysENlbiuwzB00nnbc1ZZNMT8hzi0+JS5kOFy8BZMyirh9kNF6WH9IqSaCG AvE7omkEVsFeKSuTEYXye7IAj7kppspvdvIVlVgy9QvrLrV6B5/SgDofOvT7wigyu121 mWZXPJDYFKCt1zCGixVONU9B2RbOHEP8jYjy5mmqREekmu/cMeTG/AFyu+DaxDHiK3gV 0fR9Sdf9UZ2BdFoiqVKWBQNqdBeAEmw4W6heNYdR43H+aUTKFuNNKxXokSUBKzAa4eCx IKdAE4D7HCQdEExAEqe78ghl45yrgucRYfaJCiaSY3Fh3owwmlsRQhHbnUE3Y5EYfZZc HOZg== X-Gm-Message-State: AOAM533tWIJO+Lu5gCw95tgVIfT7khnmeHvhyhvNaBI21oelh2xUtFCR basJ14TyoB4AkH3iyV+5veb2lzrodFhme0MaQd45 X-Received: by 2002:a05:6122:40e:: with SMTP id e14mr4179610vkd.20.1622852699138; Fri, 04 Jun 2021 17:24:59 -0700 (PDT) MIME-Version: 1.0 References: <20210531141538.721613-1-iwamatsu@nigauri.org> <20210604214608.GA3974358@robh.at.kernel.org> In-Reply-To: <20210604214608.GA3974358@robh.at.kernel.org> From: Nobuhiro Iwamatsu Date: Sat, 5 Jun 2021 09:24:32 +0900 Message-ID: Subject: Re: [PATCH] dt-bindings: spi: convert Cadence SPI bindings to YAML To: Rob Herring Cc: Mark Brown , Michal Simek , linux-spi , devicetree@vger.kernel.org, Linux Kernel Mailing List , Harini Katakam , linux ARM Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, Thanks for your review. 2021=E5=B9=B46=E6=9C=885=E6=97=A5(=E5=9C=9F) 6:46 Rob Herring : > > On Mon, May 31, 2021 at 11:15:38PM +0900, Nobuhiro Iwamatsu wrote: > > Convert spi for Cadence SPI bindings documentation to YAML. > > > > Signed-off-by: Nobuhiro Iwamatsu > > --- > > .../devicetree/bindings/spi/spi-cadence.txt | 30 --------- > > .../devicetree/bindings/spi/spi-cadence.yaml | 63 +++++++++++++++++++ > > 2 files changed, 63 insertions(+), 30 deletions(-) > > delete mode 100644 Documentation/devicetree/bindings/spi/spi-cadence.t= xt > > create mode 100644 Documentation/devicetree/bindings/spi/spi-cadence.y= aml > > > > diff --git a/Documentation/devicetree/bindings/spi/spi-cadence.txt b/Do= cumentation/devicetree/bindings/spi/spi-cadence.txt > > deleted file mode 100644 > > index 05a2ef945664be..00000000000000 > > --- a/Documentation/devicetree/bindings/spi/spi-cadence.txt > > +++ /dev/null > > @@ -1,30 +0,0 @@ > > -Cadence SPI controller Device Tree Bindings > > -------------------------------------------- > > - > > -Required properties: > > -- compatible : Should be "cdns,spi-r1p6" or "xlnx,zynq-spi-r1p= 6". > > -- reg : Physical base address and size of SPI r= egisters map. > > -- interrupts : Property with a value describing the interrupt > > - number. > > -- clock-names : List of input clock names - "ref_clk", = "pclk" > > - (See clock bindings for details). > > -- clocks : Clock phandles (see clock bindings for details)= . > > - > > -Optional properties: > > -- num-cs : Number of chip selects used. > > - If a decoder is used, this will be the number o= f > > - chip selects after the decoder. > > -- is-decoded-cs : Flag to indicate whether decoder is use= d or not. > > - > > -Example: > > - > > - spi@e0007000 { > > - compatible =3D "xlnx,zynq-spi-r1p6"; > > - clock-names =3D "ref_clk", "pclk"; > > - clocks =3D <&clkc 26>, <&clkc 35>; > > - interrupt-parent =3D <&intc>; > > - interrupts =3D <0 49 4>; > > - num-cs =3D <4>; > > - is-decoded-cs =3D <0>; > > - reg =3D <0xe0007000 0x1000>; > > - } ; > > diff --git a/Documentation/devicetree/bindings/spi/spi-cadence.yaml b/D= ocumentation/devicetree/bindings/spi/spi-cadence.yaml > > new file mode 100644 > > index 00000000000000..27a7121ed0f9ae > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/spi/spi-cadence.yaml > > @@ -0,0 +1,63 @@ > > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > > +%YAML 1.2 > > +--- > > +$id: http://devicetree.org/schemas/spi/spi-cadence.yaml# > > +$schema: http://devicetree.org/meta-schemas/core.yaml# > > + > > +title: Cadence SPI controller Device Tree Bindings > > + > > +maintainers: > > + - Michal Simek > > + > > +allOf: > > + - $ref: "spi-controller.yaml#" > > + > > +properties: > > + compatible: > > + enum: > > + - cdns,spi-r1p6 > > + - xlnx,zynq-spi-r1p6 > > + > > + reg: > > + maxItems: 1 > > + > > + interrupts: > > + maxItems: 1 > > + > > + clock-names: > > + items: > > + - const: ref_clk > > + - const: pclk > > + > > + clocks: > > + maxItems: 2 > > + > > + num-cs: > > + description: | > > + Number of chip selects used. If a decoder is used, > > + this will be the number of chip selects after the > > + decoder. > > + minimum: 1 > > + maximum: 4 > > + default: 4 > > + > > + is-decoded-cs: > > Needs a type ref. Despite being called a 'flag' looks like it's an > uint32. Presumably, it also needs: > > enum: [ 0, 1 ] Yeah, this is a flag, but the driver source code uses it as u32. So, I will add ref and enum, thanks. > > > + description: | > > + Flag to indicate whether decoder is used or not. > > + default: 0 Best regards, Nobuhiro