Re: [PATCH 1/4] staging:iio: Add support for multiple buffers

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

 



On 06/09/2012 02:17 PM, Jonathan Cameron wrote:
>>> [...]
>>> @@ -534,31 +453,186 @@ static int iio_compute_scan_bytes(struct iio_dev *indio_dev, const long *mask,
>>>  	return bytes;
>>>  }
>>>  
>>> -int iio_sw_buffer_preenable(struct iio_dev *indio_dev)
>>> +int iio_update_buffers(struct iio_dev *indio_dev,
>>> +		       struct iio_buffer *insert_buffer,
>>> +		       struct iio_buffer *remove_buffer)
>>>  {
>>> -	struct iio_buffer *buffer = indio_dev->buffer;
>>> -	dev_dbg(&indio_dev->dev, "%s\n", __func__);
>>> +	int ret;
>>
>> drivers/iio/industrialio-buffer.c: In function ‘iio_update_buffers’:
>> drivers/iio/industrialio-buffer.c:460: warning: ‘ret’ may be used
>> uninitialized in this function
>>
>> I think there a missing 'return 0', before the error handling. Right now the
>> code always sets active_scan_mask to NULL.
> The setting to NULL definitely isn't right, but we do still need to
> free the compoundmask.  I'll fix that up.

Only if compoundmask != active_scan_mask
--
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