linux-media.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Antti Palosaari <crope@iki.fi>
To: Hans de Goede <hdegoede@redhat.com>
Cc: Linux Media Mailing List <linux-media@vger.kernel.org>,
	hverkuil@xs4all.nl
Subject: Re: [PATCH 4/4] radio-si470x: Lower firmware version requirements
Date: Wed, 04 Jul 2012 18:23:35 +0300	[thread overview]
Message-ID: <4FF45FF7.4020300@iki.fi> (raw)
In-Reply-To: <1339681394-11348-4-git-send-email-hdegoede@redhat.com>

On 06/14/2012 04:43 PM, Hans de Goede wrote:
> With the changes from the previous patches device firmware version 14 +
> usb microcontroller software version 1 works fine too.
>
> Signed-off-by: Hans de Goede <hdegoede@redhat.com>
> ---
>   drivers/media/radio/si470x/radio-si470x-usb.c |    2 +-
>   drivers/media/radio/si470x/radio-si470x.h     |    2 +-
>   2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/media/radio/si470x/radio-si470x-usb.c b/drivers/media/radio/si470x/radio-si470x-usb.c
> index 66b1ba8..40b963c 100644
> --- a/drivers/media/radio/si470x/radio-si470x-usb.c
> +++ b/drivers/media/radio/si470x/radio-si470x-usb.c
> @@ -143,7 +143,7 @@ MODULE_PARM_DESC(max_rds_errors, "RDS maximum block errors: *1*");
>    * Software/Hardware Versions from Scratch Page
>    **************************************************************************/
>   #define RADIO_SW_VERSION_NOT_BOOTLOADABLE	6
> -#define RADIO_SW_VERSION			7
> +#define RADIO_SW_VERSION			1
>   #define RADIO_HW_VERSION			1
>
>
> diff --git a/drivers/media/radio/si470x/radio-si470x.h b/drivers/media/radio/si470x/radio-si470x.h
> index fbf713d..b3b612f 100644
> --- a/drivers/media/radio/si470x/radio-si470x.h
> +++ b/drivers/media/radio/si470x/radio-si470x.h
> @@ -189,7 +189,7 @@ struct si470x_device {
>    * Firmware Versions
>    **************************************************************************/
>
> -#define RADIO_FW_VERSION	15
> +#define RADIO_FW_VERSION	14

I just found out this patch serie and tested it - but no luck. Could you 
say if I do something wrong or is it due to my device firmware version?

I compiled radio from http://git.linuxtv.org/xawtv3.git to tune and 
"arecord  -r96000 -c2 -f S16_LE | aplay - " to play sound. Just silent 
white noise is heard.

Jul  4 18:04:33 localhost kernel: [   78.006361] usb 5-2: new full-speed 
USB device number 2 using ohci_hcd
Jul  4 18:04:34 localhost kernel: [   78.165127] usb 5-2: New USB device 
found, idVendor=10c5, idProduct=819a
Jul  4 18:04:34 localhost kernel: [   78.165131] usb 5-2: New USB device 
strings: Mfr=1, Product=2, SerialNumber=0
Jul  4 18:04:34 localhost kernel: [   78.165133] usb 5-2: Manufacturer: 
www.rding.cn
Jul  4 18:04:34 localhost udevd[412]: specified group 'plugdev' unknown
Jul  4 18:04:34 localhost mtp-probe: checking bus 5, device 2: 
"/sys/devices/pci0000:00/0000:00:13.0/usb5/5-2"
Jul  4 18:04:34 localhost mtp-probe: bus: 5, device: 2 was not an MTP device
Jul  4 18:04:34 localhost kernel: [   78.189884] Linux media interface: 
v0.10
Jul  4 18:04:34 localhost kernel: [   78.191940] Linux video capture 
interface: v2.00
Jul  4 18:04:34 localhost kernel: [   78.194058] radio-si470x 5-2:1.2: 
DeviceID=0x1242 ChipID=0x060c
Jul  4 18:04:34 localhost kernel: [   78.194061] radio-si470x 5-2:1.2: 
This driver is known to work with firmware version 14,
Jul  4 18:04:34 localhost kernel: [   78.194062] radio-si470x 5-2:1.2: 
but the device has firmware version 12.
Jul  4 18:04:34 localhost kernel: [   78.196041] radio-si470x 5-2:1.2: 
software version 1, hardware version 7
Jul  4 18:04:34 localhost kernel: [   78.196043] radio-si470x 5-2:1.2: 
If you have some trouble using this driver,
Jul  4 18:04:34 localhost kernel: [   78.196045] radio-si470x 5-2:1.2: 
please report to V4L ML at linux-media@vger.kernel.org
Jul  4 18:04:34 localhost kernel: [   78.339041] usbcore: registered new 
interface driver radio-si470x
Jul  4 18:04:34 localhost kernel: [   78.357056] usbcore: registered new 
interface driver snd-usb-audio


And is there any cheap USB FM-radio device that is known to work on 
Linux ?  I would like to order one...

regards
Antti

-- 
http://palosaari.fi/



  reply	other threads:[~2012-07-04 15:23 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-06-14 13:43 [PATCH 1/4] radio-si470x: Don't unnecesarily read registers on G_TUNER Hans de Goede
2012-06-14 13:43 ` [PATCH 2/4] radio-si470x: Always use interrupt to wait for tune/seek completion Hans de Goede
2012-06-14 13:43 ` [PATCH 3/4] radio-si470x: Lower hardware freq seek signal treshold Hans de Goede
2012-06-14 13:43 ` [PATCH 4/4] radio-si470x: Lower firmware version requirements Hans de Goede
2012-07-04 15:23   ` Antti Palosaari [this message]
2012-07-05  8:33     ` Hans de Goede
2012-07-05 13:35       ` Antti Palosaari
2012-07-05 13:41         ` Hans de Goede
2012-07-05 14:13           ` Antti Palosaari
2012-07-05 15:08             ` Hans de Goede
2012-07-05 15:09               ` Antti Palosaari
2012-07-07  7:58               ` Hans de Goede
2012-07-07  9:00                 ` Antti Palosaari
2012-07-07 18:53                   ` Hans de Goede
2012-07-09  9:17                     ` Antti Palosaari
2012-07-09 10:03                       ` Hans de Goede
2012-07-09 11:58                         ` Antti Palosaari
2012-07-09 13:02                           ` Hans de Goede
2012-07-09 13:34                             ` 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=4FF45FF7.4020300@iki.fi \
    --to=crope@iki.fi \
    --cc=hdegoede@redhat.com \
    --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;
as well as URLs for NNTP newsgroup(s).