Re: IIO:ADC Scan Modes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Michael
> The way available scan modes are handled in the max1363 driver doesn't
> nicely equate to the AD7997, AD7997 or AD7991 series of ADCs.
> These devices feature up to 8 Channels while each channel can be independently
> included into the scan sequence.
> (this would require up to 256 'named' modes)
I agree entirely.  The approach originally used in max1363 has gone in the latest
patch set posted the other day (indeed the set you reference below)
> 
> A while ago we talked about this - and you said you would abandon the
> 'named' scan mode selection - and come up with something like a scan_elements
> directory with channel control files (enable/disable), and some sort of mechanism
> that rejects invalid combinations.
> 
> I noticed some additions ...
> [PATCH 04/19] staging:iio: Support functions for scan mask matching
Yup.  That's what max1363 now does. In the same series, see
[PATCH 08/19] staging:iio:max1363 move to new abi.

(as the new abi didn't allow the 'named' scan mode approach!)

Other than some renaming, the lis3l20dq for example has done the scan_elements
approach from the start (as have some of the others.)

Jonathan


> 
> But the sysfs interface is not in place.
> Is this something you're still planning to put into place?
> 
> Best regards,
> Michael
> 
> 
> ------------------------------------------------------------------
> ********* Analog Devices GmbH              Open Platform Solutions
> **  *****
> **     ** Wilhelm-Wagenfeld-Strasse 6
> **  ***** D-80807 Munich
> ********* Germany
> Registergericht München HRB 40368,  Geschäftsführer: Thomas Wessel, William A. Martin, Margaret K. Seif
> 
> 

--
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

[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux