* [U-Boot] [PATCH] arm: am57xx: Update bootloader partition size
@ 2017-02-21 19:51 Sam Protsenko
2017-02-21 20:05 ` Bajjuri, Praneeth
2017-02-21 20:35 ` Felipe Balbi
0 siblings, 2 replies; 8+ messages in thread
From: Sam Protsenko @ 2017-02-21 19:51 UTC (permalink / raw)
To: u-boot
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(-)
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};" \
--
2.11.0
^ permalink raw reply related [flat|nested] 8+ messages in thread* [U-Boot] [PATCH] arm: am57xx: Update bootloader partition size
2017-02-21 19:51 [U-Boot] [PATCH] arm: am57xx: Update bootloader partition size Sam Protsenko
@ 2017-02-21 20:05 ` Bajjuri, Praneeth
2017-02-21 20:35 ` Felipe Balbi
1 sibling, 0 replies; 8+ messages in thread
From: Bajjuri, Praneeth @ 2017-02-21 20:05 UTC (permalink / raw)
To: u-boot
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};" \
^ permalink raw reply [flat|nested] 8+ messages in thread* [U-Boot] [PATCH] arm: am57xx: Update bootloader partition size
2017-02-21 19:51 [U-Boot] [PATCH] arm: am57xx: Update bootloader partition size Sam Protsenko
2017-02-21 20:05 ` Bajjuri, Praneeth
@ 2017-02-21 20:35 ` Felipe Balbi
2017-02-21 20:53 ` Sam Protsenko
1 sibling, 1 reply; 8+ messages in thread
From: Felipe Balbi @ 2017-02-21 20:35 UTC (permalink / raw)
To: u-boot
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
^ permalink raw reply [flat|nested] 8+ messages in thread
* [U-Boot] [PATCH] arm: am57xx: Update bootloader partition size
2017-02-21 20:35 ` Felipe Balbi
@ 2017-02-21 20:53 ` Sam Protsenko
2017-02-21 21:48 ` Felipe Balbi
0 siblings, 1 reply; 8+ messages in thread
From: Sam Protsenko @ 2017-02-21 20:53 UTC (permalink / raw)
To: u-boot
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
^ permalink raw reply [flat|nested] 8+ messages in thread
* [U-Boot] [PATCH] arm: am57xx: Update bootloader partition size
2017-02-21 20:53 ` Sam Protsenko
@ 2017-02-21 21:48 ` Felipe Balbi
2017-02-22 3:00 ` Bajjuri, Praneeth
0 siblings, 1 reply; 8+ messages in thread
From: Felipe Balbi @ 2017-02-21 21:48 UTC (permalink / raw)
To: u-boot
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
^ permalink raw reply [flat|nested] 8+ messages in thread
* [U-Boot] [PATCH] arm: am57xx: Update bootloader partition size
2017-02-21 21:48 ` Felipe Balbi
@ 2017-02-22 3:00 ` Bajjuri, Praneeth
2017-02-22 18:47 ` Sam Protsenko
2017-02-25 1:37 ` Tom Rini
0 siblings, 2 replies; 8+ messages in thread
From: Bajjuri, Praneeth @ 2017-02-22 3:00 UTC (permalink / raw)
To: u-boot
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
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* [U-Boot] [PATCH] arm: am57xx: Update bootloader partition size
2017-02-22 3:00 ` Bajjuri, Praneeth
@ 2017-02-22 18:47 ` Sam Protsenko
2017-02-25 1:37 ` Tom Rini
1 sibling, 0 replies; 8+ messages in thread
From: Sam Protsenko @ 2017-02-22 18:47 UTC (permalink / raw)
To: u-boot
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.
>>
>>
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* [U-Boot] [PATCH] arm: am57xx: Update bootloader partition size
2017-02-22 3:00 ` Bajjuri, Praneeth
2017-02-22 18:47 ` Sam Protsenko
@ 2017-02-25 1:37 ` Tom Rini
1 sibling, 0 replies; 8+ messages in thread
From: Tom Rini @ 2017-02-25 1:37 UTC (permalink / raw)
To: u-boot
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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170224/dbf34a7c/attachment.sig>
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2017-02-25 1:37 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-02-21 19:51 [U-Boot] [PATCH] arm: am57xx: Update bootloader partition size Sam Protsenko
2017-02-21 20:05 ` Bajjuri, Praneeth
2017-02-21 20:35 ` Felipe Balbi
2017-02-21 20:53 ` Sam Protsenko
2017-02-21 21:48 ` Felipe Balbi
2017-02-22 3:00 ` Bajjuri, Praneeth
2017-02-22 18:47 ` Sam Protsenko
2017-02-25 1:37 ` Tom Rini
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox