From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail.kapsi.fi ([217.30.184.167]:45552 "EHLO mail.kapsi.fi" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750910Ab2GENfT (ORCPT ); Thu, 5 Jul 2012 09:35:19 -0400 Message-ID: <4FF5980F.8030109@iki.fi> Date: Thu, 05 Jul 2012 16:35:11 +0300 From: Antti Palosaari MIME-Version: 1.0 To: Hans de Goede CC: Linux Media Mailing List , hverkuil@xs4all.nl Subject: Re: [PATCH 4/4] radio-si470x: Lower firmware version requirements References: <1339681394-11348-1-git-send-email-hdegoede@redhat.com> <1339681394-11348-4-git-send-email-hdegoede@redhat.com> <4FF45FF7.4020300@iki.fi> <4FF5515A.1030704@redhat.com> In-Reply-To: <4FF5515A.1030704@redhat.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-media-owner@vger.kernel.org List-ID: On 07/05/2012 11:33 AM, Hans de Goede wrote: > Hi, > > On 07/04/2012 05:23 PM, Antti Palosaari wrote: >> 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 >>> --- >>> 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 believe you're doing something wrong ... > >> 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. > > You're not specifying which device arecord should record from so likely > it is taking > the default input of your soundcard (line/mic in), rather then recording > from the > radio device. I tried to define hw:1,0 etc. but only hw:0,0 exists. > Note the latest radio from http://git.linuxtv.org/xawtv3.git will do the > digital loopback of > the sound itself, so try things again with running arecord / aplay, if > you then start radio > and exit again (so that you can see its startup messages) you should see > something like this: > > "Using alsa loopback: cap: hw:1,0 (/dev/radio0), out: default" > > Note radio will automatically select the correct alsa device to record > from for the radio-usb-stick. For some reason I don't see that happening. >> 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... > > I've done my testing with this device: > http://www.tinydeal.com/usb-pcear-radio-recorder-for-pc-p-8603.html > > And that one works, but I think yours should work fine too. Mine device looks just similar. I have bought it maybe 3-4 years ago. Could you check these logs: last commit of xawtv3: commit 2550984180f940c4379651245db468602cb38354 Author: Hans de Goede Date: Thu Jun 21 14:42:42 2012 +0200 Don't call strdup on NULL This fixes radio / xawtv crashing when no associated alsa capture device can be found. Signed-off-by: Hans de Goede **************************************************** Jul 5 16:21:38 localhost kernel: [ 5932.284058] usb 5-2: new full-speed USB device number 2 using ohci_hcd Jul 5 16:21:38 localhost kernel: [ 5932.442821] usb 5-2: New USB device found, idVendor=10c5, idProduct=819a Jul 5 16:21:38 localhost kernel: [ 5932.442825] usb 5-2: New USB device strings: Mfr=1, Product=2, SerialNumber=0 Jul 5 16:21:38 localhost kernel: [ 5932.442827] usb 5-2: Manufacturer: www.rding.cn Jul 5 16:21:38 localhost kernel: [ 5932.467874] Linux media interface: v0.10 Jul 5 16:21:38 localhost kernel: [ 5932.469898] Linux video capture interface: v2.00 Jul 5 16:21:38 localhost kernel: [ 5932.471732] radio-si470x 5-2:1.2: DeviceID=0x1242 ChipID=0x060c Jul 5 16:21:38 localhost kernel: [ 5932.471736] radio-si470x 5-2:1.2: This driver is known to work with firmware version 15, Jul 5 16:21:38 localhost kernel: [ 5932.471738] radio-si470x 5-2:1.2: but the device has firmware version 12. Jul 5 16:21:38 localhost kernel: [ 5932.473736] radio-si470x 5-2:1.2: software version 1, hardware version 7 Jul 5 16:21:38 localhost kernel: [ 5932.473739] radio-si470x 5-2:1.2: This driver is known to work with software version 7, Jul 5 16:21:38 localhost kernel: [ 5932.473741] radio-si470x 5-2:1.2: but the device has software version 1. Jul 5 16:21:38 localhost kernel: [ 5932.473743] radio-si470x 5-2:1.2: If you have some trouble using this driver, Jul 5 16:21:38 localhost kernel: [ 5932.473744] radio-si470x 5-2:1.2: please report to V4L ML at linux-media@vger.kernel.org Jul 5 16:21:38 localhost kernel: [ 5932.561895] usbcore: registered new interface driver radio-si470x Jul 5 16:21:38 localhost kernel: [ 5932.579893] usbcore: registered new interface driver snd-usb-audio [crope@localhost console]$ ./radio -f 90.4 Tuning to 90.40 MHz [crope@localhost console]$ **************************************************** install 4 new radio-si470x patches **************************************************** Jul 5 16:29:36 localhost kernel: [ 6409.204169] usb 5-2: new full-speed USB device number 5 using ohci_hcd Jul 5 16:29:36 localhost kernel: [ 6409.362926] usb 5-2: New USB device found, idVendor=10c5, idProduct=819a Jul 5 16:29:36 localhost kernel: [ 6409.362930] usb 5-2: New USB device strings: Mfr=1, Product=2, SerialNumber=0 Jul 5 16:29:36 localhost kernel: [ 6409.362932] usb 5-2: Manufacturer: www.rding.cn Jul 5 16:29:36 localhost kernel: [ 6409.375781] Linux media interface: v0.10 Jul 5 16:29:36 localhost kernel: [ 6409.378671] Linux video capture interface: v2.00 Jul 5 16:29:36 localhost kernel: [ 6409.396168] usbcore: registered new interface driver snd-usb-audio Jul 5 16:29:36 localhost kernel: [ 6409.397822] radio-si470x 5-2:1.2: DeviceID=0x1242 ChipID=0x060c Jul 5 16:29:36 localhost kernel: [ 6409.397825] radio-si470x 5-2:1.2: This driver is known to work with firmware version 14, Jul 5 16:29:36 localhost kernel: [ 6409.397827] radio-si470x 5-2:1.2: but the device has firmware version 12. Jul 5 16:29:36 localhost kernel: [ 6409.399829] radio-si470x 5-2:1.2: software version 1, hardware version 7 Jul 5 16:29:36 localhost kernel: [ 6409.399832] radio-si470x 5-2:1.2: If you have some trouble using this driver, Jul 5 16:29:36 localhost kernel: [ 6409.399834] radio-si470x 5-2:1.2: please report to V4L ML at linux-media@vger.kernel.org Jul 5 16:29:36 localhost kernel: [ 6409.515805] usbcore: registered new interface driver radio-si470x [crope@localhost console]$ ./radio -f 90.4 Tuning to 90.40 MHz [crope@localhost console]$ If I am not able to get FM-radio working I could guess how hard it is for "normal" user :] regards Antti -- http://palosaari.fi/