All of lore.kernel.org
 help / color / mirror / Atom feed
From: khilman@baylibre.com (Kevin Hilman)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] ARM: davinci_all_defconfig: enable CONFIG_FHANDLE
Date: Mon, 18 Apr 2016 15:46:40 -0700	[thread overview]
Message-ID: <7hh9eycz73.fsf@baylibre.com> (raw)
In-Reply-To: <5710B87A.7060201@ti.com> (Sekhar Nori's message of "Fri, 15 Apr 2016 15:16:34 +0530")

Sekhar Nori <nsekhar@ti.com> writes:

> 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>

Acked-by: Kevin Hilman <khilman@baylibre.com>

  reply	other threads:[~2016-04-18 22: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
2016-04-18 22:46       ` Kevin Hilman [this message]
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=7hh9eycz73.fsf@baylibre.com \
    --to=khilman@baylibre.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.