Hi Sergei, On Tue, Jun 26, 2012 at 16:06:01, Sergei Shtylyov wrote: > Hello. > > On 25-06-2012 15:07, Manjunath Hadli wrote: > > > fix setting of data width in config_vpif_params() function, > > which was wrongly set. > > > Signed-off-by: Manjunath Hadli<manjunath.hadli@xxxxxx> > > Signed-off-by: Lad, Prabhakar<prabhakar.lad@xxxxxx> > > --- > > drivers/media/video/davinci/vpif.c | 2 +- > > 1 files changed, 1 insertions(+), 1 deletions(-) > > > diff --git a/drivers/media/video/davinci/vpif.c b/drivers/media/video/davinci/vpif.c > > index 774bcd3..08fb81f 100644 > > --- a/drivers/media/video/davinci/vpif.c > > +++ b/drivers/media/video/davinci/vpif.c > > @@ -346,7 +346,7 @@ static void config_vpif_params(struct vpif_params *vpifparams, > > > > value = regr(reg); > > /* Set data width */ > > - value&= ((~(unsigned int)(0x3))<< > > + value&= ~(((unsigned int)(0x3))<< > > Why not just 0x3u instead of (unsigned int)(0x3)? > Ok. Thx, --Manju > WBR, Sergei > -- 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