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