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

Wadim Egorov posted 1 patch 1 year, 11 months ago
Documentation/devicetree/bindings/mfd/ti,am3359-tscadc.yaml | 2 --
1 file changed, 2 deletions(-)
[PATCH] dt-bindings: mfd: ti,am3359-tscadc: Make dmas & dma-names optional
Posted by Wadim Egorov 1 year, 11 months ago
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 <w.egorov@phytec.de>
---
 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
Re: [PATCH] dt-bindings: mfd: ti,am3359-tscadc: Make dmas & dma-names optional
Posted by Conor Dooley 1 year, 11 months ago
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 <w.egorov@phytec.de>
> ---
>  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
> 
AW: [PATCH] dt-bindings: mfd: ti,am3359-tscadc: Make dmas & dma-names optional
Posted by Wadim Egorov 1 year, 11 months ago
Please ignore, a similar patch was already accepted

https://lore.kernel.org/linux-arm-kernel/20231124045019.21003-1-s-k6@ti.com/T/
________________________________________
Von: Conor Dooley <conor@kernel.org>
Gesendet: Mittwoch, 17. Januar 2024 16:16:45
An: Wadim Egorov
Cc: lee@kernel.org; robh+dt@kernel.org; krzysztof.kozlowski+dt@linaro.org; conor+dt@kernel.org; miquel.raynal@bootlin.com; devicetree@vger.kernel.org; linux-kernel@vger.kernel.org; upstream@lists.phytec.de
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 <w.egorov@phytec.de>
> ---
>  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
>