diff mbox

[1/2] ARM: mvebu: Enable nfsroot in the defconfig

Message ID 1396539014-8673-1-git-send-email-ezequiel.garcia@free-electrons.com (mailing list archive)
State New, archived
Headers show

Commit Message

Ezequiel Garcia April 3, 2014, 3:30 p.m. UTC
NFSroot is very frequently used by developers to boot, so let's make
our lives simpler and enable it by default.

Signed-off-by: Ezequiel Garcia <ezequiel.garcia@free-electrons.com>
---
 arch/arm/configs/mvebu_v7_defconfig | 5 +++++
 1 file changed, 5 insertions(+)

Comments

Ezequiel Garcia April 16, 2014, 3:52 p.m. UTC | #1
Hello guys,

On Apr 03, Ezequiel Garcia wrote:
> NFSroot is very frequently used by developers to boot, so let's make
> our lives simpler and enable it by default.
> 
> Signed-off-by: Ezequiel Garcia <ezequiel.garcia@free-electrons.com>
> ---
>  arch/arm/configs/mvebu_v7_defconfig | 5 +++++
>  1 file changed, 5 insertions(+)
> 

Any comments about this two defconfig changes?

After the recent systemd debug mess, now I'm not sure we should add a defconfig,
just because systemd breaks without it.

Maybe we can merge this one and drop the second one?
Jason Cooper April 16, 2014, 5:01 p.m. UTC | #2
On Wed, Apr 16, 2014 at 12:52:12PM -0300, Ezequiel Garcia wrote:
> Hello guys,
> 
> On Apr 03, Ezequiel Garcia wrote:
> > NFSroot is very frequently used by developers to boot, so let's make
> > our lives simpler and enable it by default.
> > 
> > Signed-off-by: Ezequiel Garcia <ezequiel.garcia@free-electrons.com>
> > ---
> >  arch/arm/configs/mvebu_v7_defconfig | 5 +++++
> >  1 file changed, 5 insertions(+)
> > 
> 
> Any comments about this two defconfig changes?

Nope.

> After the recent systemd debug mess, now I'm not sure we should add a defconfig,
> just because systemd breaks without it.

If we (mvebu maintainers) don't mind changing the mvebu_*_defconfigs,
who's going to complain?  If it helps your workflow, then we'll add it.

My only issue would be if the change breaks the bootfarm testing.

The debug thread and this series are completely orthogonal.

thx,

Jason.
Jason Cooper April 24, 2014, 4:34 a.m. UTC | #3
On Thu, Apr 03, 2014 at 12:30:13PM -0300, Ezequiel Garcia wrote:
> NFSroot is very frequently used by developers to boot, so let's make
> our lives simpler and enable it by default.
> 
> Signed-off-by: Ezequiel Garcia <ezequiel.garcia@free-electrons.com>
> ---
>  arch/arm/configs/mvebu_v7_defconfig | 5 +++++
>  1 file changed, 5 insertions(+)

Both applied to mvebu/defconfig

thx,

Jason.
diff mbox

Patch

diff --git a/arch/arm/configs/mvebu_v7_defconfig b/arch/arm/configs/mvebu_v7_defconfig
index a34713d..bfcc362 100644
--- a/arch/arm/configs/mvebu_v7_defconfig
+++ b/arch/arm/configs/mvebu_v7_defconfig
@@ -29,6 +29,9 @@  CONFIG_ARM_ATAG_DTB_COMPAT=y
 CONFIG_VFP=y
 CONFIG_NET=y
 CONFIG_INET=y
+CONFIG_IP_PNP=y
+CONFIG_IP_PNP_DHCP=y
+CONFIG_IP_PNP_BOOTP=y
 CONFIG_BT=y
 CONFIG_BT_MRVL=y
 CONFIG_BT_MRVL_SDIO=y
@@ -103,6 +106,8 @@  CONFIG_UDF_FS=m
 CONFIG_MSDOS_FS=y
 CONFIG_VFAT_FS=y
 CONFIG_TMPFS=y
+CONFIG_NFS_FS=y
+CONFIG_ROOT_NFS=y
 CONFIG_NLS_CODEPAGE_437=y
 CONFIG_NLS_CODEPAGE_850=y
 CONFIG_NLS_ISO8859_1=y