dra7x: fastboot: Increase recovery partition size

PATCH In review Praneeth Bajjuri 4 participants 5 messages Nov 8, 2017 – Nov 21, 2017

lore

Share & tools

lore

Praneeth Bajjuri

author

2017-11-08 23:12 UTC #0 lore raw

PATCH 1/1 dra7x: fastboot: Increase recovery partition size

As per current android recommendation
https://source.android.com/devices/architecture/kernel/modular-kernels

1. Android recovery mode should contain both SOC and ODM kernel modules in the recovery partition.

2. If a kernel module is required both in recovery and normal boot mode, the module has to be located in recovery and vendor partition seperately.

3. Kernel modules used in recovery mode should be independent of vendor and odm partition

4. Recovery image should contain atleast storage, display, keypad, battery and pmic modules.

Due to these requirements, recovery image size has increased to >10MB.

This patch is to increase recovery partition size for TI devices so that we dont see such flashing error

log:
sending 'recovery' (12560 KB)...
OKAY [ 0.436s]
writing 'recovery'...
FAILED (remote: too large for partition)
finished. total time: 0.458s

Signed-off-by Praneeth Bajjuri <[email protected]>

---
 include/configs/am57xx_evm.h | 2 +-
 include/configs/dra7xx_evm.h | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

include/configs/am57xx_evm.h +1 -1

index ebb0474..dc05bea 100644
@@ -56,7 +56,7 @@
 	"name=reserved,size=256K,uuid=${uuid_gpt_reserved};" \
 	"name=efs,size=16M,uuid=${uuid_gpt_efs};" \
 	"name=crypto,size=16K,uuid=${uuid_gpt_crypto};" \
-	"name=recovery,size=10M,uuid=${uuid_gpt_recovery};" \
+	"name=recovery,size=40M,uuid=${uuid_gpt_recovery};" \
 	"name=boot,size=10M,uuid=${uuid_gpt_boot};" \
 	"name=system,size=768M,uuid=${uuid_gpt_system};" \
 	"name=vendor,size=256M,uuid=${uuid_gpt_vendor};" \

include/configs/dra7xx_evm.h +1 -1

index 34a4555..f84c1f0 100644
@@ -63,7 +63,7 @@
 	"name=reserved,size=256K,uuid=${uuid_gpt_reserved};" \
 	"name=efs,size=16M,uuid=${uuid_gpt_efs};" \
 	"name=crypto,size=16K,uuid=${uuid_gpt_crypto};" \
-	"name=recovery,size=10M,uuid=${uuid_gpt_recovery};" \
+	"name=recovery,size=40M,uuid=${uuid_gpt_recovery};" \
 	"name=boot,size=10M,uuid=${uuid_gpt_boot};" \
 	"name=system,size=768M,uuid=${uuid_gpt_system};" \
 	"name=vendor,size=256M,uuid=${uuid_gpt_vendor};" \

Signature · 1 line

1.9.1

Sam Protsenko

2017-11-10 19:22 UTC #1 lore raw

PATCH 1/1 dra7x: fastboot: Increase recovery partition size

Quoted 62 lines · Praneeth Bajjuri

As per current android recommendation
https://source.android.com/devices/architecture/kernel/modular-kernels
1. Android recovery mode should contain both SOC and ODM kernel modules in the recovery partition.
2. If a kernel module is required both in recovery and normal boot mode, the module has to be located in recovery and vendor partition seperately.
3. Kernel modules used in recovery mode should be independent of vendor and odm partition
4. Recovery image should contain atleast storage, display, keypad, battery and pmic modules.
Due to these requirements, recovery image size has increased to >10MB.
This patch is to increase recovery partition size for TI devices so that we dont see such flashing error
log:
sending 'recovery' (12560 KB)...
OKAY [ 0.436s]
writing 'recovery'...
FAILED (remote: too large for partition)
finished. total time: 0.458s
Signed-off-by Praneeth Bajjuri <[email protected]>
---
 include/configs/am57xx_evm.h | 2 +-
 include/configs/dra7xx_evm.h | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)
include/configs/am57xx_evm.h +1 -1
index ebb0474..dc05bea 100644
@@ -56,7 +56,7 @@
        "name=reserved,size=256K,uuid=${uuid_gpt_reserved};" \
        "name=efs,size=16M,uuid=${uuid_gpt_efs};" \
        "name=crypto,size=16K,uuid=${uuid_gpt_crypto};" \
