Re: Falcon Mode Support For Uncompressed Kernel Images

2022-04-19 Thread Tom Rini
On Tue, Apr 19, 2022 at 05:31:52PM -0400, Nathan Barrett-Morrison wrote: > Hi Tom, > > While trying to bring up Falcon Mode boot on an ARM64 board, I discovered > that there is no path which allows you to use an uncompressed kernel image > (booti). I've added this path and attached the relevant

Falcon Mode Support For Uncompressed Kernel Images

2022-04-19 Thread Nathan Barrett-Morrison
Hi Tom, While trying to bring up Falcon Mode boot on an ARM64 board, I discovered that there is no path which allows you to use an uncompressed kernel image (booti). I've added this path and attached the relevant patch. Sincerely, Nathan >From f1c34333f79996bd2927a60f4858c01699431cba Mon Sep

Re: Falcon Mode Support For Uncompressed Kernel Images

2022-04-19 Thread Tom Rini
On Tue, Apr 19, 2022 at 05:16:21PM -0400, Nathan Barrett-Morrison wrote: > Hi Tom, > > Let's see if this works any better. Using a new mail client that is supposed > to not modify whitespace: > > From f1c34333f79996bd2927a60f4858c01699431cba Mon Sep 17 00:00:00 2001 > From: Nathan Barrett

Re: Falcon Mode Support For Uncompressed Kernel Images

2022-04-19 Thread Nathan Barrett-Morrison
One more time -- >From f1c34333f79996bd2927a60f4858c01699431cba Mon Sep 17 00:00:00 2001 From: Nathan Barrett-Morrison Date: Wed, 2 Feb 2022 15:05:18 -0500 Subject: [PATCH v2] Add in the ability to load and boot an uncompressed kernel image during the Falcon Mode boot sequence. This is

Re: Falcon Mode Support For Uncompressed Kernel Images

2022-04-19 Thread Nathan Barrett-Morrison
Hi Tom, Let's see if this works any better. Using a new mail client that is supposed to not modify whitespace: >From f1c34333f79996bd2927a60f4858c01699431cba Mon Sep 17 00:00:00 2001 From: Nathan Barrett Morrison Date: Wed, 2 Feb 2022 15:05:18 -0500 Subject: [PATCH v2] Add in the ability to

Re: Falcon Mode Support For Uncompressed Kernel Images

2022-04-19 Thread Tom Rini
On Wed, Feb 02, 2022 at 04:53:11PM -0500, Nathan Barrett-Morrison wrote: > Hi Tom, > > I've attached version 2 of the patch. I believe this should adhere more > closely to your suggestions. > > Keep in mind, CONFIG_IS_ENABLED will not work for checking > CMD_BOOTI/CMD_BOOTZ in this case, as

Re: Falcon Mode Support For Uncompressed Kernel Images

2022-02-02 Thread Nathan Barrett-Morrison
Hi Tom, I've attached version 2 of the patch. I believe this should adhere more closely to your suggestions. Keep in mind, CONFIG_IS_ENABLED will not work for checking CMD_BOOTI/CMD_BOOTZ in this case, as there is no CONFIG_SPL_CMD_BOOTx. So, I'm just directly checking if it's defined instead.

Re: Falcon Mode Support For Uncompressed Kernel Images

2022-01-31 Thread Tom Rini
On Mon, Jan 31, 2022 at 10:23:58AM -0500, Nathan Barrett-Morrison wrote: > Hi Tom, > > Yea, I'm not sure if uncompressed ARM32 would work, but I don't believe it > was ever working to begin with... as bootz_setup is being called right now > ( @

Re: Falcon Mode Support For Uncompressed Kernel Images

2022-01-31 Thread Nathan Barrett-Morrison
Hi Tom, Yea, I'm not sure if uncompressed ARM32 would work, but I don't believe it was ever working to begin with... as bootz_setup is being called right now ( @ https://source.denx.de/u-boot/u-boot/-/blob/master/common/spl/spl.c#L366 ) My intent was for SPL_OS_BOOT_UNCOMPRESSED to only be used

Re: Falcon Mode Support For Uncompressed Kernel Images

2022-01-31 Thread Tom Rini
On Mon, Jan 17, 2022 at 07:58:00PM -0500, Nathan Barrett-Morrison wrote: > Hi All, > > While trying to bring up Falcon Mode boot on an ARM64 board, I discovered > that there is no path which allows you to use an uncompressed kernel image > (booti). I've added this path and attached the relevant

Falcon Mode Support For Uncompressed Kernel Images

2022-01-17 Thread Nathan Barrett-Morrison
Hi All, While trying to bring up Falcon Mode boot on an ARM64 board, I discovered that there is no path which allows you to use an uncompressed kernel image (booti). I've added this path and attached the relevant patch. I've made this a separate if/else CONFIG option instead of allowing both