[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <0ae2e424-f5d4-83d0-90ea-56acde3d4c9b@ti.com>
Date: Thu, 16 Jul 2020 10:49:24 +0300
From: Roger Quadros <rogerq@...com>
To: Rob Herring <robh@...nel.org>
CC: <vigneshr@...com>, <linux-kernel@...r.kernel.org>,
<robh+dt@...nel.org>, <kishon@...com>, <nsekhar@...com>,
<devicetree@...r.kernel.org>
Subject: Re: [PATCH v3 1/3] dt-binding: phy: convert ti,omap-usb2 to YAML
On 03/07/2020 11:58, Roger Quadros wrote:
> Hi Rob,
>
> On 02/07/2020 23:47, Rob Herring wrote:
>> On Tue, 30 Jun 2020 12:27:27 +0300, Roger Quadros wrote:
>>> Move ti,omap-usb2 to its own YAML schema.
>>>
>>> Signed-off-by: Roger Quadros <rogerq@...com>
>>> Reviewed-by: Rob Herring <robh@...nel.org>
>>> ---
>>> .../devicetree/bindings/phy/ti,omap-usb2.yaml | 69 +++++++++++++++++++
>>> .../devicetree/bindings/phy/ti-phy.txt | 37 ----------
>>> 2 files changed, 69 insertions(+), 37 deletions(-)
>>> create mode 100644 Documentation/devicetree/bindings/phy/ti,omap-usb2.yaml
>>>
>>
>>
>> My bot found errors running 'make dt_binding_check' on your patch:
>>
>> /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/phy/ti,omap-usb2.example.dt.yaml: example-0: phy@...0000:reg:0: [0, 68157440, 0, 84] is too long
>>
>
> In my local build in the dt example I see
>
> phy@...0000:
> compatible: ["ti,am654-usb2", "ti,omap-usb2"]
> reg: [[0x0, 0x4100000, 0x0, 0x54]]
>
> And I don't see any errors. I've updated my dt-schema as well.
I'm able to see the issue now. Will fix it and send v4.
>
>>
>> See https://patchwork.ozlabs.org/patch/1319665
>>
>> If you already ran 'make dt_binding_check' and didn't see the above
>> error(s), then make sure dt-schema is up to date:
>>
>> pip3 install git+https://github.com/devicetree-org/dt-schema.git@master --upgrade
>>
>> Please check and re-submit.
>>
>
cheers,
-roger
--
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki
Powered by blists - more mailing lists