Shawn Guo wrote: > +static unsigned char port_mapping[] = { > + 0x0, 0x4, 0x8, 0x10, 0x14, 0x1c, > +}; May as well make this a 'const' array while you're at it. And perhaps uint8_t is a better choice than "unsigned char". -- Timur Tabi Linux kernel developer at Freescale