| Message ID | 20170221195124.28580-1-semen.protsenko@linaro.org |
|---|---|
| State | Superseded |
| Delegated to: | Tom Rini |
| Headers | show |
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};" \
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?
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
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 ? ;-)
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 >
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. >> >> >
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.
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};" \