diff mbox series

configs/aarch64_rdinfra: add new config for infra rd plaforms

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

Commit Message

Vivek Kumar Gautam Nov. 26, 2020, 10:05 a.m. UTC
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

Comments

Vivek Kumar Gautam Dec. 5, 2020, 5:29 a.m. UTC | #1
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.
Vivek Kumar Gautam Jan. 8, 2021, 8:50 a.m. UTC | #2
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.
Vivek Kumar Gautam Jan. 11, 2021, 2:22 p.m. UTC | #3
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
Thomas Petazzoni Jan. 21, 2021, 9:54 p.m. UTC | #4
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 mbox series

Patch

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