[U-Boot] [PATCH 1/2] tegra: display: add board pinmux

Tom Warren
2012-11-26T16:55:49Z

Marc,

>Boards may require a different pinmux setup for DISPALY than the default one.

s/b 'DISPLAY'

>-void pin_mux_usb(void); /* overrideable USB pinmux setup */
>-void pin_mux_spi(void); /* overrideable SPI pinmux setup */
>-void pin_mux_nand(void); /* overrideable NAND pinmux setup */
>+void pin_mux_usb(void); /* overrideable USB pinmux setup */
>+void pin_mux_spi(void); /* overrideable SPI pinmux setup */
>+void pin_mux_nand(void); /* overrideable NAND pinmux setup */
>+void pin_mux_display(void); /* overrideable DISPLAY pinmux setup */

Don't pad the end of the other comments just to match up with your new
DISPLAY proto comment. Had to look at this twice to see what you were
really doing here.

Otherwise this patchset LGTM. I'll apply it to tegra/next once you've
fixed these nits.

Tom

On Sun, Nov 25, 2012 at 2:26 PM, Marc Dietrich <marvin24@gmx.de> wrote:
> Boards may require a different pinmux setup for DISPALY than the default one.
> Add a way to call into board specific code to set this up.
>
> Signed-off-by: Marc Dietrich <marvin24@gmx.de>
> ---
> arch/arm/include/asm/arch-tegra/board.h | 7 ++++---
> board/nvidia/common/board.c | 7 +++++++
> 2 files changed, 11 insertions(+), 3 deletions(-)
>
> diff --git a/arch/arm/include/asm/arch-tegra/board.h b/arch/arm/include/asm/arch-tegra/board.h
> index be6bf25..3db0d93 100644
> --- a/arch/arm/include/asm/arch-tegra/board.h
> +++ b/arch/arm/include/asm/arch-tegra/board.h
> @@ -41,8 +41,9 @@ void gpio_early_init(void); /* overrideable GPIO config */
> * an empty stub function will be called.
> */
>
> -void pin_mux_usb(void); /* overrideable USB pinmux setup */
> -void pin_mux_spi(void); /* overrideable SPI pinmux setup */
> -void pin_mux_nand(void); /* overrideable NAND pinmux setup */
> +void pin_mux_usb(void); /* overrideable USB pinmux setup */
> +void pin_mux_spi(void); /* overrideable SPI pinmux setup */
> +void pin_mux_nand(void); /* overrideable NAND pinmux setup */
> +void pin_mux_display(void); /* overrideable DISPLAY pinmux setup */
>
> #endif
> diff --git a/board/nvidia/common/board.c b/board/nvidia/common/board.c
> index 76ec687..819c120 100644
> --- a/board/nvidia/common/board.c
> +++ b/board/nvidia/common/board.c
> @@ -87,6 +87,12 @@ void __pin_mux_nand(void)
>
> void pin_mux_nand(void) __attribute__((weak, alias("__pin_mux_nand")));
>
> +void __pin_mux_display(void)
> +{
> +}
> +
> +void pin_mux_display(void) __attribute__((weak, alias("__pin_mux_display")));
> +
> /*
> * Routine: power_det_init
> * Description: turn off power detects
> @@ -126,6 +132,7 @@ int board_init(void)
> debug("%s: Failed to init pwm\n", __func__);
> #endif
> #ifdef CONFIG_LCD
> + pin_mux_display();
> tegra_lcd_check_next_stage(gd->fdt_blob, 0);
> #endif
> /* boot param addr */
> --
> 1.7.9.5
>