mbox series

[edk2,edk2-platforms,v2,0/2] DeveloperBox: prepare for expanding the capsule payload

Message ID 20180613162826.19986-1-ard.biesheuvel@linaro.org
Headers show
Series DeveloperBox: prepare for expanding the capsule payload | expand

Message

Ard Biesheuvel June 13, 2018, 4:28 p.m. UTC 
We intend to start distributing firmware update capsules that include the SCP
firmware partition. In order to allow for more flexibility regarding whether
a capsule contains this piece or not, update the flash access routines and
the flash partition descriptions so we can update any part of the first
4 MB, consisting of the CM3 bootstrap code, emergency flasher, pseudo
EEPROM, SCP firmware, ARM-TF and UEFI.

Note that this means we can drop the 'TEMPORARY' patch I proposed a while
ago to hack around this limitation.

Changes since v1:
- use more parentheses in if () condition expression (#1)
- use symbolic constants for code and env NOR regions (#2)

Ard Biesheuvel (2):
  Silicon/SynQuacerPlatformFlashAccessLib: relax FV address check
  Silicon/NorFlashSynQuacerLib: describe entire firmware region as FV

 .../NorFlashSynQuacerLib/NorFlashSynQuacer.c  | 18 ++++--
 .../SynQuacerPlatformFlashAccessLib.c         | 56 +++++++++----------
 2 files changed, 39 insertions(+), 35 deletions(-)

-- 
2.17.1

_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel

Comments

Leif Lindholm June 13, 2018, 6:56 p.m. UTC | #1 
On Wed, Jun 13, 2018 at 06:28:24PM +0200, Ard Biesheuvel wrote:
> We intend to start distributing firmware update capsules that include the SCP

> firmware partition. In order to allow for more flexibility regarding whether

> a capsule contains this piece or not, update the flash access routines and

> the flash partition descriptions so we can update any part of the first

> 4 MB, consisting of the CM3 bootstrap code, emergency flasher, pseudo

> EEPROM, SCP firmware, ARM-TF and UEFI.

> 

> Note that this means we can drop the 'TEMPORARY' patch I proposed a while

> ago to hack around this limitation.

> 

> Changes since v1:

> - use more parentheses in if () condition expression (#1)

> - use symbolic constants for code and env NOR regions (#2)


For the series:
Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org>



> Ard Biesheuvel (2):

>   Silicon/SynQuacerPlatformFlashAccessLib: relax FV address check

>   Silicon/NorFlashSynQuacerLib: describe entire firmware region as FV

> 

>  .../NorFlashSynQuacerLib/NorFlashSynQuacer.c  | 18 ++++--

>  .../SynQuacerPlatformFlashAccessLib.c         | 56 +++++++++----------

>  2 files changed, 39 insertions(+), 35 deletions(-)

> 

> -- 

> 2.17.1

> 

_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
Ard Biesheuvel June 15, 2018, 10:51 a.m. UTC | #2 
On 13 June 2018 at 20:56, Leif Lindholm <leif.lindholm@linaro.org> wrote:
> On Wed, Jun 13, 2018 at 06:28:24PM +0200, Ard Biesheuvel wrote:

>> We intend to start distributing firmware update capsules that include the SCP

>> firmware partition. In order to allow for more flexibility regarding whether

>> a capsule contains this piece or not, update the flash access routines and

>> the flash partition descriptions so we can update any part of the first

>> 4 MB, consisting of the CM3 bootstrap code, emergency flasher, pseudo

>> EEPROM, SCP firmware, ARM-TF and UEFI.

>>

>> Note that this means we can drop the 'TEMPORARY' patch I proposed a while

>> ago to hack around this limitation.

>>

>> Changes since v1:

>> - use more parentheses in if () condition expression (#1)

>> - use symbolic constants for code and env NOR regions (#2)

>

> For the series:

> Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org>

>


Thanks

Pushed as 1cd14dea5fc1..fff2e99a5ee6

>

>> Ard Biesheuvel (2):

>>   Silicon/SynQuacerPlatformFlashAccessLib: relax FV address check

>>   Silicon/NorFlashSynQuacerLib: describe entire firmware region as FV

>>

>>  .../NorFlashSynQuacerLib/NorFlashSynQuacer.c  | 18 ++++--

>>  .../SynQuacerPlatformFlashAccessLib.c         | 56 +++++++++----------

>>  2 files changed, 39 insertions(+), 35 deletions(-)

>>

>> --

>> 2.17.1

>>

_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel