cx88: Enable additional cx2388x features
PATCH 1/1 cx88: Enable additional cx2388x features
The cx2388x family of broadcast decoders all have features not enabled by the standard cx88 driver. This patch series adds module parameters allowing the chroma AGC and the color killer to be enabled. By default both features are disabled as in previous versions of the driver.
The Chroma AGC and the color killer is sometimes needed when using signal sources of less than optimal quality.
The patches applies cleanly to 7330:ad0b1f882ad9 of
http://linuxtv.org/hg/v4l-dvb/. The patches should be applied in order.
--
video4linux-list mailing list
Unsubscribe mailto:[email protected]?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list
PATCH 1/1 cx88: Add module parameter to control chroma AGC
1 file changed, 5 insertions(+), 1 deletion(-)
linux/drivers/media/video/cx88/cx88-core.c | 6 +++++-
# HG changeset patch
# User "Frej Drejhammar <[email protected]>" # Date 1204989936 -3600
# Node ID d628824bec6646cc474d13fd47ff03034119d83a # Parent ad0b1f882ad988ae2f80ebbbe914092a4a963f04
cx88: Add module parameter to control chroma AGC
From: "Frej Drejhammar <[email protected]>"
The cx2388x family has support for chroma AGC but no way for a casual user to enable it. This patch adds the module parameter chroma_agc which if non-zero enables the AGC. By default chroma AGC is disabled, as in previous versions of the driver.
linux/drivers/media/video/cx88/cx88-core.c +5 -1
@@ -61,6 +61,10 @@ static unsigned int nocomb;static unsigned int nocomb;module_param (nocomb,int,0644 );MODULE_PARM_DESC (nocomb,"disable comb filter" ); + +static unsigned int chroma_agc; +module_param (chroma_agc, int,0444 ); +MODULE_PARM_DESC (chroma_agc,"enable chroma agc" ); #define dprintk(level,fmt, arg...) if (core_debug >= level) \ printk(KERN_DEBUG "%s: " fmt, core->name , ## arg)
@@ -628,7 +632,7 @@ int cx88_reset(struct cx88_core *core)cx_write (MO_INPUT_FORMAT , ((1 <<13 ) |// agc enable (1 <<12 ) |// agc gain (1 <<11 ) |// adaptibe agc - (0 <<10 ) |// chroma agc + chroma_agc ? (1 <<10 ) :0 | (0 <<9 ) |// ckillen (7 )));
--
video4linux-list mailing list
Unsubscribe mailto:[email protected]?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list
PATCH 1/1 cx88: Add module parameter to control color killer
In reply to Frej Drejhammar1 file changed, 5 insertions(+), 1 deletion(-)
linux/drivers/media/video/cx88/cx88-core.c | 6 +++++-
# HG changeset patch
# User "Frej Drejhammar <[email protected]>" # Date 1204990024 -3600
# Node ID 37b97e1d079850e00df8bd8cb540ae7978341489 # Parent d628824bec6646cc474d13fd47ff03034119d83a
cx88: Add module parameter to control color killer
From: "Frej Drejhammar <[email protected]>"
The cx2388x family has support for a color killer but no way for a casual user to enable it. This patch adds the module parameter color_killer which if non-zero enables the killer. By default the color killer is disabled, as in previous versions of the driver.
linux/drivers/media/video/cx88/cx88-core.c +5 -1
@@ -65,6 +65,10 @@ static unsigned int chroma_agc;static unsigned int chroma_agc;module_param (chroma_agc, int,0444 );MODULE_PARM_DESC (chroma_agc,"enable chroma agc" ); + +static unsigned int color_killer; +module_param (color_killer, int,0444 ); +MODULE_PARM_DESC (color_killer,"enable color killer" ); #define dprintk(level,fmt, arg...) if (core_debug >= level) \ printk(KERN_DEBUG "%s: " fmt, core->name , ## arg)
@@ -633,7 +637,7 @@ int cx88_reset(struct cx88_core *core) (1 <<12 ) |// agc gain (1 <<11 ) |// adaptibe agc chroma_agc ? (1 <<10 ) :0 | - (0 <<9 ) |// ckillen + color_killer ? (1 <<9 ) :0 | (7)));/* setup image format */
--
video4linux-list mailing list
Unsubscribe mailto:[email protected]?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list
Quoted 7 lines · Frej Drejhammar
The cx2388x family of broadcast decoders all have features not enabled by the standard cx88 driver. This patch series adds module parameters allowing the chroma AGC and the color killer to be enabled. By default both features are disabled as in previous versions of the driver.
The Chroma AGC and the color killer is sometimes needed when using signal sources of less than optimal quality.
This really should be done with controls, not more module parameters.
--
video4linux-list mailing list
Unsubscribe mailto:[email protected]?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list
Quoted 1 line
This really should be done with controls, not more module parameters.
I was not aware that you could add custom controls, your comment prompted me to find V4L2_CID_PRIVATE_BASE in the V4L-spec. Expect a revised patch using controls within a week or so...
--Frej
--
video4linux-list mailing list
Unsubscribe mailto:[email protected]?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list
Am Sonntag, den 09.03.2008, 11:32 -0700 schrieb Trent Piepho:
Quoted 7 lines · Frej Drejhammar
Quoted 6 lines
The cx2388x family of broadcast decoders all have features not enabled by the standard cx88 driver. This patch series adds module parameters allowing the chroma AGC and the color killer to be enabled. By default both features are disabled as in previous versions of the driver.
Quoted 5 lines
Quoted 2 lines
The Chroma AGC and the color killer is sometimes needed when using signal sources of less than optimal quality.
This really should be done with controls, not more module parameters.
This is a fully harmless example, still.
In general, we are flooded with patches currently, which seem to be clever, but those people have not even any hardware to test them and are easily happy to go over whole subsystems ...
I would like to see them piss in their own shoes in the first place and not into ours.
Cheers,
Hermann
--
video4linux-list mailing list
Unsubscribe mailto:[email protected]?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list