All of lore.kernel.org
 help / color / mirror / Atom feed
From: Antti Palosaari <crope@iki.fi>
To: Devin Heitmueller <dheitmueller@kernellabs.com>
Cc: Jelle de Jong <jelledejong@powercraft.nl>,
	"linux-media@vger.kernel.org" <linux-media@vger.kernel.org>
Subject: Re: Afatech AF9013 DVB-T not working with mplayer radio streams
Date: Sat, 11 Jul 2009 00:12:41 +0300	[thread overview]
Message-ID: <4A57AEC9.9040602@iki.fi> (raw)
In-Reply-To: <829197380907100816o4a3daa22k78a424da5bebed1e@mail.gmail.com>

On 07/10/2009 06:16 PM, Devin Heitmueller wrote:
> On Fri, Jul 10, 2009 at 8:09 AM, Antti Palosaari<crope@iki.fi>  wrote:
>> af9013 is correct in my mind. af9013 will return -EINVAL (error invalid
>> value) in case of first garbage value met (maybe better to switch auto mode
>> when garbage value meet and print debug log?).
>>
>> Of course there should be at least debug printing to inform that... but fix
>> you suggest is better for compatibility. You can do that, it is ok for me.
>
>> From a purist standpoint, I agree that the application at fault, and
> if it were some no-name application I would just say "fix the broken
> application".  Except it's not a no-name application - it's mplayer.
>
> Are you familiar with Postel's Law?

No :)

> http://en.wikipedia.org/wiki/Postel%27s_Law
>
> Saying "this demod is not going to work properly with all versions of
> one of the most popular applications", especially when other demods
> handle the condition gracefully, is the sort of thing that causes real
> problems for the Linux community.
>
> I'm not the maintainer for this demod, so I'm not the best person to
> make such a fix.  I spent four hours and debugged the issue as a favor
> to Jelle de Jong since he loaned me some hardware a couple of months
> ago.  I guess I can make the fix, but it's just going to take away
> from time better spent on things I am more qualified to work on.
>
> Devin

I will fix that just right now. I think I will change demodulator from 
"return error invalid value" to "force detect transmission parameters 
automatically" in case of broken parameters given.

thanks,
Antti
-- 
http://palosaari.fi/

  parent reply	other threads:[~2009-07-10 21:12 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-06-26  8:07 Afatech AF9013 DVB-T not working with mplayer radio streams Jelle de Jong
2009-06-30 20:12 ` Jelle de Jong
2009-07-02  8:43   ` Jelle de Jong
2009-07-02 16:09     ` Devin Heitmueller
2009-07-02 20:51       ` Jelle de Jong
2009-07-02 21:18         ` Devin Heitmueller
2009-07-02 22:44     ` Antti Palosaari
2009-07-02 22:29 ` Antti Palosaari
2009-07-03 16:01   ` Jelle de Jong
2009-07-10  1:05     ` Devin Heitmueller
2009-07-10  8:25       ` Jelle de Jong
2009-07-10 12:09       ` Antti Palosaari
2009-07-10 15:16         ` Devin Heitmueller
2009-07-10 15:40           ` Jelle de Jong
2009-07-10 17:09             ` Devin Heitmueller
2009-07-10 17:09             ` Devin Heitmueller
2009-07-10 21:12           ` Antti Palosaari [this message]
2009-07-10 23:11             ` Antti Palosaari
2009-07-11 10:31               ` Jelle de Jong
2009-07-11 11:48                 ` Jelle de Jong

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=4A57AEC9.9040602@iki.fi \
    --to=crope@iki.fi \
    --cc=dheitmueller@kernellabs.com \
    --cc=jelledejong@powercraft.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.