On Mon, Jul 19, 2021 at 3:47 PM Rob Herring <robh@xxxxxxxxxx> wrote: > 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: > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/bus/intel,ixp4xx-expansion-bus-controller.example.dt.yaml: serial@1,0: compatible: 'oneOf' conditional failed, one must be fixed: > ['exar,xr16l2551', 'ns8250'] is too long > Additional items are not allowed ('ns8250' was unexpected) > ['exar,xr16l2551', 'ns8250'] is too short This is because the patch adding these compatibles was sent separately so it can be merged into the tty subsystem. So pretty much false alarm, but the bot can't know everything. Yours, Linus Walleij