* [PATCH v2 0/2] ARM: st: add new compatible for PWR regulators on STM32MP13
@ 2024-04-26 18:32 ` Patrick Delaunay
0 siblings, 0 replies; 14+ messages in thread
From: Patrick Delaunay @ 2024-04-26 18:32 UTC (permalink / raw
To: Alexandre TORGUE, Liam Girdwood, Mark Brown, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Maxime Coquelin,
Pascal Paillet
Cc: Patrick Delaunay, devicetree, linux-arm-kernel, linux-kernel,
linux-stm32
This patchset adds the new PWR regulators compatible for STM32MP13:
"st,stm32mp13-pwr-reg".
As this node is just introduced by [1] and it is is not used by any
board in Linux, it is the good time to introduced this compatible
and update the STM32MP13 SoC dtsi without ABI break.
A new compatible is needed as the content of the PWR_CR3 register,
used by this driver change with new bits on STM32MP13 for SD IO domain:
- bit 23: VDDSD2VALID
- bit 22: VDDSD1VALID
- bit 16: VDDSD2RDY
- bit 15: VDDSD2EN
- bit 14: VDDSD1RDY
- bit 13: VDDSD1EN
I will push a update on STM32MP13 SoC dtsi if this new compatible
is accepted to preserve the bisectability.
[1] commit f798f7079233 ("ARM: dts: stm32: add PWR regulators support on stm32mp131")
https://lore.kernel.org/linux-arm-kernel/b89d0531-067f-4356-91b0-ed7434cee3d7@foss.st.com/
Changes in v2:
- Add new compatible for STM32MP13 and change title after Rob remarks
V1: "ARM: st: use a correct pwr compatible for stm32mp15"
Patrick Delaunay (2):
dt-bindings: regulator: st,stm32mp1-pwr-reg: add compatible for
STM32MP13
regulator: stm32-pwr: add support of STM32MP13
.../devicetree/bindings/regulator/st,stm32mp1-pwr-reg.yaml | 4 +++-
drivers/regulator/stm32-pwr.c | 1 +
2 files changed, 4 insertions(+), 1 deletion(-)
--
2.25.1
^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH v2 0/2] ARM: st: add new compatible for PWR regulators on STM32MP13
@ 2024-04-26 18:32 ` Patrick Delaunay
0 siblings, 0 replies; 14+ messages in thread
From: Patrick Delaunay @ 2024-04-26 18:32 UTC (permalink / raw
To: Alexandre TORGUE, Liam Girdwood, Mark Brown, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Maxime Coquelin,
Pascal Paillet
Cc: Patrick Delaunay, devicetree, linux-arm-kernel, linux-kernel,
linux-stm32
This patchset adds the new PWR regulators compatible for STM32MP13:
"st,stm32mp13-pwr-reg".
As this node is just introduced by [1] and it is is not used by any
board in Linux, it is the good time to introduced this compatible
and update the STM32MP13 SoC dtsi without ABI break.
A new compatible is needed as the content of the PWR_CR3 register,
used by this driver change with new bits on STM32MP13 for SD IO domain:
- bit 23: VDDSD2VALID
- bit 22: VDDSD1VALID
- bit 16: VDDSD2RDY
- bit 15: VDDSD2EN
- bit 14: VDDSD1RDY
- bit 13: VDDSD1EN
I will push a update on STM32MP13 SoC dtsi if this new compatible
is accepted to preserve the bisectability.
[1] commit f798f7079233 ("ARM: dts: stm32: add PWR regulators support on stm32mp131")
https://lore.kernel.org/linux-arm-kernel/b89d0531-067f-4356-91b0-ed7434cee3d7@foss.st.com/
Changes in v2:
- Add new compatible for STM32MP13 and change title after Rob remarks
V1: "ARM: st: use a correct pwr compatible for stm32mp15"
Patrick Delaunay (2):
dt-bindings: regulator: st,stm32mp1-pwr-reg: add compatible for
STM32MP13
regulator: stm32-pwr: add support of STM32MP13
.../devicetree/bindings/regulator/st,stm32mp1-pwr-reg.yaml | 4 +++-
drivers/regulator/stm32-pwr.c | 1 +
2 files changed, 4 insertions(+), 1 deletion(-)
--
2.25.1
_______________________________________________
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/2] dt-bindings: regulator: st,stm32mp1-pwr-reg: add compatible for STM32MP13
2024-04-26 18:32 ` Patrick Delaunay
@ 2024-04-26 18:32 ` Patrick Delaunay
-1 siblings, 0 replies; 14+ messages in thread
From: Patrick Delaunay @ 2024-04-26 18:32 UTC (permalink / raw
To: Alexandre TORGUE, Liam Girdwood, Mark Brown, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Maxime Coquelin,
Pascal Paillet
Cc: Patrick Delaunay, devicetree, linux-arm-kernel, linux-kernel,
linux-stm32
Add new compatible "st,stm32mp13-pwr-reg" for STM32MP13 SoC family.
Signed-off-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
---
Changes in v2:
- update for Rob review, only add compatible for STM32MP13 family
.../devicetree/bindings/regulator/st,stm32mp1-pwr-reg.yaml | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/Documentation/devicetree/bindings/regulator/st,stm32mp1-pwr-reg.yaml b/Documentation/devicetree/bindings/regulator/st,stm32mp1-pwr-reg.yaml
index c9586d277f41..b0c6c62a84ea 100644
--- a/Documentation/devicetree/bindings/regulator/st,stm32mp1-pwr-reg.yaml
+++ b/Documentation/devicetree/bindings/regulator/st,stm32mp1-pwr-reg.yaml
@@ -11,7 +11,9 @@ maintainers:
properties:
compatible:
- const: st,stm32mp1,pwr-reg
+ oneOf:
+ - const: st,stm32mp1,pwr-reg
+ - const: st,stm32mp13-pwr-reg
reg:
maxItems: 1
--
2.25.1
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH v2 1/2] dt-bindings: regulator: st,stm32mp1-pwr-reg: add compatible for STM32MP13
@ 2024-04-26 18:32 ` Patrick Delaunay
0 siblings, 0 replies; 14+ messages in thread
From: Patrick Delaunay @ 2024-04-26 18:32 UTC (permalink / raw
To: Alexandre TORGUE, Liam Girdwood, Mark Brown, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Maxime Coquelin,
Pascal Paillet
Cc: Patrick Delaunay, devicetree, linux-arm-kernel, linux-kernel,
linux-stm32
Add new compatible "st,stm32mp13-pwr-reg" for STM32MP13 SoC family.
Signed-off-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
---
Changes in v2:
- update for Rob review, only add compatible for STM32MP13 family
.../devicetree/bindings/regulator/st,stm32mp1-pwr-reg.yaml | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/Documentation/devicetree/bindings/regulator/st,stm32mp1-pwr-reg.yaml b/Documentation/devicetree/bindings/regulator/st,stm32mp1-pwr-reg.yaml
index c9586d277f41..b0c6c62a84ea 100644
--- a/Documentation/devicetree/bindings/regulator/st,stm32mp1-pwr-reg.yaml
+++ b/Documentation/devicetree/bindings/regulator/st,stm32mp1-pwr-reg.yaml
@@ -11,7 +11,9 @@ maintainers:
properties:
compatible:
- const: st,stm32mp1,pwr-reg
+ oneOf:
+ - const: st,stm32mp1,pwr-reg
+ - const: st,stm32mp13-pwr-reg
reg:
maxItems: 1
--
2.25.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/2] regulator: stm32-pwr: add support of STM32MP13
2024-04-26 18:32 ` Patrick Delaunay
@ 2024-04-26 18:32 ` Patrick Delaunay
-1 siblings, 0 replies; 14+ messages in thread
From: Patrick Delaunay @ 2024-04-26 18:32 UTC (permalink / raw
To: Alexandre TORGUE, Liam Girdwood, Mark Brown, Maxime Coquelin
Cc: Patrick Delaunay, linux-arm-kernel, linux-kernel, linux-stm32
Add support of the new compatible "st,stm32mp13-pwr-reg" for STM32MP13.
Signed-off-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
---
Changes in v2:
- Add new compatible for STM32MP13 and change title after Rob remarks
V1: "ARM: st: use a correct pwr compatible for stm32mp15"
drivers/regulator/stm32-pwr.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/regulator/stm32-pwr.c b/drivers/regulator/stm32-pwr.c
index 85b0102fb9b1..b7aeef6e09e7 100644
--- a/drivers/regulator/stm32-pwr.c
+++ b/drivers/regulator/stm32-pwr.c
@@ -166,6 +166,7 @@ static int stm32_pwr_regulator_probe(struct platform_device *pdev)
static const struct of_device_id __maybe_unused stm32_pwr_of_match[] = {
{ .compatible = "st,stm32mp1,pwr-reg", },
+ { .compatible = "st,stm32mp13-pwr-reg", },
{},
};
MODULE_DEVICE_TABLE(of, stm32_pwr_of_match);
--
2.25.1
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH v2 2/2] regulator: stm32-pwr: add support of STM32MP13
@ 2024-04-26 18:32 ` Patrick Delaunay
0 siblings, 0 replies; 14+ messages in thread
From: Patrick Delaunay @ 2024-04-26 18:32 UTC (permalink / raw
To: Alexandre TORGUE, Liam Girdwood, Mark Brown, Maxime Coquelin
Cc: Patrick Delaunay, linux-arm-kernel, linux-kernel, linux-stm32
Add support of the new compatible "st,stm32mp13-pwr-reg" for STM32MP13.
Signed-off-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
---
Changes in v2:
- Add new compatible for STM32MP13 and change title after Rob remarks
V1: "ARM: st: use a correct pwr compatible for stm32mp15"
drivers/regulator/stm32-pwr.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/regulator/stm32-pwr.c b/drivers/regulator/stm32-pwr.c
index 85b0102fb9b1..b7aeef6e09e7 100644
--- a/drivers/regulator/stm32-pwr.c
+++ b/drivers/regulator/stm32-pwr.c
@@ -166,6 +166,7 @@ static int stm32_pwr_regulator_probe(struct platform_device *pdev)
static const struct of_device_id __maybe_unused stm32_pwr_of_match[] = {
{ .compatible = "st,stm32mp1,pwr-reg", },
+ { .compatible = "st,stm32mp13-pwr-reg", },
{},
};
MODULE_DEVICE_TABLE(of, stm32_pwr_of_match);
--
2.25.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: [PATCH v2 1/2] dt-bindings: regulator: st,stm32mp1-pwr-reg: add compatible for STM32MP13
2024-04-26 18:32 ` Patrick Delaunay
@ 2024-04-26 19:22 ` Rob Herring
-1 siblings, 0 replies; 14+ messages in thread
From: Rob Herring @ 2024-04-26 19:22 UTC (permalink / raw
To: Patrick Delaunay
Cc: linux-arm-kernel, Alexandre TORGUE, devicetree, Maxime Coquelin,
Liam Girdwood, linux-stm32, Mark Brown, Conor Dooley,
Pascal Paillet, linux-kernel, Krzysztof Kozlowski
On Fri, 26 Apr 2024 20:32:50 +0200, Patrick Delaunay wrote:
> Add new compatible "st,stm32mp13-pwr-reg" for STM32MP13 SoC family.
>
> Signed-off-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
> ---
>
> Changes in v2:
> - update for Rob review, only add compatible for STM32MP13 family
>
> .../devicetree/bindings/regulator/st,stm32mp1-pwr-reg.yaml | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
My bot found errors running 'make dt_binding_check' on your patch:
yamllint warnings/errors:
./Documentation/devicetree/bindings/regulator/st,stm32mp1-pwr-reg.yaml:15:5: [warning] wrong indentation: expected 6 but found 4 (indentation)
dtschema/dtc warnings/errors:
/builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/regulator/st,stm32mp1-pwr-reg.yaml: properties:compatible:oneOf: [{'const': 'st,stm32mp1,pwr-reg'}, {'const': 'st,stm32mp13-pwr-reg'}] should not be valid under {'items': {'propertyNames': {'const': 'const'}, 'required': ['const']}}
hint: Use 'enum' rather than 'oneOf' + 'const' entries
from schema $id: http://devicetree.org/meta-schemas/keywords.yaml#
doc reference errors (make refcheckdocs):
See https://patchwork.ozlabs.org/project/devicetree-bindings/patch/20240426203245.v2.1.Ia0a99d90acb512aa020a6e7a8cca8cc1b71f1759@changeid
The base for the series is generally the latest rc1. A different dependency
should be noted in *this* patch.
If you already ran 'make dt_binding_check' and didn't see the above
error(s), then make sure 'yamllint' is installed and dt-schema is up to
date:
pip3 install dtschema --upgrade
Please check and re-submit after running the above command yourself. Note
that DT_SCHEMA_FILES can be set to your schema file to speed up checking
your schema. However, it must be unset to test all examples with your schema.
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v2 1/2] dt-bindings: regulator: st,stm32mp1-pwr-reg: add compatible for STM32MP13
@ 2024-04-26 19:22 ` Rob Herring
0 siblings, 0 replies; 14+ messages in thread
From: Rob Herring @ 2024-04-26 19:22 UTC (permalink / raw
To: Patrick Delaunay
Cc: linux-arm-kernel, Alexandre TORGUE, devicetree, Maxime Coquelin,
Liam Girdwood, linux-stm32, Mark Brown, Conor Dooley,
Pascal Paillet, linux-kernel, Krzysztof Kozlowski
On Fri, 26 Apr 2024 20:32:50 +0200, Patrick Delaunay wrote:
> Add new compatible "st,stm32mp13-pwr-reg" for STM32MP13 SoC family.
>
> Signed-off-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
> ---
>
> Changes in v2:
> - update for Rob review, only add compatible for STM32MP13 family
>
> .../devicetree/bindings/regulator/st,stm32mp1-pwr-reg.yaml | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
My bot found errors running 'make dt_binding_check' on your patch:
yamllint warnings/errors:
./Documentation/devicetree/bindings/regulator/st,stm32mp1-pwr-reg.yaml:15:5: [warning] wrong indentation: expected 6 but found 4 (indentation)
dtschema/dtc warnings/errors:
/builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/regulator/st,stm32mp1-pwr-reg.yaml: properties:compatible:oneOf: [{'const': 'st,stm32mp1,pwr-reg'}, {'const': 'st,stm32mp13-pwr-reg'}] should not be valid under {'items': {'propertyNames': {'const': 'const'}, 'required': ['const']}}
hint: Use 'enum' rather than 'oneOf' + 'const' entries
from schema $id: http://devicetree.org/meta-schemas/keywords.yaml#
doc reference errors (make refcheckdocs):
See https://patchwork.ozlabs.org/project/devicetree-bindings/patch/20240426203245.v2.1.Ia0a99d90acb512aa020a6e7a8cca8cc1b71f1759@changeid
The base for the series is generally the latest rc1. A different dependency
should be noted in *this* patch.
If you already ran 'make dt_binding_check' and didn't see the above
error(s), then make sure 'yamllint' is installed and dt-schema is up to
date:
pip3 install dtschema --upgrade
Please check and re-submit after running the above command yourself. Note
that DT_SCHEMA_FILES can be set to your schema file to speed up checking
your schema. However, it must be unset to test all examples with your schema.
_______________________________________________
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 1/2] dt-bindings: regulator: st,stm32mp1-pwr-reg: add compatible for STM32MP13
2024-04-26 18:32 ` Patrick Delaunay
@ 2024-04-29 5:11 ` Krzysztof Kozlowski
-1 siblings, 0 replies; 14+ messages in thread
From: Krzysztof Kozlowski @ 2024-04-29 5:11 UTC (permalink / raw
To: Patrick Delaunay, Alexandre TORGUE, Liam Girdwood, Mark Brown,
Rob Herring, Krzysztof Kozlowski, Conor Dooley, Maxime Coquelin,
Pascal Paillet
Cc: devicetree, linux-arm-kernel, linux-kernel, linux-stm32
On 26/04/2024 20:32, Patrick Delaunay wrote:
> Add new compatible "st,stm32mp13-pwr-reg" for STM32MP13 SoC family.
>
> Signed-off-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
> ---
>
> Changes in v2:
> - update for Rob review, only add compatible for STM32MP13 family
Please start testing your patches BEFORE sending.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v2 1/2] dt-bindings: regulator: st,stm32mp1-pwr-reg: add compatible for STM32MP13
@ 2024-04-29 5:11 ` Krzysztof Kozlowski
0 siblings, 0 replies; 14+ messages in thread
From: Krzysztof Kozlowski @ 2024-04-29 5:11 UTC (permalink / raw
To: Patrick Delaunay, Alexandre TORGUE, Liam Girdwood, Mark Brown,
Rob Herring, Krzysztof Kozlowski, Conor Dooley, Maxime Coquelin,
Pascal Paillet
Cc: devicetree, linux-arm-kernel, linux-kernel, linux-stm32
On 26/04/2024 20:32, Patrick Delaunay wrote:
> Add new compatible "st,stm32mp13-pwr-reg" for STM32MP13 SoC family.
>
> Signed-off-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
> ---
>
> Changes in v2:
> - update for Rob review, only add compatible for STM32MP13 family
Please start testing your patches BEFORE sending.
Best regards,
Krzysztof
_______________________________________________
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 1/2] dt-bindings: regulator: st,stm32mp1-pwr-reg: add compatible for STM32MP13
2024-04-29 5:11 ` Krzysztof Kozlowski
@ 2024-05-13 9:46 ` Patrick DELAUNAY
-1 siblings, 0 replies; 14+ messages in thread
From: Patrick DELAUNAY @ 2024-05-13 9:46 UTC (permalink / raw
To: Krzysztof Kozlowski, Alexandre TORGUE, Liam Girdwood, Mark Brown,
Rob Herring, Krzysztof Kozlowski, Conor Dooley, Maxime Coquelin,
Pascal Paillet
Cc: devicetree, linux-arm-kernel, linux-kernel, linux-stm32
On 4/29/24 07:11, Krzysztof Kozlowski wrote:
> On 26/04/2024 20:32, Patrick Delaunay wrote:
>> Add new compatible "st,stm32mp13-pwr-reg" for STM32MP13 SoC family.
>>
>> Signed-off-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
>> ---
>>
>> Changes in v2:
>> - update for Rob review, only add compatible for STM32MP13 family
> Please start testing your patches BEFORE sending.
Sorry,
I don't know why I don't see the issue even if I executed
make dt_binding_check SCHEMA_FILES=stm32
I will sent a V3
and I update my way of working to avoid this error
=> clean the output biding directory before to execute again
"dt_binding_check"
>
> Best regards,
> Krzysztof
Regards
Patrick
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v2 1/2] dt-bindings: regulator: st,stm32mp1-pwr-reg: add compatible for STM32MP13
@ 2024-05-13 9:46 ` Patrick DELAUNAY
0 siblings, 0 replies; 14+ messages in thread
From: Patrick DELAUNAY @ 2024-05-13 9:46 UTC (permalink / raw
To: Krzysztof Kozlowski, Alexandre TORGUE, Liam Girdwood, Mark Brown,
Rob Herring, Krzysztof Kozlowski, Conor Dooley, Maxime Coquelin,
Pascal Paillet
Cc: devicetree, linux-arm-kernel, linux-kernel, linux-stm32
On 4/29/24 07:11, Krzysztof Kozlowski wrote:
> On 26/04/2024 20:32, Patrick Delaunay wrote:
>> Add new compatible "st,stm32mp13-pwr-reg" for STM32MP13 SoC family.
>>
>> Signed-off-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
>> ---
>>
>> Changes in v2:
>> - update for Rob review, only add compatible for STM32MP13 family
> Please start testing your patches BEFORE sending.
Sorry,
I don't know why I don't see the issue even if I executed
make dt_binding_check SCHEMA_FILES=stm32
I will sent a V3
and I update my way of working to avoid this error
=> clean the output biding directory before to execute again
"dt_binding_check"
>
> Best regards,
> Krzysztof
Regards
Patrick
_______________________________________________
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 1/2] dt-bindings: regulator: st,stm32mp1-pwr-reg: add compatible for STM32MP13
2024-05-13 9:46 ` Patrick DELAUNAY
@ 2024-05-13 11:56 ` Krzysztof Kozlowski
-1 siblings, 0 replies; 14+ messages in thread
From: Krzysztof Kozlowski @ 2024-05-13 11:56 UTC (permalink / raw
To: Patrick DELAUNAY, Alexandre TORGUE, Liam Girdwood, Mark Brown,
Rob Herring, Krzysztof Kozlowski, Conor Dooley, Maxime Coquelin,
Pascal Paillet
Cc: devicetree, linux-arm-kernel, linux-kernel, linux-stm32
On 13/05/2024 11:46, Patrick DELAUNAY wrote:
>
> On 4/29/24 07:11, Krzysztof Kozlowski wrote:
>> On 26/04/2024 20:32, Patrick Delaunay wrote:
>>> Add new compatible "st,stm32mp13-pwr-reg" for STM32MP13 SoC family.
>>>
>>> Signed-off-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
>>> ---
>>>
>>> Changes in v2:
>>> - update for Rob review, only add compatible for STM32MP13 family
>> Please start testing your patches BEFORE sending.
>
>
> Sorry,
>
> I don't know why I don't see the issue even if I executed
>
> make dt_binding_check SCHEMA_FILES=stm32
The code is incorrect, which is easily visible by the diff. dtschema
detects it so, your setup is probably not complete, if you claim you run
it before sending. writing-schema doc has short guide how to setup, but
that's basically only one command - pip install dtschema
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v2 1/2] dt-bindings: regulator: st,stm32mp1-pwr-reg: add compatible for STM32MP13
@ 2024-05-13 11:56 ` Krzysztof Kozlowski
0 siblings, 0 replies; 14+ messages in thread
From: Krzysztof Kozlowski @ 2024-05-13 11:56 UTC (permalink / raw
To: Patrick DELAUNAY, Alexandre TORGUE, Liam Girdwood, Mark Brown,
Rob Herring, Krzysztof Kozlowski, Conor Dooley, Maxime Coquelin,
Pascal Paillet
Cc: devicetree, linux-arm-kernel, linux-kernel, linux-stm32
On 13/05/2024 11:46, Patrick DELAUNAY wrote:
>
> On 4/29/24 07:11, Krzysztof Kozlowski wrote:
>> On 26/04/2024 20:32, Patrick Delaunay wrote:
>>> Add new compatible "st,stm32mp13-pwr-reg" for STM32MP13 SoC family.
>>>
>>> Signed-off-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
>>> ---
>>>
>>> Changes in v2:
>>> - update for Rob review, only add compatible for STM32MP13 family
>> Please start testing your patches BEFORE sending.
>
>
> Sorry,
>
> I don't know why I don't see the issue even if I executed
>
> make dt_binding_check SCHEMA_FILES=stm32
The code is incorrect, which is easily visible by the diff. dtschema
detects it so, your setup is probably not complete, if you claim you run
it before sending. writing-schema doc has short guide how to setup, but
that's basically only one command - pip install dtschema
Best regards,
Krzysztof
_______________________________________________
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-05-13 11:57 UTC | newest]
Thread overview: 14+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-04-26 18:32 [PATCH v2 0/2] ARM: st: add new compatible for PWR regulators on STM32MP13 Patrick Delaunay
2024-04-26 18:32 ` Patrick Delaunay
2024-04-26 18:32 ` [PATCH v2 1/2] dt-bindings: regulator: st,stm32mp1-pwr-reg: add compatible for STM32MP13 Patrick Delaunay
2024-04-26 18:32 ` Patrick Delaunay
2024-04-26 19:22 ` Rob Herring
2024-04-26 19:22 ` Rob Herring
2024-04-29 5:11 ` Krzysztof Kozlowski
2024-04-29 5:11 ` Krzysztof Kozlowski
2024-05-13 9:46 ` Patrick DELAUNAY
2024-05-13 9:46 ` Patrick DELAUNAY
2024-05-13 11:56 ` Krzysztof Kozlowski
2024-05-13 11:56 ` Krzysztof Kozlowski
2024-04-26 18:32 ` [PATCH v2 2/2] regulator: stm32-pwr: add support of STM32MP13 Patrick Delaunay
2024-04-26 18:32 ` Patrick Delaunay
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.