[U-Boot] [PATCH 1/3] sunxi: video: Add lvds support

[Ian Campbell <ijc@hellion.org.uk>]

On Sun, 2015-01-04 at 21:05 +0100, Hans de Goede wrote:
> diff --git a/board/sunxi/Kconfig b/board/sunxi/Kconfig
> index 8782394..fdb18a4 100644
> --- a/board/sunxi/Kconfig
> +++ b/board/sunxi/Kconfig
> @@ -345,6 +345,20 @@ config VIDEO_LCD_BL_PWM
>  	Set the backlight pwm pin for the LCD panel. This takes a string in the
>  	format understood by sunxi_name_to_gpio, e.g. PH1 for pin 1 of port H.
>  
> +choice
> +	prompt "LCD panel support"
> +	depends on VIDEO
> +	---help---
> +	Select which type of LCD panel to support.
> +
> +config VIDEO_LCD_PANEL_PARALLEL
> +	bool "Generic parallel interface LCD panel"

I don't see any use of this choice, I suppose it is the #else case?

Are there other (third) options for LCDs? Or could this be reworded as a
boolean rather than a choice? If it's to remain a choice I think it'd be
better to have explicit #ifdef ..._PARALLEL rather than #else. Maybe an
#else /* ..._PARALLEL */ style would be an ok compromise though.

The code itself is fine by me, not that I know **** about graphics
stuff ;-)

Ian.