Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752140AbbEFVhv (ORCPT ); Wed, 6 May 2015 17:37:51 -0400 Received: from mail-bn1on0111.outbound.protection.outlook.com ([157.56.110.111]:39223 "EHLO na01-bn1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751896AbbEFVgt (ORCPT ); Wed, 6 May 2015 17:36:49 -0400 Authentication-Results: spf=fail (sender IP is 192.88.168.50) smtp.mailfrom=freescale.com; freescale.mail.onmicrosoft.com; dkim=none (message not signed) header.d=none; From: "J. German Rivera" To: , , , CC: , , , , , , , , , "J. German Rivera" Subject: [PATCH v2 5/7] staging: fsl-mc: Allow the MC bus driver to run without GIC support Date: Wed, 6 May 2015 16:28:26 -0500 Message-ID: <1430947708-10521-6-git-send-email-German.Rivera@freescale.com> X-Mailer: git-send-email 2.3.3 In-Reply-To: <1430947708-10521-1-git-send-email-German.Rivera@freescale.com> References: <1430947708-10521-1-git-send-email-German.Rivera@freescale.com> X-EOPAttributedMessage: 0 X-Forefront-Antispam-Report: CIP:192.88.168.50;CTRY:US;IPV:NLI;EFV:NLI;SFV:NSPM;SFS:(10019020)(6009001)(339900001)(189002)(199003)(87936001)(50226001)(106466001)(2201001)(19580405001)(46102003)(36756003)(19580395003)(229853001)(47776003)(189998001)(50986999)(86362001)(50466002)(85426001)(76176999)(104016003)(48376002)(77096005)(62966003)(105606002)(5001960100002)(2950100001)(92566002)(77156002)(5001770100001)(107886002)(5001920100001)(6806004)(4001430100001);DIR:OUT;SFP:1102;SCL:1;SRVR:BY2PR03MB458;H:tx30smr01.am.freescale.net;FPR:;SPF:Fail;MLV:sfv;MX:1;A:1;LANG:en; MIME-Version: 1.0 Content-Type: text/plain X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:BY2PR03MB458; X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(601004)(5005006)(3002001);SRVR:BY2PR03MB458;BCL:0;PCL:0;RULEID:;SRVR:BY2PR03MB458; X-Forefront-PRVS: 0568F32D91 X-OriginatorOrg: freescale.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 06 May 2015 21:36:44.1906 (UTC) X-MS-Exchange-CrossTenant-Id: 710a03f5-10f6-4d38-9ff4-a80b81da590d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=710a03f5-10f6-4d38-9ff4-a80b81da590d;Ip=[192.88.168.50];Helo=[tx30smr01.am.freescale.net] X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: BY2PR03MB458 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 7071 Lines: 200 If the lpi-parent property is not present in the fsl,qoriq-mc node of the device tree, the MC bus driver will assume that the GIC is not supported. This change is made in order to be able to use the MC bus driver in a KVM VM, without having GIC-ITS support in guests. Added function fsl_mc_interrupts_supported(), which can be called from DPAA2 object drivers. Signed-off-by: J. German Rivera Reviewed-by: Stuart Yoder --- Changes in v2: none drivers/staging/fsl-mc/bus/dprc-driver.c | 31 +++++++++++++++++++---------- drivers/staging/fsl-mc/bus/mc-allocator.c | 4 ++++ drivers/staging/fsl-mc/bus/mc-bus.c | 16 +++++++++++++-- drivers/staging/fsl-mc/include/mc-private.h | 4 ++++ drivers/staging/fsl-mc/include/mc.h | 2 ++ 5 files changed, 45 insertions(+), 12 deletions(-) diff --git a/drivers/staging/fsl-mc/bus/dprc-driver.c b/drivers/staging/fsl-mc/bus/dprc-driver.c index 338fd7d..72eacee 100644 --- a/drivers/staging/fsl-mc/bus/dprc-driver.c +++ b/drivers/staging/fsl-mc/bus/dprc-driver.c @@ -323,6 +323,7 @@ static int dprc_scan_container(struct fsl_mc_device *mc_bus_dev) int error; unsigned int irq_count; struct fsl_mc_bus *mc_bus = to_fsl_mc_bus(mc_bus_dev); + struct fsl_mc *mc = dev_get_drvdata(fsl_mc_bus_type.dev_root->parent); dprc_init_all_resource_pools(mc_bus_dev); @@ -335,7 +336,7 @@ static int dprc_scan_container(struct fsl_mc_device *mc_bus_dev) if (error < 0) return error; - if (!mc_bus->irq_resources) { + if (mc->gic_supported && !mc_bus->irq_resources) { irq_count += FSL_MC_IRQ_POOL_MAX_EXTRA_IRQS; error = fsl_mc_populate_irq_pool(mc_bus, irq_count); if (error < 0) @@ -634,6 +635,7 @@ static int dprc_probe(struct fsl_mc_device *mc_dev) int error; size_t region_size; struct fsl_mc_bus *mc_bus = to_fsl_mc_bus(mc_dev); + struct fsl_mc *mc = dev_get_drvdata(fsl_mc_bus_type.dev_root->parent); if (WARN_ON(strcmp(mc_dev->obj_desc.type, "dprc") != 0)) return -EINVAL; @@ -672,19 +674,23 @@ static int dprc_probe(struct fsl_mc_device *mc_dev) if (error < 0) goto error_cleanup_open; - /* - * Configure interrupts for the DPRC object associated with this MC bus: - */ - error = dprc_setup_irqs(mc_dev); - if (error < 0) - goto error_cleanup_dprc_scan; + if (mc->gic_supported) { + /* + * Configure interrupts for the DPRC object associated with + * this MC bus: + */ + error = dprc_setup_irqs(mc_dev); + if (error < 0) + goto error_cleanup_dprc_scan; + } dev_info(&mc_dev->dev, "DPRC device bound to driver"); return 0; error_cleanup_dprc_scan: device_for_each_child(&mc_dev->dev, NULL, __fsl_mc_device_remove); - fsl_mc_cleanup_irq_pool(mc_bus); + if (mc->gic_supported) + fsl_mc_cleanup_irq_pool(mc_bus); error_cleanup_open: (void)dprc_close(mc_dev->mc_io, mc_dev->mc_handle); @@ -717,6 +723,7 @@ static int dprc_remove(struct fsl_mc_device *mc_dev) { int error; struct fsl_mc_bus *mc_bus = to_fsl_mc_bus(mc_dev); + struct fsl_mc *mc = dev_get_drvdata(fsl_mc_bus_type.dev_root->parent); if (WARN_ON(strcmp(mc_dev->obj_desc.type, "dprc") != 0)) return -EINVAL; @@ -726,13 +733,17 @@ static int dprc_remove(struct fsl_mc_device *mc_dev) if (WARN_ON(!mc_bus->irq_resources)) return -EINVAL; - dprc_teardown_irqs(mc_dev); + if (mc->gic_supported) + dprc_teardown_irqs(mc_dev); + device_for_each_child(&mc_dev->dev, NULL, __fsl_mc_device_remove); error = dprc_close(mc_dev->mc_io, mc_dev->mc_handle); if (error < 0) dev_err(&mc_dev->dev, "dprc_close() failed: %d\n", error); - fsl_mc_cleanup_irq_pool(mc_bus); + if (mc->gic_supported) + fsl_mc_cleanup_irq_pool(mc_bus); + dev_info(&mc_dev->dev, "DPRC device unbound from driver"); return 0; } diff --git a/drivers/staging/fsl-mc/bus/mc-allocator.c b/drivers/staging/fsl-mc/bus/mc-allocator.c index e445f79..3bdfefb 100644 --- a/drivers/staging/fsl-mc/bus/mc-allocator.c +++ b/drivers/staging/fsl-mc/bus/mc-allocator.c @@ -485,6 +485,10 @@ int __must_check fsl_mc_allocate_irqs(struct fsl_mc_device *mc_dev) struct fsl_mc_device_irq **irqs = NULL; struct fsl_mc_bus *mc_bus; struct fsl_mc_resource_pool *res_pool; + struct fsl_mc *mc = dev_get_drvdata(fsl_mc_bus_type.dev_root->parent); + + if (!mc->gic_supported) + return -ENOTSUPP; if (WARN_ON(mc_dev->irqs)) return -EINVAL; diff --git a/drivers/staging/fsl-mc/bus/mc-bus.c b/drivers/staging/fsl-mc/bus/mc-bus.c index b82fd7b..36116d7 100644 --- a/drivers/staging/fsl-mc/bus/mc-bus.c +++ b/drivers/staging/fsl-mc/bus/mc-bus.c @@ -287,6 +287,14 @@ void fsl_mc_driver_unregister(struct fsl_mc_driver *mc_driver) } EXPORT_SYMBOL_GPL(fsl_mc_driver_unregister); +bool fsl_mc_interrupts_supported(void) +{ + struct fsl_mc *mc = dev_get_drvdata(fsl_mc_bus_type.dev_root->parent); + + return mc->gic_supported; +} +EXPORT_SYMBOL_GPL(fsl_mc_interrupts_supported); + static int get_dprc_icid(struct fsl_mc_io *mc_io, int container_id, uint16_t *icid) { @@ -1048,8 +1056,12 @@ static int fsl_mc_bus_probe(struct platform_device *pdev) */ #ifdef GIC_ITS_MC_SUPPORT error = create_mc_irq_domain(pdev, &mc->irq_domain); - if (error < 0) - return error; + if (error < 0) { + dev_warn(&pdev->dev, + "WARNING: MC bus driver will run without interrupt support\n"); + } else { + mc->gic_supported = true; + } #endif /* diff --git a/drivers/staging/fsl-mc/include/mc-private.h b/drivers/staging/fsl-mc/include/mc-private.h index 67ba488..0757258 100644 --- a/drivers/staging/fsl-mc/include/mc-private.h +++ b/drivers/staging/fsl-mc/include/mc-private.h @@ -42,11 +42,15 @@ * struct fsl_mc - Private data of a "fsl,qoriq-mc" platform device * @root_mc_bus_dev: MC object device representing the root DPRC * @irq_domain: IRQ domain for the fsl-mc bus type + * @gic_supported: boolean flag that indicates if the GIC interrupt controller + * is supported. + * @num_translation_ranges: number of entries in addr_translation_ranges * @addr_translation_ranges: array of bus to system address translation ranges */ struct fsl_mc { struct fsl_mc_device *root_mc_bus_dev; struct irq_domain *irq_domain; + bool gic_supported; uint8_t num_translation_ranges; struct fsl_mc_addr_translation_range *translation_ranges; }; diff --git a/drivers/staging/fsl-mc/include/mc.h b/drivers/staging/fsl-mc/include/mc.h index 0a2f381d..c2a702e4 100644 --- a/drivers/staging/fsl-mc/include/mc.h +++ b/drivers/staging/fsl-mc/include/mc.h @@ -209,6 +209,8 @@ int __must_check __fsl_mc_driver_register(struct fsl_mc_driver *fsl_mc_driver, void fsl_mc_driver_unregister(struct fsl_mc_driver *driver); +bool fsl_mc_interrupts_supported(void); + int __must_check fsl_mc_portal_allocate(struct fsl_mc_device *mc_dev, uint16_t mc_io_flags, struct fsl_mc_io **new_mc_io); -- 2.3.3 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/