All of lore.kernel.org
 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.