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.
> + 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
--
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
