[PATCH v2 1/3] dt-bindings: mailbox: fsl,mu: Fix i.MX 8QXP compatible matching

[PATCH v2 1/3] dt-bindings: mailbox: fsl,mu: Fix i.MX 8QXP compatible matching

[Krzysztof Kozlowski]

The Mailbox on i.MX 8QXP (fsl,imx8qxp-mu) can also be compatible with
fsl,imx8-mu-scu (for fast IPC) so adjust the compatibles to fix
dtbs_check warnings like:

  arch/arm64/boot/dts/freescale/imx8qxp-mek.dt.yaml: mailbox@5d1f0000:
    compatible: ['fsl,imx8-mu-scu', 'fsl,imx8qxp-mu', 'fsl,imx6sx-mu']
    is not valid under any of the given schemas (Possible causes of the failure):

  arch/arm64/boot/dts/freescale/imx8qxp-mek.dt.yaml: mailbox@5d1f0000:
    compatible: ['fsl,imx8-mu-scu', 'fsl,imx8qxp-mu', 'fsl,imx6sx-mu'] is too long

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>

---

Changes since v1:
1. None
---
 Documentation/devicetree/bindings/mailbox/fsl,mu.yaml | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/mailbox/fsl,mu.yaml b/Documentation/devicetree/bindings/mailbox/fsl,mu.yaml
index 8a3470b64d06..7ed096360be2 100644
--- a/Documentation/devicetree/bindings/mailbox/fsl,mu.yaml
+++ b/Documentation/devicetree/bindings/mailbox/fsl,mu.yaml
@@ -38,8 +38,9 @@ properties:
           - const: fsl,imx6sx-mu
       - description: To communicate with i.MX8 SCU with fast IPC
         items:
-          - const: fsl,imx8qxp-mu
           - const: fsl,imx8-mu-scu
+          - const: fsl,imx8qxp-mu
+          - const: fsl,imx6sx-mu
 
   reg:
     maxItems: 1
-- 
2.17.1

[PATCH v2 2/3] dt-bindings: mailbox: fsl,mu: Add missing power-domains

[Krzysztof Kozlowski]

Add quite common property - power-domains - to fix dtbs_check warnings
like:

  arch/arm64/boot/dts/freescale/imx8qxp-mek.dt.yaml:
    mailbox@5d280000: 'power-domains' does not match any of the regexes: 'pinctrl-[0-9]+'

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>

---

Changes since v1:
1. Add missing properties instead of unevaluatedProperties
---
 Documentation/devicetree/bindings/mailbox/fsl,mu.yaml | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/Documentation/devicetree/bindings/mailbox/fsl,mu.yaml b/Documentation/devicetree/bindings/mailbox/fsl,mu.yaml
index 7ed096360be2..e372dcce8037 100644
--- a/Documentation/devicetree/bindings/mailbox/fsl,mu.yaml
+++ b/Documentation/devicetree/bindings/mailbox/fsl,mu.yaml
@@ -72,6 +72,8 @@ properties:
     description: boolean, if present, means it is for side B MU.
     type: boolean
 
+  power-domains: true
+
 required:
   - compatible
   - reg
-- 
2.17.1

Re: [PATCH v2 2/3] dt-bindings: mailbox: fsl,mu: Add missing power-domains

[Rob Herring]

On Thu, Sep 03, 2020 at 09:53:24PM +0200, Krzysztof Kozlowski wrote:
> Add quite common property - power-domains - to fix dtbs_check warnings
> like:
> 
>   arch/arm64/boot/dts/freescale/imx8qxp-mek.dt.yaml:
>     mailbox@5d280000: 'power-domains' does not match any of the regexes: 'pinctrl-[0-9]+'
> 
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> 
> ---
> 
> Changes since v1:
> 1. Add missing properties instead of unevaluatedProperties
> ---
>  Documentation/devicetree/bindings/mailbox/fsl,mu.yaml | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/mailbox/fsl,mu.yaml b/Documentation/devicetree/bindings/mailbox/fsl,mu.yaml
> index 7ed096360be2..e372dcce8037 100644
> --- a/Documentation/devicetree/bindings/mailbox/fsl,mu.yaml
> +++ b/Documentation/devicetree/bindings/mailbox/fsl,mu.yaml
> @@ -72,6 +72,8 @@ properties:
>      description: boolean, if present, means it is for side B MU.
>      type: boolean
>  
> +  power-domains: true

How many?

> +
>  required:
>    - compatible
>    - reg
> -- 
> 2.17.1
> 

[PATCH v2 3/3] MAINTAINERS: Add dt-bindings to mailbox entry

[Krzysztof Kozlowski]

Cover the dt-bindings of mailbox drivers by mailbox maintainers entry.

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>

---

Changes since v1:
1. New patch
---
 MAINTAINERS | 1 +
 1 file changed, 1 insertion(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index bb0cb31d323b..d3d8dedbaf22 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -10417,6 +10417,7 @@ MAILBOX API
 M:	Jassi Brar <jassisinghbrar@gmail.com>
 L:	linux-kernel@vger.kernel.org
 S:	Maintained
+F:	Documentation/devicetree/bindings/mailbox/
 F:	drivers/mailbox/
 F:	include/linux/mailbox_client.h
 F:	include/linux/mailbox_controller.h
-- 
2.17.1

Re: [PATCH v2 3/3] MAINTAINERS: Add dt-bindings to mailbox entry

[Krzysztof Kozlowski]

On Thu, Sep 03, 2020 at 09:53:25PM +0200, Krzysztof Kozlowski wrote:
> Cover the dt-bindings of mailbox drivers by mailbox maintainers entry.
> 
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> 
> ---
> 
> Changes since v1:
> 1. New patch

Jassi,

Any comments on this patch?

Best regards,
Krzysztof

Re: [PATCH v2 1/3] dt-bindings: mailbox: fsl,mu: Fix i.MX 8QXP compatible matching

[Rob Herring]

On Thu, 03 Sep 2020 21:53:23 +0200, Krzysztof Kozlowski wrote:
> The Mailbox on i.MX 8QXP (fsl,imx8qxp-mu) can also be compatible with
> fsl,imx8-mu-scu (for fast IPC) so adjust the compatibles to fix
> dtbs_check warnings like:
> 
>   arch/arm64/boot/dts/freescale/imx8qxp-mek.dt.yaml: mailbox@5d1f0000:
>     compatible: ['fsl,imx8-mu-scu', 'fsl,imx8qxp-mu', 'fsl,imx6sx-mu']
>     is not valid under any of the given schemas (Possible causes of the failure):
> 
>   arch/arm64/boot/dts/freescale/imx8qxp-mek.dt.yaml: mailbox@5d1f0000:
>     compatible: ['fsl,imx8-mu-scu', 'fsl,imx8qxp-mu', 'fsl,imx6sx-mu'] is too long
> 
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> 
> ---
> 
> Changes since v1:
> 1. None
> ---
>  Documentation/devicetree/bindings/mailbox/fsl,mu.yaml | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 

Applied, thanks!