2024-01-17 05:21:34

by Wadim Egorov

[permalink] [raw]
Subject: [PATCH] dt-bindings: mfd: ti,am3359-tscadc: Make dmas & dma-names optional

The driver does not require dmas or dma-names to be configured for the
ADC to work properly. Remove dmas and dma-names from the required
property list.

Signed-off-by: Wadim Egorov <[email protected]>
---
Documentation/devicetree/bindings/mfd/ti,am3359-tscadc.yaml | 2 --
1 file changed, 2 deletions(-)

diff --git a/Documentation/devicetree/bindings/mfd/ti,am3359-tscadc.yaml b/Documentation/devicetree/bindings/mfd/ti,am3359-tscadc.yaml
index 23a63265be3c..70b5dfce07d2 100644
--- a/Documentation/devicetree/bindings/mfd/ti,am3359-tscadc.yaml
+++ b/Documentation/devicetree/bindings/mfd/ti,am3359-tscadc.yaml
@@ -61,8 +61,6 @@ required:
- interrupts
- clocks
- clock-names
- - dmas
- - dma-names

additionalProperties: false

--
2.25.1



2024-01-17 15:17:11

by Conor Dooley

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: mfd: ti,am3359-tscadc: Make dmas & dma-names optional

On Wed, Jan 17, 2024 at 06:21:04AM +0100, Wadim Egorov wrote:
> The driver does not require dmas or dma-names to be configured for the
> ADC to work properly. Remove dmas and dma-names from the required
> property list.

You say "the driver" - I assume that this is the linux driver. What
about drivers for other operating systems?

Thanks,
Conor.

>
> Signed-off-by: Wadim Egorov <[email protected]>
> ---
> Documentation/devicetree/bindings/mfd/ti,am3359-tscadc.yaml | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/mfd/ti,am3359-tscadc.yaml b/Documentation/devicetree/bindings/mfd/ti,am3359-tscadc.yaml
> index 23a63265be3c..70b5dfce07d2 100644
> --- a/Documentation/devicetree/bindings/mfd/ti,am3359-tscadc.yaml
> +++ b/Documentation/devicetree/bindings/mfd/ti,am3359-tscadc.yaml
> @@ -61,8 +61,6 @@ required:
> - interrupts
> - clocks
> - clock-names
> - - dmas
> - - dma-names
>
> additionalProperties: false
>
> --
> 2.25.1
>

Attachments:
(No filename) (1.01 kB)
 signature.asc (235.00 B)
 Download all attachments

2024-01-18 09:17:27

by Wadim Egorov

[permalink] [raw]
Subject: AW: [PATCH] dt-bindings: mfd: ti,am3359-tscadc: Make dmas & dma-names optional

Please ignore, a similar patch was already accepted

https://lore.kernel.org/linux-arm-kernel/[email protected]/T/
________________________________________
Von: Conor Dooley <[email protected]>
Gesendet: Mittwoch, 17. Januar 2024 16:16:45
An: Wadim Egorov
Cc: [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]
Betreff: Re: [PATCH] dt-bindings: mfd: ti,am3359-tscadc: Make dmas & dma-names optional

On Wed, Jan 17, 2024 at 06:21:04AM +0100, Wadim Egorov wrote:
> The driver does not require dmas or dma-names to be configured for the
> ADC to work properly. Remove dmas and dma-names from the required
> property list.

You say "the driver" - I assume that this is the linux driver. What
about drivers for other operating systems?

Thanks,
Conor.

>
> Signed-off-by: Wadim Egorov <[email protected]>
> ---
> Documentation/devicetree/bindings/mfd/ti,am3359-tscadc.yaml | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/mfd/ti,am3359-tscadc.yaml b/Documentation/devicetree/bindings/mfd/ti,am3359-tscadc.yaml
> index 23a63265be3c..70b5dfce07d2 100644
> --- a/Documentation/devicetree/bindings/mfd/ti,am3359-tscadc.yaml
> +++ b/Documentation/devicetree/bindings/mfd/ti,am3359-tscadc.yaml
> @@ -61,8 +61,6 @@ required:
> - interrupts
> - clocks
> - clock-names
> - - dmas
> - - dma-names
>
> additionalProperties: false
>
> --
> 2.25.1
>