[U-Boot] i.MX6SL pad declarations (was [RFC PATCH 4/3] i.MX6DQ/DLS: remove unused pad declarations)

Eric Nelson eric.nelson at boundarydevices.com
Fri Sep 20 16:42:33 CEST 2013


Thanks Shawn,

I noticed that Otavio's e-mail had a bad reference for the list,
and we've veered off the original topic.

On 09/20/2013 01:44 AM, Shawn Guo wrote:
> On Thu, Sep 19, 2013 at 07:08:35AM -0700, Eric Nelson wrote:
>> There are currently 8 pads defined in U-Boot that aren't defined
>> in the Linux kernel sources:
>>
>> 	SD2_CLK__USDHC2_CLK
>> 	SD2_CMD__USDHC2_CMD
>> 	SD2_DAT0__USDHC2_DAT0
>> 	SD2_DAT1__USDHC2_DAT1
>> 	SD2_DAT2__USDHC2_DAT2
>> 	SD2_DAT3__USDHC2_DAT3
>
> They are the following ones in kernel.
>
> 	MX6QDL_PAD_SD2_CLK__SD2_CLK
> 	MX6QDL_PAD_SD2_CMD__SD2_CMD
> 	MX6QDL_PAD_SD2_DAT0__SD2_DATA0
> 	MX6QDL_PAD_SD2_DAT1__SD2_DATA1
> 	MX6QDL_PAD_SD2_DAT2__SD2_DATA2
> 	MX6QDL_PAD_SD2_DAT3__SD2_DATA3
>
This comment was about SL, not DQ or DLS.

And my comments were wrong.

I used a tool to do a quick comparison and didn't check my work.

The reality is that U-Boot's pad declarations for SL are
almost non-existent:
	http://git.denx.de/u-boot.git/?p=u-boot.git;a=blob;f=arch/arm/include/asm/arch-mx6/mx6sl_pins.h;h=b39a354f39568b11ea010ae08485ded610b00f1a;hb=HEAD

The 8 declarations that are present all have name mis-matches
with the Linux kernel (USDHC2 vs. SD2, UART1_RXD vs. UART1_RX_DATA).

> They are named per IOMUXC registers description in reference manual.
>
>> 	UART1_RXD__UART1_RXD
>> 	UART1_TXD__UART1_TXD
>
> Yes, kernel does not define them.  That's because the reference manual
> does not define them.  The macros in kernel are generated from reference
> manual data source.
>
Cool. Thanks for your work on the kernel side. Things there
are in much better shape than in the U-Boot source tree.

Regards,


Eric



More information about the U-Boot mailing list