-       "name=recovery,size=10M,uuid=${uuid_gpt_recovery};" \
+       "name=recovery,size=40M,uuid=${uuid_gpt_recovery};" \
        "name=boot,size=10M,uuid=${uuid_gpt_boot};" \
        "name=system,size=768M,uuid=${uuid_gpt_system};" \
        "name=vendor,size=256M,uuid=${uuid_gpt_vendor};" \
include/configs/dra7xx_evm.h +1 -2
index 34a4555..f84c1f0 100644
@@ -63,7 +63,7 @@
        "name=reserved,size=256K,uuid=${uuid_gpt_reserved};" \
        "name=efs,size=16M,uuid=${uuid_gpt_efs};" \
        "name=crypto,size=16K,uuid=${uuid_gpt_crypto};" \
-       "name=recovery,size=10M,uuid=${uuid_gpt_recovery};" \
+       "name=recovery,size=40M,uuid=${uuid_gpt_recovery};" \
        "name=boot,size=10M,uuid=${uuid_gpt_boot};" \
        "name=system,size=768M,uuid=${uuid_gpt_system};" \
        "name=vendor,size=256M,uuid=${uuid_gpt_vendor};" \
--

Quoted 2 lines

1.9.1

Reviewed-by Sam Protsenko <[email protected]>

Bajjuri

2017-11-20 19:29 UTC #2 lore raw

PATCH 1/1 dra7x: fastboot: Increase recovery partition size

Tom,

Can you pull this , if there is nothing waiting to be addressed on this one.

--praneeth

Quoted 66 lines · Sam Protsenko

Quoted 62 lines · Praneeth Bajjuri
As per current android recommendation
https://source.android.com/devices/architecture/kernel/modular-kernels
1. Android recovery mode should contain both SOC and ODM kernel modules in the recovery partition.
2. If a kernel module is required both in recovery and normal boot mode, the module has to be located in recovery and vendor partition seperately.
3. Kernel modules used in recovery mode should be independent of vendor and odm partition
4. Recovery image should contain atleast storage, display, keypad, battery and pmic modules.
Due to these requirements, recovery image size has increased to >10MB.
This patch is to increase recovery partition size for TI devices so that we dont see such flashing error
log:
sending 'recovery' (12560 KB)...
OKAY [ 0.436s]
writing 'recovery'...
FAILED (remote: too large for partition)
finished. total time: 0.458s
Signed-off-by Praneeth Bajjuri <[email protected]>
---
  include/configs/am57xx_evm.h | 2 +-
  include/configs/dra7xx_evm.h | 2 +-
  2 files changed, 2 insertions(+), 2 deletions(-)
include/configs/am57xx_evm.h +1 -1
index ebb0474..dc05bea 100644
@@ -56,7 +56,7 @@
         "name=reserved,size=256K,uuid=${uuid_gpt_reserved};" \
         "name=efs,size=16M,uuid=${uuid_gpt_efs};" \
         "name=crypto,size=16K,uuid=${uuid_gpt_crypto};" \
-       "name=recovery,size=10M,uuid=${uuid_gpt_recovery};" \
+       "name=recovery,size=40M,uuid=${uuid_gpt_recovery};" \
         "name=boot,size=10M,uuid=${uuid_gpt_boot};" \
         "name=system,size=768M,uuid=${uuid_gpt_system};" \
         "name=vendor,size=256M,uuid=${uuid_gpt_vendor};" \
include/configs/dra7xx_evm.h +1 -2
index 34a4555..f84c1f0 100644
@@ -63,7 +63,7 @@
         "name=reserved,size=256K,uuid=${uuid_gpt_reserved};" \
         "name=efs,size=16M,uuid=${uuid_gpt_efs};" \
         "name=crypto,size=16K,uuid=${uuid_gpt_crypto};" \
-       "name=recovery,size=10M,uuid=${uuid_gpt_recovery};" \
+       "name=recovery,size=40M,uuid=${uuid_gpt_recovery};" \
         "name=boot,size=10M,uuid=${uuid_gpt_boot};" \
         "name=system,size=768M,uuid=${uuid_gpt_system};" \
         "name=vendor,size=256M,uuid=${uuid_gpt_vendor};" \
--
Quoted 2 lines
1.9.1
Reviewed-by Sam Protsenko <[email protected]>

Bajjuri

2017-11-20 19:33 UTC #3 lore raw

PATCH 1/1 dra7x: fastboot: Increase recovery partition size

In reply to Sam Protsenko

Hi Tom,

if there is nothing waiting to be addressed, Can you pull this patch.

-- Praneeth

Quoted 66 lines · Sam Protsenko

Quoted 62 lines · Praneeth Bajjuri
As per current android recommendation
https://source.android.com/devices/architecture/kernel/modular-kernels
1. Android recovery mode should contain both SOC and ODM kernel modules in the recovery partition.
2. If a kernel module is required both in recovery and normal boot mode, the module has to be located in recovery and vendor partition seperately.
3. Kernel modules used in recovery mode should be independent of vendor and odm partition
4. Recovery image should contain atleast storage, display, keypad, battery and pmic modules.
Due to these requirements, recovery image size has increased to >10MB.
This patch is to increase recovery partition size for TI devices so that we dont see such flashing error
log:
sending 'recovery' (12560 KB)...
OKAY [ 0.436s]
writing 'recovery'...
FAILED (remote: too large for partition)
finished. total time: 0.458s
Signed-off-by Praneeth Bajjuri <[email protected]>
---
  include/configs/am57xx_evm.h | 2 +-
  include/configs/dra7xx_evm.h | 2 +-
  2 files changed, 2 insertions(+), 2 deletions(-)
include/configs/am57xx_evm.h +1 -1
index ebb0474..dc05bea 100644
@@ -56,7 +56,7 @@
         "name=reserved,size=256K,uuid=${uuid_gpt_reserved};" \
         "name=efs,size=16M,uuid=${uuid_gpt_efs};" \
         "name=crypto,size=16K,uuid=${uuid_gpt_crypto};" \
-       "name=recovery,size=10M,uuid=${uuid_gpt_recovery};" \
+       "name=recovery,size=40M,uuid=${uuid_gpt_recovery};" \
         "name=boot,size=10M,uuid=${uuid_gpt_boot};" \
         "name=system,size=768M,uuid=${uuid_gpt_system};" \
         "name=vendor,size=256M,uuid=${uuid_gpt_vendor};" \
include/configs/dra7xx_evm.h +1 -2
index 34a4555..f84c1f0 100644
@@ -63,7 +63,7 @@
         "name=reserved,size=256K,uuid=${uuid_gpt_reserved};" \
         "name=efs,size=16M,uuid=${uuid_gpt_efs};" \
         "name=crypto,size=16K,uuid=${uuid_gpt_crypto};" \
-       "name=recovery,size=10M,uuid=${uuid_gpt_recovery};" \
+       "name=recovery,size=40M,uuid=${uuid_gpt_recovery};" \
         "name=boot,size=10M,uuid=${uuid_gpt_boot};" \
         "name=system,size=768M,uuid=${uuid_gpt_system};" \
         "name=vendor,size=256M,uuid=${uuid_gpt_vendor};" \
--
Quoted 2 lines
1.9.1
Reviewed-by Sam Protsenko <[email protected]>

Tom Rini

2017-11-21 13:05 UTC #4 lore raw

In reply to Praneeth Bajjuri

Quoted 29 lines · Praneeth Bajjuri

As per current android recommendation
https://source.android.com/devices/architecture/kernel/modular-kernels
1. Android recovery mode should contain both SOC and ODM kernel modules in the recovery partition.
2. If a kernel module is required both in recovery and normal boot mode, the module has to be located in recovery and vendor partition seperately.
3. Kernel modules used in recovery mode should be independent of vendor and odm partition
4. Recovery image should contain atleast storage, display, keypad, battery and pmic modules.
Due to these requirements, recovery image size has increased to >10MB.
This patch is to increase recovery partition size for TI devices so that we dont see such flashing error
log:
sending 'recovery' (12560 KB)...
OKAY [ 0.436s]
writing 'recovery'...
FAILED (remote: too large for partition)
finished. total time: 0.458s

Quoted 2 lines

Signed-off-by Praneeth Bajjuri <[email protected]>
Reviewed-by Sam Protsenko <[email protected]>

Applied to u-boot/master, thanks!

Signature · 8 lines

Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20171121/080e83ec/attachment.sig>