On 06/13/2012 10:19 AM, Simon Glass wrote:
> Add calls to the LCD driver from Nvidia board code.
> diff --git a/board/nvidia/common/board.c b/board/nvidia/common/board.c
> @@ -87,6 +88,9 @@ int board_init(void)
> +#ifdef CONFIG_VIDEO_TEGRA2
> + tegra_lcd_check_next_stage(gd->blob, 0);
> +#endif
This seems to be conflating video support with LCD support. It would be
quite possible to have a board with no LCD, yet supporting display over
HDMI for example. In other words, shouldn't the ifdef above be something
more like:
#if define(CONFIG_LCD_SUPPORT)
register_lcd_driver();
#endif
#if defined(CONFIG_VIDEO_TEGRA2)
tegra_display_init(...);
#endif
and internal to tegra_display_init(), the DT is searched for LCD
controller nodes, and if any are found, they're matched to the LCD
driver registered by the first call above.
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
