[U-Boot] [PATCH v2 2/2] sunxi: video: Add support for Hitachi tx18d42vm LVDS LCD panels

Anatolij Gustschin agust at denx.de
Thu Jan 22 18:37:20 CET 2015


On Tue, 20 Jan 2015 15:08:02 +0100
Hans de Goede <hdegoede at redhat.com> wrote:
...
> @@ -976,6 +977,14 @@ static int sunxi_ssd2828_init(const struct ctfb_res_modes *mode)
>  }
>  #endif /* CONFIG_VIDEO_LCD_SSD2828 */
>  
> +#ifdef CONFIG_VIDEO_LCD_HITACHI_TX18D42VM
> +static void sunxi_hitachi_tx18d42vm_init(void)
> +{
> +	mdelay(50); /* Wait for lcd controller power on */
> +	hitachi_tx18d42vm_init();
> +}
> +#endif

Could you please test if 

#if IS_ENABLED(CONFIG_VIDEO_LCD_HITACHI_TX18D42VM)

#endif

works here around sunxi_hitachi_tx18d42vm_init() definition ?

Now with Kconfig it should. I'd prefer this variant instead
of #ifdef.

...
> @@ -1004,6 +1013,9 @@ static void sunxi_mode_set(const struct ctfb_res_modes *mode,
>  		break;
>  	case sunxi_monitor_lcd:
>  		sunxi_lcdc_panel_enable();
> +#ifdef CONFIG_VIDEO_LCD_HITACHI_TX18D42VM
> +		sunxi_hitachi_tx18d42vm_init();
> +#endif

And here, please try to use

	if (IS_ENABLED(CONFIG_VIDEO_LCD_HITACHI_TX18D42VM))
		sunxi_hitachi_tx18d42vm_init();

Thanks,

Anatolij


More information about the U-Boot mailing list