On 8/10/24 4:35 AM, Jonathan Cameron wrote: > On Wed, 7 Aug 2024 15:02:10 -0500 > David Lechner <dlechner@xxxxxxxxxxxx> wrote: > >> This implements buffered reads for the ad4695 driver using the typical >> triggered buffer implementation, including adding a soft timestamp >> channel. >> >> The chip has 4 different modes for doing conversions. The driver is >> using the advanced sequencer mode since that is the only mode that >> allows individual configuration of all aspects each channel (e.g. >> bipolar config currently and oversampling to be added in the future). >> >> Signed-off-by: David Lechner <dlechner@xxxxxxxxxxxx> > > Main thing in here is I think you can use available_scan_masks > to avoid the need for the error path on just the temperature channel > being enabled. > I had not thought about doing it that way, but now that I am thinking about it, it seems like we would need to have a scan mask in the list for every possible combination of channels. This would be 10s of thousands of possible scan masks for 16 channel chips so that doesn't seem like the best way to go. But adding some special handling to make the temperature channel just work should be easy enough to add.