Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1613961 > unrolled thread

[PATCH v2 0/2] iio: allow to set STM32 ADC resolution

Started byFabrice Gasnier <fabrice.gasnier@st.com>
First post2017-03-31 14:40 +0200
Last post2017-04-03 21:50 +0200
Articles 4 — 3 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH v2 0/2] iio: allow to set STM32 ADC resolution  Fabrice Gasnier <fabrice.gasnier@st.com> - 2017-03-31 14:40 +0200
    [PATCH v2 1/2] dt-bindings: iio: stm32-adc: add option to set resolution Fabrice Gasnier <fabrice.gasnier@st.com> - 2017-03-31 14:40 +0200
      Re: [PATCH v2 1/2] dt-bindings: iio: stm32-adc: add option to set  resolution Rob Herring <robh@kernel.org> - 2017-04-03 18:50 +0200
        Re: [PATCH v2 1/2] dt-bindings: iio: stm32-adc: add option to set  resolution Jonathan Cameron <jic23@kernel.org> - 2017-04-03 21:50 +0200

#1613961 — [PATCH v2 0/2] iio: allow to set STM32 ADC resolution

FromFabrice Gasnier <fabrice.gasnier@st.com>
Date2017-03-31 14:40 +0200
Subject[PATCH v2 0/2] iio: allow to set STM32 ADC resolution
Message-ID<tr40p-4Bg-7@gated-at.bofh.it>
STM32 ADC supports several resolution. Add dt option so resolution
can be tuned at probe time. By default, maximum resolution is used
when it isn't set.

---
Changes in v2:
 - rename property from 'st,adc-res' to 'assigned-resolution-bits' so it
 is more common.

Fabrice Gasnier (2):
  dt-bindings: iio: stm32-adc: add option to set resolution
  iio: adc: stm32: add dt option to set resolution

 .../devicetree/bindings/iio/adc/st,stm32-adc.txt   |  4 ++
 drivers/iio/adc/stm32-adc.c                        | 50 +++++++++++++++++++++-
 2 files changed, 53 insertions(+), 1 deletion(-)

-- 
1.9.1

[toc] | [next] | [standalone]


#1613962 — [PATCH v2 1/2] dt-bindings: iio: stm32-adc: add option to set resolution

FromFabrice Gasnier <fabrice.gasnier@st.com>
Date2017-03-31 14:40 +0200
Subject[PATCH v2 1/2] dt-bindings: iio: stm32-adc: add option to set resolution
Message-ID<tr40q-4Bg-15@gated-at.bofh.it>
In reply to#1613961
Add documentation for 'assigned-resolution-bits' dt optional property.

Signed-off-by: Fabrice Gasnier <fabrice.gasnier@st.com>
---
Changes in v2:
- rename property from 'st,adc-res' to 'assigned-resolution-bits' so it
  is more common.
---
 Documentation/devicetree/bindings/iio/adc/st,stm32-adc.txt | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/Documentation/devicetree/bindings/iio/adc/st,stm32-adc.txt b/Documentation/devicetree/bindings/iio/adc/st,stm32-adc.txt
index 5dfc88e..e35f9f1 100644
--- a/Documentation/devicetree/bindings/iio/adc/st,stm32-adc.txt
+++ b/Documentation/devicetree/bindings/iio/adc/st,stm32-adc.txt
@@ -57,6 +57,9 @@ Optional properties:
 - dmas: Phandle to dma channel for this ADC instance.
   See ../../dma/dma.txt for details.
 - dma-names: Must be "rx" when dmas property is being used.
+- assigned-resolution-bits: Resolution (bits) to use for conversions. Must
+  match device available resolutions (e.g. can be 6, 8, 10 or 12 on stm32f4).
+  Default is maximum resolution if unset.
 
 Example:
 	adc: adc@40012000 {
@@ -84,6 +87,7 @@ Example:
 			st,adc-channels = <8>;
 			dmas = <&dma2 0 0 0x400 0x0>;
 			dma-names = "rx";
+			assigned-resolution-bits = <8>;
 		};
 		...
 		other adc child nodes follow...
-- 
1.9.1

[toc] | [prev] | [next] | [standalone]


#1615412 — Re: [PATCH v2 1/2] dt-bindings: iio: stm32-adc: add option to set resolution

FromRob Herring <robh@kernel.org>
Date2017-04-03 18:50 +0200
SubjectRe: [PATCH v2 1/2] dt-bindings: iio: stm32-adc: add option to set resolution
Message-ID<tsdl0-Vb-25@gated-at.bofh.it>
In reply to#1613962
On Fri, Mar 31, 2017 at 02:32:37PM +0200, Fabrice Gasnier wrote:
> Add documentation for 'assigned-resolution-bits' dt optional property.
> 
> Signed-off-by: Fabrice Gasnier <fabrice.gasnier@st.com>
> ---
> Changes in v2:
> - rename property from 'st,adc-res' to 'assigned-resolution-bits' so it
>   is more common.
> ---
>  Documentation/devicetree/bindings/iio/adc/st,stm32-adc.txt | 4 ++++
>  1 file changed, 4 insertions(+)

Acked-by: Rob Herring <robh@kernel.org> 

[toc] | [prev] | [next] | [standalone]


#1615513 — Re: [PATCH v2 1/2] dt-bindings: iio: stm32-adc: add option to set resolution

FromJonathan Cameron <jic23@kernel.org>
Date2017-04-03 21:50 +0200
SubjectRe: [PATCH v2 1/2] dt-bindings: iio: stm32-adc: add option to set resolution
Message-ID<tsg9b-2Mf-1@gated-at.bofh.it>
In reply to#1615412
On 03/04/17 17:46, Rob Herring wrote:
> On Fri, Mar 31, 2017 at 02:32:37PM +0200, Fabrice Gasnier wrote:
>> Add documentation for 'assigned-resolution-bits' dt optional property.
>>
>> Signed-off-by: Fabrice Gasnier <fabrice.gasnier@st.com>
>> ---
>> Changes in v2:
>> - rename property from 'st,adc-res' to 'assigned-resolution-bits' so it
>>   is more common.
>> ---
>>  Documentation/devicetree/bindings/iio/adc/st,stm32-adc.txt | 4 ++++
>>  1 file changed, 4 insertions(+)
> 
> Acked-by: Rob Herring <robh@kernel.org> 
Thanks.

Applied to the togreg branch of iio.git.  Will push out as testing for
the autobuilders to have a chance to play with it.

Jonathan
> --
> To unsubscribe from this list: send the line "unsubscribe linux-iio" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web