Hi Rama, On 10/31/2016 09:12 AM, Rama Krishna Phani A wrote: > Hi Jonathan, > > On 30-Oct-16 10:43 PM, Jonathan Cameron wrote: >> On 27/10/16 18:37, Phani A, Rama Krishna wrote: >>> Hi Stan, >>> >>> On 27-Oct-16 4:48 PM, Stanimir Varbanov wrote: >>>> Hi Rama, >>>> >>>> On 10/26/2016 05:41 PM, Rama Krishna Phani A wrote: >>>>> Polling can also be used for End of conversion completion. >>>>> Implement logic >>>>> to choose either polling or interrupt for End of conversion >>>>> completion. >>>>> Scaling can be done on the voltage to report adc code in physical >>>>> units. >>>>> Add changes to support different scale functions to convert adc >>>>> code to >>>>> physical units. >>>>> >>>>> Signed-off-by: Rama Krishna Phani A <rphani@xxxxxxxxxxxxxx> >>>>> --- >>>>> .../devicetree/bindings/iio/adc/qcom,spmi-vadc.txt | 14 ++ >>>>> drivers/iio/adc/qcom-spmi-vadc.c | 263 >>>>> +++++++++++++++++---- >>>>> 2 files changed, 236 insertions(+), 41 deletions(-) >>>>> >>>>> diff --git >>>>> a/Documentation/devicetree/bindings/iio/adc/qcom,spmi-vadc.txt >>>>> b/Documentation/devicetree/bindings/iio/adc/qcom,spmi-vadc.txt >>>>> index 0fb4613..39e31c0e 100644 >>>>> --- a/Documentation/devicetree/bindings/iio/adc/qcom,spmi-vadc.txt >>>>> +++ b/Documentation/devicetree/bindings/iio/adc/qcom,spmi-vadc.txt >>>>> @@ -37,6 +37,12 @@ VADC node: >>>>> Value type: <prop-encoded-array> >>>>> Definition: End of conversion interrupt. >>>>> >>>>> +- qcom,vadc-poll-eoc: >>>>> + Usage: optional >>>>> + Value type: <bool> >>>>> + Definition: Use polling instead of interrupts for End of >>>>> Conversion >>>>> + completion. >>>> >>>> Why you need to add such a flag in DT? >>>> >>>> The DT should describe hardware details not how the driver will choose >>>> pooling vs interrupt. >>>> >>>> On which use-case you would prefer pooling? >>>> >>> >>> Few PMIC's support interrupt functionality for ADC where as few >>> PMIC's dont support. Based on the functionality that is supported in >>> hardware we choose whether to go for polling or for interrupt. >> Can't use the usual trick of an optional interrupt in DT? >> If it's there we try to use it, if not then fall back to polling? >> > Ok., Will check this logic for implementation and will post next patch. The interrupts DT property in binding doc is marked as optional already, so I can't really understand what you are trying to achieve with this new qcom,vadc-poll-eoc boolean property? -- regards, Stan -- To unsubscribe from this list: send the line "unsubscribe linux-iio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html