Re: [U-Boot] [PATCH 5/7 V4] SPI: Add SPI Driver for EXYNOS.

2012-08-19 Thread Joonyoung Shim
Hi, 2012/8/2 Rajeshwari Shinde : > This patch adds SPI driver for EXYNOS. > > Signed-off-by: Simon Glass > Signed-off-by: Padmavathi Venna > Signed-off-by: Gabe Black > Signed-off-by: Rajeshwari Shinde > --- > Changes in V2: > - None. > Changes in V3: > - Removed SPI_SLAVE flag

[U-Boot] [PATCH 1/4] spiflash: at25: using common spi flash operation

2012-08-19 Thread Bo Shen
Using common spi flash operation function to replace private operation funtion This patch is based on http://patchwork.ozlabs.org/patch/177896/ which has been merged by Mike frysinger Signed-off-by: Bo Shen --- drivers/mtd/spi/atmel.c | 11 --- 1 file changed, 8 insertions(+), 3 delet

[U-Boot] [PATCH 4/4] atmel: at91sam9x5: add spi flash boot support

2012-08-19 Thread Bo Shen
Add at91sam9x5 series spi flash boot support Using at91sam9x5ek_spiflash to configure, then it can boot from at25df321 serial flash SPI mater work in 30Mhz speed, while not 1Mhz speed. This will base on atmel_spi patch, or else, it will occur receive overrun Signed-off-by: Bo Shen --- boards.c

[U-Boot] [PATCH 3/4] atmel: at91sam9x5: fix name error for spi

2012-08-19 Thread Bo Shen
Fix the name error Signed-off-by: Bo Shen --- arch/arm/cpu/arm926ejs/at91/at91sam9x5_devices.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/arm/cpu/arm926ejs/at91/at91sam9x5_devices.c b/arch/arm/cpu/arm926ejs/at91/at91sam9x5_devices.c index 7558ca2..6d77219 10

[U-Boot] [PATCH 2/4] spi: atmel: add WDRBT bit to avoid receive overrun

2012-08-19 Thread Bo Shen
The atmel at91sam9x5 series spi has feature to avoid receive overren Using the patch to enable it Signed-off-by: Bo Shen --- drivers/spi/atmel_spi.c |3 +++ drivers/spi/atmel_spi.h |1 + 2 files changed, 4 insertions(+) diff --git a/drivers/spi/atmel_spi.c b/drivers/spi/atmel_spi.c ind

Re: [U-Boot] [PATCH V3 0/4] common/lcd cleanup

2012-08-19 Thread Nikita Kiryanov
Gentle ping. On 08/09/2012 01:14 PM, Nikita Kiryanov wrote: This patch series attempts to simplify #ifdef complexity in common/lcd.c. It was compile tested on Arm and PowerPC using MAKEALL Changes in V3: - Rebased on latest u-boot-video - Removed volatile qualifiers in patch 3

Re: [U-Boot] [PATCH] common/lcd: add protection from null bmp pointer

2012-08-19 Thread Nikita Kiryanov
On 08/18/2012 02:09 PM, Jeroen Hofstee wrote: Hi, On 08/16/2012 12:43 PM, Nikita Kiryanov wrote: If the bmp pointer is null (for example because the environment variable "splashimage" was not defined) then U-Boot will get stuck when trying to load the image. Which branch is this? At [1] there

Re: [U-Boot] [U-Boot-DM] [PATCH 3/3] dm: mips: Fix warnings in lb60 board

2012-08-19 Thread Xiangfu Liu
Thanks Marek Vasu, Daniel Schwierzeck, Xiangfu On 08/20/2012 02:19 AM, Daniel Schwierzeck wrote: 2012/7/27 Marek Vasut : The lb60 board accesses the clkgr register, which is 32bit via 16bit IO ops. This causes malfunction. Fix this. qi_lb60.c: In function ‘cpm_init’: qi_lb60.c:72:2: warning:

[U-Boot] [PATCH V2] common/lcd: add protection from null bmp pointer

2012-08-19 Thread Nikita Kiryanov
If the bmp pointer is null then U-Boot will get stuck when trying to load the image. What's worse, it will get stuck before the U-Boot shell becomes available to the user, thus making it difficult to correct the situation. To protect from the above scenario, check if the pointer is valid. Signed-

