Re: [U-Boot] [PATCH v0 13/20] efi_loader: use proper device-paths for partitions

On Mon, Aug 07, 2017 at 12:16:54PM -0400, Rob Clark wrote: > On Mon, Aug 7, 2017 at 11:47 AM, Jonathan Gray wrote: > > On Sun, Aug 06, 2017 at 11:34:15AM -0400, Rob Clark wrote: > >> On Sun, Aug 6, 2017 at 10:45 AM, Rob Clark wrote: > >> > > >> > I've started

Re: [U-Boot] [PATCH v0 13/20] efi_loader: use proper device-paths for partitions

On Fri, Aug 04, 2017 at 10:41:32PM +0200, Mark Kettenis wrote: [...] > ..and what you're sketching out here should work with recent enough > versions of our bootloader. > > However, to me having an ACPI Device Path component in there doesn't > make much sense on a board without ACPI. It

Re: [U-Boot] [PATCH v0 13/20] efi_loader: use proper device-paths for partitions

On Mon, Aug 7, 2017 at 11:47 AM, Jonathan Gray wrote: > On Sun, Aug 06, 2017 at 11:34:15AM -0400, Rob Clark wrote: >> On Sun, Aug 6, 2017 at 10:45 AM, Rob Clark wrote: >> > >> > I've started trying to hack up test_efi_loader.py to add a test that >> > loads

Re: [U-Boot] [PATCH v0 13/20] efi_loader: use proper device-paths for partitions

On Sun, Aug 06, 2017 at 11:34:15AM -0400, Rob Clark wrote: > On Sun, Aug 6, 2017 at 10:45 AM, Rob Clark wrote: > > > > I've started trying to hack up test_efi_loader.py to add a test that > > loads OpenBSD's bootloader.. kinda muddling through it at this point, > > since not

Re: [U-Boot] [PATCH v0 13/20] efi_loader: use proper device-paths for partitions

On Sun, Aug 6, 2017 at 2:47 PM, Rob Clark wrote: > On Sun, Aug 6, 2017 at 2:37 PM, Mark Kettenis wrote: >>> Date: Sun, 6 Aug 2017 20:21:45 +0200 (CEST) >>> From: Mark Kettenis >>> >>> > Mind sending me or pastebin'ing your

Re: [U-Boot] [PATCH v0 13/20] efi_loader: use proper device-paths for partitions

On Sun, Aug 6, 2017 at 2:37 PM, Mark Kettenis wrote: >> Date: Sun, 6 Aug 2017 20:21:45 +0200 (CEST) >> From: Mark Kettenis >> >> > Mind sending me or pastebin'ing your u-boot .config? There are some >> > different device-path construction

Re: [U-Boot] [PATCH v0 13/20] efi_loader: use proper device-paths for partitions

On Sun, Aug 6, 2017 at 2:21 PM, Mark Kettenis wrote: >> From: Rob Clark >> Date: Sun, 6 Aug 2017 13:49:43 -0400 >> >> On Sun, Aug 6, 2017 at 1:28 PM, Mark Kettenis >> wrote: >> >> From: Rob Clark >> >>

Re: [U-Boot] [PATCH v0 13/20] efi_loader: use proper device-paths for partitions

> Date: Sun, 6 Aug 2017 20:21:45 +0200 (CEST) > From: Mark Kettenis > > > Mind sending me or pastebin'ing your u-boot .config? There are some > > different device-path construction depending on legacy vs > > CONFIG_DM+CONFIG_BLK (the legacy case *looks* right to me, and

Re: [U-Boot] [PATCH v0 13/20] efi_loader: use proper device-paths for partitions

> From: Rob Clark > Date: Sun, 6 Aug 2017 13:49:43 -0400 > > On Sun, Aug 6, 2017 at 1:28 PM, Mark Kettenis wrote: > >> From: Rob Clark > >> Date: Sun, 6 Aug 2017 11:34:15 -0400 > >> > >> On Sun, Aug 6, 2017 at 10:45 AM, Rob

