On 06/08/2013 11:24 AM, Jonathan Cameron wrote:
> On 06/05/2013 05:24 PM, Sebastian Andrzej Siewior wrote:
>> The TSC part allows to specify the input lines. The IIO part assumes
>> that it usues always the last few, that means if IIO has adc-channels
>> set to 2 it will use channel 6 and 7. However it might make sense to use
>> only 6.
>> This patch changes the device property (which was introduced recently
>> and was never in an official release) in a way that the user can specify
>> which of the AIN lines should be used. In Addition to this, the name is
>> now AINx where x is the channel number i.e. for AIN6 we would have 6.
>> Prior this, it always started counting at 0 which is confusing. In
>> addition to this, it also checks for correct step number during reading
>> and does not rely on proper FIFO depth.
>>
> Looks like a typo below... Also the change that is in doesn't directly seem
> to be described in this description.
You can use up to 8 channels and it was checked that the user does not
specify more. However, in case of the touch you can read a channel
multiple times. Each read process is programmed in the FSM and referred
as a step in the manual. You can have up to 15 steps. The additional
check ensures that the user does not try to more "steps" then available.
>> diff --git a/drivers/mfd/ti_am335x_tscadc.c b/drivers/mfd/ti_am335x_tscadc.c
>> index 8bdab8d..acdb7b9 100644
>> --- a/drivers/mfd/ti_am335x_tscadc.c
>> +++ b/drivers/mfd/ti_am335x_tscadc.c
>> @@ -91,10 +95,17 @@ static int ti_tscadc_probe(struct platform_device *pdev)
>>
>> node = of_get_child_by_name(pdev->dev.of_node, "tsc");
>> of_property_read_u32(node, "ti,wires", &tsc_wires);
>> + of_property_read_u32(node, "ti,coordiante-readouts", &readouts);
> Coordiante? I'm guessing coordinate
You guess correctly. It is a typo in multiple files, I fix it up…
>>
>> node = of_get_child_by_name(pdev->dev.of_node, "adc");
>> - of_property_read_u32(node, "ti,adc-channels", &adc_channels);
>> -
>> + of_property_for_each_u32(node, "ti,adc-channels", prop, cur, val) {
>> + adc_channels++;
>> + if (val > 7) {
>> + dev_err(&pdev->dev, " PIN numbers are 0..7 (not %d)\n",
>> + val);
>> + return -EINVAL;
>> + }
>> + }
>> total_channels = tsc_wires + adc_channels;
>> if (total_channels > 8) {
>> dev_err(&pdev->dev, "Number of i/p channels more than 8\n");
Sebastian
--
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
