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:   Thu, 2 Jun 2022 09:29:17 -0500
From:   Rob Herring <robh@kernel.org>
To:     Palmer Dabbelt <palmer@rivosinc.com>
Cc:     kuba@kernel.org, michael.hennerich@analog.com, davem@davemloft.net,
        edumazet@google.com, pabeni@redhat.com,
        krzysztof.kozlowski+dt@linaro.org, alexandru.ardelean@analog.com,
        netdev@vger.kernel.org, devicetree@vger.kernel.org,
        linux-kernel@vger.kernel.org, linux@rivosinc.com
Subject: Re: [PATCH] dt-bindings: net: adin: Escape a trailing ":"
Message-ID: <20220602142917.GA2254348-robh@kernel.org>
References: <20220602012809.8384-1-palmer@rivosinc.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20220602012809.8384-1-palmer@rivosinc.com>
Precedence: bulk

On Wed, Jun 01, 2022 at 06:28:09PM -0700, Palmer Dabbelt wrote:
> From: Palmer Dabbelt <palmer@rivosinc.com>
> 
> 1f77204e11f8 ("dt-bindings: net: adin: document phy clock output
> properties") added a line with a ":" at the end, which is tripping up my
> attempts to run the DT schema checks due to this being invalid YAML
> syntax.  I get a schema check failure with the following error
> 
>     ruamel.yaml.scanner.ScannerError: mapping values are not allowed in this context
> 
> This just escapes the line in question, so it can parse.
> 
> Fixes: 1f77204e11f8 ("dt-bindings: net: adin: document phy clock output properties")
> Signed-off-by: Palmer Dabbelt <palmer@rivosinc.com>
> ---
>  Documentation/devicetree/bindings/net/adi,adin.yaml | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)

Already have a fix queued in netdev.

Rob