On Tue, Apr 6, 2010 at 10:17 AM, Felipe Balbi <felipe.balbi@xxxxxxxxx> wrote: >> +/* Helper macros and defines for the omap_sdrc_params registers */ >> +#define ACTIM_TRFC(a) (((a) & 0x1f)<<27) > > you need to add more spacing in all these macros, something like: > > #define ACTIM_TRFC(a) (((a) & 0x1f) << 27) > > similar to all others. > > -- > balbi > I'll do it, however checkpatch.pl didn't flag that as a warning. -- Laine Walker-Avina Firmware Engineer PASCO scientific -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html