From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from arroyo.ext.ti.com (arroyo.ext.ti.com [192.94.94.40]) by arago-project.org (Postfix) with ESMTPS id 45747529DD for ; Mon, 8 Sep 2014 21:42:47 +0000 (UTC) Received: from dflxv15.itg.ti.com ([128.247.5.124]) by arroyo.ext.ti.com (8.13.7/8.13.7) with ESMTP id s88LgkPO025984; Mon, 8 Sep 2014 16:42:46 -0500 Received: from DFLE72.ent.ti.com (dfle72.ent.ti.com [128.247.5.109]) by dflxv15.itg.ti.com (8.14.3/8.13.8) with ESMTP id s88Lgkgq008792; Mon, 8 Sep 2014 16:42:46 -0500 Received: from dflp32.itg.ti.com (10.64.6.15) by DFLE72.ent.ti.com (128.247.5.109) with Microsoft SMTP Server id 14.3.174.1; Mon, 8 Sep 2014 16:42:46 -0500 Received: from localhost (ileax41-snat.itg.ti.com [10.172.224.153]) by dflp32.itg.ti.com (8.14.3/8.13.8) with ESMTP id s88LgklJ030801; Mon, 8 Sep 2014 16:42:46 -0500 Date: Mon, 8 Sep 2014 17:42:46 -0400 From: Denys Dmytriyenko To: "Cooper Jr., Franklin" Message-ID: <20140908214245.GK18620@edge> References: <1410205737-7590-1-git-send-email-denis@denix.org> <8F29D6B095ED194EA1980491A5E029710C72C4DB@DFLE08.ent.ti.com> MIME-Version: 1.0 In-Reply-To: <8F29D6B095ED194EA1980491A5E029710C72C4DB@DFLE08.ent.ti.com> User-Agent: Mutt/1.5.20 (2009-06-14) Cc: "meta-arago@arago-project.org" Subject: Re: [tisdk-build-scripts][PATCH 1/2] run-opentest: only pass u-boot and spl/uart when enabled in config X-BeenThere: meta-arago@arago-project.org X-Mailman-Version: 2.1.12 Precedence: list List-Id: Arago metadata layer for TI SDKs - OE-Core/Yocto compatible List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 08 Sep 2014 21:42:47 -0000 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline On Mon, Sep 08, 2014 at 09:39:29PM +0000, Cooper Jr., Franklin wrote: > Should we make this variable specific to am335x some how then? I don't think there's easy way to control a config variable based on machine... > > -----Original Message----- > > From: meta-arago-bounces@arago-project.org [mailto:meta-arago- > > bounces@arago-project.org] On Behalf Of Denys Dmytriyenko > > Sent: Monday, September 08, 2014 2:49 PM > > To: meta-arago@arago-project.org > > Subject: [meta-arago] [tisdk-build-scripts][PATCH 1/2] run-opentest: only > > pass u-boot and spl/uart when enabled in config > > > > From: Denys Dmytriyenko > > > > Introduce new variable OPENTEST_BOOTLOADERS to control it, but defaults > > to false. > > This never worked as expected, since SPL/UART is only supported by am335x > > and on other platforms it causes OpenTest to miss the escape window and it > > can't stop boot sequence to configure the necessary parameters. > > > > Signed-off-by: Denys Dmytriyenko > > --- > > configs/arago-core-tisdk-daisy-puget.txt | 4 ++++ > > lib/run-opentest | 17 ++++++++++------- > > 2 files changed, 14 insertions(+), 7 deletions(-) > > > > diff --git a/configs/arago-core-tisdk-daisy-puget.txt b/configs/arago-core- > > tisdk-daisy-puget.txt > > index c132baf..f8bfb7a 100644 > > --- a/configs/arago-core-tisdk-daisy-puget.txt > > +++ b/configs/arago-core-tisdk-daisy-puget.txt > > @@ -177,6 +177,10 @@ OPENTEST_TEE_IP="none" > > # Limit the board selection to official TI GT farm, avoiding personal boards > > OPENTEST_HW_CAP="linux_tigt_farm" > > > > +# Determine whether SPL/UART and U-boot binaries are passed to > > OpenTest > > +for # testing. Otherwise uses existing bootloaders on the board (SD, > > +NAND, etc) OPENTEST_BOOTLOADERS="false" > > + > > # This variable will allow control over whether to perform build testing or # > > skip it. This should usually be set to true but may be changed for # > > prototyping. > > diff --git a/lib/run-opentest b/lib/run-opentest index a64e4cc..1a71ae8 > > 100644 > > --- a/lib/run-opentest > > +++ b/lib/run-opentest > > @@ -206,15 +206,18 @@ run_target_side_test() { > > fi > > > > uboot="" > > - if [ -f $test_dir/temp/board-support/prebuilt-images/u-boot-$m.* ] > > - then > > - uboot="~secondary_bootloader=$(ls $test_dir/temp/board- > > support/prebuilt-images/u-boot-$m.*)" > > - fi > > - > > spluart="" > > - if [ -f $test_dir/temp/board-support/prebuilt-images/u-boot-spl.bin-$m ] > > + if [ "$OPENTEST_BOOTLOADERS" == "true" ] > > then > > - spluart="~primary_bootloader=$(ls $test_dir/temp/board- > > support/prebuilt-images/u-boot-spl.bin-$m)" > > + if [ -f $test_dir/temp/board-support/prebuilt-images/u-boot-$m.* ] > > + then > > + uboot="~secondary_bootloader=$(ls $test_dir/temp/board- > > support/prebuilt-images/u-boot-$m.*)" > > + fi > > + > > + if [ -f $test_dir/temp/board-support/prebuilt-images/u-boot-spl.bin-$m > > ] > > + then > > + spluart="~primary_bootloader=$(ls $test_dir/temp/board- > > support/prebuilt-images/u-boot-spl.bin-$m)" > > + fi > > fi > > > > # Start the STAF process if not already started > > -- > > 2.0.4 > > > > _______________________________________________ > > meta-arago mailing list > > meta-arago@arago-project.org > > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago > _______________________________________________ > meta-arago mailing list > meta-arago@arago-project.org > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago