[PATCH 2/2 v2] devicetree: spi: add spi-mosi-idle-low property support

[PATCH 2/2 v2] devicetree: spi: add spi-mosi-idle-low property support

[charles-antoine.couret]

From: Charles-Antoine Couret <charles-antoine.couret@mind.be>

This flag means that device requires the MOSI line to be low
when it's in idle state.

Signed-off-by: Charles-Antoine Couret <charles-antoine.couret@mind.be>
---
 Documentation/devicetree/bindings/spi/spi-controller.yaml | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/Documentation/devicetree/bindings/spi/spi-controller.yaml b/Documentation/devicetree/bindings/spi/spi-controller.yaml
index 3b8e990e30c4..5f5b143cb27b 100644
--- a/Documentation/devicetree/bindings/spi/spi-controller.yaml
+++ b/Documentation/devicetree/bindings/spi/spi-controller.yaml
@@ -132,6 +132,11 @@ patternProperties:
         description:
           The device requires inverse clock polarity (CPOL) mode.
 
+      spi-mosi-idle-low:
+        $ref: /schemas/types.yaml#/definitions/flag
+        description:
+          The device requires the MOSI line to be low when it's in idle state.
+
     required:
       - compatible
       - reg
-- 
2.53.0

Re: [PATCH 2/2 v2] devicetree: spi: add spi-mosi-idle-low property support

[Marcelo Schmitt]

On 03/29, charles-antoine.couret@mind.be wrote:
> From: Charles-Antoine Couret <charles-antoine.couret@mind.be>
> 
> This flag means that device requires the MOSI line to be low
> when it's in idle state.

Same question asked on the other patch [1] applies here.

By the way, for SPI bindings, the patch subject uses a different prefix order.
See submitting-patches.rst [2].

[1]: https://lore.kernel.org/linux-spi/ack2dPiLpO0uE2VE@debian-BULLSEYE-live-builder-AMD64/
[2]: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/devicetree/bindings/submitting-patches.rst?h=v7.0-rc5#n18

> 
> Signed-off-by: Charles-Antoine Couret <charles-antoine.couret@mind.be>
> ---
>  Documentation/devicetree/bindings/spi/spi-controller.yaml | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/spi/spi-controller.yaml b/Documentation/devicetree/bindings/spi/spi-controller.yaml
> index 3b8e990e30c4..5f5b143cb27b 100644
> --- a/Documentation/devicetree/bindings/spi/spi-controller.yaml
> +++ b/Documentation/devicetree/bindings/spi/spi-controller.yaml
> @@ -132,6 +132,11 @@ patternProperties:
>          description:
>            The device requires inverse clock polarity (CPOL) mode.
>  
> +      spi-mosi-idle-low:
> +        $ref: /schemas/types.yaml#/definitions/flag
> +        description:
> +          The device requires the MOSI line to be low when it's in idle state.
> +
>      required:
>        - compatible
>        - reg
> -- 
> 2.53.0
> 
> 

Re: [PATCH 2/2 v2] devicetree: spi: add spi-mosi-idle-low property support

[Krzysztof Kozlowski]

On 29/03/2026 16:32, Marcelo Schmitt wrote:
> On 03/29, charles-antoine.couret@mind.be wrote:
>> From: Charles-Antoine Couret <charles-antoine.couret@mind.be>
>>
>> This flag means that device requires the MOSI line to be low
>> when it's in idle state.
> 
> Same question asked on the other patch [1] applies here.
> 
> By the way, for SPI bindings, the patch subject uses a different prefix order.
> See submitting-patches.rst [2].
> 
> [1]: https://lore.kernel.org/linux-spi/ack2dPiLpO0uE2VE@debian-BULLSEYE-live-builder-AMD64/
> [2]: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/devicetree/bindings/submitting-patches.rst?h=v7.0-rc5#n18
> 


They also do not use "devicetree" prefix at all. Nowhere...

Best regards,
Krzysztof

Re: [PATCH 2/2 v2] devicetree: spi: add spi-mosi-idle-low property support

[Krzysztof Kozlowski]

On Sun, Mar 29, 2026 at 02:58:32PM +0200, charles-antoine.couret@mind.be wrote:
> From: Charles-Antoine Couret <charles-antoine.couret@mind.be>
> 
> This flag means that device requires the MOSI line to be low
> when it's in idle state.
> 
> Signed-off-by: Charles-Antoine Couret <charles-antoine.couret@mind.be>
> ---
>  Documentation/devicetree/bindings/spi/spi-controller.yaml | 5 +++++
>  1 file changed, 5 insertions(+)


There is no patch 1/2 here, even on mailing lists, so probably this has
messed threading.

Please use subject prefixes matching the subsystem. You can get them for
example with 'git log --oneline -- DIRECTORY_OR_FILE' on the directory
your patch is touching. For bindings, the preferred subjects are
explained here:
https://www.kernel.org/doc/html/latest/devicetree/bindings/submitting-patches.html#i-for-patch-submitters

Best regards,
Krzysztof

Re: [PATCH 2/2 v2] devicetree: spi: add spi-mosi-idle-low property support

[Mark Brown]

[-- Attachment #1: Type: text/plain, Size: 429 bytes --]

On Mon, Mar 30, 2026 at 09:16:56AM +0200, Krzysztof Kozlowski wrote:
> On Sun, Mar 29, 2026 at 02:58:32PM +0200, charles-antoine.couret@mind.be wrote:

> > This flag means that device requires the MOSI line to be low
> > when it's in idle state.

> There is no patch 1/2 here, even on mailing lists, so probably this has
> messed threading.

Yes: https://lore.kernel.org/r/20260329125725.2984756-1-charles-antoine.couret@mind.be

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]