| Message ID | 20210721140424.725744-28-maxime@cerno.tech (mailing list archive) |
|---|---|
| State | Not Applicable |
| Delegated to: | Netdev Maintainers |
| Headers | show |
| Series | None | expand |
On Wed, 21 Jul 2021 16:03:57 +0200, Maxime Ripard wrote: > The ESP8089 Wireless Chip is supported by Linux (through an out-of-tree > driver) thanks to its device tree binding. > > Now that we have the DT validation in place, let's convert the device > tree bindings for that driver over to a YAML schema. > > Cc: "David S. Miller" <davem@davemloft.net> > Cc: de Goede <hdegoede@redhat.com> > Cc: Jakub Kicinski <kuba@kernel.org> > Cc: Kalle Valo <kvalo@codeaurora.org> > Cc: linux-wireless@vger.kernel.org > Cc: netdev@vger.kernel.org > Signed-off-by: Maxime Ripard <maxime@cerno.tech> > --- > .../bindings/net/wireless/esp,esp8089.txt | 30 ------------- > .../bindings/net/wireless/esp,esp8089.yaml | 43 +++++++++++++++++++ > 2 files changed, 43 insertions(+), 30 deletions(-) > delete mode 100644 Documentation/devicetree/bindings/net/wireless/esp,esp8089.txt > create mode 100644 Documentation/devicetree/bindings/net/wireless/esp,esp8089.yaml > Reviewed-by: Rob Herring <robh@kernel.org>
Maxime Ripard <maxime@cerno.tech> wrote: > The ESP8089 Wireless Chip is supported by Linux (through an out-of-tree > driver) thanks to its device tree binding. > > Now that we have the DT validation in place, let's convert the device > tree bindings for that driver over to a YAML schema. > > Cc: "David S. Miller" <davem@davemloft.net> > Cc: de Goede <hdegoede@redhat.com> > Cc: Jakub Kicinski <kuba@kernel.org> > Cc: Kalle Valo <kvalo@codeaurora.org> > Cc: linux-wireless@vger.kernel.org > Cc: netdev@vger.kernel.org > Signed-off-by: Maxime Ripard <maxime@cerno.tech> > Reviewed-by: Rob Herring <robh@kernel.org> We support out-of-tree drivers in DT? Via which tree is this supposed to go? I guess not via wireless-drivers-next as this is an out-of-tree driver.
Hi Kalle, On Fri, Aug 06, 2021 at 08:47:09AM +0000, Kalle Valo wrote: > Maxime Ripard <maxime@cerno.tech> wrote: > > > The ESP8089 Wireless Chip is supported by Linux (through an out-of-tree > > driver) thanks to its device tree binding. > > > > Now that we have the DT validation in place, let's convert the device > > tree bindings for that driver over to a YAML schema. > > > > Cc: "David S. Miller" <davem@davemloft.net> > > Cc: de Goede <hdegoede@redhat.com> > > Cc: Jakub Kicinski <kuba@kernel.org> > > Cc: Kalle Valo <kvalo@codeaurora.org> > > Cc: linux-wireless@vger.kernel.org > > Cc: netdev@vger.kernel.org > > Signed-off-by: Maxime Ripard <maxime@cerno.tech> > > Reviewed-by: Rob Herring <robh@kernel.org> > > We support out-of-tree drivers in DT? Yeah, as long as the binding is stable we can merge it. This controller is one of these examples, but we supported multiple GPU that way too. > Via which tree is this supposed to go? I guess not via > wireless-drivers-next as this is an out-of-tree driver. It's up to you I guess. It was already part of the dt-bindings wireless directory, so either your tree or the DT maintainers if you'd prefer not to. Maxime
diff --git a/Documentation/devicetree/bindings/net/wireless/esp,esp8089.txt b/Documentation/devicetree/bindings/net/wireless/esp,esp8089.txt deleted file mode 100644 index 6830c4786f8a..000000000000 --- a/Documentation/devicetree/bindings/net/wireless/esp,esp8089.txt +++ /dev/null @@ -1,30 +0,0 @@ -Espressif ESP8089 wireless SDIO devices - -This node provides properties for controlling the ESP8089 wireless device. -The node is expected to be specified as a child node to the SDIO controller -that connects the device to the system. - -Required properties: - - - compatible : Should be "esp,esp8089". - -Optional properties: - - esp,crystal-26M-en: Integer value for the crystal_26M_en firmware parameter - -Example: - -&mmc1 { - #address-cells = <1>; - #size-cells = <0>; - - vmmc-supply = <®_dldo1>; - mmc-pwrseq = <&wifi_pwrseq>; - bus-width = <4>; - non-removable; - - esp8089: sdio_wifi@1 { - compatible = "esp,esp8089"; - reg = <1>; - esp,crystal-26M-en = <2>; - }; -}; diff --git a/Documentation/devicetree/bindings/net/wireless/esp,esp8089.yaml b/Documentation/devicetree/bindings/net/wireless/esp,esp8089.yaml new file mode 100644 index 000000000000..284ef45add99 --- /dev/null +++ b/Documentation/devicetree/bindings/net/wireless/esp,esp8089.yaml @@ -0,0 +1,43 @@ +# SPDX-License-Identifier: GPL-2.0 +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/net/wireless/esp,esp8089.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: Espressif ESP8089 Device Tree Bindings + +maintainers: + - Hans de Goede <hdegoede@redhat.com> + +properties: + compatible: + const: esp,esp8089 + + reg: + maxItems: 1 + + esp,crystal-26M-en: + $ref: /schemas/types.yaml#/definitions/uint32 + description: > + Value for the crystal_26M_en firmware parameter + +required: + - compatible + - reg + +additionalProperties: false + +examples: + - | + mmc { + #address-cells = <1>; + #size-cells = <0>; + + wifi@1 { + compatible = "esp,esp8089"; + reg = <1>; + esp,crystal-26M-en = <2>; + }; + }; + +...
The ESP8089 Wireless Chip is supported by Linux (through an out-of-tree driver) thanks to its device tree binding. Now that we have the DT validation in place, let's convert the device tree bindings for that driver over to a YAML schema. Cc: "David S. Miller" <davem@davemloft.net> Cc: de Goede <hdegoede@redhat.com> Cc: Jakub Kicinski <kuba@kernel.org> Cc: Kalle Valo <kvalo@codeaurora.org> Cc: linux-wireless@vger.kernel.org Cc: netdev@vger.kernel.org Signed-off-by: Maxime Ripard <maxime@cerno.tech> --- .../bindings/net/wireless/esp,esp8089.txt | 30 ------------- .../bindings/net/wireless/esp,esp8089.yaml | 43 +++++++++++++++++++ 2 files changed, 43 insertions(+), 30 deletions(-) delete mode 100644 Documentation/devicetree/bindings/net/wireless/esp,esp8089.txt create mode 100644 Documentation/devicetree/bindings/net/wireless/esp,esp8089.yaml