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 BB8C4C001DC for ; Mon, 31 Jul 2023 12:01:20 +0000 (UTC) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id 1745086A55; Mon, 31 Jul 2023 14:01:10 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=none (p=none dis=none) header.from=linux.intel.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Authentication-Results: phobos.denx.de; dkim=pass (2048-bit key; unprotected) header.d=intel.com header.i=@intel.com header.b="NPp5WI6w"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id 61D9F86A47; Mon, 31 Jul 2023 10:09:33 +0200 (CEST) Received: from mgamail.intel.com (unknown [134.134.136.31]) (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 0A42486A49 for ; Mon, 31 Jul 2023 10:09:29 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=none (p=none dis=none) header.from=linux.intel.com Authentication-Results: phobos.denx.de; spf=none smtp.mailfrom=andriy.shevchenko@linux.intel.com DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1690790970; x=1722326970; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=82SZtLvPiW2HvT5+fUjnATM0OmRv1asou0N4BmY3ICE=; b=NPp5WI6wnQkYu8xRP/YbeLzvZr2VSFf7PsbYde9hrRh9QaUF3KvJtv31 lZ89lHA5qle+Z6a3tLFWjWht3m5C2WwV2TJhLQ9Rbjs56G5W0u+sTpAG+ 83Xn7SUVmflgjcGhnKb13QBWYvYGbW5J7ueanpQFCh5zlgDiUXJDb+BOp S7wmYCZjYYddMzt895YX8mulhbQvol5NTc7+HusTd04pjgOH6QqYOQXSX SYH6Lc/o0popv9PCBQ1IVlCxocDMkbr9+bnk3h4BSJ9BYXjP8nRf2+oGp /hRYtFdoF3FpfrHTiVQuBNnZK6WVDT20dBNGtmPW4O/jm+8rJjsOM6B7k Q==; X-IronPort-AV: E=McAfee;i="6600,9927,10787"; a="432762056" X-IronPort-AV: E=Sophos;i="6.01,244,1684825200"; d="scan'208";a="432762056" Received: from orsmga005.jf.intel.com ([10.7.209.41]) by orsmga104.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 31 Jul 2023 01:09:27 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10787"; a="902022221" X-IronPort-AV: E=Sophos;i="6.01,244,1684825200"; d="scan'208";a="902022221" Received: from smile.fi.intel.com ([10.237.72.54]) by orsmga005.jf.intel.com with ESMTP; 31 Jul 2023 01:09:20 -0700 Received: from andy by smile.fi.intel.com with local (Exim 4.96) (envelope-from ) id 1qQNxs-00EmjN-2X; Mon, 31 Jul 2023 11:09:16 +0300 Date: Mon, 31 Jul 2023 11:09:16 +0300 From: Andy Shevchenko To: Simon Glass Cc: U-Boot Mailing List , Bin Meng , Tom Rini , Adam Ford , Aiden Park , Aleksandar Gerasimovski , Andrew Davis , Anup Patel , Aswath Govindraju , Atish Patra , Bryan Brattlof , Dave Gerlach , "Derald D. Woods" , Heiko Schocher , Heinrich Schuchardt , Holger Brunck , Jan Kiszka , Joe Hershberger , Judith Mendez , Le Jin , Manorit Chawdhry , Neha Malcom Francis , Niel Fourie , Nikhil M Jain , Pali =?iso-8859-1?Q?Roh=E1r?= , Palmer Dabbelt , Parthiban Nallathambi , Paul Walmsley , Philip Oberfichtner , Rainer Boschung , Vignesh Raghavendra , Wolfgang Denk Subject: Re: [PATCH 00/22] x86: Move some boards to text environment Message-ID: References: <20230731030211.955875-1-sjg@chromium.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20230731030211.955875-1-sjg@chromium.org> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo X-Mailman-Approved-At: Mon, 31 Jul 2023 14:01:06 +0200 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 On Sun, Jul 30, 2023 at 09:01:43PM -0600, Simon Glass wrote: > This series starts the process of moving x86 boards to a text environment. > > This includes conversions for about half the boards. > > The rest (including the more complicated Chromebooks) can come in a later > series. Patch wise it looks good to me Reviewed-by: Andy Shevchenko # Intel Edison Where can I read about this new edison.env file and what is expected to be there? > doc: Explain how to avoid the distro-boot scripts > env: Use include/env for text-environment includes > env: Explain how to use #include files in text environment > x86: Drop CFG_SYS_STACK_SIZE > x86: i8254: Include required ibmpc.h header > x86: edison: Drop inclusion of ibmpc.h > x86: Drop inclusion of ibmpc.h > x86: Drop unused distro settings > x86: Add a common include for environment settings > x86: coreboot: Convert to text environment > x86: crownbay: Convert to text environment > x86: bayleybay: Convert to text environment > x86: galileo: Convert to text environment > x86: edison: Convert to text environment > x86: cherryhill: Convert to text environment > x86: cougarcanyon2: Convert to text environment > x86: minnowmax: Convert to text environment > x86: slimbootloader: Convert to text environment > x86: efi-x86_app: Convert to text environment > x86: efi-x86_payload: Convert to text environment > x86: qemu: Add required linux/sizes.h include > x86: qemu-x86: Convert to text environment > > arch/x86/cpu/qemu/dram.c | 1 + > arch/x86/cpu/qemu/e820.c | 1 + > arch/x86/lib/i8254.c | 1 + > arch/x86/lib/physmem.c | 3 +- > board/coreboot/coreboot/coreboot.env | 9 +++++ > board/efi/efi-x86_app/efi-x86_app.env | 6 ++++ > board/efi/efi-x86_payload/efi-x86_payload.env | 6 ++++ > board/emulation/qemu-x86/qemu-x86.env | 6 ++++ > board/intel/bayleybay/bayleybay.env | 9 +++++ > board/intel/cherryhill/cherryhill.env | 9 +++++ > board/intel/cougarcanyon2/cougarcanyon2.env | 6 ++++ > board/intel/crownbay/crownbay.env | 6 ++++ > board/intel/edison/edison.env | 6 ++++ > board/intel/galileo/galileo.env | 11 ++++++ > board/intel/minnowmax/minnowmax.env | 11 ++++++ > board/intel/slimbootloader/slimbootloader.env | 25 +++++++++++++ > board/keymile/km83xx/km83xx.env | 4 +-- > board/keymile/kmcent2/kmcent2.env | 2 +- > .../keymile/pg-wcom-ls102xa/pg-wcom-expu1.env | 2 +- > .../keymile/pg-wcom-ls102xa/pg-wcom-seli8.env | 2 +- > board/siemens/iot2050/iot2050.env | 2 +- > board/ti/am62ax/am62ax.env | 4 +-- > board/ti/am62x/am62x.env | 4 +-- > board/ti/am64x/am64x.env | 6 ++-- > board/ti/am65x/am65x.env | 8 ++--- > board/ti/j721e/j721e.env | 10 +++--- > board/ti/j721s2/j721s2.env | 10 +++--- > board/ti/ks2_evm/k2e_evm.env | 4 +-- > board/ti/ks2_evm/k2g_evm.env | 6 ++-- > board/ti/ks2_evm/k2hk_evm.env | 4 +-- > board/ti/ks2_evm/k2l_evm.env | 4 +-- > doc/usage/environment.rst | 8 ++++- > include/configs/am335x_evm.h | 2 +- > include/configs/am43xx_evm.h | 2 +- > include/configs/am57xx_evm.h | 2 +- > include/configs/am62ax_evm.h | 4 +-- > include/configs/am62x_evm.h | 2 +- > include/configs/am64x_evm.h | 4 +-- > include/configs/am65x_evm.h | 6 ++-- > include/configs/bayleybay.h | 17 --------- > include/configs/cherryhill.h | 13 ------- > include/configs/coreboot.h | 20 ----------- > include/configs/cougarcanyon2.h | 13 ------- > include/configs/crownbay.h | 17 --------- > include/configs/da850evm.h | 2 +- > include/configs/dra7xx_evm.h | 2 +- > include/configs/edison.h | 11 ------ > include/configs/efi-x86_app.h | 11 ------ > include/configs/efi-x86_payload.h | 17 --------- > include/configs/galileo.h | 19 ---------- > include/configs/minnowmax.h | 16 --------- > include/configs/omap3_evm.h | 2 +- > include/configs/omapl138_lcdk.h | 2 +- > include/configs/phycore_am335x_r2.h | 4 +-- > include/configs/qemu-x86.h | 23 ------------ > include/configs/sifive-unleashed.h | 2 +- > include/configs/slimbootloader.h | 35 ------------------- > include/configs/ti_omap4_common.h | 2 +- > include/configs/ti_omap5_common.h | 4 +-- > include/configs/x86-common.h | 19 ---------- > include/{environment => env}/distro/sf.h | 0 > .../{environment => env}/pg-wcom/common.env | 0 > .../{environment => env}/pg-wcom/ls102xa.env | 2 +- > .../{environment => env}/pg-wcom/powerpc.env | 0 > include/{environment => env}/ti/dfu.h | 0 > include/{environment => env}/ti/k3_dfu.env | 0 > include/{environment => env}/ti/k3_dfu.h | 0 > include/{environment => env}/ti/k3_rproc.env | 0 > include/{environment => env}/ti/k3_rproc.h | 0 > include/{environment => env}/ti/mmc.env | 0 > include/{environment => env}/ti/mmc.h | 0 > include/{environment => env}/ti/nand.env | 0 > include/{environment => env}/ti/nand.h | 0 > .../ti/ti_armv7_common.env | 0 > .../ti/ti_armv7_keystone2.env | 0 > include/{environment => env}/ti/ufs.env | 0 > include/{environment => env}/ti/ufs.h | 0 > include/env/x86.env | 20 +++++++++++ > 78 files changed, 200 insertions(+), 291 deletions(-) > create mode 100644 board/coreboot/coreboot/coreboot.env > create mode 100644 board/efi/efi-x86_app/efi-x86_app.env > create mode 100644 board/efi/efi-x86_payload/efi-x86_payload.env > create mode 100644 board/emulation/qemu-x86/qemu-x86.env > create mode 100644 board/intel/bayleybay/bayleybay.env > create mode 100644 board/intel/cherryhill/cherryhill.env > create mode 100644 board/intel/cougarcanyon2/cougarcanyon2.env > create mode 100644 board/intel/crownbay/crownbay.env > create mode 100644 board/intel/edison/edison.env > create mode 100644 board/intel/galileo/galileo.env > create mode 100644 board/intel/minnowmax/minnowmax.env > create mode 100644 board/intel/slimbootloader/slimbootloader.env > rename include/{environment => env}/distro/sf.h (100%) > rename include/{environment => env}/pg-wcom/common.env (100%) > rename include/{environment => env}/pg-wcom/ls102xa.env (95%) > rename include/{environment => env}/pg-wcom/powerpc.env (100%) > rename include/{environment => env}/ti/dfu.h (100%) > rename include/{environment => env}/ti/k3_dfu.env (100%) > rename include/{environment => env}/ti/k3_dfu.h (100%) > rename include/{environment => env}/ti/k3_rproc.env (100%) > rename include/{environment => env}/ti/k3_rproc.h (100%) > rename include/{environment => env}/ti/mmc.env (100%) > rename include/{environment => env}/ti/mmc.h (100%) > rename include/{environment => env}/ti/nand.env (100%) > rename include/{environment => env}/ti/nand.h (100%) > rename include/{environment => env}/ti/ti_armv7_common.env (100%) > rename include/{environment => env}/ti/ti_armv7_keystone2.env (100%) > rename include/{environment => env}/ti/ufs.env (100%) > rename include/{environment => env}/ti/ufs.h (100%) > create mode 100644 include/env/x86.env > > -- > 2.41.0.487.g6d72f3e995-goog > -- With Best Regards, Andy Shevchenko