Message ID | 20201126100507.27484-1-vivek.gautam@arm.com |
---|---|
State | Rejected |
Headers | show |
Series | configs/aarch64_rdinfra: add new config for infra rd plaforms | expand |
Hi, On 11/26/20 3:35 PM, Vivek Kumar Gautam wrote: > This defconfig enables Arm aarch64 Infra RD platforms to use > buildroot backed ramdisks to perform simple unit/sanity testing. > These infra platforms are currently enabled with a scripts based > build system as can be seen in linaro public repos[1]. A busybox > based boot is also enabled[2]. > We plan to use buildroot for a richer experience while testing and > therefore this config. > This config will also be first step towards enabling buildroot > based build system for infra rd platforms in near future. > It comes with following minimal stuff - > -neoverse_n1 cpu > -bash shell > -vim > -pciutils > -libdtc > -screen > > [1] https://git.linaro.org/landing-teams/working/arm/arm-reference-platforms.git/about/ > [2] https://git.linaro.org/landing-teams/working/arm/arm-reference-platforms.git/about/docs/rdn1edge/how-to/busybox-boot.rst > > Signed-off-by: Vivek Kumar Gautam <vivek.gautam@arm.com> > --- > configs/aarch64_rdinfra_defconfig | 21 +++++++++++++++++++++ > 1 file changed, 21 insertions(+) > create mode 100644 configs/aarch64_rdinfra_defconfig > > diff --git a/configs/aarch64_rdinfra_defconfig b/configs/aarch64_rdinfra_defconfig > new file mode 100644 > index 0000000000..71e44e537c > --- /dev/null > +++ b/configs/aarch64_rdinfra_defconfig > @@ -0,0 +1,21 @@ > +# Architecture > +BR2_aarch64=y > + > +# Processor > +BR2_neoverse_n1=y > + > +# Toolchain > +BR2_TOOLCHAIN_EXTERNAL=y > + > +# Additional system packages > +BR2_PACKAGE_BUSYBOX_SHOW_OTHERS=y > +BR2_PACKAGE_PCIUTILS=y > +BR2_PACKAGE_DTC=y > +BR2_PACKAGE_DTC_PROGRAMS=y > +BR2_PACKAGE_LIBEVENT=y > +BR2_PACKAGE_BASH=y > +BR2_PACKAGE_SCREEN=y > +BR2_PACKAGE_VIM=y > + > +# Target filesystem > +BR2_TARGET_ROOTFS_CPIO=y Gentle ping. Any comments on this? Best regards Vivek IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.
Hi all, On 12/5/20 10:59 AM, Vivek Kumar Gautam wrote: > Hi, > > > On 11/26/20 3:35 PM, Vivek Kumar Gautam wrote: >> This defconfig enables Arm aarch64 Infra RD platforms to use >> buildroot backed ramdisks to perform simple unit/sanity testing. >> These infra platforms are currently enabled with a scripts based >> build system as can be seen in linaro public repos[1]. A busybox >> based boot is also enabled[2]. >> We plan to use buildroot for a richer experience while testing and >> therefore this config. >> This config will also be first step towards enabling buildroot >> based build system for infra rd platforms in near future. >> It comes with following minimal stuff - >> -neoverse_n1 cpu >> -bash shell >> -vim >> -pciutils >> -libdtc >> -screen >> >> [1] >> https://git.linaro.org/landing-teams/working/arm/arm-reference-platforms.git/about/ >> >> [2] >> https://git.linaro.org/landing-teams/working/arm/arm-reference-platforms.git/about/docs/rdn1edge/how-to/busybox-boot.rst >> >> >> Signed-off-by: Vivek Kumar Gautam <vivek.gautam@arm.com> >> --- >> configs/aarch64_rdinfra_defconfig | 21 +++++++++++++++++++++ >> 1 file changed, 21 insertions(+) >> create mode 100644 configs/aarch64_rdinfra_defconfig >> >> diff --git a/configs/aarch64_rdinfra_defconfig >> b/configs/aarch64_rdinfra_defconfig >> new file mode 100644 >> index 0000000000..71e44e537c >> --- /dev/null >> +++ b/configs/aarch64_rdinfra_defconfig >> @@ -0,0 +1,21 @@ >> +# Architecture >> +BR2_aarch64=y >> + >> +# Processor >> +BR2_neoverse_n1=y >> + >> +# Toolchain >> +BR2_TOOLCHAIN_EXTERNAL=y >> + >> +# Additional system packages >> +BR2_PACKAGE_BUSYBOX_SHOW_OTHERS=y >> +BR2_PACKAGE_PCIUTILS=y >> +BR2_PACKAGE_DTC=y >> +BR2_PACKAGE_DTC_PROGRAMS=y >> +BR2_PACKAGE_LIBEVENT=y >> +BR2_PACKAGE_BASH=y >> +BR2_PACKAGE_SCREEN=y >> +BR2_PACKAGE_VIM=y >> + >> +# Target filesystem >> +BR2_TARGET_ROOTFS_CPIO=y > > Gentle ping. Any comments on this? I will be more than happy to address any feedback on this config patch. Please let me know if there are missing pieces. Best regards Vivek IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.
On 1/8/21 2:20 PM, Vivek Kumar Gautam wrote: > Hi all, > > > On 12/5/20 10:59 AM, Vivek Kumar Gautam wrote: >> Hi, >> >> >> On 11/26/20 3:35 PM, Vivek Kumar Gautam wrote: >>> This defconfig enables Arm aarch64 Infra RD platforms to use >>> buildroot backed ramdisks to perform simple unit/sanity testing. >>> These infra platforms are currently enabled with a scripts based >>> build system as can be seen in linaro public repos[1]. A busybox >>> based boot is also enabled[2]. >>> We plan to use buildroot for a richer experience while testing and >>> therefore this config. >>> This config will also be first step towards enabling buildroot >>> based build system for infra rd platforms in near future. >>> It comes with following minimal stuff - >>> -neoverse_n1 cpu >>> -bash shell >>> -vim >>> -pciutils >>> -libdtc >>> -screen >>> >>> [1] >>> https://git.linaro.org/landing-teams/working/arm/arm-reference-platforms.git/about/ >>> >>> [2] >>> https://git.linaro.org/landing-teams/working/arm/arm-reference-platforms.git/about/docs/rdn1edge/how-to/busybox-boot.rst >>> >>> >>> Signed-off-by: Vivek Kumar Gautam <vivek.gautam@arm.com> >>> --- >>> configs/aarch64_rdinfra_defconfig | 21 +++++++++++++++++++++ >>> 1 file changed, 21 insertions(+) >>> create mode 100644 configs/aarch64_rdinfra_defconfig >>> >>> diff --git a/configs/aarch64_rdinfra_defconfig >>> b/configs/aarch64_rdinfra_defconfig >>> new file mode 100644 >>> index 0000000000..71e44e537c >>> --- /dev/null >>> +++ b/configs/aarch64_rdinfra_defconfig >>> @@ -0,0 +1,21 @@ >>> +# Architecture >>> +BR2_aarch64=y >>> + >>> +# Processor >>> +BR2_neoverse_n1=y >>> + >>> +# Toolchain >>> +BR2_TOOLCHAIN_EXTERNAL=y >>> + >>> +# Additional system packages >>> +BR2_PACKAGE_BUSYBOX_SHOW_OTHERS=y >>> +BR2_PACKAGE_PCIUTILS=y >>> +BR2_PACKAGE_DTC=y >>> +BR2_PACKAGE_DTC_PROGRAMS=y >>> +BR2_PACKAGE_LIBEVENT=y >>> +BR2_PACKAGE_BASH=y >>> +BR2_PACKAGE_SCREEN=y >>> +BR2_PACKAGE_VIM=y >>> + >>> +# Target filesystem >>> +BR2_TARGET_ROOTFS_CPIO=y >> >> Gentle ping. Any comments on this? > > I will be more than happy to address any feedback on this config patch. > Please let me know if there are missing pieces. > My apologies for spamming as it looks like a 'Disclaimer' message managed to sneak in with my responses. I have tried to fix it this time. Kindly let me know comments on this patch. Thanks & regards Vivek
Hello Vivek, On Thu, 26 Nov 2020 15:35:07 +0530 Vivek Kumar Gautam <vivek.gautam@arm.com> wrote: > This defconfig enables Arm aarch64 Infra RD platforms to use > buildroot backed ramdisks to perform simple unit/sanity testing. > These infra platforms are currently enabled with a scripts based > build system as can be seen in linaro public repos[1]. A busybox > based boot is also enabled[2]. > We plan to use buildroot for a richer experience while testing and > therefore this config. > This config will also be first step towards enabling buildroot > based build system for infra rd platforms in near future. > It comes with following minimal stuff - > -neoverse_n1 cpu > -bash shell > -vim > -pciutils > -libdtc > -screen > > [1] https://git.linaro.org/landing-teams/working/arm/arm-reference-platforms.git/about/ > [2] https://git.linaro.org/landing-teams/working/arm/arm-reference-platforms.git/about/docs/rdn1edge/how-to/busybox-boot.rst > > Signed-off-by: Vivek Kumar Gautam <vivek.gautam@arm.com> Thanks for the patch and contribution. However, I kind of fail to see the point of a defconfig like the one you propose. It is really doing nothing but building a CPIO archive with a bunch of somewhat random selection of user-space programs and libraries. Our defconfigs are meant to support specific HW platforms, and used to easily build a combination of firmware/bootloader/kernel that works on this hardware platforms, and those defconfigs are typically minimal in terms of user-space packages enabled, as it is very use-case specific what are the user-space packages that make sense to have. Therefore, as it is, your configuration that does not build a kernel, does not build any bootloader, doesn't bring much. Anyone can easily fire up Buildroot, select ARM64, an external toolchain, and a random selection of user-space packages. No defconfig is necessary to achieve this. Could you provide more details about what you are trying to achieve ? Best regards, Thomas Petazzoni
diff --git a/configs/aarch64_rdinfra_defconfig b/configs/aarch64_rdinfra_defconfig new file mode 100644 index 0000000000..71e44e537c --- /dev/null +++ b/configs/aarch64_rdinfra_defconfig @@ -0,0 +1,21 @@ +# Architecture +BR2_aarch64=y + +# Processor +BR2_neoverse_n1=y + +# Toolchain +BR2_TOOLCHAIN_EXTERNAL=y + +# Additional system packages +BR2_PACKAGE_BUSYBOX_SHOW_OTHERS=y +BR2_PACKAGE_PCIUTILS=y +BR2_PACKAGE_DTC=y +BR2_PACKAGE_DTC_PROGRAMS=y +BR2_PACKAGE_LIBEVENT=y +BR2_PACKAGE_BASH=y +BR2_PACKAGE_SCREEN=y +BR2_PACKAGE_VIM=y + +# Target filesystem +BR2_TARGET_ROOTFS_CPIO=y
This defconfig enables Arm aarch64 Infra RD platforms to use buildroot backed ramdisks to perform simple unit/sanity testing. These infra platforms are currently enabled with a scripts based build system as can be seen in linaro public repos[1]. A busybox based boot is also enabled[2]. We plan to use buildroot for a richer experience while testing and therefore this config. This config will also be first step towards enabling buildroot based build system for infra rd platforms in near future. It comes with following minimal stuff - -neoverse_n1 cpu -bash shell -vim -pciutils -libdtc -screen [1] https://git.linaro.org/landing-teams/working/arm/arm-reference-platforms.git/about/ [2] https://git.linaro.org/landing-teams/working/arm/arm-reference-platforms.git/about/docs/rdn1edge/how-to/busybox-boot.rst Signed-off-by: Vivek Kumar Gautam <vivek.gautam@arm.com> --- configs/aarch64_rdinfra_defconfig | 21 +++++++++++++++++++++ 1 file changed, 21 insertions(+) create mode 100644 configs/aarch64_rdinfra_defconfig