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

2012-11-26 Thread Marc Dietrich 
Hi Tom,

On Monday 26 November 2012 09:55:49 Tom Warren wrote:
> >Boards may require a different pinmux setup for DISPALY than the default
> >one.
> s/b 'DISPLAY'

ups.

> >-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.

yeah, I had to decide whether to create an ugly patch or an ugly comment.
Will choose the other solution then.

Marc
 
> 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  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 
> > ---
> > 
> >  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
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

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

2012-11-26 Thread Tom Warren 
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  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 
> ---
>  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
>
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

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

2012-11-25 Thread Marc Dietrich 
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 
---
 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

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot