diff mbox

arm: am57xx: Update bootloader partition size

Message ID 20170221195124.28580-1-semen.protsenko@linaro.org
State Superseded
Headers show

Commit Message

Sam Protsenko Feb. 21, 2017, 7:51 p.m. UTC
From: Praneeth Bajjuri <praneeth@ti.com>


The current size of u-boot is already ~840 KiB.

This patch is to increase bootloader partition size up to 1 MiB
and thereby fix flashing error like below:

    sending 'bootloader' (840 KB)...
    OKAY [  0.048s]
    writing 'bootloader'...
    FAILED (remote: too large for partition)

Signed-off-by: Sam Protsenko <semen.protsenko@linaro.org>

---
 include/configs/am57xx_evm.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
2.11.0

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Comments

Bajjuri, Praneeth Feb. 21, 2017, 8:05 p.m. UTC | #1
On 2/21/2017 1:51 PM, Sam Protsenko wrote:
> From: Praneeth Bajjuri <praneeth@ti.com>

>

> The current size of u-boot is already ~840 KiB.

>

> This patch is to increase bootloader partition size up to 1 MiB

> and thereby fix flashing error like below:

>

>      sending 'bootloader' (840 KB)...

>      OKAY [  0.048s]

>      writing 'bootloader'...

>      FAILED (remote: too large for partition)

>

> Signed-off-by: Sam Protsenko <semen.protsenko@linaro.org>

Verified
Signed-off-by: Praneeth Bajjuri <praneeth@ti.com>

> ---

>   include/configs/am57xx_evm.h | 2 +-

>   1 file changed, 1 insertion(+), 1 deletion(-)

>

> diff --git a/include/configs/am57xx_evm.h b/include/configs/am57xx_evm.h

> index 3d8b996054..aa63b24ca4 100644

> --- a/include/configs/am57xx_evm.h

> +++ b/include/configs/am57xx_evm.h

> @@ -45,7 +45,7 @@

>   	"partitions_android=" \

>   	"uuid_disk=${uuid_gpt_disk};" \

>   	"name=xloader,start=128K,size=256K,uuid=${uuid_gpt_xloader};" \

> -	"name=bootloader,size=768K,uuid=${uuid_gpt_bootloader};" \

> +	"name=bootloader,size=1M,uuid=${uuid_gpt_bootloader};" \

>   	"name=environment,size=128K,uuid=${uuid_gpt_environment};" \

>   	"name=misc,size=128K,uuid=${uuid_gpt_misc};" \

>   	"name=reserved,size=256K,uuid=${uuid_gpt_reserved};" \


_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
Felipe Balbi Feb. 21, 2017, 8:35 p.m. UTC | #2
Hi,

Sam Protsenko <semen.protsenko@linaro.org> writes:
> From: Praneeth Bajjuri <praneeth@ti.com>

>

> The current size of u-boot is already ~840 KiB.

>

> This patch is to increase bootloader partition size up to 1 MiB

> and thereby fix flashing error like below:

>

>     sending 'bootloader' (840 KB)...

>     OKAY [  0.048s]

>     writing 'bootloader'...

>     FAILED (remote: too large for partition)

>

> Signed-off-by: Sam Protsenko <semen.protsenko@linaro.org>


Missing Praneeth's Signed-off-by?

-- 
balbi
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
Sam Protsenko Feb. 21, 2017, 8:53 p.m. UTC | #3
On Tue, Feb 21, 2017 at 10:35 PM, Felipe Balbi
<felipe.balbi@linux.intel.com> wrote:
>

> Hi,

>

> Sam Protsenko <semen.protsenko@linaro.org> writes:

>> From: Praneeth Bajjuri <praneeth@ti.com>

>>

>> The current size of u-boot is already ~840 KiB.

>>

>> This patch is to increase bootloader partition size up to 1 MiB

>> and thereby fix flashing error like below:

>>

>>     sending 'bootloader' (840 KB)...

>>     OKAY [  0.048s]

>>     writing 'bootloader'...

>>     FAILED (remote: too large for partition)

>>

>> Signed-off-by: Sam Protsenko <semen.protsenko@linaro.org>

>

> Missing Praneeth's Signed-off-by?

>


Yes. But not sure If I should resend.

Tom, will you please add Praneeth's Signed-off-by line when applying patch?

> --

> balbi

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
Felipe Balbi Feb. 21, 2017, 9:48 p.m. UTC | #4
Hi,

Sam Protsenko <semen.protsenko@linaro.org> writes:
> On Tue, Feb 21, 2017 at 10:35 PM, Felipe Balbi

> <felipe.balbi@linux.intel.com> wrote:

>>

>> Hi,

>>

>> Sam Protsenko <semen.protsenko@linaro.org> writes:

>>> From: Praneeth Bajjuri <praneeth@ti.com>

>>>

>>> The current size of u-boot is already ~840 KiB.

>>>

>>> This patch is to increase bootloader partition size up to 1 MiB

>>> and thereby fix flashing error like below:

>>>

>>>     sending 'bootloader' (840 KB)...

>>>     OKAY [  0.048s]

>>>     writing 'bootloader'...

>>>     FAILED (remote: too large for partition)

>>>

>>> Signed-off-by: Sam Protsenko <semen.protsenko@linaro.org>

>>

>> Missing Praneeth's Signed-off-by?

>>

>

> Yes. But not sure If I should resend.

>

> Tom, will you please add Praneeth's Signed-off-by line when applying patch?


Only if Praneeth agrees, right ? ;-)

-- 
balbi
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
Bajjuri, Praneeth Feb. 22, 2017, 3 a.m. UTC | #5
On 2/21/2017 3:48 PM, Felipe Balbi wrote:
> Hi,

>

> Sam Protsenko <semen.protsenko@linaro.org> writes:

>> On Tue, Feb 21, 2017 at 10:35 PM, Felipe Balbi

>> <felipe.balbi@linux.intel.com> wrote:

>>> Hi,

>>>

>>> Sam Protsenko <semen.protsenko@linaro.org> writes:

>>>> From: Praneeth Bajjuri <praneeth@ti.com>

>>>>

>>>> The current size of u-boot is already ~840 KiB.

>>>>

>>>> This patch is to increase bootloader partition size up to 1 MiB

>>>> and thereby fix flashing error like below:

>>>>

>>>>      sending 'bootloader' (840 KB)...

>>>>      OKAY [  0.048s]

>>>>      writing 'bootloader'...

>>>>      FAILED (remote: too large for partition)

>>>>

>>>> Signed-off-by: Sam Protsenko <semen.protsenko@linaro.org>

>>> Missing Praneeth's Signed-off-by?

>>>

>> Yes. But not sure If I should resend.

>>

>> Tom, will you please add Praneeth's Signed-off-by line when applying patch?

> Only if Praneeth agrees, right ? ;-)

agreed and ok with me
thanks felipe , sam
>


_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
Sam Protsenko Feb. 22, 2017, 6:47 p.m. UTC | #6
On Wed, Feb 22, 2017 at 5:00 AM, Bajjuri, Praneeth <praneeth@ti.com> wrote:
>

>

> On 2/21/2017 3:48 PM, Felipe Balbi wrote:

>>

>> Hi,

>>

>> Sam Protsenko <semen.protsenko@linaro.org> writes:

>>>

>>> On Tue, Feb 21, 2017 at 10:35 PM, Felipe Balbi

>>> <felipe.balbi@linux.intel.com> wrote:

>>>>

>>>> Hi,

>>>>

>>>> Sam Protsenko <semen.protsenko@linaro.org> writes:

>>>>>

>>>>> From: Praneeth Bajjuri <praneeth@ti.com>

>>>>>

>>>>> The current size of u-boot is already ~840 KiB.

>>>>>

>>>>> This patch is to increase bootloader partition size up to 1 MiB

>>>>> and thereby fix flashing error like below:

>>>>>

>>>>>      sending 'bootloader' (840 KB)...

>>>>>      OKAY [  0.048s]

>>>>>      writing 'bootloader'...

>>>>>      FAILED (remote: too large for partition)

>>>>>

>>>>> Signed-off-by: Sam Protsenko <semen.protsenko@linaro.org>

>>>>

>>>> Missing Praneeth's Signed-off-by?

>>>>

>>> Yes. But not sure If I should resend.

>>>

>>> Tom, will you please add Praneeth's Signed-off-by line when applying

>>> patch?

>>

>> Only if Praneeth agrees, right ? ;-)

>

> agreed and ok with me

> thanks felipe , sam


Tom,

Please apply this one.

Thanks.

>>

>>

>

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
Tom Rini Feb. 25, 2017, 1:37 a.m. UTC | #7
On Tue, Feb 21, 2017 at 09:00:04PM -0600, Bajjuri, Praneeth wrote:
> 

> 

> On 2/21/2017 3:48 PM, Felipe Balbi wrote:

> >Hi,

> >

> >Sam Protsenko <semen.protsenko@linaro.org> writes:

> >>On Tue, Feb 21, 2017 at 10:35 PM, Felipe Balbi

> >><felipe.balbi@linux.intel.com> wrote:

> >>>Hi,

> >>>

> >>>Sam Protsenko <semen.protsenko@linaro.org> writes:

> >>>>From: Praneeth Bajjuri <praneeth@ti.com>

> >>>>

> >>>>The current size of u-boot is already ~840 KiB.

> >>>>

> >>>>This patch is to increase bootloader partition size up to 1 MiB

> >>>>and thereby fix flashing error like below:

> >>>>

> >>>>     sending 'bootloader' (840 KB)...

> >>>>     OKAY [  0.048s]

> >>>>     writing 'bootloader'...

> >>>>     FAILED (remote: too large for partition)

> >>>>

> >>>>Signed-off-by: Sam Protsenko <semen.protsenko@linaro.org>

> >>>Missing Praneeth's Signed-off-by?

> >>>

> >>Yes. But not sure If I should resend.

> >>

> >>Tom, will you please add Praneeth's Signed-off-by line when applying patch?

> >Only if Praneeth agrees, right ? ;-)

> agreed and ok with me

> thanks felipe , sam


You can reply with a Signed-off-by line and patchwork will collect it,
in general.

-- 
Tom
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
diff mbox

Patch

diff --git a/include/configs/am57xx_evm.h b/include/configs/am57xx_evm.h
index 3d8b996054..aa63b24ca4 100644
--- a/include/configs/am57xx_evm.h
+++ b/include/configs/am57xx_evm.h
@@ -45,7 +45,7 @@ 
 	"partitions_android=" \
 	"uuid_disk=${uuid_gpt_disk};" \
 	"name=xloader,start=128K,size=256K,uuid=${uuid_gpt_xloader};" \
-	"name=bootloader,size=768K,uuid=${uuid_gpt_bootloader};" \
+	"name=bootloader,size=1M,uuid=${uuid_gpt_bootloader};" \
 	"name=environment,size=128K,uuid=${uuid_gpt_environment};" \
 	"name=misc,size=128K,uuid=${uuid_gpt_misc};" \
 	"name=reserved,size=256K,uuid=${uuid_gpt_reserved};" \