On Thu, Jul 07, 2022 at 08:53:32AM +0200, Krzysztof Kozlowski wrote: > On 07/07/2022 08:17, Thierry Reding wrote: > > On Wed, Jul 06, 2022 at 10:59:49PM -0600, Rob Herring wrote: > >> On Wed, 06 Jul 2022 23:32:51 +0200, Thierry Reding wrote: > >>> From: Bhadram Varka <vbhadram@xxxxxxxxxx> > >>> > >>> Add device-tree binding documentation for the Multi-Gigabit Ethernet > >>> (MGBE) controller found on NVIDIA Tegra234 SoCs. > >>> > >>> Signed-off-by: Jon Hunter <jonathanh@xxxxxxxxxx> > >>> Signed-off-by: Bhadram Varka <vbhadram@xxxxxxxxxx> > >>> Signed-off-by: Thierry Reding <treding@xxxxxxxxxx> > >>> --- > >>> Changes in v3: > >>> - add macsec and macsec-ns interrupt names > >>> - improve mdio bus node description > >>> - drop power-domains description > >>> - improve bindings title > >>> > >>> Changes in v2: > >>> - add supported PHY modes > >>> - change to dual license > >>> > >>> .../bindings/net/nvidia,tegra234-mgbe.yaml | 169 ++++++++++++++++++ > >>> 1 file changed, 169 insertions(+) > >>> create mode 100644 Documentation/devicetree/bindings/net/nvidia,tegra234-mgbe.yaml > >>> > >> > >> My bot found errors running 'make DT_CHECKER_FLAGS=-m dt_binding_check' > >> on your patch (DT_CHECKER_FLAGS is new in v5.13): > >> > >> yamllint warnings/errors: > >> > >> dtschema/dtc warnings/errors: > >> Error: Documentation/devicetree/bindings/net/nvidia,tegra234-mgbe.example.dts:53.34-35 syntax error > >> FATAL ERROR: Unable to parse input tree > > > > This is an error that you'd get if patch 3 is not applied. Not sure if I > > managed to confuse the bot somehow, but I cannot reproduce this if I > > apply the series on top of v5.19-rc1 or linux-next. > > Patch number 3 does not apply on v5.19-rc1 or linux-next, so maybe the > bot (which applies on rc1) did not have it. Good point. I'll rebase v4 on top of v5.19-rc1 then. This shouldn't cause a problem for net-next because there's no conflict there for patch 9. I did notice that the devicetree-bindings patchwork instance doesn't have all of the patches, so perhaps that tripped up the bot as well. Not sure what happened there, the linux-tegra instance has all 9 patches. Thierry
Attachment:
signature.asc
Description: PGP signature