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 B73CCEB64DD for ; Tue, 25 Jul 2023 21:36:02 +0000 (UTC) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id 8C48386679; Tue, 25 Jul 2023 23:36:00 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=konsulko.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=konsulko.com header.i=@konsulko.com header.b="KUeZTG3T"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id E0B368636F; Tue, 25 Jul 2023 23:35:58 +0200 (CEST) Received: from mail-ua1-x92f.google.com (mail-ua1-x92f.google.com [IPv6:2607:f8b0:4864:20::92f]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits)) (No client certificate requested) by phobos.denx.de (Postfix) with ESMTPS id C981E86698 for ; Tue, 25 Jul 2023 23:35:55 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=konsulko.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=trini@konsulko.com Received: by mail-ua1-x92f.google.com with SMTP id a1e0cc1a2514c-79a23db59c3so864192241.2 for ; Tue, 25 Jul 2023 14:35:55 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=konsulko.com; s=google; t=1690320954; x=1690925754; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=YFhOgMvBsL1gRwS1YHIp3yqk9boMmofOfKRkv6LelZM=; b=KUeZTG3T6QfXfhCyjXCetTZ3xf2o2NDCbk/5sSVl7zJ/63Bht18+A9K6+rzKfIJ7Fp aSzqCIhCbfQOzNpMvN+mXtEonc5U5p+R47zbFeGyuYb/Cn+MtT6E0I49a7tPnNYxm/X8 VD2nBHJFjsnfPa/SN7NRzGdMwyPcvb452UnT0= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1690320954; x=1690925754; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=YFhOgMvBsL1gRwS1YHIp3yqk9boMmofOfKRkv6LelZM=; b=Vh+TCps8n+ZfnUi63xi3o25RSbE+IsUKHgSdWrt5DBM6ffOhX9Ov7VfMK/5XfvoOzU oHtO39gn8top6HS2dMs63Q3c17DKdNzWzJPE5TftT6ODi1PT8/VE2RdC6F0hTUBHbBfa A2b+X4ZRGV3U9fjz6y9pMPD4Fg/8qhIcyOSMR07VAvMEY7pqBrrx05o/FK5cTWDSlbS/ FOb0E8ch3I65k17b2PobBk79CH6mJxKQNqSykntT3SRpS56Koa1RvPWh8od131U+fLuN ZqSbD+pEu6ia2BKWSFl8dy8X0Mk8r7ch4PO7mwLwf7fuxRLFibf3GtHixIUdEc0aT6xR 1PGA== X-Gm-Message-State: ABy/qLZsIMd7qYUuQ17hMI5aguHVmclKp4m3JHqUSVUXhZ2buPtHoRcZ 7dMREZxeUPrGTJSA6RpcYEbFAw== X-Google-Smtp-Source: APBJJlGfyHLYQb7QUyuvhZUxz68naDPGLgjPftUjA480cLPvAn5DeA8NuVZGZr7O4p342uDAj0oopw== X-Received: by 2002:a67:fc92:0:b0:445:942:5c81 with SMTP id x18-20020a67fc92000000b0044509425c81mr108833vsp.20.1690320954510; Tue, 25 Jul 2023 14:35:54 -0700 (PDT) Received: from bill-the-cat (2603-6081-7b00-6400-b236-e3a5-57be-379d.res6.spectrum.com. [2603:6081:7b00:6400:b236:e3a5:57be:379d]) by smtp.gmail.com with ESMTPSA id u18-20020ab05b12000000b0075ef51c257dsm1800497uae.32.2023.07.25.14.35.53 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 25 Jul 2023 14:35:54 -0700 (PDT) Date: Tue, 25 Jul 2023 17:35:52 -0400 From: Tom Rini To: Nishanth Menon Cc: Neha Malcom Francis , Bryan Brattlof , Praneeth Bajjuri , Andrew Davis , Robert Nelson , Vignesh R , u-boot@lists.denx.de Subject: Re: [PATCH 6/6] doc: board: ti: Add beagleplay documentation Message-ID: <20230725213552.GJ3630934@bill-the-cat> References: <20230725185253.2123433-1-nm@ti.com> <20230725185253.2123433-7-nm@ti.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="A6E0OKdkZNue+16v" Content-Disposition: inline In-Reply-To: <20230725185253.2123433-7-nm@ti.com> X-Clacks-Overhead: GNU Terry Pratchett 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 --A6E0OKdkZNue+16v Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Tue, Jul 25, 2023 at 01:52:53PM -0500, Nishanth Menon wrote: > Add base documentation for beagleplay >=20 > Signed-off-by: Nishanth Menon > --- > doc/board/ti/am62x_beagleplay.rst | 169 ++++++++++++++++++++++++++++++ > doc/board/ti/k3.rst | 1 + > 2 files changed, 170 insertions(+) > create mode 100644 doc/board/ti/am62x_beagleplay.rst I think we have some room to improve things here, some right now and some as follow-up work. [snip] > +Sources: > +-------- > +1. Trusted Firmware-A: > + Tree: https://git.trustedfirmware.org/TF-A/trusted-firmware-a.git/ > + Branch: master > + > +2. OP-TEE: > + Tree: https://github.com/OP-TEE/optee_os.git > + Branch: master > + > +3. U-Boot: > + Tree: https://source.denx.de/u-boot/u-boot > + Branch: master > + > +4. TI Linux Firmware: > + Tree: git://git.ti.com/processor-firmware/ti-linux-firmware.git > + Branch: ti-linux-firmware > + > +Build procedure: > +---------------- > +1. Trusted Firmware-A: > + > +.. code-block:: bash > + > + $ make CROSS_COMPILE=3Daarch64-none-linux-gnu- ARCH=3Daarch64 PLAT=3Dk3= \ > + TARGET_BOARD=3Dlite SPD=3Dopteed > + > +2. OP-TEE: > + > +.. code-block:: bash > + > + $ make PLATFORM=3Dk3 CFG_ARM64_core=3Dy CROSS_COMPILE=3Darm-none-linux-= gnueabihf- \ > + CROSS_COMPILE64=3Daarch64-none-linux-gnu- So, we have some generic wording on how to build TF-A and OP-TEE up in doc/board/ti/k3.rst. It should be made clearer if needed that unless otherwise indicated, top of tree is expected to be used for TF-A/OP-TEE and ti-linux-firmware for the ti-linux-firmware repo. Then just state that TF-A should be which of {lite|generic|j784s4} as it is a $family device. And we then drop much of the duplication here. > +3. U-Boot: > + > +* 3.1 R5: > + > +.. code-block:: bash > + > + $ make ARCH=3Darm am62x_beagleplay_r5_defconfig > + $ make ARCH=3Darm CROSS_COMPILE=3Darm-none-linux-gnueabihf- \ > + BINMAN_INDIRS=3D > + > +* 3.2 A53: > + > +.. code-block:: bash > + > + $ make ARCH=3Darm am62x_beagleplay_a53_defconfig > + $ make ARCH=3Darm CROSS_COMPILE=3Daarch64-none-linux-gnu- \ > + BL31=3D/build/k3/lite/release/bl= 31.bin \ > + TEE=3D/out/arm-plat-k3/core/tee-raw.bin \ > + BINMAN_INDIRS=3D And can drop this. > +Target Images > +-------------- > +Copy the below images to an SD card and boot: > + > +* tiboot3-am62x-gp-evm.bin from step 3.1 as tiboot3.bin > +* tispl.bin_unsigned from step 3.2 as tispl.bin > +* u-boot.img_unsigned from step 3.2 as uboot.img Then reword this to be "from R5" or "from Axx" build. > +Image formats: > +-------------- > + > +The Image format follows the standard AM62X image format [1]_. > + > +A53 SPL DDR Memory Layout > +------------------------- This is unchanged for all AM62 platforms, yes? That should be in a generic am62 doc. --=20 Tom --A6E0OKdkZNue+16v Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQGzBAABCgAdFiEEGjx/cOCPqxcHgJu/FHw5/5Y0tywFAmTAQDUACgkQFHw5/5Y0 tyyIEwv/UX9m8uUnf0f8irqsjE0ClmYx/0MpCeVw5gJkbIqf/wTu1k1vCY13Jgjx smeZus/ps+Fe9pfQof8jk4Y1A/rBydnX0rtn57e6nRb/GMOBP2Wx+uPoAaKez1IT 0iEfbnJtRkewkSgxda/EX5I/T8MGuZWAvztnJ4j7XJO6K3rWmsZkk1oUoYR0mmt9 zwBZcjED2qo+z7D43HPmkxv6WgjHX2kW/LFHhK+lOB1g5YJ8+FCa7Yiv4w7sLLK/ kubhySBYjbsJPpGoNiQCSIjLx4w1zcYFXuFTGXE/1hrYgJFXAPL7CkOJWiG38qpV D11bQUzzIxIyrCISpi1Fz6pPPzsyJiYc8IxzHqNBf9XHII1ElQJvEZwQfT3OMsjt ETaletubW/wnk7ce18+USG1xrjiki422b9GjpWeTYpA30WVZJROKVSGO827dIE6h tJeyFmeRTsubcn9NsG7OgI3YeVSI1Bq1O89dUOmiph/pZbttwxiyvp5bJ5JTj7Ls S4z3IYYE =dRri -----END PGP SIGNATURE----- --A6E0OKdkZNue+16v--