From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefano Babic Date: Tue, 24 May 2016 16:07:37 +0200 Subject: [U-Boot] [PATCH] board: ge: bx50v3: don't configure the backlight when there's no display In-Reply-To: <1463746931-7440-1-git-send-email-andrew.shadura@collabora.co.uk> References: <1463746931-7440-1-git-send-email-andrew.shadura@collabora.co.uk> Message-ID: <57446029.3030803@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Andrew, On 20/05/2016 14:22, Andrew Shadura wrote: > Don't try to configure the backlight when CONFIG_VIDEO_IPUV3 isn't set. > > Signed-off-by: Andrew Shadura > --- > board/ge/bx50v3/bx50v3.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/board/ge/bx50v3/bx50v3.c b/board/ge/bx50v3/bx50v3.c > index e167cd5..fb112ee 100644 > --- a/board/ge/bx50v3/bx50v3.c > +++ b/board/ge/bx50v3/bx50v3.c > @@ -601,6 +601,8 @@ int board_late_init(void) > #ifdef CONFIG_CMD_BMODE > add_board_boot_modes(board_boot_modes); > #endif > + > +#ifdef CONFIG_VIDEO_IPUV3 > /* We need at least 200ms between power on and backlight on > * as per specifications from CHI MEI */ > mdelay(250); > @@ -615,6 +617,7 @@ int board_late_init(void) > gpio_direction_output(LVDS_BACKLIGHT_GP, 1); > > pwm_enable(0); > +#endif ...but CONFIG_VIDEO_IPUV3 is set in include/configs/ge_bx50v3.h, that means your changes works on a out-of-tree version of the board. To get this real working, you need to set CONFIG_VIDEO_IPUV3 via menuconfig, that is outside ge_bx50v3.h. Best regards, Stefano Babic -- ===================================================================== DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de =====================================================================