Re: Fw: [PATCH] E506r-composite-input
Hi,
Am Donnerstag, den 15.01.2009, 23:35 -0200 schrieb Mauro Carvalho
Chehab:
Message sent to the wrong address... it is not *-owner ;)
Forwarded message:
Date: Thu, 15 Jan 2009 21:58:55 +0900
From: Tim Farrington t...@iinet.net.au
To: Mauro Carvalho Chehab mche...@infradead.org,
linux-media-ow...@vger.kernel.org
Subject: [PATCH] E506r-composite-input
Make correction to composite input plus svideo input
to Avermedia E506R
Signed-off-by: Tim Farrington t...@iinet.net.au
Cheers,
Mauro
Unterschied
zwischen
Dateien-Anlage
(E506r_composite.patch)
Only in .: E506r_composite.patch
diff
-upr ./linux/drivers/media/video/saa7134/saa7134-cards.c
../a/v4l-dvb/linux/drivers/media/video/saa7134/saa7134-cards.c
--- ./linux/drivers/media/video/saa7134/saa7134-cards.c 2009-01-15
21:42:05.0 +0900
+++ ../a/v4l-dvb/linux/drivers/media/video/saa7134/saa7134-cards.c
2009-01-15 21:45:29.0 +0900
@@ -4362,13 +4362,13 @@ struct saa7134_board saa7134_boards[] =
.amux = TV,
.tv = 1,
}, {
-.name = name_comp,
-.vmux = 0,
+.name = name_comp1,
+.vmux = 3,
.amux = LINE1,
}, {
.name = name_svideo,
.vmux = 8,
-.amux = LINE1,
+.amux = LINE2,
} },
.radio = {
.name = name_radio,
Mauro, I was never sure about why that patch, which introduced name_comp
was accepted very, very late in the drivers history. It previously
always started the enumeration with name_comp1.
If it should have any sense at all, I thought it was to avoid ambiguity
on such devices which have only one Composite input.
Tim, are you sure that Composite amux is LINE1 and S-Video LINE2?
It would be the first and only card ever seen with different amux for
those inputs and should be noted as unusual case. I doubt it has two
different audio-in connectors.
Cheers,
Hermann
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: Fw: [PATCH] E506r-composite-input
hermann pitton wrote:
Hi,
Am Donnerstag, den 15.01.2009, 23:35 -0200 schrieb Mauro Carvalho
Chehab:
Message sent to the wrong address... it is not *-owner ;)
Forwarded message:
Date: Thu, 15 Jan 2009 21:58:55 +0900
From: Tim Farrington t...@iinet.net.au
To: Mauro Carvalho Chehab mche...@infradead.org,
linux-media-ow...@vger.kernel.org
Subject: [PATCH] E506r-composite-input
Make correction to composite input plus svideo input
to Avermedia E506R
Signed-off-by: Tim Farrington t...@iinet.net.au
Cheers,
Mauro
Unterschied
zwischen
Dateien-Anlage
(E506r_composite.patch)
Only in .: E506r_composite.patch
diff
-upr ./linux/drivers/media/video/saa7134/saa7134-cards.c
../a/v4l-dvb/linux/drivers/media/video/saa7134/saa7134-cards.c
--- ./linux/drivers/media/video/saa7134/saa7134-cards.c 2009-01-15
21:42:05.0 +0900
+++ ../a/v4l-dvb/linux/drivers/media/video/saa7134/saa7134-cards.c
2009-01-15 21:45:29.0 +0900
@@ -4362,13 +4362,13 @@ struct saa7134_board saa7134_boards[] =
.amux = TV,
.tv = 1,
}, {
-.name = name_comp,
-.vmux = 0,
+.name = name_comp1,
+.vmux = 3,
.amux = LINE1,
}, {
.name = name_svideo,
.vmux = 8,
-.amux = LINE1,
+.amux = LINE2,
} },
.radio = {
.name = name_radio,
Mauro, I was never sure about why that patch, which introduced name_comp
was accepted very, very late in the drivers history. It previously
always started the enumeration with name_comp1.
If it should have any sense at all, I thought it was to avoid ambiguity
on such devices which have only one Composite input.
Tim, are you sure that Composite amux is LINE1 and S-Video LINE2?
It would be the first and only card ever seen with different amux for
those inputs and should be noted as unusual case. I doubt it has two
different audio-in connectors.
Cheers,
Hermann
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Hermann,
For the record, Avermedia are well known for not respecting the GPL, and
finding ways to
have as many different chip combinations on as many different cards as
they can,
and as many different input configurations as they can invent.
(Try looking at all the different chip combinations they use on all
their Volar sticks.)
Avermedia make the E506R, which is a PCMCIA card.
It has a FM Radio input, a DVB-T/Analog Television input and a Remote
Control.
It also has a mini-SCSI input, to which is connected an Audio Left
cable, an Audio Right cable, and a Composite Video cable;
also separately connected to the SCSI is a S-Video input cable.
Regards,
Timf
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: Fw: [PATCH] E506r-composite-input
Hi,
Am Freitag, den 16.01.2009, 20:26 +0900 schrieb Tim Farrington:
hermann pitton wrote:
Hi,
Am Donnerstag, den 15.01.2009, 23:35 -0200 schrieb Mauro Carvalho
Chehab:
Message sent to the wrong address... it is not *-owner ;)
Forwarded message:
Date: Thu, 15 Jan 2009 21:58:55 +0900
From: Tim Farrington t...@iinet.net.au
To: Mauro Carvalho Chehab mche...@infradead.org,
linux-media-ow...@vger.kernel.org
Subject: [PATCH] E506r-composite-input
Make correction to composite input plus svideo input
to Avermedia E506R
Signed-off-by: Tim Farrington t...@iinet.net.au
Cheers,
Mauro
Unterschied
zwischen
Dateien-Anlage
(E506r_composite.patch)
Only in .: E506r_composite.patch
diff
-upr ./linux/drivers/media/video/saa7134/saa7134-cards.c
../a/v4l-dvb/linux/drivers/media/video/saa7134/saa7134-cards.c
--- ./linux/drivers/media/video/saa7134/saa7134-cards.c 2009-01-15
21:42:05.0 +0900
+++ ../a/v4l-dvb/linux/drivers/media/video/saa7134/saa7134-cards.c
2009-01-15 21:45:29.0 +0900
@@ -4362,13 +4362,13 @@ struct saa7134_board saa7134_boards[] =
.amux = TV,
.tv = 1,
}, {
-.name = name_comp,
-.vmux = 0,
+.name = name_comp1,
+.vmux = 3,
.amux = LINE1,
}, {
.name = name_svideo,
.vmux = 8,
-.amux = LINE1,
+.amux = LINE2,
} },
.radio = {
.name = name_radio,
Mauro, I was never sure about why that patch, which introduced name_comp
was accepted very, very late in the drivers history. It previously
always started the enumeration with name_comp1.
If it should have any sense at all, I thought it was to avoid ambiguity
on such devices which have only one Composite input.
Tim, are you sure that Composite amux is LINE1 and S-Video LINE2?
It would be the first and only card ever seen with different amux for
those inputs and should be noted as unusual case. I doubt it has two
different audio-in connectors.
Cheers,
Hermann
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Hermann,
For the record, Avermedia are well known for not respecting the GPL, and
finding ways to
have as many different chip combinations on as many different cards as
they can,
and as many different input configurations as they can invent.
(Try looking at all the different chip combinations they use on all
their Volar sticks.)
At least they use unique PCI subsystem IDs these days.
Avermedia make the E506R, which is a PCMCIA card.
Can remember when it for the first time appeared on the list ;)
It has a FM Radio input, a DVB-T/Analog Television input and a Remote
Control.
It also has a mini-SCSI input, to which is connected an Audio Left
cable, an Audio Right cable, and a Composite Video cable;
also separately connected to the SCSI is a S-Video input cable.
If you have a single left/right pair as analog audio in, how can it be
for Composite one the LINE1 input pair and for S-Video on the LINE2
input pair of the chip.
There is no gpio switching on an external audio mux visible for these
inputs in the card's entry.
The error is probably taken from the E500 cardbus or I simply don't
understand what should happen on that card with the external audio in.
Unless you confirm you have tested audio on both inputs you modified,
it looks for me like a bug is left.
I thought I did help to fix this on Markus tree already and there was
sound reported for s-video working on LINE1!
I know it have been hard times with this one, but yourself introduced
vmux 0 for Composite over S-Video and now you take it away again ...
Cheers,
Hermann
Regards,
Timf
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Fw: [PATCH] E506r-composite-input
Message sent to the wrong address... it is not *-owner ;)
Forwarded message:
Date: Thu, 15 Jan 2009 21:58:55 +0900
From: Tim Farrington t...@iinet.net.au
To: Mauro Carvalho Chehab mche...@infradead.org,
linux-media-ow...@vger.kernel.org
Subject: [PATCH] E506r-composite-input
Make correction to composite input plus svideo input
to Avermedia E506R
Signed-off-by: Tim Farrington t...@iinet.net.au
Cheers,
Mauro
Only in .: E506r_composite.patch
diff -upr ./linux/drivers/media/video/saa7134/saa7134-cards.c ../a/v4l-dvb/linux/drivers/media/video/saa7134/saa7134-cards.c
--- ./linux/drivers/media/video/saa7134/saa7134-cards.c 2009-01-15 21:42:05.0 +0900
+++ ../a/v4l-dvb/linux/drivers/media/video/saa7134/saa7134-cards.c 2009-01-15 21:45:29.0 +0900
@@ -4362,13 +4362,13 @@ struct saa7134_board saa7134_boards[] =
.amux = TV,
.tv = 1,
}, {
- .name = name_comp,
- .vmux = 0,
+ .name = name_comp1,
+ .vmux = 3,
.amux = LINE1,
}, {
.name = name_svideo,
.vmux = 8,
- .amux = LINE1,
+ .amux = LINE2,
} },
.radio = {
.name = name_radio,
