All of lore.kernel.org
 help / color / mirror / Atom feed
From: Catalin Marinas <catalin.marinas@arm.com>
To: Olof Johansson <olof@lixom.net>
Cc: John Stultz <john.stultz@linaro.org>,
	Arnd Bergmann <arnd@arndb.de>, Kukjin Kim <kgene.kim@samsung.com>,
	"linux-samsung-soc@vger.kernel.org"
	<linux-samsung-soc@vger.kernel.org>,
	Ilho Lee <ilho215.lee@samsung.com>,
	Thomas Abraham <thomas.ab@samsung.com>,
	Kumar Gala <galak@codeaurora.org>,
	"linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>,
	Daniel Lezcano <daniel.lezcano@linaro.org>,
	Thomas Gleixner <tglx@linutronix.de>
Subject: Re: [PATCH 2/3] arm64: Add Kconfig option for Samsung GH7 SoC family
Date: Thu, 20 Feb 2014 11:22:48 +0000	[thread overview]
Message-ID: <20140220112248.GC32456@localhost> (raw)
In-Reply-To: <CAOesGMjSjzZ4iX3NVA6eS4LHJF2MQQ=KJpNgKWikJb6YkQxNpQ@mail.gmail.com>

On Thu, Feb 20, 2014 at 09:03:30AM +0000, Olof Johansson wrote:
> So, after giving this some more thought (and getting my hands dirty in
> some of this code), I think I'm going to change my mind on this. For
> mobile platforms I think it might make sense to bring over the
> toplevel platform Kconfig from arch/arm, to simplify dependencies
> without tearing up the driver subtree with churn like this.
> 
> This, of course, only holds true for v8 mobile platforms. Samsung
> isn't saying if GH7 is a server platform and not, and they don't have
> to tell us. But I think we should consider only enabling and bringing
> over the mobile ones (and ideally try to avoid even that, but it might
> make sense to do some of them at least initially -- it does provide
> some convenient ways to enable larger subsets of default drivers per
> platform/vendor family).
> 
> I.e. I'd be OK with an
> ARCH_EXYNOS/ARCH_TEGRA/ARCH_IMX/ARCH_<whatever>, but I don't think we
> should add more finegrained options than that globally on ARM64, at
> least not until truly proven to be needed. We're trying to push back
> against new per-SoC Kconfig entries on 32-bit as well right now.

I'm fine with this. Do we still need something for ARMv8 server
platforms like ARCH_ARM_SBSA? The only advantage would be to make it
easier for mobile targeted kernel builds to disable server features but
I'm not sure there are so many such features, people can trim the
.config manually.

Two additional points:

1. Single arm64 defconfig file covering everything
2. Modules rather than built-in by default where possible (especially
   for server platforms)

-- 
Catalin

WARNING: multiple messages have this Message-ID (diff)
From: catalin.marinas@arm.com (Catalin Marinas)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 2/3] arm64: Add Kconfig option for Samsung GH7 SoC family
Date: Thu, 20 Feb 2014 11:22:48 +0000	[thread overview]
Message-ID: <20140220112248.GC32456@localhost> (raw)
In-Reply-To: <CAOesGMjSjzZ4iX3NVA6eS4LHJF2MQQ=KJpNgKWikJb6YkQxNpQ@mail.gmail.com>

On Thu, Feb 20, 2014 at 09:03:30AM +0000, Olof Johansson wrote:
> So, after giving this some more thought (and getting my hands dirty in
> some of this code), I think I'm going to change my mind on this. For
> mobile platforms I think it might make sense to bring over the
> toplevel platform Kconfig from arch/arm, to simplify dependencies
> without tearing up the driver subtree with churn like this.
> 
> This, of course, only holds true for v8 mobile platforms. Samsung
> isn't saying if GH7 is a server platform and not, and they don't have
> to tell us. But I think we should consider only enabling and bringing
> over the mobile ones (and ideally try to avoid even that, but it might
> make sense to do some of them at least initially -- it does provide
> some convenient ways to enable larger subsets of default drivers per
> platform/vendor family).
> 
> I.e. I'd be OK with an
> ARCH_EXYNOS/ARCH_TEGRA/ARCH_IMX/ARCH_<whatever>, but I don't think we
> should add more finegrained options than that globally on ARM64, at
> least not until truly proven to be needed. We're trying to push back
> against new per-SoC Kconfig entries on 32-bit as well right now.

I'm fine with this. Do we still need something for ARMv8 server
platforms like ARCH_ARM_SBSA? The only advantage would be to make it
easier for mobile targeted kernel builds to disable server features but
I'm not sure there are so many such features, people can trim the
.config manually.

Two additional points:

1. Single arm64 defconfig file covering everything
2. Modules rather than built-in by default where possible (especially
   for server platforms)

-- 
Catalin

  reply	other threads:[~2014-02-20 11:22 UTC|newest]

