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/
next prev parent 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).