From mboxrd@z Thu Jan 1 00:00:00 1970 From: Maxime Ripard Date: Thu, 9 Mar 2017 09:33:06 +0100 Subject: [U-Boot] [PATCH 4/8] sunxi: video: Split out TCON code In-Reply-To: <20170308233444.26172-5-jernej.skrabec@siol.net> References: <20170308233444.26172-1-jernej.skrabec@siol.net> <20170308233444.26172-5-jernej.skrabec@siol.net> Message-ID: <20170309083306.ohemyp6zjgujy3qf@lukather> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi, Thanks for your great work. On Thu, Mar 09, 2017 at 12:34:40AM +0100, Jernej Skrabec wrote: > - writel(0, &lcdc->tcon0_io_tristate); > + sunxi_ctfb_mode_to_display_timing(mode, &timing); > + lcdc_tcon0_mode_set(lcdc, &timing, clk_div, for_ext_vga_dac, > + sunxi_display.depth, CONFIG_VIDEO_LCD_DCLK_PHASE); I'm not sure what this sunxi_ctfb_mode_to_display_timing function is useful for, but it's introduction and the conversion of the users would probably need to be moved to another patch. > #elif defined CONFIG_VIDEO_VGA_VIA_LCD > sunxi_composer_mode_set(mode, address); > sunxi_lcdc_tcon0_mode_set(mode, true); > - sunxi_composer_enable(); > - sunxi_lcdc_enable(); > + lcdc_enable((struct sunxi_lcdc_reg *)SUNXI_LCD0_BASE, > + sunxi_display.depth); > + lcdc_enable((struct sunxi_lcdc_reg *)SUNXI_LCD0_BASE, 0); That one is suspicious. Shouldn't sunxi_composer_enable be left, and lcdc_enable called only once? Maxime -- Maxime Ripard, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 801 bytes Desc: not available URL: