Re: [U-Boot] [PATCH] dra7x: fastboot: Increase recovery partition size

[Bajjuri, Praneeth]

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  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 
---
  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 


___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH] dra7x: fastboot: Increase recovery partition size

[Bajjuri, Praneeth]

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  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 
---
  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 


___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH] dra7x: fastboot: Increase recovery partition size

[Sam Protsenko]

On 9 November 2017 at 01:12, 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 
> ---
>  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 
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

[U-Boot] [PATCH] dra7x: fastboot: Increase recovery partition size

[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 
---
 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

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot