public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
From: Antti Palosaari <crope@iki.fi>
To: Hans Verkuil <hverkuil@xs4all.nl>, linux-media@vger.kernel.org
Subject: Re: [PATCH] airspy: AirSpy SDR driver
Date: Fri, 18 Jul 2014 17:57:29 +0300	[thread overview]
Message-ID: <53C935D9.50202@iki.fi> (raw)
In-Reply-To: <53C8AAB7.1080305@xs4all.nl>



On 07/18/2014 08:03 AM, Hans Verkuil wrote:
> On 07/18/2014 02:14 AM, Antti Palosaari wrote:
>>
>>
>> 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)
>
> That suggests you were not using the very latest media tree.
>
> But now you'll get a new error: !(dcaps & V4L2_CAP_EXT_PIX_FORMAT)
>
> That's because of a bug in the kernel that I mailed Laurent about.
>
> Perhaps I was a bit too hasty in adding that to v4l2-compliance :-)

Yes, I am stuck on 3.15-rc6 media/fixes as Mauro did not apply patches 
needed from fixes to master. Also, master is about always unusable as it 
is very buggy rc1.

regards
Antti

-- 
http://palosaari.fi/

  reply	other threads:[~2014-07-18 14:57 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-07-14 19:27 [PATCH] airspy: AirSpy SDR driver Antti Palosaari
2014-07-14 19:34 ` Hans Verkuil
2014-07-14 19:55   ` Antti Palosaari
2014-07-14 20:01     ` Hans Verkuil
2014-07-15  2:35       ` Antti Palosaari
2014-07-15  3:50         ` Hans Verkuil
2014-07-18  0:14           ` Antti Palosaari
2014-07-18  5:03             ` Hans Verkuil
2014-07-18 14:57               ` Antti Palosaari [this message]
2014-07-18 15:01                 ` Hans Verkuil
2014-07-18 15:10                   ` Antti Palosaari

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=53C935D9.50202@iki.fi \
    --to=crope@iki.fi \
    --cc=hverkuil@xs4all.nl \
    --cc=linux-media@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox