From patchwork Mon Sep 26 08:48:31 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: David Engraf X-Patchwork-Id: 675018 Return-Path: X-Original-To: patchwork-incoming@ozlabs.org Delivered-To: patchwork-incoming@ozlabs.org Received: from lists.ozlabs.org (lists.ozlabs.org [103.22.144.68]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 3sjHrt1F35z9s65 for ; Mon, 26 Sep 2016 18:56:46 +1000 (AEST) Received: from lists.ozlabs.org (lists.ozlabs.org [IPv6:2401:3900:2:1::3]) by lists.ozlabs.org (Postfix) with ESMTP id 3sjHrt03t4zDrRR for ; Mon, 26 Sep 2016 18:56:46 +1000 (AEST) X-Original-To: linuxppc-dev@lists.ozlabs.org Delivered-To: linuxppc-dev@lists.ozlabs.org X-Greylist: delayed 421 seconds by postgrey-1.36 at bilbo; Mon, 26 Sep 2016 18:55:40 AEST Received: from mail.sysgo.com (mail.sysgo.com [176.9.12.79]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 3sjHqc6xWMzDrPk for ; Mon, 26 Sep 2016 18:55:40 +1000 (AEST) From: David Engraf Subject: Re: powerpc64: Enable CONFIG_E500 and CONFIG_PPC_E500MC for e5500/e6500 To: Scott Wood References: <1471877203-29327-1-git-send-email-david.engraf@sysgo.com> <20160925062017.GA10754@home.buserror.net> Message-ID: <87664de8-7aea-96a8-9906-cf2ce8d325a2@sysgo.com> Date: Mon, 26 Sep 2016 10:48:31 +0200 MIME-Version: 1.0 In-Reply-To: <20160925062017.GA10754@home.buserror.net> X-BeenThere: linuxppc-dev@lists.ozlabs.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: paulus@samba.org, linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org Errors-To: linuxppc-dev-bounces+patchwork-incoming=ozlabs.org@lists.ozlabs.org Sender: "Linuxppc-dev" Am 25.09.2016 um 08:20 schrieb Scott Wood: > On Mon, Aug 22, 2016 at 04:46:43PM +0200, David Engraf wrote: >> The PowerPC e5500/e6500 architecture is based on the e500mc core. Enable >> CONFIG_E500 and CONFIG_PPC_E500MC when e5500/e6500 is used. >> >> This will also fix using CONFIG_PPC_QEMU_E500 on PPC64. >> >> Signed-off-by: David Engraf >> --- >> arch/powerpc/platforms/Kconfig.cputype | 6 ++++-- >> 1 file changed, 4 insertions(+), 2 deletions(-) >> >> diff --git a/arch/powerpc/platforms/Kconfig.cputype b/arch/powerpc/platforms/Kconfig.cputype >> index f32edec..0382da7 100644 >> --- a/arch/powerpc/platforms/Kconfig.cputype >> +++ b/arch/powerpc/platforms/Kconfig.cputype >> @@ -125,11 +125,13 @@ config POWER8_CPU >> >> config E5500_CPU >> bool "Freescale e5500" >> - depends on E500 >> + select E500 >> + select PPC_E500MC >> >> config E6500_CPU >> bool "Freescale e6500" >> - depends on E500 >> + select E500 >> + select PPC_E500MC > > These config symbols are for setting -mcpu. Kernels built with > CONFIG_GENERIC_CPU should also work on e5500/e6500. I don't think so. At least on QEMU it is not working because e5500/e6500 is based on the e500mc core and the option CONFIG_PPC_E500MC also controls the cpu features (check cputable.h). > The problem is that CONFIG_PPC_QEMU_E500 doesn't select E500 (I didn't > notice it before because usually CORENET_GENERIC is enabled as well). I noticed that as well, but I think it makes more sense to select E500/PPC_E500MC within the cputype menu instead of having a dependency which might be not clear for the user. Right now the way how to configure such a BSP is not clear, you need to open "Processor support" and select the "Processor Type", then switch to "Platform support" to select the BSP and afterward got back to "Processor support" to switch from the generic CPU type to e5500/e6500. > Note that your patch, by eliminating the dependency on E500, would make > it possible to build a book3s kernel with E5500_CPU/E6500_CPU, which > doesn't make any sense. You're right. The attached version fixes this. - David diff --git a/arch/powerpc/platforms/Kconfig.cputype b/arch/powerpc/platforms/Kconfig.cputype index f32edec..abd345e 100644 --- a/arch/powerpc/platforms/Kconfig.cputype +++ b/arch/powerpc/platforms/Kconfig.cputype @@ -125,11 +125,15 @@ config POWER8_CPU config E5500_CPU bool "Freescale e5500" - depends on E500 + depends on PPC_BOOK3E_64 + select E500 + select PPC_E500MC config E6500_CPU bool "Freescale e6500" - depends on E500 + depends on PPC_BOOK3E_64 + select E500 + select PPC_E500MC endchoice