[U-Boot] [PATCH V2 3/6] ARM: Tegra210: Add SoC code/include files for T210
Tom Warren
TWarren at nvidia.com
Thu Jul 23 17:34:47 CEST 2015
Stephen,
> -----Original Message-----
> From: Stephen Warren [mailto:swarren at wwwdotorg.org]
> Sent: Wednesday, July 22, 2015 10:45 AM
> To: Tom Warren
> Cc: u-boot at lists.denx.de; Thierry Reding; Stephen Warren;
> tomcwarren3959 at gmail.com
> Subject: Re: [U-Boot] [PATCH V2 3/6] ARM: Tegra210: Add SoC code/include
> files for T210
>
> On 07/20/2015 01:50 PM, Tom Warren wrote:
> > All based off of Tegra124. As a Tegra210 board is brought up, these
> > may change a bit to match the HW more closely, but probably 90% of
> > this is identical to T124.
> >
> > Note that since T210 is a 64-bit build, it has no SPL component, and
> > hence no cpu.c for Tegra210.
>
> > diff --git a/arch/arm/mach-tegra/tegra210/funcmux.c
> > b/arch/arm/mach-tegra/tegra210/funcmux.c
>
> > +int funcmux_select(enum periph_id id, int config) {
> > + int bad_config = config != FUNCMUX_DEFAULT;
> > +
> > + switch (id) {
> > + case PERIPH_ID_UART4:
> > + switch (config) {
> > + case FUNCMUX_UART4_GPIO: /* TXD,RXD,CTS,RTS */
> > + pinmux_set_func(PMUX_PINGRP_UART4_TX_PI4,
> > + PMUX_FUNC_UARTD);
> > + pinmux_set_func(PMUX_PINGRP_UART4_RX_PI5,
> > + PMUX_FUNC_UARTD);
> > + pinmux_set_func(PMUX_PINGRP_UART4_CTS_PI7,
> > + PMUX_FUNC_UARTD);
> > + pinmux_set_func(PMUX_PINGRP_UART4_RTS_PI6,
> > + PMUX_FUNC_UARTD);
> ...
> > + case PERIPH_ID_UART1:
> > + switch (config) {
> > + case FUNCMUX_UART1_KBC:
> > + pinmux_set_func(PMUX_PINGRP_UART1_TX_PU0,
> > + PMUX_FUNC_UARTA);
> > + pinmux_set_func(PMUX_PINGRP_UART1_RX_PU1,
>
> Those are the wrong FUNCMUX_* enum names; they're supposed to be
> FUNCMUX_${hwblock}_${pinset}. ${hwblock} is correctly UART1/4 above.
> ${pinset} doesn't look right to me; I see no GPIO or KBC pins being used.
> Rather, I'd expect FUNCMUX_UART4_UART4 and FUNCMUX_UART1_UART1.
These were cloned from T124. I'll add FUNCMUX_UARTx_ to uart_configs[] in board.c and use them here. Thanks.
Tom
--
nvpublic
More information about the U-Boot
mailing list