* [PATCH v2 0/4] arm: dts: nxp: imx6: fix esai related dtb_check warning
@ 2024-04-16 14:47 ` Frank Li
0 siblings, 0 replies; 14+ messages in thread
From: Frank Li @ 2024-04-16 14:47 UTC (permalink / raw)
To: Liam Girdwood, Mark Brown, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Shengjiu Wang, Shawn Guo, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam
Cc: linux-sound, devicetree, linux-kernel, imx, linux-arm-kernel, Frank Li
two patches update binding doc. two patches fix dts file.
dt_binding_check each patch.
git rebase -i aa8a0e0038936 --exec "make ARCH=arm CROSS_COMPILE=aarch64-linux-gnu- -j8 dt_binding_check DT_SCHEMA_FILES=fsl,esai.yaml"
Executing: make ARCH=arm CROSS_COMPILE=aarch64-linux-gnu- -j8 dt_binding_check DT_SCHEMA_FILES=fsl,esai.yaml
LINT Documentation/devicetree/bindings
DTEX Documentation/devicetree/bindings/sound/fsl,esai.example.dts
CHKDT Documentation/devicetree/bindings/processed-schema.json
SCHEMA Documentation/devicetree/bindings/processed-schema.json
DTC_CHK Documentation/devicetree/bindings/sound/fsl,esai.example.dtb
Executing: make ARCH=arm CROSS_COMPILE=aarch64-linux-gnu- -j8 dt_binding_check DT_SCHEMA_FILES=fsl,esai.yaml
DTEX Documentation/devicetree/bindings/sound/fsl,esai.example.dts
LINT Documentation/devicetree/bindings
CHKDT Documentation/devicetree/bindings/processed-schema.json
SCHEMA Documentation/devicetree/bindings/processed-schema.json
DTC_CHK Documentation/devicetree/bindings/sound/fsl,esai.example.dtb
Executing: make ARCH=arm CROSS_COMPILE=aarch64-linux-gnu- -j8 dt_binding_check DT_SCHEMA_FILES=fsl,esai.yaml
Executing: make ARCH=arm CROSS_COMPILE=aarch64-linux-gnu- -j8 dt_binding_check DT_SCHEMA_FILES=fsl,esai.yaml
Successfully rebased and updated refs/heads/b4/esai_arm_dts_warning.
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
Changes in v2:
- Add rob's review tag at patch 2/4
- default: 64 at patch 1/4
- Link to v1: https://lore.kernel.org/r/20240411-esai_arm_dts_warning-v1-0-5e1a7c7be9b9@nxp.com
---
Frank Li (4):
ASoC: dt-bindings: fsl-esai: Remove 'fsl,*' from required list
ASoC: dt-bindings: fsl-esai: Add ref: dai-common.yaml
arm: dts: nxp: imx6sx: fix esai related warning when do dtb_check
arm: dts: nxp: imx6qdl: fix esai clock warning when do dtb_check
Documentation/devicetree/bindings/sound/fsl,esai.yaml | 6 ++++--
arch/arm/boot/dts/nxp/imx/imx6qdl.dtsi | 3 +--
arch/arm/boot/dts/nxp/imx/imx6sx.dtsi | 5 ++---
3 files changed, 7 insertions(+), 7 deletions(-)
---
base-commit: a053fd3ca5d1b927a8655f239c84b0d790218fda
change-id: 20240411-esai_arm_dts_warning-86753197f796
Best regards,
---
Frank Li <Frank.Li@nxp.com>
^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH v2 0/4] arm: dts: nxp: imx6: fix esai related dtb_check warning
@ 2024-04-16 14:47 ` Frank Li
0 siblings, 0 replies; 14+ messages in thread
From: Frank Li @ 2024-04-16 14:47 UTC (permalink / raw)
To: Liam Girdwood, Mark Brown, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Shengjiu Wang, Shawn Guo, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam
Cc: linux-sound, devicetree, linux-kernel, imx, linux-arm-kernel, Frank Li
two patches update binding doc. two patches fix dts file.
dt_binding_check each patch.
git rebase -i aa8a0e0038936 --exec "make ARCH=arm CROSS_COMPILE=aarch64-linux-gnu- -j8 dt_binding_check DT_SCHEMA_FILES=fsl,esai.yaml"
Executing: make ARCH=arm CROSS_COMPILE=aarch64-linux-gnu- -j8 dt_binding_check DT_SCHEMA_FILES=fsl,esai.yaml
LINT Documentation/devicetree/bindings
DTEX Documentation/devicetree/bindings/sound/fsl,esai.example.dts
CHKDT Documentation/devicetree/bindings/processed-schema.json
SCHEMA Documentation/devicetree/bindings/processed-schema.json
DTC_CHK Documentation/devicetree/bindings/sound/fsl,esai.example.dtb
Executing: make ARCH=arm CROSS_COMPILE=aarch64-linux-gnu- -j8 dt_binding_check DT_SCHEMA_FILES=fsl,esai.yaml
DTEX Documentation/devicetree/bindings/sound/fsl,esai.example.dts
LINT Documentation/devicetree/bindings
CHKDT Documentation/devicetree/bindings/processed-schema.json
SCHEMA Documentation/devicetree/bindings/processed-schema.json
DTC_CHK Documentation/devicetree/bindings/sound/fsl,esai.example.dtb
Executing: make ARCH=arm CROSS_COMPILE=aarch64-linux-gnu- -j8 dt_binding_check DT_SCHEMA_FILES=fsl,esai.yaml
Executing: make ARCH=arm CROSS_COMPILE=aarch64-linux-gnu- -j8 dt_binding_check DT_SCHEMA_FILES=fsl,esai.yaml
Successfully rebased and updated refs/heads/b4/esai_arm_dts_warning.
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
Changes in v2:
- Add rob's review tag at patch 2/4
- default: 64 at patch 1/4
- Link to v1: https://lore.kernel.org/r/20240411-esai_arm_dts_warning-v1-0-5e1a7c7be9b9@nxp.com
---
Frank Li (4):
ASoC: dt-bindings: fsl-esai: Remove 'fsl,*' from required list
ASoC: dt-bindings: fsl-esai: Add ref: dai-common.yaml
arm: dts: nxp: imx6sx: fix esai related warning when do dtb_check
arm: dts: nxp: imx6qdl: fix esai clock warning when do dtb_check
Documentation/devicetree/bindings/sound/fsl,esai.yaml | 6 ++++--
arch/arm/boot/dts/nxp/imx/imx6qdl.dtsi | 3 +--
arch/arm/boot/dts/nxp/imx/imx6sx.dtsi | 5 ++---
3 files changed, 7 insertions(+), 7 deletions(-)
---
base-commit: a053fd3ca5d1b927a8655f239c84b0d790218fda
change-id: 20240411-esai_arm_dts_warning-86753197f796
Best regards,
---
Frank Li <Frank.Li@nxp.com>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH v2 1/4] ASoC: dt-bindings: fsl-esai: Remove 'fsl,*' from required list
2024-04-16 14:47 ` Frank Li
@ 2024-04-16 14:47 ` Frank Li
-1 siblings, 0 replies; 14+ messages in thread
From: Frank Li @ 2024-04-16 14:47 UTC (permalink / raw)
To: Liam Girdwood, Mark Brown, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Shengjiu Wang, Shawn Guo, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam
Cc: linux-sound, devicetree, linux-kernel, imx, linux-arm-kernel, Frank Li
fsl,fifo-depth have default value 64 in driver(sound/soc/fsl/fsl_esai.c).
fsl,esai-synchronous is flag(bool) type. It doesn't make sense to put flag
type into 'required'.
Fix warning:
arch/arm/boot/dts/nxp/imx/imx6q-sabreauto.dtb: esai@2024000: 'fsl,fifo-depth' is a required property
arch/arm/boot/dts/nxp/imx/imx6q-sabreauto.dtb: esai@2024000: 'fsl,esai-synchronous' is a required property
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
Documentation/devicetree/bindings/sound/fsl,esai.yaml | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/Documentation/devicetree/bindings/sound/fsl,esai.yaml b/Documentation/devicetree/bindings/sound/fsl,esai.yaml
index f167f1634d7e3..2a64e682b5178 100644
--- a/Documentation/devicetree/bindings/sound/fsl,esai.yaml
+++ b/Documentation/devicetree/bindings/sound/fsl,esai.yaml
@@ -67,6 +67,7 @@ properties:
fsl,fifo-depth:
$ref: /schemas/types.yaml#/definitions/uint32
+ default: 64
description:
The number of elements in the transmit and receive
FIFOs. This number is the maximum allowed value for
@@ -95,8 +96,6 @@ required:
- clock-names
- dmas
- dma-names
- - fsl,fifo-depth
- - fsl,esai-synchronous
unevaluatedProperties: false
--
2.34.1
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH v2 1/4] ASoC: dt-bindings: fsl-esai: Remove 'fsl,*' from required list
@ 2024-04-16 14:47 ` Frank Li
0 siblings, 0 replies; 14+ messages in thread
From: Frank Li @ 2024-04-16 14:47 UTC (permalink / raw)
To: Liam Girdwood, Mark Brown, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Shengjiu Wang, Shawn Guo, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam
Cc: linux-sound, devicetree, linux-kernel, imx, linux-arm-kernel, Frank Li
fsl,fifo-depth have default value 64 in driver(sound/soc/fsl/fsl_esai.c).
fsl,esai-synchronous is flag(bool) type. It doesn't make sense to put flag
type into 'required'.
Fix warning:
arch/arm/boot/dts/nxp/imx/imx6q-sabreauto.dtb: esai@2024000: 'fsl,fifo-depth' is a required property
arch/arm/boot/dts/nxp/imx/imx6q-sabreauto.dtb: esai@2024000: 'fsl,esai-synchronous' is a required property
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
Documentation/devicetree/bindings/sound/fsl,esai.yaml | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/Documentation/devicetree/bindings/sound/fsl,esai.yaml b/Documentation/devicetree/bindings/sound/fsl,esai.yaml
index f167f1634d7e3..2a64e682b5178 100644
--- a/Documentation/devicetree/bindings/sound/fsl,esai.yaml
+++ b/Documentation/devicetree/bindings/sound/fsl,esai.yaml
@@ -67,6 +67,7 @@ properties:
fsl,fifo-depth:
$ref: /schemas/types.yaml#/definitions/uint32
+ default: 64
description:
The number of elements in the transmit and receive
FIFOs. This number is the maximum allowed value for
@@ -95,8 +96,6 @@ required:
- clock-names
- dmas
- dma-names
- - fsl,fifo-depth
- - fsl,esai-synchronous
unevaluatedProperties: false
--
2.34.1
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH v2 2/4] ASoC: dt-bindings: fsl-esai: Add ref: dai-common.yaml
2024-04-16 14:47 ` Frank Li
@ 2024-04-16 14:47 ` Frank Li
-1 siblings, 0 replies; 14+ messages in thread
From: Frank Li @ 2024-04-16 14:47 UTC (permalink / raw)
To: Liam Girdwood, Mark Brown, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Shengjiu Wang, Shawn Guo, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam
Cc: linux-sound, devicetree, linux-kernel, imx, linux-arm-kernel, Frank Li
Add ref: dai-common.yaml to fix below warning.
arch/arm/boot/dts/nxp/imx/imx6q-sabreauto.dtb: esai@2024000: Unevaluated properties are not allowed ('#sound-dai-cells' was unexpected)
Reviewed-by: Rob Herring <robh@kernel.org>
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
Documentation/devicetree/bindings/sound/fsl,esai.yaml | 3 +++
1 file changed, 3 insertions(+)
diff --git a/Documentation/devicetree/bindings/sound/fsl,esai.yaml b/Documentation/devicetree/bindings/sound/fsl,esai.yaml
index 2a64e682b5178..f99ed20fa684a 100644
--- a/Documentation/devicetree/bindings/sound/fsl,esai.yaml
+++ b/Documentation/devicetree/bindings/sound/fsl,esai.yaml
@@ -99,6 +99,9 @@ required:
unevaluatedProperties: false
+allOf:
+ - $ref: dai-common.yaml#
+
examples:
- |
esai@2024000 {
--
2.34.1
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH v2 2/4] ASoC: dt-bindings: fsl-esai: Add ref: dai-common.yaml
@ 2024-04-16 14:47 ` Frank Li
0 siblings, 0 replies; 14+ messages in thread
From: Frank Li @ 2024-04-16 14:47 UTC (permalink / raw)
To: Liam Girdwood, Mark Brown, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Shengjiu Wang, Shawn Guo, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam
Cc: linux-sound, devicetree, linux-kernel, imx, linux-arm-kernel, Frank Li
Add ref: dai-common.yaml to fix below warning.
arch/arm/boot/dts/nxp/imx/imx6q-sabreauto.dtb: esai@2024000: Unevaluated properties are not allowed ('#sound-dai-cells' was unexpected)
Reviewed-by: Rob Herring <robh@kernel.org>
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
Documentation/devicetree/bindings/sound/fsl,esai.yaml | 3 +++
1 file changed, 3 insertions(+)
diff --git a/Documentation/devicetree/bindings/sound/fsl,esai.yaml b/Documentation/devicetree/bindings/sound/fsl,esai.yaml
index 2a64e682b5178..f99ed20fa684a 100644
--- a/Documentation/devicetree/bindings/sound/fsl,esai.yaml
+++ b/Documentation/devicetree/bindings/sound/fsl,esai.yaml
@@ -99,6 +99,9 @@ required:
unevaluatedProperties: false
+allOf:
+ - $ref: dai-common.yaml#
+
examples:
- |
esai@2024000 {
--
2.34.1
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH v2 3/4] arm: dts: nxp: imx6sx: fix esai related warning when do dtb_check
2024-04-16 14:47 ` Frank Li
@ 2024-04-16 14:47 ` Frank Li
-1 siblings, 0 replies; 14+ messages in thread
From: Frank Li @ 2024-04-16 14:47 UTC (permalink / raw)
To: Liam Girdwood, Mark Brown, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Shengjiu Wang, Shawn Guo, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam
Cc: linux-sound, devicetree, linux-kernel, imx, linux-arm-kernel, Frank Li
Remove undocumented compatible string 'fsl,imx6sx-esai', which never used
in driver sound/soc/fsl/fsl_esai.c.
Remove unused clock-names 'mem'. Driver never use clock name 'mem'.
Fix below warning:
arch/arm/boot/dts/nxp/imx/imx6sx-sdb.dtb: esai@2024000: compatible:0: 'fsl,imx6sx-esai' is not one of ['fsl,imx35-esai', 'fsl,imx6ull-esai', 'fsl,imx8qm-esai', 'fsl,vf610-esai']
from schema $id: http://devicetree.org/schemas/sound/fsl,esai.yaml#
arm/arch/arm/boot/dts/nxp/imx/imx6sx-sdb.dtb: esai@2024000: compatible: ['fsl,imx6sx-esai', 'fsl,imx35-esai'] is too long
from schema $id: http://devicetree.org/schemas/sound/fsl,esai.yaml#
arm/arch/arm/boot/dts/nxp/imx/imx6sx-sdb.dtb: esai@2024000: clocks: [[2, 239], [2, 240], [2, 152], [2, 239], [2, 196]] is too long
from schema $id: http://devicetree.org/schemas/sound/fsl,esai.yaml#
arm/boot/dts/nxp/imx/imx6sx-sdb.dtb: esai@2024000: clock-names:1: 'extal' was expected
from schema $id: http://devicetree.org/schemas/sound/fsl,esai.yaml#
arch/arm/boot/dts/nxp/imx/imx6sx-sdb.dtb: esai@2024000: clock-names:2: 'fsys' was expected
from schema $id: http://devicetree.org/schemas/sound/fsl,esai.yaml#
arch/arm/boot/dts/nxp/imx/imx6sx-sdb.dtb: esai@2024000: clock-names:3: 'spba' was expected
from schema $id: http://devicetree.org/schemas/sound/fsl,esai.yaml#
arch/arm/boot/dts/nxp/imx/imx6sx-sdb.dtb: esai@2024000: clock-names: ['core', 'mem', 'extal', 'fsys', 'spba'] is too long
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
arch/arm/boot/dts/nxp/imx/imx6sx.dtsi | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/arch/arm/boot/dts/nxp/imx/imx6sx.dtsi b/arch/arm/boot/dts/nxp/imx/imx6sx.dtsi
index 0de359d62a472..874f33fb17227 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6sx.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx6sx.dtsi
@@ -339,15 +339,14 @@ uart1: serial@2020000 {
};
esai: esai@2024000 {
- compatible = "fsl,imx6sx-esai", "fsl,imx35-esai";
+ compatible = "fsl,imx35-esai";
reg = <0x02024000 0x4000>;
interrupts = <GIC_SPI 51 IRQ_TYPE_LEVEL_HIGH>;
clocks = <&clks IMX6SX_CLK_ESAI_IPG>,
- <&clks IMX6SX_CLK_ESAI_MEM>,
<&clks IMX6SX_CLK_ESAI_EXTAL>,
<&clks IMX6SX_CLK_ESAI_IPG>,
<&clks IMX6SX_CLK_SPBA>;
- clock-names = "core", "mem", "extal",
+ clock-names = "core", "extal",
"fsys", "spba";
dmas = <&sdma 23 21 0>,
<&sdma 24 21 0>;
--
2.34.1
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH v2 3/4] arm: dts: nxp: imx6sx: fix esai related warning when do dtb_check
@ 2024-04-16 14:47 ` Frank Li
0 siblings, 0 replies; 14+ messages in thread
From: Frank Li @ 2024-04-16 14:47 UTC (permalink / raw)
To: Liam Girdwood, Mark Brown, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Shengjiu Wang, Shawn Guo, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam
Cc: linux-sound, devicetree, linux-kernel, imx, linux-arm-kernel, Frank Li
Remove undocumented compatible string 'fsl,imx6sx-esai', which never used
in driver sound/soc/fsl/fsl_esai.c.
Remove unused clock-names 'mem'. Driver never use clock name 'mem'.
Fix below warning:
arch/arm/boot/dts/nxp/imx/imx6sx-sdb.dtb: esai@2024000: compatible:0: 'fsl,imx6sx-esai' is not one of ['fsl,imx35-esai', 'fsl,imx6ull-esai', 'fsl,imx8qm-esai', 'fsl,vf610-esai']
from schema $id: http://devicetree.org/schemas/sound/fsl,esai.yaml#
arm/arch/arm/boot/dts/nxp/imx/imx6sx-sdb.dtb: esai@2024000: compatible: ['fsl,imx6sx-esai', 'fsl,imx35-esai'] is too long
from schema $id: http://devicetree.org/schemas/sound/fsl,esai.yaml#
arm/arch/arm/boot/dts/nxp/imx/imx6sx-sdb.dtb: esai@2024000: clocks: [[2, 239], [2, 240], [2, 152], [2, 239], [2, 196]] is too long
from schema $id: http://devicetree.org/schemas/sound/fsl,esai.yaml#
arm/boot/dts/nxp/imx/imx6sx-sdb.dtb: esai@2024000: clock-names:1: 'extal' was expected
from schema $id: http://devicetree.org/schemas/sound/fsl,esai.yaml#
arch/arm/boot/dts/nxp/imx/imx6sx-sdb.dtb: esai@2024000: clock-names:2: 'fsys' was expected
from schema $id: http://devicetree.org/schemas/sound/fsl,esai.yaml#
arch/arm/boot/dts/nxp/imx/imx6sx-sdb.dtb: esai@2024000: clock-names:3: 'spba' was expected
from schema $id: http://devicetree.org/schemas/sound/fsl,esai.yaml#
arch/arm/boot/dts/nxp/imx/imx6sx-sdb.dtb: esai@2024000: clock-names: ['core', 'mem', 'extal', 'fsys', 'spba'] is too long
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
arch/arm/boot/dts/nxp/imx/imx6sx.dtsi | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/arch/arm/boot/dts/nxp/imx/imx6sx.dtsi b/arch/arm/boot/dts/nxp/imx/imx6sx.dtsi
index 0de359d62a472..874f33fb17227 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6sx.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx6sx.dtsi
@@ -339,15 +339,14 @@ uart1: serial@2020000 {
};
esai: esai@2024000 {
- compatible = "fsl,imx6sx-esai", "fsl,imx35-esai";
+ compatible = "fsl,imx35-esai";
reg = <0x02024000 0x4000>;
interrupts = <GIC_SPI 51 IRQ_TYPE_LEVEL_HIGH>;
clocks = <&clks IMX6SX_CLK_ESAI_IPG>,
- <&clks IMX6SX_CLK_ESAI_MEM>,
<&clks IMX6SX_CLK_ESAI_EXTAL>,
<&clks IMX6SX_CLK_ESAI_IPG>,
<&clks IMX6SX_CLK_SPBA>;
- clock-names = "core", "mem", "extal",
+ clock-names = "core", "extal",
"fsys", "spba";
dmas = <&sdma 23 21 0>,
<&sdma 24 21 0>;
--
2.34.1
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH v2 4/4] arm: dts: nxp: imx6qdl: fix esai clock warning when do dtb_check
2024-04-16 14:47 ` Frank Li
@ 2024-04-16 14:47 ` Frank Li
-1 siblings, 0 replies; 14+ messages in thread
From: Frank Li @ 2024-04-16 14:47 UTC (permalink / raw)
To: Liam Girdwood, Mark Brown, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Shengjiu Wang, Shawn Guo, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam
Cc: linux-sound, devicetree, linux-kernel, imx, linux-arm-kernel, Frank Li
Remove unused clock-names 'mem'. Driver (sound/soc/fsl/fsl_esai.c.) never
use clock name 'mem'.
arch/arm/boot/dts/nxp/imx/imx6q-sabreauto.dtb: esai@2024000: clocks: [[2, 208], [2, 209], [2, 118], [2, 208], [2, 156]] is too long
from schema $id: http://devicetree.org/schemas/sound/fsl,esai.yaml#
arch/arm/boot/dts/nxp/imx/imx6q-sabreauto.dtb: esai@2024000: clock-names:1: 'extal' was expected
from schema $id: http://devicetree.org/schemas/sound/fsl,esai.yaml#
arch/arm/boot/dts/nxp/imx/imx6q-sabreauto.dtb: esai@2024000: clock-names:2: 'fsys' was expected
from schema $id: http://devicetree.org/schemas/sound/fsl,esai.yaml#
arch/arm/boot/dts/nxp/imx/imx6q-sabreauto.dtb: esai@2024000: clock-names:3: 'spba' was expected
from schema $id: http://devicetree.org/schemas/sound/fsl,esai.yaml#
arch/arm/boot/dts/nxp/imx/imx6q-sabreauto.dtb: esai@2024000: clock-names: ['core', 'mem', 'extal', 'fsys', 'spba'] is too long
from schema $id: http://devicetree.org/schemas/sound/fsl,esai.yaml#
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
arch/arm/boot/dts/nxp/imx/imx6qdl.dtsi | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/arch/arm/boot/dts/nxp/imx/imx6qdl.dtsi b/arch/arm/boot/dts/nxp/imx/imx6qdl.dtsi
index 8431b8a994f4c..d2200c9db25ae 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6qdl.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx6qdl.dtsi
@@ -397,11 +397,10 @@ esai: esai@2024000 {
reg = <0x02024000 0x4000>;
interrupts = <0 51 IRQ_TYPE_LEVEL_HIGH>;
clocks = <&clks IMX6QDL_CLK_ESAI_IPG>,
- <&clks IMX6QDL_CLK_ESAI_MEM>,
<&clks IMX6QDL_CLK_ESAI_EXTAL>,
<&clks IMX6QDL_CLK_ESAI_IPG>,
<&clks IMX6QDL_CLK_SPBA>;
- clock-names = "core", "mem", "extal", "fsys", "spba";
+ clock-names = "core", "extal", "fsys", "spba";
dmas = <&sdma 23 21 0>, <&sdma 24 21 0>;
dma-names = "rx", "tx";
status = "disabled";
--
2.34.1
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH v2 4/4] arm: dts: nxp: imx6qdl: fix esai clock warning when do dtb_check
@ 2024-04-16 14:47 ` Frank Li
0 siblings, 0 replies; 14+ messages in thread
From: Frank Li @ 2024-04-16 14:47 UTC (permalink / raw)
To: Liam Girdwood, Mark Brown, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Shengjiu Wang, Shawn Guo, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam
Cc: linux-sound, devicetree, linux-kernel, imx, linux-arm-kernel, Frank Li
Remove unused clock-names 'mem'. Driver (sound/soc/fsl/fsl_esai.c.) never
use clock name 'mem'.
arch/arm/boot/dts/nxp/imx/imx6q-sabreauto.dtb: esai@2024000: clocks: [[2, 208], [2, 209], [2, 118], [2, 208], [2, 156]] is too long
from schema $id: http://devicetree.org/schemas/sound/fsl,esai.yaml#
arch/arm/boot/dts/nxp/imx/imx6q-sabreauto.dtb: esai@2024000: clock-names:1: 'extal' was expected
from schema $id: http://devicetree.org/schemas/sound/fsl,esai.yaml#
arch/arm/boot/dts/nxp/imx/imx6q-sabreauto.dtb: esai@2024000: clock-names:2: 'fsys' was expected
from schema $id: http://devicetree.org/schemas/sound/fsl,esai.yaml#
arch/arm/boot/dts/nxp/imx/imx6q-sabreauto.dtb: esai@2024000: clock-names:3: 'spba' was expected
from schema $id: http://devicetree.org/schemas/sound/fsl,esai.yaml#
arch/arm/boot/dts/nxp/imx/imx6q-sabreauto.dtb: esai@2024000: clock-names: ['core', 'mem', 'extal', 'fsys', 'spba'] is too long
from schema $id: http://devicetree.org/schemas/sound/fsl,esai.yaml#
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
arch/arm/boot/dts/nxp/imx/imx6qdl.dtsi | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/arch/arm/boot/dts/nxp/imx/imx6qdl.dtsi b/arch/arm/boot/dts/nxp/imx/imx6qdl.dtsi
index 8431b8a994f4c..d2200c9db25ae 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6qdl.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx6qdl.dtsi
@@ -397,11 +397,10 @@ esai: esai@2024000 {
reg = <0x02024000 0x4000>;
interrupts = <0 51 IRQ_TYPE_LEVEL_HIGH>;
clocks = <&clks IMX6QDL_CLK_ESAI_IPG>,
- <&clks IMX6QDL_CLK_ESAI_MEM>,
<&clks IMX6QDL_CLK_ESAI_EXTAL>,
<&clks IMX6QDL_CLK_ESAI_IPG>,
<&clks IMX6QDL_CLK_SPBA>;
- clock-names = "core", "mem", "extal", "fsys", "spba";
+ clock-names = "core", "extal", "fsys", "spba";
dmas = <&sdma 23 21 0>, <&sdma 24 21 0>;
dma-names = "rx", "tx";
status = "disabled";
--
2.34.1
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: (subset) [PATCH v2 0/4] arm: dts: nxp: imx6: fix esai related dtb_check warning
2024-04-16 14:47 ` Frank Li
@ 2024-04-17 2:46 ` Mark Brown
-1 siblings, 0 replies; 14+ messages in thread
From: Mark Brown @ 2024-04-17 2:46 UTC (permalink / raw)
To: Liam Girdwood, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Shengjiu Wang, Shawn Guo, Sascha Hauer, Pengutronix Kernel Team,
Fabio Estevam, Frank Li
Cc: linux-sound, devicetree, linux-kernel, imx, linux-arm-kernel
On Tue, 16 Apr 2024 10:47:46 -0400, Frank Li wrote:
> two patches update binding doc. two patches fix dts file.
>
> dt_binding_check each patch.
>
> git rebase -i aa8a0e0038936 --exec "make ARCH=arm CROSS_COMPILE=aarch64-linux-gnu- -j8 dt_binding_check DT_SCHEMA_FILES=fsl,esai.yaml"
> Executing: make ARCH=arm CROSS_COMPILE=aarch64-linux-gnu- -j8 dt_binding_check DT_SCHEMA_FILES=fsl,esai.yaml
> LINT Documentation/devicetree/bindings
> DTEX Documentation/devicetree/bindings/sound/fsl,esai.example.dts
> CHKDT Documentation/devicetree/bindings/processed-schema.json
> SCHEMA Documentation/devicetree/bindings/processed-schema.json
> DTC_CHK Documentation/devicetree/bindings/sound/fsl,esai.example.dtb
> Executing: make ARCH=arm CROSS_COMPILE=aarch64-linux-gnu- -j8 dt_binding_check DT_SCHEMA_FILES=fsl,esai.yaml
> DTEX Documentation/devicetree/bindings/sound/fsl,esai.example.dts
> LINT Documentation/devicetree/bindings
> CHKDT Documentation/devicetree/bindings/processed-schema.json
> SCHEMA Documentation/devicetree/bindings/processed-schema.json
> DTC_CHK Documentation/devicetree/bindings/sound/fsl,esai.example.dtb
> Executing: make ARCH=arm CROSS_COMPILE=aarch64-linux-gnu- -j8 dt_binding_check DT_SCHEMA_FILES=fsl,esai.yaml
> Executing: make ARCH=arm CROSS_COMPILE=aarch64-linux-gnu- -j8 dt_binding_check DT_SCHEMA_FILES=fsl,esai.yaml
> Successfully rebased and updated refs/heads/b4/esai_arm_dts_warning.
>
> [...]
Applied to
https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next
Thanks!
[1/4] ASoC: dt-bindings: fsl-esai: Remove 'fsl,*' from required list
commit: 5894ff6c4707af645f2faa36cf07cc2745e1658f
[2/4] ASoC: dt-bindings: fsl-esai: Add ref: dai-common.yaml
commit: 9aea6d64bb3acbff6b2f33cb5bbff65e824ab9a7
All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.
You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.
If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.
Please add any relevant lists and maintainers to the CCs when replying
to this mail.
Thanks,
Mark
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: (subset) [PATCH v2 0/4] arm: dts: nxp: imx6: fix esai related dtb_check warning
@ 2024-04-17 2:46 ` Mark Brown
0 siblings, 0 replies; 14+ messages in thread
From: Mark Brown @ 2024-04-17 2:46 UTC (permalink / raw)
To: Liam Girdwood, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Shengjiu Wang, Shawn Guo, Sascha Hauer, Pengutronix Kernel Team,
Fabio Estevam, Frank Li
Cc: linux-sound, devicetree, linux-kernel, imx, linux-arm-kernel
On Tue, 16 Apr 2024 10:47:46 -0400, Frank Li wrote:
> two patches update binding doc. two patches fix dts file.
>
> dt_binding_check each patch.
>
> git rebase -i aa8a0e0038936 --exec "make ARCH=arm CROSS_COMPILE=aarch64-linux-gnu- -j8 dt_binding_check DT_SCHEMA_FILES=fsl,esai.yaml"
> Executing: make ARCH=arm CROSS_COMPILE=aarch64-linux-gnu- -j8 dt_binding_check DT_SCHEMA_FILES=fsl,esai.yaml
> LINT Documentation/devicetree/bindings
> DTEX Documentation/devicetree/bindings/sound/fsl,esai.example.dts
> CHKDT Documentation/devicetree/bindings/processed-schema.json
> SCHEMA Documentation/devicetree/bindings/processed-schema.json
> DTC_CHK Documentation/devicetree/bindings/sound/fsl,esai.example.dtb
> Executing: make ARCH=arm CROSS_COMPILE=aarch64-linux-gnu- -j8 dt_binding_check DT_SCHEMA_FILES=fsl,esai.yaml
> DTEX Documentation/devicetree/bindings/sound/fsl,esai.example.dts
> LINT Documentation/devicetree/bindings
> CHKDT Documentation/devicetree/bindings/processed-schema.json
> SCHEMA Documentation/devicetree/bindings/processed-schema.json
> DTC_CHK Documentation/devicetree/bindings/sound/fsl,esai.example.dtb
> Executing: make ARCH=arm CROSS_COMPILE=aarch64-linux-gnu- -j8 dt_binding_check DT_SCHEMA_FILES=fsl,esai.yaml
> Executing: make ARCH=arm CROSS_COMPILE=aarch64-linux-gnu- -j8 dt_binding_check DT_SCHEMA_FILES=fsl,esai.yaml
> Successfully rebased and updated refs/heads/b4/esai_arm_dts_warning.
>
> [...]
Applied to
https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next
Thanks!
[1/4] ASoC: dt-bindings: fsl-esai: Remove 'fsl,*' from required list
commit: 5894ff6c4707af645f2faa36cf07cc2745e1658f
[2/4] ASoC: dt-bindings: fsl-esai: Add ref: dai-common.yaml
commit: 9aea6d64bb3acbff6b2f33cb5bbff65e824ab9a7
All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.
You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.
If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.
Please add any relevant lists and maintainers to the CCs when replying
to this mail.
Thanks,
Mark
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v2 0/4] arm: dts: nxp: imx6: fix esai related dtb_check warning
2024-04-16 14:47 ` Frank Li
@ 2024-04-25 6:04 ` Shawn Guo
-1 siblings, 0 replies; 14+ messages in thread
From: Shawn Guo @ 2024-04-25 6:04 UTC (permalink / raw)
To: Frank Li
Cc: Liam Girdwood, Mark Brown, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Shengjiu Wang, Shawn Guo, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam, linux-sound, devicetree,
linux-kernel, imx, linux-arm-kernel
On Tue, Apr 16, 2024 at 10:47:46AM -0400, Frank Li wrote:
> Frank Li (4):
...
> arm: dts: nxp: imx6sx: fix esai related warning when do dtb_check
> arm: dts: nxp: imx6qdl: fix esai clock warning when do dtb_check
We historically use prefix "ARM: dts: ..." for i.MX arm DTS changes.
Fixed them up and applied.
Shawn
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v2 0/4] arm: dts: nxp: imx6: fix esai related dtb_check warning
@ 2024-04-25 6:04 ` Shawn Guo
0 siblings, 0 replies; 14+ messages in thread
From: Shawn Guo @ 2024-04-25 6:04 UTC (permalink / raw)
To: Frank Li
Cc: Liam Girdwood, Mark Brown, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Shengjiu Wang, Shawn Guo, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam, linux-sound, devicetree,
linux-kernel, imx, linux-arm-kernel
On Tue, Apr 16, 2024 at 10:47:46AM -0400, Frank Li wrote:
> Frank Li (4):
...
> arm: dts: nxp: imx6sx: fix esai related warning when do dtb_check
> arm: dts: nxp: imx6qdl: fix esai clock warning when do dtb_check
We historically use prefix "ARM: dts: ..." for i.MX arm DTS changes.
Fixed them up and applied.
Shawn
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2024-04-25 6:04 UTC | newest]
Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-04-16 14:47 [PATCH v2 0/4] arm: dts: nxp: imx6: fix esai related dtb_check warning Frank Li
2024-04-16 14:47 ` Frank Li
2024-04-16 14:47 ` [PATCH v2 1/4] ASoC: dt-bindings: fsl-esai: Remove 'fsl,*' from required list Frank Li
2024-04-16 14:47 ` Frank Li
2024-04-16 14:47 ` [PATCH v2 2/4] ASoC: dt-bindings: fsl-esai: Add ref: dai-common.yaml Frank Li
2024-04-16 14:47 ` Frank Li
2024-04-16 14:47 ` [PATCH v2 3/4] arm: dts: nxp: imx6sx: fix esai related warning when do dtb_check Frank Li
2024-04-16 14:47 ` Frank Li
2024-04-16 14:47 ` [PATCH v2 4/4] arm: dts: nxp: imx6qdl: fix esai clock " Frank Li
2024-04-16 14:47 ` Frank Li
2024-04-17 2:46 ` (subset) [PATCH v2 0/4] arm: dts: nxp: imx6: fix esai related dtb_check warning Mark Brown
2024-04-17 2:46 ` Mark Brown
2024-04-25 6:04 ` Shawn Guo
2024-04-25 6:04 ` Shawn Guo
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.