From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mx1.redhat.com ([209.132.183.28]:51785 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751031Ab2GGSxM (ORCPT ); Sat, 7 Jul 2012 14:53:12 -0400 Message-ID: <4FF885B2.3070509@redhat.com> Date: Sat, 07 Jul 2012 20:53:38 +0200 From: Hans de Goede MIME-Version: 1.0 To: Antti Palosaari 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> <4FF5980F.8030109@iki.fi> <4FF59995.4010604@redhat.com> <4FF5A119.6020903@iki.fi> <4FF5ADE3.5040600@redhat.com> <4FF7EC0E.7060200@redhat.com> <4FF7FAB6.7040508@iki.fi> In-Reply-To: <4FF7FAB6.7040508@iki.fi> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-media-owner@vger.kernel.org List-ID: Hi, On 07/07/2012 11:00 AM, Antti Palosaari wrote: > Hello Hans, > > On 07/07/2012 10:58 AM, Hans de Goede wrote: >> So is your device working properly now? The reason I'm asking it >> because it is still causing a firmware version warning, and if >> it works fine I would like to lower the firmware version warning >> point, so that the warning goes away. > > I don't know what is definition of properly in that case. > > Problem is that when I use radio application from xawtv3 with that new loopback I hear very often cracks and following errors are printed to the radio screen: > ALSA lib pcm.c:7339:(snd_pcm_recover) underrun occurred > or > ALSA lib pcm.c:7339:(snd_pcm_recover) overrun occurred > > Looks like those does not appear, at least it does not crack so often nor errors seen, when I use Rhythmbox to tune and "arecord -D hw:2,0 -r96000 -c2 -f S16_LE | aplay -" to listen. > > I can guess those are not firmware related so warning texts could be removed. Actually they may very well be firmware related. At least with my firmware there is a bug where actively asking the device for its register contents, it lets its audio stream drop. My patches fix this by waiting for the device to volunteer it register contents through its usb interrupt in endpoint, which it does at xx times / sec. So the first question would be, does this dropping of sound happen approx 1 / sec when using radio? If so this is caused by radio upating the signal strength it displays 1 / sec. If you look at radio.c line 981 you will see a radio_getsignal_n_stereo(fd); call there in the main loop which gets called 1/sec. Try commenting this out. If commenting this out fixes your sound issues with radio, then the next question is are you using my 4 recent si470x patches, if not please give them a try. If you are already using them then I'm afraid that your older firmware may be broken even more then my also not so new firmware. Regards, Hans