Re: [U-Boot] [PATCH v0 13/20] efi_loader: use proper device-paths for partitions

On Sun, Aug 6, 2017 at 6:49 PM, Rob Clark wrote: > On Sun, Aug 6, 2017 at 1:28 PM, Mark Kettenis wrote: >>> From: Rob Clark >>> Date: Sun, 6 Aug 2017 11:34:15 -0400 >>> >>> On Sun, Aug 6, 2017 at 10:45 AM, Rob Clark

Re: [U-Boot] [PATCH v0 13/20] efi_loader: use proper device-paths for partitions

On Sun, Aug 6, 2017 at 1:28 PM, Mark Kettenis wrote: >> From: Rob Clark >> Date: Sun, 6 Aug 2017 11:34:15 -0400 >> >> On Sun, Aug 6, 2017 at 10:45 AM, Rob Clark wrote: >> > >> > I've started trying to hack up test_efi_loader.py

Re: [U-Boot] [PATCH v0 13/20] efi_loader: use proper device-paths for partitions

> From: Rob Clark > Date: Sun, 6 Aug 2017 11:34:15 -0400 > > On Sun, Aug 6, 2017 at 10:45 AM, Rob Clark wrote: > > > > I've started trying to hack up test_efi_loader.py to add a test that > > loads OpenBSD's bootloader.. kinda muddling through it at

Re: [U-Boot] [PATCH v0 13/20] efi_loader: use proper device-paths for partitions

On Sun, Aug 06, 2017 at 11:34:15AM -0400, Rob Clark wrote: > On Sun, Aug 6, 2017 at 10:45 AM, Rob Clark wrote: > > > > I've started trying to hack up test_efi_loader.py to add a test that > > loads OpenBSD's bootloader.. kinda muddling through it at this point, > > since not

Re: [U-Boot] [PATCH v0 13/20] efi_loader: use proper device-paths for partitions

On 08/06/2017 05:34 PM, Rob Clark wrote: > On Sun, Aug 6, 2017 at 10:45 AM, Rob Clark wrote: >> >> I've started trying to hack up test_efi_loader.py to add a test that >> loads OpenBSD's bootloader.. kinda muddling through it at this point, >> since not a py expert or too

Re: [U-Boot] [PATCH v0 13/20] efi_loader: use proper device-paths for partitions

On Sun, Aug 6, 2017 at 10:45 AM, Rob Clark wrote: > > I've started trying to hack up test_efi_loader.py to add a test that > loads OpenBSD's bootloader.. kinda muddling through it at this point, > since not a py expert or too familiar w/ u-boot's test framework. But > I'll

Re: [U-Boot] [PATCH v0 13/20] efi_loader: use proper device-paths for partitions

On Sun, Aug 6, 2017 at 10:28 AM, Mark Kettenis wrote: >> Date: Sun, 6 Aug 2017 15:16:09 +0200 (CEST) >> From: Mark Kettenis >> >> Things are already broken before that commit though, so there is >> another problem. I'll see if I can figure out

Re: [U-Boot] [PATCH v0 13/20] efi_loader: use proper device-paths for partitions

> Date: Sun, 6 Aug 2017 15:16:09 +0200 (CEST) > From: Mark Kettenis > > Things are already broken before that commit though, so there is > another problem. I'll see if I can figure out what it is... data abort pc : [<7ef59160>] lr : [<7ef59118>] reloc pc :

Re: [U-Boot] [PATCH v0 13/20] efi_loader: use proper device-paths for partitions

On Sun, Aug 6, 2017 at 10:17 AM, Rob Clark wrote: > On Sun, Aug 6, 2017 at 9:16 AM, Mark Kettenis wrote: >>> From: Rob Clark >>> Date: Sat, 5 Aug 2017 11:36:25 -0400 >>> >>> On Sat, Aug 5, 2017 at 11:24 AM, Rob Clark

