From: "Torsten Seeboth" <Torsten.Seeboth@t-online.de>
To: "Marton Balint" <cus@fazekas.hu>
Cc: video4linux-list@redhat.com
Subject: Re: [PATCH 0 of 3] cx88: fix oops on rmmod and implement stereodetection
Date: Wed, 26 Mar 2008 21:39:34 +0100 [thread overview]
Message-ID: <001a01c88f81$80aec670$6402a8c0@desktop> (raw)
In-Reply-To: patchbomb.1206497254@bluegene.athome
> The second and the third patches are enhachments of the cx88 audio code, I
> tried to implement the detection of stereo TV channels for A2 mode. I had
> no idea how to detect it, and falling back to EN_A2_AUTO_STEREO instead of
> EN_A2_FORCE_MONO1 did not help either. (The card changed the audio mode
> periodically on both mono and stereo channels) Forcing STEREO mode also
> did not help, because it resulted a loud static noise on mono tv channels.
It's a bug of the audio dsp part.
> Testing proved that AUD_NICAM_STATUS1 and AUD_NICAM_STATUS2 registers
> change randomly if and only if the second audio channel is missing, so if
> these registers are constant (Usually 0x0000 and 0x01), we can assume that
> the
> tv channel has two audio channels, so we can use STEREO mode. This method
> seems a bit ugly, but nicam detection works the same way, so to avoid
> further msleep()-ing, the A2 stereo detection code is in the nicam
> detection
> function.
No, the Nicam_Status_Regs contain only and only if Nicam is forced before.
Your patch _can_ work for B/G, but I don't think so for others, like D/K
etc.
> By the way, the audio thread in the cx88 code is totally useless, in fact,
> it occaisonally sets the audio to MONO after starting a TV application, so
> i think it should be removed. My patch does NOT fix cx88_get_stereo, and
> even if it would, the audio thread would not work as expected, because
> core->audiomode_current is not set in cx88_set_tvaudio, and AUTO stereo
> modes (EN_BTSC_AUTO_STEREO, EN_NICAM_AUTO_STEREO) would also cause
> problems, the autodetected audio mode should be set to
> core->audiomode_current
> to make it work.
Sorry, not an native english speaker here.
If it were easy it would already be done. ;)
I am the one who did that in cx88-tvaudio.c. With Mauro's and many others
help I have moved this part from DScaler project where I am from into this
file.
Still not finished as you can see in comments. There is much more to do if
you want to get safe mono/stereo or a2/nicam/btsc detection.
I did many things trying to understand on how audio things are going on on
this chip. Together witht some others from this list I have learned from m$
driver how it really works by using de-asm/debuggers tools. To make a long
story short: The only way is to take some audio samples into memory, compare
it to tables and do some calculations depands on video-mode.
Torsten
--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list
next prev parent reply other threads:[~2008-03-26 20:39 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-03-26 2:07 [PATCH 0 of 3] cx88: fix oops on rmmod and implement stereo detection Marton Balint
2008-03-26 2:07 ` [PATCH 1 of 3] cx88: fix oops on module removal caused by IR worker Marton Balint
2008-03-28 18:38 ` Mauro Carvalho Chehab
2008-03-26 2:07 ` [PATCH 2 of 3] cx88: fix stereo dematrix for A2 sound system Marton Balint
2008-03-26 2:07 ` [PATCH 3 of 3] cx88: detect stereo output instead of mono fallback in " Marton Balint
2008-03-26 7:02 ` [PATCH 0 of 3] cx88: fix oops on rmmod and implement stereo detection Peter Vágner
2008-03-26 14:40 ` Balint Marton
2008-03-26 17:39 ` Peter Vágner
2008-03-26 19:40 ` Balint Marton
2008-03-28 6:59 ` Peter Vágner
2008-03-26 20:39 ` Torsten Seeboth [this message]
2008-03-26 22:25 ` [PATCH 0 of 3] cx88: fix oops on rmmod and implement stereodetection Balint Marton
2008-04-17 21:35 ` [PATCH 0 of 3] cx88: fix oops on rmmod and implement stereo detection Marton Balint
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='001a01c88f81$80aec670$6402a8c0@desktop' \
--to=torsten.seeboth@t-online.de \
--cc=cus@fazekas.hu \
--cc=video4linux-list@redhat.com \
/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