Re: [PATCH v4] fpga manager: Add Altera CvP driver

From: Joe Perches
Date: Tue May 02 2017 - 19:37:31 EST


On Tue, 2017-05-02 at 11:53 +0200, Anatolij Gustschin wrote:
> On Mon, 1 May 2017 23:06:16 +0300 Andy Shevchenko andy.shevchenko@xxxxxxxxx wrote:
> > > +#define VSEC_CVP_MODE_CTRL_NUMCLKS (0xff<<8) /* CVP_NUMCLKS */
> >
> > Is 0xff a mask here? (Btw, you missed spaces around <<)
>
> yes, it is. Will add spaces (checkpatch didn't warn here).

It would with command line option --strict, otherwise not.