| Message ID | 1509439943-32213-1-git-send-email-kever.yang@rock-chips.com |
|---|---|
| State | Accepted |
| Commit | c5a4141343e601bac2789c137e8be665c074f898 |
| Delegated to: | Philipp Tomsich |
| Headers | show |
| Series | [U-Boot] rockchip: config: update part table | expand |
Hi Kever The reserved partition is not only for U-Boot but also for system. There are customers using reserved partition in system now. 2017-10-31 16:52 GMT+08:00 Kever Yang <kever.yang@rock-chips.com>: > User do not need to access the reserved part in system, remove them > from partition table. > Rename atf to trust as generic name for armv7 do not use ATF. > > Signed-off-by: Kever Yang <kever.yang@rock-chips.com> > --- > > include/configs/rockchip-common.h | 6 ++---- > 1 file changed, 2 insertions(+), 4 deletions(-) > > diff --git a/include/configs/rockchip-common.h b/include/configs/rockchip-common.h > index 35d948a..4139a7b 100644 > --- a/include/configs/rockchip-common.h > +++ b/include/configs/rockchip-common.h > @@ -52,10 +52,8 @@ > #define PARTS_DEFAULT \ > "uuid_disk=${uuid_gpt_disk};" \ > "name=loader1,start=32K,size=4000K,uuid=${uuid_gpt_loader1};" \ > - "name=reserved1,size=64K,uuid=${uuid_gpt_reserved1};" \ > - "name=reserved2,size=4M,uuid=${uuid_gpt_reserved2};" \ > - "name=loader2,size=4MB,uuid=${uuid_gpt_loader2};" \ > - "name=atf,size=4M,uuid=${uuid_gpt_atf};" \ > + "name=loader2,start=8MB,size=4MB,uuid=${uuid_gpt_loader2};" \ > + "name=trust,size=4M,uuid=${uuid_gpt_atf};" \ > "name=boot,size=112M,bootable,uuid=${uuid_gpt_boot};" \ > "name=rootfs,size=-,uuid="ROOT_UUID > > -- > 1.9.1 >
Hi, 2017-11-01 8:36 GMT+08:00 Eddie Cai <eddie.cai.linux@gmail.com>: > Hi Kever > The reserved partition is not only for U-Boot but also for system. > There are customers using reserved partition in system now. > It's ok to hide them, since we just use offset, not partition index. > 2017-10-31 16:52 GMT+08:00 Kever Yang <kever.yang@rock-chips.com>: >> User do not need to access the reserved part in system, remove them >> from partition table. >> Rename atf to trust as generic name for armv7 do not use ATF. >> >> Signed-off-by: Kever Yang <kever.yang@rock-chips.com> >> --- >> >> include/configs/rockchip-common.h | 6 ++---- >> 1 file changed, 2 insertions(+), 4 deletions(-) >> >> diff --git a/include/configs/rockchip-common.h b/include/configs/rockchip-common.h >> index 35d948a..4139a7b 100644 >> --- a/include/configs/rockchip-common.h >> +++ b/include/configs/rockchip-common.h >> @@ -52,10 +52,8 @@ >> #define PARTS_DEFAULT \ >> "uuid_disk=${uuid_gpt_disk};" \ >> "name=loader1,start=32K,size=4000K,uuid=${uuid_gpt_loader1};" \ >> - "name=reserved1,size=64K,uuid=${uuid_gpt_reserved1};" \ >> - "name=reserved2,size=4M,uuid=${uuid_gpt_reserved2};" \ >> - "name=loader2,size=4MB,uuid=${uuid_gpt_loader2};" \ >> - "name=atf,size=4M,uuid=${uuid_gpt_atf};" \ >> + "name=loader2,start=8MB,size=4MB,uuid=${uuid_gpt_loader2};" \ >> + "name=trust,size=4M,uuid=${uuid_gpt_atf};" \ >> "name=boot,size=112M,bootable,uuid=${uuid_gpt_boot};" \ >> "name=rootfs,size=-,uuid="ROOT_UUID >> >> -- >> 1.9.1 >> > _______________________________________________ > U-Boot mailing list > U-Boot@lists.denx.de > https://lists.denx.de/listinfo/u-boot
On Tue, 31 Oct 2017, Kever Yang wrote: > User do not need to access the reserved part in system, remove them > from partition table. > Rename atf to trust as generic name for armv7 do not use ATF. Is this still something that is required following Jacob's comment? Thanks, Phil. > > Signed-off-by: Kever Yang <kever.yang@rock-chips.com> > --- > > include/configs/rockchip-common.h | 6 ++---- > 1 file changed, 2 insertions(+), 4 deletions(-) > > diff --git a/include/configs/rockchip-common.h b/include/configs/rockchip-common.h > index 35d948a..4139a7b 100644 > --- a/include/configs/rockchip-common.h > +++ b/include/configs/rockchip-common.h > @@ -52,10 +52,8 @@ > #define PARTS_DEFAULT \ > "uuid_disk=${uuid_gpt_disk};" \ > "name=loader1,start=32K,size=4000K,uuid=${uuid_gpt_loader1};" \ > - "name=reserved1,size=64K,uuid=${uuid_gpt_reserved1};" \ > - "name=reserved2,size=4M,uuid=${uuid_gpt_reserved2};" \ > - "name=loader2,size=4MB,uuid=${uuid_gpt_loader2};" \ > - "name=atf,size=4M,uuid=${uuid_gpt_atf};" \ > + "name=loader2,start=8MB,size=4MB,uuid=${uuid_gpt_loader2};" \ > + "name=trust,size=4M,uuid=${uuid_gpt_atf};" \ > "name=boot,size=112M,bootable,uuid=${uuid_gpt_boot};" \ > "name=rootfs,size=-,uuid="ROOT_UUID > >
Hi Philipp,
Yes, this is required, we do not need to show the reserved
partitions to users.
Thanks,
- Kever
On 11/20/2017 10:52 PM, Philipp Tomsich wrote:
>
>
> On Tue, 31 Oct 2017, Kever Yang wrote:
>
>> User do not need to access the reserved part in system, remove them
>> from partition table.
>> Rename atf to trust as generic name for armv7 do not use ATF.
>
> Is this still something that is required following Jacob's comment?
>
> Thanks,
> Phil.
>
>>
>> Signed-off-by: Kever Yang <kever.yang@rock-chips.com>
>> ---
>>
>> include/configs/rockchip-common.h | 6 ++----
>> 1 file changed, 2 insertions(+), 4 deletions(-)
>>
>> diff --git a/include/configs/rockchip-common.h
>> b/include/configs/rockchip-common.h
>> index 35d948a..4139a7b 100644
>> --- a/include/configs/rockchip-common.h
>> +++ b/include/configs/rockchip-common.h
>> @@ -52,10 +52,8 @@
>> #define PARTS_DEFAULT \
>> "uuid_disk=${uuid_gpt_disk};" \
>> "name=loader1,start=32K,size=4000K,uuid=${uuid_gpt_loader1};" \
>> - "name=reserved1,size=64K,uuid=${uuid_gpt_reserved1};" \
>> - "name=reserved2,size=4M,uuid=${uuid_gpt_reserved2};" \
>> - "name=loader2,size=4MB,uuid=${uuid_gpt_loader2};" \
>> - "name=atf,size=4M,uuid=${uuid_gpt_atf};" \
>> + "name=loader2,start=8MB,size=4MB,uuid=${uuid_gpt_loader2};" \
>> + "name=trust,size=4M,uuid=${uuid_gpt_atf};" \
>> "name=boot,size=112M,bootable,uuid=${uuid_gpt_boot};" \
>> "name=rootfs,size=-,uuid="ROOT_UUID
>>
>>
>
> User do not need to access the reserved part in system, remove them > from partition table. > Rename atf to trust as generic name for armv7 do not use ATF. > > Signed-off-by: Kever Yang <kever.yang@rock-chips.com> > --- > > include/configs/rockchip-common.h | 6 ++---- > 1 file changed, 2 insertions(+), 4 deletions(-) > Acked-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
> User do not need to access the reserved part in system, remove them > from partition table. > Rename atf to trust as generic name for armv7 do not use ATF. > > Signed-off-by: Kever Yang <kever.yang@rock-chips.com> > Acked-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com> > --- > > include/configs/rockchip-common.h | 6 ++---- > 1 file changed, 2 insertions(+), 4 deletions(-) > Reviewed-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
> User do not need to access the reserved part in system, remove them > from partition table. > Rename atf to trust as generic name for armv7 do not use ATF. > > Signed-off-by: Kever Yang <kever.yang@rock-chips.com> > Acked-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com> > Reviewed-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com> > --- > > include/configs/rockchip-common.h | 6 ++---- > 1 file changed, 2 insertions(+), 4 deletions(-) > Applied to u-boot-rockchip, thanks!
> User do not need to access the reserved part in system, remove them > from partition table. > Rename atf to trust as generic name for armv7 do not use ATF. > > Signed-off-by: Kever Yang <kever.yang@rock-chips.com> > Acked-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com> > Reviewed-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com> > --- > > include/configs/rockchip-common.h | 6 ++---- > 1 file changed, 2 insertions(+), 4 deletions(-) > Reviewed-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
diff --git a/include/configs/rockchip-common.h b/include/configs/rockchip-common.h index 35d948a..4139a7b 100644 --- a/include/configs/rockchip-common.h +++ b/include/configs/rockchip-common.h @@ -52,10 +52,8 @@ #define PARTS_DEFAULT \ "uuid_disk=${uuid_gpt_disk};" \ "name=loader1,start=32K,size=4000K,uuid=${uuid_gpt_loader1};" \ - "name=reserved1,size=64K,uuid=${uuid_gpt_reserved1};" \ - "name=reserved2,size=4M,uuid=${uuid_gpt_reserved2};" \ - "name=loader2,size=4MB,uuid=${uuid_gpt_loader2};" \ - "name=atf,size=4M,uuid=${uuid_gpt_atf};" \ + "name=loader2,start=8MB,size=4MB,uuid=${uuid_gpt_loader2};" \ + "name=trust,size=4M,uuid=${uuid_gpt_atf};" \ "name=boot,size=112M,bootable,uuid=${uuid_gpt_boot};" \ "name=rootfs,size=-,uuid="ROOT_UUID
User do not need to access the reserved part in system, remove them from partition table. Rename atf to trust as generic name for armv7 do not use ATF. Signed-off-by: Kever Yang <kever.yang@rock-chips.com> --- include/configs/rockchip-common.h | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-)