public inbox for linux-arm-kernel@lists.infradead.org 
 help / color / mirror / Atom feed
From: nsekhar@ti•com (Sekhar Nori)
To: linux-arm-kernel@lists•infradead.org
Subject: [PATCH] ARM: davinci_all_defconfig: enable CONFIG_FHANDLE
Date: Fri, 15 Apr 2016 15:16:34 +0530	[thread overview]
Message-ID: <5710B87A.7060201@ti.com> (raw)
In-Reply-To: <570B7C89.9030200@ti.com>

On Monday 11 April 2016 03:59 PM, Sekhar Nori wrote:
> On Saturday 09 April 2016 03:24 AM, Kevin Hilman wrote:
>> Sekhar Nori <nsekhar@ti•com> writes:
>>
>>> CONFIG_FHANDLE is required by systemd, which is the
>>> default init system in more and more distributions.
>>
>> There's a bunch more options required for basic systemd support also[1],
>> so those should probably be added with the changelog referencing the
>> source[1]
> 
> Good point. Will send a v2.

I looked at this bit more closely. CONFIG_EPOLL, CONFIG_SIGNALFD, and
CONFIG_TIMERFD are all default y and so actually get removed when the
next 'make savedefconfig' syncup will be done. Same thing for PROC_FS
and SYSFS.

Looking at the list of required options, FHANDLE is the only one which
needs to get enabled explicitly. Also, CONFIG_INOTIFY is actually a
stale entry that can be removed.

Based on this, I think the updated patch can be:

---8<---
From: Sekhar Nori <nsekhar@ti•com>
Date: Fri, 15 Apr 2016 14:19:26 +0530
Subject: [PATCH] ARM: davinci_all_defconfig: support systemd

CONFIG_FHANDLE is required by systemd[1], which is the
default init system in more and more distributions.
So lets enable it for DaVinci platforms as well.

While at it, remove stale entry CONFIG_INOTIFY=y

[1] https://github.com/systemd/systemd/blob/master/README#L37

Signed-off-by: Sekhar Nori <nsekhar@ti•com>
---
 arch/arm/configs/davinci_all_defconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/configs/davinci_all_defconfig
b/arch/arm/configs/davinci_all_defconfig
index c5bccd9d010e..776fd8ad792f 100644
--- a/arch/arm/configs/davinci_all_defconfig
+++ b/arch/arm/configs/davinci_all_defconfig
@@ -2,6 +2,7 @@ CONFIG_EXPERIMENTAL=y
 # CONFIG_SWAP is not set
 CONFIG_SYSVIPC=y
 CONFIG_POSIX_MQUEUE=y
+CONFIG_FHANDLE=y
 CONFIG_IKCONFIG=y
 CONFIG_IKCONFIG_PROC=y
 CONFIG_LOG_BUF_SHIFT=14
@@ -188,7 +189,6 @@ CONFIG_TI_EDMA=y
 CONFIG_EXT2_FS=y
 CONFIG_EXT3_FS=y
 CONFIG_XFS_FS=m
-CONFIG_INOTIFY=y
 CONFIG_AUTOFS4_FS=m
 CONFIG_MSDOS_FS=y
 CONFIG_VFAT_FS=y
-- 
2.6.3

  reply	other threads:[~2016-04-15  9:46 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-04-07 10:06 [PATCH] ARM: davinci_all_defconfig: enable CONFIG_FHANDLE Sekhar Nori
2016-04-08 21:54 ` Kevin Hilman
2016-04-11 10:29   ` Sekhar Nori
2016-04-15  9:46     ` Sekhar Nori [this message]
2016-04-18 22:46       ` Kevin Hilman
2016-04-25 12:49         ` Sekhar Nori

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5710B87A.7060201@ti.com \
    --to=nsekhar@ti$(echo .)com \
    --cc=linux-arm-kernel@lists$(echo .)infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox