From patchwork Thu Sep 20 17:14:01 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Paul Walmsley X-Patchwork-Id: 1486731 Return-Path: X-Original-To: patchwork-linux-omap@patchwork.kernel.org Delivered-To: patchwork-process-083081@patchwork2.kernel.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by patchwork2.kernel.org (Postfix) with ESMTP id 1FCF1DF2D2 for ; Thu, 20 Sep 2012 17:14:05 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752004Ab2ITROD (ORCPT ); Thu, 20 Sep 2012 13:14:03 -0400 Received: from utopia.booyaka.com ([74.50.51.50]:59895 "EHLO utopia.booyaka.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751247Ab2ITROC (ORCPT ); Thu, 20 Sep 2012 13:14:02 -0400 Received: (qmail 14920 invoked by uid 1019); 20 Sep 2012 17:14:01 -0000 Date: Thu, 20 Sep 2012 17:14:01 +0000 (UTC) From: Paul Walmsley To: Jon Hunter cc: Tony Lindgren , linux-omap , linux-arm , Ming Lei , Will Deacon , Benoit Cousson , Kevin Hilman Subject: Re: [PATCH V3 4/8] ARM: OMAP4430: Create PMU device via HWMOD In-Reply-To: <1347290626-21164-5-git-send-email-jon-hunter@ti.com> Message-ID: References: <1347290626-21164-1-git-send-email-jon-hunter@ti.com> <1347290626-21164-5-git-send-email-jon-hunter@ti.com> User-Agent: Alpine 2.00 (DEB 1167 2008-08-23) MIME-Version: 1.0 Sender: linux-omap-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-omap@vger.kernel.org Hi On Mon, 10 Sep 2012, Jon Hunter wrote: > From: Ming Lei > > For OMAP4430 PMU events are routed to the CPU via the cross trigger interface > (CTI) because there are no dedicated interrupts. In order to route the PMU > events via the CTI IRQs, the following modules must be enabled: > > l3_instr, l3_main_3, debugss > > Therefore, build the arm-pmu device via these three HWMODs. > > Cc: Ming Lei > Cc: Will Deacon > Cc: Benoit Cousson > Cc: Paul Walmsley > Cc: Kevin Hilman > > Signed-off-by: Ming Lei > Signed-off-by: Will Deacon > Signed-off-by: Jon Hunter Modified this one to skip the OMAP4430 PMU device creation for now and log a message; updated patch below. - Paul From: Ming Lei Date: Mon, 10 Sep 2012 10:23:42 -0500 Subject: [PATCH] ARM: OMAP4430: PMU: prepare to create PMU device via HWMOD For OMAP4430 PMU events are routed to the CPU via the cross trigger interface (CTI) because there are no dedicated interrupts. In order to route the PMU events via the CTI IRQs, the following modules must be enabled: l3_instr, l3_main_3, debugss Therefore, build the arm-pmu device via these three HWMODs. However, the CTI support for this platform still needs some work. Until that's finished, temporarily disable the PMU on OMAP4430. Cc: Ming Lei Cc: Will Deacon Cc: Benoit Cousson Cc: Paul Walmsley Cc: Kevin Hilman Signed-off-by: Ming Lei Signed-off-by: Will Deacon Signed-off-by: Jon Hunter [paul@pwsan.com: temporarily disabled OMAP4430 PMU support until a better CTI interface can be implemented; added patch description note] Signed-off-by: Paul Walmsley --- arch/arm/mach-omap2/pmu.c | 16 ++++++++++++---- 1 file changed, 12 insertions(+), 4 deletions(-) diff --git a/arch/arm/mach-omap2/pmu.c b/arch/arm/mach-omap2/pmu.c index cf68bab..c49e058 100644 --- a/arch/arm/mach-omap2/pmu.c +++ b/arch/arm/mach-omap2/pmu.c @@ -19,6 +19,7 @@ static char *omap2_pmu_oh_names[] = {"mpu"}; static char *omap3_pmu_oh_names[] = {"mpu", "debugss"}; +static char *omap4430_pmu_oh_names[] = {"l3_main_3", "l3_instr", "debugss"}; static struct platform_device *omap_pmu_dev; /** @@ -27,16 +28,16 @@ static struct platform_device *omap_pmu_dev; * @oh_names: Array of OMAP HWMODS names required to create PMU device * * Uses OMAP HWMOD framework to create and register an ARM PMU device - * from a list of HWMOD names passed. Currently supports OMAP2 and - * OMAP3 devices. + * from a list of HWMOD names passed. Currently supports OMAP2, OMAP3 + * and OMAP4430 devices. */ static int __init omap2_init_pmu(unsigned oh_num, char *oh_names[]) { int i; - struct omap_hwmod *oh[2]; + struct omap_hwmod *oh[3]; char *dev_name = "arm-pmu"; - if ((!oh_num) || (oh_num > 2)) + if ((!oh_num) || (oh_num > 3)) return -EINVAL; for (i = 0; i < oh_num; i++) { @@ -66,6 +67,7 @@ static int __init omap_init_pmu(void) * * OMAP24xx: mpu * OMAP3xxx: mpu, debugss + * OMAP4430: l3_main_3, l3_instr, debugss */ if (cpu_is_omap24xx()) { oh_num = ARRAY_SIZE(omap2_pmu_oh_names); @@ -73,6 +75,12 @@ static int __init omap_init_pmu(void) } else if (cpu_is_omap34xx()) { oh_num = ARRAY_SIZE(omap3_pmu_oh_names); oh_names = omap3_pmu_oh_names; + } else if (cpu_is_omap443x()) { + oh_num = ARRAY_SIZE(omap4430_pmu_oh_names); + oh_names = omap4430_pmu_oh_names; + /* XXX Remove the next two lines when CTI driver available */ + pr_info("ARM PMU: not yet supported on OMAP4430 due to missing CTI driver\n"); + return 0; } else { return 0; }