From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from phobos.denx.de (phobos.denx.de [85.214.62.61]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 6E27AC4707C for ; Fri, 12 Jan 2024 21:28:05 +0000 (UTC) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id E553F879DF; Fri, 12 Jan 2024 22:26:47 +0100 (CET) Authentication-Results: phobos.denx.de; dmarc=pass (p=quarantine dis=none) header.from=ti.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Authentication-Results: phobos.denx.de; dkim=pass (1024-bit key; unprotected) header.d=ti.com header.i=@ti.com header.b="y5GfNkyz"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id 70341807A7; Fri, 12 Jan 2024 22:15:44 +0100 (CET) Received: from fllv0015.ext.ti.com (fllv0015.ext.ti.com [198.47.19.141]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by phobos.denx.de (Postfix) with ESMTPS id 0A154875FA for ; Fri, 12 Jan 2024 22:15:41 +0100 (CET) Authentication-Results: phobos.denx.de; dmarc=pass (p=quarantine dis=none) header.from=ti.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=j-humphreys@ti.com Received: from fllv0035.itg.ti.com ([10.64.41.0]) by fllv0015.ext.ti.com (8.15.2/8.15.2) with ESMTP id 40CLFY7Y052989; Fri, 12 Jan 2024 15:15:34 -0600 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ti.com; s=ti-com-17Q1; t=1705094134; bh=/G5t57/lO0AIN04eZT2wMgI0PpSHpxPzQcUAvk/lZnE=; h=From:To:CC:Subject:In-Reply-To:References:Date; b=y5GfNkyzGRq0OJtdaC5Yixn/lWhzi2OeYpJnZ4ZiHvSBSsi0x1moJLwlDl6WNLNu9 s9qMK2qcln+22wuaJ5gRwVaswPnhemB3tG5M0BW+SbeCGW1avfF/dIzh0qq/qwRgNX ZdnCYHxO9olKrqVl2/COn9V5l0SJxvjanQbVmtx4= Received: from DLEE104.ent.ti.com (dlee104.ent.ti.com [157.170.170.34]) by fllv0035.itg.ti.com (8.15.2/8.15.2) with ESMTPS id 40CLFYDH080747 (version=TLSv1.2 cipher=AES256-GCM-SHA384 bits=256 verify=FAIL); Fri, 12 Jan 2024 15:15:34 -0600 Received: from DLEE103.ent.ti.com (157.170.170.33) by DLEE104.ent.ti.com (157.170.170.34) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2507.23; Fri, 12 Jan 2024 15:15:34 -0600 Received: from lelvsmtp6.itg.ti.com (10.180.75.249) by DLEE103.ent.ti.com (157.170.170.33) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2507.23 via Frontend Transport; Fri, 12 Jan 2024 15:15:34 -0600 Received: from localhost (udb0321960.dhcp.ti.com [128.247.81.241]) by lelvsmtp6.itg.ti.com (8.15.2/8.15.2) with ESMTP id 40CLFXgZ062265; Fri, 12 Jan 2024 15:15:33 -0600 From: Jon Humphreys To: Nishanth Menon , Tom Rini CC: Kamlesh Gurudasani , Sinthu Raja , Neha Malcom Francis , Heinrich Schuchardt , Roger Quadros , Simon Glass , Andrew Davis , Mattijs Korpershoek , Nikhil M Jain , "Manorit Chawdhry" , Bryan Brattlof , Robert Nelson , , Nishanth Menon Subject: Re: [PATCH V2 09/10] board: beagle: beagleplay: Set fdtfile from C code instead of findfdt script In-Reply-To: <20240109191506.3820908-10-nm@ti.com> References: <20240109191506.3820908-1-nm@ti.com> <20240109191506.3820908-10-nm@ti.com> Date: Fri, 12 Jan 2024 15:15:33 -0600 Message-ID: <86frz2cky2.fsf@udb0321960.dhcp.ti.com> MIME-Version: 1.0 Content-Type: text/plain X-EXCLAIMER-MD-CONFIG: e1e8a2fd-e40a-4ac6-ac9b-f7e9cc9ee180 X-Mailman-Approved-At: Fri, 12 Jan 2024 22:26:42 +0100 X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.39 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" X-Virus-Scanned: clamav-milter 0.103.8 at phobos.denx.de X-Virus-Status: Clean Nishanth Menon writes: > Stop using the findfdt script and switch to setting the fdtfile from C > code. > > Signed-off-by: Nishanth Menon > --- > Changes from V1: > * Just macro name change s/TI_EVM_FDT_FOLDER_PATH/TI_FDT_FOLDER_PATH > * Commit message update to drop the "warning added to findfdt" since > that is not done. > > I have retained the explicit setting of fdtfile to remove dependency on > TI evm specific logic. Also, the dynamic population of fdtfile instead > of env_set("fdtfile", "ti/k3-am625-beagleplay.dtb") to allow for the > upcoming board variants to be able to configure the dtb name via a > config fragment. > > V1: https://lore.kernel.org/r/20240108173301.2692332-10-nm@ti.com > board/beagle/beagleplay/beagleplay.c | 14 ++++++++++++++ > board/beagle/beagleplay/beagleplay.env | 1 - > configs/am62x_beagleplay_a53_defconfig | 3 ++- > 3 files changed, 16 insertions(+), 2 deletions(-) > > diff --git a/board/beagle/beagleplay/beagleplay.c b/board/beagle/beagleplay/beagleplay.c > index 1c376dea372f..20819ecf45b4 100644 > --- a/board/beagle/beagleplay/beagleplay.c > +++ b/board/beagle/beagleplay/beagleplay.c > @@ -27,3 +27,17 @@ int dram_init_banksize(void) > { > return fdtdec_setup_memory_banksize(); > } > + > +#ifdef CONFIG_BOARD_LATE_INIT > +int board_late_init(void) > +{ > + char fdtfile[50]; > + > + snprintf(fdtfile, sizeof(fdtfile), "%s/%s.dtb", > + CONFIG_TI_FDT_FOLDER_PATH, CONFIG_DEFAULT_DEVICE_TREE); > + > + env_set("fdtfile", fdtfile); > + > + return 0; > +} > +#endif > diff --git a/board/beagle/beagleplay/beagleplay.env b/board/beagle/beagleplay/beagleplay.env > index 4f0a94a8113e..647b25d14c8e 100644 > --- a/board/beagle/beagleplay/beagleplay.env > +++ b/board/beagle/beagleplay/beagleplay.env > @@ -1,5 +1,4 @@ > #include > -#include > #include > > name_kern=Image > diff --git a/configs/am62x_beagleplay_a53_defconfig b/configs/am62x_beagleplay_a53_defconfig > index 0be20045a974..1f43891d10bb 100644 > --- a/configs/am62x_beagleplay_a53_defconfig > +++ b/configs/am62x_beagleplay_a53_defconfig > @@ -33,7 +33,8 @@ CONFIG_AUTOBOOT_KEYED=y > CONFIG_AUTOBOOT_PROMPT="Press SPACE to abort autoboot in %d seconds\n" > CONFIG_AUTOBOOT_DELAY_STR="d" > CONFIG_AUTOBOOT_STOP_STR=" " > -CONFIG_BOOTCOMMAND="run set_led_state_start_load;run findfdt; run envboot; bootflow scan -lb;run set_led_state_fail_load" > +CONFIG_BOOTCOMMAND="run set_led_state_start_load; run envboot; bootflow scan -lb;run set_led_state_fail_load" > +CONFIG_BOARD_LATE_INIT=y > CONFIG_SPL_MAX_SIZE=0x58000 > CONFIG_SPL_HAS_BSS_LINKER_SECTION=y > CONFIG_SPL_BSS_START_ADDR=0x80c80000 > -- > 2.43.0 Reviewed-by: Jonathan Humphreys