On Thu, 14 Jul 2011, Laurent Pinchart wrote: > Hi Michael, > > On Tuesday 12 July 2011 17:39:04 Michael Grzeschik wrote: > > Signed-off-by: Michael Grzeschik <m.grzeschik@xxxxxxxxxxxxxx> > > --- > > drivers/media/video/mt9m111.c | 7 ++++--- > > 1 files changed, 4 insertions(+), 3 deletions(-) > > > > diff --git a/drivers/media/video/mt9m111.c b/drivers/media/video/mt9m111.c > > index e08b46c..8ad99a9 100644 > > --- a/drivers/media/video/mt9m111.c > > +++ b/drivers/media/video/mt9m111.c > > @@ -170,6 +170,7 @@ struct mt9m111 { > > enum mt9m111_context context; > > struct v4l2_rect rect; > > const struct mt9m111_datafmt *fmt; > > + int lastpage; > > unsigned int gain; > > unsigned char autoexposure; > > unsigned char datawidth; > > @@ -192,17 +193,17 @@ static int reg_page_map_set(struct i2c_client > > *client, const u16 reg) { > > int ret = 0; > > u16 page; > > - static int lastpage = -1; /* PageMap cache value */ > > You're loosing lastpage initialization to -1. Seconded. A fixed version of this patch will ve welcome for 3.2. Thanks Guennadi > > > + struct mt9m111 *mt9m111 = to_mt9m111(client); > > > > page = (reg >> 8); > > - if (page == lastpage) > > + if (page == mt9m111->lastpage) > > return 0; > > if (page > 2) > > return -EINVAL; > > > > ret = i2c_smbus_write_word_data(client, MT9M111_PAGE_MAP, swab16(page)); > > if (!ret) > > - lastpage = page; > > + mt9m111->lastpage = page; > > return ret; > > } > > -- > Regards, > > Laurent Pinchart > --- Guennadi Liakhovetski, Ph.D. Freelance Open-Source Software Developer http://www.open-technology.de/ -- 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