Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Tue, 3 May 2022 23:38:39 +0200
From:   Alexandre Belloni <alexandre.belloni@bootlin.com>
To:     Rob Herring <robh@kernel.org>
Cc:     Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        linux-i3c@lists.infradead.org, devicetree@vger.kernel.org,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] dt-bindings: i3c: Convert snps,dw-i3c-master to DT schema
Message-ID: <YnGg31lPwxAQOUtg@mail.local>
References: <20220422192236.2594577-1-robh@kernel.org>
 <CAL_Jsq+t5e-Dr0z9Zd_+jaqwb1afEVx4SdjbQZA4bHpWf8a2jg@mail.gmail.com>
 <YmMD3RPXfPdiYXyg@mail.local>
 <Ymb61YHl62AfSe2R@robh.at.kernel.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <Ymb61YHl62AfSe2R@robh.at.kernel.org>
Precedence: bulk

On 25/04/2022 14:47:33-0500, Rob Herring wrote:
> On Fri, Apr 22, 2022 at 09:37:01PM +0200, Alexandre Belloni wrote:
> > On 22/04/2022 14:28:53-0500, Rob Herring wrote:
> > > On Fri, Apr 22, 2022 at 2:22 PM Rob Herring <robh@kernel.org> wrote:
> > > >
> > > > Convert the Synopsys Designware I3C master to DT schema format.
> > > >
> > > > Signed-off-by: Rob Herring <robh@kernel.org>
> > > > ---
> > > >  .../bindings/i3c/snps,dw-i3c-master.txt       | 41 ---------------
> > > >  .../bindings/i3c/snps,dw-i3c-master.yaml      | 52 +++++++++++++++++++
> > > >  2 files changed, 52 insertions(+), 41 deletions(-)
> > > >  delete mode 100644 Documentation/devicetree/bindings/i3c/snps,dw-i3c-master.txt
> > > >  create mode 100644 Documentation/devicetree/bindings/i3c/snps,dw-i3c-master.yaml
> > > >
> > > > diff --git a/Documentation/devicetree/bindings/i3c/snps,dw-i3c-master.txt b/Documentation/devicetree/bindings/i3c/snps,dw-i3c-master.txt
> > > > deleted file mode 100644
> > > > index 07f35f36085d..000000000000
> > > > --- a/Documentation/devicetree/bindings/i3c/snps,dw-i3c-master.txt
> > > > +++ /dev/null
> > > > @@ -1,41 +0,0 @@
> > > > -Bindings for Synopsys DesignWare I3C master block
> > > > -=================================================
> > > > -
> > > > -Required properties:
> > > > ---------------------
> > > > -- compatible: shall be "snps,dw-i3c-master-1.00a"
> > > > -- clocks: shall reference the core_clk
> > > > -- interrupts: the interrupt line connected to this I3C master
> > > > -- reg: Offset and length of I3C master registers
> > > > -
> > > > -Mandatory properties defined by the generic binding (see
> > > > -Documentation/devicetree/bindings/i3c/i3c.yaml for more details):
> > > > -
> > > > -- #address-cells: shall be set to 3
> > > > -- #size-cells: shall be set to 0
> > > > -
> > > > -Optional properties defined by the generic binding (see
> > > > -Documentation/devicetree/bindings/i3c/i3c.yaml for more details):
> > > > -
> > > > -- i2c-scl-hz
> > > > -- i3c-scl-hz
> > > > -
> > > > -I3C device connected on the bus follow the generic description (see
> > > > -Documentation/devicetree/bindings/i3c/i3c.yaml for more details).
> > > > -
> > > > -Example:
> > > > -
> > > > -       i3c-master@2000 {
> > > > -               compatible = "snps,dw-i3c-master-1.00a";
> > > > -               #address-cells = <3>;
> > > > -               #size-cells = <0>;
> > > > -               reg = <0x02000 0x1000>;
> > > > -               interrupts = <0>;
> > > > -               clocks = <&i3cclk>;
> > > > -
> > > > -               eeprom@57{
> > > > -                       compatible = "atmel,24c01";
> > > > -                       reg = <0x57 0x0 0x10>;
> > > > -                       pagesize = <0x8>;
> > > > -               };
> > > > -       };
> > > > diff --git a/Documentation/devicetree/bindings/i3c/snps,dw-i3c-master.yaml b/Documentation/devicetree/bindings/i3c/snps,dw-i3c-master.yaml
> > > > new file mode 100644
> > > > index 000000000000..5b8b43e6f1e4
> > > > --- /dev/null
> > > > +++ b/Documentation/devicetree/bindings/i3c/snps,dw-i3c-master.yaml
> > > > @@ -0,0 +1,52 @@
> > > > +# SPDX-License-Identifier: GPL-2.0
> > > > +%YAML 1.2
> > > > +---
> > > > +$id: http://devicetree.org/schemas/i3c/snps,dw-i3c-master.yaml#
> > > > +$schema: http://devicetree.org/meta-schemas/core.yaml#
> > > > +
> > > > +title: Synopsys DesignWare I3C master block
> > > > +
> > > > +maintainers:
> > > > +  - Vitor Soares <vitor.soares@synopsys.com>
> > > 
> > > And it bounces. Someone else want it?
> > > 
> > 
> > Probably no one but me.
> 
> Can you change it to yourself when applying?
> 

Done!


-- 
Alexandre Belloni, co-owner and COO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com