On 11/10/2011 12:05 AM, Robert Jarzmik wrote: > -#define DOC_BCH_SYNDROM(idx) (0x1048 + (idx << 1)) > +#define DOC_BCH_SYNDROM(idx) (0x1048 + (idx << 0)) Minor nit: why the 0, and not just (0x1048 + idx) ? Mike