Hi Vikram, On Sun, Jun 10, 2012 at 10:02 AM, Vikram Narayanan <vikram...@gmail.com> wrote:
> + > +/* Converts a GPIO port number and the internal bit position > + * to the GPIO number > + */ > +#define MXC_GPIO_PORT_TO_NUM(port, bit) (((port - 1) << 5) + (bit & 0x1f)) Just a minor comment: MXC_GPIO_PORT_TO_NUM looks like a very looong string. Couldn't we use the same macro as in the Linux kernel (IMX_GPIO_NR) instead? It is more concise and it would nice to have the same macro name for kernel and U-boot. What do you think? Thanks, Fabio Estevam _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot