> On Nov 9, 2016, at 9:07 PM, Vignesh R <vigne...@ti.com> wrote:
>
> Hi,
>
> On Thursday 10 November 2016 05:23 AM, John Syne wrote:
>> OK, then back to my original question. Given that these DT properties are
>> supported in the driver
>>
>
> Below properties are supported by only by ti_am3335x_adc driver and not
> ti_am335x_tsc driver. As author of this patch pointed out in another
> reply, there is no need to change step-opendelay for tsc. AFAIK, I don't
> see a use case where these values needs to be tweaked for tsc channels,
> therefore it does not make sense to be DT properties.
Yeah, the confusion was mine because the author of this patch series was
proposing to hard code these settings while DT properties already existed in
the ti_am335x_adc driver. I use the ADC for sensor measurement and do not use
the touchscreen features. I already pointed out where these DT parameters
should be added by referencing how this was done in one of the BBB DT overlay
files [1]. I am just proposing the same is done as a default in the AM33xx.dtsi
and AM4372.dtsi files. Here is what I was proposing. Granted, the adc-channels
should be restricted to the subset of channels not used by tsc, but you get the
idea.
tscadc: tscadc@44e0d000 {
compatible = "ti,am3359-tscadc";
reg = <0x44e0d000 0x1000>;
interrupt-parent = <&intc>;
interrupts = <16>;
ti,hwmods = "adc_tsc";
status = "disabled";
dmas = <&edma 53 0>, <&edma 57 0>;
dma-names = "fifo0", "fifo1”;
tsc {
compatible = "ti,am3359-tsc";
};
am335x_adc: adc {
#io-channel-cells = <1>;
ti,adc-channels = <0 1 2 3 4 5 6>;
ti,chan-step-avg = <0x16 0x16 0x16 0x16 0x16
0x16 0x16>;
ti,chan-step-opendelay = <0x98 0x98 0x98 0x98
0x98 0x98 0x98>;
ti,chan-step-sampledelay = <0x0 0x0 0x0 0x0 0x0
0x0 0x0>;
compatible = "ti,am3359-adc";
};
};
[1]https://github.com/RobertCNelson/bb.org-overlays/blob/master/src/arm/BB-ADC-00A0.dts
Regards,
John
>
>> shouldn’t the following be added to am33xx.dtsi and am4372.dtsi?
>
> Its totally upto board dts files to allocate channels for tsc and adc.
> So, how could these be added to dtsi files?
>
>> ti,chan-step-avg = <0x16 0x16 0x16 0x16 0x16 0x16 0x16>;
>> ti,chan-step-opendelay = <0x500 0x500 0x500 0x500 0x500 0x500 0x500>;
>> ti,chan-step-sampledelay = <0x0 0x0 0x0 0x0 0x0 0x0 0x0>;
>>
>> Regards,
>> John
>>>
>>> --
>>> Regards
>>> Vignesh
>>
>
> --
> Regards
> Vignesh
