@@ -1,36 +0,0 @@-Atmel SPI device--Required properties:-- compatible : should be "atmel,at91rm9200-spi" or "microchip,sam9x60-spi".-- reg: Address and length of the register set for the device-- interrupts: Should contain spi interrupt-- cs-gpios: chipselects (optional for SPI controller version >= 2 with the- Chip Select Active After Transfer feature).-- clock-names: tuple listing input clock names.- Required elements: "spi_clk"-- clocks: phandles to input clocks.--Optional properties:-- atmel,fifo-size: maximum number of data the RX and TX FIFOs can store for FIFO- capable SPI controllers.--Example:--spi1: spi@fffcc000 {- compatible = "atmel,at91rm9200-spi";- reg = <0xfffcc000 0x4000>;- interrupts = <13 4 5>;- #address-cells = <1>;- #size-cells = <0>;- clocks = <&spi1_clk>;- clock-names = "spi_clk";- cs-gpios = <&pioB 3 0>;- atmel,fifo-size = <32>;-- mmc-slot@0 {- compatible = "mmc-spi-slot";- reg = <0>;- gpios = <&pioC 4 0>; /* CD */- spi-max-frequency = <25000000>;- };-};
--
2.25.1
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
@@ -0,0 +1,82 @@+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)+# Copyright (C) 2022 Microchip Technology, Inc. and its subsidiaries+%YAML1.2+---+$id:http://devicetree.org/schemas/spi/atmel,spi.yaml#+$schema:http://devicetree.org/meta-schemas/core.yaml#++title:Atmel SPI device++maintainers:+-Mark Brown <broonie@kernel.org>
+
+ clocks:
+ maxItems: 1
+
+ atmel,fifo-size:
+ $ref: /schemas/types.yaml#/definitions/uint32
+ description: |
+ Maximum number of data the RX and TX FIFOs can store for FIFO
+ capable SPI controllers.
From: Mark Brown <broonie@kernel.org> Date: 2022-06-29 14:13:25
On Wed, Jun 29, 2022 at 03:58:04PM +0300, Sergiu Moga wrote:
Convert SPI binding for Atmel/Microchip SoCs to Device Tree Schema
format.
Please submit patches using subject lines reflecting the style for the
subsystem, this makes it easier for people to identify relevant patches.
Look at what existing commits in the area you're changing are doing and
make sure your subject lines visually resemble what they're doing.
There's no need to resubmit to fix this alone.
Tudor Ambarus [off-list ref] is our maintainer for SPI
controller, sorry for not having advised you internally about this ;-)
Best regards,
Nicolas
--
Nicolas Ferre
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
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/spi/atmel,spi.example.dtb:0:0: /example-0/spi@fffcc000/mmc@0: failed to match any schema with compatible: ['mmc-spi-slot']
doc reference errors (make refcheckdocs):
See https://patchwork.ozlabs.org/patch/
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.
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
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/spi/atmel,spi.example.dtb:0:0: /example-0/spi@fffcc000/mmc@0: failed to match any schema with compatible: ['mmc-spi-slot']
My conversion of this should be in -next right?
Is this just an incorrect base for the bot, or am I missing
something?
Thanks,
Conor.
doc reference errors (make refcheckdocs):
See https://patchwork.ozlabs.org/patch/
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.
@@ -0,0 +1,82 @@+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)+# Copyright (C) 2022 Microchip Technology, Inc. and its subsidiaries+%YAML1.2+---+$id:http://devicetree.org/schemas/spi/atmel,spi.yaml#+$schema:http://devicetree.org/meta-schemas/core.yaml#++title:Atmel SPI device++maintainers:+-Mark Brown <broonie@kernel.org>
This is wrong. Either this is a fallback or it is not. It's not the
Schroedinger's cat... Maybe your DTS are wrong.
Indeed, this looks unnecessary. This here is only to avoid the error
caused by arm/boot/dts/at91-sam9x60ek.dts, since its spi0 node contains
both compatibles.
+
+ clocks:
+ maxItems: 1
+
+ atmel,fifo-size:
+ $ref: /schemas/types.yaml#/definitions/uint32
+ description: |
+ Maximum number of data the RX and TX FIFOs can store for FIFO
+ capable SPI controllers.
This does not make really sense. You have one clock.
I am sorry about this. Yes, you are right, the original binding did not
really have it either. I just put it here while testing it to ensure
that the property accepts more than just "spi_clk". It seems that I
forgot to remove it before sending it. Thank you for pointing it out!
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/spi/atmel,spi.example.dtb:0:0: /example-0/spi@fffcc000/mmc@0: failed to match any schema with compatible: ['mmc-spi-slot']
doc reference errors (make refcheckdocs):
See https://patchwork.ozlabs.org/patch/
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.
I do not receive this error when running the specified checking command
on my end. I am on the next branch as well and I have just updated
dt-schema. There already seems to be such a schema at
Documentation/devicetree/bindings/mmc/mmc-spi-slot.yaml.
Regards,
Sergiu
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
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/spi/atmel,spi.example.dtb:0:0: /example-0/spi@fffcc000/mmc@0: failed to match any schema with compatible: ['mmc-spi-slot']
My conversion of this should be in -next right?
I don't know, you tell me.
Is this just an incorrect base for the bot, or am I missing
something?
EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
On Wed, Jun 29, 2022 at 10:45:20PM +0000, Conor.Dooley@microchip.com wrote:
quoted
On 29/06/2022 23:36, Rob Herring wrote:
quoted
On Wed, 29 Jun 2022 15:58:04 +0300, Sergiu Moga 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:
Documentation/devicetree/bindings/spi/atmel,spi.example.dtb:0:0: /example-0/spi@fffcc000/mmc@0: failed to match any schema with compatible: ['mmc-spi-slot']
My conversion of this should be in -next right?
I don't know, you tell me.
Well, I was hoping Sergiu knew, been/am away so hard to keep track!
quoted
Is this just an incorrect base for the bot, or am I missing
something?
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/spi/atmel,spi.example.dtb:0:0: /example-0/spi@fffcc000/mmc@0: failed to match any schema with compatible: ['mmc-spi-slot']
My conversion of this should be in -next right?
Aren't you talking about
Documentation/devicetree/bindings/spi/microchip,mpfs-spi.yaml or
Documentation/devicetree/bindings/fpga/microchip,mpf-spi-fpga-mgr.yaml
rather than atmel,spi.yaml ?
Is this just an incorrect base for the bot, or am I missing
something?
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/spi/atmel,spi.example.dtb:0:0: /example-0/spi@fffcc000/mmc@0: failed to match any schema with compatible: ['mmc-spi-slot']
My conversion of this should be in -next right?
Aren't you talking about
Documentation/devicetree/bindings/spi/microchip,mpfs-spi.yaml or Documentation/devicetree/bindings/fpga/microchip,mpf-spi-fpga-mgr.yaml
rather than atmel,spi.yaml ?
Nope, I converted mmc-spi-slot :)
And I checked, it is in -next 226e09de0acd ("dt-bindings: mmc:
convert mmc-spi-slot to yaml"))
Was just pointing out that this error from the bot is not really
a problem.
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
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/spi/atmel,spi.example.dtb:0:0: /example-0/spi@fffcc000/mmc@0: failed to match any schema with compatible: ['mmc-spi-slot']
My conversion of this should be in -next right?
Aren't you talking about
Documentation/devicetree/bindings/spi/microchip,mpfs-spi.yaml or Documentation/devicetree/bindings/fpga/microchip,mpf-spi-fpga-mgr.yaml
rather than atmel,spi.yaml ?
Nope, I converted mmc-spi-slot :)
And I checked, it is in -next 226e09de0acd ("dt-bindings: mmc:
convert mmc-spi-slot to yaml"))
Was just pointing out that this error from the bot is not really
a problem.
Ah, all right Conor. Thanks for the info.
Best regards,
Nicolas
--
Nicolas Ferre
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel