From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by yocto-www.yoctoproject.org (Postfix, from userid 118) id 71F60E009EB; Thu, 11 Jun 2015 07:16:46 -0700 (PDT) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on yocto-www.yoctoproject.org X-Spam-Level: X-Spam-Status: No, score=-2.7 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, FREEMAIL_FROM, RCVD_IN_DNSWL_LOW autolearn=ham version=3.3.1 X-Spam-HAM-Report: * 0.0 FREEMAIL_FROM Sender email is commonly abused enduser mail provider * (picmaster[at]mail.bg) * -0.7 RCVD_IN_DNSWL_LOW RBL: Sender listed at http://www.dnswl.org/, low * trust * [193.201.172.119 listed in list.dnswl.org] * -1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1% * [score: 0.0000] * -0.1 DKIM_VALID_AU Message has a valid DKIM or DK signature from author's * domain * 0.1 DKIM_SIGNED Message has a DKIM or DK signature, not necessarily * valid * -0.1 DKIM_VALID Message has at least one valid DKIM or DK signature Received: from mx3.mail.bg (mx3.mail.bg [193.201.172.119]) by yocto-www.yoctoproject.org (Postfix) with ESMTP id 88342E00A76 for ; Thu, 11 Jun 2015 07:16:24 -0700 (PDT) Received: from [192.168.0.62] (unknown [93.152.143.60]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by mx3.mail.bg (Postfix) with ESMTPSA id 9A2E2203851A; Thu, 11 Jun 2015 17:16:20 +0300 (EEST) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=mail.bg; s=default; t=1434032180; bh=EDUtGr69K2nwcheGefvYdQJ0R8onv2pGUF520mJKfd4=; h=Message-ID:Date:From:MIME-Version:To:CC:Subject:References: In-Reply-To:Content-Type:Content-Transfer-Encoding; b=kY4l7k6VFVnYKqSz1X1zI/qnfwObzQj4zYUmSO1h+sJsYiEA1EH7tk/AjNPryWdMP B9BYnWJdvD8+WBeBpegeeGlABDLLv3VHentFKhZ30F+Pzpdk5FxnwxSLOBFhIC7Hr0 /k8p1XWTwQHxQCAR5cEJbJjLVZIHMDuoYpPS5EkE= Message-ID: <55799834.8030503@mail.bg> Date: Thu, 11 Jun 2015 17:16:20 +0300 From: Nikolay Dimitrov User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Icedove/31.7.0 MIME-Version: 1.0 To: Fabio Estevam , Otavio Salvador References: <1433106098-7683-1-git-send-email-picmaster@mail.bg> In-Reply-To: Cc: "meta-freescale@yoctoproject.org" , Fabio Estevam Subject: Re: [meta-fsl-arm][PATCH] u-boot-fslc: mx6sabresd: Enable HDMI video in bootargs X-BeenThere: meta-freescale@yoctoproject.org X-Mailman-Version: 2.1.13 Precedence: list List-Id: Usage and development list for the meta-fsl-* layers List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 11 Jun 2015 14:16:46 -0000 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Hi Fabio, On 06/11/2015 03:47 PM, Fabio Estevam wrote: > Hi Nikolay, > > Sorry for the delay. Np, I'm sorry for poking you. > This only affects mmc boot, but what if people boot from NFS? For sure I realize that there are lots of possible permutations, but I would prefer someone who really needs this functionality to test and propose a patch for it, which patch extends the current solution. I just don't have the resources to implement all the possible variants, but I will always welcome if you send such a patch. > To be honest I don't know what is the best way to handle this. Same here, but this my 4th attempt to provide a better solution (mainline u-boot, yocto master, 2 patch attempts to u-boot-fslc), and I'm starting to run out of ideas :D. Actually the code in your email is from my old patch you rejected. Here's the actual code from the Github u-boot-fslc pull-request: From 1dd053085a7c7dbe7b40aa81b29c35f9de7ef001 Mon Sep 17 00:00:00 2001 From: Nikolay Dimitrov Date: Tue, 2 Jun 2015 18:44:17 +0300 Subject: [PATCH] mx6sabresd: Enable video interfaces in bootargs Generate kernel video bootargs for sabresd, based on a list of needed video interfaces ("video_interfaces" U-Boot env-var). The order of initialization and video settings can be fully customized. Signed-off-by: Nikolay Dimitrov --- include/configs/mx6sabre_common.h | 25 ++++++++++++++++++++++++- include/configs/mx6sabresd.h | 3 +++ 2 files changed, 27 insertions(+), 1 deletion(-) diff --git a/include/configs/mx6sabre_common.h b/include/configs/mx6sabre_common.h index b72522b..a8746a4 100644 --- a/include/configs/mx6sabre_common.h +++ b/include/configs/mx6sabre_common.h @@ -121,6 +121,14 @@ #define EMMC_ENV "" #endif +#ifndef VIDEO_ARGS +#define VIDEO_ARGS "" +#endif + +#ifndef VIDEO_ARGS_SCRIPT +#define VIDEO_ARGS_SCRIPT "" +#endif + #define CONFIG_EXTRA_ENV_SETTINGS \ "script=boot.scr\0" \ "image=zImage\0" \ @@ -148,8 +156,22 @@ "fi; " \ "fi\0" \ EMMC_ENV \ + "video_args_hdmi=setenv video_args $video_args " \ + "video=mxcfb${fb}:dev=hdmi,1280x720M@60,if=RGB24\0" \ + "video_args_lvds=setenv video_args $video_args " \ + "video=mxcfb${fb}:dev=ldb,LDB-XGA,if=RGB666\0" \ + "video_args_lcd=setenv video_args $video_args " \ + "video=mxcfb${fb}:dev=lcd,CLAA-WVGA,if=RGB666\0" \ + "fb=0\0" \ + "video_interfaces=hdmi lvds lcd\0" \ + "video_args_script=" \ + "for v in ${video_interfaces}; do " \ + "run video_args_${v}; " \ + "setexpr fb $fb + 1; " \ + "done\0" \ "mmcargs=setenv bootargs console=${console},${baudrate} " \ - "root=${mmcroot}\0" \ + "root=${mmcroot} " \ + VIDEO_ARGS "\0" \ "loadbootscript=" \ "fatload mmc ${mmcdev}:${mmcpart} ${loadaddr} ${script};\0" \ "bootscript=echo Running bootscript from mmc ...; " \ @@ -157,6 +179,7 @@ "loadimage=fatload mmc ${mmcdev}:${mmcpart} ${loadaddr} ${image}\0" \ "loadfdt=fatload mmc ${mmcdev}:${mmcpart} ${fdt_addr} ${fdt_file}\0" \ "mmcboot=echo Booting from mmc ...; " \ + VIDEO_ARGS_SCRIPT \ "run mmcargs; " \ "if test ${boot_fdt} = yes || test ${boot_fdt} = try; then " \ "if run loadfdt; then " \ diff --git a/include/configs/mx6sabresd.h b/include/configs/mx6sabresd.h index fe71321..9835510 100644 --- a/include/configs/mx6sabresd.h +++ b/include/configs/mx6sabresd.h @@ -31,6 +31,9 @@ #define CONFIG_SUPPORT_EMMC_BOOT /* eMMC specific */ +#define VIDEO_ARGS "${video_args}" +#define VIDEO_ARGS_SCRIPT "run video_args_script; " + #include "mx6sabre_common.h" #define CONFIG_SYS_FSL_USDHC_NUM 3 -- 1.7.10.4 > We could also use the 'hdmidet' command in U-boot and set HDMI as > video argument if a HDMI is connected, but then there is also the > issue that the FSL kernel hangs with HDMI enabled in U-boot. That's correct. I don't think that hdmidet can work without U-Boot HDMI support, so I would propose the dumb idea of just disabling the U-Boot HDMI support and have things working in Linux. Regards, Nikolay