Commit 0c41acf1 authored by Antonio Ospite's avatar Antonio Ospite Committed by Mauro Carvalho Chehab

V4L/DVB: gspca - ov534: Remove hue control

Hue control doesn't work and the sensor datasheet is not clear about how
to set hue properly.
Signed-off-by: default avatarAntonio Ospite <ospite@studenti.unina.it>
Signed-off-by: default avatarJean-François Moine <moinejf@free.fr>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@redhat.com>
parent c3f52f63
...@@ -60,7 +60,6 @@ struct sd { ...@@ -60,7 +60,6 @@ struct sd {
u8 contrast; u8 contrast;
u8 gain; u8 gain;
u8 exposure; u8 exposure;
u8 hue;
u8 autogain; u8 autogain;
u8 awb; u8 awb;
s8 sharpness; s8 sharpness;
...@@ -82,8 +81,6 @@ static int sd_sethflip(struct gspca_dev *gspca_dev, __s32 val); ...@@ -82,8 +81,6 @@ static int sd_sethflip(struct gspca_dev *gspca_dev, __s32 val);
static int sd_gethflip(struct gspca_dev *gspca_dev, __s32 *val); static int sd_gethflip(struct gspca_dev *gspca_dev, __s32 *val);
static int sd_setvflip(struct gspca_dev *gspca_dev, __s32 val); static int sd_setvflip(struct gspca_dev *gspca_dev, __s32 val);
static int sd_getvflip(struct gspca_dev *gspca_dev, __s32 *val); static int sd_getvflip(struct gspca_dev *gspca_dev, __s32 *val);
static int sd_sethue(struct gspca_dev *gspca_dev, __s32 val);
static int sd_gethue(struct gspca_dev *gspca_dev, __s32 *val);
static int sd_setawb(struct gspca_dev *gspca_dev, __s32 val); static int sd_setawb(struct gspca_dev *gspca_dev, __s32 val);
static int sd_getawb(struct gspca_dev *gspca_dev, __s32 *val); static int sd_getawb(struct gspca_dev *gspca_dev, __s32 *val);
static int sd_setbrightness(struct gspca_dev *gspca_dev, __s32 val); static int sd_setbrightness(struct gspca_dev *gspca_dev, __s32 val);
...@@ -149,20 +146,6 @@ static const struct ctrl sd_ctrls[] = { ...@@ -149,20 +146,6 @@ static const struct ctrl sd_ctrls[] = {
.get = sd_getexposure, .get = sd_getexposure,
}, },
{ /* 4 */ { /* 4 */
{
.id = V4L2_CID_HUE,
.type = V4L2_CTRL_TYPE_INTEGER,
.name = "Hue",
.minimum = 0,
.maximum = 255,
.step = 1,
#define HUE_DEF 143
.default_value = HUE_DEF,
},
.set = sd_sethue,
.get = sd_gethue,
},
{ /* 5 */
{ {
.id = V4L2_CID_AUTOGAIN, .id = V4L2_CID_AUTOGAIN,
.type = V4L2_CTRL_TYPE_BOOLEAN, .type = V4L2_CTRL_TYPE_BOOLEAN,
...@@ -176,8 +159,8 @@ static const struct ctrl sd_ctrls[] = { ...@@ -176,8 +159,8 @@ static const struct ctrl sd_ctrls[] = {
.set = sd_setautogain, .set = sd_setautogain,
.get = sd_getautogain, .get = sd_getautogain,
}, },
#define AWB_IDX 6 #define AWB_IDX 5
{ /* 6 */ { /* 5 */
{ {
.id = V4L2_CID_AUTO_WHITE_BALANCE, .id = V4L2_CID_AUTO_WHITE_BALANCE,
.type = V4L2_CTRL_TYPE_BOOLEAN, .type = V4L2_CTRL_TYPE_BOOLEAN,
...@@ -191,7 +174,7 @@ static const struct ctrl sd_ctrls[] = { ...@@ -191,7 +174,7 @@ static const struct ctrl sd_ctrls[] = {
.set = sd_setawb, .set = sd_setawb,
.get = sd_getawb, .get = sd_getawb,
}, },
{ /* 7 */ { /* 6 */
{ {
.id = V4L2_CID_SHARPNESS, .id = V4L2_CID_SHARPNESS,
.type = V4L2_CTRL_TYPE_INTEGER, .type = V4L2_CTRL_TYPE_INTEGER,
...@@ -205,7 +188,7 @@ static const struct ctrl sd_ctrls[] = { ...@@ -205,7 +188,7 @@ static const struct ctrl sd_ctrls[] = {
.set = sd_setsharpness, .set = sd_setsharpness,
.get = sd_getsharpness, .get = sd_getsharpness,
}, },
{ /* 8 */ { /* 7 */
{ {
.id = V4L2_CID_HFLIP, .id = V4L2_CID_HFLIP,
.type = V4L2_CTRL_TYPE_BOOLEAN, .type = V4L2_CTRL_TYPE_BOOLEAN,
...@@ -219,7 +202,7 @@ static const struct ctrl sd_ctrls[] = { ...@@ -219,7 +202,7 @@ static const struct ctrl sd_ctrls[] = {
.set = sd_sethflip, .set = sd_sethflip,
.get = sd_gethflip, .get = sd_gethflip,
}, },
{ /* 9 */ { /* 8 */
{ {
.id = V4L2_CID_VFLIP, .id = V4L2_CID_VFLIP,
.type = V4L2_CTRL_TYPE_BOOLEAN, .type = V4L2_CTRL_TYPE_BOOLEAN,
...@@ -688,13 +671,6 @@ static void setexposure(struct gspca_dev *gspca_dev) ...@@ -688,13 +671,6 @@ static void setexposure(struct gspca_dev *gspca_dev)
sccb_reg_write(gspca_dev, 0x10, val << 1); sccb_reg_write(gspca_dev, 0x10, val << 1);
} }
static void sethue(struct gspca_dev *gspca_dev)
{
struct sd *sd = (struct sd *) gspca_dev;
sccb_reg_write(gspca_dev, 0x01, sd->hue);
}
static void setautogain(struct gspca_dev *gspca_dev) static void setautogain(struct gspca_dev *gspca_dev)
{ {
struct sd *sd = (struct sd *) gspca_dev; struct sd *sd = (struct sd *) gspca_dev;
...@@ -777,7 +753,6 @@ static int sd_config(struct gspca_dev *gspca_dev, ...@@ -777,7 +753,6 @@ static int sd_config(struct gspca_dev *gspca_dev,
sd->contrast = CONTRAST_DEF; sd->contrast = CONTRAST_DEF;
sd->gain = GAIN_DEF; sd->gain = GAIN_DEF;
sd->exposure = EXPO_DEF; sd->exposure = EXPO_DEF;
sd->hue = HUE_DEF;
#if AUTOGAIN_DEF != 0 #if AUTOGAIN_DEF != 0
sd->autogain = AUTOGAIN_DEF; sd->autogain = AUTOGAIN_DEF;
#else #else
...@@ -857,7 +832,6 @@ static int sd_start(struct gspca_dev *gspca_dev) ...@@ -857,7 +832,6 @@ static int sd_start(struct gspca_dev *gspca_dev)
setautogain(gspca_dev); setautogain(gspca_dev);
setawb(gspca_dev); setawb(gspca_dev);
setgain(gspca_dev); setgain(gspca_dev);
sethue(gspca_dev);
setexposure(gspca_dev); setexposure(gspca_dev);
setbrightness(gspca_dev); setbrightness(gspca_dev);
setcontrast(gspca_dev); setcontrast(gspca_dev);
...@@ -1040,24 +1014,6 @@ static int sd_getcontrast(struct gspca_dev *gspca_dev, __s32 *val) ...@@ -1040,24 +1014,6 @@ static int sd_getcontrast(struct gspca_dev *gspca_dev, __s32 *val)
return 0; return 0;
} }
static int sd_sethue(struct gspca_dev *gspca_dev, __s32 val)
{
struct sd *sd = (struct sd *) gspca_dev;
sd->hue = val;
if (gspca_dev->streaming)
sethue(gspca_dev);
return 0;
}
static int sd_gethue(struct gspca_dev *gspca_dev, __s32 *val)
{
struct sd *sd = (struct sd *) gspca_dev;
*val = sd->hue;
return 0;
}
static int sd_setautogain(struct gspca_dev *gspca_dev, __s32 val) static int sd_setautogain(struct gspca_dev *gspca_dev, __s32 val)
{ {
struct sd *sd = (struct sd *) gspca_dev; struct sd *sd = (struct sd *) gspca_dev;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment