From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=2a00:1450:4864:20::431; helo=mail-wr1-x431.google.com; envelope-from=leif.lindholm@linaro.org; receiver=edk2-devel@lists.01.org Received: from mail-wr1-x431.google.com (mail-wr1-x431.google.com [IPv6:2a00:1450:4864:20::431]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id B9AB921194897 for ; Wed, 12 Dec 2018 10:32:48 -0800 (PST) Received: by mail-wr1-x431.google.com with SMTP id u3so18733752wrs.3 for ; Wed, 12 Dec 2018 10:32:48 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=WoDdzwmYUQ6XGt4Jwp+QeNAfWYLicWgXnWGlV4afKZY=; b=U4lD5WPUDJawDPMeHI1L64tLd+iCqq+hA6bWGwaZVV8fAc7ZDuhxQKmQ+DyGLfhI1P VZ9ZNcji0wECgs2rg3ymzj2yi2O7fj/fWPgA3QSfel21vKMbPDJUbJaiddVZiFFApZ3f pvS1P0UAJZiJFHyBhVD5eodzjd7iLZTutNFJU= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=WoDdzwmYUQ6XGt4Jwp+QeNAfWYLicWgXnWGlV4afKZY=; b=YJgTECSyqS9ogDwpwI3Zs6uyFz9FHB/RERPabs1tNTZnbdduXoNOPyhzy2E4biuD1U ID2/dWeBBaFOlbh0l+lYHGsbPl/29Fa/kCKGdb7vRwoXD/9ANUbseUJ2Z84cv6eSumOq s5YN3pVT1fjhwK0YASz4UAoFVvHj8FPMC2j9jIURDB6KKL6ZJa+0j5el4lkj1HCKgg36 7e7TtjSoiNHL9/JYQYpk517Yqa4ARo/PmECMFyZIObjy1ZzmHLpXONvlCAKrNUmG0Ia9 jAoQJDNRoI3sgDXqtqeZnwkCxfIX04FLESSQVol6i0w9Y8ZtKr+I+ZpTAw/2tG3NAdA0 zNiw== X-Gm-Message-State: AA+aEWYVUnmT0Y6STSMbWAgFW4R0Yb9fS+Q2/eL3RDd9r1PghM1bZDF8 DHsH11m5MVsUjdw9E8WAg+z+iOZbegk= X-Google-Smtp-Source: AFSGD/W4zD3FQfYKXXpCUxDNe/4CvUoQHbK+zn9rEdbLBhpC/o0bYr4KiiAT1ibWJa7WCmqAaQGI5A== X-Received: by 2002:adf:d089:: with SMTP id y9mr18684861wrh.22.1544639566986; Wed, 12 Dec 2018 10:32:46 -0800 (PST) Received: from bivouac.eciton.net (bivouac.eciton.net. [2a00:1098:0:86:1000:23:0:2]) by smtp.gmail.com with ESMTPSA id a12sm18636139wrm.45.2018.12.12.10.32.45 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Wed, 12 Dec 2018 10:32:45 -0800 (PST) Date: Wed, 12 Dec 2018 18:32:44 +0000 From: Leif Lindholm To: Pete Batard Cc: edk2-devel@lists.01.org, Ard Biesheuvel Message-ID: <20181212183244.sn3h6sgsmz6s4hti@bivouac.eciton.net> References: <20181210123853.4864-1-pete@akeo.ie> <20181211181040.7i6dfxrl4kfcxspz@bivouac.eciton.net> MIME-Version: 1.0 In-Reply-To: User-Agent: NeoMutt/20170113 (1.7.2) Subject: Re: [PATCH v2 edk2-platforms 00/20] Platform/Broadcom: Add Raspberry Pi 3 support X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 12 Dec 2018 18:32:49 -0000 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Tue, Dec 11, 2018 at 08:16:07PM +0000, Pete Batard wrote: > > I _think_ all of the ATF binaries we have in non-osi are > > non-upstream. If the port for the rpi3 is upstream, I would be just as > > happy to have simple build instructions of a known good commit (with > > notes on toolchain version tested) in the Readme.md - and possibly a > > placeholder directory with a .inf in to drop a prebuilt image into. > > Well, while it is upstream, it is built using a custom rpi3 specific option > which we had ATF to add, so that we could get a memory mapping that works > with Windows (default one was okay for Linux but not Windows). So I doubt we > will ever get upstream binaries that we can use as is, if that's what you > are alluding to. I don't really care all that much about that, but if we need modifications to run Windows (and nothing prevents that from working with Linux), then should we try to change the upstream defaults? But for me, I'd be happy with just the build instructions you have, and no binaries/license, in edk2-platforms/Platform/RaspberryPi/RPi3/Arm-Tf/, with the user having to drop in their own binaries. > For the record, there's a readme located in the directory where the ATF > binaries are provided, that has the full build command we used, as well as > the description of the memory mapping. Good stuff. > > (If it isn't upstream, non-osi is the way to go for now.) > > ((This isn't a "do what I say", this is a "you don't have to".)) > > I guess that means we'll keep the binaries in non-osi for now then. > > However, I have been thinking about renaming the directory that contains the > ATF blobs from "Binary/" to "Atf/" to make it more explicit. I'll probably > do for the v3, unless someone has a different idea. Your call, after reading the above. > > I fully agree with this, but... > > > > Now for the bikeshedding: Bcm2837 is the SoC used in Pi 3. It is not > > an alternative name for the Pi 3. And since the board design is open, > > it is plausible that there may be derivative boards. > > So ideally, I would like to see something like: > > > > Platform/RaspberryPi/Pi3 > > Silicon/Broadcom/Bcm283x > > > > With (if practically possible) a split between SoC and board modules > > and configuration files. > > Okay. I was half expecting such a request, so I'll see what I can do. Thanks :) > I'll be waiting to see if Ard has additional feedback before I start working > on this as part of a v3. > > > I would expect the Pi3.dsc/.fdf to be fairly minimal and including > > .dsc.inc/.fdf.inc files from Bcm283x. > > > > > * The ARM Trusted Firmware being used is a vanilla version built from the > > > latest tree, as we worked with that project to get necessary patches > > > integrated. > > > > Sweet! > > > > > * Detailed instructions on how to build and test the platform firmware are > > > included in the Readme.md found at the root of the platform. > > > > Splendid! > > > > > * As detailed in the Readme, the resulting platform firmware has been > > > successfully used to install and run Linux OSes, such as Ubuntu 18.10, as > > > well as Windows 10 1809 (*full* UI version, not IoT). > > > > Very nice! > > > > Could you also add an entry to the top-level Readme.md with a link to > > the rpi3 Readme.md? (But hold off until I push some updates from > > Nariman.) > > Will do. (This has now been partly pushed.) Regards, Leif