From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Subject: Re: [PATCH] iio: buffer-dma: Expose data available To: Matthew Fornero Cc: Jonathan Cameron , linux-iio@vger.kernel.org, Matt Fornero References: <1512161180-30116-1-git-send-email-mfornero@gmail.com> <20171202115257.5e561988@archlinux> From: Lars-Peter Clausen Message-ID: Date: Tue, 5 Dec 2017 19:38:43 +0100 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 List-ID: On 12/05/2017 07:11 PM, Matthew Fornero wrote: > On Tue, Dec 5, 2017 at 10:08 AM Lars-Peter Clausen wrote: >>> What about the case when the data_available element in >>> iio_buffer_access_funcs is undefined (e.g. industrialio-buffer-cb)? >>> Do we modify iio_buffer_data_available() to return 0, do we make the >>> sysfs function return -ENOENT or -EINVAL, or simply not expose the sysfs >>> interface for this case? >> >> The callback buffer does not support any of the interfaces required for the >> userspace facing side. E.g. there is no read() callback. >> >> For proper support of the userspace interfaces data_available() is required, >> otherwise read() would never return. So I think this is fine, we'd never >> register the userspacing interface if it didn't have the data_available >> callback. > > So there should already be provisions in place for *not* registering the > cb buffer with user-space, meaning no additional logic would be required > for exposing / not exposing a "data_available" sysfs attribute, correct? Yes.