From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kumar Gala Subject: Re: [PATCH 2/3] arm64: Add Kconfig option for Samsung GH7 SoC family Date: Wed, 12 Feb 2014 10:25:59 -0600 Message-ID: References: <1392100183-30930-1-git-send-email-kgene.kim@samsung.com> <1392100183-30930-3-git-send-email-kgene.kim@samsung.com> <20140212103824.GF29702@arm.com> Mime-Version: 1.0 (Mac OS X Mail 7.1 \(1827\)) Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 8BIT Return-path: Received: from smtp.codeaurora.org ([198.145.11.231]:60071 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753175AbaBLQ0J convert rfc822-to-8bit (ORCPT ); Wed, 12 Feb 2014 11:26:09 -0500 In-Reply-To: <20140212103824.GF29702@arm.com> Sender: linux-samsung-soc-owner@vger.kernel.org List-Id: linux-samsung-soc@vger.kernel.org To: Catalin Marinas Cc: Olof Johansson , Thomas Abraham , "linux-samsung-soc@vger.kernel.org" , Kukjin Kim , Ilho Lee , "linux-arm-kernel@lists.infradead.org" On Feb 12, 2014, at 4:38 AM, Catalin Marinas wrote: > On Tue, Feb 11, 2014 at 11:39:27PM +0000, Olof Johansson wrote: >> On Mon, Feb 10, 2014 at 10:29 PM, Kukjin Kim wrote: >>> This patch adds support for Samsung GH7 SoC in arm64/Kconfig. >>> >>> Signed-off-by: Kukjin Kim >>> Cc: Catalin Marinas >> >> The overhead of building one more device tree isn't very large, and I >> don't see any other need to have a Kconfig entry per SoC at this time. >> It's of course up to Catalin, but you might just want to always >> compile all dts files instead. > > For arm64, I thought of getting rid of ARCH_* Kconfig entries entirely, > only that I haven't heard any strong opinion either way (in which case > I'll do it, with a risk of single Image getting bigger and bigger and > people needing smaller Image can trim their .config). One reason to keep around ARCH_* is for drivers shared between arm and arm64 that depend on it. - k -- Employee of Qualcomm Innovation Center, Inc. Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation