From: Rob Herring <robh@kernel.org>
Sent: Friday, September 10, 2021 11:58 PM

On Thu, Sep 9, 2021 at 8:38 PM ChiaWei Wang
[off-list ref] wrote:

quoted

Hi Rob,

quoted

From: Rob Herring <robh@kernel.org>
Sent: Thursday, September 9, 2021 8:33 PM

On Thu, 09 Sep 2021 18:29:05 +0800, Chia-Wei Wang wrote:

quoted

Add dt-bindings for Aspeed UART routing controller.

Signed-off-by: Oskar Senft <redacted>
Signed-off-by: Chia-Wei Wang <redacted>
---
 .../bindings/soc/aspeed/uart-routing.yaml     | 70

+++++++++++++++++++

quoted

 1 file changed, 70 insertions(+)
 create mode 100644
Documentation/devicetree/bindings/soc/aspeed/uart-routing.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/soc/aspeed/uart-routing.example.dt.yaml:

quoted

quoted

0:0: /example-0/lpc at 1e789000: failed to match any schema with

compatible:

quoted

quoted

['aspeed,ast2600-lpc-v2', 'simple-mfd', 'syscon']

The "aspeed,ast2600-lpc-v2" compatible string is described in the .txt file

'aspeed-lpc.txt'.

quoted

Do we have to firstly convert the text file into YAML one to resolve this

dependency issue?

Yes. This is a significant change to the LPC block binding.

Understood. Will send a v4 patch which includes the YAML conversion.
Thanks.

Chiawei