Re: [PATCH 3/5] tm6000: add audio mode parameter

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Am 03.05.2011 16:52, schrieb Stefan Ringel:
Am 03.05.2011 16:42, schrieb Mauro Carvalho Chehab:
Em 03-05-2011 11:25, Stefan Ringel arcor escreveu:
Am 03.05.2011 12:07, schrieb Mauro Carvalho Chehab:
Em 20-04-2011 05:36, Stefan Ringel escreveu:
Am 19.04.2011 23:26, schrieb Mauro Carvalho Chehab:
Em 04-04-2011 17:18, stefan.ringel@xxxxxxxx escreveu:
From: Stefan Ringel<stefan.ringel@xxxxxxxx>

add audio mode parameter
Why we need a parameter for it? It should be determined based on
the standard.

tm6010 has a sif decoder, and I think if auto detect doesn't work, use can set the audio standard, which it has in your region. Or it's better if users can see image but can hear audio?
I did some tests with SIF and MTS here. None of them were capable of working with BTSC signals with my devices. Adding a parameter won't help it at all. What we need to do is to fix the audio
decoding.

In the next patch I will send it. A preview I have send to be test (
https://patchwork.kernel.org/patch/722021/ ).
I tested your preview. Didn't make any difference.
Has you test with all setting variants? (BG_A2 works auto and A2 audio mode).
I use mplayer, and have in the first use the wrong parameter. Now I know the right parameter for mplayer:

mplayer -vc rawyuy2 -ac pcm -ao alsa:device=hw=0.0 -tv driver=v4l2:device=/dev/video2:input=0:outfmt=yv12:freq=210.25:normid=3:alsa:adevice=hw.2,0:amode=1:immediatemode=0 -fs tv://
Signed-off-by: Stefan Ringel<stefan.ringel@xxxxxxxx>
---
   drivers/staging/tm6000/tm6000-stds.c |    5 +++++
   1 files changed, 5 insertions(+), 0 deletions(-)

diff --git a/drivers/staging/tm6000/tm6000-stds.c b/drivers/staging/tm6000/tm6000-stds.c
index da3e51b..a9e1921 100644
--- a/drivers/staging/tm6000/tm6000-stds.c
+++ b/drivers/staging/tm6000/tm6000-stds.c
@@ -22,12 +22,17 @@
   #include "tm6000.h"
   #include "tm6000-regs.h"

+static unsigned int tm6010_a_mode;
+module_param(tm6010_a_mode, int, 0644);
+MODULE_PARM_DESC(tm6010_a_mode, "set sif audio mode (tm6010 only)");
+
   struct tm6000_reg_settings {
       unsigned char req;
       unsigned char reg;
       unsigned char value;
   };

+/* must be updated */
   enum tm6000_audio_std {
       BG_NICAM,
       BTSC,
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux