Re: [PATCH] airspy: AirSpy SDR driver

[Antti Palosaari <crope@iki.fi>]

On 07/15/2014 06:50 AM, Hans Verkuil wrote:
> On 07/15/2014 04:35 AM, Antti Palosaari wrote:
>> On 07/14/2014 11:01 PM, Hans Verkuil wrote:
>>> On 07/14/2014 09:55 PM, Antti Palosaari wrote:
>>>> I actually ran v4l2-compliance and there was problem with ADC band
>>>> enumeration. v4l2-compliance didn't liked as ADC freq was just 20MHz,
>>>> both upper and lower limit. Due to that I added even small hack to driver,
>>>>
>>>> +		.rangelow   = 20000000,
>>>> +		.rangehigh  = 20000001, /* FIXME: make v4l2-compliance happy */
>>>
>>> Hmm, does the latest v4l2-compliance (direct from the git repo) still fail on
>>> that? That shouldn't be a problem, and I don't see that here either if I try that
>>> myself.
>>>
>>> If it still fails, can you show me the error message?
>>
>> [crope@localhost gr-analog]$ ls -l /usr/local/bin/v4l2-compliance
>> -rwxr-xr-x. 1 root root 1497964 Jul 14 22:50 /usr/local/bin/v4l2-compliance
>> [crope@localhost gr-analog]$ /usr/local/bin/v4l2-compliance -S
>> /dev/swradio0 -s
>> Driver Info:
>> 	Driver name   : airspy
>> 	Card type     : AirSpy SDR
>> 	Bus info      : usb-0000:00:13.2-2
>> 	Driver version: 3.15.0
>> 	Capabilities  : 0x85110000
>> 		SDR Capture
>> 		Tuner
>> 		Read/Write
>> 		Streaming
>> 		Device Capabilities
>> 	Device Caps   : 0x05110000
>> 		SDR Capture
>> 		Tuner
>> 		Read/Write
>> 		Streaming
>>
>> Compliance test for device /dev/swradio0 (not using libv4l2):
>>
>> Required ioctls:
>> 	test VIDIOC_QUERYCAP: OK
>>
>> Allow for multiple opens:
>> 	test second sdr open: OK
>> 	test VIDIOC_QUERYCAP: OK
>> 	test VIDIOC_G/S_PRIORITY: OK
>>
>> Debug ioctls:
>> 	test VIDIOC_DBG_G/S_REGISTER: OK
>> 	test VIDIOC_LOG_STATUS: OK
>>
>> Input ioctls:
>> 		fail: v4l2-test-input-output.cpp(107): rangelow >= rangehigh
>> 		fail: v4l2-test-input-output.cpp(190): invalid tuner 0
>> 	test VIDIOC_G/S_TUNER: FAIL
>> 		fail: v4l2-test-input-output.cpp(290): could get frequency for invalid
>
> Try again, it should be fixed now.

Old error has gone, but two new comes:

Compliance test for device /dev/swradio0 (not using libv4l2):

Required ioctls:
		fail: v4l2-compliance.cpp(354): !(caps & V4L2_CAP_EXT_PIX_FORMAT)
	test VIDIOC_QUERYCAP: FAIL

Allow for multiple opens:
	test second sdr open: OK
		fail: v4l2-compliance.cpp(354): !(caps & V4L2_CAP_EXT_PIX_FORMAT)
	test VIDIOC_QUERYCAP: FAIL
	test VIDIOC_G/S_PRIORITY: OK

regards
Antti

-- 
http://palosaari.fi/