From miquel.raynal at bootlin.com  Thu Aug  3 02:04:08 2023
From: miquel.raynal at bootlin.com (Miquel Raynal)
Date: Thu, 3 Aug 2023 11:04:08 +0200
Subject: [PATCH -next] i3c: master: svc: Do not check for 0 return after
 calling platform_get_irq()
In-Reply-To: <20230803085149.149248-1-ruanjinjie@huawei.com>
References: <20230803085149.149248-1-ruanjinjie@huawei.com>
Message-ID: <20230803110408.48bbe8ec@xps-13>

Hi Ruan,

ruanjinjie at huawei.com wrote on Thu, 3 Aug 2023 16:51:49 +0800:

> It is not possible for platform_get_irq() to return 0. Use the
> return value from platform_get_irq().

Reviewed-by: Miquel Raynal <miquel.raynal at bootlin.com>

> 
> Signed-off-by: Ruan Jinjie <ruanjinjie at huawei.com>
> ---
>  drivers/i3c/master/svc-i3c-master.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/i3c/master/svc-i3c-master.c b/drivers/i3c/master/svc-i3c-master.c
> index 0d63b732ef0c..770b40e28015 100644
> --- a/drivers/i3c/master/svc-i3c-master.c
> +++ b/drivers/i3c/master/svc-i3c-master.c
> @@ -1518,8 +1518,8 @@ static int svc_i3c_master_probe(struct platform_device *pdev)
>  		return PTR_ERR(master->sclk);
>  
>  	master->irq = platform_get_irq(pdev, 0);
> -	if (master->irq <= 0)
> -		return -ENOENT;
> +	if (master->irq < 0)
> +		return master->irq;
>  
>  	master->dev = dev;
>  


Thanks,
Miqu?l


From billy_tsai at aspeedtech.com  Wed Aug  2 03:09:09 2023
From: billy_tsai at aspeedtech.com (Billy Tsai)
Date: Wed, 2 Aug 2023 18:09:09 +0800
Subject: [PATCH] i3c/master: cmd_v1: Fix the exit criteria for the daa procedure
Message-ID: <20230802100909.2568215-1-billy_tsai@aspeedtech.com>

The exit criteria for the DAA should check if the data length is equal to
1, instead of checking if the response status is equal to 1.

Signed-off-by: Billy Tsai <billy_tsai at aspeedtech.com>
---
 drivers/i3c/master/mipi-i3c-hci/cmd_v1.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/i3c/master/mipi-i3c-hci/cmd_v1.c b/drivers/i3c/master/mipi-i3c-hci/cmd_v1.c
index d97c3175e0e2..6a781f89b0e4 100644
--- a/drivers/i3c/master/mipi-i3c-hci/cmd_v1.c
+++ b/drivers/i3c/master/mipi-i3c-hci/cmd_v1.c
@@ -339,7 +339,7 @@ static int hci_cmd_v1_daa(struct i3c_hci *hci)
 			break;
 		}
 		if (RESP_STATUS(xfer[0].response) == RESP_ERR_NACK &&
-		    RESP_STATUS(xfer[0].response) == 1) {
+		    RESP_DATA_LENGTH(xfer->response) == 1) {
 			ret = 0;  /* no more devices to be assigned */
 			break;
 		}
-- 
2.25.1



From ruanjinjie at huawei.com  Thu Aug  3 01:51:49 2023
From: ruanjinjie at huawei.com (Ruan Jinjie)
Date: Thu, 3 Aug 2023 16:51:49 +0800
Subject: [PATCH -next] i3c: master: svc: Do not check for 0 return after calling platform_get_irq()
Message-ID: <20230803085149.149248-1-ruanjinjie@huawei.com>

It is not possible for platform_get_irq() to return 0. Use the
return value from platform_get_irq().

Signed-off-by: Ruan Jinjie <ruanjinjie at huawei.com>
---
 drivers/i3c/master/svc-i3c-master.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/i3c/master/svc-i3c-master.c b/drivers/i3c/master/svc-i3c-master.c
index 0d63b732ef0c..770b40e28015 100644
--- a/drivers/i3c/master/svc-i3c-master.c
+++ b/drivers/i3c/master/svc-i3c-master.c
@@ -1518,8 +1518,8 @@ static int svc_i3c_master_probe(struct platform_device *pdev)
 		return PTR_ERR(master->sclk);
 
 	master->irq = platform_get_irq(pdev, 0);
-	if (master->irq <= 0)
-		return -ENOENT;
+	if (master->irq < 0)
+		return master->irq;
 
 	master->dev = dev;
 
-- 
2.34.1



From krzysztof.kozlowski at linaro.org  Tue Aug  8 08:59:38 2023
From: krzysztof.kozlowski at linaro.org (Krzysztof Kozlowski)
Date: Tue, 8 Aug 2023 17:59:38 +0200
Subject: [PATCH 2/3] dt-bindings: i3c: ast2600: Add resets and reset-names
In-Reply-To: <20230808154241.749641-3-dylan_hung@aspeedtech.com>
References: <20230808154241.749641-1-dylan_hung@aspeedtech.com>
 <20230808154241.749641-3-dylan_hung@aspeedtech.com>
Message-ID: <49edf46a-b35c-1135-c500-2382a453e58d@linaro.org>

On 08/08/2023 17:42, Dylan Hung wrote:
> Add two reset lines to the AST2600 I3C controller:
> - core_rst: the reset line of the controller itself
> - global_rst: the reset line of the I3C global register block. Since all
> six I3C controllers in AST2600 share this global register block, the
> driver needs to handle this carefully. Generally, this reset line should
> only need to be de-asserted.

The commit msg does not explain why this is now required.

> 
> Signed-off-by: Dylan Hung <dylan_hung at aspeedtech.com>
> ---
>  .../devicetree/bindings/i3c/aspeed,ast2600-i3c.yaml  | 12 ++++++++++--
>  1 file changed, 10 insertions(+), 2 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/i3c/aspeed,ast2600-i3c.yaml b/Documentation/devicetree/bindings/i3c/aspeed,ast2600-i3c.yaml
> index fcc3dbff9c9a..3166d6f3a39c 100644
> --- a/Documentation/devicetree/bindings/i3c/aspeed,ast2600-i3c.yaml
> +++ b/Documentation/devicetree/bindings/i3c/aspeed,ast2600-i3c.yaml
> @@ -23,7 +23,12 @@ properties:
>      maxItems: 1
>  
>    resets:
> -    maxItems: 1
> +    maxItems: 2
> +
> +  reset-names:
> +    items:
> +      - const: core_rst
> +      - const: global_rst
>  

Drop "_rst" suffixes from both.

>    interrupts:
>      maxItems: 1
> @@ -48,6 +53,8 @@ required:
>    - compatible
>    - reg
>    - clocks
> +  - resets
> +  - reset-names


Best regards,
Krzysztof



From krzysztof.kozlowski at linaro.org  Tue Aug  8 09:00:01 2023
From: krzysztof.kozlowski at linaro.org (Krzysztof Kozlowski)
Date: Tue, 8 Aug 2023 18:00:01 +0200
Subject: [PATCH 3/3] i3c: ast2600: Add reset deassertion for global
 registers
In-Reply-To: <20230808154241.749641-4-dylan_hung@aspeedtech.com>
References: <20230808154241.749641-1-dylan_hung@aspeedtech.com>
 <20230808154241.749641-4-dylan_hung@aspeedtech.com>
Message-ID: <f85df327-c321-8c1e-2043-c60210fbed56@linaro.org>

On 08/08/2023 17:42, Dylan Hung wrote:
> Add missing reset deassertion of the I3C global control registers.
> 
> Signed-off-by: Dylan Hung <dylan_hung at aspeedtech.com>
> ---
>  drivers/i3c/master/ast2600-i3c-master.c | 9 +++++++++
>  1 file changed, 9 insertions(+)
> 
> diff --git a/drivers/i3c/master/ast2600-i3c-master.c b/drivers/i3c/master/ast2600-i3c-master.c
> index 09ed19d489e9..5d9d060134e0 100644
> --- a/drivers/i3c/master/ast2600-i3c-master.c
> +++ b/drivers/i3c/master/ast2600-i3c-master.c
> @@ -11,6 +11,7 @@
>  #include <linux/of_device.h>
>  #include <linux/platform_device.h>
>  #include <linux/regmap.h>
> +#include <linux/reset.h>
>  
>  #include "dw-i3c-master.h"
>  
> @@ -128,6 +129,7 @@ static int ast2600_i3c_probe(struct platform_device *pdev)
>  	struct device_node *np = pdev->dev.of_node;
>  	struct of_phandle_args gspec;
>  	struct ast2600_i3c *i3c;
> +	struct reset_control *rst;
>  	int rc;
>  
>  	i3c = devm_kzalloc(&pdev->dev, sizeof(*i3c), GFP_KERNEL);
> @@ -156,6 +158,13 @@ static int ast2600_i3c_probe(struct platform_device *pdev)
>  		dev_err(&pdev->dev, "invalid sda-pullup value %d\n",
>  			i3c->sda_pullup);
>  
> +	rst = devm_reset_control_get_shared(&pdev->dev, "global_rst");
> +	if (IS_ERR(rst)) {
> +		dev_err(&pdev->dev, "missing of invalid reset entry");
> +		return PTR_ERR(rst);

return dev_err_probe

Best regards,
Krzysztof



From jk at codeconstruct.com.au  Tue Aug  8 17:07:58 2023
From: jk at codeconstruct.com.au (Jeremy Kerr)
Date: Wed, 09 Aug 2023 08:07:58 +0800
Subject: [PATCH 0/3] Add Aspeed AST2600 I3C support
In-Reply-To: <20230808154241.749641-1-dylan_hung@aspeedtech.com>
References: <20230808154241.749641-1-dylan_hung@aspeedtech.com>
Message-ID: <d8d577e91779e045d5a2c7d701f65133a15b5d64.camel@codeconstruct.com.au>

Hi Dylan,

> This patch series introduces the necessary changes to enable I3C
> support for the Aspeed AST2600 I3C controller. Specifically, it addresses the
> missing pinctrl configuration and reset control for the I3C
> functionality.

+1 for the pinctrl changes for the I3C1 and I3C2 controllers (I'll
review and ack separately). I have been testing on I3C3 and up, but just
not with the HVI3C on 1 & 2, hence no pinctrl definition there.

However, I don't think the other two are needed.

For 2/3 and 3/3, you're adding a reset control for the global register
block within the per-controller driver, but we can already do that on a
global basis with the existing syscon device. Hence this earlier change:

  https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/drivers/mfd/syscon.c?id=7d1e3bd94828ad9fc86f55253cd6fec8edd65394

For this, I have:

    &i3c {
        i3c_global: i3c-global {
            compatible = "aspeed,ast2600-i3c-global", "simple-mfd", "syscon";
            resets = <&syscon ASPEED_RESET_I3C_DMA>;
            reg = <0x0 0x1000>;
        };

        i3c2: i3c-master at 4000 {
            compatible = "aspeed,ast2600-i3c";
            reg = <0x4000 0x1000>;
            clocks = <&syscon ASPEED_CLK_GATE_I3C2CLK>;
            pinctrl-names = "default";
            pinctrl-0 = <&pinctrl_i3c3_default>;
            interrupts = <GIC_SPI 104 IRQ_TYPE_LEVEL_HIGH>;
            aspeed,global-regs = <&i3c_global 2>;
            status = "disabled";
        };

        /* ... */
    };

- with no changes needed to any bindings. I haven't needed any other
resets; are there per-controller resets specified in the HW docs you
have?

Does that work for you? If you'd like to test, feel free to use my
sample dts at:

  https://github.com/CodeConstruct/linux/commit/05cac24705fa62d2176ecbbbf15d955cfe86e753

Cheers,


Jeremy


From jk at codeconstruct.com.au  Tue Aug  8 17:28:57 2023
From: jk at codeconstruct.com.au (Jeremy Kerr)
Date: Wed, 09 Aug 2023 08:28:57 +0800
Subject: [PATCH 0/3] Add Aspeed AST2600 I3C support
In-Reply-To: <d8d577e91779e045d5a2c7d701f65133a15b5d64.camel@codeconstruct.com.au>
References: <20230808154241.749641-1-dylan_hung@aspeedtech.com>
	 <d8d577e91779e045d5a2c7d701f65133a15b5d64.camel@codeconstruct.com.au>
Message-ID: <a670192024911f7f2d676e245f5f240e9ed5126b.camel@codeconstruct.com.au>

Hi Dylan,

> - with no changes needed to any bindings. I haven't needed any other
> resets; are there per-controller resets specified in the HW docs you
> have?

... keeping in mind the reset control that the aspeed SCU driver
already provides for you, when you enable the appropriate clock gate:

  https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/clk/clk-aspeed.c#n224

Cheers,


Jeremy


From jk at codeconstruct.com.au  Tue Aug  8 20:12:51 2023
From: jk at codeconstruct.com.au (Jeremy Kerr)
Date: Wed, 09 Aug 2023 11:12:51 +0800
Subject: [PATCH 0/3] Add Aspeed AST2600 I3C support
In-Reply-To: <TYZPR06MB65674B8AA26D959254101A749C12A@TYZPR06MB6567.apcprd06.prod.outlook.com>
References: <20230808154241.749641-1-dylan_hung@aspeedtech.com>
	 <d8d577e91779e045d5a2c7d701f65133a15b5d64.camel@codeconstruct.com.au>
	 <TYZPR06MB65674B8AA26D959254101A749C12A@TYZPR06MB6567.apcprd06.prod.outlook.com>
Message-ID: <ee48336f94f7e32de81e1f07facf0663c70091b0.camel@codeconstruct.com.au>

Hi Dylan,

> Thank you for your review. I3C1 and I3C2 can only operate in low
> voltage (1.0V/1.2V), which is why there are no HVI3C1 and HVI3C2
> pinctrl definitions.

Yep, and that was config that I hadn't tested (so hadn't proposed
pinctrl definitions for those).

> > For 2/3 and 3/3, you're adding a reset control for the global
> > register block within the per-controller driver, but we can already
> > do that on a global basis with the existing syscon device. Hence
> > this earlier change:
> ?
> I followed your recommendation and verified that it worked on my end.

OK, excellent!

> Should I resend the pinctrl patch as a stand-alone submission?

Yes, and feel free to add:

Reviewed-by: Jeremy Kerr <jk at codeconstruct.com.au>

Did your test use my i3c DTS definitions? If so, that's a decent
datapoint that the config works (on something other than my setup), and
so I'll submit upstream. Alternatively, feel free to include it with
your pinctrl change, if you like.

Cheers,


Jeremy


From dylan_hung at aspeedtech.com  Tue Aug  8 08:42:38 2023
From: dylan_hung at aspeedtech.com (Dylan Hung)
Date: Tue, 8 Aug 2023 23:42:38 +0800
Subject: [PATCH 0/3] Add Aspeed AST2600 I3C support
Message-ID: <20230808154241.749641-1-dylan_hung@aspeedtech.com>

This patch series introduces the necessary changes to enable I3C support
for the Aspeed AST2600 I3C controller. Specifically, it addresses the
missing pinctrl configuration and reset control for the I3C
functionality.

Dylan Hung (3):
  ARM: dts: pinctrl-aspeed-g6: Add I3C1 and I3C2 control pins
  dt-bindings: i3c: ast2600: Add resets and reset-names
  i3c: ast2600: Add reset deassertion for global registers

 .../devicetree/bindings/i3c/aspeed,ast2600-i3c.yaml  | 12 ++++++++++--
 arch/arm/boot/dts/aspeed/aspeed-g6-pinctrl.dtsi      | 10 ++++++++++
 drivers/i3c/master/ast2600-i3c-master.c              |  9 +++++++++
 3 files changed, 29 insertions(+), 2 deletions(-)

-- 
2.25.1



From dylan_hung at aspeedtech.com  Tue Aug  8 08:42:41 2023
From: dylan_hung at aspeedtech.com (Dylan Hung)
Date: Tue, 8 Aug 2023 23:42:41 +0800
Subject: [PATCH 3/3] i3c: ast2600: Add reset deassertion for global registers
In-Reply-To: <20230808154241.749641-1-dylan_hung@aspeedtech.com>
References: <20230808154241.749641-1-dylan_hung@aspeedtech.com>
Message-ID: <20230808154241.749641-4-dylan_hung@aspeedtech.com>

Add missing reset deassertion of the I3C global control registers.

Signed-off-by: Dylan Hung <dylan_hung at aspeedtech.com>
---
 drivers/i3c/master/ast2600-i3c-master.c | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/drivers/i3c/master/ast2600-i3c-master.c b/drivers/i3c/master/ast2600-i3c-master.c
index 09ed19d489e9..5d9d060134e0 100644
--- a/drivers/i3c/master/ast2600-i3c-master.c
+++ b/drivers/i3c/master/ast2600-i3c-master.c
@@ -11,6 +11,7 @@
 #include <linux/of_device.h>
 #include <linux/platform_device.h>
 #include <linux/regmap.h>
+#include <linux/reset.h>
 
 #include "dw-i3c-master.h"
 
@@ -128,6 +129,7 @@ static int ast2600_i3c_probe(struct platform_device *pdev)
 	struct device_node *np = pdev->dev.of_node;
 	struct of_phandle_args gspec;
 	struct ast2600_i3c *i3c;
+	struct reset_control *rst;
 	int rc;
 
 	i3c = devm_kzalloc(&pdev->dev, sizeof(*i3c), GFP_KERNEL);
@@ -156,6 +158,13 @@ static int ast2600_i3c_probe(struct platform_device *pdev)
 		dev_err(&pdev->dev, "invalid sda-pullup value %d\n",
 			i3c->sda_pullup);
 
+	rst = devm_reset_control_get_shared(&pdev->dev, "global_rst");
+	if (IS_ERR(rst)) {
+		dev_err(&pdev->dev, "missing of invalid reset entry");
+		return PTR_ERR(rst);
+	}
+	reset_control_deassert(rst);
+
 	i3c->dw.platform_ops = &ast2600_i3c_ops;
 	i3c->dw.ibi_capable = true;
 	return dw_i3c_common_probe(&i3c->dw, pdev);
-- 
2.25.1



From dylan_hung at aspeedtech.com  Tue Aug  8 08:42:39 2023
From: dylan_hung at aspeedtech.com (Dylan Hung)
Date: Tue, 8 Aug 2023 23:42:39 +0800
Subject: [PATCH 1/3] ARM: dts: pinctrl-aspeed-g6: Add I3C1 and I3C2 control pins
In-Reply-To: <20230808154241.749641-1-dylan_hung@aspeedtech.com>
References: <20230808154241.749641-1-dylan_hung@aspeedtech.com>
Message-ID: <20230808154241.749641-2-dylan_hung@aspeedtech.com>

Add pinctrl support for the I3C1 and I3C2 pins.

Signed-off-by: Dylan Hung <dylan_hung at aspeedtech.com>
---
 arch/arm/boot/dts/aspeed/aspeed-g6-pinctrl.dtsi | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/arch/arm/boot/dts/aspeed/aspeed-g6-pinctrl.dtsi b/arch/arm/boot/dts/aspeed/aspeed-g6-pinctrl.dtsi
index 7cd4f075e325..289668f051eb 100644
--- a/arch/arm/boot/dts/aspeed/aspeed-g6-pinctrl.dtsi
+++ b/arch/arm/boot/dts/aspeed/aspeed-g6-pinctrl.dtsi
@@ -297,6 +297,16 @@ pinctrl_i2c9_default: i2c9_default {
 		groups = "I2C9";
 	};
 
