Message ID | 20230514165651.2199-1-jszhang@kernel.org |
---|---|
Headers | show |
Series | riscv: add Bouffalolab bl808 support | expand |
On Mon, 15 May 2023 00:56:44 +0800, Jisheng Zhang wrote: > Add bindings doc for Bouffalolab UART Driver > > Signed-off-by: Jisheng Zhang <jszhang@kernel.org> > Acked-by: Palmer Dabbelt <palmer@rivosinc.com> > --- > .../serial/bouffalolab,bl808-uart.yaml | 47 +++++++++++++++++++ > 1 file changed, 47 insertions(+) > create mode 100644 Documentation/devicetree/bindings/serial/bouffalolab,bl808-uart.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: ./Documentation/devicetree/bindings/serial/bouffalolab,bl808-uart.yaml: $id: relative path/filename doesn't match actual path or filename expected: http://devicetree.org/schemas/serial/bouffalolab,bl808-uart.yaml# See https://patchwork.ozlabs.org/patch/1781053 This check can fail if there are any dependencies. The base for a patch series is generally the most recent rc1. If you already ran 'make dt_binding_check' and didn't see the above error(s), then make sure 'yamllint' is installed and dt-schema is up to date: pip3 install dtschema --upgrade Please check and re-submit.
On Mon, May 15, 2023 at 07:28:10AM +0800, Jisheng Zhang wrote: > On Sun, May 14, 2023 at 07:17:27PM +0100, Conor Dooley wrote: > > On Mon, May 15, 2023 at 12:56:44AM +0800, Jisheng Zhang wrote: > > > > > +$id: http://devicetree.org/schemas/serial/bouffalolab,uart.yaml# > > > > $id: relative path/filename doesn't match actual path or filename > > expected: http://devicetree.org/schemas/serial/bouffalolab,bl808-uart.yaml# > > > > Please test the bindings before submission - even dtbs_check catches > > that one! > > Aha, I knew the reason. I did run dt_binding_check and dtbs_check, > but then I read one of comments in v2 which suggests the renaming, > that's to say the dtbs_check is done before the renmaing. Sorry for > confusion No worries. Serial aisde, rest of the series looks grand to me, perhaps you want to resubmit the serial bits separately & I'll grab the riscv bits once the serial side is sorted out?