From patchwork Mon Nov 25 16:31:01 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Ciprian Marian Costea X-Patchwork-Id: 13885141 Received: from EUR05-VI1-obe.outbound.protection.outlook.com (mail-vi1eur05on2086.outbound.protection.outlook.com [40.107.21.86]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 41EF91B0F25 for ; Mon, 25 Nov 2024 16:31:17 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=40.107.21.86 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1732552279; cv=fail; b=OcAt5eQqhxOGXeengJbiek3M8nsT6Lx9gppfhQnC6ioov5BWYpvfAqUJserWpsRm4A5kAkXypRNBpd959L0Tfw/8pkNzzwF7upe2VYKDvw2tUgOLz2Rtlk462eHUF8nSQZ7yuoFCmlK8LKswKAEMPA/Kz2gNQ9cdHTpktqIRFVw= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1732552279; c=relaxed/simple; bh=R4cigYs3WvxA18SHZWDLkmPrTPBp80c5KL4PHc/KKdk=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: Content-Type:MIME-Version; b=oQkydnJJyF8Az70yV42fLUE+TDv8DC7Q9ka1omNNoOFGXt8tNqJhT6QDGfMEU5Wwh+9jTDaNWi9aAPxZQBvr/DBTSzxvJei9auZgELK4tdI0d876SafvOESlDpNaKvrkRyYc8eprPp55T94RXvOSeVNxJsTzgKY4/YqnIcL8dAI= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=oss.nxp.com; spf=pass smtp.mailfrom=oss.nxp.com; dkim=pass (2048-bit key) header.d=NXP1.onmicrosoft.com header.i=@NXP1.onmicrosoft.com header.b=Auqn4V5k; arc=fail smtp.client-ip=40.107.21.86 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=oss.nxp.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=oss.nxp.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=NXP1.onmicrosoft.com header.i=@NXP1.onmicrosoft.com header.b="Auqn4V5k" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=RdRrbehseQPpX3IMgXWFJNfBRKQ/3Zpxejth4zoWWHGw1K/1Y+P/M8wSzDPKMkFKm66jyF8bG+DBx1AiwUE8PupfSR5fVihn9JrSax8+4V+2JS384KO4dHRFCLrxFDZ1Efffh6+i49PQriGS1MrqEBHZv/dLjQKKsOjLikgwN7tlRlxEYG9+kiLaY/cqLp8EQZi2jBzVEwj1lFk0WLsVnT1I8NF9X5lc/DnZ/6GYrmGCmosdx9qo7fMBySrBK+Y3OoYYbVOewl2t9bsBVSjw4x1F3ltqfxOrdtfma2CiWfLsPs6kd7bexnaMrmY287sIJUYVWWxz/K6+r1QSa2yzNQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=wXKfRfRos/K6nkIpe7cPHuVxoJKUztfgDSE7u8Mi6Xo=; b=uL+4ifmrr2uVCfh2RzLi3g9gxObst/Wu4ij48zm11NPuN1T2X8biHMG9m67F22Uf+XsaGWoZQyXwxZD0EdJDw7G56JRWuuJhW3yBvwcKucd9F3V7sXmrvk2J7577MPARbeL1YOuQMmugA1JSaNbqBSJetThQLfnBeJwp48iXeRAmQfx7VmXuH6UzLeGJtVlFkB0a4VSJ+LDs67gfZxhfPkQ6hrh7HxP6WWdX0d3YK0gbs35D/dJ+JgcSnourvIJ7RtLNqgNQjjoUb5g4kSG6m4htBSGKZoxsEioJyPWWRF+IcbbMrxcT/Kg/d11neyXStWraf8He6mCYG469AdG8cQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=oss.nxp.com; dmarc=pass action=none header.from=oss.nxp.com; dkim=pass header.d=oss.nxp.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=NXP1.onmicrosoft.com; s=selector1-NXP1-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=wXKfRfRos/K6nkIpe7cPHuVxoJKUztfgDSE7u8Mi6Xo=; b=Auqn4V5kNj3ICmql9UOewiwPJc8uOVZtFdQivljCDVCz3WQ/VqolLkgVLQmuzJU5raq6ktJJ45HV5CvmESAhFaH7kH5zOuaII+1ecTvgrlZybjWmOSeY4taH7nG4LW0vV4JPOX7dUzwDb8Hst28nvB5kMxNiNo/rEiDo23a9Om4cxePbvA89uH7MNF8fphX9eWdvhvikhbRMsyE8WO0NOlOo1bj3AoSKpzj/dvbbGf62uNjfWGrYmiQ+3xx5PFao2JRG52DRSZquNdwN8dzE9yKmuR+hX9tkg6AR0TGCcMHXjBi4aGlIvjL77DVHfsehlB+O65N+HtlCxuz7YZ4oXA== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=oss.nxp.com; Received: from AS1PR04MB9261.eurprd04.prod.outlook.com (2603:10a6:20b:4c7::13) by DU0PR04MB9658.eurprd04.prod.outlook.com (2603:10a6:10:31f::22) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8182.19; Mon, 25 Nov 2024 16:31:11 +0000 Received: from AS1PR04MB9261.eurprd04.prod.outlook.com ([fe80::7b16:7596:6bb4:c5fb]) by AS1PR04MB9261.eurprd04.prod.outlook.com ([fe80::7b16:7596:6bb4:c5fb%4]) with mapi id 15.20.8182.019; Mon, 25 Nov 2024 16:31:11 +0000 From: Ciprian Costea To: Marc Kleine-Budde , Vincent Mailhol , Rob Herring , Krzysztof Kozlowski , Conor Dooley Cc: linux-can@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, NXP S32 Linux , imx@lists.linux.dev, Christophe Lizzi , Alberto Ruiz , Enric Balletbo , Ciprian Marian Costea Subject: [PATCH v2 2/3] can: flexcan: Add quirk to handle separate interrupt lines for mailboxes Date: Mon, 25 Nov 2024 18:31:01 +0200 Message-ID: <20241125163103.4166207-3-ciprianmarian.costea@oss.nxp.com> X-Mailer: git-send-email 2.45.2 In-Reply-To: <20241125163103.4166207-1-ciprianmarian.costea@oss.nxp.com> References: <20241125163103.4166207-1-ciprianmarian.costea@oss.nxp.com> X-ClientProxiedBy: BE1P281CA0039.DEUP281.PROD.OUTLOOK.COM (2603:10a6:b10:22::10) To AS1PR04MB9261.eurprd04.prod.outlook.com (2603:10a6:20b:4c7::13) Precedence: bulk X-Mailing-List: imx@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: AS1PR04MB9261:EE_|DU0PR04MB9658:EE_ X-MS-Office365-Filtering-Correlation-Id: ec47abe8-062c-4d91-8998-08dd0d6e928a X-MS-Exchange-SharedMailbox-RoutingAgent-Processed: True X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|7416014|376014|1800799024|366016; X-Microsoft-Antispam-Message-Info: =?utf-8?q?Sw1+3jw+y2A6ndGpXTAc6hdgapwLbjr?= =?utf-8?q?kJqE9zpzWb3a4sSdkbH7TqW6OHEukvcEwkOlQRRmhRTer2q2wMAWpX21R+dyCNmOp?= =?utf-8?q?atlQLOD8bK9wKlZ5JY9D4unRuytl65Aq/IINKaBD5VrsVKovsR31RqQffP3kTbmOj?= =?utf-8?q?ekr3mStjYdJS8Km1O60fpN0w/SfOdCpDkzBU5MFKIsbsEcTkX3FSQkIJbiAxhvWwZ?= =?utf-8?q?eDJk422pEXaGhatFeQTBvh9qb++0tNcDZz2CoxBb+aBbQx0ozQV8Lx86tFWrFbBYv?= =?utf-8?q?FtLX87LJugNb4FXesMNjsnx4u80jE7e7EpJHlGdJj6X5XiQMx1K5IdyXGrvaWHQ9D?= =?utf-8?q?/umQPbnrbzQ8EChDNDhpFm+t/4I5kwKxN5sL15XESL8dVg1xzyODq17Tc71RIR4wY?= =?utf-8?q?8gVyVIzaqvbYSRD7hj9zhHHOmv2TizhJKUCm1g39GSs2tvKXItIGD4Sejghmshqcb?= =?utf-8?q?i2t0+eoOBC7A2x0e1Rats6/IuVI2uQf4JosCSIcFq1SHrLCTR2Nzowxq/4t2UfspG?= =?utf-8?q?eNHH1OU9wgp700n2TrWJHbJ+WU7V1VAOsmsTvZZusq/3kEB+E/PoK7ziXsetF0RoF?= =?utf-8?q?+qAO6Bcnv0PnvN0RvCAUri9L1VodwMXV701dCIMVjbcyIPo1cz0K8BkAsKiAc8OR1?= =?utf-8?q?EQSFWxoN4hVSgSOJo7r33Ek9FzB3UNWvO+exDZ8XQY8CmEocWS0WyUmVU6PB4UBGG?= =?utf-8?q?aTvlAcuqy8QLflzY+kYQXGLBe8PGKeVqmOVjKUV3Mqi9NN1KrvJo/PfTDX6qRKNO0?= =?utf-8?q?EyMi7JpKx7MHPAlV8pxaZmMGk4eOGjcqE1/2WyhTCDe8HERazZu60RuXkL/+wSyhV?= =?utf-8?q?DHScBHrKglrVPBafQrovkayUeqd5dOhBnmCO+auhV2Vg/ilnLSqz2Fv0FQ0ulE0gm?= =?utf-8?q?8rHXOuEA2KMXsCg00NAV4z3A/wwonuWrxX/eRFTUAmWfTTmPkHAJomb186Zu2snNW?= =?utf-8?q?hfMDhPRKbSur5PlhBTtkox1uOuXHI+AEpc//sxex5Cgmf+10uDL0CP9fsut0WkOcR?= =?utf-8?q?3K5BSINT/JZV0LKmANMj+JzgJ/tcT9L4f/nuv6QqSG1cOktu/HxJ0IXO9OQY6oOCS?= =?utf-8?q?+s4JrAP/Z1LK5APQc8MND47XqzaGI3yWVQxpKEFNzHe+5JMFE1mKbhhoP1kCEGx1D?= =?utf-8?q?POIPvM5H7nr3VhId/IYhE4bfx1/Y7ZMcMTTuqQCWBKJwLXtQQXW9BYDlyUNhOXzsg?= =?utf-8?q?4osqulUvYj19kxGAq+QiUcofXJbHil2mo9fA+aRk4u1OIGOgn8ToyLbesbvg9yKsC?= =?utf-8?q?j/NLxaDa6GrU3NxnGwveobtqTza59v2Qkaw=3D=3D?= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:AS1PR04MB9261.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(7416014)(376014)(1800799024)(366016);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?q?RKc+7UEpfg54jtjiQQS33cHOKHB4?= =?utf-8?q?SaEV8xqAqq1kLHykB9zYD7DozUjgHoJmCFjExVgDgaYYvyvnhezVdMa+O+0X2Kvc2?= =?utf-8?q?oUc//PnwgMKs/q7Li8wVWEMCjm9fUoob+MmgVB/lGQmoEGSD+k/27KseCcgDpJQm5?= =?utf-8?q?hEmKRbF0m14zd6ONJ05iKKuD/35/Y9swqYLUJDKlvOXIXkImO2qKYpq+wM4sSIjjK?= =?utf-8?q?1mjAaJQbVzFBcPe6jS/UoVoIyiM+GZbmNmPgYUh3tbaq+jqxqJZn45NHWOULZUj8N?= =?utf-8?q?b5g/YEs6VAiwLaa6cUucS+VqX4J/BrSYlWcDmrpYbC0qSwYofqkGBIOse6leTVdxI?= =?utf-8?q?1kADgPzWvic8nObuARwz3vfrDucKBiy+pYSPTYR0C0SF7sE1fKT+NQPten7BmqXCc?= =?utf-8?q?P7c62Harf8R2/u7ioZ9pAh2v2vEozmcnLAZrHI7jkFy3cwlNGeweId1JXZseuuoB9?= =?utf-8?q?UmVRkY1pst9C5+cLb2bYRvB+YSySgn3x4fuhN5zlogwN/s+vqYTMY0jHGYIb4qE1c?= =?utf-8?q?lEsAqJXNXGvXVBxMTdCg3buPBO6HsDrCx41R2XVeE+yrWa47StpotQNmGKsBYpCuA?= =?utf-8?q?APz1J+P7OkfHG2CvuxhxXSQ4Ll7LrGwxX982pFxc6Fqi+xi8fwB1kgWb5zkaSCDNX?= =?utf-8?q?tZTXMCEESYSjRg9NMA0hHVVtHKEw/JhpxKoh9j8pf7UMLsn5VNt8wtKIBmDOo855m?= =?utf-8?q?OmZF+8lj57YydBwuHKh+jjS5k2sZ0FMQSoEJJ/d6+RHcDs/xazmJkk5+yCSjJEVTb?= =?utf-8?q?+LN5P2hEuBVKFcL5wkMWosjdM0BtCwU0sOLc7Iun8XUp0NieE0rVzhKbQ9PxLCrzh?= =?utf-8?q?nQs+3qMUOn/85wVHFUn5J219hyFxjFBgcoePLPXY0RUBUNqD9J25oqGrYmd+zi9SG?= =?utf-8?q?kCHdrfXB5MrHNtupTbgsVc/fxnd1/tc6E+BEaFWaICDFmyruXhoRNBCLo2bixjov6?= =?utf-8?q?fDGiqbgVuRRuTOa2o8OOZMTQ08AsC3Q5FvdKL0rGg0Iha26Ftrs5eu0fIabV9bMuP?= =?utf-8?q?tiWfBardSDTt8mtNQtqGMVbL6yCdkjk6bF+K67O8ROycoo6vEfh8vjerO/nMOR1QL?= =?utf-8?q?c3GEItbxVw2ATH9ixY3CznxJ0TuJ7fkfdGPKHF7T7F3EQdYNZqeh+5BCMlQnS4OGA?= =?utf-8?q?e5vItCqcn4pmsY/RB8p06SIpcZVmBzALdIR7Y6kniGbMoBR3zgdHVfpU8ieRn3e9q?= =?utf-8?q?MSzdFKqm6/VYK7evwPafUY0sNh0KWcpJKQuD75tJm56hwd4Kj4+mdazdkc8yjymDt?= =?utf-8?q?dscHRa/PAiG+P/G/S69G9LXCf/fVS2hDBJhVKrkxkcnmn9dHt//YtPmniy1BHNZo3?= =?utf-8?q?T0t14SDDCQR5deXI4aqwcGA1LDEcWu8LulOP274OGt8xLWPCnYjjx8f2ZN0O4qUzq?= =?utf-8?q?jTn1oAoq9rcFm+UD+b+o7toA+rmxuQrBHTChrezaXUEH2vIVFkHBXqwiPpe6LEfi9?= =?utf-8?q?KFUFfVgFA59x/Qp9GrK7YvYlRFDSWLMoChlSKTPerABnh9HshJ2tdLpPvyHLceb6b?= =?utf-8?q?v2FLJM0T4njbqJcMBDRwFVjuFchinpXRyw=3D=3D?= X-OriginatorOrg: oss.nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: ec47abe8-062c-4d91-8998-08dd0d6e928a X-MS-Exchange-CrossTenant-AuthSource: AS1PR04MB9261.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 25 Nov 2024 16:31:11.3662 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: hIR7VFoXvl8jR6/Dhm3JfXHJwpFdrBXI6y+DTlY5/junkz8U4ldqjvW6tO/IXWjrFLq9ol5bPH/vciK0q4VS4aojcUSZdTvfx2a6cVQOPQI= X-MS-Exchange-Transport-CrossTenantHeadersStamped: DU0PR04MB9658 From: Ciprian Marian Costea Introduce 'FLEXCAN_QUIRK_SECONDARY_MB_IRQ' quirk to handle a FlexCAN hardware module integration particularity where two ranges of mailboxes are controlled by separate hardware interrupt lines. The same 'flexcan_irq' handler is used for both separate mailbox interrupt lines, with no other changes. Signed-off-by: Ciprian Marian Costea Reviewed-by: Vincent Mailhol --- drivers/net/can/flexcan/flexcan-core.c | 24 +++++++++++++++++++++++- drivers/net/can/flexcan/flexcan.h | 5 +++++ 2 files changed, 28 insertions(+), 1 deletion(-) diff --git a/drivers/net/can/flexcan/flexcan-core.c b/drivers/net/can/flexcan/flexcan-core.c index ac1a860986df..7d7f3fa1d67b 100644 --- a/drivers/net/can/flexcan/flexcan-core.c +++ b/drivers/net/can/flexcan/flexcan-core.c @@ -1762,14 +1762,25 @@ static int flexcan_open(struct net_device *dev) goto out_free_irq_boff; } + if (priv->devtype_data.quirks & FLEXCAN_QUIRK_SECONDARY_MB_IRQ) { + err = request_irq(priv->irq_secondary_mb, + flexcan_irq, IRQF_SHARED, dev->name, dev); + if (err) + goto out_free_irq_err; + } + flexcan_chip_interrupts_enable(dev); netif_start_queue(dev); return 0; + out_free_irq_err: + if (priv->devtype_data.quirks & FLEXCAN_QUIRK_NR_IRQ_3) + free_irq(priv->irq_err, dev); out_free_irq_boff: - free_irq(priv->irq_boff, dev); + if (priv->devtype_data.quirks & FLEXCAN_QUIRK_NR_IRQ_3) + free_irq(priv->irq_boff, dev); out_free_irq: free_irq(dev->irq, dev); out_can_rx_offload_disable: @@ -1799,6 +1810,9 @@ static int flexcan_close(struct net_device *dev) free_irq(priv->irq_boff, dev); } + if (priv->devtype_data.quirks & FLEXCAN_QUIRK_SECONDARY_MB_IRQ) + free_irq(priv->irq_secondary_mb, dev); + free_irq(dev->irq, dev); can_rx_offload_disable(&priv->offload); flexcan_chip_stop_disable_on_error(dev); @@ -2187,6 +2201,14 @@ static int flexcan_probe(struct platform_device *pdev) } } + if (priv->devtype_data.quirks & FLEXCAN_QUIRK_SECONDARY_MB_IRQ) { + priv->irq_secondary_mb = platform_get_irq(pdev, 3); + if (priv->irq_secondary_mb < 0) { + err = priv->irq_secondary_mb; + goto failed_platform_get_irq; + } + } + if (priv->devtype_data.quirks & FLEXCAN_QUIRK_SUPPORT_FD) { priv->can.ctrlmode_supported |= CAN_CTRLMODE_FD | CAN_CTRLMODE_FD_NON_ISO; diff --git a/drivers/net/can/flexcan/flexcan.h b/drivers/net/can/flexcan/flexcan.h index 4933d8c7439e..2cf886618c96 100644 --- a/drivers/net/can/flexcan/flexcan.h +++ b/drivers/net/can/flexcan/flexcan.h @@ -70,6 +70,10 @@ #define FLEXCAN_QUIRK_SUPPORT_RX_FIFO BIT(16) /* Setup stop mode with ATF SCMI protocol to support wakeup */ #define FLEXCAN_QUIRK_SETUP_STOP_MODE_SCMI BIT(17) +/* Device has two separate interrupt lines for two mailbox ranges, which + * both need to have an interrupt handler registered. + */ +#define FLEXCAN_QUIRK_SECONDARY_MB_IRQ BIT(18) struct flexcan_devtype_data { u32 quirks; /* quirks needed for different IP cores */ @@ -107,6 +111,7 @@ struct flexcan_priv { int irq_boff; int irq_err; + int irq_secondary_mb; /* IPC handle when setup stop mode by System Controller firmware(scfw) */ struct imx_sc_ipc *sc_ipc_handle;