* [U-Boot] [PATCH] dra7x: fastboot: Increase recovery partition size
@ 2017-11-08 23:12 Praneeth Bajjuri
2017-11-10 19:22 ` Sam Protsenko
2017-11-21 13:05 ` [U-Boot] " Tom Rini
0 siblings, 2 replies; 5+ messages in thread
From: Praneeth Bajjuri @ 2017-11-08 23:12 UTC (permalink / raw)
To: u-boot
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 <praneeth@ti.com>
---
include/configs/am57xx_evm.h | 2 +-
include/configs/dra7xx_evm.h | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/include/configs/am57xx_evm.h b/include/configs/am57xx_evm.h
index ebb0474..dc05bea 100644
--- a/include/configs/am57xx_evm.h
+++ b/include/configs/am57xx_evm.h
@@ -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};" \
diff --git a/include/configs/dra7xx_evm.h b/include/configs/dra7xx_evm.h
index 34a4555..f84c1f0 100644
--- a/include/configs/dra7xx_evm.h
+++ b/include/configs/dra7xx_evm.h
@@ -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};" \
--
1.9.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [U-Boot] [PATCH] dra7x: fastboot: Increase recovery partition size
2017-11-08 23:12 [U-Boot] [PATCH] dra7x: fastboot: Increase recovery partition size Praneeth Bajjuri
@ 2017-11-10 19:22 ` Sam Protsenko
2017-11-20 19:29 ` Bajjuri, Praneeth
2017-11-20 19:33 ` Bajjuri, Praneeth
2017-11-21 13:05 ` [U-Boot] " Tom Rini
1 sibling, 2 replies; 5+ messages in thread
From: Sam Protsenko @ 2017-11-10 19:22 UTC (permalink / raw)
To: u-boot
On 9 November 2017 at 01:12, Praneeth Bajjuri <praneeth@ti.com> wrote:
> 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 <praneeth@ti.com>
> ---
> include/configs/am57xx_evm.h | 2 +-
> include/configs/dra7xx_evm.h | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/include/configs/am57xx_evm.h b/include/configs/am57xx_evm.h
> index ebb0474..dc05bea 100644
> --- a/include/configs/am57xx_evm.h
> +++ b/include/configs/am57xx_evm.h
> @@ -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};" \
> diff --git a/include/configs/dra7xx_evm.h b/include/configs/dra7xx_evm.h
> index 34a4555..f84c1f0 100644
> --- a/include/configs/dra7xx_evm.h
> +++ b/include/configs/dra7xx_evm.h
> @@ -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};" \
> --
> 1.9.1
>
Reviewed-by: Sam Protsenko <semen.protsenko@linaro.org>
^ permalink raw reply [flat|nested] 5+ messages in thread
* [U-Boot] [PATCH] dra7x: fastboot: Increase recovery partition size
2017-11-10 19:22 ` Sam Protsenko
@ 2017-11-20 19:29 ` Bajjuri, Praneeth
2017-11-20 19:33 ` Bajjuri, Praneeth
1 sibling, 0 replies; 5+ messages in thread
From: Bajjuri, Praneeth @ 2017-11-20 19:29 UTC (permalink / raw)
To: u-boot
Tom,
Can you pull this , if there is nothing waiting to be addressed on this one.
--praneeth
On 11/10/2017 1:22 PM, Sam Protsenko wrote:
> On 9 November 2017 at 01:12, Praneeth Bajjuri <praneeth@ti.com> wrote:
>> 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 <praneeth@ti.com>
>> ---
>> include/configs/am57xx_evm.h | 2 +-
>> include/configs/dra7xx_evm.h | 2 +-
>> 2 files changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/include/configs/am57xx_evm.h b/include/configs/am57xx_evm.h
>> index ebb0474..dc05bea 100644
>> --- a/include/configs/am57xx_evm.h
>> +++ b/include/configs/am57xx_evm.h
>> @@ -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};" \
>> diff --git a/include/configs/dra7xx_evm.h b/include/configs/dra7xx_evm.h
>> index 34a4555..f84c1f0 100644
>> --- a/include/configs/dra7xx_evm.h
>> +++ b/include/configs/dra7xx_evm.h
>> @@ -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};" \
>> --
>> 1.9.1
>>
> Reviewed-by: Sam Protsenko <semen.protsenko@linaro.org>
^ permalink raw reply [flat|nested] 5+ messages in thread
* [U-Boot] [PATCH] dra7x: fastboot: Increase recovery partition size
2017-11-10 19:22 ` Sam Protsenko
2017-11-20 19:29 ` Bajjuri, Praneeth
@ 2017-11-20 19:33 ` Bajjuri, Praneeth
1 sibling, 0 replies; 5+ messages in thread
From: Bajjuri, Praneeth @ 2017-11-20 19:33 UTC (permalink / raw)
To: u-boot
Hi Tom,
if there is nothing waiting to be addressed, Can you pull this patch.
-- Praneeth
On 11/10/2017 1:22 PM, Sam Protsenko wrote:
> On 9 November 2017 at 01:12, Praneeth Bajjuri <praneeth@ti.com> wrote:
>> 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 <praneeth@ti.com>
>> ---
>> include/configs/am57xx_evm.h | 2 +-
>> include/configs/dra7xx_evm.h | 2 +-
>> 2 files changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/include/configs/am57xx_evm.h b/include/configs/am57xx_evm.h
>> index ebb0474..dc05bea 100644
>> --- a/include/configs/am57xx_evm.h
>> +++ b/include/configs/am57xx_evm.h
>> @@ -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};" \
>> diff --git a/include/configs/dra7xx_evm.h b/include/configs/dra7xx_evm.h
>> index 34a4555..f84c1f0 100644
>> --- a/include/configs/dra7xx_evm.h
>> +++ b/include/configs/dra7xx_evm.h
>> @@ -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};" \
>> --
>> 1.9.1
>>
> Reviewed-by: Sam Protsenko <semen.protsenko@linaro.org>
^ permalink raw reply [flat|nested] 5+ messages in thread
* [U-Boot] dra7x: fastboot: Increase recovery partition size
2017-11-08 23:12 [U-Boot] [PATCH] dra7x: fastboot: Increase recovery partition size Praneeth Bajjuri
2017-11-10 19:22 ` Sam Protsenko
@ 2017-11-21 13:05 ` Tom Rini
1 sibling, 0 replies; 5+ messages in thread
From: Tom Rini @ 2017-11-21 13:05 UTC (permalink / raw)
To: u-boot
On Wed, Nov 08, 2017 at 05:12:56PM -0600, Praneeth Bajjuri wrote:
> 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 <praneeth@ti.com>
> Reviewed-by: Sam Protsenko <semen.protsenko@linaro.org>
Applied to u-boot/master, thanks!
--
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>
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2017-11-21 13:05 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-11-08 23:12 [U-Boot] [PATCH] dra7x: fastboot: Increase recovery partition size Praneeth Bajjuri
2017-11-10 19:22 ` Sam Protsenko
2017-11-20 19:29 ` Bajjuri, Praneeth
2017-11-20 19:33 ` Bajjuri, Praneeth
2017-11-21 13:05 ` [U-Boot] " Tom Rini
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.