Hi Serge.
> -----Original Message-----
> From: Serge Semin <sergey.se...@baikalelectronics.ru>
> Sent: Saturday, December 5, 2020 11:24 PM
> To: Nyman, Mathias <mathias.ny...@intel.com>; Felipe Balbi
> <ba...@kernel.org>; Krzysztof Kozlowski <k...@kernel.org>; Greg Kroah-
> Hartman <gre...@linuxfoundation.org>; Rob Herring
> <robh...@kernel.org>; Chunfeng Yun <chunfeng....@mediatek.com>;
> Wan Mohamad, Wan Ahmad Zainie
> <wan.ahmad.zainie.wan.moha...@intel.com>
> Cc: Serge Semin <sergey.se...@baikalelectronics.ru>; Serge Semin
> <fancer.lan...@gmail.com>; Alexey Malahov
> <alexey.mala...@baikalelectronics.ru>; Pavel Parkhomenko
> <pavel.parkhome...@baikalelectronics.ru>; Andy Gross
> <agr...@kernel.org>; Bjorn Andersson <bjorn.anders...@linaro.org>;
> Manu Gautam <mgau...@codeaurora.org>; Roger Quadros
> <rog...@ti.com>; Lad Prabhakar <prabhakar.mahadev-
> lad...@bp.renesas.com>; Yoshihiro Shimoda
> <yoshihiro.shimoda...@renesas.com>; narmstrong
> <narmstr...@baylibre.com>; Kevin Hilman <khil...@baylibre.com>;
> Martin Blumenstingl <martin.blumensti...@googlemail.com>; linux-arm-
> ker...@lists.infradead.org; linux-snps-...@lists.infradead.org; linux-
> m...@vger.kernel.org; linuxppc-...@lists.ozlabs.org; linux-
> u...@vger.kernel.org; devicet...@vger.kernel.org; linux-
> ker...@vger.kernel.org
> Subject: [PATCH v5 19/19] dt-bindings: usb: intel,keembay-dwc3: Validate
> DWC3 sub-node
>
> Intel Keem Bay DWC3 compatible DT nodes are supposed to have a DWC
> USB3 compatible sub-node to describe a fully functioning USB interface. Let's
> use the available DWC USB3 DT schema to validate the Qualcomm DWC3 sub-
> nodes.
>
> Note since the generic DWC USB3 DT node is supposed to be named as
> generic USB HCD ("^usb(@.*)?") one we have to accordingly fix the sub-
> nodes name regexp and fix the DT node example.
>
> Signed-off-by: Serge Semin <sergey.se...@baikalelectronics.ru>
LGTM. With minor change to fix the typo above, Qualcomm to Intel
Keem Bay,
Acked-by: Wan Ahmad Zainie <wan.ahmad.zainie.wan.moha...@intel.com>
>
> ---
>
> Changelog v5:
> - This is a new patch created for the new Intel Keem Bay bindings file,
> which has been added just recently.
> ---
> .../devicetree/bindings/usb/intel,keembay-dwc3.yaml | 9 +++------
> 1 file changed, 3 insertions(+), 6 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/usb/intel,keembay-
> dwc3.yaml b/Documentation/devicetree/bindings/usb/intel,keembay-
> dwc3.yaml
> index dd32c10ce6c7..43b91ab62004 100644
> --- a/Documentation/devicetree/bindings/usb/intel,keembay-dwc3.yaml
> +++ b/Documentation/devicetree/bindings/usb/intel,keembay-dwc3.yaml
> @@ -34,11 +34,8 @@ properties:
> # Required child node:
>
> patternProperties:
> - "^dwc3@[0-9a-f]+$":
> - type: object
> - description:
> - A child node must exist to represent the core DWC3 IP block.
> - The content of the node is defined in dwc3.txt.
> + "^usb@[0-9a-f]+$":
> + $ref: snps,dwc3.yaml#
>
> required:
> - compatible
> @@ -68,7 +65,7 @@ examples:
> #address-cells = <1>;
> #size-cells = <1>;
>
> - dwc3@34000000 {
> + usb@34000000 {
> compatible = "snps,dwc3";
> reg = <0x34000000 0x10000>;
> interrupts = <GIC_SPI 91 IRQ_TYPE_LEVEL_HIGH>;
> --
> 2.29.2
Best regards,
Zainie