+	pinctrl_i3c1_default: i3c1_default {
+		function = "I3C1";
+		groups = "I3C1";
+	};
+
+	pinctrl_i3c2_default: i3c2_default {
+		function = "I3C2";
+		groups = "I3C2";
+	};
+
 	pinctrl_i3c3_default: i3c3_default {
 		function = "I3C3";
 		groups = "I3C3";
-- 
2.25.1



From dylan_hung at aspeedtech.com  Tue Aug  8 08:42:40 2023
From: dylan_hung at aspeedtech.com (Dylan Hung)
Date: Tue, 8 Aug 2023 23:42:40 +0800
Subject: [PATCH 2/3] dt-bindings: i3c: ast2600: Add resets and reset-names
In-Reply-To: <20230808154241.749641-1-dylan_hung@aspeedtech.com>
References: <20230808154241.749641-1-dylan_hung@aspeedtech.com>
Message-ID: <20230808154241.749641-3-dylan_hung@aspeedtech.com>

Add two reset lines to the AST2600 I3C controller:
- core_rst: the reset line of the controller itself
- global_rst: the reset line of the I3C global register block. Since all
six I3C controllers in AST2600 share this global register block, the
driver needs to handle this carefully. Generally, this reset line should
only need to be de-asserted.

Signed-off-by: Dylan Hung <dylan_hung at aspeedtech.com>
---
 .../devicetree/bindings/i3c/aspeed,ast2600-i3c.yaml  | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/Documentation/devicetree/bindings/i3c/aspeed,ast2600-i3c.yaml b/Documentation/devicetree/bindings/i3c/aspeed,ast2600-i3c.yaml
index fcc3dbff9c9a..3166d6f3a39c 100644
--- a/Documentation/devicetree/bindings/i3c/aspeed,ast2600-i3c.yaml
+++ b/Documentation/devicetree/bindings/i3c/aspeed,ast2600-i3c.yaml
@@ -23,7 +23,12 @@ properties:
     maxItems: 1
 
   resets:
-    maxItems: 1
+    maxItems: 2
+
+  reset-names:
+    items:
+      - const: core_rst
+      - const: global_rst
 
   interrupts:
     maxItems: 1
@@ -48,6 +53,8 @@ required:
   - compatible
   - reg
   - clocks
+  - resets
+  - reset-names
   - interrupts
   - aspeed,global-regs
 
@@ -63,7 +70,8 @@ examples:
         #address-cells = <3>;
         #size-cells = <0>;
         clocks = <&syscon 0>;
-        resets = <&syscon 0>;
+        resets = <&syscon 40>, <&syscon 39>;
+        reset-names = "core_rst", "global_rst";
         aspeed,global-regs = <&i3c_global 0>;
         pinctrl-names = "default";
         pinctrl-0 = <&pinctrl_i3c1_default>;
-- 
2.25.1



From alexandre.belloni at bootlin.com  Thu Aug 10 00:38:03 2023
From: alexandre.belloni at bootlin.com (Alexandre Belloni)
Date: Thu, 10 Aug 2023 09:38:03 +0200
Subject: [PATCH] i3c/master: cmd_v1: Fix the exit criteria for the daa
 procedure
In-Reply-To: <20230802100909.2568215-1-billy_tsai@aspeedtech.com>
References: <20230802100909.2568215-1-billy_tsai@aspeedtech.com>
Message-ID: <169165306480.966832.11791541519030436327.b4-ty@bootlin.com>


On Wed, 02 Aug 2023 18:09:09 +0800, Billy Tsai wrote:
> The exit criteria for the DAA should check if the data length is equal to
> 1, instead of checking if the response status is equal to 1.
> 
> 

Applied, thanks!

[1/1] i3c/master: cmd_v1: Fix the exit criteria for the daa procedure
      commit: cbf871e6d8ce23dd4d458d8b7ab9d4a267e7bc03

Best regards,

-- 
Alexandre Belloni, co-owner and COO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com


From alexandre.belloni at bootlin.com  Thu Aug 10 01:13:20 2023
From: alexandre.belloni at bootlin.com (Alexandre Belloni)
Date: Thu, 10 Aug 2023 10:13:20 +0200
Subject: [PATCH -next] i3c: master: svc: Do not check for 0 return after
 calling platform_get_irq()
In-Reply-To: <20230803085149.149248-1-ruanjinjie@huawei.com>
References: <20230803085149.149248-1-ruanjinjie@huawei.com>
Message-ID: <169165518426.989932.4311192647570957813.b4-ty@bootlin.com>


On Thu, 03 Aug 2023 16:51:49 +0800, Ruan Jinjie wrote:
> It is not possible for platform_get_irq() to return 0. Use the
> return value from platform_get_irq().
> 
> 

Applied, thanks!

[1/1] i3c: master: svc: Do not check for 0 return after calling platform_get_irq()
      commit: 300098637900f7ff99cafed2be7221407b82df83

Best regards,

-- 
Alexandre Belloni, co-owner and COO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com


From miquel.raynal at bootlin.com  Thu Aug 17 03:18:53 2023
From: miquel.raynal at bootlin.com (Miquel Raynal)
Date: Thu, 17 Aug 2023 12:18:53 +0200
Subject: [PATCH] i3c: master: svc: Describe member 'saved_regs'
Message-ID: <20230817101853.16805-1-miquel.raynal@bootlin.com>

The 'saved_regs' member of the 'svc_i3c_master'	structure is not
described in the kernel doc, which produces the following warning:

    Function parameter or member 'saved_regs' not described in 'svc_i3c_master'

Add the missing line in the kernel documentation of the parent
structure.

Fixes: 1c5ee2a77b1b ("i3c: master: svc: fix i3c suspend/resume issue")
Reported-by: kernel test robot <lkp at intel.com>
Closes: https://lore.kernel.org/oe-kbuild-all/202308171435.0xQ82lvu-lkp at intel.com/
Signed-off-by: Miquel Raynal <miquel.raynal at bootlin.com>
---
 drivers/i3c/master/svc-i3c-master.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/i3c/master/svc-i3c-master.c b/drivers/i3c/master/svc-i3c-master.c
index 0d63b732ef0c..77a70e9cd69b 100644
--- a/drivers/i3c/master/svc-i3c-master.c
+++ b/drivers/i3c/master/svc-i3c-master.c
@@ -156,6 +156,7 @@ struct svc_i3c_regs_save {
  * @base: I3C master controller
  * @dev: Corresponding device
  * @regs: Memory mapping
+ * @saved_regs: Volatile values for PM operations
  * @free_slots: Bit array of available slots
  * @addrs: Array containing the dynamic addresses of each attached device
  * @descs: Array of descriptors, one per attached device
-- 
2.34.1



From aniketmaurya at google.com  Mon Aug 21 22:12:09 2023
From: aniketmaurya at google.com (Aniket)
Date: Tue, 22 Aug 2023 05:12:09 +0000
Subject: [PATCH] dt-bindings: i3c: Fix description for assigned-address
Message-ID: <20230822051209.2837818-1-aniketmaurya@google.com>

assigned-address is a valid property for I3C device with or without
static address. Fix the description and an example for the same.

Signed-off-by: Aniket <aniketmaurya at google.com>
---
 Documentation/devicetree/bindings/i3c/i3c.yaml | 15 +++++++++++----
 1 file changed, 11 insertions(+), 4 deletions(-)

diff --git a/Documentation/devicetree/bindings/i3c/i3c.yaml b/Documentation/devicetree/bindings/i3c/i3c.yaml
index fdb4212149e7..ab69f4115de4 100644
--- a/Documentation/devicetree/bindings/i3c/i3c.yaml
+++ b/Documentation/devicetree/bindings/i3c/i3c.yaml
@@ -135,9 +135,10 @@ patternProperties:
         minimum: 0x1
         maximum: 0xff
         description: |
-          Dynamic address to be assigned to this device. This property is only
-          valid if the I3C device has a static address (first cell of the reg
-          property != 0).
+          Dynamic address to be assigned to this device. In case static address is
+          present (first cell of the reg property != 0), this address is assigned
+          through SETDASA. If static address is not present, this address is assigned
+          through SETNEWDA after assigning a temporary address via ENTDAA.
 
     required:
       - reg
@@ -163,12 +164,18 @@ examples:
             pagesize = <0x8>;
         };
 
-        /* I3C device with a static I2C address. */
+        /* I3C device with a static I2C address and assigned address. */
         thermal_sensor: sensor at 68,39200144004 {
             reg = <0x68 0x392 0x144004>;
             assigned-address = <0xa>;
         };
 
+        /* I3C device with only assigned address. */
+        pressure_sensor: sensor at 0,39200124004 {
+            reg = <0x0 0x392 0x124000>;
+            assigned-address = <0xc>;
+        };
+
         /*
          * I3C device without a static I2C address but requiring
          * resources described in the DT.
-- 
2.42.0.rc1.204.g551eb34607-goog



From aniketmaurya at google.com  Mon Aug 21 22:19:38 2023
From: aniketmaurya at google.com (Aniket)
Date: Tue, 22 Aug 2023 05:19:38 +0000
Subject: [PATCH] i3c: master: Fix SETDASA process
Message-ID: <20230822051938.2852567-1-aniketmaurya@google.com>

When a I3C DT node has a static_addr and an init_dyn_addr,the
init_dyn_addr is reserved in i3c_master_bus_init() and then
the static_addr is reserved in i3c_master_early_i3c_dev_add().
But if the dynamic address is same as static then above
procedure would fail.
Add a check to pass i3c_bus_get_addr_slot_status() when static
and dynamic address are equal.

Signed-off-by: Aniket <aniketmaurya at google.com>
---
 drivers/i3c/master.c | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/drivers/i3c/master.c b/drivers/i3c/master.c
index 08aeb69a7800..87283e4a4607 100644
--- a/drivers/i3c/master.c
+++ b/drivers/i3c/master.c
@@ -1308,7 +1308,11 @@ static int i3c_master_get_i3c_addrs(struct i3c_dev_desc *dev)
 	if (dev->info.static_addr) {
 		status = i3c_bus_get_addr_slot_status(&master->bus,
 						      dev->info.static_addr);
-		if (status != I3C_ADDR_SLOT_FREE)
+		/* Since static address and assigned dynamic address can be
+		 * equal, allow this case to pass.
+		 */
+		if (status != I3C_ADDR_SLOT_FREE &&
+		    dev->info.static_addr != dev->boardinfo->init_dyn_addr)
 			return -EBUSY;
 
 		i3c_bus_set_addr_slot_status(&master->bus,
-- 
2.42.0.rc1.204.g551eb34607-goog



From robh at kernel.org  Wed Aug 23 06:20:06 2023
From: robh at kernel.org (Rob Herring)
Date: Wed, 23 Aug 2023 08:20:06 -0500
Subject: [PATCH] dt-bindings: i3c: Fix description for assigned-address
In-Reply-To: <20230822051209.2837818-1-aniketmaurya@google.com>
References: <20230822051209.2837818-1-aniketmaurya@google.com>
Message-ID: <169279680573.2204361.10063597911204356200.robh@kernel.org>


On Tue, 22 Aug 2023 05:12:09 +0000, Aniket wrote:
> assigned-address is a valid property for I3C device with or without
> static address. Fix the description and an example for the same.
> 
> Signed-off-by: Aniket <aniketmaurya at google.com>
> ---
>  Documentation/devicetree/bindings/i3c/i3c.yaml | 15 +++++++++++----
>  1 file changed, 11 insertions(+), 4 deletions(-)
> 

Acked-by: Rob Herring <robh at kernel.org>



From robh at kernel.org  Wed Aug 23 11:28:47 2023
From: robh at kernel.org (Rob Herring)
Date: Wed, 23 Aug 2023 13:28:47 -0500
Subject: [PATCH] dt-bindings: Drop remaining unneeded quotes
Message-ID: <20230823183749.2609013-1-robh@kernel.org>

Cleanup bindings dropping the last remaining unneeded quotes. With this,
the check for this can be enabled in yamllint.

Signed-off-by: Rob Herring <robh at kernel.org>
---
 .../bindings/arm/arm,embedded-trace-extension.yaml   |  4 ++--
 .../bindings/arm/arm,trace-buffer-extension.yaml     |  7 ++++---
 .../devicetree/bindings/arm/arm,vexpress-juno.yaml   |  2 +-
 .../devicetree/bindings/arm/aspeed/aspeed,sbc.yaml   |  4 ++--
 .../arm/firmware/tlm,trusted-foundations.yaml        |  4 ++--
 .../bindings/arm/mstar/mstar,l3bridge.yaml           |  4 ++--
 .../devicetree/bindings/arm/mstar/mstar,smpctrl.yaml |  4 ++--
 .../devicetree/bindings/arm/stm32/st,mlahb.yaml      |  4 ++--
 .../bindings/arm/stm32/st,stm32-syscon.yaml          |  4 ++--
 .../devicetree/bindings/connector/usb-connector.yaml |  4 ++--
 Documentation/devicetree/bindings/eeprom/at24.yaml   |  4 ++--
 Documentation/devicetree/bindings/eeprom/at25.yaml   |  4 ++--
 .../intel,ixp4xx-network-processing-engine.yaml      |  4 ++--
 .../bindings/gpio/x-powers,axp209-gpio.yaml          |  4 ++--
 .../bindings/gpio/xlnx,zynqmp-gpio-modepin.yaml      |  4 ++--
 .../devicetree/bindings/gpio/xylon,logicvc-gpio.yaml |  4 ++--
 .../devicetree/bindings/hwmon/iio-hwmon.yaml         |  4 ++--
 .../bindings/hwmon/starfive,jh71x0-temp.yaml         |  8 ++++----
 .../devicetree/bindings/i3c/mipi-i3c-hci.yaml        |  4 ++--
 .../devicetree/bindings/iio/accel/fsl,mma7455.yaml   |  4 ++--
 .../bindings/iio/adc/atmel,sama9260-adc.yaml         |  4 ++--
 .../bindings/ipmi/aspeed,ast2400-kcs-bmc.yaml        |  8 ++++----
 .../devicetree/bindings/ipmi/ipmi-ipmb.yaml          |  2 +-
 .../devicetree/bindings/ipmi/ipmi-smic.yaml          |  2 +-
 .../bindings/media/qcom,msm8916-venus.yaml           |  4 ++--
 .../bindings/mips/loongson/ls2k-reset.yaml           |  4 ++--
 .../bindings/mips/loongson/rs780e-acpi.yaml          |  4 ++--
 .../misc/intel,ixp4xx-ahb-queue-manager.yaml         |  4 ++--
 .../devicetree/bindings/mmc/marvell,xenon-sdhci.yaml |  4 ++--
 .../bindings/mtd/microchip,mchp48l640.yaml           |  4 ++--
 .../devicetree/bindings/soc/aspeed/uart-routing.yaml |  4 ++--
 .../bindings/soc/intel/intel,hps-copy-engine.yaml    |  4 ++--
 .../bindings/soc/litex/litex,soc-controller.yaml     |  4 ++--
 .../bindings/soc/renesas/renesas,rzg2l-sysc.yaml     |  4 ++--
 .../devicetree/bindings/soc/ti/k3-ringacc.yaml       |  4 ++--
 .../devicetree/bindings/sound/dialog,da7219.yaml     |  4 ++--
 .../bindings/sound/nvidia,tegra-audio-max9808x.yaml  | 12 ++++++------
 .../bindings/sound/nvidia,tegra-audio-rt5631.yaml    |  8 ++++----
 .../devicetree/bindings/ufs/ufs-common.yaml          |  2 +-
 .../bindings/watchdog/toshiba,visconti-wdt.yaml      |  4 ++--
 40 files changed, 88 insertions(+), 87 deletions(-)

diff --git a/Documentation/devicetree/bindings/arm/arm,embedded-trace-extension.yaml b/Documentation/devicetree/bindings/arm/arm,embedded-trace-extension.yaml
index 108460627d9a..a477a810f9e9 100644
--- a/Documentation/devicetree/bindings/arm/arm,embedded-trace-extension.yaml
+++ b/Documentation/devicetree/bindings/arm/arm,embedded-trace-extension.yaml
@@ -2,8 +2,8 @@
 # Copyright 2021, Arm Ltd
 %YAML 1.2
 ---
-$id: "http://devicetree.org/schemas/arm/arm,embedded-trace-extension.yaml#"
-$schema: "http://devicetree.org/meta-schemas/core.yaml#"
+$id: http://devicetree.org/schemas/arm/arm,embedded-trace-extension.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
 
 title: ARM Embedded Trace Extensions
 
diff --git a/Documentation/devicetree/bindings/arm/arm,trace-buffer-extension.yaml b/Documentation/devicetree/bindings/arm/arm,trace-buffer-extension.yaml
index b1322658063a..8c27e510cb71 100644
--- a/Documentation/devicetree/bindings/arm/arm,trace-buffer-extension.yaml
+++ b/Documentation/devicetree/bindings/arm/arm,trace-buffer-extension.yaml
@@ -2,8 +2,8 @@
 # Copyright 2021, Arm Ltd
 %YAML 1.2
 ---
-$id: "http://devicetree.org/schemas/arm/arm,trace-buffer-extension.yaml#"
-$schema: "http://devicetree.org/meta-schemas/core.yaml#"
+$id: http://devicetree.org/schemas/arm/arm,trace-buffer-extension.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
 
 title: ARM Trace Buffer Extensions
 
@@ -19,7 +19,8 @@ description: |
 
 properties:
   $nodename:
-    const: "trbe"
+    const: trbe
+
   compatible:
     items:
       - const: arm,trace-buffer-extension
diff --git a/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml b/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml
index cdd65881fcdd..8dd6b6446394 100644
--- a/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml
+++ b/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml
@@ -143,7 +143,7 @@ patternProperties:
       "simple-bus". If the compatible is placed in the "motherboard-bus" node,
       it is stricter and always has two compatibles.
     type: object
-    $ref: '/schemas/simple-bus.yaml'
+    $ref: /schemas/simple-bus.yaml
     unevaluatedProperties: false
 
     properties:
diff --git a/Documentation/devicetree/bindings/arm/aspeed/aspeed,sbc.yaml b/Documentation/devicetree/bindings/arm/aspeed/aspeed,sbc.yaml
index c72aab706484..b8c5cacb09bd 100644
--- a/Documentation/devicetree/bindings/arm/aspeed/aspeed,sbc.yaml
+++ b/Documentation/devicetree/bindings/arm/aspeed/aspeed,sbc.yaml
@@ -2,8 +2,8 @@
 # Copyright 2021 Joel Stanley, IBM Corp.
 %YAML 1.2
 ---
-$id: "http://devicetree.org/schemas/arm/aspeed/aspeed,sbc.yaml#"
-$schema: "http://devicetree.org/meta-schemas/core.yaml#"
+$id: http://devicetree.org/schemas/arm/aspeed/aspeed,sbc.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
 
 title: ASPEED Secure Boot Controller
 
diff --git a/Documentation/devicetree/bindings/arm/firmware/tlm,trusted-foundations.yaml b/Documentation/devicetree/bindings/arm/firmware/tlm,trusted-foundations.yaml
index 9d1857c0aa07..e3980b659f63 100644
--- a/Documentation/devicetree/bindings/arm/firmware/tlm,trusted-foundations.yaml
+++ b/Documentation/devicetree/bindings/arm/firmware/tlm,trusted-foundations.yaml
@@ -1,8 +1,8 @@
 # SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
 %YAML 1.2
 ---
-$id: "http://devicetree.org/schemas/arm/firmware/tlm,trusted-foundations.yaml#"
-$schema: "http://devicetree.org/meta-schemas/core.yaml#"
+$id: http://devicetree.org/schemas/arm/firmware/tlm,trusted-foundations.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
 
 title: Trusted Foundations
 
diff --git a/Documentation/devicetree/bindings/arm/mstar/mstar,l3bridge.yaml b/Documentation/devicetree/bindings/arm/mstar/mstar,l3bridge.yaml
index 6816bd68f9cf..a8ac4a2d672d 100644
--- a/Documentation/devicetree/bindings/arm/mstar/mstar,l3bridge.yaml
+++ b/Documentation/devicetree/bindings/arm/mstar/mstar,l3bridge.yaml
@@ -2,8 +2,8 @@
 # Copyright 2020 thingy.jp.
 %YAML 1.2
 ---
-$id: "http://devicetree.org/schemas/arm/mstar/mstar,l3bridge.yaml#"
-$schema: "http://devicetree.org/meta-schemas/core.yaml#"
+$id: http://devicetree.org/schemas/arm/mstar/mstar,l3bridge.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
 
 title: MStar/SigmaStar Armv7 SoC l3bridge
 
diff --git a/Documentation/devicetree/bindings/arm/mstar/mstar,smpctrl.yaml b/Documentation/devicetree/bindings/arm/mstar/mstar,smpctrl.yaml
index 599c65980f5d..5739848000b1 100644
--- a/Documentation/devicetree/bindings/arm/mstar/mstar,smpctrl.yaml
+++ b/Documentation/devicetree/bindings/arm/mstar/mstar,smpctrl.yaml
@@ -2,8 +2,8 @@
 # Copyright 2020 thingy.jp.
 %YAML 1.2
 ---
-$id: "http://devicetree.org/schemas/arm/mstar/mstar,smpctrl.yaml#"
-$schema: "http://devicetree.org/meta-schemas/core.yaml#"
+$id: http://devicetree.org/schemas/arm/mstar/mstar,smpctrl.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
 
 title: MStar/SigmaStar Armv7 SoC SMP control registers
 
diff --git a/Documentation/devicetree/bindings/arm/stm32/st,mlahb.yaml b/Documentation/devicetree/bindings/arm/stm32/st,mlahb.yaml
index 2297ad3f4774..d2dce238ff5d 100644
--- a/Documentation/devicetree/bindings/arm/stm32/st,mlahb.yaml
+++ b/Documentation/devicetree/bindings/arm/stm32/st,mlahb.yaml
@@ -1,8 +1,8 @@
 # SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
 %YAML 1.2
 ---
-$id: "http://devicetree.org/schemas/arm/stm32/st,mlahb.yaml#"
-$schema: "http://devicetree.org/meta-schemas/core.yaml#"
+$id: http://devicetree.org/schemas/arm/stm32/st,mlahb.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
 
 title: STMicroelectronics STM32 ML-AHB interconnect
 
diff --git a/Documentation/devicetree/bindings/arm/stm32/st,stm32-syscon.yaml b/Documentation/devicetree/bindings/arm/stm32/st,stm32-syscon.yaml
index b63ff591ef8f..d083d8ad48b7 100644
--- a/Documentation/devicetree/bindings/arm/stm32/st,stm32-syscon.yaml
+++ b/Documentation/devicetree/bindings/arm/stm32/st,stm32-syscon.yaml
@@ -1,8 +1,8 @@
 # SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
 %YAML 1.2
 ---
-$id: "http://devicetree.org/schemas/arm/stm32/st,stm32-syscon.yaml#"
-$schema: "http://devicetree.org/meta-schemas/core.yaml#"
+$id: http://devicetree.org/schemas/arm/stm32/st,stm32-syscon.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
 
 title: STMicroelectronics STM32 Platforms System Controller
 
diff --git a/Documentation/devicetree/bindings/connector/usb-connector.yaml b/Documentation/devicetree/bindings/connector/usb-connector.yaml
index 3ecb51f55a71..7c8a3e8430d3 100644
--- a/Documentation/devicetree/bindings/connector/usb-connector.yaml
+++ b/Documentation/devicetree/bindings/connector/usb-connector.yaml
@@ -232,8 +232,8 @@ properties:
     type: boolean
 
 dependencies:
-  sink-vdos-v1: [ 'sink-vdos' ]
-  sink-vdos: [ 'sink-vdos-v1' ]
+  sink-vdos-v1: [ sink-vdos ]
+  sink-vdos: [ sink-vdos-v1 ]
 
 required:
   - compatible
diff --git a/Documentation/devicetree/bindings/eeprom/at24.yaml b/Documentation/devicetree/bindings/eeprom/at24.yaml
index 84af0d5f52aa..b1e5dddf9bf4 100644
--- a/Documentation/devicetree/bindings/eeprom/at24.yaml
+++ b/Documentation/devicetree/bindings/eeprom/at24.yaml
@@ -2,8 +2,8 @@
 # Copyright 2019 BayLibre SAS
 %YAML 1.2
 ---
-$id: "http://devicetree.org/schemas/eeprom/at24.yaml#"
-$schema: "http://devicetree.org/meta-schemas/core.yaml#"
+$id: http://devicetree.org/schemas/eeprom/at24.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
 
 title: I2C EEPROMs compatible with Atmel's AT24
 
diff --git a/Documentation/devicetree/bindings/eeprom/at25.yaml b/Documentation/devicetree/bindings/eeprom/at25.yaml
index 0e31bb36ebb1..1715b0c9feea 100644
--- a/Documentation/devicetree/bindings/eeprom/at25.yaml
+++ b/Documentation/devicetree/bindings/eeprom/at25.yaml
@@ -1,8 +1,8 @@
 # SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
 %YAML 1.2
 ---
-$id: "http://devicetree.org/schemas/eeprom/at25.yaml#"
-$schema: "http://devicetree.org/meta-schemas/core.yaml#"
+$id: http://devicetree.org/schemas/eeprom/at25.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
 
 title: SPI EEPROMs or FRAMs compatible with Atmel's AT25
 
diff --git a/Documentation/devicetree/bindings/firmware/intel,ixp4xx-network-processing-engine.yaml b/Documentation/devicetree/bindings/firmware/intel,ixp4xx-network-processing-engine.yaml
index 9a785bbaafb7..e6bed7d93e2d 100644
--- a/Documentation/devicetree/bindings/firmware/intel,ixp4xx-network-processing-engine.yaml
+++ b/Documentation/devicetree/bindings/firmware/intel,ixp4xx-network-processing-engine.yaml
@@ -2,8 +2,8 @@
 # Copyright 2019 Linaro Ltd.
 %YAML 1.2
 ---
-$id: "http://devicetree.org/schemas/firmware/intel,ixp4xx-network-processing-engine.yaml#"
-$schema: "http://devicetree.org/meta-schemas/core.yaml#"
+$id: http://devicetree.org/schemas/firmware/intel,ixp4xx-network-processing-engine.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
 
 title: Intel IXP4xx Network Processing Engine
 
diff --git a/Documentation/devicetree/bindings/gpio/x-powers,axp209-gpio.yaml b/Documentation/devicetree/bindings/gpio/x-powers,axp209-gpio.yaml
index 1638cfe90f1c..5eeb29bcdd21 100644
--- a/Documentation/devicetree/bindings/gpio/x-powers,axp209-gpio.yaml
+++ b/Documentation/devicetree/bindings/gpio/x-powers,axp209-gpio.yaml
@@ -1,8 +1,8 @@
 # SPDX-License-Identifier: GPL-2.0
 %YAML 1.2
 ---
-$id: "http://devicetree.org/schemas/gpio/x-powers,axp209-gpio.yaml#"
-$schema: "http://devicetree.org/meta-schemas/core.yaml#"
+$id: http://devicetree.org/schemas/gpio/x-powers,axp209-gpio.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
 
 title: X-Powers AXP209 GPIO
 
diff --git a/Documentation/devicetree/bindings/gpio/xlnx,zynqmp-gpio-modepin.yaml b/Documentation/devicetree/bindings/gpio/xlnx,zynqmp-gpio-modepin.yaml
index 18e61aff2185..56143f1fe84a 100644
--- a/Documentation/devicetree/bindings/gpio/xlnx,zynqmp-gpio-modepin.yaml
+++ b/Documentation/devicetree/bindings/gpio/xlnx,zynqmp-gpio-modepin.yaml
@@ -1,8 +1,8 @@
 # SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
 %YAML 1.2
 ---
-$id: "http://devicetree.org/schemas/gpio/xlnx,zynqmp-gpio-modepin.yaml#"
-$schema: "http://devicetree.org/meta-schemas/core.yaml#"
+$id: http://devicetree.org/schemas/gpio/xlnx,zynqmp-gpio-modepin.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
 
 title: ZynqMP Mode Pin GPIO controller
 
diff --git a/Documentation/devicetree/bindings/gpio/xylon,logicvc-gpio.yaml b/Documentation/devicetree/bindings/gpio/xylon,logicvc-gpio.yaml
index a36aec27069c..59c79a6943ec 100644
--- a/Documentation/devicetree/bindings/gpio/xylon,logicvc-gpio.yaml
+++ b/Documentation/devicetree/bindings/gpio/xylon,logicvc-gpio.yaml
@@ -2,8 +2,8 @@
 # Copyright 2019 Bootlin
 %YAML 1.2
 ---
-$id: "http://devicetree.org/schemas/gpio/xylon,logicvc-gpio.yaml#"
-$schema: "http://devicetree.org/meta-schemas/core.yaml#"
+$id: http://devicetree.org/schemas/gpio/xylon,logicvc-gpio.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
 
 title: Xylon LogiCVC GPIO controller
 
diff --git a/Documentation/devicetree/bindings/hwmon/iio-hwmon.yaml b/Documentation/devicetree/bindings/hwmon/iio-hwmon.yaml
index c54b5986b365..e5b24782f448 100644
--- a/Documentation/devicetree/bindings/hwmon/iio-hwmon.yaml
+++ b/Documentation/devicetree/bindings/hwmon/iio-hwmon.yaml
@@ -1,8 +1,8 @@
 # SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
 %YAML 1.2
 ---
-$id: "http://devicetree.org/schemas/hwmon/iio-hwmon.yaml#"
-$schema: "http://devicetree.org/meta-schemas/core.yaml#"
+$id: http://devicetree.org/schemas/hwmon/iio-hwmon.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
 
 title: ADC-attached Hardware Sensor
 
diff --git a/Documentation/devicetree/bindings/hwmon/starfive,jh71x0-temp.yaml b/Documentation/devicetree/bindings/hwmon/starfive,jh71x0-temp.yaml
index f5b34528928d..733cba780186 100644
--- a/Documentation/devicetree/bindings/hwmon/starfive,jh71x0-temp.yaml
+++ b/Documentation/devicetree/bindings/hwmon/starfive,jh71x0-temp.yaml
@@ -27,8 +27,8 @@ properties:
 
   clock-names:
     items:
-      - const: "sense"
-      - const: "bus"
+      - const: sense
+      - const: bus
 
   '#thermal-sensor-cells':
     const: 0
@@ -39,8 +39,8 @@ properties:
 
   reset-names:
     items:
-      - const: "sense"
-      - const: "bus"
+      - const: sense
+      - const: bus
 
 required:
   - compatible
diff --git a/Documentation/devicetree/bindings/i3c/mipi-i3c-hci.yaml b/Documentation/devicetree/bindings/i3c/mipi-i3c-hci.yaml
index c002afdbfc7c..5dda8cb44cdb 100644
--- a/Documentation/devicetree/bindings/i3c/mipi-i3c-hci.yaml
+++ b/Documentation/devicetree/bindings/i3c/mipi-i3c-hci.yaml
@@ -1,8 +1,8 @@
 # SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
 %YAML 1.2
 ---
-$id: "http://devicetree.org/schemas/i3c/mipi-i3c-hci.yaml#"
-$schema: "http://devicetree.org/meta-schemas/core.yaml#"
+$id: http://devicetree.org/schemas/i3c/mipi-i3c-hci.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
 
 title: MIPI I3C HCI
 
diff --git a/Documentation/devicetree/bindings/iio/accel/fsl,mma7455.yaml b/Documentation/devicetree/bindings/iio/accel/fsl,mma7455.yaml
index c8659c5eba2a..cb31e75ba680 100644
--- a/Documentation/devicetree/bindings/iio/accel/fsl,mma7455.yaml
+++ b/Documentation/devicetree/bindings/iio/accel/fsl,mma7455.yaml
@@ -36,8 +36,8 @@ properties:
     maxItems: 2
     items:
       enum:
-        - "INT1"
-        - "INT2"
+        - INT1
+        - INT2
 
 required:
   - compatible
diff --git a/Documentation/devicetree/bindings/iio/adc/atmel,sama9260-adc.yaml b/Documentation/devicetree/bindings/iio/adc/atmel,sama9260-adc.yaml
index e6a1f915b542..1f30a8569187 100644
--- a/Documentation/devicetree/bindings/iio/adc/atmel,sama9260-adc.yaml
+++ b/Documentation/devicetree/bindings/iio/adc/atmel,sama9260-adc.yaml
@@ -56,8 +56,8 @@ properties:
       String corresponding to an identifier from atmel,adc-res-names property.
       If not specified, the highest resolution will be used.
     enum:
-      - "lowres"
-      - "highres"
+      - lowres
+      - highres
 
   atmel,adc-sleep-mode:
     $ref: /schemas/types.yaml#/definitions/flag
diff --git a/Documentation/devicetree/bindings/ipmi/aspeed,ast2400-kcs-bmc.yaml b/Documentation/devicetree/bindings/ipmi/aspeed,ast2400-kcs-bmc.yaml
index 4ff6fabfcb30..129e32c4c774 100644
--- a/Documentation/devicetree/bindings/ipmi/aspeed,ast2400-kcs-bmc.yaml
+++ b/Documentation/devicetree/bindings/ipmi/aspeed,ast2400-kcs-bmc.yaml
@@ -41,7 +41,7 @@ properties:
       - description: STR register
 
   aspeed,lpc-io-reg:
-    $ref: '/schemas/types.yaml#/definitions/uint32-array'
+    $ref: /schemas/types.yaml#/definitions/uint32-array
     minItems: 1
     maxItems: 2
     description: |
@@ -50,7 +50,7 @@ properties:
       status address may be optionally provided.
 
   aspeed,lpc-interrupts:
-    $ref: "/schemas/types.yaml#/definitions/uint32-array"
+    $ref: /schemas/types.yaml#/definitions/uint32-array
     minItems: 2
     maxItems: 2
     description: |
@@ -63,12 +63,12 @@ properties:
 
   kcs_chan:
     deprecated: true
-    $ref: '/schemas/types.yaml#/definitions/uint32'
+    $ref: /schemas/types.yaml#/definitions/uint32
     description: The LPC channel number in the controller
 
   kcs_addr:
     deprecated: true
-    $ref: '/schemas/types.yaml#/definitions/uint32'
+    $ref: /schemas/types.yaml#/definitions/uint32
     description: The host CPU IO map address
 
 required:
diff --git a/Documentation/devicetree/bindings/ipmi/ipmi-ipmb.yaml b/Documentation/devicetree/bindings/ipmi/ipmi-ipmb.yaml
index 3f25cdb4e99b..52647bff31af 100644
--- a/Documentation/devicetree/bindings/ipmi/ipmi-ipmb.yaml
+++ b/Documentation/devicetree/bindings/ipmi/ipmi-ipmb.yaml
@@ -18,7 +18,7 @@ properties:
 
   device_type:
     items:
-      - const: "ipmi"
+      - const: ipmi
 
   reg:
     maxItems: 1
diff --git a/Documentation/devicetree/bindings/ipmi/ipmi-smic.yaml b/Documentation/devicetree/bindings/ipmi/ipmi-smic.yaml
index c1b4bf95ef99..4bffa3d86128 100644
--- a/Documentation/devicetree/bindings/ipmi/ipmi-smic.yaml
+++ b/Documentation/devicetree/bindings/ipmi/ipmi-smic.yaml
@@ -20,7 +20,7 @@ properties:
 
   device_type:
     items:
-      - const: "ipmi"
+      - const: ipmi
 
   reg:
     maxItems: 1
diff --git a/Documentation/devicetree/bindings/media/qcom,msm8916-venus.yaml b/Documentation/devicetree/bindings/media/qcom,msm8916-venus.yaml
index 2350bf4b370e..9410f13ca97c 100644
--- a/Documentation/devicetree/bindings/media/qcom,msm8916-venus.yaml
+++ b/Documentation/devicetree/bindings/media/qcom,msm8916-venus.yaml
@@ -40,7 +40,7 @@ properties:
 
     properties:
       compatible:
-        const: "venus-decoder"
+        const: venus-decoder
 
     required:
       - compatible
@@ -52,7 +52,7 @@ properties:
 
     properties:
       compatible:
-        const: "venus-encoder"
+        const: venus-encoder
 
     required:
       - compatible
diff --git a/Documentation/devicetree/bindings/mips/loongson/ls2k-reset.yaml b/Documentation/devicetree/bindings/mips/loongson/ls2k-reset.yaml
index 20b5836efd90..358ac8cd4d1d 100644
--- a/Documentation/devicetree/bindings/mips/loongson/ls2k-reset.yaml
+++ b/Documentation/devicetree/bindings/mips/loongson/ls2k-reset.yaml
@@ -1,8 +1,8 @@
 # SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
 %YAML 1.2
 ---
-$id: "http://devicetree.org/schemas/mips/loongson/ls2k-reset.yaml#"
-$schema: "http://devicetree.org/meta-schemas/core.yaml#"
+$id: http://devicetree.org/schemas/mips/loongson/ls2k-reset.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
 
 title: Loongson 2K1000 PM Controller
 
diff --git a/Documentation/devicetree/bindings/mips/loongson/rs780e-acpi.yaml b/Documentation/devicetree/bindings/mips/loongson/rs780e-acpi.yaml
index 7c0f9022202c..3e3a3705e879 100644
--- a/Documentation/devicetree/bindings/mips/loongson/rs780e-acpi.yaml
+++ b/Documentation/devicetree/bindings/mips/loongson/rs780e-acpi.yaml
@@ -1,8 +1,8 @@
 # SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
 %YAML 1.2
 ---
-$id: "http://devicetree.org/schemas/mips/loongson/rs780e-acpi.yaml#"
-$schema: "http://devicetree.org/meta-schemas/core.yaml#"
+$id: http://devicetree.org/schemas/mips/loongson/rs780e-acpi.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
 
 title: Loongson RS780E PCH ACPI Controller
 
diff --git a/Documentation/devicetree/bindings/misc/intel,ixp4xx-ahb-queue-manager.yaml b/Documentation/devicetree/bindings/misc/intel,ixp4xx-ahb-queue-manager.yaml
index 38ab0499102d..36a9dbdf3f03 100644
--- a/Documentation/devicetree/bindings/misc/intel,ixp4xx-ahb-queue-manager.yaml
+++ b/Documentation/devicetree/bindings/misc/intel,ixp4xx-ahb-queue-manager.yaml
@@ -2,8 +2,8 @@
 # Copyright 2019 Linaro Ltd.
 %YAML 1.2
 ---
-$id: "http://devicetree.org/schemas/misc/intel,ixp4xx-ahb-queue-manager.yaml#"
-$schema: "http://devicetree.org/meta-schemas/core.yaml#"
+$id: http://devicetree.org/schemas/misc/intel,ixp4xx-ahb-queue-manager.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
 
 title: Intel IXP4xx AHB Queue Manager
 
diff --git a/Documentation/devicetree/bindings/mmc/marvell,xenon-sdhci.yaml b/Documentation/devicetree/bindings/mmc/marvell,xenon-sdhci.yaml
index 3ee758886558..3a8e74894ae0 100644
--- a/Documentation/devicetree/bindings/mmc/marvell,xenon-sdhci.yaml
+++ b/Documentation/devicetree/bindings/mmc/marvell,xenon-sdhci.yaml
@@ -71,8 +71,8 @@ properties:
   marvell,xenon-phy-type:
     $ref: /schemas/types.yaml#/definitions/string
     enum:
-      - "emmc 5.1 phy"
-      - "emmc 5.0 phy"
+      - emmc 5.1 phy
+      - emmc 5.0 phy
     description: |
       Xenon support multiple types of PHYs. To select eMMC 5.1 PHY, set:
       marvell,xenon-phy-type = "emmc 5.1 phy" eMMC 5.1 PHY is the default
diff --git a/Documentation/devicetree/bindings/mtd/microchip,mchp48l640.yaml b/Documentation/devicetree/bindings/mtd/microchip,mchp48l640.yaml
index 00882892f47e..0ff32bd00bf6 100644
--- a/Documentation/devicetree/bindings/mtd/microchip,mchp48l640.yaml
+++ b/Documentation/devicetree/bindings/mtd/microchip,mchp48l640.yaml
@@ -1,8 +1,8 @@
 # SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
 %YAML 1.2
 ---
-$id: "http://devicetree.org/schemas/mtd/microchip,mchp48l640.yaml#"
-$schema: "http://devicetree.org/meta-schemas/core.yaml#"
+$id: http://devicetree.org/schemas/mtd/microchip,mchp48l640.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
 
 title: Microchip 48l640 (and similar) serial EERAM
 
diff --git a/Documentation/devicetree/bindings/soc/aspeed/uart-routing.yaml b/Documentation/devicetree/bindings/soc/aspeed/uart-routing.yaml
index 6876407124dc..51aaf34acb32 100644
--- a/Documentation/devicetree/bindings/soc/aspeed/uart-routing.yaml
+++ b/Documentation/devicetree/bindings/soc/aspeed/uart-routing.yaml
@@ -3,8 +3,8 @@
 # # Copyright (c) 2021 Aspeed Technology Inc.
 %YAML 1.2
 ---
-$id: "http://devicetree.org/schemas/soc/aspeed/uart-routing.yaml#"
-$schema: "http://devicetree.org/meta-schemas/core.yaml#"
+$id: http://devicetree.org/schemas/soc/aspeed/uart-routing.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
 
 title: Aspeed UART Routing Controller
 
diff --git a/Documentation/devicetree/bindings/soc/intel/intel,hps-copy-engine.yaml b/Documentation/devicetree/bindings/soc/intel/intel,hps-copy-engine.yaml
index 8634865015cd..ceb81646fe75 100644
--- a/Documentation/devicetree/bindings/soc/intel/intel,hps-copy-engine.yaml
+++ b/Documentation/devicetree/bindings/soc/intel/intel,hps-copy-engine.yaml
@@ -2,8 +2,8 @@
 # Copyright (C) 2022, Intel Corporation
 %YAML 1.2
 ---
-$id: "http://devicetree.org/schemas/soc/intel/intel,hps-copy-engine.yaml#"
-$schema: "http://devicetree.org/meta-schemas/core.yaml#"
+$id: http://devicetree.org/schemas/soc/intel/intel,hps-copy-engine.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
 
 title: Intel HPS Copy Engine
 
diff --git a/Documentation/devicetree/bindings/soc/litex/litex,soc-controller.yaml b/Documentation/devicetree/bindings/soc/litex/litex,soc-controller.yaml
index ecae9fa8561b..a64406ca17b5 100644
--- a/Documentation/devicetree/bindings/soc/litex/litex,soc-controller.yaml
+++ b/Documentation/devicetree/bindings/soc/litex/litex,soc-controller.yaml
@@ -2,8 +2,8 @@
 # Copyright 2020 Antmicro <www.antmicro.com>
 %YAML 1.2
 ---
-$id: "http://devicetree.org/schemas/soc/litex/litex,soc-controller.yaml#"
-$schema: "http://devicetree.org/meta-schemas/core.yaml#"
+$id: http://devicetree.org/schemas/soc/litex/litex,soc-controller.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
 
 title: LiteX SoC Controller driver
 
diff --git a/Documentation/devicetree/bindings/soc/renesas/renesas,rzg2l-sysc.yaml b/Documentation/devicetree/bindings/soc/renesas/renesas,rzg2l-sysc.yaml
index 398663d21ab1..e52e176d8cb3 100644
--- a/Documentation/devicetree/bindings/soc/renesas/renesas,rzg2l-sysc.yaml
+++ b/Documentation/devicetree/bindings/soc/renesas/renesas,rzg2l-sysc.yaml
@@ -1,8 +1,8 @@
 # SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
 %YAML 1.2
 ---
-$id: "http://devicetree.org/schemas/soc/renesas/renesas,rzg2l-sysc.yaml#"
-$schema: "http://devicetree.org/meta-schemas/core.yaml#"
+$id: http://devicetree.org/schemas/soc/renesas/renesas,rzg2l-sysc.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
 
 title: Renesas RZ/{G2L,V2L} System Controller (SYSC)
 
diff --git a/Documentation/devicetree/bindings/soc/ti/k3-ringacc.yaml b/Documentation/devicetree/bindings/soc/ti/k3-ringacc.yaml
index 22cf9002fee7..4ac00716885e 100644
--- a/Documentation/devicetree/bindings/soc/ti/k3-ringacc.yaml
+++ b/Documentation/devicetree/bindings/soc/ti/k3-ringacc.yaml
@@ -2,8 +2,8 @@
 # Copyright (C) 2020 Texas Instruments Incorporated - http://www.ti.com/
 %YAML 1.2
 ---
-$id: "http://devicetree.org/schemas/soc/ti/k3-ringacc.yaml#"
-$schema: "http://devicetree.org/meta-schemas/core.yaml#"
+$id: http://devicetree.org/schemas/soc/ti/k3-ringacc.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
 
 title: Texas Instruments K3 NavigatorSS Ring Accelerator
 
diff --git a/Documentation/devicetree/bindings/sound/dialog,da7219.yaml b/Documentation/devicetree/bindings/sound/dialog,da7219.yaml
index 2d01956cefbb..19137abdba3e 100644
--- a/Documentation/devicetree/bindings/sound/dialog,da7219.yaml
+++ b/Documentation/devicetree/bindings/sound/dialog,da7219.yaml
@@ -74,7 +74,7 @@ properties:
     $ref: /schemas/types.yaml#/definitions/uint32
 
   dlg,mic-amp-in-sel:
-    enum: ["diff", "se_p", "se_n"]
+    enum: [diff, se_p, se_n]
     description:
       Mic input source type.
 
@@ -124,7 +124,7 @@ properties:
         $ref: /schemas/types.yaml#/definitions/uint32
 
       dlg,jack-ins-det-pty:
-        enum: ["low", "high"]
+        enum: [low, high]
         description:
           Polarity for jack insertion detection.
         $ref: /schemas/types.yaml#/definitions/string
diff --git a/Documentation/devicetree/bindings/sound/nvidia,tegra-audio-max9808x.yaml b/Documentation/devicetree/bindings/sound/nvidia,tegra-audio-max9808x.yaml
index fc89dbd6bf24..c29d7942915c 100644
--- a/Documentation/devicetree/bindings/sound/nvidia,tegra-audio-max9808x.yaml
+++ b/Documentation/devicetree/bindings/sound/nvidia,tegra-audio-max9808x.yaml
@@ -35,12 +35,12 @@ properties:
     items:
       enum:
         # Board Connectors
-        - "Int Spk"
-        - "Headphone Jack"
-        - "Earpiece"
-        - "Headset Mic"
-        - "Internal Mic 1"
-        - "Internal Mic 2"
+        - Int Spk
+        - Headphone Jack
+        - Earpiece
+        - Headset Mic
+        - Internal Mic 1
+        - Internal Mic 2
 
         # CODEC Pins
         - HPL
diff --git a/Documentation/devicetree/bindings/sound/nvidia,tegra-audio-rt5631.yaml b/Documentation/devicetree/bindings/sound/nvidia,tegra-audio-rt5631.yaml
index a04487002e88..0c8067c3b056 100644
--- a/Documentation/devicetree/bindings/sound/nvidia,tegra-audio-rt5631.yaml
+++ b/Documentation/devicetree/bindings/sound/nvidia,tegra-audio-rt5631.yaml
@@ -31,10 +31,10 @@ properties:
     items:
       enum:
         # Board Connectors
-        - "Int Spk"
-        - "Headphone Jack"
-        - "Mic Jack"
-        - "Int Mic"
+        - Int Spk
+        - Headphone Jack
+        - Mic Jack
+        - Int Mic
 
         # CODEC Pins
         - MIC1
diff --git a/Documentation/devicetree/bindings/ufs/ufs-common.yaml b/Documentation/devicetree/bindings/ufs/ufs-common.yaml
index 47a4e9e1a775..bbaee4f5f7b2 100644
--- a/Documentation/devicetree/bindings/ufs/ufs-common.yaml
+++ b/Documentation/devicetree/bindings/ufs/ufs-common.yaml
@@ -74,7 +74,7 @@ properties:
       Specifies max. load that can be drawn from VCCQ2 supply.
 
 dependencies:
-  freq-table-hz: [ 'clocks' ]
+  freq-table-hz: [ clocks ]
 
 required:
   - interrupts
diff --git a/Documentation/devicetree/bindings/watchdog/toshiba,visconti-wdt.yaml b/Documentation/devicetree/bindings/watchdog/toshiba,visconti-wdt.yaml
index 51d03d5b08ad..3e9fd49d935e 100644
--- a/Documentation/devicetree/bindings/watchdog/toshiba,visconti-wdt.yaml
+++ b/Documentation/devicetree/bindings/watchdog/toshiba,visconti-wdt.yaml
@@ -2,8 +2,8 @@
 # Copyright 2020 Toshiba Electronic Devices & Storage Corporation
 %YAML 1.2
 ---
-$id: "http://devicetree.org/schemas/watchdog/toshiba,visconti-wdt.yaml#"
-$schema: "http://devicetree.org/meta-schemas/core.yaml#"
+$id: http://devicetree.org/schemas/watchdog/toshiba,visconti-wdt.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
 
 title: Toshiba Visconti SoCs PIUWDT Watchdog timer
 
-- 
2.40.1



From miquel.raynal at bootlin.com  Thu Aug 24 01:06:48 2023
From: miquel.raynal at bootlin.com (Miquel Raynal)
Date: Thu, 24 Aug 2023 10:06:48 +0200
Subject: [PATCH] dt-bindings: Drop remaining unneeded quotes
In-Reply-To: <20230823183749.2609013-1-robh@kernel.org>
References: <20230823183749.2609013-1-robh@kernel.org>
Message-ID: <20230824100648.5b6e8b70@xps-13>

Hi Rob,

robh at kernel.org wrote on Wed, 23 Aug 2023 13:28:47 -0500:

> Cleanup bindings dropping the last remaining unneeded quotes. With this,
> the check for this can be enabled in yamllint.
> 
> Signed-off-by: Rob Herring <robh at kernel.org>

Acked-by: Miquel Raynal <miquel.raynal at bootlin.com> # for mtd

Thanks,
Miqu?l


From alexandre.belloni at bootlin.com  Thu Aug 24 13:24:02 2023
From: alexandre.belloni at bootlin.com (Alexandre Belloni)
Date: Thu, 24 Aug 2023 22:24:02 +0200
Subject: [PATCH] dt-bindings: Drop remaining unneeded quotes
In-Reply-To: <20230823183749.2609013-1-robh@kernel.org>
References: <20230823183749.2609013-1-robh@kernel.org>
Message-ID: <20230824201756563976f6@mail.local>

On 23/08/2023 13:28:47-0500, Rob Herring wrote:
> Cleanup bindings dropping the last remaining unneeded quotes. With this,
> the check for this can be enabled in yamllint.
> 
> Signed-off-by: Rob Herring <robh at kernel.org>
Acked-by: Alexandre Belloni <alexandre.belloni at bootlin.com>

> ---
>  .../bindings/arm/arm,embedded-trace-extension.yaml   |  4 ++--
>  .../bindings/arm/arm,trace-buffer-extension.yaml     |  7 ++++---
>  .../devicetree/bindings/arm/arm,vexpress-juno.yaml   |  2 +-
>  .../devicetree/bindings/arm/aspeed/aspeed,sbc.yaml   |  4 ++--
>  .../arm/firmware/tlm,trusted-foundations.yaml        |  4 ++--
>  .../bindings/arm/mstar/mstar,l3bridge.yaml           |  4 ++--
>  .../devicetree/bindings/arm/mstar/mstar,smpctrl.yaml |  4 ++--
>  .../devicetree/bindings/arm/stm32/st,mlahb.yaml      |  4 ++--
>  .../bindings/arm/stm32/st,stm32-syscon.yaml          |  4 ++--
>  .../devicetree/bindings/connector/usb-connector.yaml |  4 ++--
>  Documentation/devicetree/bindings/eeprom/at24.yaml   |  4 ++--
>  Documentation/devicetree/bindings/eeprom/at25.yaml   |  4 ++--
>  .../intel,ixp4xx-network-processing-engine.yaml      |  4 ++--
>  .../bindings/gpio/x-powers,axp209-gpio.yaml          |  4 ++--
>  .../bindings/gpio/xlnx,zynqmp-gpio-modepin.yaml      |  4 ++--
>  .../devicetree/bindings/gpio/xylon,logicvc-gpio.yaml |  4 ++--
>  .../devicetree/bindings/hwmon/iio-hwmon.yaml         |  4 ++--
>  .../bindings/hwmon/starfive,jh71x0-temp.yaml         |  8 ++++----
>  .../devicetree/bindings/i3c/mipi-i3c-hci.yaml        |  4 ++--
>  .../devicetree/bindings/iio/accel/fsl,mma7455.yaml   |  4 ++--
>  .../bindings/iio/adc/atmel,sama9260-adc.yaml         |  4 ++--
>  .../bindings/ipmi/aspeed,ast2400-kcs-bmc.yaml        |  8 ++++----
>  .../devicetree/bindings/ipmi/ipmi-ipmb.yaml          |  2 +-
>  .../devicetree/bindings/ipmi/ipmi-smic.yaml          |  2 +-
>  .../bindings/media/qcom,msm8916-venus.yaml           |  4 ++--
>  .../bindings/mips/loongson/ls2k-reset.yaml           |  4 ++--
>  .../bindings/mips/loongson/rs780e-acpi.yaml          |  4 ++--
>  .../misc/intel,ixp4xx-ahb-queue-manager.yaml         |  4 ++--
>  .../devicetree/bindings/mmc/marvell,xenon-sdhci.yaml |  4 ++--
>  .../bindings/mtd/microchip,mchp48l640.yaml           |  4 ++--
>  .../devicetree/bindings/soc/aspeed/uart-routing.yaml |  4 ++--
>  .../bindings/soc/intel/intel,hps-copy-engine.yaml    |  4 ++--
>  .../bindings/soc/litex/litex,soc-controller.yaml     |  4 ++--
>  .../bindings/soc/renesas/renesas,rzg2l-sysc.yaml     |  4 ++--
>  .../devicetree/bindings/soc/ti/k3-ringacc.yaml       |  4 ++--
>  .../devicetree/bindings/sound/dialog,da7219.yaml     |  4 ++--
>  .../bindings/sound/nvidia,tegra-audio-max9808x.yaml  | 12 ++++++------
>  .../bindings/sound/nvidia,tegra-audio-rt5631.yaml    |  8 ++++----
>  .../devicetree/bindings/ufs/ufs-common.yaml          |  2 +-
>  .../bindings/watchdog/toshiba,visconti-wdt.yaml      |  4 ++--
>  40 files changed, 88 insertions(+), 87 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/arm/arm,embedded-trace-extension.yaml b/Documentation/devicetree/bindings/arm/arm,embedded-trace-extension.yaml
> index 108460627d9a..a477a810f9e9 100644
> --- a/Documentation/devicetree/bindings/arm/arm,embedded-trace-extension.yaml
> +++ b/Documentation/devicetree/bindings/arm/arm,embedded-trace-extension.yaml
> @@ -2,8 +2,8 @@
>  # Copyright 2021, Arm Ltd
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/arm/arm,embedded-trace-extension.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/arm/arm,embedded-trace-extension.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: ARM Embedded Trace Extensions
>  
> diff --git a/Documentation/devicetree/bindings/arm/arm,trace-buffer-extension.yaml b/Documentation/devicetree/bindings/arm/arm,trace-buffer-extension.yaml
> index b1322658063a..8c27e510cb71 100644
> --- a/Documentation/devicetree/bindings/arm/arm,trace-buffer-extension.yaml
> +++ b/Documentation/devicetree/bindings/arm/arm,trace-buffer-extension.yaml
> @@ -2,8 +2,8 @@
>  # Copyright 2021, Arm Ltd
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/arm/arm,trace-buffer-extension.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/arm/arm,trace-buffer-extension.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: ARM Trace Buffer Extensions
>  
> @@ -19,7 +19,8 @@ description: |
>  
>  properties:
>    $nodename:
> -    const: "trbe"
> +    const: trbe
> +
>    compatible:
>      items:
>        - const: arm,trace-buffer-extension
> diff --git a/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml b/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml
> index cdd65881fcdd..8dd6b6446394 100644
> --- a/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml
> +++ b/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml
> @@ -143,7 +143,7 @@ patternProperties:
>        "simple-bus". If the compatible is placed in the "motherboard-bus" node,
>        it is stricter and always has two compatibles.
>      type: object
> -    $ref: '/schemas/simple-bus.yaml'
> +    $ref: /schemas/simple-bus.yaml
>      unevaluatedProperties: false
>  
>      properties:
> diff --git a/Documentation/devicetree/bindings/arm/aspeed/aspeed,sbc.yaml b/Documentation/devicetree/bindings/arm/aspeed/aspeed,sbc.yaml
> index c72aab706484..b8c5cacb09bd 100644
> --- a/Documentation/devicetree/bindings/arm/aspeed/aspeed,sbc.yaml
> +++ b/Documentation/devicetree/bindings/arm/aspeed/aspeed,sbc.yaml
> @@ -2,8 +2,8 @@
>  # Copyright 2021 Joel Stanley, IBM Corp.
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/arm/aspeed/aspeed,sbc.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/arm/aspeed/aspeed,sbc.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: ASPEED Secure Boot Controller
>  
> diff --git a/Documentation/devicetree/bindings/arm/firmware/tlm,trusted-foundations.yaml b/Documentation/devicetree/bindings/arm/firmware/tlm,trusted-foundations.yaml
> index 9d1857c0aa07..e3980b659f63 100644
> --- a/Documentation/devicetree/bindings/arm/firmware/tlm,trusted-foundations.yaml
> +++ b/Documentation/devicetree/bindings/arm/firmware/tlm,trusted-foundations.yaml
> @@ -1,8 +1,8 @@
>  # SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/arm/firmware/tlm,trusted-foundations.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/arm/firmware/tlm,trusted-foundations.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: Trusted Foundations
>  
> diff --git a/Documentation/devicetree/bindings/arm/mstar/mstar,l3bridge.yaml b/Documentation/devicetree/bindings/arm/mstar/mstar,l3bridge.yaml
> index 6816bd68f9cf..a8ac4a2d672d 100644
> --- a/Documentation/devicetree/bindings/arm/mstar/mstar,l3bridge.yaml
> +++ b/Documentation/devicetree/bindings/arm/mstar/mstar,l3bridge.yaml
> @@ -2,8 +2,8 @@
>  # Copyright 2020 thingy.jp.
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/arm/mstar/mstar,l3bridge.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/arm/mstar/mstar,l3bridge.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: MStar/SigmaStar Armv7 SoC l3bridge
>  
> diff --git a/Documentation/devicetree/bindings/arm/mstar/mstar,smpctrl.yaml b/Documentation/devicetree/bindings/arm/mstar/mstar,smpctrl.yaml
> index 599c65980f5d..5739848000b1 100644
> --- a/Documentation/devicetree/bindings/arm/mstar/mstar,smpctrl.yaml
> +++ b/Documentation/devicetree/bindings/arm/mstar/mstar,smpctrl.yaml
> @@ -2,8 +2,8 @@
>  # Copyright 2020 thingy.jp.
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/arm/mstar/mstar,smpctrl.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/arm/mstar/mstar,smpctrl.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: MStar/SigmaStar Armv7 SoC SMP control registers
>  
> diff --git a/Documentation/devicetree/bindings/arm/stm32/st,mlahb.yaml b/Documentation/devicetree/bindings/arm/stm32/st,mlahb.yaml
> index 2297ad3f4774..d2dce238ff5d 100644
> --- a/Documentation/devicetree/bindings/arm/stm32/st,mlahb.yaml
> +++ b/Documentation/devicetree/bindings/arm/stm32/st,mlahb.yaml
> @@ -1,8 +1,8 @@
>  # SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/arm/stm32/st,mlahb.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/arm/stm32/st,mlahb.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: STMicroelectronics STM32 ML-AHB interconnect
>  
> diff --git a/Documentation/devicetree/bindings/arm/stm32/st,stm32-syscon.yaml b/Documentation/devicetree/bindings/arm/stm32/st,stm32-syscon.yaml
> index b63ff591ef8f..d083d8ad48b7 100644
> --- a/Documentation/devicetree/bindings/arm/stm32/st,stm32-syscon.yaml
> +++ b/Documentation/devicetree/bindings/arm/stm32/st,stm32-syscon.yaml
> @@ -1,8 +1,8 @@
>  # SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/arm/stm32/st,stm32-syscon.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/arm/stm32/st,stm32-syscon.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: STMicroelectronics STM32 Platforms System Controller
>  
> diff --git a/Documentation/devicetree/bindings/connector/usb-connector.yaml b/Documentation/devicetree/bindings/connector/usb-connector.yaml
> index 3ecb51f55a71..7c8a3e8430d3 100644
> --- a/Documentation/devicetree/bindings/connector/usb-connector.yaml
> +++ b/Documentation/devicetree/bindings/connector/usb-connector.yaml
> @@ -232,8 +232,8 @@ properties:
>      type: boolean
>  
>  dependencies:
> -  sink-vdos-v1: [ 'sink-vdos' ]
> -  sink-vdos: [ 'sink-vdos-v1' ]
> +  sink-vdos-v1: [ sink-vdos ]
> +  sink-vdos: [ sink-vdos-v1 ]
>  
>  required:
>    - compatible
> diff --git a/Documentation/devicetree/bindings/eeprom/at24.yaml b/Documentation/devicetree/bindings/eeprom/at24.yaml
> index 84af0d5f52aa..b1e5dddf9bf4 100644
> --- a/Documentation/devicetree/bindings/eeprom/at24.yaml
> +++ b/Documentation/devicetree/bindings/eeprom/at24.yaml
> @@ -2,8 +2,8 @@
>  # Copyright 2019 BayLibre SAS
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/eeprom/at24.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/eeprom/at24.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: I2C EEPROMs compatible with Atmel's AT24
>  
> diff --git a/Documentation/devicetree/bindings/eeprom/at25.yaml b/Documentation/devicetree/bindings/eeprom/at25.yaml
> index 0e31bb36ebb1..1715b0c9feea 100644
> --- a/Documentation/devicetree/bindings/eeprom/at25.yaml
> +++ b/Documentation/devicetree/bindings/eeprom/at25.yaml
> @@ -1,8 +1,8 @@
>  # SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/eeprom/at25.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/eeprom/at25.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: SPI EEPROMs or FRAMs compatible with Atmel's AT25
>  
> diff --git a/Documentation/devicetree/bindings/firmware/intel,ixp4xx-network-processing-engine.yaml b/Documentation/devicetree/bindings/firmware/intel,ixp4xx-network-processing-engine.yaml
> index 9a785bbaafb7..e6bed7d93e2d 100644
> --- a/Documentation/devicetree/bindings/firmware/intel,ixp4xx-network-processing-engine.yaml
> +++ b/Documentation/devicetree/bindings/firmware/intel,ixp4xx-network-processing-engine.yaml
> @@ -2,8 +2,8 @@
>  # Copyright 2019 Linaro Ltd.
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/firmware/intel,ixp4xx-network-processing-engine.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/firmware/intel,ixp4xx-network-processing-engine.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: Intel IXP4xx Network Processing Engine
>  
> diff --git a/Documentation/devicetree/bindings/gpio/x-powers,axp209-gpio.yaml b/Documentation/devicetree/bindings/gpio/x-powers,axp209-gpio.yaml
> index 1638cfe90f1c..5eeb29bcdd21 100644
> --- a/Documentation/devicetree/bindings/gpio/x-powers,axp209-gpio.yaml
> +++ b/Documentation/devicetree/bindings/gpio/x-powers,axp209-gpio.yaml
> @@ -1,8 +1,8 @@
>  # SPDX-License-Identifier: GPL-2.0
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/gpio/x-powers,axp209-gpio.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/gpio/x-powers,axp209-gpio.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: X-Powers AXP209 GPIO
>  
> diff --git a/Documentation/devicetree/bindings/gpio/xlnx,zynqmp-gpio-modepin.yaml b/Documentation/devicetree/bindings/gpio/xlnx,zynqmp-gpio-modepin.yaml
> index 18e61aff2185..56143f1fe84a 100644
> --- a/Documentation/devicetree/bindings/gpio/xlnx,zynqmp-gpio-modepin.yaml
> +++ b/Documentation/devicetree/bindings/gpio/xlnx,zynqmp-gpio-modepin.yaml
> @@ -1,8 +1,8 @@
>  # SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/gpio/xlnx,zynqmp-gpio-modepin.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/gpio/xlnx,zynqmp-gpio-modepin.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: ZynqMP Mode Pin GPIO controller
>  
> diff --git a/Documentation/devicetree/bindings/gpio/xylon,logicvc-gpio.yaml b/Documentation/devicetree/bindings/gpio/xylon,logicvc-gpio.yaml
> index a36aec27069c..59c79a6943ec 100644
> --- a/Documentation/devicetree/bindings/gpio/xylon,logicvc-gpio.yaml
> +++ b/Documentation/devicetree/bindings/gpio/xylon,logicvc-gpio.yaml
> @@ -2,8 +2,8 @@
>  # Copyright 2019 Bootlin
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/gpio/xylon,logicvc-gpio.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/gpio/xylon,logicvc-gpio.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: Xylon LogiCVC GPIO controller
>  
> diff --git a/Documentation/devicetree/bindings/hwmon/iio-hwmon.yaml b/Documentation/devicetree/bindings/hwmon/iio-hwmon.yaml
> index c54b5986b365..e5b24782f448 100644
> --- a/Documentation/devicetree/bindings/hwmon/iio-hwmon.yaml
> +++ b/Documentation/devicetree/bindings/hwmon/iio-hwmon.yaml
> @@ -1,8 +1,8 @@
>  # SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/hwmon/iio-hwmon.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/hwmon/iio-hwmon.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: ADC-attached Hardware Sensor
>  
> diff --git a/Documentation/devicetree/bindings/hwmon/starfive,jh71x0-temp.yaml b/Documentation/devicetree/bindings/hwmon/starfive,jh71x0-temp.yaml
> index f5b34528928d..733cba780186 100644
> --- a/Documentation/devicetree/bindings/hwmon/starfive,jh71x0-temp.yaml
> +++ b/Documentation/devicetree/bindings/hwmon/starfive,jh71x0-temp.yaml
> @@ -27,8 +27,8 @@ properties:
>  
>    clock-names:
>      items:
> -      - const: "sense"
> -      - const: "bus"
> +      - const: sense
> +      - const: bus
>  
>    '#thermal-sensor-cells':
>      const: 0
> @@ -39,8 +39,8 @@ properties:
>  
>    reset-names:
>      items:
> -      - const: "sense"
> -      - const: "bus"
> +      - const: sense
> +      - const: bus
>  
>  required:
>    - compatible
> diff --git a/Documentation/devicetree/bindings/i3c/mipi-i3c-hci.yaml b/Documentation/devicetree/bindings/i3c/mipi-i3c-hci.yaml
> index c002afdbfc7c..5dda8cb44cdb 100644
> --- a/Documentation/devicetree/bindings/i3c/mipi-i3c-hci.yaml
> +++ b/Documentation/devicetree/bindings/i3c/mipi-i3c-hci.yaml
> @@ -1,8 +1,8 @@
>  # SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/i3c/mipi-i3c-hci.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/i3c/mipi-i3c-hci.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: MIPI I3C HCI
>  
> diff --git a/Documentation/devicetree/bindings/iio/accel/fsl,mma7455.yaml b/Documentation/devicetree/bindings/iio/accel/fsl,mma7455.yaml
> index c8659c5eba2a..cb31e75ba680 100644
> --- a/Documentation/devicetree/bindings/iio/accel/fsl,mma7455.yaml
> +++ b/Documentation/devicetree/bindings/iio/accel/fsl,mma7455.yaml
> @@ -36,8 +36,8 @@ properties:
>      maxItems: 2
>      items:
>        enum:
> -        - "INT1"
> -        - "INT2"
> +        - INT1
> +        - INT2
>  
>  required:
>    - compatible
> diff --git a/Documentation/devicetree/bindings/iio/adc/atmel,sama9260-adc.yaml b/Documentation/devicetree/bindings/iio/adc/atmel,sama9260-adc.yaml
> index e6a1f915b542..1f30a8569187 100644
> --- a/Documentation/devicetree/bindings/iio/adc/atmel,sama9260-adc.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/atmel,sama9260-adc.yaml
> @@ -56,8 +56,8 @@ properties:
>        String corresponding to an identifier from atmel,adc-res-names property.
>        If not specified, the highest resolution will be used.
>      enum:
> -      - "lowres"
> -      - "highres"
> +      - lowres
> +      - highres
>  
>    atmel,adc-sleep-mode:
>      $ref: /schemas/types.yaml#/definitions/flag
> diff --git a/Documentation/devicetree/bindings/ipmi/aspeed,ast2400-kcs-bmc.yaml b/Documentation/devicetree/bindings/ipmi/aspeed,ast2400-kcs-bmc.yaml
> index 4ff6fabfcb30..129e32c4c774 100644
> --- a/Documentation/devicetree/bindings/ipmi/aspeed,ast2400-kcs-bmc.yaml
> +++ b/Documentation/devicetree/bindings/ipmi/aspeed,ast2400-kcs-bmc.yaml
> @@ -41,7 +41,7 @@ properties:
>        - description: STR register
>  
>    aspeed,lpc-io-reg:
> -    $ref: '/schemas/types.yaml#/definitions/uint32-array'
> +    $ref: /schemas/types.yaml#/definitions/uint32-array
>      minItems: 1
>      maxItems: 2
>      description: |
> @@ -50,7 +50,7 @@ properties:
>        status address may be optionally provided.
>  
>    aspeed,lpc-interrupts:
> -    $ref: "/schemas/types.yaml#/definitions/uint32-array"
> +    $ref: /schemas/types.yaml#/definitions/uint32-array
>      minItems: 2
>      maxItems: 2
>      description: |
> @@ -63,12 +63,12 @@ properties:
>  
>    kcs_chan:
>      deprecated: true
> -    $ref: '/schemas/types.yaml#/definitions/uint32'
> +    $ref: /schemas/types.yaml#/definitions/uint32
>      description: The LPC channel number in the controller
>  
>    kcs_addr:
>      deprecated: true
> -    $ref: '/schemas/types.yaml#/definitions/uint32'
> +    $ref: /schemas/types.yaml#/definitions/uint32
>      description: The host CPU IO map address
>  
>  required:
> diff --git a/Documentation/devicetree/bindings/ipmi/ipmi-ipmb.yaml b/Documentation/devicetree/bindings/ipmi/ipmi-ipmb.yaml
> index 3f25cdb4e99b..52647bff31af 100644
> --- a/Documentation/devicetree/bindings/ipmi/ipmi-ipmb.yaml
> +++ b/Documentation/devicetree/bindings/ipmi/ipmi-ipmb.yaml
> @@ -18,7 +18,7 @@ properties:
>  
>    device_type:
>      items:
> -      - const: "ipmi"
> +      - const: ipmi
>  
>    reg:
>      maxItems: 1
> diff --git a/Documentation/devicetree/bindings/ipmi/ipmi-smic.yaml b/Documentation/devicetree/bindings/ipmi/ipmi-smic.yaml
> index c1b4bf95ef99..4bffa3d86128 100644
> --- a/Documentation/devicetree/bindings/ipmi/ipmi-smic.yaml
> +++ b/Documentation/devicetree/bindings/ipmi/ipmi-smic.yaml
> @@ -20,7 +20,7 @@ properties:
>  
>    device_type:
>      items:
> -      - const: "ipmi"
> +      - const: ipmi
>  
>    reg:
>      maxItems: 1
> diff --git a/Documentation/devicetree/bindings/media/qcom,msm8916-venus.yaml b/Documentation/devicetree/bindings/media/qcom,msm8916-venus.yaml
> index 2350bf4b370e..9410f13ca97c 100644
> --- a/Documentation/devicetree/bindings/media/qcom,msm8916-venus.yaml
> +++ b/Documentation/devicetree/bindings/media/qcom,msm8916-venus.yaml
> @@ -40,7 +40,7 @@ properties:
>  
>      properties:
>        compatible:
> -        const: "venus-decoder"
> +        const: venus-decoder
>  
>      required:
>        - compatible
> @@ -52,7 +52,7 @@ properties:
>  
>      properties:
>        compatible:
> -        const: "venus-encoder"
> +        const: venus-encoder
>  
>      required:
>        - compatible
> diff --git a/Documentation/devicetree/bindings/mips/loongson/ls2k-reset.yaml b/Documentation/devicetree/bindings/mips/loongson/ls2k-reset.yaml
> index 20b5836efd90..358ac8cd4d1d 100644
> --- a/Documentation/devicetree/bindings/mips/loongson/ls2k-reset.yaml
> +++ b/Documentation/devicetree/bindings/mips/loongson/ls2k-reset.yaml
> @@ -1,8 +1,8 @@
>  # SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/mips/loongson/ls2k-reset.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/mips/loongson/ls2k-reset.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: Loongson 2K1000 PM Controller
>  
> diff --git a/Documentation/devicetree/bindings/mips/loongson/rs780e-acpi.yaml b/Documentation/devicetree/bindings/mips/loongson/rs780e-acpi.yaml
> index 7c0f9022202c..3e3a3705e879 100644
> --- a/Documentation/devicetree/bindings/mips/loongson/rs780e-acpi.yaml
> +++ b/Documentation/devicetree/bindings/mips/loongson/rs780e-acpi.yaml
> @@ -1,8 +1,8 @@
>  # SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/mips/loongson/rs780e-acpi.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/mips/loongson/rs780e-acpi.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: Loongson RS780E PCH ACPI Controller
>  
> diff --git a/Documentation/devicetree/bindings/misc/intel,ixp4xx-ahb-queue-manager.yaml b/Documentation/devicetree/bindings/misc/intel,ixp4xx-ahb-queue-manager.yaml
> index 38ab0499102d..36a9dbdf3f03 100644
> --- a/Documentation/devicetree/bindings/misc/intel,ixp4xx-ahb-queue-manager.yaml
> +++ b/Documentation/devicetree/bindings/misc/intel,ixp4xx-ahb-queue-manager.yaml
> @@ -2,8 +2,8 @@
>  # Copyright 2019 Linaro Ltd.
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/misc/intel,ixp4xx-ahb-queue-manager.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/misc/intel,ixp4xx-ahb-queue-manager.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: Intel IXP4xx AHB Queue Manager
>  
> diff --git a/Documentation/devicetree/bindings/mmc/marvell,xenon-sdhci.yaml b/Documentation/devicetree/bindings/mmc/marvell,xenon-sdhci.yaml
> index 3ee758886558..3a8e74894ae0 100644
> --- a/Documentation/devicetree/bindings/mmc/marvell,xenon-sdhci.yaml
> +++ b/Documentation/devicetree/bindings/mmc/marvell,xenon-sdhci.yaml
> @@ -71,8 +71,8 @@ properties:
>    marvell,xenon-phy-type:
>      $ref: /schemas/types.yaml#/definitions/string
>      enum:
> -      - "emmc 5.1 phy"
> -      - "emmc 5.0 phy"
> +      - emmc 5.1 phy
> +      - emmc 5.0 phy
>      description: |
>        Xenon support multiple types of PHYs. To select eMMC 5.1 PHY, set:
>        marvell,xenon-phy-type = "emmc 5.1 phy" eMMC 5.1 PHY is the default
> diff --git a/Documentation/devicetree/bindings/mtd/microchip,mchp48l640.yaml b/Documentation/devicetree/bindings/mtd/microchip,mchp48l640.yaml
> index 00882892f47e..0ff32bd00bf6 100644
> --- a/Documentation/devicetree/bindings/mtd/microchip,mchp48l640.yaml
> +++ b/Documentation/devicetree/bindings/mtd/microchip,mchp48l640.yaml
> @@ -1,8 +1,8 @@
>  # SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/mtd/microchip,mchp48l640.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/mtd/microchip,mchp48l640.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: Microchip 48l640 (and similar) serial EERAM
>  
> diff --git a/Documentation/devicetree/bindings/soc/aspeed/uart-routing.yaml b/Documentation/devicetree/bindings/soc/aspeed/uart-routing.yaml
> index 6876407124dc..51aaf34acb32 100644
> --- a/Documentation/devicetree/bindings/soc/aspeed/uart-routing.yaml
> +++ b/Documentation/devicetree/bindings/soc/aspeed/uart-routing.yaml
> @@ -3,8 +3,8 @@
>  # # Copyright (c) 2021 Aspeed Technology Inc.
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/soc/aspeed/uart-routing.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/soc/aspeed/uart-routing.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: Aspeed UART Routing Controller
>  
> diff --git a/Documentation/devicetree/bindings/soc/intel/intel,hps-copy-engine.yaml b/Documentation/devicetree/bindings/soc/intel/intel,hps-copy-engine.yaml
> index 8634865015cd..ceb81646fe75 100644
> --- a/Documentation/devicetree/bindings/soc/intel/intel,hps-copy-engine.yaml
> +++ b/Documentation/devicetree/bindings/soc/intel/intel,hps-copy-engine.yaml
> @@ -2,8 +2,8 @@
>  # Copyright (C) 2022, Intel Corporation
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/soc/intel/intel,hps-copy-engine.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/soc/intel/intel,hps-copy-engine.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: Intel HPS Copy Engine
>  
> diff --git a/Documentation/devicetree/bindings/soc/litex/litex,soc-controller.yaml b/Documentation/devicetree/bindings/soc/litex/litex,soc-controller.yaml
> index ecae9fa8561b..a64406ca17b5 100644
> --- a/Documentation/devicetree/bindings/soc/litex/litex,soc-controller.yaml
> +++ b/Documentation/devicetree/bindings/soc/litex/litex,soc-controller.yaml
> @@ -2,8 +2,8 @@
>  # Copyright 2020 Antmicro <www.antmicro.com>
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/soc/litex/litex,soc-controller.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/soc/litex/litex,soc-controller.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: LiteX SoC Controller driver
>  
> diff --git a/Documentation/devicetree/bindings/soc/renesas/renesas,rzg2l-sysc.yaml b/Documentation/devicetree/bindings/soc/renesas/renesas,rzg2l-sysc.yaml
> index 398663d21ab1..e52e176d8cb3 100644
> --- a/Documentation/devicetree/bindings/soc/renesas/renesas,rzg2l-sysc.yaml
> +++ b/Documentation/devicetree/bindings/soc/renesas/renesas,rzg2l-sysc.yaml
> @@ -1,8 +1,8 @@
>  # SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/soc/renesas/renesas,rzg2l-sysc.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/soc/renesas/renesas,rzg2l-sysc.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: Renesas RZ/{G2L,V2L} System Controller (SYSC)
>  
> diff --git a/Documentation/devicetree/bindings/soc/ti/k3-ringacc.yaml b/Documentation/devicetree/bindings/soc/ti/k3-ringacc.yaml
> index 22cf9002fee7..4ac00716885e 100644
> --- a/Documentation/devicetree/bindings/soc/ti/k3-ringacc.yaml
> +++ b/Documentation/devicetree/bindings/soc/ti/k3-ringacc.yaml
> @@ -2,8 +2,8 @@
>  # Copyright (C) 2020 Texas Instruments Incorporated - http://www.ti.com/
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/soc/ti/k3-ringacc.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/soc/ti/k3-ringacc.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: Texas Instruments K3 NavigatorSS Ring Accelerator
>  
> diff --git a/Documentation/devicetree/bindings/sound/dialog,da7219.yaml b/Documentation/devicetree/bindings/sound/dialog,da7219.yaml
> index 2d01956cefbb..19137abdba3e 100644
> --- a/Documentation/devicetree/bindings/sound/dialog,da7219.yaml
> +++ b/Documentation/devicetree/bindings/sound/dialog,da7219.yaml
> @@ -74,7 +74,7 @@ properties:
>      $ref: /schemas/types.yaml#/definitions/uint32
>  
>    dlg,mic-amp-in-sel:
> -    enum: ["diff", "se_p", "se_n"]
> +    enum: [diff, se_p, se_n]
>      description:
>        Mic input source type.
>  
> @@ -124,7 +124,7 @@ properties:
>          $ref: /schemas/types.yaml#/definitions/uint32
>  
>        dlg,jack-ins-det-pty:
> -        enum: ["low", "high"]
> +        enum: [low, high]
>          description:
>            Polarity for jack insertion detection.
>          $ref: /schemas/types.yaml#/definitions/string
> diff --git a/Documentation/devicetree/bindings/sound/nvidia,tegra-audio-max9808x.yaml b/Documentation/devicetree/bindings/sound/nvidia,tegra-audio-max9808x.yaml
> index fc89dbd6bf24..c29d7942915c 100644
> --- a/Documentation/devicetree/bindings/sound/nvidia,tegra-audio-max9808x.yaml
> +++ b/Documentation/devicetree/bindings/sound/nvidia,tegra-audio-max9808x.yaml
> @@ -35,12 +35,12 @@ properties:
>      items:
>        enum:
>          # Board Connectors
> -        - "Int Spk"
> -        - "Headphone Jack"
> -        - "Earpiece"
> -        - "Headset Mic"
> -        - "Internal Mic 1"
> -        - "Internal Mic 2"
> +        - Int Spk
> +        - Headphone Jack
> +        - Earpiece
> +        - Headset Mic
> +        - Internal Mic 1
> +        - Internal Mic 2
>  
>          # CODEC Pins
>          - HPL
> diff --git a/Documentation/devicetree/bindings/sound/nvidia,tegra-audio-rt5631.yaml b/Documentation/devicetree/bindings/sound/nvidia,tegra-audio-rt5631.yaml
> index a04487002e88..0c8067c3b056 100644
> --- a/Documentation/devicetree/bindings/sound/nvidia,tegra-audio-rt5631.yaml
> +++ b/Documentation/devicetree/bindings/sound/nvidia,tegra-audio-rt5631.yaml
> @@ -31,10 +31,10 @@ properties:
>      items:
>        enum:
>          # Board Connectors
> -        - "Int Spk"
> -        - "Headphone Jack"
> -        - "Mic Jack"
> -        - "Int Mic"
> +        - Int Spk
> +        - Headphone Jack
> +        - Mic Jack
> +        - Int Mic
>  
>          # CODEC Pins
>          - MIC1
> diff --git a/Documentation/devicetree/bindings/ufs/ufs-common.yaml b/Documentation/devicetree/bindings/ufs/ufs-common.yaml
> index 47a4e9e1a775..bbaee4f5f7b2 100644
> --- a/Documentation/devicetree/bindings/ufs/ufs-common.yaml
> +++ b/Documentation/devicetree/bindings/ufs/ufs-common.yaml
> @@ -74,7 +74,7 @@ properties:
>        Specifies max. load that can be drawn from VCCQ2 supply.
>  
>  dependencies:
> -  freq-table-hz: [ 'clocks' ]
> +  freq-table-hz: [ clocks ]
>  
>  required:
>    - interrupts
> diff --git a/Documentation/devicetree/bindings/watchdog/toshiba,visconti-wdt.yaml b/Documentation/devicetree/bindings/watchdog/toshiba,visconti-wdt.yaml
> index 51d03d5b08ad..3e9fd49d935e 100644
> --- a/Documentation/devicetree/bindings/watchdog/toshiba,visconti-wdt.yaml
> +++ b/Documentation/devicetree/bindings/watchdog/toshiba,visconti-wdt.yaml
> @@ -2,8 +2,8 @@
>  # Copyright 2020 Toshiba Electronic Devices & Storage Corporation
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/watchdog/toshiba,visconti-wdt.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/watchdog/toshiba,visconti-wdt.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: Toshiba Visconti SoCs PIUWDT Watchdog timer
>  
> -- 
> 2.40.1
> 
> 
> -- 
> linux-i3c mailing list
> linux-i3c at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-i3c

-- 
Alexandre Belloni, co-owner and COO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com


From broonie at kernel.org  Wed Aug 23 11:39:29 2023
From: broonie at kernel.org (Mark Brown)
Date: Wed, 23 Aug 2023 19:39:29 +0100
Subject: [PATCH] dt-bindings: Drop remaining unneeded quotes
In-Reply-To: <20230823183749.2609013-1-robh@kernel.org>
References: <20230823183749.2609013-1-robh@kernel.org>
Message-ID: <603cf881-b423-40cc-8bbf-b09d929a8597@sirena.org.uk>

On Wed, Aug 23, 2023 at 01:28:47PM -0500, Rob Herring wrote:
> Cleanup bindings dropping the last remaining unneeded quotes. With this,
> the check for this can be enabled in yamllint.

Acked-by: Mark Brown <broonie at kernel.org>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-i3c/attachments/20230823/fbf969e1/attachment-0001.sig>

From wsa at kernel.org  Wed Aug 23 12:19:25 2023
From: wsa at kernel.org (Wolfram Sang)
Date: Wed, 23 Aug 2023 21:19:25 +0200
Subject: [PATCH] dt-bindings: Drop remaining unneeded quotes
In-Reply-To: <20230823183749.2609013-1-robh@kernel.org>
References: <20230823183749.2609013-1-robh@kernel.org>
Message-ID: <ZOZbvbQCfE/7za7A@shikoro>

On Wed, Aug 23, 2023 at 01:28:47PM -0500, Rob Herring wrote:
> Cleanup bindings dropping the last remaining unneeded quotes. With this,
> the check for this can be enabled in yamllint.
> 
> Signed-off-by: Rob Herring <robh at kernel.org>

Acked-by: Wolfram Sang <wsa at kernel.org> # for AT24/I2C

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-i3c/attachments/20230823/d0270443/attachment-0001.sig>

From linus.walleij at linaro.org  Wed Aug 23 14:16:22 2023
From: linus.walleij at linaro.org (Linus Walleij)
Date: Wed, 23 Aug 2023 23:16:22 +0200
Subject: [PATCH] dt-bindings: Drop remaining unneeded quotes
In-Reply-To: <20230823183749.2609013-1-robh@kernel.org>
References: <20230823183749.2609013-1-robh@kernel.org>
Message-ID: <CACRpkdbuB-NotnMXc9vZj-byjk7spCSkZ+Dx=R_eOqoSV5CdTA@mail.gmail.com>

On Wed, Aug 23, 2023 at 8:38?PM Rob Herring <robh at kernel.org> wrote:

> Cleanup bindings dropping the last remaining unneeded quotes. With this,
> the check for this can be enabled in yamllint.
>
> Signed-off-by: Rob Herring <robh at kernel.org>

Reviewed-by: Linus Walleij <linus.walleij at linaro.org>

Yours,
Linus Walleij


From linux at roeck-us.net  Wed Aug 23 19:56:56 2023
From: linux at roeck-us.net (Guenter Roeck)
Date: Wed, 23 Aug 2023 19:56:56 -0700
Subject: [PATCH] dt-bindings: Drop remaining unneeded quotes
In-Reply-To: <20230823183749.2609013-1-robh@kernel.org>
References: <20230823183749.2609013-1-robh@kernel.org>
Message-ID: <0f6ee9b9-3792-4865-8183-c50d4f3896e1@roeck-us.net>

On Wed, Aug 23, 2023 at 01:28:47PM -0500, Rob Herring wrote:
> Cleanup bindings dropping the last remaining unneeded quotes. With this,
> the check for this can be enabled in yamllint.
> 
> Signed-off-by: Rob Herring <robh at kernel.org>
> ---
>  .../devicetree/bindings/hwmon/iio-hwmon.yaml         |  4 ++--
>  .../bindings/watchdog/toshiba,visconti-wdt.yaml      |  4 ++--

For hwmon and watchdog:

Acked-by: Guenter Roeck <linux at roeck-us.net>

Guenter


From geert at linux-m68k.org  Thu Aug 24 00:41:56 2023
From: geert at linux-m68k.org (Geert Uytterhoeven)
Date: Thu, 24 Aug 2023 09:41:56 +0200
Subject: [PATCH] dt-bindings: Drop remaining unneeded quotes
In-Reply-To: <20230823183749.2609013-1-robh@kernel.org>
References: <20230823183749.2609013-1-robh@kernel.org>
Message-ID: <CAMuHMdWSrpjUK4Fa9cBiRqsnhh2GxyXK5fCt8B-RmaDgEy6OeA@mail.gmail.com>

On Wed, Aug 23, 2023 at 8:38?PM Rob Herring <robh at kernel.org> wrote:
> Cleanup bindings dropping the last remaining unneeded quotes. With this,
> the check for this can be enabled in yamllint.
>
> Signed-off-by: Rob Herring <robh at kernel.org>

>  .../bindings/soc/renesas/renesas,rzg2l-sysc.yaml     |  4 ++--

Acked-by: Geert Uytterhoeven <geert+renesas at glider.be>

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds


From sudeep.holla at arm.com  Thu Aug 24 03:28:16 2023
From: sudeep.holla at arm.com (Sudeep Holla)
Date: Thu, 24 Aug 2023 11:28:16 +0100
Subject: [PATCH] dt-bindings: Drop remaining unneeded quotes
In-Reply-To: <20230823183749.2609013-1-robh@kernel.org>
References: <20230823183749.2609013-1-robh@kernel.org>
Message-ID: <20230824102816.q3qoub6vzen5uomj@bogus>

On Wed, Aug 23, 2023 at 01:28:47PM -0500, Rob Herring wrote:
> Cleanup bindings dropping the last remaining unneeded quotes. With this,
> the check for this can be enabled in yamllint.
> 
> Signed-off-by: Rob Herring <robh at kernel.org>
> ---
>  .../devicetree/bindings/arm/arm,vexpress-juno.yaml   |  2 +-

Acked-by: Sudeep Holla <sudeep.holla at arm.com>

-- 
Regards,
Sudeep


From ulf.hansson at linaro.org  Thu Aug 24 03:42:07 2023
From: ulf.hansson at linaro.org (Ulf Hansson)
Date: Thu, 24 Aug 2023 12:42:07 +0200
Subject: [PATCH] dt-bindings: Drop remaining unneeded quotes
In-Reply-To: <20230823183749.2609013-1-robh@kernel.org>
References: <20230823183749.2609013-1-robh@kernel.org>
Message-ID: <CAPDyKFqExoA8--A-xHdiAkwz70iz-3VuwkvmiJgCDz1zmGPOCw@mail.gmail.com>

On Wed, 23 Aug 2023 at 20:38, Rob Herring <robh at kernel.org> wrote:
>
> Cleanup bindings dropping the last remaining unneeded quotes. With this,
> the check for this can be enabled in yamllint.
>
> Signed-off-by: Rob Herring <robh at kernel.org>

Acked-by: Ulf Hansson <ulf.hansson at linaro.org> # For MMC

Kind regards
Uffe

> ---
>  .../bindings/arm/arm,embedded-trace-extension.yaml   |  4 ++--
>  .../bindings/arm/arm,trace-buffer-extension.yaml     |  7 ++++---
>  .../devicetree/bindings/arm/arm,vexpress-juno.yaml   |  2 +-
>  .../devicetree/bindings/arm/aspeed/aspeed,sbc.yaml   |  4 ++--
>  .../arm/firmware/tlm,trusted-foundations.yaml        |  4 ++--
>  .../bindings/arm/mstar/mstar,l3bridge.yaml           |  4 ++--
>  .../devicetree/bindings/arm/mstar/mstar,smpctrl.yaml |  4 ++--
>  .../devicetree/bindings/arm/stm32/st,mlahb.yaml      |  4 ++--
>  .../bindings/arm/stm32/st,stm32-syscon.yaml          |  4 ++--
>  .../devicetree/bindings/connector/usb-connector.yaml |  4 ++--
>  Documentation/devicetree/bindings/eeprom/at24.yaml   |  4 ++--
>  Documentation/devicetree/bindings/eeprom/at25.yaml   |  4 ++--
>  .../intel,ixp4xx-network-processing-engine.yaml      |  4 ++--
>  .../bindings/gpio/x-powers,axp209-gpio.yaml          |  4 ++--
>  .../bindings/gpio/xlnx,zynqmp-gpio-modepin.yaml      |  4 ++--
>  .../devicetree/bindings/gpio/xylon,logicvc-gpio.yaml |  4 ++--
>  .../devicetree/bindings/hwmon/iio-hwmon.yaml         |  4 ++--
>  .../bindings/hwmon/starfive,jh71x0-temp.yaml         |  8 ++++----
>  .../devicetree/bindings/i3c/mipi-i3c-hci.yaml        |  4 ++--
>  .../devicetree/bindings/iio/accel/fsl,mma7455.yaml   |  4 ++--
>  .../bindings/iio/adc/atmel,sama9260-adc.yaml         |  4 ++--
>  .../bindings/ipmi/aspeed,ast2400-kcs-bmc.yaml        |  8 ++++----
>  .../devicetree/bindings/ipmi/ipmi-ipmb.yaml          |  2 +-
>  .../devicetree/bindings/ipmi/ipmi-smic.yaml          |  2 +-
>  .../bindings/media/qcom,msm8916-venus.yaml           |  4 ++--
>  .../bindings/mips/loongson/ls2k-reset.yaml           |  4 ++--
>  .../bindings/mips/loongson/rs780e-acpi.yaml          |  4 ++--
>  .../misc/intel,ixp4xx-ahb-queue-manager.yaml         |  4 ++--
>  .../devicetree/bindings/mmc/marvell,xenon-sdhci.yaml |  4 ++--
>  .../bindings/mtd/microchip,mchp48l640.yaml           |  4 ++--
>  .../devicetree/bindings/soc/aspeed/uart-routing.yaml |  4 ++--
>  .../bindings/soc/intel/intel,hps-copy-engine.yaml    |  4 ++--
>  .../bindings/soc/litex/litex,soc-controller.yaml     |  4 ++--
>  .../bindings/soc/renesas/renesas,rzg2l-sysc.yaml     |  4 ++--
>  .../devicetree/bindings/soc/ti/k3-ringacc.yaml       |  4 ++--
>  .../devicetree/bindings/sound/dialog,da7219.yaml     |  4 ++--
>  .../bindings/sound/nvidia,tegra-audio-max9808x.yaml  | 12 ++++++------
>  .../bindings/sound/nvidia,tegra-audio-rt5631.yaml    |  8 ++++----
>  .../devicetree/bindings/ufs/ufs-common.yaml          |  2 +-
>  .../bindings/watchdog/toshiba,visconti-wdt.yaml      |  4 ++--
>  40 files changed, 88 insertions(+), 87 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/arm/arm,embedded-trace-extension.yaml b/Documentation/devicetree/bindings/arm/arm,embedded-trace-extension.yaml
> index 108460627d9a..a477a810f9e9 100644
> --- a/Documentation/devicetree/bindings/arm/arm,embedded-trace-extension.yaml
> +++ b/Documentation/devicetree/bindings/arm/arm,embedded-trace-extension.yaml
> @@ -2,8 +2,8 @@
>  # Copyright 2021, Arm Ltd
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/arm/arm,embedded-trace-extension.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/arm/arm,embedded-trace-extension.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>
>  title: ARM Embedded Trace Extensions
>
> diff --git a/Documentation/devicetree/bindings/arm/arm,trace-buffer-extension.yaml b/Documentation/devicetree/bindings/arm/arm,trace-buffer-extension.yaml
> index b1322658063a..8c27e510cb71 100644
> --- a/Documentation/devicetree/bindings/arm/arm,trace-buffer-extension.yaml
> +++ b/Documentation/devicetree/bindings/arm/arm,trace-buffer-extension.yaml
> @@ -2,8 +2,8 @@
>  # Copyright 2021, Arm Ltd
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/arm/arm,trace-buffer-extension.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/arm/arm,trace-buffer-extension.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>
>  title: ARM Trace Buffer Extensions
>
> @@ -19,7 +19,8 @@ description: |
>
>  properties:
>    $nodename:
> -    const: "trbe"
> +    const: trbe
> +
>    compatible:
>      items:
>        - const: arm,trace-buffer-extension
> diff --git a/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml b/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml
> index cdd65881fcdd..8dd6b6446394 100644
> --- a/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml
> +++ b/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml
> @@ -143,7 +143,7 @@ patternProperties:
>        "simple-bus". If the compatible is placed in the "motherboard-bus" node,
>        it is stricter and always has two compatibles.
>      type: object
> -    $ref: '/schemas/simple-bus.yaml'
> +    $ref: /schemas/simple-bus.yaml
>      unevaluatedProperties: false
>
>      properties:
> diff --git a/Documentation/devicetree/bindings/arm/aspeed/aspeed,sbc.yaml b/Documentation/devicetree/bindings/arm/aspeed/aspeed,sbc.yaml
> index c72aab706484..b8c5cacb09bd 100644
> --- a/Documentation/devicetree/bindings/arm/aspeed/aspeed,sbc.yaml
> +++ b/Documentation/devicetree/bindings/arm/aspeed/aspeed,sbc.yaml
> @@ -2,8 +2,8 @@
>  # Copyright 2021 Joel Stanley, IBM Corp.
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/arm/aspeed/aspeed,sbc.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/arm/aspeed/aspeed,sbc.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>
>  title: ASPEED Secure Boot Controller
>
> diff --git a/Documentation/devicetree/bindings/arm/firmware/tlm,trusted-foundations.yaml b/Documentation/devicetree/bindings/arm/firmware/tlm,trusted-foundations.yaml
> index 9d1857c0aa07..e3980b659f63 100644
> --- a/Documentation/devicetree/bindings/arm/firmware/tlm,trusted-foundations.yaml
> +++ b/Documentation/devicetree/bindings/arm/firmware/tlm,trusted-foundations.yaml
> @@ -1,8 +1,8 @@
>  # SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/arm/firmware/tlm,trusted-foundations.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/arm/firmware/tlm,trusted-foundations.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>
>  title: Trusted Foundations
>
> diff --git a/Documentation/devicetree/bindings/arm/mstar/mstar,l3bridge.yaml b/Documentation/devicetree/bindings/arm/mstar/mstar,l3bridge.yaml
> index 6816bd68f9cf..a8ac4a2d672d 100644
> --- a/Documentation/devicetree/bindings/arm/mstar/mstar,l3bridge.yaml
> +++ b/Documentation/devicetree/bindings/arm/mstar/mstar,l3bridge.yaml
> @@ -2,8 +2,8 @@
>  # Copyright 2020 thingy.jp.
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/arm/mstar/mstar,l3bridge.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/arm/mstar/mstar,l3bridge.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>
>  title: MStar/SigmaStar Armv7 SoC l3bridge
>
> diff --git a/Documentation/devicetree/bindings/arm/mstar/mstar,smpctrl.yaml b/Documentation/devicetree/bindings/arm/mstar/mstar,smpctrl.yaml
> index 599c65980f5d..5739848000b1 100644
> --- a/Documentation/devicetree/bindings/arm/mstar/mstar,smpctrl.yaml
> +++ b/Documentation/devicetree/bindings/arm/mstar/mstar,smpctrl.yaml
> @@ -2,8 +2,8 @@
>  # Copyright 2020 thingy.jp.
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/arm/mstar/mstar,smpctrl.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/arm/mstar/mstar,smpctrl.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>
>  title: MStar/SigmaStar Armv7 SoC SMP control registers
>
> diff --git a/Documentation/devicetree/bindings/arm/stm32/st,mlahb.yaml b/Documentation/devicetree/bindings/arm/stm32/st,mlahb.yaml
> index 2297ad3f4774..d2dce238ff5d 100644
> --- a/Documentation/devicetree/bindings/arm/stm32/st,mlahb.yaml
> +++ b/Documentation/devicetree/bindings/arm/stm32/st,mlahb.yaml
> @@ -1,8 +1,8 @@
>  # SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/arm/stm32/st,mlahb.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/arm/stm32/st,mlahb.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>
>  title: STMicroelectronics STM32 ML-AHB interconnect
>
> diff --git a/Documentation/devicetree/bindings/arm/stm32/st,stm32-syscon.yaml b/Documentation/devicetree/bindings/arm/stm32/st,stm32-syscon.yaml
> index b63ff591ef8f..d083d8ad48b7 100644
> --- a/Documentation/devicetree/bindings/arm/stm32/st,stm32-syscon.yaml
> +++ b/Documentation/devicetree/bindings/arm/stm32/st,stm32-syscon.yaml
> @@ -1,8 +1,8 @@
>  # SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/arm/stm32/st,stm32-syscon.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/arm/stm32/st,stm32-syscon.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>
>  title: STMicroelectronics STM32 Platforms System Controller
>
> diff --git a/Documentation/devicetree/bindings/connector/usb-connector.yaml b/Documentation/devicetree/bindings/connector/usb-connector.yaml
> index 3ecb51f55a71..7c8a3e8430d3 100644
> --- a/Documentation/devicetree/bindings/connector/usb-connector.yaml
> +++ b/Documentation/devicetree/bindings/connector/usb-connector.yaml
> @@ -232,8 +232,8 @@ properties:
>      type: boolean
>
>  dependencies:
> -  sink-vdos-v1: [ 'sink-vdos' ]
> -  sink-vdos: [ 'sink-vdos-v1' ]
> +  sink-vdos-v1: [ sink-vdos ]
> +  sink-vdos: [ sink-vdos-v1 ]
>
>  required:
>    - compatible
> diff --git a/Documentation/devicetree/bindings/eeprom/at24.yaml b/Documentation/devicetree/bindings/eeprom/at24.yaml
> index 84af0d5f52aa..b1e5dddf9bf4 100644
> --- a/Documentation/devicetree/bindings/eeprom/at24.yaml
> +++ b/Documentation/devicetree/bindings/eeprom/at24.yaml
> @@ -2,8 +2,8 @@
>  # Copyright 2019 BayLibre SAS
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/eeprom/at24.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/eeprom/at24.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>
>  title: I2C EEPROMs compatible with Atmel's AT24
>
> diff --git a/Documentation/devicetree/bindings/eeprom/at25.yaml b/Documentation/devicetree/bindings/eeprom/at25.yaml
> index 0e31bb36ebb1..1715b0c9feea 100644
> --- a/Documentation/devicetree/bindings/eeprom/at25.yaml
> +++ b/Documentation/devicetree/bindings/eeprom/at25.yaml
> @@ -1,8 +1,8 @@
>  # SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/eeprom/at25.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/eeprom/at25.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>
>  title: SPI EEPROMs or FRAMs compatible with Atmel's AT25
>
> diff --git a/Documentation/devicetree/bindings/firmware/intel,ixp4xx-network-processing-engine.yaml b/Documentation/devicetree/bindings/firmware/intel,ixp4xx-network-processing-engine.yaml
> index 9a785bbaafb7..e6bed7d93e2d 100644
> --- a/Documentation/devicetree/bindings/firmware/intel,ixp4xx-network-processing-engine.yaml
> +++ b/Documentation/devicetree/bindings/firmware/intel,ixp4xx-network-processing-engine.yaml
> @@ -2,8 +2,8 @@
>  # Copyright 2019 Linaro Ltd.
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/firmware/intel,ixp4xx-network-processing-engine.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/firmware/intel,ixp4xx-network-processing-engine.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>
>  title: Intel IXP4xx Network Processing Engine
>
> diff --git a/Documentation/devicetree/bindings/gpio/x-powers,axp209-gpio.yaml b/Documentation/devicetree/bindings/gpio/x-powers,axp209-gpio.yaml
> index 1638cfe90f1c..5eeb29bcdd21 100644
> --- a/Documentation/devicetree/bindings/gpio/x-powers,axp209-gpio.yaml
> +++ b/Documentation/devicetree/bindings/gpio/x-powers,axp209-gpio.yaml
> @@ -1,8 +1,8 @@
>  # SPDX-License-Identifier: GPL-2.0
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/gpio/x-powers,axp209-gpio.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/gpio/x-powers,axp209-gpio.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>
>  title: X-Powers AXP209 GPIO
>
> diff --git a/Documentation/devicetree/bindings/gpio/xlnx,zynqmp-gpio-modepin.yaml b/Documentation/devicetree/bindings/gpio/xlnx,zynqmp-gpio-modepin.yaml
> index 18e61aff2185..56143f1fe84a 100644
> --- a/Documentation/devicetree/bindings/gpio/xlnx,zynqmp-gpio-modepin.yaml
> +++ b/Documentation/devicetree/bindings/gpio/xlnx,zynqmp-gpio-modepin.yaml
> @@ -1,8 +1,8 @@
>  # SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/gpio/xlnx,zynqmp-gpio-modepin.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/gpio/xlnx,zynqmp-gpio-modepin.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>
>  title: ZynqMP Mode Pin GPIO controller
>
> diff --git a/Documentation/devicetree/bindings/gpio/xylon,logicvc-gpio.yaml b/Documentation/devicetree/bindings/gpio/xylon,logicvc-gpio.yaml
> index a36aec27069c..59c79a6943ec 100644
> --- a/Documentation/devicetree/bindings/gpio/xylon,logicvc-gpio.yaml
> +++ b/Documentation/devicetree/bindings/gpio/xylon,logicvc-gpio.yaml
> @@ -2,8 +2,8 @@
>  # Copyright 2019 Bootlin
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/gpio/xylon,logicvc-gpio.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/gpio/xylon,logicvc-gpio.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>
>  title: Xylon LogiCVC GPIO controller
>
> diff --git a/Documentation/devicetree/bindings/hwmon/iio-hwmon.yaml b/Documentation/devicetree/bindings/hwmon/iio-hwmon.yaml
> index c54b5986b365..e5b24782f448 100644
> --- a/Documentation/devicetree/bindings/hwmon/iio-hwmon.yaml
> +++ b/Documentation/devicetree/bindings/hwmon/iio-hwmon.yaml
> @@ -1,8 +1,8 @@
>  # SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/hwmon/iio-hwmon.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/hwmon/iio-hwmon.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>
>  title: ADC-attached Hardware Sensor
>
> diff --git a/Documentation/devicetree/bindings/hwmon/starfive,jh71x0-temp.yaml b/Documentation/devicetree/bindings/hwmon/starfive,jh71x0-temp.yaml
> index f5b34528928d..733cba780186 100644
> --- a/Documentation/devicetree/bindings/hwmon/starfive,jh71x0-temp.yaml
> +++ b/Documentation/devicetree/bindings/hwmon/starfive,jh71x0-temp.yaml
> @@ -27,8 +27,8 @@ properties:
>
>    clock-names:
>      items:
> -      - const: "sense"
> -      - const: "bus"
> +      - const: sense
> +      - const: bus
>
>    '#thermal-sensor-cells':
>      const: 0
> @@ -39,8 +39,8 @@ properties:
>
>    reset-names:
>      items:
> -      - const: "sense"
> -      - const: "bus"
> +      - const: sense
> +      - const: bus
>
>  required:
>    - compatible
> diff --git a/Documentation/devicetree/bindings/i3c/mipi-i3c-hci.yaml b/Documentation/devicetree/bindings/i3c/mipi-i3c-hci.yaml
> index c002afdbfc7c..5dda8cb44cdb 100644
> --- a/Documentation/devicetree/bindings/i3c/mipi-i3c-hci.yaml
> +++ b/Documentation/devicetree/bindings/i3c/mipi-i3c-hci.yaml
> @@ -1,8 +1,8 @@
>  # SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/i3c/mipi-i3c-hci.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/i3c/mipi-i3c-hci.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>
>  title: MIPI I3C HCI
>
> diff --git a/Documentation/devicetree/bindings/iio/accel/fsl,mma7455.yaml b/Documentation/devicetree/bindings/iio/accel/fsl,mma7455.yaml
> index c8659c5eba2a..cb31e75ba680 100644
> --- a/Documentation/devicetree/bindings/iio/accel/fsl,mma7455.yaml
> +++ b/Documentation/devicetree/bindings/iio/accel/fsl,mma7455.yaml
> @@ -36,8 +36,8 @@ properties:
>      maxItems: 2
>      items:
>        enum:
> -        - "INT1"
> -        - "INT2"
> +        - INT1
> +        - INT2
>
>  required:
>    - compatible
> diff --git a/Documentation/devicetree/bindings/iio/adc/atmel,sama9260-adc.yaml b/Documentation/devicetree/bindings/iio/adc/atmel,sama9260-adc.yaml
> index e6a1f915b542..1f30a8569187 100644
> --- a/Documentation/devicetree/bindings/iio/adc/atmel,sama9260-adc.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/atmel,sama9260-adc.yaml
> @@ -56,8 +56,8 @@ properties:
>        String corresponding to an identifier from atmel,adc-res-names property.
>        If not specified, the highest resolution will be used.
>      enum:
> -      - "lowres"
> -      - "highres"
> +      - lowres
> +      - highres
>
>    atmel,adc-sleep-mode:
>      $ref: /schemas/types.yaml#/definitions/flag
> diff --git a/Documentation/devicetree/bindings/ipmi/aspeed,ast2400-kcs-bmc.yaml b/Documentation/devicetree/bindings/ipmi/aspeed,ast2400-kcs-bmc.yaml
> index 4ff6fabfcb30..129e32c4c774 100644
> --- a/Documentation/devicetree/bindings/ipmi/aspeed,ast2400-kcs-bmc.yaml
> +++ b/Documentation/devicetree/bindings/ipmi/aspeed,ast2400-kcs-bmc.yaml
> @@ -41,7 +41,7 @@ properties:
>        - description: STR register
>
>    aspeed,lpc-io-reg:
> -    $ref: '/schemas/types.yaml#/definitions/uint32-array'
> +    $ref: /schemas/types.yaml#/definitions/uint32-array
>      minItems: 1
>      maxItems: 2
>      description: |
> @@ -50,7 +50,7 @@ properties:
>        status address may be optionally provided.
>
>    aspeed,lpc-interrupts:
> -    $ref: "/schemas/types.yaml#/definitions/uint32-array"
> +    $ref: /schemas/types.yaml#/definitions/uint32-array
>      minItems: 2
>      maxItems: 2
>      description: |
> @@ -63,12 +63,12 @@ properties:
>
>    kcs_chan:
>      deprecated: true
> -    $ref: '/schemas/types.yaml#/definitions/uint32'
> +    $ref: /schemas/types.yaml#/definitions/uint32
>      description: The LPC channel number in the controller
>
>    kcs_addr:
>      deprecated: true
> -    $ref: '/schemas/types.yaml#/definitions/uint32'
> +    $ref: /schemas/types.yaml#/definitions/uint32
>      description: The host CPU IO map address
>
>  required:
> diff --git a/Documentation/devicetree/bindings/ipmi/ipmi-ipmb.yaml b/Documentation/devicetree/bindings/ipmi/ipmi-ipmb.yaml
> index 3f25cdb4e99b..52647bff31af 100644
> --- a/Documentation/devicetree/bindings/ipmi/ipmi-ipmb.yaml
> +++ b/Documentation/devicetree/bindings/ipmi/ipmi-ipmb.yaml
> @@ -18,7 +18,7 @@ properties:
>
>    device_type:
>      items:
> -      - const: "ipmi"
> +      - const: ipmi
>
>    reg:
>      maxItems: 1
> diff --git a/Documentation/devicetree/bindings/ipmi/ipmi-smic.yaml b/Documentation/devicetree/bindings/ipmi/ipmi-smic.yaml
> index c1b4bf95ef99..4bffa3d86128 100644
> --- a/Documentation/devicetree/bindings/ipmi/ipmi-smic.yaml
> +++ b/Documentation/devicetree/bindings/ipmi/ipmi-smic.yaml
> @@ -20,7 +20,7 @@ properties:
>
>    device_type:
>      items:
> -      - const: "ipmi"
> +      - const: ipmi
>
>    reg:
>      maxItems: 1
> diff --git a/Documentation/devicetree/bindings/media/qcom,msm8916-venus.yaml b/Documentation/devicetree/bindings/media/qcom,msm8916-venus.yaml
> index 2350bf4b370e..9410f13ca97c 100644
> --- a/Documentation/devicetree/bindings/media/qcom,msm8916-venus.yaml
> +++ b/Documentation/devicetree/bindings/media/qcom,msm8916-venus.yaml
> @@ -40,7 +40,7 @@ properties:
>
>      properties:
>        compatible:
> -        const: "venus-decoder"
> +        const: venus-decoder
>
>      required:
>        - compatible
> @@ -52,7 +52,7 @@ properties:
>
>      properties:
>        compatible:
> -        const: "venus-encoder"
> +        const: venus-encoder
>
>      required:
>        - compatible
> diff --git a/Documentation/devicetree/bindings/mips/loongson/ls2k-reset.yaml b/Documentation/devicetree/bindings/mips/loongson/ls2k-reset.yaml
> index 20b5836efd90..358ac8cd4d1d 100644
> --- a/Documentation/devicetree/bindings/mips/loongson/ls2k-reset.yaml
> +++ b/Documentation/devicetree/bindings/mips/loongson/ls2k-reset.yaml
> @@ -1,8 +1,8 @@
>  # SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/mips/loongson/ls2k-reset.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/mips/loongson/ls2k-reset.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>
>  title: Loongson 2K1000 PM Controller
>
> diff --git a/Documentation/devicetree/bindings/mips/loongson/rs780e-acpi.yaml b/Documentation/devicetree/bindings/mips/loongson/rs780e-acpi.yaml
> index 7c0f9022202c..3e3a3705e879 100644
> --- a/Documentation/devicetree/bindings/mips/loongson/rs780e-acpi.yaml
> +++ b/Documentation/devicetree/bindings/mips/loongson/rs780e-acpi.yaml
> @@ -1,8 +1,8 @@
>  # SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/mips/loongson/rs780e-acpi.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/mips/loongson/rs780e-acpi.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>
>  title: Loongson RS780E PCH ACPI Controller
>
> diff --git a/Documentation/devicetree/bindings/misc/intel,ixp4xx-ahb-queue-manager.yaml b/Documentation/devicetree/bindings/misc/intel,ixp4xx-ahb-queue-manager.yaml
> index 38ab0499102d..36a9dbdf3f03 100644
> --- a/Documentation/devicetree/bindings/misc/intel,ixp4xx-ahb-queue-manager.yaml
> +++ b/Documentation/devicetree/bindings/misc/intel,ixp4xx-ahb-queue-manager.yaml
> @@ -2,8 +2,8 @@
>  # Copyright 2019 Linaro Ltd.
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/misc/intel,ixp4xx-ahb-queue-manager.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/misc/intel,ixp4xx-ahb-queue-manager.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>
>  title: Intel IXP4xx AHB Queue Manager
>
> diff --git a/Documentation/devicetree/bindings/mmc/marvell,xenon-sdhci.yaml b/Documentation/devicetree/bindings/mmc/marvell,xenon-sdhci.yaml
> index 3ee758886558..3a8e74894ae0 100644
> --- a/Documentation/devicetree/bindings/mmc/marvell,xenon-sdhci.yaml
> +++ b/Documentation/devicetree/bindings/mmc/marvell,xenon-sdhci.yaml
> @@ -71,8 +71,8 @@ properties:
>    marvell,xenon-phy-type:
>      $ref: /schemas/types.yaml#/definitions/string
>      enum:
> -      - "emmc 5.1 phy"
> -      - "emmc 5.0 phy"
> +      - emmc 5.1 phy
> +      - emmc 5.0 phy
>      description: |
>        Xenon support multiple types of PHYs. To select eMMC 5.1 PHY, set:
>        marvell,xenon-phy-type = "emmc 5.1 phy" eMMC 5.1 PHY is the default
> diff --git a/Documentation/devicetree/bindings/mtd/microchip,mchp48l640.yaml b/Documentation/devicetree/bindings/mtd/microchip,mchp48l640.yaml
> index 00882892f47e..0ff32bd00bf6 100644
> --- a/Documentation/devicetree/bindings/mtd/microchip,mchp48l640.yaml
> +++ b/Documentation/devicetree/bindings/mtd/microchip,mchp48l640.yaml
> @@ -1,8 +1,8 @@
>  # SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/mtd/microchip,mchp48l640.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/mtd/microchip,mchp48l640.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>
>  title: Microchip 48l640 (and similar) serial EERAM
>
> diff --git a/Documentation/devicetree/bindings/soc/aspeed/uart-routing.yaml b/Documentation/devicetree/bindings/soc/aspeed/uart-routing.yaml
> index 6876407124dc..51aaf34acb32 100644
> --- a/Documentation/devicetree/bindings/soc/aspeed/uart-routing.yaml
> +++ b/Documentation/devicetree/bindings/soc/aspeed/uart-routing.yaml
> @@ -3,8 +3,8 @@
>  # # Copyright (c) 2021 Aspeed Technology Inc.
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/soc/aspeed/uart-routing.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/soc/aspeed/uart-routing.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>
>  title: Aspeed UART Routing Controller
>
> diff --git a/Documentation/devicetree/bindings/soc/intel/intel,hps-copy-engine.yaml b/Documentation/devicetree/bindings/soc/intel/intel,hps-copy-engine.yaml
> index 8634865015cd..ceb81646fe75 100644
> --- a/Documentation/devicetree/bindings/soc/intel/intel,hps-copy-engine.yaml
> +++ b/Documentation/devicetree/bindings/soc/intel/intel,hps-copy-engine.yaml
> @@ -2,8 +2,8 @@
>  # Copyright (C) 2022, Intel Corporation
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/soc/intel/intel,hps-copy-engine.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/soc/intel/intel,hps-copy-engine.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>
>  title: Intel HPS Copy Engine
>
> diff --git a/Documentation/devicetree/bindings/soc/litex/litex,soc-controller.yaml b/Documentation/devicetree/bindings/soc/litex/litex,soc-controller.yaml
> index ecae9fa8561b..a64406ca17b5 100644
> --- a/Documentation/devicetree/bindings/soc/litex/litex,soc-controller.yaml
> +++ b/Documentation/devicetree/bindings/soc/litex/litex,soc-controller.yaml
> @@ -2,8 +2,8 @@
>  # Copyright 2020 Antmicro <www.antmicro.com>
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/soc/litex/litex,soc-controller.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/soc/litex/litex,soc-controller.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>
>  title: LiteX SoC Controller driver
>
> diff --git a/Documentation/devicetree/bindings/soc/renesas/renesas,rzg2l-sysc.yaml b/Documentation/devicetree/bindings/soc/renesas/renesas,rzg2l-sysc.yaml
> index 398663d21ab1..e52e176d8cb3 100644
> --- a/Documentation/devicetree/bindings/soc/renesas/renesas,rzg2l-sysc.yaml
> +++ b/Documentation/devicetree/bindings/soc/renesas/renesas,rzg2l-sysc.yaml
> @@ -1,8 +1,8 @@
>  # SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/soc/renesas/renesas,rzg2l-sysc.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/soc/renesas/renesas,rzg2l-sysc.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>
>  title: Renesas RZ/{G2L,V2L} System Controller (SYSC)
>
> diff --git a/Documentation/devicetree/bindings/soc/ti/k3-ringacc.yaml b/Documentation/devicetree/bindings/soc/ti/k3-ringacc.yaml
> index 22cf9002fee7..4ac00716885e 100644
> --- a/Documentation/devicetree/bindings/soc/ti/k3-ringacc.yaml
> +++ b/Documentation/devicetree/bindings/soc/ti/k3-ringacc.yaml
> @@ -2,8 +2,8 @@
>  # Copyright (C) 2020 Texas Instruments Incorporated - http://www.ti.com/
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/soc/ti/k3-ringacc.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/soc/ti/k3-ringacc.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>
>  title: Texas Instruments K3 NavigatorSS Ring Accelerator
>
> diff --git a/Documentation/devicetree/bindings/sound/dialog,da7219.yaml b/Documentation/devicetree/bindings/sound/dialog,da7219.yaml
> index 2d01956cefbb..19137abdba3e 100644
> --- a/Documentation/devicetree/bindings/sound/dialog,da7219.yaml
> +++ b/Documentation/devicetree/bindings/sound/dialog,da7219.yaml
> @@ -74,7 +74,7 @@ properties:
>      $ref: /schemas/types.yaml#/definitions/uint32
>
>    dlg,mic-amp-in-sel:
> -    enum: ["diff", "se_p", "se_n"]
> +    enum: [diff, se_p, se_n]
>      description:
>        Mic input source type.
>
> @@ -124,7 +124,7 @@ properties:
>          $ref: /schemas/types.yaml#/definitions/uint32
>
>        dlg,jack-ins-det-pty:
> -        enum: ["low", "high"]
> +        enum: [low, high]
>          description:
>            Polarity for jack insertion detection.
>          $ref: /schemas/types.yaml#/definitions/string
> diff --git a/Documentation/devicetree/bindings/sound/nvidia,tegra-audio-max9808x.yaml b/Documentation/devicetree/bindings/sound/nvidia,tegra-audio-max9808x.yaml
> index fc89dbd6bf24..c29d7942915c 100644
> --- a/Documentation/devicetree/bindings/sound/nvidia,tegra-audio-max9808x.yaml
> +++ b/Documentation/devicetree/bindings/sound/nvidia,tegra-audio-max9808x.yaml
> @@ -35,12 +35,12 @@ properties:
>      items:
>        enum:
>          # Board Connectors
> -        - "Int Spk"
> -        - "Headphone Jack"
> -        - "Earpiece"
> -        - "Headset Mic"
> -        - "Internal Mic 1"
> -        - "Internal Mic 2"
> +        - Int Spk
> +        - Headphone Jack
> +        - Earpiece
> +        - Headset Mic
> +        - Internal Mic 1
> +        - Internal Mic 2
>
>          # CODEC Pins
>          - HPL
> diff --git a/Documentation/devicetree/bindings/sound/nvidia,tegra-audio-rt5631.yaml b/Documentation/devicetree/bindings/sound/nvidia,tegra-audio-rt5631.yaml
> index a04487002e88..0c8067c3b056 100644
> --- a/Documentation/devicetree/bindings/sound/nvidia,tegra-audio-rt5631.yaml
> +++ b/Documentation/devicetree/bindings/sound/nvidia,tegra-audio-rt5631.yaml
> @@ -31,10 +31,10 @@ properties:
>      items:
>        enum:
>          # Board Connectors
> -        - "Int Spk"
> -        - "Headphone Jack"
> -        - "Mic Jack"
> -        - "Int Mic"
> +        - Int Spk
> +        - Headphone Jack
> +        - Mic Jack
> +        - Int Mic
>
>          # CODEC Pins
>          - MIC1
> diff --git a/Documentation/devicetree/bindings/ufs/ufs-common.yaml b/Documentation/devicetree/bindings/ufs/ufs-common.yaml
> index 47a4e9e1a775..bbaee4f5f7b2 100644
> --- a/Documentation/devicetree/bindings/ufs/ufs-common.yaml
> +++ b/Documentation/devicetree/bindings/ufs/ufs-common.yaml
> @@ -74,7 +74,7 @@ properties:
>        Specifies max. load that can be drawn from VCCQ2 supply.
>
>  dependencies:
> -  freq-table-hz: [ 'clocks' ]
> +  freq-table-hz: [ clocks ]
>
>  required:
>    - interrupts
> diff --git a/Documentation/devicetree/bindings/watchdog/toshiba,visconti-wdt.yaml b/Documentation/devicetree/bindings/watchdog/toshiba,visconti-wdt.yaml
> index 51d03d5b08ad..3e9fd49d935e 100644
> --- a/Documentation/devicetree/bindings/watchdog/toshiba,visconti-wdt.yaml
> +++ b/Documentation/devicetree/bindings/watchdog/toshiba,visconti-wdt.yaml
> @@ -2,8 +2,8 @@
>  # Copyright 2020 Toshiba Electronic Devices & Storage Corporation
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/watchdog/toshiba,visconti-wdt.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/watchdog/toshiba,visconti-wdt.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>
>  title: Toshiba Visconti SoCs PIUWDT Watchdog timer
>
> --
> 2.40.1
>


From robh at kernel.org  Thu Aug 24 07:08:33 2023
From: robh at kernel.org (Rob Herring)
Date: Thu, 24 Aug 2023 09:08:33 -0500
Subject: [PATCH] dt-bindings: Drop remaining unneeded quotes
In-Reply-To: <20230823183749.2609013-1-robh@kernel.org>
References: <20230823183749.2609013-1-robh@kernel.org>
Message-ID: <169288610889.715445.11107086315099689865.robh@kernel.org>


On Wed, 23 Aug 2023 13:28:47 -0500, Rob Herring wrote:
> Cleanup bindings dropping the last remaining unneeded quotes. With this,
> the check for this can be enabled in yamllint.
> 
> Signed-off-by: Rob Herring <robh at kernel.org>
> ---
>  .../bindings/arm/arm,embedded-trace-extension.yaml   |  4 ++--
>  .../bindings/arm/arm,trace-buffer-extension.yaml     |  7 ++++---
>  .../devicetree/bindings/arm/arm,vexpress-juno.yaml   |  2 +-
>  .../devicetree/bindings/arm/aspeed/aspeed,sbc.yaml   |  4 ++--
>  .../arm/firmware/tlm,trusted-foundations.yaml        |  4 ++--
>  .../bindings/arm/mstar/mstar,l3bridge.yaml           |  4 ++--
>  .../devicetree/bindings/arm/mstar/mstar,smpctrl.yaml |  4 ++--
>  .../devicetree/bindings/arm/stm32/st,mlahb.yaml      |  4 ++--
>  .../bindings/arm/stm32/st,stm32-syscon.yaml          |  4 ++--
>  .../devicetree/bindings/connector/usb-connector.yaml |  4 ++--
>  Documentation/devicetree/bindings/eeprom/at24.yaml   |  4 ++--
>  Documentation/devicetree/bindings/eeprom/at25.yaml   |  4 ++--
>  .../intel,ixp4xx-network-processing-engine.yaml      |  4 ++--
>  .../bindings/gpio/x-powers,axp209-gpio.yaml          |  4 ++--
>  .../bindings/gpio/xlnx,zynqmp-gpio-modepin.yaml      |  4 ++--
>  .../devicetree/bindings/gpio/xylon,logicvc-gpio.yaml |  4 ++--
>  .../devicetree/bindings/hwmon/iio-hwmon.yaml         |  4 ++--
>  .../bindings/hwmon/starfive,jh71x0-temp.yaml         |  8 ++++----
>  .../devicetree/bindings/i3c/mipi-i3c-hci.yaml        |  4 ++--
>  .../devicetree/bindings/iio/accel/fsl,mma7455.yaml   |  4 ++--
>  .../bindings/iio/adc/atmel,sama9260-adc.yaml         |  4 ++--
>  .../bindings/ipmi/aspeed,ast2400-kcs-bmc.yaml        |  8 ++++----
>  .../devicetree/bindings/ipmi/ipmi-ipmb.yaml          |  2 +-
>  .../devicetree/bindings/ipmi/ipmi-smic.yaml          |  2 +-
>  .../bindings/media/qcom,msm8916-venus.yaml           |  4 ++--
>  .../bindings/mips/loongson/ls2k-reset.yaml           |  4 ++--
>  .../bindings/mips/loongson/rs780e-acpi.yaml          |  4 ++--
>  .../misc/intel,ixp4xx-ahb-queue-manager.yaml         |  4 ++--
>  .../devicetree/bindings/mmc/marvell,xenon-sdhci.yaml |  4 ++--
>  .../bindings/mtd/microchip,mchp48l640.yaml           |  4 ++--
>  .../devicetree/bindings/soc/aspeed/uart-routing.yaml |  4 ++--
>  .../bindings/soc/intel/intel,hps-copy-engine.yaml    |  4 ++--
>  .../bindings/soc/litex/litex,soc-controller.yaml     |  4 ++--
>  .../bindings/soc/renesas/renesas,rzg2l-sysc.yaml     |  4 ++--
>  .../devicetree/bindings/soc/ti/k3-ringacc.yaml       |  4 ++--
>  .../devicetree/bindings/sound/dialog,da7219.yaml     |  4 ++--
>  .../bindings/sound/nvidia,tegra-audio-max9808x.yaml  | 12 ++++++------
>  .../bindings/sound/nvidia,tegra-audio-rt5631.yaml    |  8 ++++----
>  .../devicetree/bindings/ufs/ufs-common.yaml          |  2 +-
>  .../bindings/watchdog/toshiba,visconti-wdt.yaml      |  4 ++--
>  40 files changed, 88 insertions(+), 87 deletions(-)
> 

Applied, thanks!



From Jonathan.Cameron at Huawei.com  Thu Aug 24 08:01:10 2023
From: Jonathan.Cameron at Huawei.com (Jonathan Cameron)
Date: Thu, 24 Aug 2023 16:01:10 +0100
Subject: [PATCH] dt-bindings: Drop remaining unneeded quotes
In-Reply-To: <20230823183749.2609013-1-robh@kernel.org>
References: <20230823183749.2609013-1-robh@kernel.org>
Message-ID: <20230824160110.00002272@Huawei.com>

On Wed, 23 Aug 2023 13:28:47 -0500
Rob Herring <robh at kernel.org> wrote:

> Cleanup bindings dropping the last remaining unneeded quotes. With this,
> the check for this can be enabled in yamllint.
> 
> Signed-off-by: Rob Herring <robh at kernel.org>
For IIO

Acked-by: Jonathan Cameron <Jonathan.Cameron at huawei.com>

> ---
>  .../bindings/arm/arm,embedded-trace-extension.yaml   |  4 ++--
>  .../bindings/arm/arm,trace-buffer-extension.yaml     |  7 ++++---
>  .../devicetree/bindings/arm/arm,vexpress-juno.yaml   |  2 +-
>  .../devicetree/bindings/arm/aspeed/aspeed,sbc.yaml   |  4 ++--
>  .../arm/firmware/tlm,trusted-foundations.yaml        |  4 ++--
>  .../bindings/arm/mstar/mstar,l3bridge.yaml           |  4 ++--
>  .../devicetree/bindings/arm/mstar/mstar,smpctrl.yaml |  4 ++--
>  .../devicetree/bindings/arm/stm32/st,mlahb.yaml      |  4 ++--
>  .../bindings/arm/stm32/st,stm32-syscon.yaml          |  4 ++--
>  .../devicetree/bindings/connector/usb-connector.yaml |  4 ++--
>  Documentation/devicetree/bindings/eeprom/at24.yaml   |  4 ++--
>  Documentation/devicetree/bindings/eeprom/at25.yaml   |  4 ++--
>  .../intel,ixp4xx-network-processing-engine.yaml      |  4 ++--
>  .../bindings/gpio/x-powers,axp209-gpio.yaml          |  4 ++--
>  .../bindings/gpio/xlnx,zynqmp-gpio-modepin.yaml      |  4 ++--
>  .../devicetree/bindings/gpio/xylon,logicvc-gpio.yaml |  4 ++--
>  .../devicetree/bindings/hwmon/iio-hwmon.yaml         |  4 ++--
>  .../bindings/hwmon/starfive,jh71x0-temp.yaml         |  8 ++++----
>  .../devicetree/bindings/i3c/mipi-i3c-hci.yaml        |  4 ++--
>  .../devicetree/bindings/iio/accel/fsl,mma7455.yaml   |  4 ++--
>  .../bindings/iio/adc/atmel,sama9260-adc.yaml         |  4 ++--
>  .../bindings/ipmi/aspeed,ast2400-kcs-bmc.yaml        |  8 ++++----
>  .../devicetree/bindings/ipmi/ipmi-ipmb.yaml          |  2 +-
>  .../devicetree/bindings/ipmi/ipmi-smic.yaml          |  2 +-
>  .../bindings/media/qcom,msm8916-venus.yaml           |  4 ++--
>  .../bindings/mips/loongson/ls2k-reset.yaml           |  4 ++--
>  .../bindings/mips/loongson/rs780e-acpi.yaml          |  4 ++--
>  .../misc/intel,ixp4xx-ahb-queue-manager.yaml         |  4 ++--
>  .../devicetree/bindings/mmc/marvell,xenon-sdhci.yaml |  4 ++--
>  .../bindings/mtd/microchip,mchp48l640.yaml           |  4 ++--
>  .../devicetree/bindings/soc/aspeed/uart-routing.yaml |  4 ++--
>  .../bindings/soc/intel/intel,hps-copy-engine.yaml    |  4 ++--
>  .../bindings/soc/litex/litex,soc-controller.yaml     |  4 ++--
>  .../bindings/soc/renesas/renesas,rzg2l-sysc.yaml     |  4 ++--
>  .../devicetree/bindings/soc/ti/k3-ringacc.yaml       |  4 ++--
>  .../devicetree/bindings/sound/dialog,da7219.yaml     |  4 ++--
>  .../bindings/sound/nvidia,tegra-audio-max9808x.yaml  | 12 ++++++------
>  .../bindings/sound/nvidia,tegra-audio-rt5631.yaml    |  8 ++++----
>  .../devicetree/bindings/ufs/ufs-common.yaml          |  2 +-
>  .../bindings/watchdog/toshiba,visconti-wdt.yaml      |  4 ++--
>  40 files changed, 88 insertions(+), 87 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/arm/arm,embedded-trace-extension.yaml b/Documentation/devicetree/bindings/arm/arm,embedded-trace-extension.yaml
> index 108460627d9a..a477a810f9e9 100644
> --- a/Documentation/devicetree/bindings/arm/arm,embedded-trace-extension.yaml
> +++ b/Documentation/devicetree/bindings/arm/arm,embedded-trace-extension.yaml
> @@ -2,8 +2,8 @@
>  # Copyright 2021, Arm Ltd
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/arm/arm,embedded-trace-extension.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/arm/arm,embedded-trace-extension.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: ARM Embedded Trace Extensions
>  
> diff --git a/Documentation/devicetree/bindings/arm/arm,trace-buffer-extension.yaml b/Documentation/devicetree/bindings/arm/arm,trace-buffer-extension.yaml
> index b1322658063a..8c27e510cb71 100644
> --- a/Documentation/devicetree/bindings/arm/arm,trace-buffer-extension.yaml
> +++ b/Documentation/devicetree/bindings/arm/arm,trace-buffer-extension.yaml
> @@ -2,8 +2,8 @@
>  # Copyright 2021, Arm Ltd
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/arm/arm,trace-buffer-extension.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/arm/arm,trace-buffer-extension.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: ARM Trace Buffer Extensions
>  
> @@ -19,7 +19,8 @@ description: |
>  
>  properties:
>    $nodename:
> -    const: "trbe"
> +    const: trbe
> +
>    compatible:
>      items:
>        - const: arm,trace-buffer-extension
> diff --git a/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml b/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml
> index cdd65881fcdd..8dd6b6446394 100644
> --- a/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml
> +++ b/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml
> @@ -143,7 +143,7 @@ patternProperties:
>        "simple-bus". If the compatible is placed in the "motherboard-bus" node,
>        it is stricter and always has two compatibles.
>      type: object
> -    $ref: '/schemas/simple-bus.yaml'
> +    $ref: /schemas/simple-bus.yaml
>      unevaluatedProperties: false
>  
>      properties:
> diff --git a/Documentation/devicetree/bindings/arm/aspeed/aspeed,sbc.yaml b/Documentation/devicetree/bindings/arm/aspeed/aspeed,sbc.yaml
> index c72aab706484..b8c5cacb09bd 100644
> --- a/Documentation/devicetree/bindings/arm/aspeed/aspeed,sbc.yaml
> +++ b/Documentation/devicetree/bindings/arm/aspeed/aspeed,sbc.yaml
> @@ -2,8 +2,8 @@
>  # Copyright 2021 Joel Stanley, IBM Corp.
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/arm/aspeed/aspeed,sbc.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/arm/aspeed/aspeed,sbc.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: ASPEED Secure Boot Controller
>  
> diff --git a/Documentation/devicetree/bindings/arm/firmware/tlm,trusted-foundations.yaml b/Documentation/devicetree/bindings/arm/firmware/tlm,trusted-foundations.yaml
> index 9d1857c0aa07..e3980b659f63 100644
> --- a/Documentation/devicetree/bindings/arm/firmware/tlm,trusted-foundations.yaml
> +++ b/Documentation/devicetree/bindings/arm/firmware/tlm,trusted-foundations.yaml
> @@ -1,8 +1,8 @@
>  # SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/arm/firmware/tlm,trusted-foundations.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/arm/firmware/tlm,trusted-foundations.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: Trusted Foundations
>  
> diff --git a/Documentation/devicetree/bindings/arm/mstar/mstar,l3bridge.yaml b/Documentation/devicetree/bindings/arm/mstar/mstar,l3bridge.yaml
> index 6816bd68f9cf..a8ac4a2d672d 100644
> --- a/Documentation/devicetree/bindings/arm/mstar/mstar,l3bridge.yaml
> +++ b/Documentation/devicetree/bindings/arm/mstar/mstar,l3bridge.yaml
> @@ -2,8 +2,8 @@
>  # Copyright 2020 thingy.jp.
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/arm/mstar/mstar,l3bridge.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/arm/mstar/mstar,l3bridge.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: MStar/SigmaStar Armv7 SoC l3bridge
>  
> diff --git a/Documentation/devicetree/bindings/arm/mstar/mstar,smpctrl.yaml b/Documentation/devicetree/bindings/arm/mstar/mstar,smpctrl.yaml
> index 599c65980f5d..5739848000b1 100644
> --- a/Documentation/devicetree/bindings/arm/mstar/mstar,smpctrl.yaml
> +++ b/Documentation/devicetree/bindings/arm/mstar/mstar,smpctrl.yaml
> @@ -2,8 +2,8 @@
>  # Copyright 2020 thingy.jp.
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/arm/mstar/mstar,smpctrl.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/arm/mstar/mstar,smpctrl.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: MStar/SigmaStar Armv7 SoC SMP control registers
>  
> diff --git a/Documentation/devicetree/bindings/arm/stm32/st,mlahb.yaml b/Documentation/devicetree/bindings/arm/stm32/st,mlahb.yaml
> index 2297ad3f4774..d2dce238ff5d 100644
> --- a/Documentation/devicetree/bindings/arm/stm32/st,mlahb.yaml
> +++ b/Documentation/devicetree/bindings/arm/stm32/st,mlahb.yaml
> @@ -1,8 +1,8 @@
>  # SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/arm/stm32/st,mlahb.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/arm/stm32/st,mlahb.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: STMicroelectronics STM32 ML-AHB interconnect
>  
> diff --git a/Documentation/devicetree/bindings/arm/stm32/st,stm32-syscon.yaml b/Documentation/devicetree/bindings/arm/stm32/st,stm32-syscon.yaml
> index b63ff591ef8f..d083d8ad48b7 100644
> --- a/Documentation/devicetree/bindings/arm/stm32/st,stm32-syscon.yaml
> +++ b/Documentation/devicetree/bindings/arm/stm32/st,stm32-syscon.yaml
> @@ -1,8 +1,8 @@
>  # SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/arm/stm32/st,stm32-syscon.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/arm/stm32/st,stm32-syscon.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: STMicroelectronics STM32 Platforms System Controller
>  
> diff --git a/Documentation/devicetree/bindings/connector/usb-connector.yaml b/Documentation/devicetree/bindings/connector/usb-connector.yaml
> index 3ecb51f55a71..7c8a3e8430d3 100644
> --- a/Documentation/devicetree/bindings/connector/usb-connector.yaml
> +++ b/Documentation/devicetree/bindings/connector/usb-connector.yaml
> @@ -232,8 +232,8 @@ properties:
>      type: boolean
>  
>  dependencies:
> -  sink-vdos-v1: [ 'sink-vdos' ]
> -  sink-vdos: [ 'sink-vdos-v1' ]
> +  sink-vdos-v1: [ sink-vdos ]
> +  sink-vdos: [ sink-vdos-v1 ]
>  
>  required:
>    - compatible
> diff --git a/Documentation/devicetree/bindings/eeprom/at24.yaml b/Documentation/devicetree/bindings/eeprom/at24.yaml
> index 84af0d5f52aa..b1e5dddf9bf4 100644
> --- a/Documentation/devicetree/bindings/eeprom/at24.yaml
> +++ b/Documentation/devicetree/bindings/eeprom/at24.yaml
> @@ -2,8 +2,8 @@
>  # Copyright 2019 BayLibre SAS
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/eeprom/at24.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/eeprom/at24.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: I2C EEPROMs compatible with Atmel's AT24
>  
> diff --git a/Documentation/devicetree/bindings/eeprom/at25.yaml b/Documentation/devicetree/bindings/eeprom/at25.yaml
> index 0e31bb36ebb1..1715b0c9feea 100644
> --- a/Documentation/devicetree/bindings/eeprom/at25.yaml
> +++ b/Documentation/devicetree/bindings/eeprom/at25.yaml
> @@ -1,8 +1,8 @@
>  # SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/eeprom/at25.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/eeprom/at25.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: SPI EEPROMs or FRAMs compatible with Atmel's AT25
>  
> diff --git a/Documentation/devicetree/bindings/firmware/intel,ixp4xx-network-processing-engine.yaml b/Documentation/devicetree/bindings/firmware/intel,ixp4xx-network-processing-engine.yaml
> index 9a785bbaafb7..e6bed7d93e2d 100644
> --- a/Documentation/devicetree/bindings/firmware/intel,ixp4xx-network-processing-engine.yaml
> +++ b/Documentation/devicetree/bindings/firmware/intel,ixp4xx-network-processing-engine.yaml
> @@ -2,8 +2,8 @@
>  # Copyright 2019 Linaro Ltd.
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/firmware/intel,ixp4xx-network-processing-engine.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/firmware/intel,ixp4xx-network-processing-engine.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: Intel IXP4xx Network Processing Engine
>  
> diff --git a/Documentation/devicetree/bindings/gpio/x-powers,axp209-gpio.yaml b/Documentation/devicetree/bindings/gpio/x-powers,axp209-gpio.yaml
> index 1638cfe90f1c..5eeb29bcdd21 100644
> --- a/Documentation/devicetree/bindings/gpio/x-powers,axp209-gpio.yaml
> +++ b/Documentation/devicetree/bindings/gpio/x-powers,axp209-gpio.yaml
> @@ -1,8 +1,8 @@
>  # SPDX-License-Identifier: GPL-2.0
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/gpio/x-powers,axp209-gpio.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/gpio/x-powers,axp209-gpio.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: X-Powers AXP209 GPIO
>  
> diff --git a/Documentation/devicetree/bindings/gpio/xlnx,zynqmp-gpio-modepin.yaml b/Documentation/devicetree/bindings/gpio/xlnx,zynqmp-gpio-modepin.yaml
> index 18e61aff2185..56143f1fe84a 100644
> --- a/Documentation/devicetree/bindings/gpio/xlnx,zynqmp-gpio-modepin.yaml
> +++ b/Documentation/devicetree/bindings/gpio/xlnx,zynqmp-gpio-modepin.yaml
> @@ -1,8 +1,8 @@
>  # SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/gpio/xlnx,zynqmp-gpio-modepin.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/gpio/xlnx,zynqmp-gpio-modepin.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: ZynqMP Mode Pin GPIO controller
>  
> diff --git a/Documentation/devicetree/bindings/gpio/xylon,logicvc-gpio.yaml b/Documentation/devicetree/bindings/gpio/xylon,logicvc-gpio.yaml
> index a36aec27069c..59c79a6943ec 100644
> --- a/Documentation/devicetree/bindings/gpio/xylon,logicvc-gpio.yaml
> +++ b/Documentation/devicetree/bindings/gpio/xylon,logicvc-gpio.yaml
> @@ -2,8 +2,8 @@
>  # Copyright 2019 Bootlin
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/gpio/xylon,logicvc-gpio.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/gpio/xylon,logicvc-gpio.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: Xylon LogiCVC GPIO controller
>  
> diff --git a/Documentation/devicetree/bindings/hwmon/iio-hwmon.yaml b/Documentation/devicetree/bindings/hwmon/iio-hwmon.yaml
> index c54b5986b365..e5b24782f448 100644
> --- a/Documentation/devicetree/bindings/hwmon/iio-hwmon.yaml
> +++ b/Documentation/devicetree/bindings/hwmon/iio-hwmon.yaml
> @@ -1,8 +1,8 @@
>  # SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/hwmon/iio-hwmon.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/hwmon/iio-hwmon.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: ADC-attached Hardware Sensor
>  
> diff --git a/Documentation/devicetree/bindings/hwmon/starfive,jh71x0-temp.yaml b/Documentation/devicetree/bindings/hwmon/starfive,jh71x0-temp.yaml
> index f5b34528928d..733cba780186 100644
> --- a/Documentation/devicetree/bindings/hwmon/starfive,jh71x0-temp.yaml
> +++ b/Documentation/devicetree/bindings/hwmon/starfive,jh71x0-temp.yaml
> @@ -27,8 +27,8 @@ properties:
>  
>    clock-names:
>      items:
> -      - const: "sense"
> -      - const: "bus"
> +      - const: sense
> +      - const: bus
>  
>    '#thermal-sensor-cells':
>      const: 0
> @@ -39,8 +39,8 @@ properties:
>  
>    reset-names:
>      items:
> -      - const: "sense"
> -      - const: "bus"
> +      - const: sense
> +      - const: bus
>  
>  required:
>    - compatible
> diff --git a/Documentation/devicetree/bindings/i3c/mipi-i3c-hci.yaml b/Documentation/devicetree/bindings/i3c/mipi-i3c-hci.yaml
> index c002afdbfc7c..5dda8cb44cdb 100644
> --- a/Documentation/devicetree/bindings/i3c/mipi-i3c-hci.yaml
> +++ b/Documentation/devicetree/bindings/i3c/mipi-i3c-hci.yaml
> @@ -1,8 +1,8 @@
>  # SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/i3c/mipi-i3c-hci.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/i3c/mipi-i3c-hci.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: MIPI I3C HCI
>  
> diff --git a/Documentation/devicetree/bindings/iio/accel/fsl,mma7455.yaml b/Documentation/devicetree/bindings/iio/accel/fsl,mma7455.yaml
> index c8659c5eba2a..cb31e75ba680 100644
> --- a/Documentation/devicetree/bindings/iio/accel/fsl,mma7455.yaml
> +++ b/Documentation/devicetree/bindings/iio/accel/fsl,mma7455.yaml
> @@ -36,8 +36,8 @@ properties:
>      maxItems: 2
>      items:
>        enum:
> -        - "INT1"
> -        - "INT2"
> +        - INT1
> +        - INT2
>  
>  required:
>    - compatible
> diff --git a/Documentation/devicetree/bindings/iio/adc/atmel,sama9260-adc.yaml b/Documentation/devicetree/bindings/iio/adc/atmel,sama9260-adc.yaml
> index e6a1f915b542..1f30a8569187 100644
> --- a/Documentation/devicetree/bindings/iio/adc/atmel,sama9260-adc.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/atmel,sama9260-adc.yaml
> @@ -56,8 +56,8 @@ properties:
>        String corresponding to an identifier from atmel,adc-res-names property.
>        If not specified, the highest resolution will be used.
>      enum:
> -      - "lowres"
> -      - "highres"
> +      - lowres
> +      - highres
>  
>    atmel,adc-sleep-mode:
>      $ref: /schemas/types.yaml#/definitions/flag
> diff --git a/Documentation/devicetree/bindings/ipmi/aspeed,ast2400-kcs-bmc.yaml b/Documentation/devicetree/bindings/ipmi/aspeed,ast2400-kcs-bmc.yaml
> index 4ff6fabfcb30..129e32c4c774 100644
> --- a/Documentation/devicetree/bindings/ipmi/aspeed,ast2400-kcs-bmc.yaml
> +++ b/Documentation/devicetree/bindings/ipmi/aspeed,ast2400-kcs-bmc.yaml
> @@ -41,7 +41,7 @@ properties:
>        - description: STR register
>  
>    aspeed,lpc-io-reg:
> -    $ref: '/schemas/types.yaml#/definitions/uint32-array'
> +    $ref: /schemas/types.yaml#/definitions/uint32-array
>      minItems: 1
>      maxItems: 2
>      description: |
> @@ -50,7 +50,7 @@ properties:
>        status address may be optionally provided.
>  
>    aspeed,lpc-interrupts:
> -    $ref: "/schemas/types.yaml#/definitions/uint32-array"
> +    $ref: /schemas/types.yaml#/definitions/uint32-array
>      minItems: 2
>      maxItems: 2
>      description: |
> @@ -63,12 +63,12 @@ properties:
>  
>    kcs_chan:
>      deprecated: true
> -    $ref: '/schemas/types.yaml#/definitions/uint32'
> +    $ref: /schemas/types.yaml#/definitions/uint32
>      description: The LPC channel number in the controller
>  
>    kcs_addr:
>      deprecated: true
> -    $ref: '/schemas/types.yaml#/definitions/uint32'
> +    $ref: /schemas/types.yaml#/definitions/uint32
>      description: The host CPU IO map address
>  
>  required:
> diff --git a/Documentation/devicetree/bindings/ipmi/ipmi-ipmb.yaml b/Documentation/devicetree/bindings/ipmi/ipmi-ipmb.yaml
> index 3f25cdb4e99b..52647bff31af 100644
> --- a/Documentation/devicetree/bindings/ipmi/ipmi-ipmb.yaml
> +++ b/Documentation/devicetree/bindings/ipmi/ipmi-ipmb.yaml
> @@ -18,7 +18,7 @@ properties:
>  
>    device_type:
>      items:
> -      - const: "ipmi"
> +      - const: ipmi
>  
>    reg:
>      maxItems: 1
> diff --git a/Documentation/devicetree/bindings/ipmi/ipmi-smic.yaml b/Documentation/devicetree/bindings/ipmi/ipmi-smic.yaml
> index c1b4bf95ef99..4bffa3d86128 100644
> --- a/Documentation/devicetree/bindings/ipmi/ipmi-smic.yaml
> +++ b/Documentation/devicetree/bindings/ipmi/ipmi-smic.yaml
> @@ -20,7 +20,7 @@ properties:
>  
>    device_type:
>      items:
> -      - const: "ipmi"
> +      - const: ipmi
>  
>    reg:
>      maxItems: 1
> diff --git a/Documentation/devicetree/bindings/media/qcom,msm8916-venus.yaml b/Documentation/devicetree/bindings/media/qcom,msm8916-venus.yaml
> index 2350bf4b370e..9410f13ca97c 100644
> --- a/Documentation/devicetree/bindings/media/qcom,msm8916-venus.yaml
> +++ b/Documentation/devicetree/bindings/media/qcom,msm8916-venus.yaml
> @@ -40,7 +40,7 @@ properties:
>  
>      properties:
>        compatible:
> -        const: "venus-decoder"
> +        const: venus-decoder
>  
>      required:
>        - compatible
> @@ -52,7 +52,7 @@ properties:
>  
>      properties:
>        compatible:
> -        const: "venus-encoder"
> +        const: venus-encoder
>  
>      required:
>        - compatible
> diff --git a/Documentation/devicetree/bindings/mips/loongson/ls2k-reset.yaml b/Documentation/devicetree/bindings/mips/loongson/ls2k-reset.yaml
> index 20b5836efd90..358ac8cd4d1d 100644
> --- a/Documentation/devicetree/bindings/mips/loongson/ls2k-reset.yaml
> +++ b/Documentation/devicetree/bindings/mips/loongson/ls2k-reset.yaml
> @@ -1,8 +1,8 @@
>  # SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/mips/loongson/ls2k-reset.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/mips/loongson/ls2k-reset.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: Loongson 2K1000 PM Controller
>  
> diff --git a/Documentation/devicetree/bindings/mips/loongson/rs780e-acpi.yaml b/Documentation/devicetree/bindings/mips/loongson/rs780e-acpi.yaml
> index 7c0f9022202c..3e3a3705e879 100644
> --- a/Documentation/devicetree/bindings/mips/loongson/rs780e-acpi.yaml
> +++ b/Documentation/devicetree/bindings/mips/loongson/rs780e-acpi.yaml
> @@ -1,8 +1,8 @@
>  # SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/mips/loongson/rs780e-acpi.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/mips/loongson/rs780e-acpi.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: Loongson RS780E PCH ACPI Controller
>  
> diff --git a/Documentation/devicetree/bindings/misc/intel,ixp4xx-ahb-queue-manager.yaml b/Documentation/devicetree/bindings/misc/intel,ixp4xx-ahb-queue-manager.yaml
> index 38ab0499102d..36a9dbdf3f03 100644
> --- a/Documentation/devicetree/bindings/misc/intel,ixp4xx-ahb-queue-manager.yaml
> +++ b/Documentation/devicetree/bindings/misc/intel,ixp4xx-ahb-queue-manager.yaml
> @@ -2,8 +2,8 @@
>  # Copyright 2019 Linaro Ltd.
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/misc/intel,ixp4xx-ahb-queue-manager.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/misc/intel,ixp4xx-ahb-queue-manager.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: Intel IXP4xx AHB Queue Manager
>  
> diff --git a/Documentation/devicetree/bindings/mmc/marvell,xenon-sdhci.yaml b/Documentation/devicetree/bindings/mmc/marvell,xenon-sdhci.yaml
> index 3ee758886558..3a8e74894ae0 100644
> --- a/Documentation/devicetree/bindings/mmc/marvell,xenon-sdhci.yaml
> +++ b/Documentation/devicetree/bindings/mmc/marvell,xenon-sdhci.yaml
> @@ -71,8 +71,8 @@ properties:
>    marvell,xenon-phy-type:
>      $ref: /schemas/types.yaml#/definitions/string
>      enum:
> -      - "emmc 5.1 phy"
> -      - "emmc 5.0 phy"
> +      - emmc 5.1 phy
> +      - emmc 5.0 phy
>      description: |
>        Xenon support multiple types of PHYs. To select eMMC 5.1 PHY, set:
>        marvell,xenon-phy-type = "emmc 5.1 phy" eMMC 5.1 PHY is the default
> diff --git a/Documentation/devicetree/bindings/mtd/microchip,mchp48l640.yaml b/Documentation/devicetree/bindings/mtd/microchip,mchp48l640.yaml
> index 00882892f47e..0ff32bd00bf6 100644
> --- a/Documentation/devicetree/bindings/mtd/microchip,mchp48l640.yaml
> +++ b/Documentation/devicetree/bindings/mtd/microchip,mchp48l640.yaml
> @@ -1,8 +1,8 @@
>  # SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/mtd/microchip,mchp48l640.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/mtd/microchip,mchp48l640.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: Microchip 48l640 (and similar) serial EERAM
>  
> diff --git a/Documentation/devicetree/bindings/soc/aspeed/uart-routing.yaml b/Documentation/devicetree/bindings/soc/aspeed/uart-routing.yaml
> index 6876407124dc..51aaf34acb32 100644
> --- a/Documentation/devicetree/bindings/soc/aspeed/uart-routing.yaml
> +++ b/Documentation/devicetree/bindings/soc/aspeed/uart-routing.yaml
> @@ -3,8 +3,8 @@
>  # # Copyright (c) 2021 Aspeed Technology Inc.
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/soc/aspeed/uart-routing.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/soc/aspeed/uart-routing.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: Aspeed UART Routing Controller
>  
> diff --git a/Documentation/devicetree/bindings/soc/intel/intel,hps-copy-engine.yaml b/Documentation/devicetree/bindings/soc/intel/intel,hps-copy-engine.yaml
> index 8634865015cd..ceb81646fe75 100644
> --- a/Documentation/devicetree/bindings/soc/intel/intel,hps-copy-engine.yaml
> +++ b/Documentation/devicetree/bindings/soc/intel/intel,hps-copy-engine.yaml
> @@ -2,8 +2,8 @@
>  # Copyright (C) 2022, Intel Corporation
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/soc/intel/intel,hps-copy-engine.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/soc/intel/intel,hps-copy-engine.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: Intel HPS Copy Engine
>  
> diff --git a/Documentation/devicetree/bindings/soc/litex/litex,soc-controller.yaml b/Documentation/devicetree/bindings/soc/litex/litex,soc-controller.yaml
> index ecae9fa8561b..a64406ca17b5 100644
> --- a/Documentation/devicetree/bindings/soc/litex/litex,soc-controller.yaml
> +++ b/Documentation/devicetree/bindings/soc/litex/litex,soc-controller.yaml
> @@ -2,8 +2,8 @@
>  # Copyright 2020 Antmicro <www.antmicro.com>
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/soc/litex/litex,soc-controller.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/soc/litex/litex,soc-controller.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: LiteX SoC Controller driver
>  
> diff --git a/Documentation/devicetree/bindings/soc/renesas/renesas,rzg2l-sysc.yaml b/Documentation/devicetree/bindings/soc/renesas/renesas,rzg2l-sysc.yaml
> index 398663d21ab1..e52e176d8cb3 100644
> --- a/Documentation/devicetree/bindings/soc/renesas/renesas,rzg2l-sysc.yaml
> +++ b/Documentation/devicetree/bindings/soc/renesas/renesas,rzg2l-sysc.yaml
> @@ -1,8 +1,8 @@
>  # SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/soc/renesas/renesas,rzg2l-sysc.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/soc/renesas/renesas,rzg2l-sysc.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: Renesas RZ/{G2L,V2L} System Controller (SYSC)
>  
> diff --git a/Documentation/devicetree/bindings/soc/ti/k3-ringacc.yaml b/Documentation/devicetree/bindings/soc/ti/k3-ringacc.yaml
> index 22cf9002fee7..4ac00716885e 100644
> --- a/Documentation/devicetree/bindings/soc/ti/k3-ringacc.yaml
> +++ b/Documentation/devicetree/bindings/soc/ti/k3-ringacc.yaml
> @@ -2,8 +2,8 @@
>  # Copyright (C) 2020 Texas Instruments Incorporated - http://www.ti.com/
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/soc/ti/k3-ringacc.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/soc/ti/k3-ringacc.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: Texas Instruments K3 NavigatorSS Ring Accelerator
>  
> diff --git a/Documentation/devicetree/bindings/sound/dialog,da7219.yaml b/Documentation/devicetree/bindings/sound/dialog,da7219.yaml
> index 2d01956cefbb..19137abdba3e 100644
> --- a/Documentation/devicetree/bindings/sound/dialog,da7219.yaml
> +++ b/Documentation/devicetree/bindings/sound/dialog,da7219.yaml
> @@ -74,7 +74,7 @@ properties:
>      $ref: /schemas/types.yaml#/definitions/uint32
>  
>    dlg,mic-amp-in-sel:
> -    enum: ["diff", "se_p", "se_n"]
> +    enum: [diff, se_p, se_n]
>      description:
>        Mic input source type.
>  
> @@ -124,7 +124,7 @@ properties:
>          $ref: /schemas/types.yaml#/definitions/uint32
>  
>        dlg,jack-ins-det-pty:
> -        enum: ["low", "high"]
> +        enum: [low, high]
>          description:
>            Polarity for jack insertion detection.
>          $ref: /schemas/types.yaml#/definitions/string
> diff --git a/Documentation/devicetree/bindings/sound/nvidia,tegra-audio-max9808x.yaml b/Documentation/devicetree/bindings/sound/nvidia,tegra-audio-max9808x.yaml
> index fc89dbd6bf24..c29d7942915c 100644
> --- a/Documentation/devicetree/bindings/sound/nvidia,tegra-audio-max9808x.yaml
> +++ b/Documentation/devicetree/bindings/sound/nvidia,tegra-audio-max9808x.yaml
> @@ -35,12 +35,12 @@ properties:
>      items:
>        enum:
>          # Board Connectors
> -        - "Int Spk"
> -        - "Headphone Jack"
> -        - "Earpiece"
> -        - "Headset Mic"
> -        - "Internal Mic 1"
> -        - "Internal Mic 2"
> +        - Int Spk
> +        - Headphone Jack
> +        - Earpiece
> +        - Headset Mic
> +        - Internal Mic 1
> +        - Internal Mic 2
>  
>          # CODEC Pins
>          - HPL
> diff --git a/Documentation/devicetree/bindings/sound/nvidia,tegra-audio-rt5631.yaml b/Documentation/devicetree/bindings/sound/nvidia,tegra-audio-rt5631.yaml
> index a04487002e88..0c8067c3b056 100644
> --- a/Documentation/devicetree/bindings/sound/nvidia,tegra-audio-rt5631.yaml
> +++ b/Documentation/devicetree/bindings/sound/nvidia,tegra-audio-rt5631.yaml
> @@ -31,10 +31,10 @@ properties:
>      items:
>        enum:
>          # Board Connectors
> -        - "Int Spk"
> -        - "Headphone Jack"
> -        - "Mic Jack"
> -        - "Int Mic"
> +        - Int Spk
> +        - Headphone Jack
> +        - Mic Jack
> +        - Int Mic
>  
>          # CODEC Pins
>          - MIC1
> diff --git a/Documentation/devicetree/bindings/ufs/ufs-common.yaml b/Documentation/devicetree/bindings/ufs/ufs-common.yaml
> index 47a4e9e1a775..bbaee4f5f7b2 100644
> --- a/Documentation/devicetree/bindings/ufs/ufs-common.yaml
> +++ b/Documentation/devicetree/bindings/ufs/ufs-common.yaml
> @@ -74,7 +74,7 @@ properties:
>        Specifies max. load that can be drawn from VCCQ2 supply.
>  
>  dependencies:
> -  freq-table-hz: [ 'clocks' ]
> +  freq-table-hz: [ clocks ]
>  
>  required:
>    - interrupts
> diff --git a/Documentation/devicetree/bindings/watchdog/toshiba,visconti-wdt.yaml b/Documentation/devicetree/bindings/watchdog/toshiba,visconti-wdt.yaml
> index 51d03d5b08ad..3e9fd49d935e 100644
> --- a/Documentation/devicetree/bindings/watchdog/toshiba,visconti-wdt.yaml
> +++ b/Documentation/devicetree/bindings/watchdog/toshiba,visconti-wdt.yaml
> @@ -2,8 +2,8 @@
>  # Copyright 2020 Toshiba Electronic Devices & Storage Corporation
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/watchdog/toshiba,visconti-wdt.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/watchdog/toshiba,visconti-wdt.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: Toshiba Visconti SoCs PIUWDT Watchdog timer
>  



From alexandre.belloni at bootlin.com  Sun Aug 27 15:45:34 2023
From: alexandre.belloni at bootlin.com (Alexandre Belloni)
Date: Mon, 28 Aug 2023 00:45:34 +0200
Subject: [PATCH] dt-bindings: i3c: Fix description for assigned-address
In-Reply-To: <20230822051209.2837818-1-aniketmaurya@google.com>
References: <20230822051209.2837818-1-aniketmaurya@google.com>
Message-ID: <169317630418.562439.4628786119246659411.b4-ty@bootlin.com>


On Tue, 22 Aug 2023 05:12:09 +0000, Aniket wrote:
> assigned-address is a valid property for I3C device with or without
> static address. Fix the description and an example for the same.
> 
> 

Applied, thanks!

[1/1] dt-bindings: i3c: Fix description for assigned-address
      commit: 32e6b31a6a80ba9493fad802d68d41f4d63230cc

Best regards,

-- 
Alexandre Belloni, co-owner and COO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com


From alexandre.belloni at bootlin.com  Sun Aug 27 15:45:46 2023
From: alexandre.belloni at bootlin.com (Alexandre Belloni)
Date: Mon, 28 Aug 2023 00:45:46 +0200
Subject: [PATCH] i3c: master: svc: Describe member 'saved_regs'
In-Reply-To: <20230817101853.16805-1-miquel.raynal@bootlin.com>
References: <20230817101853.16805-1-miquel.raynal@bootlin.com>
Message-ID: <169317630417.562439.10455463939243073936.b4-ty@bootlin.com>


On Thu, 17 Aug 2023 12:18:53 +0200, Miquel Raynal wrote:
> The 'saved_regs' member of the 'svc_i3c_master'	structure is not
> described in the kernel doc, which produces the following warning:
> 
>     Function parameter or member 'saved_regs' not described in 'svc_i3c_master'
> 
> Add the missing line in the kernel documentation of the parent
> structure.
> 
> [...]

Applied, thanks!

[1/1] i3c: master: svc: Describe member 'saved_regs'
      commit: 5496eac6ad7428fa06811a8c34b3a15beb93b86d

Best regards,

-- 
Alexandre Belloni, co-owner and COO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com


From alexandre.belloni at bootlin.com  Sun Aug 27 15:46:45 2023
From: alexandre.belloni at bootlin.com (Alexandre Belloni)
Date: Mon, 28 Aug 2023 00:46:45 +0200
Subject: [PATCH] i3c: master: Fix SETDASA process
In-Reply-To: <20230822051938.2852567-1-aniketmaurya@google.com>
References: <20230822051938.2852567-1-aniketmaurya@google.com>
Message-ID: <169317639314.563617.17285767371678104909.b4-ty@bootlin.com>


On Tue, 22 Aug 2023 05:19:38 +0000, Aniket wrote:
> When a I3C DT node has a static_addr and an init_dyn_addr,the
> init_dyn_addr is reserved in i3c_master_bus_init() and then
> the static_addr is reserved in i3c_master_early_i3c_dev_add().
> But if the dynamic address is same as static then above
> procedure would fail.
> Add a check to pass i3c_bus_get_addr_slot_status() when static
> and dynamic address are equal.
> 
> [...]

Applied, thanks!

[1/1] i3c: master: Fix SETDASA process
      commit: c6a7550458922181bb922863d5f5feaf3c453fe5

Best regards,

-- 
Alexandre Belloni, co-owner and COO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com


From joshua.yeong at starfivetech.com  Mon Aug 28 08:14:58 2023
From: joshua.yeong at starfivetech.com (Joshua Yeong)
Date: Mon, 28 Aug 2023 23:14:58 +0800
Subject: [PATCH] i3c: master: cdns: Fix reading status register
Message-ID: <20230828151458.15085-1-joshua.yeong@starfivetech.com>

IBIR_DEPTH and CMDR_DEPTH should read from status0 instead of status1

Signed-off-by: Joshua Yeong <joshua.yeong at starfivetech.com>
---
 drivers/i3c/master/i3c-master-cdns.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/i3c/master/i3c-master-cdns.c b/drivers/i3c/master/i3c-master-cdns.c
index 5b37ffe5ad5b..4a49c75a9408 100644
--- a/drivers/i3c/master/i3c-master-cdns.c
+++ b/drivers/i3c/master/i3c-master-cdns.c
@@ -192,7 +192,7 @@
 #define SLV_STATUS1_HJ_DIS		BIT(18)
 #define SLV_STATUS1_MR_DIS		BIT(17)
 #define SLV_STATUS1_PROT_ERR		BIT(16)
-#define SLV_STATUS1_DA(x)		(((s) & GENMASK(15, 9)) >> 9)
+#define SLV_STATUS1_DA(s)		(((s) & GENMASK(15, 9)) >> 9)
 #define SLV_STATUS1_HAS_DA		BIT(8)
 #define SLV_STATUS1_DDR_RX_FULL		BIT(7)
 #define SLV_STATUS1_DDR_TX_FULL		BIT(6)
@@ -1624,13 +1624,13 @@ static int cdns_i3c_master_probe(struct platform_device *pdev)
 	/* Device ID0 is reserved to describe this master. */
 	master->maxdevs = CONF_STATUS0_DEVS_NUM(val);
 	master->free_rr_slots = GENMASK(master->maxdevs, 1);
+	master->caps.ibirfifodepth = CONF_STATUS0_IBIR_DEPTH(val);
+	master->caps.cmdrfifodepth = CONF_STATUS0_CMDR_DEPTH(val);
 
 	val = readl(master->regs + CONF_STATUS1);
 	master->caps.cmdfifodepth = CONF_STATUS1_CMD_DEPTH(val);
 	master->caps.rxfifodepth = CONF_STATUS1_RX_DEPTH(val);
 	master->caps.txfifodepth = CONF_STATUS1_TX_DEPTH(val);
-	master->caps.ibirfifodepth = CONF_STATUS0_IBIR_DEPTH(val);
-	master->caps.cmdrfifodepth = CONF_STATUS0_CMDR_DEPTH(val);
 
 	spin_lock_init(&master->ibi.lock);
 	master->ibi.num_slots = CONF_STATUS1_IBI_HW_RES(val);
-- 
2.25.1



From joshua.yeong at starfivetech.com  Mon Aug 28 08:20:35 2023
From: joshua.yeong at starfivetech.com (Joshua Yeong)
Date: Mon, 28 Aug 2023 23:20:35 +0800
Subject: [PATCH] i3c: Add fallback method for GETMXDS CCC
Message-ID: <20230828152035.15498-1-joshua.yeong@starfivetech.com>

Some I3C hardware will report error when incorrect length is received from device. GETMXDS CCC are availble in 2 formats; without turnaround time (format 1) and with turnaround time (format 2). There is no mechanics to determine which format is supported by device. In case sending GETMXDS CCC format 2 resulted failure, try sending GETMXDS CCC format 1 instead.

Signed-off-by: Joshua Yeong <joshua.yeong at starfivetech.com>
---
 drivers/i3c/master.c    | 33 ++++++++++++++++++++++++++++-----
 include/linux/i3c/ccc.h | 17 +++++++++++++++--
 2 files changed, 43 insertions(+), 7 deletions(-)

diff --git a/drivers/i3c/master.c b/drivers/i3c/master.c
index 351c81a929a6..8e6449ca724b 100644
--- a/drivers/i3c/master.c
+++ b/drivers/i3c/master.c
@@ -1076,10 +1076,37 @@ static int i3c_master_getmxds_locked(struct i3c_master_controller *master,
 				     struct i3c_device_info *info)
 {
 	struct i3c_ccc_getmxds *getmaxds;
+	struct i3c_ccc_getmxds_turnaround *getmaxds_ta;
 	struct i3c_ccc_cmd_dest dest;
 	struct i3c_ccc_cmd cmd;
 	int ret;
 
+	getmaxds_ta = i3c_ccc_cmd_dest_init(&dest, info->dyn_addr,
+					 sizeof(*getmaxds_ta));
+	if (!getmaxds_ta)
+		return -ENOMEM;
+
+	i3c_ccc_cmd_init(&cmd, true, I3C_CCC_GETMXDS, &dest, 1);
+	ret = i3c_master_send_ccc_cmd_locked(master, &cmd);
+	if (ret) {
+		goto alternative;
+	}
+
+	if (dest.payload.len != 2 && dest.payload.len != 5) {
+		ret = -EIO;
+		goto out;
+	}
+
+	info->max_read_ds = getmaxds_ta->maxrd;
+	info->max_write_ds = getmaxds_ta->maxwr;
+	if (dest.payload.len == 5)
+		info->max_read_turnaround = getmaxds_ta->maxrdturn[0] |
+					    ((u32)getmaxds_ta->maxrdturn[1] << 8) |
+					    ((u32)getmaxds_ta->maxrdturn[2] << 16);
+
+alternative:
+	i3c_ccc_cmd_dest_cleanup(&dest);
+
 	getmaxds = i3c_ccc_cmd_dest_init(&dest, info->dyn_addr,
 					 sizeof(*getmaxds));
 	if (!getmaxds)
@@ -1090,17 +1117,13 @@ static int i3c_master_getmxds_locked(struct i3c_master_controller *master,
 	if (ret)
 		goto out;
 
-	if (dest.payload.len != 2 && dest.payload.len != 5) {
+	if (dest.payload.len != 2) {
 		ret = -EIO;
 		goto out;
 	}
 
 	info->max_read_ds = getmaxds->maxrd;
 	info->max_write_ds = getmaxds->maxwr;
-	if (dest.payload.len == 5)
-		info->max_read_turnaround = getmaxds->maxrdturn[0] |
-					    ((u32)getmaxds->maxrdturn[1] << 8) |
-					    ((u32)getmaxds->maxrdturn[2] << 16);
 
 out:
 	i3c_ccc_cmd_dest_cleanup(&dest);
diff --git a/include/linux/i3c/ccc.h b/include/linux/i3c/ccc.h
index ad59a4ae60d1..50ed41d4d5a1 100644
--- a/include/linux/i3c/ccc.h
+++ b/include/linux/i3c/ccc.h
@@ -269,14 +269,27 @@ enum i3c_tsco {
 #define I3C_CCC_MAX_SDR_FSCL(x)		((x) & I3C_CCC_MAX_SDR_FSCL_MASK)
 
 /**
- * struct i3c_ccc_getmxds - payload passed to GETMXDS CCC
+ * struct i3c_ccc_getmxds - payload passed to GETMXDS CCC without turnaround
+ * (format 1)
+ *
+ * @maxwr: write limitations
+ * @maxrd: read limitations
+ */
+struct i3c_ccc_getmxds {
+	u8 maxwr;
+	u8 maxrd;
+} __packed;
+
+/**
+ * struct i3c_ccc_getmxds_ta - payload passed to GETMXDS CCC with turnaround
+ * (format 2)
  *
  * @maxwr: write limitations
  * @maxrd: read limitations
  * @maxrdturn: maximum read turn-around expressed micro-seconds and
  *	       little-endian formatted
  */
-struct i3c_ccc_getmxds {
+struct i3c_ccc_getmxds_turnaround {
 	u8 maxwr;
 	u8 maxrd;
 	u8 maxrdturn[3];
-- 
2.25.1



From miquel.raynal at bootlin.com  Tue Aug 29 02:16:11 2023
From: miquel.raynal at bootlin.com (Miquel Raynal)
Date: Tue, 29 Aug 2023 11:16:11 +0200
Subject: [PATCH 2/2] i3c: master: svc: fix probe failure when no i3c
 device exist
In-Reply-To: <20230828192502.3525418-2-Frank.Li@nxp.com>
References: <20230828192502.3525418-1-Frank.Li@nxp.com>
	<20230828192502.3525418-2-Frank.Li@nxp.com>
Message-ID: <20230829111611.12f91348@xps-13>

Hi Frank,

Frank.Li at nxp.com wrote on Mon, 28 Aug 2023 15:25:02 -0400:

> If there are not i3c device, all ccc command will get NACK. Set

		no					NACKed?

> i3c_ccc_cmd::err as I3C_ERROR_M2.

This sentence should come last and be slightly more explicit.

> Return success when no i3c device found at svc_i3c_master_do_daa_locked().

Please explain why this is important/useful.

> Fixes: dd3c52846d59 ("i3c: master: svc: Add Silvaco I3C master driver")

Shall we consider a backport into stable kernels?

> Signed-off-by: Frank Li <Frank.Li at nxp.com>
> ---
>  drivers/i3c/master/svc-i3c-master.c | 13 +++++++++++--
>  1 file changed, 11 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/i3c/master/svc-i3c-master.c b/drivers/i3c/master/svc-i3c-master.c
> index 770b40e28015e..a5620103acb73 100644
> --- a/drivers/i3c/master/svc-i3c-master.c
> +++ b/drivers/i3c/master/svc-i3c-master.c
> @@ -789,6 +789,9 @@ static int svc_i3c_master_do_daa_locked(struct svc_i3c_master *master,
>  				 */
>  				break;
>  			} else if (SVC_I3C_MSTATUS_NACKED(reg)) {
> +				/* No I3C devices attached */
> +				if (dev_nb == 0)
> +					break;

\n ?

>  				/*
>  				 * A slave device nacked the address, this is
>  				 * allowed only once, DAA will be stopped and
> @@ -1263,11 +1266,17 @@ static int svc_i3c_master_send_ccc_cmd(struct i3c_master_controller *m,
>  {
>  	struct svc_i3c_master *master = to_svc_i3c_master(m);
>  	bool broadcast = cmd->id < 0x80;
> +	int ret;
>  
>  	if (broadcast)
> -		return svc_i3c_master_send_bdcast_ccc_cmd(master, cmd);
> +		ret = svc_i3c_master_send_bdcast_ccc_cmd(master, cmd);
>  	else
> -		return svc_i3c_master_send_direct_ccc_cmd(master, cmd);
> +		ret = svc_i3c_master_send_direct_ccc_cmd(master, cmd);
> +
> +	if (ret)
> +		cmd->err = I3C_ERROR_M2;
> +
> +	return ret;
>  }
>  
>  static int svc_i3c_master_priv_xfers(struct i3c_dev_desc *dev,


Thanks,
Miqu?l


From Frank.li at nxp.com  Tue Aug 29 08:26:42 2023
From: Frank.li at nxp.com (Frank Li)
Date: Tue, 29 Aug 2023 11:26:42 -0400
Subject: [PATCH 2/2] i3c: master: svc: fix probe failure when no i3c
 device exist
In-Reply-To: <20230829111611.12f91348@xps-13>
References: <20230828192502.3525418-1-Frank.Li@nxp.com>
 <20230828192502.3525418-2-Frank.Li@nxp.com>
 <20230829111611.12f91348@xps-13>
Message-ID: <ZO4OMnPNCYecYImQ@lizhi-Precision-Tower-5810>

On Tue, Aug 29, 2023 at 11:16:11AM +0200, Miquel Raynal wrote:
> Hi Frank,
> 
> Frank.Li at nxp.com wrote on Mon, 28 Aug 2023 15:25:02 -0400:
> 
> > If there are not i3c device, all ccc command will get NACK. Set
> 
> 		no					NACKed?
> 
> > i3c_ccc_cmd::err as I3C_ERROR_M2.
> 
> This sentence should come last and be slightly more explicit.
> 
> > Return success when no i3c device found at svc_i3c_master_do_daa_locked().
> 
> Please explain why this is important/useful.

If return failure, driver will probe failure when there are no any i3c
devices. I3C master supposed support hot join. The probe failure don't
make sense if no i3c devices found when master driver probe.

How about rewrite commit log as

"I3C master supports device hot join, it doesn't make sense master driver
probe failure when there are no I3C devices.

When there are no I3C devices attached, all CCC commands are NACKed. So
SVC I3C master fails to probe.

Set i3c_cc_cmd:err as I3C_ERROR_M2. So I3C master framework
i3c_master_rstdaa_locked() can return expected ERROR code and continue
driver probe process.

Return success at svc_i3c_master_do_daa_locked() if no i3c devices found.
So SVC master driver can probe successfully even if no I3C devices are
attached"

> 
> > Fixes: dd3c52846d59 ("i3c: master: svc: Add Silvaco I3C master driver")
> 
> Shall we consider a backport into stable kernels?

Yes, I can add stable tag at next version.
> 
> > Signed-off-by: Frank Li <Frank.Li at nxp.com>
> > ---
> >  drivers/i3c/master/svc-i3c-master.c | 13 +++++++++++--
> >  1 file changed, 11 insertions(+), 2 deletions(-)
> > 
> > diff --git a/drivers/i3c/master/svc-i3c-master.c b/drivers/i3c/master/svc-i3c-master.c
> > index 770b40e28015e..a5620103acb73 100644
> > --- a/drivers/i3c/master/svc-i3c-master.c
> > +++ b/drivers/i3c/master/svc-i3c-master.c
> > @@ -789,6 +789,9 @@ static int svc_i3c_master_do_daa_locked(struct svc_i3c_master *master,
> >  				 */
> >  				break;
> >  			} else if (SVC_I3C_MSTATUS_NACKED(reg)) {
> > +				/* No I3C devices attached */
> > +				if (dev_nb == 0)
> > +					break;
> 
> \n ?
> 
> >  				/*
> >  				 * A slave device nacked the address, this is
> >  				 * allowed only once, DAA will be stopped and
> > @@ -1263,11 +1266,17 @@ static int svc_i3c_master_send_ccc_cmd(struct i3c_master_controller *m,
> >  {
> >  	struct svc_i3c_master *master = to_svc_i3c_master(m);
> >  	bool broadcast = cmd->id < 0x80;
> > +	int ret;
> >  
> >  	if (broadcast)
> > -		return svc_i3c_master_send_bdcast_ccc_cmd(master, cmd);
> > +		ret = svc_i3c_master_send_bdcast_ccc_cmd(master, cmd);
> >  	else
> > -		return svc_i3c_master_send_direct_ccc_cmd(master, cmd);
> > +		ret = svc_i3c_master_send_direct_ccc_cmd(master, cmd);
> > +
> > +	if (ret)
> > +		cmd->err = I3C_ERROR_M2;
> > +
> > +	return ret;
> >  }
> >  
> >  static int svc_i3c_master_priv_xfers(struct i3c_dev_desc *dev,
> 
> 
> Thanks,
> Miqu?l


From miquel.raynal at bootlin.com  Wed Aug 30 01:52:11 2023
From: miquel.raynal at bootlin.com (Miquel Raynal)
Date: Wed, 30 Aug 2023 10:52:11 +0200
Subject: [PATCH 2/2] i3c: master: svc: fix probe failure when no i3c
 device exist
In-Reply-To: <ZO4OMnPNCYecYImQ@lizhi-Precision-Tower-5810>
References: <20230828192502.3525418-1-Frank.Li@nxp.com>
	<20230828192502.3525418-2-Frank.Li@nxp.com>
	<20230829111611.12f91348@xps-13>
	<ZO4OMnPNCYecYImQ@lizhi-Precision-Tower-5810>
Message-ID: <20230830105211.07a2c790@xps-13>

Hi Frank,

Frank.li at nxp.com wrote on Tue, 29 Aug 2023 11:26:42 -0400:

> On Tue, Aug 29, 2023 at 11:16:11AM +0200, Miquel Raynal wrote:
> > Hi Frank,
> > 
> > Frank.Li at nxp.com wrote on Mon, 28 Aug 2023 15:25:02 -0400:
> >   
> > > If there are not i3c device, all ccc command will get NACK. Set  
> > 
> > 		no					NACKed?
> >   
> > > i3c_ccc_cmd::err as I3C_ERROR_M2.  
> > 
> > This sentence should come last and be slightly more explicit.
> >   
> > > Return success when no i3c device found at svc_i3c_master_do_daa_locked().  
> > 
> > Please explain why this is important/useful.  
> 
> If return failure, driver will probe failure when there are no any i3c
> devices. I3C master supposed support hot join. The probe failure don't
> make sense if no i3c devices found when master driver probe.
> 
> How about rewrite commit log as

Much better, let me propose something slightly clearer.

> "I3C master supports device hot join, it doesn't make sense master driver
> probe failure when there are no I3C devices.
>
> When there are no I3C devices attached, all CCC commands are NACKed. So
> SVC I3C master fails to probe.

What about:

I3C masters are expected to support hot-join. This means at
initialization time we might not yet discover any device and this
should not be treated as a fatal error.

During the DAA procedure which happens at probe time, if no device has
joined, all CCC will be NACKed (from a bus perspective). This leads to
an early return with an error code which fails the probe of the master.

Let's avoid this by just telling the core through an I3C_ERROR_M2
return command code that no device was discovered, which is a valid
situation. This way the master will no longer bail out and fail to
probe for a wrong reason.

> Set i3c_cc_cmd:err as I3C_ERROR_M2. So I3C master framework
> i3c_master_rstdaa_locked() can return expected ERROR code and continue
> driver probe process.
> 
> Return success at svc_i3c_master_do_daa_locked() if no i3c devices found.
> So SVC master driver can probe successfully even if no I3C devices are
> attached"
> 
> >   
> > > Fixes: dd3c52846d59 ("i3c: master: svc: Add Silvaco I3C master driver")  
> > 
> > Shall we consider a backport into stable kernels?  
> 
> Yes, I can add stable tag at next version.

Yes, please.

> >   
> > > Signed-off-by: Frank Li <Frank.Li at nxp.com>
> > > ---
> > >  drivers/i3c/master/svc-i3c-master.c | 13 +++++++++++--
> > >  1 file changed, 11 insertions(+), 2 deletions(-)
> > > 
> > > diff --git a/drivers/i3c/master/svc-i3c-master.c b/drivers/i3c/master/svc-i3c-master.c
> > > index 770b40e28015e..a5620103acb73 100644
> > > --- a/drivers/i3c/master/svc-i3c-master.c
> > > +++ b/drivers/i3c/master/svc-i3c-master.c
> > > @@ -789,6 +789,9 @@ static int svc_i3c_master_do_daa_locked(struct svc_i3c_master *master,
> > >  				 */
> > >  				break;
> > >  			} else if (SVC_I3C_MSTATUS_NACKED(reg)) {
> > > +				/* No I3C devices attached */
> > > +				if (dev_nb == 0)
> > > +					break;  
> > 
> > \n ?
> >   
> > >  				/*
> > >  				 * A slave device nacked the address, this is
> > >  				 * allowed only once, DAA will be stopped and
> > > @@ -1263,11 +1266,17 @@ static int svc_i3c_master_send_ccc_cmd(struct i3c_master_controller *m,
> > >  {
> > >  	struct svc_i3c_master *master = to_svc_i3c_master(m);
> > >  	bool broadcast = cmd->id < 0x80;
> > > +	int ret;
> > >  
> > >  	if (broadcast)
> > > -		return svc_i3c_master_send_bdcast_ccc_cmd(master, cmd);
> > > +		ret = svc_i3c_master_send_bdcast_ccc_cmd(master, cmd);
> > >  	else
> > > -		return svc_i3c_master_send_direct_ccc_cmd(master, cmd);
> > > +		ret = svc_i3c_master_send_direct_ccc_cmd(master, cmd);
> > > +
> > > +	if (ret)
> > > +		cmd->err = I3C_ERROR_M2;
> > > +
> > > +	return ret;
> > >  }
> > >  
> > >  static int svc_i3c_master_priv_xfers(struct i3c_dev_desc *dev,  
> > 
> > 
> > Thanks,
> > Miqu?l  


Thanks,
Miqu?l


From Frank.Li at nxp.com  Wed Aug 30 07:17:26 2023
From: Frank.Li at nxp.com (Frank Li)
Date: Wed, 30 Aug 2023 10:17:26 -0400
Subject: [PATCH v2 1/2] i3c: master: fixes i3c bus driver probe failure if no i3c device attached
Message-ID: <20230830141727.3794152-1-Frank.Li@nxp.com>

In i3c_master_bus_init()
{	...
	ret = i3c_master_rstdaa_locked(master, I3C_BROADCAST_ADDR);
	if (ret && ret != I3C_ERROR_M2)
			  ^^^ // it is enum i3c_error_code
	...
}

In dw-i3c-master.c implementation:
dw_i3c_ccc_set()
{	...
	ret = xfer->ret;
	if (xfer->cmds[0].error == RESPONSE_ERROR_IBA_NACK)
		ccc->err = I3C_ERROR_M2;

	dw_i3c_master_free_xfer(xfer);

	return ret;
}

Return enum i3c_error_code when error happen in i3c_master_rstdaa_locked().

Cc: stable at vger.kernel.org
Fixes: 3a379bbcea0a ("i3c: Add core I3C infrastructure")
Signed-off-by: Frank Li <Frank.Li at nxp.com>
---

Notes:
    Change from v1 to v2:
    - cc stable

 drivers/i3c/master.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/drivers/i3c/master.c b/drivers/i3c/master.c
index 08aeb69a78003..00a82f3ab9ac0 100644
--- a/drivers/i3c/master.c
+++ b/drivers/i3c/master.c
@@ -783,6 +783,9 @@ static int i3c_master_rstdaa_locked(struct i3c_master_controller *master,
 	ret = i3c_master_send_ccc_cmd_locked(master, &cmd);
 	i3c_ccc_cmd_dest_cleanup(&dest);
 
+	if (ret)
+		ret = cmd.err;
+
 	return ret;
 }
 
-- 
2.34.1



From Frank.Li at nxp.com  Wed Aug 30 07:17:27 2023
From: Frank.Li at nxp.com (Frank Li)
Date: Wed, 30 Aug 2023 10:17:27 -0400
Subject: [PATCH v2 2/2] i3c: master: svc: fix probe failure when no i3c device exist
In-Reply-To: <20230830141727.3794152-1-Frank.Li@nxp.com>
References: <20230830141727.3794152-1-Frank.Li@nxp.com>
Message-ID: <20230830141727.3794152-2-Frank.Li@nxp.com>

I3C masters are expected to support hot-join. This means at initialization
time we might not yet discover any device and this should not be treated
as a fatal error.

During the DAA procedure which happens at probe time, if no device has
joined, all CCC will be NACKed (from a bus perspective). This leads to an
early return with an error code which fails the probe of the master.

Let's avoid this by just telling the core through an I3C_ERROR_M2
return command code that no device was discovered, which is a valid
situation. This way the master will no longer bail out and fail to probe
for a wrong reason.

Cc: stable at vger.kernel.org
Fixes: dd3c52846d59 ("i3c: master: svc: Add Silvaco I3C master driver")
Signed-off-by: Frank Li <Frank.Li at nxp.com>
---

Notes:
    Change from v1 to v2:
    - rewrite commit message
    - cc stable
    - add empty line in daa()

 drivers/i3c/master/svc-i3c-master.c | 14 ++++++++++++--
 1 file changed, 12 insertions(+), 2 deletions(-)

diff --git a/drivers/i3c/master/svc-i3c-master.c b/drivers/i3c/master/svc-i3c-master.c
index 770b40e28015e..cf932ee056ef9 100644
--- a/drivers/i3c/master/svc-i3c-master.c
+++ b/drivers/i3c/master/svc-i3c-master.c
@@ -789,6 +789,10 @@ static int svc_i3c_master_do_daa_locked(struct svc_i3c_master *master,
 				 */
 				break;
 			} else if (SVC_I3C_MSTATUS_NACKED(reg)) {
+				/* No I3C devices attached */
+				if (dev_nb == 0)
+					break;
+
 				/*
 				 * A slave device nacked the address, this is
 				 * allowed only once, DAA will be stopped and
@@ -1263,11 +1267,17 @@ static int svc_i3c_master_send_ccc_cmd(struct i3c_master_controller *m,
 {
 	struct svc_i3c_master *master = to_svc_i3c_master(m);
 	bool broadcast = cmd->id < 0x80;
+	int ret;
 
 	if (broadcast)
-		return svc_i3c_master_send_bdcast_ccc_cmd(master, cmd);
+		ret = svc_i3c_master_send_bdcast_ccc_cmd(master, cmd);
 	else
-		return svc_i3c_master_send_direct_ccc_cmd(master, cmd);
+		ret = svc_i3c_master_send_direct_ccc_cmd(master, cmd);
+
+	if (ret)
+		cmd->err = I3C_ERROR_M2;
+
+	return ret;
 }
 
 static int svc_i3c_master_priv_xfers(struct i3c_dev_desc *dev,
-- 
2.34.1



From miquel.raynal at bootlin.com  Wed Aug 30 08:53:55 2023
From: miquel.raynal at bootlin.com (Miquel Raynal)
Date: Wed, 30 Aug 2023 17:53:55 +0200
Subject: [PATCH v2 1/2] i3c: master: fixes i3c bus driver probe failure
 if no i3c device attached
In-Reply-To: <20230830141727.3794152-1-Frank.Li@nxp.com>
References: <20230830141727.3794152-1-Frank.Li@nxp.com>
Message-ID: <20230830175355.4bc2b8d2@xps-13>

Hi Frank,

Frank.Li at nxp.com wrote on Wed, 30 Aug 2023 10:17:26 -0400:

> In i3c_master_bus_init()
> {	...
> 	ret = i3c_master_rstdaa_locked(master, I3C_BROADCAST_ADDR);
> 	if (ret && ret != I3C_ERROR_M2)
> 			  ^^^ // it is enum i3c_error_code
> 	...
> }
> 
> In dw-i3c-master.c implementation:
> dw_i3c_ccc_set()
> {	...
> 	ret = xfer->ret;
> 	if (xfer->cmds[0].error == RESPONSE_ERROR_IBA_NACK)
> 		ccc->err = I3C_ERROR_M2;
> 
> 	dw_i3c_master_free_xfer(xfer);
> 
> 	return ret;
> }
> 
> Return enum i3c_error_code when error happen in i3c_master_rstdaa_locked().

I am sorry but the commit log needs to be worked on.

> 
> Cc: stable at vger.kernel.org
> Fixes: 3a379bbcea0a ("i3c: Add core I3C infrastructure")
> Signed-off-by: Frank Li <Frank.Li at nxp.com>
> ---
> 
> Notes:
>     Change from v1 to v2:
>     - cc stable
> 
>  drivers/i3c/master.c | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/drivers/i3c/master.c b/drivers/i3c/master.c
> index 08aeb69a78003..00a82f3ab9ac0 100644
> --- a/drivers/i3c/master.c
> +++ b/drivers/i3c/master.c
> @@ -783,6 +783,9 @@ static int i3c_master_rstdaa_locked(struct i3c_master_controller *master,
>  	ret = i3c_master_send_ccc_cmd_locked(master, &cmd);
>  	i3c_ccc_cmd_dest_cleanup(&dest);
>  
> +	if (ret)
> +		ret = cmd.err;

Shouldn't this happen in i3c_master_send_ccc_cmd_locked()?

> +
>  	return ret;
>  }
>  


Thanks,
Miqu?l


From Frank.li at nxp.com  Wed Aug 30 09:32:28 2023
From: Frank.li at nxp.com (Frank Li)
Date: Wed, 30 Aug 2023 12:32:28 -0400
Subject: [PATCH v2 1/2] i3c: master: fixes i3c bus driver probe failure
 if no i3c device attached
In-Reply-To: <20230830175355.4bc2b8d2@xps-13>
References: <20230830141727.3794152-1-Frank.Li@nxp.com>
 <20230830175355.4bc2b8d2@xps-13>
Message-ID: <ZO9vHAZT7M8hNueW@lizhi-Precision-Tower-5810>

On Wed, Aug 30, 2023 at 05:53:55PM +0200, Miquel Raynal wrote:
> Hi Frank,
> 
> Frank.Li at nxp.com wrote on Wed, 30 Aug 2023 10:17:26 -0400:
> 
> > In i3c_master_bus_init()
> > {	...
> > 	ret = i3c_master_rstdaa_locked(master, I3C_BROADCAST_ADDR);
> > 	if (ret && ret != I3C_ERROR_M2)
> > 			  ^^^ // it is enum i3c_error_code
> > 	...
> > }
> > 
> > In dw-i3c-master.c implementation:
> > dw_i3c_ccc_set()
> > {	...
> > 	ret = xfer->ret;
> > 	if (xfer->cmds[0].error == RESPONSE_ERROR_IBA_NACK)
> > 		ccc->err = I3C_ERROR_M2;
> > 
> > 	dw_i3c_master_free_xfer(xfer);
> > 
> > 	return ret;
> > }
> > 
> > Return enum i3c_error_code when error happen in i3c_master_rstdaa_locked().
> 
> I am sorry but the commit log needs to be worked on.
> 
> > 
> > Cc: stable at vger.kernel.org
> > Fixes: 3a379bbcea0a ("i3c: Add core I3C infrastructure")
> > Signed-off-by: Frank Li <Frank.Li at nxp.com>
> > ---
> > 
> > Notes:
> >     Change from v1 to v2:
> >     - cc stable
> > 
> >  drivers/i3c/master.c | 3 +++
> >  1 file changed, 3 insertions(+)
> > 
> > diff --git a/drivers/i3c/master.c b/drivers/i3c/master.c
> > index 08aeb69a78003..00a82f3ab9ac0 100644
> > --- a/drivers/i3c/master.c
> > +++ b/drivers/i3c/master.c
> > @@ -783,6 +783,9 @@ static int i3c_master_rstdaa_locked(struct i3c_master_controller *master,
> >  	ret = i3c_master_send_ccc_cmd_locked(master, &cmd);
> >  	i3c_ccc_cmd_dest_cleanup(&dest);
> >  
> > +	if (ret)
> > +		ret = cmd.err;
> 
> Shouldn't this happen in i3c_master_send_ccc_cmd_locked()?

Sorry, This patch is not necesary at all.
i3c_master_send_ccc_cmd_locked() already handled it.
Only need second patch. You can discard this patch.

Do you need me send 2nd patch only? Or you can just pick 2nd one?

> 
> > +
> >  	return ret;
> >  }
> >  
> 
> 
> Thanks,
> Miqu?l


From Frank.Li at nxp.com  Mon Aug 28 12:25:01 2023
From: Frank.Li at nxp.com (Frank Li)
Date: Mon, 28 Aug 2023 15:25:01 -0400
Subject: [PATCH 1/2] i3c: master: fixes i3c bus driver probe failure if no i3c device attached
Message-ID: <20230828192502.3525418-1-Frank.Li@nxp.com>

In i3c_master_bus_init()
{	...
	ret = i3c_master_rstdaa_locked(master, I3C_BROADCAST_ADDR);
	if (ret && ret != I3C_ERROR_M2)
			  ^^^ // it is enum i3c_error_code
	...
}

In dw-i3c-master.c implementation:
dw_i3c_ccc_set()
{	...
	ret = xfer->ret;
	if (xfer->cmds[0].error == RESPONSE_ERROR_IBA_NACK)
		ccc->err = I3C_ERROR_M2;

	dw_i3c_master_free_xfer(xfer);

	return ret;
}

Return enum i3c_error_code when error happen in i3c_master_rstdaa_locked().

Fixes: 3a379bbcea0a ("i3c: Add core I3C infrastructure")
Signed-off-by: Frank Li <Frank.Li at nxp.com>
---
 drivers/i3c/master.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/drivers/i3c/master.c b/drivers/i3c/master.c
index 08aeb69a78003..00a82f3ab9ac0 100644
--- a/drivers/i3c/master.c
+++ b/drivers/i3c/master.c
@@ -783,6 +783,9 @@ static int i3c_master_rstdaa_locked(struct i3c_master_controller *master,
 	ret = i3c_master_send_ccc_cmd_locked(master, &cmd);
 	i3c_ccc_cmd_dest_cleanup(&dest);
 
+	if (ret)
+		ret = cmd.err;
+
 	return ret;
 }
 
-- 
2.34.1



From Frank.Li at nxp.com  Mon Aug 28 12:25:02 2023
From: Frank.Li at nxp.com (Frank Li)
Date: Mon, 28 Aug 2023 15:25:02 -0400
Subject: [PATCH 2/2] i3c: master: svc: fix probe failure when no i3c device exist
In-Reply-To: <20230828192502.3525418-1-Frank.Li@nxp.com>
References: <20230828192502.3525418-1-Frank.Li@nxp.com>
Message-ID: <20230828192502.3525418-2-Frank.Li@nxp.com>

If there are not i3c device, all ccc command will get NACK. Set
i3c_ccc_cmd::err as I3C_ERROR_M2.

Return success when no i3c device found at svc_i3c_master_do_daa_locked().

Fixes: dd3c52846d59 ("i3c: master: svc: Add Silvaco I3C master driver")
Signed-off-by: Frank Li <Frank.Li at nxp.com>
---
 drivers/i3c/master/svc-i3c-master.c | 13 +++++++++++--
 1 file changed, 11 insertions(+), 2 deletions(-)

diff --git a/drivers/i3c/master/svc-i3c-master.c b/drivers/i3c/master/svc-i3c-master.c
index 770b40e28015e..a5620103acb73 100644
--- a/drivers/i3c/master/svc-i3c-master.c
+++ b/drivers/i3c/master/svc-i3c-master.c
@@ -789,6 +789,9 @@ static int svc_i3c_master_do_daa_locked(struct svc_i3c_master *master,
 				 */
 				break;
 			} else if (SVC_I3C_MSTATUS_NACKED(reg)) {
+				/* No I3C devices attached */
+				if (dev_nb == 0)
+					break;
 				/*
 				 * A slave device nacked the address, this is
 				 * allowed only once, DAA will be stopped and
@@ -1263,11 +1266,17 @@ static int svc_i3c_master_send_ccc_cmd(struct i3c_master_controller *m,
 {
 	struct svc_i3c_master *master = to_svc_i3c_master(m);
 	bool broadcast = cmd->id < 0x80;
+	int ret;
 
 	if (broadcast)
-		return svc_i3c_master_send_bdcast_ccc_cmd(master, cmd);
+		ret = svc_i3c_master_send_bdcast_ccc_cmd(master, cmd);
 	else
-		return svc_i3c_master_send_direct_ccc_cmd(master, cmd);
+		ret = svc_i3c_master_send_direct_ccc_cmd(master, cmd);
+
+	if (ret)
+		cmd->err = I3C_ERROR_M2;
+
+	return ret;
 }
 
 static int svc_i3c_master_priv_xfers(struct i3c_dev_desc *dev,
-- 
2.34.1



From miquel.raynal at bootlin.com  Wed Aug 30 23:57:28 2023
From: miquel.raynal at bootlin.com (Miquel Raynal)
Date: Thu, 31 Aug 2023 08:57:28 +0200
Subject: [PATCH v2 1/2] i3c: master: fixes i3c bus driver probe failure
 if no i3c device attached
In-Reply-To: <ZO9vHAZT7M8hNueW@lizhi-Precision-Tower-5810>
References: <20230830141727.3794152-1-Frank.Li@nxp.com>
	<20230830175355.4bc2b8d2@xps-13>
	<ZO9vHAZT7M8hNueW@lizhi-Precision-Tower-5810>
Message-ID: <20230831085728.0becee29@xps-13>

Hi Frank,

Frank.li at nxp.com wrote on Wed, 30 Aug 2023 12:32:28 -0400:

> On Wed, Aug 30, 2023 at 05:53:55PM +0200, Miquel Raynal wrote:
> > Hi Frank,
> > 
> > Frank.Li at nxp.com wrote on Wed, 30 Aug 2023 10:17:26 -0400:
> >   
> > > In i3c_master_bus_init()
> > > {	...
> > > 	ret = i3c_master_rstdaa_locked(master, I3C_BROADCAST_ADDR);
> > > 	if (ret && ret != I3C_ERROR_M2)
> > > 			  ^^^ // it is enum i3c_error_code
> > > 	...
> > > }
> > > 
> > > In dw-i3c-master.c implementation:
> > > dw_i3c_ccc_set()
> > > {	...
> > > 	ret = xfer->ret;
> > > 	if (xfer->cmds[0].error == RESPONSE_ERROR_IBA_NACK)
> > > 		ccc->err = I3C_ERROR_M2;
> > > 
> > > 	dw_i3c_master_free_xfer(xfer);
> > > 
> > > 	return ret;
> > > }
> > > 
> > > Return enum i3c_error_code when error happen in i3c_master_rstdaa_locked().  
> > 
> > I am sorry but the commit log needs to be worked on.
> >   
> > > 
> > > Cc: stable at vger.kernel.org
> > > Fixes: 3a379bbcea0a ("i3c: Add core I3C infrastructure")
> > > Signed-off-by: Frank Li <Frank.Li at nxp.com>
> > > ---
> > > 
> > > Notes:
> > >     Change from v1 to v2:
> > >     - cc stable
> > > 
> > >  drivers/i3c/master.c | 3 +++
> > >  1 file changed, 3 insertions(+)
> > > 
> > > diff --git a/drivers/i3c/master.c b/drivers/i3c/master.c
> > > index 08aeb69a78003..00a82f3ab9ac0 100644
> > > --- a/drivers/i3c/master.c
> > > +++ b/drivers/i3c/master.c
> > > @@ -783,6 +783,9 @@ static int i3c_master_rstdaa_locked(struct i3c_master_controller *master,
> > >  	ret = i3c_master_send_ccc_cmd_locked(master, &cmd);
> > >  	i3c_ccc_cmd_dest_cleanup(&dest);
> > >  
> > > +	if (ret)
> > > +		ret = cmd.err;  
> > 
> > Shouldn't this happen in i3c_master_send_ccc_cmd_locked()?  
> 
> Sorry, This patch is not necesary at all.
> i3c_master_send_ccc_cmd_locked() already handled it.
> Only need second patch. You can discard this patch.
> 
> Do you need me send 2nd patch only? Or you can just pick 2nd one?

It is quite unclear what you are finally trying to do. Please send a
proper v3, with a changelog.

> 
> >   
> > > +
> > >  	return ret;
> > >  }
> > >    
> > 
> > 
> > Thanks,
> > Miqu?l  


Thanks,
Miqu?l


From Frank.Li at nxp.com  Thu Aug 31 07:13:24 2023
From: Frank.Li at nxp.com (Frank Li)
Date: Thu, 31 Aug 2023 10:13:24 -0400
Subject: [PATCH v3 1/1] i3c: master: svc: fix probe failure when no i3c device exist
Message-ID: <20230831141324.2841525-1-Frank.Li@nxp.com>

I3C masters are expected to support hot-join. This means at initialization
time we might not yet discover any device and this should not be treated
as a fatal error.

During the DAA procedure which happens at probe time, if no device has
joined, all CCC will be NACKed (from a bus perspective). This leads to an
early return with an error code which fails the probe of the master.

Let's avoid this by just telling the core through an I3C_ERROR_M2
return command code that no device was discovered, which is a valid
situation. This way the master will no longer bail out and fail to probe
for a wrong reason.

Cc: stable at vger.kernel.org
Fixes: dd3c52846d59 ("i3c: master: svc: Add Silvaco I3C master driver")
Signed-off-by: Frank Li <Frank.Li at nxp.com>
---

Notes:
    Change from v3 to v3
    - No code change
    - Remove patch i3c: master: fixes i3c bus driver probe failure if no i3c
    device attached
    - After test, only need this patch to fix probe failure because
    i3c_master_rstdaa_locked() already correct handle error code.
    
    Change from v1 to v2:
    - rewrite commit message
    - cc stable
    - add empty line in daa()

 drivers/i3c/master/svc-i3c-master.c | 14 ++++++++++++--
 1 file changed, 12 insertions(+), 2 deletions(-)

diff --git a/drivers/i3c/master/svc-i3c-master.c b/drivers/i3c/master/svc-i3c-master.c
index 770b40e28015e..cf932ee056ef9 100644
--- a/drivers/i3c/master/svc-i3c-master.c
+++ b/drivers/i3c/master/svc-i3c-master.c
@@ -789,6 +789,10 @@ static int svc_i3c_master_do_daa_locked(struct svc_i3c_master *master,
 				 */
 				break;
 			} else if (SVC_I3C_MSTATUS_NACKED(reg)) {
+				/* No I3C devices attached */
+				if (dev_nb == 0)
+					break;
+
 				/*
 				 * A slave device nacked the address, this is
 				 * allowed only once, DAA will be stopped and
@@ -1263,11 +1267,17 @@ static int svc_i3c_master_send_ccc_cmd(struct i3c_master_controller *m,
 {
 	struct svc_i3c_master *master = to_svc_i3c_master(m);
 	bool broadcast = cmd->id < 0x80;
+	int ret;
 
 	if (broadcast)
-		return svc_i3c_master_send_bdcast_ccc_cmd(master, cmd);
+		ret = svc_i3c_master_send_bdcast_ccc_cmd(master, cmd);
 	else
-		return svc_i3c_master_send_direct_ccc_cmd(master, cmd);
+		ret = svc_i3c_master_send_direct_ccc_cmd(master, cmd);
+
+	if (ret)
+		cmd->err = I3C_ERROR_M2;
+
+	return ret;
 }
 
 static int svc_i3c_master_priv_xfers(struct i3c_dev_desc *dev,
-- 
2.34.1



From miquel.raynal at bootlin.com  Thu Aug 31 07:21:34 2023
From: miquel.raynal at bootlin.com (Miquel Raynal)
Date: Thu, 31 Aug 2023 16:21:34 +0200
Subject: [PATCH v3 1/1] i3c: master: svc: fix probe failure when no i3c
 device exist
In-Reply-To: <20230831141324.2841525-1-Frank.Li@nxp.com>
References: <20230831141324.2841525-1-Frank.Li@nxp.com>
Message-ID: <20230831162134.6ccbaf23@xps-13>

Hi Frank,

Frank.Li at nxp.com wrote on Thu, 31 Aug 2023 10:13:24 -0400:

> I3C masters are expected to support hot-join. This means at initialization
> time we might not yet discover any device and this should not be treated
> as a fatal error.
> 
> During the DAA procedure which happens at probe time, if no device has
> joined, all CCC will be NACKed (from a bus perspective). This leads to an
> early return with an error code which fails the probe of the master.
> 
> Let's avoid this by just telling the core through an I3C_ERROR_M2
> return command code that no device was discovered, which is a valid
> situation. This way the master will no longer bail out and fail to probe
> for a wrong reason.
> 
> Cc: stable at vger.kernel.org
> Fixes: dd3c52846d59 ("i3c: master: svc: Add Silvaco I3C master driver")
> Signed-off-by: Frank Li <Frank.Li at nxp.com>

Acked-by: Miquel Raynal <miquel.raynal at bootlin.com>

Thanks,
Miqu?l