Re: [U-Boot] [PATCH v0 13/20] efi_loader: use proper device-paths for partitions

On Sun, Aug 6, 2017 at 9:16 AM, Mark Kettenis wrote: >> From: Rob Clark >> Date: Sat, 5 Aug 2017 11:36:25 -0400 >> >> On Sat, Aug 5, 2017 at 11:24 AM, Rob Clark wrote: >> > On Sat, Aug 5, 2017 at 11:10 AM, Heinrich Schuchardt

Re: [U-Boot] [PATCH v0 13/20] efi_loader: use proper device-paths for partitions

> From: Rob Clark > Date: Sat, 5 Aug 2017 11:36:25 -0400 > > On Sat, Aug 5, 2017 at 11:24 AM, Rob Clark wrote: > > On Sat, Aug 5, 2017 at 11:10 AM, Heinrich Schuchardt > > wrote: > >> On 08/05/2017 04:35 PM, Rob Clark wrote: > >>>

Re: [U-Boot] [PATCH v0 13/20] efi_loader: use proper device-paths for partitions

> From: Rob Clark > Date: Sat, 5 Aug 2017 10:28:34 -0400 > > On Sat, Aug 5, 2017 at 10:01 AM, Mark Kettenis > wrote: > > > > OpenBSD doesn't run on the db410c. However, our EFI bootloader should > > just run. You can download it from: > > > >

Re: [U-Boot] [PATCH v0 13/20] efi_loader: use proper device-paths for partitions

On Sat, Aug 5, 2017 at 12:02 PM, Heinrich Schuchardt wrote: > On 08/05/2017 05:22 PM, Rob Clark wrote: >> On Sat, Aug 5, 2017 at 11:08 AM, Mark Kettenis >> wrote: From: Rob Clark Date: Sat, 5 Aug 2017 10:35:08 -0400

Re: [U-Boot] [PATCH v0 13/20] efi_loader: use proper device-paths for partitions

On 08/05/2017 05:22 PM, Rob Clark wrote: > On Sat, Aug 5, 2017 at 11:08 AM, Mark Kettenis > wrote: >>> From: Rob Clark >>> Date: Sat, 5 Aug 2017 10:35:08 -0400 >>> >>> On Sat, Aug 5, 2017 at 10:28 AM, Mark Kettenis >>>

Re: [U-Boot] [PATCH v0 13/20] efi_loader: use proper device-paths for partitions

On Sat, Aug 5, 2017 at 11:24 AM, Rob Clark wrote: > On Sat, Aug 5, 2017 at 11:10 AM, Heinrich Schuchardt > wrote: >> On 08/05/2017 04:35 PM, Rob Clark wrote: >>> On Sat, Aug 5, 2017 at 10:28 AM, Mark Kettenis >>> wrote: >

Re: [U-Boot] [PATCH v0 13/20] efi_loader: use proper device-paths for partitions

On Sat, Aug 5, 2017 at 11:10 AM, Heinrich Schuchardt wrote: > On 08/05/2017 04:35 PM, Rob Clark wrote: >> On Sat, Aug 5, 2017 at 10:28 AM, Mark Kettenis >> wrote: Date: Sat, 5 Aug 2017 16:01:51 +0200 (CEST) From: Mark Kettenis

Re: [U-Boot] [PATCH v0 13/20] efi_loader: use proper device-paths for partitions

On Sat, Aug 5, 2017 at 11:08 AM, Mark Kettenis wrote: >> From: Rob Clark >> Date: Sat, 5 Aug 2017 10:35:08 -0400 >> >> On Sat, Aug 5, 2017 at 10:28 AM, Mark Kettenis >> wrote: >> >> Date: Sat, 5 Aug 2017 16:01:51 +0200

Re: [U-Boot] [PATCH v0 13/20] efi_loader: use proper device-paths for partitions

