Re: [PATCH] dt-bindings: PCI: uniphier: Convert uniphier-pcie.txt to json-schema

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Rob,

On 2022/03/29 1:23, Rob Herring wrote:
On Mon, Mar 28, 2022 at 07:51:05AM -0500, Rob Herring wrote:
On Mon, 28 Mar 2022 11:11:38 +0900, Kunihiko Hayashi wrote:
Convert the file into a JSON description at the yaml format.

Signed-off-by: Kunihiko Hayashi <hayashi.kunihiko@xxxxxxxxxxxxx>
---
  .../bindings/pci/socionext,uniphier-pcie.yaml | 100
++++++++++++++++++
  .../devicetree/bindings/pci/uniphier-pcie.txt |  82 --------------
  MAINTAINERS                                   |   2 +-
  3 files changed, 101 insertions(+), 83 deletions(-)
  create mode 100644
Documentation/devicetree/bindings/pci/socionext,uniphier-pcie.yaml
  delete mode 100644
Documentation/devicetree/bindings/pci/uniphier-pcie.txt


Running 'make dtbs_check' with the schema in this patch gives the
following warnings. Consider if they are expected or the schema is
incorrect. These may not be new warnings.

Note that it is not yet a requirement to have 0 warnings for dtbs_check.
This will change in the future.

Full log is available here: https://patchwork.ozlabs.org/patch/1609988


pcie@66000000: compatible: ['socionext,uniphier-pcie', 'snps,dw-pcie']
is too long
	arch/arm64/boot/dts/socionext/uniphier-ld20-akebi96.dt.yaml
	arch/arm64/boot/dts/socionext/uniphier-ld20-global.dt.yaml
	arch/arm64/boot/dts/socionext/uniphier-ld20-ref.dt.yaml
	arch/arm64/boot/dts/socionext/uniphier-pxs3-ref.dt.yaml

Ignore this if your intent is to fix these by dropping 'snps,dw-pcie'. I
think that is the right thing to do. 'snps,dw-pcie' is not too
meaningful.

I see. I should remove "snps,dw-pcie" from the existing devicetree, so
I'll fix it as a devicetree patch.

Thank you,

---
Best Regards
Kunihiko Hayashi



[Index of Archives]     [DMA Engine]     [Linux Coverity]     [Linux USB]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Greybus]

  Powered by Linux