All of lore.kernel.org
 help / color / mirror / Atom feed
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

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