All of lore.kernel.org
 help / color / mirror / Atom feed
* [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 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.