Re: [PATCH v1 2/4] dt-bindings: bluetooth: marvell: add max-speed property

From: Krzysztof Kozlowski
Date: Thu Jan 19 2023 - 06:46:13 EST


On 18/01/2023 13:28, Francesco Dolcini wrote:
> From: Stefan Eichenberger <stefan.eichenberger@xxxxxxxxxxx>
>
> The 88W8997 bluetooth module supports setting the max-speed property.
>
> Signed-off-by: Stefan Eichenberger <stefan.eichenberger@xxxxxxxxxxx>
> Signed-off-by: Francesco Dolcini <francesco.dolcini@xxxxxxxxxxx>
> ---
> .../bindings/net/marvell-bluetooth.yaml | 16 ++++++++++++++++
> 1 file changed, 16 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/net/marvell-bluetooth.yaml b/Documentation/devicetree/bindings/net/marvell-bluetooth.yaml
> index 83b64ed730f5..2fccea30c58d 100644
> --- a/Documentation/devicetree/bindings/net/marvell-bluetooth.yaml
> +++ b/Documentation/devicetree/bindings/net/marvell-bluetooth.yaml
> @@ -19,9 +19,25 @@ properties:
> - mrvl,88w8897
> - mrvl,88w8997
>
> + max-speed:
> + description: see Documentation/devicetree/bindings/serial/serial.yaml
> +
> required:
> - compatible
>
> +allOf:
> + - if:
> + properties:

Wrong indentation, missing two spaces.

Does not look like you tested the bindings. Please run `make
dt_binding_check` (see
Documentation/devicetree/bindings/writing-schema.rst for instructions).

Best regards,
Krzysztof