RE: [PATCH v3 04/13] davinci: vpif: fix setting of data width in config_vpif_params() function

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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


[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux