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

Ard Biesheuvel posted 2 patches 5 years, 11 months ago
Failed in applying to current master (apply log)
.../NorFlashSynQuacerLib/NorFlashSynQuacer.c  | 18 ++++--
.../SynQuacerPlatformFlashAccessLib.c         | 56 +++++++++----------
2 files changed, 39 insertions(+), 35 deletions(-)
[edk2] [PATCH edk2-platforms v2 0/2]DeveloperBox: prepare for expanding the capsule payload
Posted by Ard Biesheuvel 5 years, 11 months ago
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
Re: [edk2] [PATCH edk2-platforms v2 0/2]DeveloperBox: prepare for expanding the capsule payload
Posted by Leif Lindholm 5 years, 11 months ago
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
Re: [edk2] [PATCH edk2-platforms v2 0/2]DeveloperBox: prepare for expanding the capsule payload
Posted by Ard Biesheuvel 5 years, 11 months ago
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