Re: [PATCH 4/4] radio-si470x: Lower firmware version requirements

[Hans de Goede <hdegoede@redhat.com>]

Hi,

On 07/09/2012 01:58 PM, Antti Palosaari wrote:
> On 07/09/2012 01:03 PM, Hans de Goede wrote:
> If I tune using old radio it works. If I tune using latest radio but with option -l 0 (./console/radio -l 0) it also works. Using "arecord -D hw:2,0 -r96000 -c2 -f S16_LE | aplay -" to listen. So it seems that latest radio with alsa loopback is only having those problems.
>

Ok.
>
> These are the patches:
> radio-si470x: Don't unnecesarily read registers on G_TUNER
> radio-si470x: Lower hardware freq seek signal treshold
> radio-si470x: Always use interrupt to wait for tune/seek completion
> radio-si470x: Lower firmware version requirements
>
> And from that I can see it loads new driver as it does not warn about software version - only firmware.

Right, so what I want to do is to lower the firmware requirement to 12, so that it won't complain
for your device since that seems to be working fine. Does that sound like a good idea to you?

> Jul  9 14:28:29 localhost kernel: [13403.017920] Linux media interface: v0.10
> Jul  9 14:28:29 localhost kernel: [13403.020866] Linux video capture interface: v2.00
> Jul  9 14:28:29 localhost kernel: [13403.022744] radio-si470x 5-2:1.2: DeviceID=0x1242 ChipID=0x060c
> Jul  9 14:28:29 localhost kernel: [13403.022747] radio-si470x 5-2:1.2: This driver is known to work with firmware version 14,
> Jul  9 14:28:29 localhost kernel: [13403.022749] radio-si470x 5-2:1.2: but the device has firmware version 12.
> Jul  9 14:28:29 localhost kernel: [13403.024715] radio-si470x 5-2:1.2: software version 1, hardware version 7
> Jul  9 14:28:29 localhost kernel: [13403.024717] radio-si470x 5-2:1.2: If you have some trouble using this driver,
> Jul  9 14:28:29 localhost kernel: [13403.024719] radio-si470x 5-2:1.2: please report to V4L ML at linux-media@vger.kernel.org
> Jul  9 14:28:29 localhost kernel: [13403.114583] usbcore: registered new interface driver radio-si470x

Regards,

Hans