Re: [U-Boot] [PATCH v5] arm: rmobile: Add support Renesas SH73A0

2012-08-19 Thread Mike Frysinger
On Sunday 19 August 2012 10:40:05 Nobuhiro Iwamatsu wrote: > --- /dev/null > +++ b/arch/arm/cpu/armv7/rmobile/board.c > > +#ifdef CONFIG_DISPLAY_BOARDINFO > +int checkboard(void) > +{ > + printf("Board: %s\n", sysinfo.board_string); > + return 0; > +} > +#endif /* CONFIG_DISPLAY_BOARDINFO *

Re: [U-Boot] [PATCH 2/2] rmobile: Add README

2012-08-19 Thread Nobuhiro Iwamatsu
On Sat, Aug 11, 2012 at 12:07 AM, Tom Rini wrote: > On 08/10/2012 12:19 AM, Nobuhiro Iwamatsu wrote: >> Hi, >> >> On Fri, Aug 10, 2012 at 2:03 AM, Tom Rini wrote: >>> On Thu, Aug 09, 2012 at 11:30:24PM +0900, Nobuhiro Iwamatsu wrote: This add README of Renesas RMOBILE. Based doc/README.

Re: [U-Boot] Early malloc() summary

2012-08-19 Thread Graeme Russ
Hi Tomas, On Sun, Aug 19, 2012 at 11:21 PM, Tomas Hlavacek wrote: > Hello Graeme! > > On Fri, Aug 17, 2012 at 3:15 AM, Graeme Russ wrote: >> dm_malloc(bytes, driver *) >> | >> +-> early_malloc(bytes, reloc_helper *) /* Pre-Relocation */ >> | | >> | +->register_helper(reloc_helpe

[U-Boot] Ethernet-USB adapter

2012-08-19 Thread Daniel Ścisłowski
Hello, using U-Boot 2012.07 how I can run ethernet via USB. I have Moschip Semiconductor UA0025C and USB Hub. But how to set MAC address? When board starts U-Boot shows "No ethernet found". My board is Beagleboard C4. Regards Dan ___ U-Boot mailing list

Re: [U-Boot] [PATCH V2] dm: mips: Import libgcc components from Linux

2012-08-19 Thread Daniel Schwierzeck
2012/7/27 Marek Vasut : > Import ashldr3, ashrdi3 and lshrdi3 to squash possible libgcc fp mismatch, > resulting in the following warning: > > mips-linux-gnu-ld: Warning: > /usr/lib/gcc/mips-linux-gnu/4.7/libgcc.a(_lshrdi3.o) uses hard float, u-boot > uses soft float > mips-linux-gnu-ld: Warning:

Re: [U-Boot] [U-Boot-DM] [PATCH 3/3] dm: mips: Fix warnings in lb60 board

2012-08-19 Thread Daniel Schwierzeck
2012/7/27 Marek Vasut : > The lb60 board accesses the clkgr register, which is 32bit via > 16bit IO ops. This causes malfunction. Fix this. > > qi_lb60.c: In function ‘cpm_init’: > qi_lb60.c:72:2: warning: dereferencing type-punned pointer will break > strict-aliasing rules [-Wstrict-aliasing] > q

Re: [U-Boot] [U-Boot-DM] [PATCH 2/3] dm: mips: Fix lb60 timer code

2012-08-19 Thread Daniel Schwierzeck
2012/7/27 Marek Vasut : > The timer code contains more halfword writes which trigger gcc errors. > The registers are again 32bit, yet written by 16bit writes, fix this: > > timer.c: In function ‘reset_timer_masked’: > timer.c:37:2: warning: dereferencing type-punned pointer will break > strict-ali

Re: [U-Boot] [PATCH 1/3] dm: mips: Fix lb60 WDT control

2012-08-19 Thread Daniel Schwierzeck
2012/7/27 Marek Vasut : > Write the TSCR register via 32bit write instead of 16bit one. > The register is 32bit wide and bit 16 is being set, triggering > gcc overflow error and making the code broken. > > Signed-off-by: Marek Vasut > Cc: Daniel > Cc: Shinya Kuribayashi > Cc: Xiangfu Liu > ---

[U-Boot] [PATCH] tegra20: rework UART GPIO handling

2012-08-19 Thread Lucas Stach
Rename board provided gpio_config_uart() to gpio_early_init_uart() as it does the same thing as the equally called function provided by the uart-switch code. This allows to simply call this function in early board init whether or not we are building with CONFIG_UART_SWITCH defined. Also provide a

Re: [U-Boot] [PATCH 2/2] tegra: convert gpio_config_uart to weak symbol

2012-08-19 Thread Lucas Stach
Hi Stephen, Am Dienstag, den 07.08.2012, 11:09 -0600 schrieb Stephen Warren: > On 08/06/2012 07:18 PM, Lucas Stach wrote: > > Most boards don't need this fixup hook. To avoid a lot of empty > > implementations in board files convert it to a weak symbol. > > This seems OK on the surface, but I thi

Re: [U-Boot] [PATCH] usb: fix ulpi_set_vbus prototype

2012-08-19 Thread Marek Vasut
Dear Lucas Stach, > Match the name of the header prototype with the actual > implementation. Applied, thanks > Signed-off-by: Lucas Stach > --- > include/usb/ulpi.h | 2 +- > 1 Datei geändert, 1 Zeile hinzugefügt(+), 1 Zeile entfernt(-) > > diff --git a/include/usb/ulpi.h b/include/usb/ulpi.h

Re: [U-Boot] [PATCH v3] MX: Set a common gpio.h for all i.MX

2012-08-19 Thread stefano babic
Am 19/08/2012 15:30, schrieb Benoît Thébaudeau: > Hi Stefano, > > Your detailed description from v1 is gone. > I will add it again >> Signed-off-by: Stefano Babic > > Your Cc's from v1 are gone. This is ok - I used patman this time, and CC are automatically handled by the tool and discharged

[U-Boot] [PATCH] usb: fix ulpi_set_vbus prototype

2012-08-19 Thread Lucas Stach
Match the name of the header prototype with the actual implementation. Signed-off-by: Lucas Stach --- include/usb/ulpi.h | 2 +- 1 Datei geändert, 1 Zeile hinzugefügt(+), 1 Zeile entfernt(-) diff --git a/include/usb/ulpi.h b/include/usb/ulpi.h index 4a23fd2..9a75c24 100644 --- a/include/usb/ulp

[U-Boot] [PATCH 3/3] tegra20: add USB ULPI init code

2012-08-19 Thread Lucas Stach
This adds the required code to set up a ULPI USB port. It is mostly a port of the Linux ULPI setup code with some tweaks added for more correctness, discovered along the way of debugging this. To use this both CONFIG_USB_ULPI and CONFIG_USB_ULPI_VIEWPORT have to be set in the board configuration f

[U-Boot] [PATCH 0/3] Tegra 2 USB ULPI series

2012-08-19 Thread Lucas Stach
With this series we are able to initialize USB controllers using an external ULPI phy AKA USB2 on Tegra 2 devices. This was tested to work on a Toradex Colibri T20 board, where USB2 is used to access the ASIX ethernet chipset. Testing was done with "tegra20: usb: rework set_host_mode" applied. I d

[U-Boot] [PATCH 2/3] tegra20: add clock_set_pllout function

2012-08-19 Thread Lucas Stach
Common practice on Tegra 2 boards is to use the pllp_out4 FO to generate the ULPI reference clock. For this to work we have to override the default hardware generated output divider. This function adds a clean way to do so. Signed-off-by: Lucas Stach --- arch/arm/cpu/tegra20-common/clock.c

[U-Boot] [PATCH 1/3] tegra20: complete periph_id enum

2012-08-19 Thread Lucas Stach
Most Tegra boards output the ULPI reference clock on pad DEV2. Complete the periph_id enum so that we are able to enable this clock output circuit. Signed-off-by: Lucas Stach --- arch/arm/cpu/tegra20-common/clock.c | 1 + arch/arm/include/asm/arch-tegra20/clock.h | 6 ++ 2 Dateien geä

Re: [U-Boot] [PATCH 2/2] mxs: Use correct function name to initialize dram

2012-08-19 Thread Otavio Salvador
On Sun, Aug 19, 2012 at 12:36 PM, Fabio Estevam wrote: > On Sun, Aug 19, 2012 at 12:33 PM, Fabio Estevam wrote: > >> Please discard this patch. >> >> Otavio fixed it in another patch related to mx23 series. > > Ops, sorry. Otavio based his patch on top of mine, so this one needs > to be applied.

Re: [U-Boot] [PATCH 2/2] mxs: Use correct function name to initialize dram

2012-08-19 Thread Fabio Estevam
On Sun, Aug 19, 2012 at 12:33 PM, Fabio Estevam wrote: > Please discard this patch. > > Otavio fixed it in another patch related to mx23 series. Ops, sorry. Otavio based his patch on top of mine, so this one needs to be applied. Thanks, Fabio Estevam ___

Re: [U-Boot] [PATCH 2/2] mxs: Use correct function name to initialize dram

2012-08-19 Thread Fabio Estevam
Hi Stefano, On Sat, Aug 18, 2012 at 8:28 PM, Fabio Estevam wrote: > From: Fabio Estevam > > commit d92591a (mxs: Convert sys_proto.h prefixes to 'mxs') introduced > a mxs_dram_init() function, which is not used anywhere. > > Fix it, so that the following warning goes away: > > mx28evk.c: In func

Re: [U-Boot] [PATCH 25/37] arm: rmobile: Add support TMU base timer function

2012-08-19 Thread Nobuhiro Iwamatsu
Hi, Thanks for your review. On Tue, Aug 14, 2012 at 8:26 AM, Kim Phillips wrote: > patch 25/37 showing up in an otherwise unnumbered patchseries of 6, > and with the cover letter set to 'v3' where all 6 patches are at v2 > makes for a highly confusing patchset. Sorry, this is my mistake. > > O

Re: [U-Boot] [PATCH 3/3] mxs: Rename 'mx28_dram_init' to 'mxs_dram_init'

2012-08-19 Thread Marek Vasut
Dear Otavio Salvador, > The DRAM initialization, after SPL has complete, is exactly the same > for all mxs SoCs so we should name it accordinly. Acked-by: Marek Vasut > The following boards has been changed: > > * apx4devkit > * m28evk > * mx28evk > * sc_sps_1 > > Signed-off-by: Otavio Sa

Re: [U-Boot] [PATCH 1/3] mxs: Replace i.MX233 by i.MX23 on copyright header

2012-08-19 Thread Otavio Salvador
On Sun, Aug 19, 2012 at 12:05 PM, Marek Vasut wrote: > Dear Otavio Salvador, > >> All other header are going to use i.MX23 so we change this for >> consistency. >> >> Signed-off-by: Otavio Salvador >> --- >> arch/arm/include/asm/arch-mxs/sys_proto.h |2 +- >> 1 file changed, 1 insertion(+),

Re: [U-Boot] [PATCH 2/3] mxs: Only build internal Ethernet controller for i.MX28

2012-08-19 Thread Marek Vasut
Dear Otavio Salvador, > The internal Ethernet controller is only available on i.MX28 > processors so it needs to use CONFIG_MX28 guardian to avoid having > this code called in others. > > Signed-off-by: Otavio Salvador > --- > arch/arm/cpu/arm926ejs/mxs/mxs.c |2 +- > 1 file changed, 1 inse

Re: [U-Boot] [PATCH 1/3] mxs: Replace i.MX233 by i.MX23 on copyright header

2012-08-19 Thread Marek Vasut
Dear Otavio Salvador, > All other header are going to use i.MX23 so we change this for > consistency. > > Signed-off-by: Otavio Salvador > --- > arch/arm/include/asm/arch-mxs/sys_proto.h |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm/include/asm/arch-mxs/s

[U-Boot] [PATCH 3/3] mxs: Rename 'mx28_dram_init' to 'mxs_dram_init'

2012-08-19 Thread Otavio Salvador
The DRAM initialization, after SPL has complete, is exactly the same for all mxs SoCs so we should name it accordinly. The following boards has been changed: * apx4devkit * m28evk * mx28evk * sc_sps_1 Signed-off-by: Otavio Salvador --- arch/arm/cpu/arm926ejs/mxs/mxs.c |4 ++--

[U-Boot] [PATCH 2/3] mxs: Only build internal Ethernet controller for i.MX28

2012-08-19 Thread Otavio Salvador
The internal Ethernet controller is only available on i.MX28 processors so it needs to use CONFIG_MX28 guardian to avoid having this code called in others. Signed-off-by: Otavio Salvador --- arch/arm/cpu/arm926ejs/mxs/mxs.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/a

[U-Boot] [PATCH 1/3] mxs: Replace i.MX233 by i.MX23 on copyright header

2012-08-19 Thread Otavio Salvador
All other header are going to use i.MX23 so we change this for consistency. Signed-off-by: Otavio Salvador --- arch/arm/include/asm/arch-mxs/sys_proto.h |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/include/asm/arch-mxs/sys_proto.h b/arch/arm/include/asm/arch-m

[U-Boot] [PATCH v5] arm: rmobile: Add support Renesas SH73A0

2012-08-19 Thread Nobuhiro Iwamatsu
From: Nobuhiro Iwamatsu Renesas SH73A0 is CPU with Cortex-A9. This supports the basic register definition and GPIO. Signed-off-by: Nobuhiro Iwamatsu Signed-off-by: Nobuhiro Iwamatsu --- V5: Remove dead code. V4: Revert ICCICR. V3: Set COBJS one object per line, and sort the list. Remov

Re: [U-Boot] [PATCH v3] arm: rmobile: Add support Renesas SH73A0

2012-08-19 Thread Nobuhiro Iwamatsu
Hi, Thanks for your review. On Sat, Aug 11, 2012 at 12:52 AM, Mike Frysinger wrote: > On Friday 10 August 2012 03:41:17 Nobuhiro Iwamatsu wrote: >> --- a/arch/arm/cpu/armv7/rmobile/Makefile >> +++ b/arch/arm/cpu/armv7/rmobile/Makefile >> >> +clean: >> + rm -f $(SOBJS) $(OBJS) >> + > > dead c

Re: [U-Boot] [PATCH 2/2] mxs: Use correct function name to initialize dram

2012-08-19 Thread Otavio Salvador
On Sat, Aug 18, 2012 at 8:28 PM, Fabio Estevam wrote: > From: Fabio Estevam > > commit d92591a (mxs: Convert sys_proto.h prefixes to 'mxs') introduced > a mxs_dram_init() function, which is not used anywhere. > > Fix it, so that the following warning goes away: > > mx28evk.c: In function ‘dram_in

Re: [U-Boot] [[Patch V2] mips: 15/16] add configs/qemu-mips64.h

2012-08-19 Thread Daniel Schwierzeck
2012/8/17 Zhizhou Zhang : > Signed-off-by: Zhizhou Zhang > --- > include/configs/qemu-mips64.h | 171 > + > 1 file changed, 171 insertions(+) > create mode 100644 include/configs/qemu-mips64.h you should merge this patch with patch 13 with a commit mess

Re: [U-Boot] [[Patch V2] mips: 16/16] modify qemu-mips/config.mk

2012-08-19 Thread Daniel Schwierzeck
2012/8/17 Zhizhou Zhang : > Though I defined CONFIG_SYS_TEXT_BASE in configs/qemu-mips64.h, > but the value is still affected by this file. you should delete all CONFIG_SYS_TEXT_BASE definitions from this file. The board specific config header file is the only right place. > Signed-off-by: Zhizho

Re: [U-Boot] [[Patch V2] mips: 14/16] add mips64 standalone link script

2012-08-19 Thread Daniel Schwierzeck
2012/8/18 Zhi-zhou Zhang : > On Sat, Aug 18, 2012 at 1:08 AM, Andrew Dyer wrote: > >> I think the OUTPUT_FORMAT line if wrong. This will produce little endian >> output even if explicitly told by command line switch -EB to make big >> endian. >> > Yes, But if I want make it configureable, I shou

Re: [U-Boot] [PATCH v3] MX: Set a common gpio.h for all i.MX

2012-08-19 Thread Benoît Thébaudeau
Hi Stefano, Your detailed description from v1 is gone. > Signed-off-by: Stefano Babic Your Cc's from v1 are gone. > --- > Changes in v2: > - please ignore this version, it is broken > > Changes in v3: > - call the macro IMX_GPIO_NR as in kernel (Fabio Estevam) > - change psr in gpio_psr (Ben

[U-Boot] U-boot confused - flashing

2012-08-19 Thread Daniel Ścisłowski
Hi, I use BeagleBoard rev C4. I would like to prepare u-boot from source myself, one of the newest. I tried to build first X-Loader MLO file and then U-Boot 2012.07. But X-Loader didn't loaded u-boot.bin file. I tried many versions and it only worked with some old. Then after many hours I realized

Re: [U-Boot] [[Patch V2] mips: 14/16] add mips64 standalone link script

2012-08-19 Thread Daniel Schwierzeck
2012/8/17 Zhizhou Zhang : > Signed-off-by: Zhizhou Zhang > --- > examples/standalone/mips64.lds | 59 > > 1 file changed, 59 insertions(+) > create mode 100644 examples/standalone/mips64.lds > > diff --git a/examples/standalone/mips64.lds b/examples/st

Re: [U-Boot] Early malloc() summary

2012-08-19 Thread Tomas Hlavacek
Hello Graeme! On Fri, Aug 17, 2012 at 3:15 AM, Graeme Russ wrote: > dm_malloc(bytes, driver *) > | > +-> early_malloc(bytes, reloc_helper *) /* Pre-Relocation */ > | | > | +->register_helper(reloc_helper *) > | | > | +->pre_reloc_malloc(size_t bytes) > | > +-> mal

Re: [U-Boot] [[Patch V2] mips: 00/16] add mips64 support to U-Boot

2012-08-19 Thread Daniel Schwierzeck
Hi, 2012/8/17 Zhizhou Zhang : > > This patch add mips64 support. > --- > Change log for V2: > - move mips64 build flags to cpu/mips64.mk > - add cache size probe > - remove cache.S please add changelogs to the patches which you have changed since the previous submission > > Zhizhou Zhang (

[U-Boot] [PATCH v3] MX: Set a common gpio.h for all i.MX

2012-08-19 Thread Stefano Babic
Signed-off-by: Stefano Babic --- Changes in v2: - please ignore this version, it is broken Changes in v3: - call the macro IMX_GPIO_NR as in kernel (Fabio Estevam) - change psr in gpio_psr (Benoit Thebaudeau) - drop MXC_GPIO_PORT_TO_NUM and use common macro (Benoit Thebaudeau) arch/arm/include

Re: [U-Boot] [PATCH 1/2] mx28evk: Remove unneeded 'undef'

2012-08-19 Thread Stefano Babic
On 19/08/2012 01:28, Fabio Estevam wrote: > From: Fabio Estevam > > There is no need to undef an option that is not enabled by default. > > Signed-off-by: Fabio Estevam > --- > include/configs/mx28evk.h |1 - > 1 file changed, 1 deletion(-) > > diff --git a/include/configs/mx28evk.h b/inc

[U-Boot] [PATCH v2] MX: Set a common gpio.h for all i.MX

2012-08-19 Thread Stefano Babic
Signed-off-by: Stefano Babic --- Changes in v2: - call the macro IMX_GPIO_NR as in kernel (Fabio Estevam) - change psr in gpio_psr (Benoit Thebaudeau) - drop MXC_GPIO_PORT_TO_NUM and use common macro (Benoit Thebaudeau) arch/arm/include/asm/arch-mx25/gpio.h |5 - arch/arm/includ

Re: [U-Boot] [PATCH] MX: Set a common gpio.h for all i.MX

2012-08-19 Thread Stefano Babic
On 19/08/2012 02:25, Benoît Thébaudeau wrote: > Hi Stefano, > #define MXC_GPIO_PORT_TO_NUM(port, bit) (((port - 1) << 5) + (bit & 0x1f)) >>> >>> Keeping this is also useless. GPIO_NUMBER() from the new >>> >>> can be used instead everywhere needed. >> >> That is right - I drop i

Re: [U-Boot] [PATCH] MX: Set a common gpio.h for all i.MX

2012-08-19 Thread stefano babic
Am 19/08/2012 00:59, schrieb Fabio Estevam: > Hi Stefano, > > On Sat, Aug 18, 2012 at 12:26 PM, Stefano Babic wrote: > >> +#define GPIO_NUMBER(port, index) >> port)-1)*32)+((index)&31)) > > What about calling this macro IMX_GPIO_NR instead? > > This way we can have the same