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 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.