Mauro: To be fair, this patch itself isn't sufficient to solve the problem described in the text provided. One other patch is necessary to get this goal accomplished, named, one published in this same thread in 2011-07-18. This later fix is now embedded in a wider patch provided by Jean-François Moine in 2011-08-10. I'd suggest to change the text below, if possible, mentioning only something like "fix wrong register masking". Thanks, Luiz --- Em dom, 7/8/11, Mauro Carvalho Chehab <mchehab@xxxxxxxxxx> escreveu: > De: Mauro Carvalho Chehab <mchehab@xxxxxxxxxx> > Assunto: [git:v4l-dvb/for_v3.2] [media] Fix wrong register mask in gspca/sonixj.c > Para: linuxtv-commits@xxxxxxxxxxx > Cc: "Jean-François Moine" <moinejf@xxxxxxx>, "Luiz Carlos Ramos" <lramos.prof@xxxxxxxxxxxx> > Data: Domingo, 7 de Agosto de 2011, 9:03 > This is an automatic generated email > to let you know that the following patch were queued at the > > http://git.linuxtv.org/media_tree.git > tree: > > Subject: [media] Fix wrong register mask in gspca/sonixj.c > Author: Luiz Ramos <luizzramos@xxxxxxxxxxxx> > Date: Thu Jul 14 23:08:39 2011 -0300 > > Hello, > > When migrating from Slackware 13.1 to 13.37 (kernel > 2.6.33.x to > 2.6.37.6), there was some sort of regression with the > external webcam > installed at the notebook (0x45:6128, SN9C325+OM6802). > > In the version 2.6.37.6, the images got *very* dark, making > the webcam > almost unusable, unless if used with direct sunlight. > > Tracing back what happened, I concluded that changeset > 0e4d413af > caused some sort of odd effects - including this - to this > specific model. > > Signed-off-by: Luiz Carlos Ramos <lramos.prof@xxxxxxxxxxxx> > Acked-by: Jean-François Moine <moinejf@xxxxxxx> > Signed-off-by: Mauro Carvalho Chehab <mchehab@xxxxxxxxxx> > > drivers/media/video/gspca/sonixj.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > --- > > http://git.linuxtv.org/media_tree.git?a=commitdiff;h=d1520c58eb84ad1ec973a257cd835c948215aab5 > > diff --git a/drivers/media/video/gspca/sonixj.c > b/drivers/media/video/gspca/sonixj.c > index 81b8a60..2ad757d 100644 > --- a/drivers/media/video/gspca/sonixj.c > +++ b/drivers/media/video/gspca/sonixj.c > @@ -2386,7 +2386,7 @@ static int sd_start(struct gspca_dev > *gspca_dev) > reg_w1(gspca_dev, > 0x01, 0x22); > msleep(100); > reg01 = SCL_SEL_OD | > S_PDN_INV; > - reg17 &= > MCK_SIZE_MASK; > + reg17 &= > ~MCK_SIZE_MASK; > reg17 |= > 0x04; /* clock / 4 */ > break; > } > -- 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