Thread overview: 82+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-02-11  6:29 [PATCH 0/3] arm64: add new support Samsung GH7 SoC and SSDK board Kukjin Kim
2014-02-11  6:29 ` Kukjin Kim
2014-02-11  6:29 ` [PATCH 1/3] arm64: dts: add initial dts for Samsung GH7 SoC and SSDK-GH7 board Kukjin Kim
2014-02-11  6:29   ` Kukjin Kim
2014-02-11 18:15   ` Mark Rutland
2014-02-11 18:15     ` Mark Rutland
2014-02-11  3:16     ` Kukjin Kim
2014-02-11  3:16       ` Kukjin Kim
2014-02-18 10:30       ` Mark Rutland
2014-02-18 10:30         ` Mark Rutland
2014-02-24 23:56         ` Kukjin Kim
2014-02-24 23:56           ` Kukjin Kim
2014-02-11 23:36   ` Olof Johansson
2014-02-11 23:36     ` Olof Johansson
2014-02-11  3:25     ` Kukjin Kim
2014-02-11  3:25       ` Kukjin Kim
2014-02-12 11:13     ` Marc Zyngier
2014-02-12 11:13       ` Marc Zyngier
2014-02-12 11:29       ` Mark Rutland
2014-02-12 11:29         ` Mark Rutland
2014-02-12 11:40         ` Marc Zyngier
2014-02-12 11:40           ` Marc Zyngier
2014-02-11  3:03           ` Kukjin Kim
2014-02-11  3:03             ` Kukjin Kim
2014-02-11  6:29 ` [PATCH 2/3] arm64: Add Kconfig option for Samsung GH7 SoC family Kukjin Kim
2014-02-11  6:29   ` Kukjin Kim
2014-02-11 23:39   ` Olof Johansson
2014-02-11 23:39     ` Olof Johansson
2014-02-12 10:38     ` Catalin Marinas
2014-02-12 10:38       ` Catalin Marinas
2014-02-12 16:25       ` Kumar Gala
2014-02-12 16:25         ` Kumar Gala
2014-02-12 18:12         ` Catalin Marinas
2014-02-12 18:12           ` Catalin Marinas
2014-02-12 19:04           ` Kumar Gala
2014-02-12 19:04             ` Kumar Gala
2014-02-12 19:14             ` Arnd Bergmann
2014-02-12 19:14               ` Arnd Bergmann
2014-02-11  2:52               ` Kukjin Kim
2014-02-11  2:52                 ` Kukjin Kim
2014-02-13 19:26                 ` Olof Johansson
2014-02-13 19:26                   ` Olof Johansson
2014-02-14 17:06                   ` Arnd Bergmann
2014-02-14 17:06                     ` Arnd Bergmann
2014-02-18  1:10                     ` Kukjin Kim
2014-02-18  1:10                       ` Kukjin Kim
2014-02-18 10:53                       ` Arnd Bergmann
2014-02-18 10:53                         ` Arnd Bergmann
2014-02-18 16:16                       ` Olof Johansson
2014-02-18 16:16                         ` Olof Johansson
2014-02-18 18:13                         ` Arnd Bergmann
2014-02-18 18:13                           ` Arnd Bergmann
2014-02-18 19:52                           ` John Stultz
2014-02-18 19:52                             ` John Stultz
2014-02-18 20:00                             ` Olof Johansson
2014-02-18 20:00                               ` Olof Johansson
2014-02-18 20:06                               ` John Stultz
2014-02-18 20:06                                 ` John Stultz
2014-02-20  9:03                                 ` Olof Johansson
2014-02-20  9:03                                   ` Olof Johansson
2014-02-20 11:22                                   ` Catalin Marinas [this message]
2014-02-20 11:22                                     ` Catalin Marinas
2014-02-20 12:07                                     ` Arnd Bergmann
2014-02-20 12:07                                       ` Arnd Bergmann
2014-02-20 17:09                                     ` Olof Johansson
2014-02-20 17:09                                       ` Olof Johansson
2014-02-20 18:58                                       ` Catalin Marinas
2014-02-20 18:58                                         ` Catalin Marinas
2014-02-25  0:19                                         ` Kukjin Kim
2014-02-25  0:19                                           ` Kukjin Kim
2014-02-25  0:20                                   ` Kukjin Kim
2014-02-25  0:20                                     ` Kukjin Kim
2014-02-25  0:10                               ` Kukjin Kim
2014-02-25  0:10                                 ` Kukjin Kim
2014-02-18 16:40                       ` Catalin Marinas
2014-02-18 16:40                         ` Catalin Marinas
2014-02-13 20:08     ` Rob Herring
2014-02-13 20:08       ` Rob Herring
2014-02-13 20:19       ` Olof Johansson
2014-02-13 20:19         ` Olof Johansson
2014-02-11  6:29 ` [PATCH 3/3] Documentation: DT: add new entry for Samsung GH7 SoC and SSDK board Kukjin Kim
2014-02-11  6:29   ` Kukjin Kim

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=20140220112248.GC32456@localhost \
    --to=catalin.marinas@arm.com \
    --cc=arnd@arndb.de \
    --cc=daniel.lezcano@linaro.org \
    --cc=galak@codeaurora.org \
    --cc=ilho215.lee@samsung.com \
    --cc=john.stultz@linaro.org \
    --cc=kgene.kim@samsung.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-samsung-soc@vger.kernel.org \
    --cc=olof@lixom.net \
    --cc=tglx@linutronix.de \
    --cc=thomas.ab@samsung.com \
    /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.