On 08/05/2017 04:35 PM, Rob Clark wrote: > On Sat, Aug 5, 2017 at 10:28 AM, Mark Kettenis > wrote: >>> Date: Sat, 5 Aug 2017 16:01:51 +0200 (CEST) >>> From: Mark Kettenis >>> >>> Unfortunately something in this patch series breaks things for me

Re: [U-Boot] [PATCH v0 13/20] efi_loader: use proper device-paths for partitions

> From: Rob Clark > Date: Sat, 5 Aug 2017 10:35:08 -0400 > > On Sat, Aug 5, 2017 at 10:28 AM, Mark Kettenis > wrote: > >> Date: Sat, 5 Aug 2017 16:01:51 +0200 (CEST) > >> From: Mark Kettenis > >> > >> Unfortunately

Re: [U-Boot] [PATCH v0 13/20] efi_loader: use proper device-paths for partitions

On Sat, Aug 5, 2017 at 10:28 AM, Mark Kettenis wrote: >> Date: Sat, 5 Aug 2017 16:01:51 +0200 (CEST) >> From: Mark Kettenis >> >> Unfortunately something in this patch series breaks things for me on a >> Banana Pi: > > And according to git

Re: [U-Boot] [PATCH v0 13/20] efi_loader: use proper device-paths for partitions

> Date: Sat, 5 Aug 2017 16:01:51 +0200 (CEST) > From: Mark Kettenis > > Unfortunately something in this patch series breaks things for me on a > Banana Pi: And according to git bisect: 4e3e748a50fc3f43e20c7ff407184596d7c9a589 is the first bad commit commit

Re: [U-Boot] [PATCH v0 13/20] efi_loader: use proper device-paths for partitions

On Sat, Aug 5, 2017 at 10:01 AM, Mark Kettenis wrote: > > OpenBSD doesn't run on the db410c. However, our EFI bootloader should > just run. You can download it from: > > http://ftp.openbsd.org/pub/OpenBSD/snapshots/arm64/BOOTAA64.EFI > > for the 64-bit (AArch64) and >

Re: [U-Boot] [PATCH v0 13/20] efi_loader: use proper device-paths for partitions

On Sat, Aug 5, 2017 at 10:01 AM, Mark Kettenis wrote: >> >> On Fri, Aug 4, 2017 at 4:41 PM, Mark Kettenis >> wrote: >> >> From: Rob Clark >> >> Date: Fri, 4 Aug 2017 15:31:55 -0400 >> > >> > Hi Rob, >> > >> > OpenBSD has

Re: [U-Boot] [PATCH v0 13/20] efi_loader: use proper device-paths for partitions

> Authentication-Results: xs4all.nl; spf=pass smtp.mailfrom=gmail.com; > dkim=pass header.d=gmail.com; dmarc=pass header.from=gmail.com > DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; > d=gmail.com; s=20161025; >

Re: [U-Boot] [PATCH v0 13/20] efi_loader: use proper device-paths for partitions

On Fri, Aug 4, 2017 at 4:41 PM, Mark Kettenis wrote: >> From: Rob Clark >> Date: Fri, 4 Aug 2017 15:31:55 -0400 > > Hi Rob, > > OpenBSD has been an early adopter of efi_loader and pretty much > completely relies on it for booting OpenBSD/armv7 and

Re: [U-Boot] [PATCH v0 13/20] efi_loader: use proper device-paths for partitions

> From: Rob Clark > Date: Fri, 4 Aug 2017 15:31:55 -0400 Hi Rob, OpenBSD has been an early adopter of efi_loader and pretty much completely relies on it for booting OpenBSD/armv7 and OpenBSD/arm64. We use our own bootloader which is fairly lightweight. Obviously we'd like

[U-Boot] [PATCH v0 13/20] efi_loader: use proper device-paths for partitions

Also, create disk objects for the disk itself, in addition to the partitions. (UEFI terminology is a bit confusing, a "disk" object is really a partition.) This helps grub properly identify the boot device since it is trying to match up partition "disk" object with it's parent device. Now