* [U-Boot] powerpc/mpc85xx: Increase image size @ 2013-10-11 18:29 York Sun 2013-10-11 18:39 ` Scott Wood 0 siblings, 1 reply; 15+ messages in thread From: York Sun @ 2013-10-11 18:29 UTC (permalink / raw) To: u-boot Scott, et al. I'd like to start the discussion to increase u-boot image size for some mpc85xx targets. As we all know the reset vector is at the very end and linking process start from the top. This gives us no good choice but to use fixed image size. While we have more and more features, the size increases inevitably. It's time to adjust the arbitrary size. We are now using 512KB. Shall we go with 768KB, or even 1MB? For the first step, I think we don't have to increase size for all targets. We can adjust those with CONFIG_SYS_TEXT_BASE=0xeff80000. Those are the most recent used. There are other targets which don't use NOR flash boot method. They should be considered as well. It is per board configuration. But it may be better if we keep them consistent. Please comment. Regards, York ^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] powerpc/mpc85xx: Increase image size 2013-10-11 18:29 [U-Boot] powerpc/mpc85xx: Increase image size York Sun @ 2013-10-11 18:39 ` Scott Wood 2013-10-11 18:56 ` York Sun 0 siblings, 1 reply; 15+ messages in thread From: Scott Wood @ 2013-10-11 18:39 UTC (permalink / raw) To: u-boot On Fri, 2013-10-11 at 11:29 -0700, York Sun wrote: > Scott, et al. > > I'd like to start the discussion to increase u-boot image size for some > mpc85xx targets. As we all know the reset vector is at the very end and > linking process start from the top. This gives us no good choice but to > use fixed image size. While we have more and more features, the size > increases inevitably. It's time to adjust the arbitrary size. We are now > using 512KB. Shall we go with 768KB, or even 1MB? 768K would affect fewer existing partition maps (many of which leave 1M for U-Boot and environment combined), but 1M might be better for new boards. And of course it would be nice if someone could spare some time to try to slim things down (finer-grained compile-time config, speed/size tradeoffs, etc). > For the first step, I think we don't have to increase size for all > targets. We can adjust those with CONFIG_SYS_TEXT_BASE=0xeff80000. Those > are the most recent used. There are other targets which don't use NOR > flash boot method. They should be considered as well. > > It is per board configuration. But it may be better if we keep them > consistent. I don't think it's worth trying to keep them consistent. Leave alone old boards that are not pushing the limit, and where testing and user education would be a hassle, and let newer boards where more features are wanted not be constrained by the past. -Scott ^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] powerpc/mpc85xx: Increase image size 2013-10-11 18:39 ` Scott Wood @ 2013-10-11 18:56 ` York Sun 2013-10-16 17:41 ` York Sun 0 siblings, 1 reply; 15+ messages in thread From: York Sun @ 2013-10-11 18:56 UTC (permalink / raw) To: u-boot On 10/11/2013 11:39 AM, Scott Wood wrote: > On Fri, 2013-10-11 at 11:29 -0700, York Sun wrote: >> Scott, et al. >> >> I'd like to start the discussion to increase u-boot image size for some >> mpc85xx targets. As we all know the reset vector is at the very end and >> linking process start from the top. This gives us no good choice but to >> use fixed image size. While we have more and more features, the size >> increases inevitably. It's time to adjust the arbitrary size. We are now >> using 512KB. Shall we go with 768KB, or even 1MB? > > 768K would affect fewer existing partition maps (many of which leave 1M > for U-Boot and environment combined), but 1M might be better for new > boards. And of course it would be nice if someone could spare some time > to try to slim things down (finer-grained compile-time config, > speed/size tradeoffs, etc). > >> For the first step, I think we don't have to increase size for all >> targets. We can adjust those with CONFIG_SYS_TEXT_BASE=0xeff80000. Those >> are the most recent used. There are other targets which don't use NOR >> flash boot method. They should be considered as well. >> >> It is per board configuration. But it may be better if we keep them >> consistent. > > I don't think it's worth trying to keep them consistent. Leave alone > old boards that are not pushing the limit, and where testing and user > education would be a hassle, and let newer boards where more features > are wanted not be constrained by the past. > I did a quick search for eff80000. Only these boards have it include/configs/B4860QDS.h include/configs/C29XPCIE.h include/configs/corenet_ds.h include/configs/HWW1U1A.h include/configs/MPC8536DS.h include/configs/MPC8572DS.h include/configs/P1010RDB.h include/configs/P1022DS.h include/configs/P1023RDB.h include/configs/P1023RDS.h include/configs/P1_P2_RDB.h include/configs/p1_p2_rdb_pc.h include/configs/p1_twr.h include/configs/P2020DS.h include/configs/P2041RDB.h include/configs/T1040QDS.h include/configs/t4qds.h York ^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] powerpc/mpc85xx: Increase image size 2013-10-11 18:56 ` York Sun @ 2013-10-16 17:41 ` York Sun 2013-10-16 19:37 ` Scott Wood 0 siblings, 1 reply; 15+ messages in thread From: York Sun @ 2013-10-16 17:41 UTC (permalink / raw) To: u-boot On 10/11/2013 11:56 AM, York Sun wrote: > On 10/11/2013 11:39 AM, Scott Wood wrote: >> On Fri, 2013-10-11 at 11:29 -0700, York Sun wrote: >>> Scott, et al. >>> >>> I'd like to start the discussion to increase u-boot image size for some >>> mpc85xx targets. As we all know the reset vector is at the very end and >>> linking process start from the top. This gives us no good choice but to >>> use fixed image size. While we have more and more features, the size >>> increases inevitably. It's time to adjust the arbitrary size. We are now >>> using 512KB. Shall we go with 768KB, or even 1MB? >> >> 768K would affect fewer existing partition maps (many of which leave 1M >> for U-Boot and environment combined), but 1M might be better for new >> boards. And of course it would be nice if someone could spare some time >> to try to slim things down (finer-grained compile-time config, >> speed/size tradeoffs, etc). >> >>> For the first step, I think we don't have to increase size for all >>> targets. We can adjust those with CONFIG_SYS_TEXT_BASE=0xeff80000. Those >>> are the most recent used. There are other targets which don't use NOR >>> flash boot method. They should be considered as well. >>> >>> It is per board configuration. But it may be better if we keep them >>> consistent. >> >> I don't think it's worth trying to keep them consistent. Leave alone >> old boards that are not pushing the limit, and where testing and user >> education would be a hassle, and let newer boards where more features >> are wanted not be constrained by the past. >> > > I did a quick search for eff80000. Only these boards have it > > include/configs/B4860QDS.h > include/configs/C29XPCIE.h > include/configs/corenet_ds.h > include/configs/HWW1U1A.h > include/configs/MPC8536DS.h > include/configs/MPC8572DS.h > include/configs/P1010RDB.h > include/configs/P1022DS.h > include/configs/P1023RDB.h > include/configs/P1023RDS.h > include/configs/P1_P2_RDB.h > include/configs/p1_p2_rdb_pc.h > include/configs/p1_twr.h > include/configs/P2020DS.h > include/configs/P2041RDB.h > include/configs/T1040QDS.h > include/configs/t4qds.h > Scott, Are SPL and TPL boot methods immune from the size issue here? York ^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] powerpc/mpc85xx: Increase image size 2013-10-16 17:41 ` York Sun @ 2013-10-16 19:37 ` Scott Wood 2013-10-16 20:22 ` York Sun 0 siblings, 1 reply; 15+ messages in thread From: Scott Wood @ 2013-10-16 19:37 UTC (permalink / raw) To: u-boot On Wed, 2013-10-16 at 10:41 -0700, York Sun wrote: > On 10/11/2013 11:56 AM, York Sun wrote: > > On 10/11/2013 11:39 AM, Scott Wood wrote: > >> On Fri, 2013-10-11 at 11:29 -0700, York Sun wrote: > >>> Scott, et al. > >>> > >>> I'd like to start the discussion to increase u-boot image size for some > >>> mpc85xx targets. As we all know the reset vector is at the very end and > >>> linking process start from the top. This gives us no good choice but to > >>> use fixed image size. While we have more and more features, the size > >>> increases inevitably. It's time to adjust the arbitrary size. We are now > >>> using 512KB. Shall we go with 768KB, or even 1MB? > >> > >> 768K would affect fewer existing partition maps (many of which leave 1M > >> for U-Boot and environment combined), but 1M might be better for new > >> boards. And of course it would be nice if someone could spare some time > >> to try to slim things down (finer-grained compile-time config, > >> speed/size tradeoffs, etc). > >> > >>> For the first step, I think we don't have to increase size for all > >>> targets. We can adjust those with CONFIG_SYS_TEXT_BASE=0xeff80000. Those > >>> are the most recent used. There are other targets which don't use NOR > >>> flash boot method. They should be considered as well. > >>> > >>> It is per board configuration. But it may be better if we keep them > >>> consistent. > >> > >> I don't think it's worth trying to keep them consistent. Leave alone > >> old boards that are not pushing the limit, and where testing and user > >> education would be a hassle, and let newer boards where more features > >> are wanted not be constrained by the past. > >> > > > > I did a quick search for eff80000. Only these boards have it > > > > include/configs/B4860QDS.h > > include/configs/C29XPCIE.h > > include/configs/corenet_ds.h > > include/configs/HWW1U1A.h > > include/configs/MPC8536DS.h > > include/configs/MPC8572DS.h > > include/configs/P1010RDB.h > > include/configs/P1022DS.h > > include/configs/P1023RDB.h > > include/configs/P1023RDS.h > > include/configs/P1_P2_RDB.h > > include/configs/p1_p2_rdb_pc.h > > include/configs/p1_twr.h > > include/configs/P2020DS.h > > include/configs/P2041RDB.h > > include/configs/T1040QDS.h > > include/configs/t4qds.h > > > > Scott, > > Are SPL and TPL boot methods immune from the size issue here? Sort of. We still need to fit inside existing partition tables. -Scott ^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] powerpc/mpc85xx: Increase image size 2013-10-16 19:37 ` Scott Wood @ 2013-10-16 20:22 ` York Sun 2013-10-16 20:29 ` Scott Wood 0 siblings, 1 reply; 15+ messages in thread From: York Sun @ 2013-10-16 20:22 UTC (permalink / raw) To: u-boot On 10/16/2013 12:37 PM, Scott Wood wrote: > On Wed, 2013-10-16 at 10:41 -0700, York Sun wrote: >> On 10/11/2013 11:56 AM, York Sun wrote: >>> On 10/11/2013 11:39 AM, Scott Wood wrote: >>>> On Fri, 2013-10-11 at 11:29 -0700, York Sun wrote: >>>>> Scott, et al. >>>>> >>>>> I'd like to start the discussion to increase u-boot image size for some >>>>> mpc85xx targets. As we all know the reset vector is at the very end and >>>>> linking process start from the top. This gives us no good choice but to >>>>> use fixed image size. While we have more and more features, the size >>>>> increases inevitably. It's time to adjust the arbitrary size. We are now >>>>> using 512KB. Shall we go with 768KB, or even 1MB? >>>> >>>> 768K would affect fewer existing partition maps (many of which leave 1M >>>> for U-Boot and environment combined), but 1M might be better for new >>>> boards. And of course it would be nice if someone could spare some time >>>> to try to slim things down (finer-grained compile-time config, >>>> speed/size tradeoffs, etc). >>>> >>>>> For the first step, I think we don't have to increase size for all >>>>> targets. We can adjust those with CONFIG_SYS_TEXT_BASE=0xeff80000. Those >>>>> are the most recent used. There are other targets which don't use NOR >>>>> flash boot method. They should be considered as well. >>>>> >>>>> It is per board configuration. But it may be better if we keep them >>>>> consistent. >>>> >>>> I don't think it's worth trying to keep them consistent. Leave alone >>>> old boards that are not pushing the limit, and where testing and user >>>> education would be a hassle, and let newer boards where more features >>>> are wanted not be constrained by the past. >>>> >>> >>> I did a quick search for eff80000. Only these boards have it >>> >>> include/configs/B4860QDS.h >>> include/configs/C29XPCIE.h >>> include/configs/corenet_ds.h >>> include/configs/HWW1U1A.h >>> include/configs/MPC8536DS.h >>> include/configs/MPC8572DS.h >>> include/configs/P1010RDB.h >>> include/configs/P1022DS.h >>> include/configs/P1023RDB.h >>> include/configs/P1023RDS.h >>> include/configs/P1_P2_RDB.h >>> include/configs/p1_p2_rdb_pc.h >>> include/configs/p1_twr.h >>> include/configs/P2020DS.h >>> include/configs/P2041RDB.h >>> include/configs/T1040QDS.h >>> include/configs/t4qds.h >>> >> >> Scott, >> >> Are SPL and TPL boot methods immune from the size issue here? > > Sort of. We still need to fit inside existing partition tables. > PBL boot will be broken if the image size is bigger than 512KB, right? York ^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] powerpc/mpc85xx: Increase image size 2013-10-16 20:22 ` York Sun @ 2013-10-16 20:29 ` Scott Wood 2013-10-16 20:32 ` York Sun 0 siblings, 1 reply; 15+ messages in thread From: Scott Wood @ 2013-10-16 20:29 UTC (permalink / raw) To: u-boot On Wed, 2013-10-16 at 13:22 -0700, York Sun wrote: > On 10/16/2013 12:37 PM, Scott Wood wrote: > > On Wed, 2013-10-16 at 10:41 -0700, York Sun wrote: > >> On 10/11/2013 11:56 AM, York Sun wrote: > >>> On 10/11/2013 11:39 AM, Scott Wood wrote: > >>>> On Fri, 2013-10-11 at 11:29 -0700, York Sun wrote: > >>>>> Scott, et al. > >>>>> > >>>>> I'd like to start the discussion to increase u-boot image size for some > >>>>> mpc85xx targets. As we all know the reset vector is at the very end and > >>>>> linking process start from the top. This gives us no good choice but to > >>>>> use fixed image size. While we have more and more features, the size > >>>>> increases inevitably. It's time to adjust the arbitrary size. We are now > >>>>> using 512KB. Shall we go with 768KB, or even 1MB? > >>>> > >>>> 768K would affect fewer existing partition maps (many of which leave 1M > >>>> for U-Boot and environment combined), but 1M might be better for new > >>>> boards. And of course it would be nice if someone could spare some time > >>>> to try to slim things down (finer-grained compile-time config, > >>>> speed/size tradeoffs, etc). > >>>> > >>>>> For the first step, I think we don't have to increase size for all > >>>>> targets. We can adjust those with CONFIG_SYS_TEXT_BASE=0xeff80000. Those > >>>>> are the most recent used. There are other targets which don't use NOR > >>>>> flash boot method. They should be considered as well. > >>>>> > >>>>> It is per board configuration. But it may be better if we keep them > >>>>> consistent. > >>>> > >>>> I don't think it's worth trying to keep them consistent. Leave alone > >>>> old boards that are not pushing the limit, and where testing and user > >>>> education would be a hassle, and let newer boards where more features > >>>> are wanted not be constrained by the past. > >>>> > >>> > >>> I did a quick search for eff80000. Only these boards have it > >>> > >>> include/configs/B4860QDS.h > >>> include/configs/C29XPCIE.h > >>> include/configs/corenet_ds.h > >>> include/configs/HWW1U1A.h > >>> include/configs/MPC8536DS.h > >>> include/configs/MPC8572DS.h > >>> include/configs/P1010RDB.h > >>> include/configs/P1022DS.h > >>> include/configs/P1023RDB.h > >>> include/configs/P1023RDS.h > >>> include/configs/P1_P2_RDB.h > >>> include/configs/p1_p2_rdb_pc.h > >>> include/configs/p1_twr.h > >>> include/configs/P2020DS.h > >>> include/configs/P2041RDB.h > >>> include/configs/T1040QDS.h > >>> include/configs/t4qds.h > >>> > >> > >> Scott, > >> > >> Are SPL and TPL boot methods immune from the size issue here? > > > > Sort of. We still need to fit inside existing partition tables. > > > > PBL boot will be broken if the image size is bigger than 512KB, right? It has to be even smaller than that, to make room for early data. -Scott ^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] powerpc/mpc85xx: Increase image size 2013-10-16 20:29 ` Scott Wood @ 2013-10-16 20:32 ` York Sun 2013-10-16 20:33 ` Scott Wood 0 siblings, 1 reply; 15+ messages in thread From: York Sun @ 2013-10-16 20:32 UTC (permalink / raw) To: u-boot On 10/16/2013 01:29 PM, Scott Wood wrote: > On Wed, 2013-10-16 at 13:22 -0700, York Sun wrote: >> On 10/16/2013 12:37 PM, Scott Wood wrote: >>> On Wed, 2013-10-16 at 10:41 -0700, York Sun wrote: >>>> On 10/11/2013 11:56 AM, York Sun wrote: >>>>> On 10/11/2013 11:39 AM, Scott Wood wrote: >>>>>> On Fri, 2013-10-11 at 11:29 -0700, York Sun wrote: >>>>>>> Scott, et al. >>>>>>> >>>>>>> I'd like to start the discussion to increase u-boot image size for some >>>>>>> mpc85xx targets. As we all know the reset vector is at the very end and >>>>>>> linking process start from the top. This gives us no good choice but to >>>>>>> use fixed image size. While we have more and more features, the size >>>>>>> increases inevitably. It's time to adjust the arbitrary size. We are now >>>>>>> using 512KB. Shall we go with 768KB, or even 1MB? >>>>>> >>>>>> 768K would affect fewer existing partition maps (many of which leave 1M >>>>>> for U-Boot and environment combined), but 1M might be better for new >>>>>> boards. And of course it would be nice if someone could spare some time >>>>>> to try to slim things down (finer-grained compile-time config, >>>>>> speed/size tradeoffs, etc). >>>>>> >>>>>>> For the first step, I think we don't have to increase size for all >>>>>>> targets. We can adjust those with CONFIG_SYS_TEXT_BASE=0xeff80000. Those >>>>>>> are the most recent used. There are other targets which don't use NOR >>>>>>> flash boot method. They should be considered as well. >>>>>>> >>>>>>> It is per board configuration. But it may be better if we keep them >>>>>>> consistent. >>>>>> >>>>>> I don't think it's worth trying to keep them consistent. Leave alone >>>>>> old boards that are not pushing the limit, and where testing and user >>>>>> education would be a hassle, and let newer boards where more features >>>>>> are wanted not be constrained by the past. >>>>>> >>>>> >>>>> I did a quick search for eff80000. Only these boards have it >>>>> >>>>> include/configs/B4860QDS.h >>>>> include/configs/C29XPCIE.h >>>>> include/configs/corenet_ds.h >>>>> include/configs/HWW1U1A.h >>>>> include/configs/MPC8536DS.h >>>>> include/configs/MPC8572DS.h >>>>> include/configs/P1010RDB.h >>>>> include/configs/P1022DS.h >>>>> include/configs/P1023RDB.h >>>>> include/configs/P1023RDS.h >>>>> include/configs/P1_P2_RDB.h >>>>> include/configs/p1_p2_rdb_pc.h >>>>> include/configs/p1_twr.h >>>>> include/configs/P2020DS.h >>>>> include/configs/P2041RDB.h >>>>> include/configs/T1040QDS.h >>>>> include/configs/t4qds.h >>>>> >>>> >>>> Scott, >>>> >>>> Are SPL and TPL boot methods immune from the size issue here? >>> >>> Sort of. We still need to fit inside existing partition tables. >>> >> >> PBL boot will be broken if the image size is bigger than 512KB, right? > > It has to be even smaller than that, to make room for early data. > So if we go with 768KB, do we have to convert all PBL boot to SPL boot? York ^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] powerpc/mpc85xx: Increase image size 2013-10-16 20:32 ` York Sun @ 2013-10-16 20:33 ` Scott Wood 2013-10-16 20:38 ` York Sun 0 siblings, 1 reply; 15+ messages in thread From: Scott Wood @ 2013-10-16 20:33 UTC (permalink / raw) To: u-boot On Wed, 2013-10-16 at 13:32 -0700, York Sun wrote: > On 10/16/2013 01:29 PM, Scott Wood wrote: > > On Wed, 2013-10-16 at 13:22 -0700, York Sun wrote: > >> On 10/16/2013 12:37 PM, Scott Wood wrote: > >>> On Wed, 2013-10-16 at 10:41 -0700, York Sun wrote: > >>>> Are SPL and TPL boot methods immune from the size issue here? > >>> > >>> Sort of. We still need to fit inside existing partition tables. > >>> > >> > >> PBL boot will be broken if the image size is bigger than 512KB, right? > > > > It has to be even smaller than that, to make room for early data. > > > > So if we go with 768KB, do we have to convert all PBL boot to SPL boot? Only the targets that need the extra space. -Scott ^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] powerpc/mpc85xx: Increase image size 2013-10-16 20:33 ` Scott Wood @ 2013-10-16 20:38 ` York Sun 2013-10-16 20:41 ` Scott Wood 0 siblings, 1 reply; 15+ messages in thread From: York Sun @ 2013-10-16 20:38 UTC (permalink / raw) To: u-boot On 10/16/2013 01:33 PM, Scott Wood wrote: > On Wed, 2013-10-16 at 13:32 -0700, York Sun wrote: >> On 10/16/2013 01:29 PM, Scott Wood wrote: >>> On Wed, 2013-10-16 at 13:22 -0700, York Sun wrote: >>>> On 10/16/2013 12:37 PM, Scott Wood wrote: >>>>> On Wed, 2013-10-16 at 10:41 -0700, York Sun wrote: >>>>>> Are SPL and TPL boot methods immune from the size issue here? >>>>> >>>>> Sort of. We still need to fit inside existing partition tables. >>>>> >>>> >>>> PBL boot will be broken if the image size is bigger than 512KB, right? >>> >>> It has to be even smaller than that, to make room for early data. >>> >> >> So if we go with 768KB, do we have to convert all PBL boot to SPL boot? > > Only the targets that need the extra space. > We have T4, B4 and corenet_ds using PBL boot. They most likely will exceed the 512KB soon, if not yet. It maybe easier to change all of them togther, than one by one. York ^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] powerpc/mpc85xx: Increase image size 2013-10-16 20:38 ` York Sun @ 2013-10-16 20:41 ` Scott Wood 2013-10-16 20:44 ` York Sun 0 siblings, 1 reply; 15+ messages in thread From: Scott Wood @ 2013-10-16 20:41 UTC (permalink / raw) To: u-boot On Wed, 2013-10-16 at 13:38 -0700, York Sun wrote: > On 10/16/2013 01:33 PM, Scott Wood wrote: > > On Wed, 2013-10-16 at 13:32 -0700, York Sun wrote: > >> On 10/16/2013 01:29 PM, Scott Wood wrote: > >>> On Wed, 2013-10-16 at 13:22 -0700, York Sun wrote: > >>>> On 10/16/2013 12:37 PM, Scott Wood wrote: > >>>>> On Wed, 2013-10-16 at 10:41 -0700, York Sun wrote: > >>>>>> Are SPL and TPL boot methods immune from the size issue here? > >>>>> > >>>>> Sort of. We still need to fit inside existing partition tables. > >>>>> > >>>> > >>>> PBL boot will be broken if the image size is bigger than 512KB, right? > >>> > >>> It has to be even smaller than that, to make room for early data. > >>> > >> > >> So if we go with 768KB, do we have to convert all PBL boot to SPL boot? > > > > Only the targets that need the extra space. > > > > We have T4, B4 and corenet_ds using PBL boot. They most likely will > exceed the 512KB soon, if not yet. It maybe easier to change all of them > togther, than one by one. There's no reason to change them all at once. It doesn't make anything easier; it just means you have to do a bunch of testing all at once, and force a change in procedure for users on boards where it otherwise would not have been required. Plus, the 512K limit is for e500v2-based chips. Newer chips have CPC for SRAM which is larger than 512K. -Scott ^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] powerpc/mpc85xx: Increase image size 2013-10-16 20:41 ` Scott Wood @ 2013-10-16 20:44 ` York Sun 2013-10-16 21:23 ` Scott Wood 0 siblings, 1 reply; 15+ messages in thread From: York Sun @ 2013-10-16 20:44 UTC (permalink / raw) To: u-boot On 10/16/2013 01:41 PM, Scott Wood wrote: > On Wed, 2013-10-16 at 13:38 -0700, York Sun wrote: >> On 10/16/2013 01:33 PM, Scott Wood wrote: >>> On Wed, 2013-10-16 at 13:32 -0700, York Sun wrote: >>>> On 10/16/2013 01:29 PM, Scott Wood wrote: >>>>> On Wed, 2013-10-16 at 13:22 -0700, York Sun wrote: >>>>>> On 10/16/2013 12:37 PM, Scott Wood wrote: >>>>>>> On Wed, 2013-10-16 at 10:41 -0700, York Sun wrote: >>>>>>>> Are SPL and TPL boot methods immune from the size issue here? >>>>>>> >>>>>>> Sort of. We still need to fit inside existing partition tables. >>>>>>> >>>>>> >>>>>> PBL boot will be broken if the image size is bigger than 512KB, right? >>>>> >>>>> It has to be even smaller than that, to make room for early data. >>>>> >>>> >>>> So if we go with 768KB, do we have to convert all PBL boot to SPL boot? >>> >>> Only the targets that need the extra space. >>> >> >> We have T4, B4 and corenet_ds using PBL boot. They most likely will >> exceed the 512KB soon, if not yet. It maybe easier to change all of them >> togther, than one by one. > > There's no reason to change them all at once. It doesn't make anything > easier; it just means you have to do a bunch of testing all at once, and > force a change in procedure for users on boards where it otherwise would > not have been required. You are right here. > > Plus, the 512K limit is for e500v2-based chips. Newer chips have CPC > for SRAM which is larger than 512K. > Hmm? T4240 has 512KB CPC. York ^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] powerpc/mpc85xx: Increase image size 2013-10-16 20:44 ` York Sun @ 2013-10-16 21:23 ` Scott Wood 2013-10-16 22:43 ` York Sun 0 siblings, 1 reply; 15+ messages in thread From: Scott Wood @ 2013-10-16 21:23 UTC (permalink / raw) To: u-boot On Wed, 2013-10-16 at 13:44 -0700, York Sun wrote: > On 10/16/2013 01:41 PM, Scott Wood wrote: > > On Wed, 2013-10-16 at 13:38 -0700, York Sun wrote: > >> On 10/16/2013 01:33 PM, Scott Wood wrote: > >>> On Wed, 2013-10-16 at 13:32 -0700, York Sun wrote: > >>>> On 10/16/2013 01:29 PM, Scott Wood wrote: > >>>>> On Wed, 2013-10-16 at 13:22 -0700, York Sun wrote: > >>>>>> On 10/16/2013 12:37 PM, Scott Wood wrote: > >>>>>>> On Wed, 2013-10-16 at 10:41 -0700, York Sun wrote: > >>>>>>>> Are SPL and TPL boot methods immune from the size issue here? > >>>>>>> > >>>>>>> Sort of. We still need to fit inside existing partition tables. > >>>>>>> > >>>>>> > >>>>>> PBL boot will be broken if the image size is bigger than 512KB, right? > >>>>> > >>>>> It has to be even smaller than that, to make room for early data. > >>>>> > >>>> > >>>> So if we go with 768KB, do we have to convert all PBL boot to SPL boot? > >>> > >>> Only the targets that need the extra space. > >>> > >> > >> We have T4, B4 and corenet_ds using PBL boot. They most likely will > >> exceed the 512KB soon, if not yet. It maybe easier to change all of them > >> togther, than one by one. > > > > There's no reason to change them all at once. It doesn't make anything > > easier; it just means you have to do a bunch of testing all at once, and > > force a change in procedure for users on boards where it otherwise would > > not have been required. > > You are right here. > > > > > Plus, the 512K limit is for e500v2-based chips. Newer chips have CPC > > for SRAM which is larger than 512K. > > > > Hmm? T4240 has 512KB CPC. Each CPC is 512K, but there are three of them. Is it possible to use more than one for SRAM, contiguously? Still, I find it curious that T4240 has less CPC than P4080, and less CPC than L2 cache. -Scott ^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] powerpc/mpc85xx: Increase image size 2013-10-16 21:23 ` Scott Wood @ 2013-10-16 22:43 ` York Sun 2013-10-17 1:35 ` Kushwaha Prabhakar-B32579 0 siblings, 1 reply; 15+ messages in thread From: York Sun @ 2013-10-16 22:43 UTC (permalink / raw) To: u-boot On 10/16/2013 02:23 PM, Scott Wood wrote: > On Wed, 2013-10-16 at 13:44 -0700, York Sun wrote: >> On 10/16/2013 01:41 PM, Scott Wood wrote: >>> On Wed, 2013-10-16 at 13:38 -0700, York Sun wrote: >>>> On 10/16/2013 01:33 PM, Scott Wood wrote: >>>>> On Wed, 2013-10-16 at 13:32 -0700, York Sun wrote: >>>>>> On 10/16/2013 01:29 PM, Scott Wood wrote: >>>>>>> On Wed, 2013-10-16 at 13:22 -0700, York Sun wrote: >>>>>>>> On 10/16/2013 12:37 PM, Scott Wood wrote: >>>>>>>>> On Wed, 2013-10-16 at 10:41 -0700, York Sun wrote: >>>>>>>>>> Are SPL and TPL boot methods immune from the size issue here? >>>>>>>>> >>>>>>>>> Sort of. We still need to fit inside existing partition tables. >>>>>>>>> >>>>>>>> >>>>>>>> PBL boot will be broken if the image size is bigger than 512KB, right? >>>>>>> >>>>>>> It has to be even smaller than that, to make room for early data. >>>>>>> >>>>>> >>>>>> So if we go with 768KB, do we have to convert all PBL boot to SPL boot? >>>>> >>>>> Only the targets that need the extra space. >>>>> >>>> >>>> We have T4, B4 and corenet_ds using PBL boot. They most likely will >>>> exceed the 512KB soon, if not yet. It maybe easier to change all of them >>>> togther, than one by one. >>> >>> There's no reason to change them all at once. It doesn't make anything >>> easier; it just means you have to do a bunch of testing all at once, and >>> force a change in procedure for users on boards where it otherwise would >>> not have been required. >> >> You are right here. >> >>> >>> Plus, the 512K limit is for e500v2-based chips. Newer chips have CPC >>> for SRAM which is larger than 512K. >>> >> >> Hmm? T4240 has 512KB CPC. > > Each CPC is 512K, but there are three of them. Is it possible to use > more than one for SRAM, contiguously? I think it is possible to use two CPC. The PBL boot method is hard-coded to take 512KB u-boot.bin and generate u-boot.pbl. It surely can be improved. My point is making a move for all concerned platforms may be easier for maintenance. York ^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] powerpc/mpc85xx: Increase image size 2013-10-16 22:43 ` York Sun @ 2013-10-17 1:35 ` Kushwaha Prabhakar-B32579 0 siblings, 0 replies; 15+ messages in thread From: Kushwaha Prabhakar-B32579 @ 2013-10-17 1:35 UTC (permalink / raw) To: u-boot > -----Original Message----- > From: u-boot-bounces at lists.denx.de [mailto:u-boot-bounces at lists.denx.de] > On Behalf Of York Sun > Sent: Thursday, October 17, 2013 4:14 AM > To: Wood Scott-B07421 > Cc: Tom Rini; u-boot at lists.denx.de > Subject: Re: [U-Boot] powerpc/mpc85xx: Increase image size > > On 10/16/2013 02:23 PM, Scott Wood wrote: > > On Wed, 2013-10-16 at 13:44 -0700, York Sun wrote: > >> On 10/16/2013 01:41 PM, Scott Wood wrote: > >>> On Wed, 2013-10-16 at 13:38 -0700, York Sun wrote: > >>>> On 10/16/2013 01:33 PM, Scott Wood wrote: > >>>>> On Wed, 2013-10-16 at 13:32 -0700, York Sun wrote: > >>>>>> On 10/16/2013 01:29 PM, Scott Wood wrote: > >>>>>>> On Wed, 2013-10-16 at 13:22 -0700, York Sun wrote: > >>>>>>>> On 10/16/2013 12:37 PM, Scott Wood wrote: > >>>>>>>>> On Wed, 2013-10-16 at 10:41 -0700, York Sun wrote: > >>>>>>>>>> Are SPL and TPL boot methods immune from the size issue here? > >>>>>>>>> > >>>>>>>>> Sort of. We still need to fit inside existing partition > tables. > >>>>>>>>> > >>>>>>>> > >>>>>>>> PBL boot will be broken if the image size is bigger than 512KB, > right? > >>>>>>> > >>>>>>> It has to be even smaller than that, to make room for early data. > >>>>>>> > >>>>>> > >>>>>> So if we go with 768KB, do we have to convert all PBL boot to SPL > boot? > >>>>> > >>>>> Only the targets that need the extra space. > >>>>> > >>>> > >>>> We have T4, B4 and corenet_ds using PBL boot. They most likely will > >>>> exceed the 512KB soon, if not yet. It maybe easier to change all of > >>>> them togther, than one by one. > >>> > >>> There's no reason to change them all at once. It doesn't make > >>> anything easier; it just means you have to do a bunch of testing all > >>> at once, and force a change in procedure for users on boards where > >>> it otherwise would not have been required. > >> > >> You are right here. > >> > >>> > >>> Plus, the 512K limit is for e500v2-based chips. Newer chips have > >>> CPC for SRAM which is larger than 512K. > >>> > >> > >> Hmm? T4240 has 512KB CPC. > > > > Each CPC is 512K, but there are three of them. Is it possible to use > > more than one for SRAM, contiguously? > > I think it is possible to use two CPC. The PBL boot method is hard-coded > to take 512KB u-boot.bin and generate u-boot.pbl. It surely can be > improved. My point is making a move for all concerned platforms may be > easier for maintenance. > We can go ahead with 2 stage boot loaders for SoC having < 1MB CPC. We have already sent patch for T1040 which has 256K CPC. This same patch can be used for T4 and B4. FYI, I have also sent one RFC for B4 with 2stage boot loader. It can be taken as full patch. Regards, Prabhakar ^ permalink raw reply [flat|nested] 15+ messages in thread
end of thread, other threads:[~2013-10-17 1:35 UTC | newest] Thread overview: 15+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2013-10-11 18:29 [U-Boot] powerpc/mpc85xx: Increase image size York Sun 2013-10-11 18:39 ` Scott Wood 2013-10-11 18:56 ` York Sun 2013-10-16 17:41 ` York Sun 2013-10-16 19:37 ` Scott Wood 2013-10-16 20:22 ` York Sun 2013-10-16 20:29 ` Scott Wood 2013-10-16 20:32 ` York Sun 2013-10-16 20:33 ` Scott Wood 2013-10-16 20:38 ` York Sun 2013-10-16 20:41 ` Scott Wood 2013-10-16 20:44 ` York Sun 2013-10-16 21:23 ` Scott Wood 2013-10-16 22:43 ` York Sun 2013-10-17 1:35 ` Kushwaha Prabhakar-B32579
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox