From patchwork Thu Aug 29 09:37:18 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Carlos Song X-Patchwork-Id: 823668 Received: from AS8PR04CU009.outbound.protection.outlook.com (mail-westeuropeazon11011055.outbound.protection.outlook.com [52.101.70.55]) (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 D746818DF79; Thu, 29 Aug 2024 09:28:30 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=52.101.70.55 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1724923713; cv=fail; b=YxH1CREJRyPaq5JPCczQH4IF/Nogdvb6EPuRH1kSk6rkA59aDdSvacr/MRF5Uj6+7AiNVDzIkdWIx74EinVfgSlT8K/uqvuZCAoJOuWFiUQZ+aaEMr3V7I1354GhYMHVMj50qwpwwRXPCpd3yVoQPHboQaxdVNZssFy8UelkzXU= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1724923713; c=relaxed/simple; bh=PfPF3qu2MyWZcOqml4AIx9yGeoWD4A8i7ckzWz2aiws=; h=From:To:Cc:Subject:Date:Message-Id:Content-Type:MIME-Version; b=D9d7FxzBBRzFPIXBUaxr3d48qi6rJXZoRWvaGOearPxO45WSghuJhHr8DucNzkCm8jgEoDdMmNoiV7rXugvkAEO8bUCopxQUX01+6QMD7+F01YeVnvvSVzIzf/sn/2KcBx1Afd+5PZMyw3jsKQkSg36xDeS45xNWNK702fiHDZI= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=nxp.com; spf=pass smtp.mailfrom=nxp.com; dkim=pass (2048-bit key) header.d=nxp.com header.i=@nxp.com header.b=DW64/aEg; arc=fail smtp.client-ip=52.101.70.55 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=nxp.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=nxp.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=nxp.com header.i=@nxp.com header.b="DW64/aEg" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=ofdzapnLc4TcKoYNEH8yQKX+eZSShsjzW5iYO4ymn88IxHGbHW94G8jZixumoqxS3ZjOimvgGP26B92unngug6yZvjCJ14h7zwLQ6BhHY2Yv2QIEqFhipoi6zcUBZrUSonxr9CLz/OUo566buCZpkrK9RSFoOGeIW4qV8gz6LWSMJus8hfCzALoD1CTSV4mwVL31v82lI+gy8wJ2sGrkWESU0MiiX78uPDI//1LE/sOuSvb13vQmP3wxQ/BTS7Wrt2J83HZ5n8SideLA8RG6qgsts3fXItdEhbppfmzk25DGe4CNXqfTEaeUWrUXGDv5+wejz1TPs/TdkluXaHZCiQ== 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=J4YfDm6yAAeBub8EpRX/rvJadtEFi73fGZWC4CpU3D4=; b=Nfpmh5oKIjZJEIqxs529w8qFdtYDUdurYpsYCYB2tc59rwyeloUHYnYUwsmYcCABvy3DNtA03QuVompfxdoWEvTBaZvGVGqmLQe7icqYuojVPN9jrp0UTUdZNJQz2EdTlgu0Q6BhJzzwrASiVXRTSrLpaxot6PHseIwdviwY2d5gDGzSt/eYwoRLrX2vcgfUXFOBBYN3Jct1SkwasQnhQ9TPDYRqnEG3pY8DGq2nO0RIGlXnF5h9rFXEbR9KaLyMFL0v32ggy+ggl0cJhdHo6noh5rkGCbNG5jO6uxKSt+Mfcd9j//mzc7wOXm4+IpwYp28NJtuQ3iwl6Q6weSsoJg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nxp.com; dmarc=pass action=none header.from=nxp.com; dkim=pass header.d=nxp.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=J4YfDm6yAAeBub8EpRX/rvJadtEFi73fGZWC4CpU3D4=; b=DW64/aEgkEGhyY/6vuHynjTCfpPcDLAvPcu5he55ZPRCaj109Ht2QzNcqUaoxvpz/qwtbRnWVTdAzYWXkdIw+0as2bsHF/5H8jncGWZQXGF+eJfF8ON/DbwSRdhLJlfYMJNv0AIswBcfvBNk0jHZ6Y0qAVRFGi7b6V81PqKip0rbP4ddosAn14Zxs8J9uKgTekpDWb6CZt4Bn2pvgVzkXLr6K+i8ugHCfXHGuPl02ELGtUWsxpoCiMFWYm1vrU66X9TuaeFi1w6lrq3um8WIA2OPIay18CgtkYvPSzP+4L0ykX6wf+7eJLqX0s/kX1FtgEJTi1Jx/rGTLZSDJIHARw== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=nxp.com; Received: from VI1PR04MB5005.eurprd04.prod.outlook.com (2603:10a6:803:57::30) by DB9PR04MB8364.eurprd04.prod.outlook.com (2603:10a6:10:24c::17) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7918.20; Thu, 29 Aug 2024 09:28:27 +0000 Received: from VI1PR04MB5005.eurprd04.prod.outlook.com ([fe80::3a57:67cf:6ee0:5ddb]) by VI1PR04MB5005.eurprd04.prod.outlook.com ([fe80::3a57:67cf:6ee0:5ddb%4]) with mapi id 15.20.7897.021; Thu, 29 Aug 2024 09:28:26 +0000 From: carlos.song@nxp.com To: aisheng.dong@nxp.com, andi.shyti@kernel.org, shawnguo@kernel.org, s.hauer@pengutronix.de, kernel@pengutronix.de, festevam@gmail.com Cc: linux-i2c@vger.kernel.org, imx@lists.linux.dev, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: [PATCH 1/5] i2c: imx-lpi2c: add target mode support Date: Thu, 29 Aug 2024 17:37:18 +0800 Message-Id: <20240829093722.2714921-1-carlos.song@nxp.com> X-Mailer: git-send-email 2.34.1 X-ClientProxiedBy: AS4PR10CA0023.EURPRD10.PROD.OUTLOOK.COM (2603:10a6:20b:5d8::15) To VI1PR04MB5005.eurprd04.prod.outlook.com (2603:10a6:803:57::30) Precedence: bulk X-Mailing-List: linux-i2c@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: VI1PR04MB5005:EE_|DB9PR04MB8364:EE_ X-MS-Office365-Filtering-Correlation-Id: acf33faf-184e-4221-29bb-08dcc80cef87 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; ARA:13230040|1800799024|376014|52116014|366016|38350700014; X-Microsoft-Antispam-Message-Info: IT8tLIQH1tyMdxB5I6jNl+M71VHz+bN84f1/xUxiMDZJla2nLN7FGchFmKDkHRpJjoW0CUOyDfcUH+0wa7CDY+qojg0p8zUbd1cD+ShADQjMAdKM6Pb7qN4EJA8Z4iH9whKzU84RQzCkiuXSXsauGDjT3K9/2NqH3XGCsZ4h89yTtFF44TDfzh92ExGctVokb24OD4erCSrfbGIFbz2iTVn50Vuj2+sPSUSDgvbrfVTVCapAOQV+QWhU4upNAp6HJWZDHzC0Hx+wis+EHydgW/RsQLrESNd6L/UJCpYe9/SZISrfNeifSYHHet6xA2GkUELveYKI+/6w+l9eYAlCqTrIREmacfZpALm8fNNfS2jouHIehYkkzHS080b1qb4DzU6FO0wU6i2tyv0hkGc+SOgC1TYtQlj+wqYFG7/NPym2JHjj1ugjLyCNLZaoOV4nloKHTvafKS3n0SDL92bViz7lTdc8cSGX1IaGFVDaFivk7VWlZ7xULwXKhDnNqqtmPinG3pXILbsjcmG9augVwMivE2OPe6Nfj4EAX3i87cu9l2Hy9HLQB4kAZB8uRJk78WlDO8rUw2PnezQdznU2TP516WjN/2U4LvgTxVFM8f+sUeYMXh9jMO6oyK1phy+yEuJIHQMmKDHr2MKXArHPgiTyZ2d3Dm357IGV9RQeAoT6rAeDGAh8jrYM9GkCwtUhyeGVzGMistql2SUVNeUVVvgP3kPWuX/xJ5Gu4twF3oO9VMiWYHZvydph2VleJbPyt35vYpfLPDmRLmXcSl19FYTKWhtnWkfVb4aI/Yp56mIhO7qpUEgHuB6ezBwDI8CSqMLVMsO1gAm94xucM9bz9IQ8vgKmzpfyoOmCkcG0ZQjNa52d64PM1UKrnAE6lCBuC+C/NYSwdflob4BRweDiTFBGgxCARLGMZG1o8ZorLi2pImsGFNOlWR2Tg4zD/IOp8M/+FAmSfQM9vN5C/A9zPSvZMGqzcBWT1rnQydlr2khD89LOuSDmaPI/mjbGmhC1W2WjGVAVDWyAs5IsEDAhgkG0NHwQtlbS5aFU8pPu7DRjBeBwX7Znu6vqOmM6LeUv34BUHgWemT8AoCV0KBsEq3OOdA428vjoCFVO53aNTY1AtAMK71LkSEBhcxnopbEEVA9Y5kgrjPFC1Mfas17wz0OjJLSu4uvBzswV9V+UdAp+SljI6B++wNHIT+1jYju09cDOvSrA8Csi4MaVMNf3D9bJlSMklBXKYRU8hQ91zUe98+XcZZXXJHt4JWHrccQ2uYHttZDKQC6ZIw1/Dlbet1Sg4ZQI7cS5g9ylN5tEvxa8ua4mXOVWvp7w/OZ7A2Zkq64LtVlLFHbj0CvQxK+I59h8JJLgIpMLBw0M2x0m68XaBT0v60mP5nNAVOwq84xfzIp35bDqdi+9Ch93HCjtkw== X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:VI1PR04MB5005.eurprd04.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230040)(1800799024)(376014)(52116014)(366016)(38350700014); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: eS8vEmFLQmsyF0nzP7gP5+62t2ZKD1VNCXyrJs0LFqhgO41vT93FtG5NfVgShlWtN549ZQgOeHcg9GdEnrA15VfNLW3l46zUKwLrKXg8EFC/X7gVJtMQkNGV6gg2q9UKo47nBqFk/+edYNlVumhEPCB9o6/HL+emF5fXkaDGkgmx7JhgJIyLpzJexytpW5VE+pFfNsR5/UGAX4bQfte90oOkpp0WM7oR8ovJQE9UNWaUby4UpP9sLTnWP8O2wKOJnIaEgeulsVUaIFeGJWxY/8UReV4+PuMk2pIOxmVNy61LvsF4vNeXAwqI7TWTwHCcfEjUjqEIcPNlvoH9zlpsal9E0g4BlsLIXkEMFTGWV4f4um3Egzq/2+GJrIV3MSyUJXt0ziXc5OQlYFlrKsMBKor3FCczv1RfmIN29BMmu5QEaIw0AA3PUvtZiA0uy4jk3rjWXZhSMSS1g7PvfKfzkxVSicy7w3/gY6ooEoM1dWJagg0xRtdCqSF3ZVnmtVpOG5vaeHqhOuRyxnu8SzQ5cEDzrOyc2cDIDS30HVaBXEVZcqRGdYXiuvF+4lki0qZ6KUZjcpQGE5p4C1h9wzdMSMtl9jdujpEcXYVmt+MAlwowyAXXb2DYY4LhJVpKGMC2QuoqsIo3gdO2yEEVB5vjBfkA3r0hsx/i8v3AAFnrRIW8RRSGMpkIJ5l7mZHrIVJVgmyXjRG26C6GVlDeTZgwTyrTSIQFfORcBn8YBlo1sokzizqQ44+4VNtnvtc9TECrvkDveaQMiKN9pnLEuspLKRB1AetRMnvUWYtiItweY3LAdjDl3q8RhjJc3FXbEz9ACsqzEs632YdyY92smjpcH1boviULIWWsYI8G3+d75TQ/EyrGeDEz0uQ6Se4/3JkZ8df4TI3qBkYIPouqwYdxRK0IdZsJeS/vE3L1xHH04fee9XwII7EDjlqOeNfOLfGtWlpiXRbe0SiJAIc+/2LJzJIoARIXZGEfOlsclGMlQeEF/MS7lDY8nGkubcY4XzofC/ZQoVtdym4z3Y8mE0gQXbunf/3f/DtNc+mdT8s8mm1OFqY8kmP8C0yeRfPo0t4bwwouS9Af3ijNIho/5ZOPe8M9C4HQJ19RARR7rxSmXDjVZXKX9nYto90YKErJF0fk8LyiUao8xU4U8z95YWBZY61rQVPCegtl6XCbikmoD5nblBZUuDCO/2RsQM/mEN+JjdE8xWHg+EfkAtLmF3t7jafFDwD7LuSa1rgYK5Swq6JD3cO+UM7RD3gE/uLr3onZJRkpKYWCjazZ+59UZrcIznbX+Za4xRNlhLvgRQTM5qQxhLLD55i/2zgGlVx5gB0G6xXuZL1qG1DWXKfs3mydCePEzuqHo2nJCDC3T/5i3Vmkhh8KkE/SRIU5Zw0RngkWYkMyUxlbGulQXQeRZEAloaE3FtMR2nAfF9YaUZW3c4yOe0wMn20Zu1oSrbKv9Rjy2adGBQDIz0PiEO9q122w/Xp2qDG3cjA0umbCJ6ixkjS4t4IL8eB7c7ijmFf/Ko0BGL6vLEN6Y5llv/7qDcJWbACNNBGtNB2m/BhzH5VinlPvu/5PepBAaKaX8pHWQHRy X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: acf33faf-184e-4221-29bb-08dcc80cef87 X-MS-Exchange-CrossTenant-AuthSource: VI1PR04MB5005.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 29 Aug 2024 09:28:26.9180 (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: ICtqpYR+yPPmOjlUORXaiep8OL6341G0ISIWKtX/9IWZHiifm98nynLYRQSc/e7bXmGTx4nsW1IePhEJVSZJCA== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB9PR04MB8364 From: Carlos Song Add target mode support for LPI2C. Signed-off-by: Carlos Song --- drivers/i2c/busses/i2c-imx-lpi2c.c | 253 ++++++++++++++++++++++++++++- 1 file changed, 249 insertions(+), 4 deletions(-) diff --git a/drivers/i2c/busses/i2c-imx-lpi2c.c b/drivers/i2c/busses/i2c-imx-lpi2c.c index 530ca5d76403..c8f3dadfb36e 100644 --- a/drivers/i2c/busses/i2c-imx-lpi2c.c +++ b/drivers/i2c/busses/i2c-imx-lpi2c.c @@ -43,6 +43,20 @@ #define LPI2C_MTDR 0x60 /* i2c master TX data register */ #define LPI2C_MRDR 0x70 /* i2c master RX data register */ +#define LPI2C_SCR 0x110 /* i2c target contrl register */ +#define LPI2C_SSR 0x114 /* i2c target status register */ +#define LPI2C_SIER 0x118 /* i2c target interrupt enable */ +#define LPI2C_SDER 0x11C /* i2c target DMA enable */ +#define LPI2C_SCFGR0 0x120 /* i2c target configuration */ +#define LPI2C_SCFGR1 0x124 /* i2c target configuration */ +#define LPI2C_SCFGR2 0x128 /* i2c target configuration */ +#define LPI2C_SAMR 0x140 /* i2c target address match */ +#define LPI2C_SASR 0x150 /* i2c target address status */ +#define LPI2C_STAR 0x154 /* i2c target transmit ACK */ +#define LPI2C_STDR 0x160 /* i2c target transmit data */ +#define LPI2C_SRDR 0x170 /* i2c target receive data */ +#define LPI2C_SRDROR 0x178 /* i2c target receive data read only */ + /* i2c command */ #define TRAN_DATA 0X00 #define RECV_DATA 0X01 @@ -76,6 +90,42 @@ #define MDER_TDDE BIT(0) #define MDER_RDDE BIT(1) +#define SCR_SEN BIT(0) +#define SCR_RST BIT(1) +#define SCR_FILTEN BIT(4) +#define SCR_RTF BIT(8) +#define SCR_RRF BIT(9) +#define SCFGR1_RXSTALL BIT(1) +#define SCFGR1_TXDSTALL BIT(2) +#define SCFGR2_FILTSDA_SHIFT 24 +#define SCFGR2_FILTSCL_SHIFT 16 +#define SCFGR2_CLKHOLD(x) (x) +#define SCFGR2_FILTSDA(x) ((x) << SCFGR2_FILTSDA_SHIFT) +#define SCFGR2_FILTSCL(x) ((x) << SCFGR2_FILTSCL_SHIFT) +#define SSR_TDF BIT(0) +#define SSR_RDF BIT(1) +#define SSR_AVF BIT(2) +#define SSR_TAF BIT(3) +#define SSR_RSF BIT(8) +#define SSR_SDF BIT(9) +#define SSR_BEF BIT(10) +#define SSR_FEF BIT(11) +#define SSR_SBF BIT(24) +#define SSR_BBF BIT(25) +#define SSR_CLEAR_BITS (SSR_RSF | SSR_SDF | SSR_BEF | SSR_FEF) +#define SIER_TDIE BIT(0) +#define SIER_RDIE BIT(1) +#define SIER_AVIE BIT(2) +#define SIER_TAIE BIT(3) +#define SIER_RSIE BIT(8) +#define SIER_SDIE BIT(9) +#define SIER_BEIE BIT(10) +#define SIER_FEIE BIT(11) +#define SIER_AM0F BIT(12) +#define SASR_READ_REQ 0x1 +#define SLAVE_INT_FLAG (SIER_TDIE | SIER_RDIE | SIER_AVIE | \ + SIER_SDIE | SIER_BEIE) + #define I2C_CLK_RATIO 2 #define CHUNK_DATA 256 @@ -133,6 +183,7 @@ struct lpi2c_imx_struct { struct i2c_bus_recovery_info rinfo; bool can_use_dma; struct lpi2c_imx_dma *dma; + struct i2c_client *target; }; static void lpi2c_imx_intctrl(struct lpi2c_imx_struct *lpi2c_imx, @@ -952,9 +1003,57 @@ static int lpi2c_imx_xfer(struct i2c_adapter *adapter, return (result < 0) ? result : num; } -static irqreturn_t lpi2c_imx_isr(int irq, void *dev_id) +static irqreturn_t lpi2c_imx_target_isr(struct lpi2c_imx_struct *lpi2c_imx, + u32 ssr, u32 sier_filter) +{ + u8 value; + u32 sasr; + + /* Arbitration lost */ + if (sier_filter & SSR_BEF) { + writel(0, lpi2c_imx->base + LPI2C_SIER); + return IRQ_HANDLED; + } + + /* Address detected */ + if (sier_filter & SSR_AVF) { + sasr = readl(lpi2c_imx->base + LPI2C_SASR); + if (SASR_READ_REQ & sasr) { + /* Read request */ + i2c_slave_event(lpi2c_imx->target, I2C_SLAVE_READ_REQUESTED, &value); + writel(value, lpi2c_imx->base + LPI2C_STDR); + goto ret; + } else { + /* Write request */ + i2c_slave_event(lpi2c_imx->target, I2C_SLAVE_WRITE_REQUESTED, &value); + } + } + + if (sier_filter & SSR_SDF) { + /* STOP */ + i2c_slave_event(lpi2c_imx->target, I2C_SLAVE_STOP, &value); + } + + if (sier_filter & SSR_TDF) { + /* Target send data */ + i2c_slave_event(lpi2c_imx->target, I2C_SLAVE_READ_PROCESSED, &value); + writel(value, lpi2c_imx->base + LPI2C_STDR); + } + + if (sier_filter & SSR_RDF) { + /* Target receive data */ + value = readl(lpi2c_imx->base + LPI2C_SRDR); + i2c_slave_event(lpi2c_imx->target, I2C_SLAVE_WRITE_RECEIVED, &value); + } + +ret: + /* Clear SSR */ + writel(ssr & SSR_CLEAR_BITS, lpi2c_imx->base + LPI2C_SSR); + return IRQ_HANDLED; +} + +static irqreturn_t lpi2c_imx_master_isr(struct lpi2c_imx_struct *lpi2c_imx) { - struct lpi2c_imx_struct *lpi2c_imx = dev_id; unsigned int enabled; unsigned int temp; @@ -974,6 +1073,119 @@ static irqreturn_t lpi2c_imx_isr(int irq, void *dev_id) return IRQ_HANDLED; } +static irqreturn_t lpi2c_imx_isr(int irq, void *dev_id) +{ + struct lpi2c_imx_struct *lpi2c_imx = dev_id; + u32 ssr, sier_filter; + unsigned int scr; + + if (lpi2c_imx->target) { + scr = readl(lpi2c_imx->base + LPI2C_SCR); + ssr = readl(lpi2c_imx->base + LPI2C_SSR); + sier_filter = ssr & readl(lpi2c_imx->base + LPI2C_SIER); + if ((scr & SCR_SEN) && sier_filter) + return lpi2c_imx_target_isr(lpi2c_imx, ssr, sier_filter); + else + return lpi2c_imx_master_isr(lpi2c_imx); + } else { + return lpi2c_imx_master_isr(lpi2c_imx); + } +} + +static void lpi2c_imx_target_init(struct lpi2c_imx_struct *lpi2c_imx) +{ + int temp; + + /* reset target module */ + writel(SCR_RST, lpi2c_imx->base + LPI2C_SCR); + writel(0, lpi2c_imx->base + LPI2C_SCR); + + /* Set target addr */ + writel((lpi2c_imx->target->addr << 1), lpi2c_imx->base + LPI2C_SAMR); + + writel(SCFGR1_RXSTALL | SCFGR1_TXDSTALL, lpi2c_imx->base + LPI2C_SCFGR1); + + /* + * set SCFGR2: FILTSDA, FILTSCL and CLKHOLD + * + * FILTSCL/FILTSDA can eliminate signal skew. It should generally be + * set to the same value and should be set >= 50ns. + * + * CLKHOLD is only used when clock stretching is enabled, but it will + * extend the clock stretching to ensure there is an additional delay + * between the target driving SDA and the target releasing the SCL pin. + * + * CLKHOLD setting is crucial for lpi2c target. When master read data + * from target, if there is a delay caused by cpu idle, excessive load, + * or other delays between two bytes in one message transmission. so it + * will cause a short interval time between the driving SDA signal and + * releasing SCL signal. Lpi2c master will mistakenly think it is a stop + * signal resulting in an arbitration failure. This issue can be avoided + * by setting CLKHOLD. + * + * In order to ensure lpi2c function normally when the lpi2c speed is as + * low as 100kHz, CLKHOLD should be set 3 and it is also compatible with + * higher clock frequency like 400kHz and 1MHz. + */ + temp = SCFGR2_FILTSDA(2) | SCFGR2_FILTSCL(2) | SCFGR2_CLKHOLD(3); + writel(temp, lpi2c_imx->base + LPI2C_SCFGR2); + + /* + * Enable module: + * SCR_FILTEN can enable digital filter and output delay counter for LPI2C + * target mode. So SCR_FILTEN need be asserted when enable SDA/SCL FILTER + * and CLKHOLD. + */ + writel(SCR_SEN | SCR_FILTEN, lpi2c_imx->base + LPI2C_SCR); + + /* Enable interrupt from i2c module */ + writel(SLAVE_INT_FLAG, lpi2c_imx->base + LPI2C_SIER); +} + +static int lpi2c_imx_reg_target(struct i2c_client *client) +{ + struct lpi2c_imx_struct *lpi2c_imx = i2c_get_adapdata(client->adapter); + int ret; + + if (lpi2c_imx->target) + return -EBUSY; + + lpi2c_imx->target = client; + + ret = pm_runtime_resume_and_get(lpi2c_imx->adapter.dev.parent); + if (ret < 0) { + dev_err(&lpi2c_imx->adapter.dev, "failed to resume i2c controller"); + return ret; + } + + lpi2c_imx_target_init(lpi2c_imx); + + return 0; +} + +static int lpi2c_imx_unreg_target(struct i2c_client *client) +{ + struct lpi2c_imx_struct *lpi2c_imx = i2c_get_adapdata(client->adapter); + int ret; + + if (!lpi2c_imx->target) + return -EINVAL; + + /* Reset target address. */ + writel(0, lpi2c_imx->base + LPI2C_SAMR); + + writel(SCR_RST, lpi2c_imx->base + LPI2C_SCR); + writel(0, lpi2c_imx->base + LPI2C_SCR); + + lpi2c_imx->target = NULL; + + ret = pm_runtime_put_sync(lpi2c_imx->adapter.dev.parent); + if (ret < 0) + dev_err(&lpi2c_imx->adapter.dev, "failed to suspend i2c controller"); + + return ret; +} + static int lpi2c_imx_init_recovery_info(struct lpi2c_imx_struct *lpi2c_imx, struct platform_device *pdev) { @@ -1049,6 +1261,8 @@ static u32 lpi2c_imx_func(struct i2c_adapter *adapter) static const struct i2c_algorithm lpi2c_imx_algo = { .master_xfer = lpi2c_imx_xfer, .functionality = lpi2c_imx_func, + .reg_slave = lpi2c_imx_reg_target, + .unreg_slave = lpi2c_imx_unreg_target, }; static const struct of_device_id lpi2c_imx_of_match[] = { @@ -1199,9 +1413,40 @@ static int __maybe_unused lpi2c_runtime_resume(struct device *dev) return 0; } +static int lpi2c_suspend_noirq(struct device *dev) +{ + struct lpi2c_imx_struct *lpi2c_imx = dev_get_drvdata(dev); + int ret; + + ret = pm_runtime_force_suspend(dev); + if (ret) + return ret; + + return 0; +} + +static int lpi2c_resume_noirq(struct device *dev) +{ + struct lpi2c_imx_struct *lpi2c_imx = dev_get_drvdata(dev); + int ret; + + ret = pm_runtime_force_resume(dev); + if (ret) + return ret; + + /* + * If i2c module powered down in system suspend, register + * value will lose. So reinit target when system resume. + */ + if (lpi2c_imx->target) + lpi2c_imx_target_init(lpi2c_imx); + + return 0; +} + static const struct dev_pm_ops lpi2c_pm_ops = { - SET_NOIRQ_SYSTEM_SLEEP_PM_OPS(pm_runtime_force_suspend, - pm_runtime_force_resume) + SET_NOIRQ_SYSTEM_SLEEP_PM_OPS(lpi2c_suspend_noirq, + lpi2c_resume_noirq) SET_RUNTIME_PM_OPS(lpi2c_runtime_suspend, lpi2c_runtime_resume, NULL) }; From patchwork Thu Aug 29 09:37:20 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Carlos Song X-Patchwork-Id: 823667 Received: from AM0PR83CU005.outbound.protection.outlook.com (mail-westeuropeazon11010023.outbound.protection.outlook.com [52.101.69.23]) (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 C735F18E05C; Thu, 29 Aug 2024 09:28:39 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=52.101.69.23 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1724923721; cv=fail; b=ZbuumyyHQYrgO8bRE7w4fQmV3/PyYZK3QY/nAPStMVt94dLnOksVFaDPidDgQtjijn82DhGt/syI5vfHb6B81OqLRi8DNLCJTGrIFCJNtaRNLIC6UWO4QwhT920/jfUjQJC7mP9iRBG4Gsd0Woxl/ZTu1Na8weyYhsLEAbBC0RY= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1724923721; c=relaxed/simple; bh=mhgQ5OmbHC6kPoFxwNzvH8P7CCMbvdoAf/q7al2TaUk=; h=From:To:Cc:Subject:Date:Message-Id:In-Reply-To:References: Content-Type:MIME-Version; b=DIafHb7dho++F467KsU5Wv9qrTVHziYWVkQ4L+2VtTrssCAhI9q7+k+96G2q5MkVAdzkIJ0DlIATu+IS2hvMIkXJDNvbU9StbTTAT84GbKISxMuRLs4v2V4IeE7rZs2iMtIFg1UN0346g08gY+U5B0h4geBuUOz/brr8ur3z9HY= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=nxp.com; spf=pass smtp.mailfrom=nxp.com; dkim=pass (2048-bit key) header.d=nxp.com header.i=@nxp.com header.b=Kr5RGD+A; arc=fail smtp.client-ip=52.101.69.23 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=nxp.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=nxp.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=nxp.com header.i=@nxp.com header.b="Kr5RGD+A" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=mgM9DXuBs9C+CjaVMNE3632rELAXy/R6sed+pzhiqnVyjMCrQTIwrg5kuDzhQD+JYtwb8cVZpiTkf/TpYnQ46FYgxKwyBrbhcKXZn3s2l1KE5+34FREJGkKHmUoxeB3pqh8+IQ6wG9DSxU6rMS04k3bBbALN80QNA2ojYpGHonR1r6SFgKcktYpLECJT6breq4HqMzzRP4/LoxD3RvZNT7p+iXQAfSRP5rFFd9E/REaNpWrZN7qqdrTtsz0d0WL/Bsu6joIYg+cPmu7AjCjvttTX3ib2co9O+DWLLOxxgd5IA38DTdYz2+P6UzFytajpX0K6ZjkA542ngT6HHKMhjg== 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=F5FxjuNdReSAUurDwXC3fYfUz5iT8OSKubT84fjG1pI=; b=H8d+05bHz22vCFzv54BYR0tll9WJkCxb3+1lI7X8oejP+rfsoLZdTc1wWOOU3eC1tlLfcHU9Dj98ULPiM16xhw7JXYnPRR0y6Yx1qMH5B4S6j6zjr5vaoM8XwuvlfVLfvAnDFX3YobH7875egJ5D48sbzrrp2ZXP5fIpp0CpJR8luLw0lpkJB9JtuWIKtVNOCldlzq1bOvJYkDY5NcC7uGjUU2K0zFj/sJpPDjpQVemhfjVRTId4DTv0w8GR1d/cv64w37cUa9YB+lRG9LFZFfdUkQi53CP7D/Am6c3D+01UZtilNCJ6tORGuHTL8uyvGbU/7/ClC58gCskcBViOxw== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nxp.com; dmarc=pass action=none header.from=nxp.com; dkim=pass header.d=nxp.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=F5FxjuNdReSAUurDwXC3fYfUz5iT8OSKubT84fjG1pI=; b=Kr5RGD+AH4R0nPtbvZ+FwplwmrqThbnm3AvmKyE/SBggg1aczczk25thuJesTbKEuO1WdLyyvM49O1zIeTomRoYLqgMREBO6xOYeeG+2mckdyi5z/p4j5JueRDQB9zQmuJqVANLdsDsWEJeCn1Ve+wkDOqWrC6sG8oI4nEVYXRWJguXmuzTr1DjRDE+XPkElzafvXAjVtCOOamUAd+1mB35MfaZgScnAbQ52nBecHRpELnwNXNzlxM5xgZWpOKh598cZ9ZjyH6eJggulIJ2DdvxL4UnylO5rDAnYBSDRQsa199XrE6KYeZs56GSEW6Zh/AYyVM8hPpijpwaSVBGcDw== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=nxp.com; Received: from VI1PR04MB5005.eurprd04.prod.outlook.com (2603:10a6:803:57::30) by AM7PR04MB6902.eurprd04.prod.outlook.com (2603:10a6:20b:107::20) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7897.28; Thu, 29 Aug 2024 09:28:37 +0000 Received: from VI1PR04MB5005.eurprd04.prod.outlook.com ([fe80::3a57:67cf:6ee0:5ddb]) by VI1PR04MB5005.eurprd04.prod.outlook.com ([fe80::3a57:67cf:6ee0:5ddb%4]) with mapi id 15.20.7897.021; Thu, 29 Aug 2024 09:28:35 +0000 From: carlos.song@nxp.com To: aisheng.dong@nxp.com, andi.shyti@kernel.org, shawnguo@kernel.org, s.hauer@pengutronix.de, kernel@pengutronix.de, festevam@gmail.com Cc: linux-i2c@vger.kernel.org, imx@lists.linux.dev, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: [PATCH 3/5] i2c: imx-lpi2c: manage IRQ request/release in runtime pm Date: Thu, 29 Aug 2024 17:37:20 +0800 Message-Id: <20240829093722.2714921-3-carlos.song@nxp.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20240829093722.2714921-1-carlos.song@nxp.com> References: <20240829093722.2714921-1-carlos.song@nxp.com> X-ClientProxiedBy: AS4PR10CA0023.EURPRD10.PROD.OUTLOOK.COM (2603:10a6:20b:5d8::15) To VI1PR04MB5005.eurprd04.prod.outlook.com (2603:10a6:803:57::30) Precedence: bulk X-Mailing-List: linux-i2c@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: VI1PR04MB5005:EE_|AM7PR04MB6902:EE_ X-MS-Office365-Filtering-Correlation-Id: 1eee1842-8981-4bc0-954f-08dcc80cf49d X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; ARA:13230040|366016|376014|52116014|1800799024|38350700014; X-Microsoft-Antispam-Message-Info: rxSe3Y8WZPP52qOj4K+yY39Kn4umjS5XDmABj+iri0E0cHXO/lZCqQxOk/uADoSGeuU+Ncn5ktJCKVjGp8IP7wXM1ogaMYuv+2UNepRu+xpzwrZBYyp3OZYNbb7YXfWGEscftSWpLJHSyXaAPQNjFSq5FEY8KxJtYZqvkx4yMxmZLv0Pw31bIiND3fCq3f4lvojwGfq9jQHTMN5HQPfeyz0baMbCQVct9YbTRo2Abjq4aZiXTEjCaIKhHKdcEGKo04+o4/8TYxhl84VcuH7PCfsYi0zbbMufkjadVD3Up4PBMOLf5IdhleLln2H3hkr3Z0nhYj3+sAoGARBoGDV2smyOKmcdQWYEQlACMoZ1OuHCDhfoY41WFvc/5V5kzMCj1yZ00cmGGT9Yau0F0Z3tbsa14qdt7KfMS3A9cabh/5muVa16CZzx1TlmgO2j3IfxPXfgFz2K7mlRTZfJAlwUvGhIXqDmKsOXkBT4jpoSrs5NoI2+V7D3nlSiQLOiPUwz3C7OA+42DoLqF0dul2uyszbdI0CIAEXMlWkmHOAZrvA/VqtR630zIP3/WWe1W6Z2gkcqZrSHdq/dq746aisdAeGhzb7eoV/qGHulPOp8VcVk/nV1aeIVJ0Nn1CmVAsMRTue7tFaFpOvobs/GYZwPr4f2bAczCF1bxQNcyb7oOmVOVfCBHiett4o3SYcpRPdx5ZJIVKP2/aelSacoptLAT/Mo9Pv+dPN1cq8ybqnwzX7D5aUNeGi9LrdYwrIwxYZVhjzjeb6/ZsPzuoJVq+GvatsqCX4OE1cNLtJWMuip/DRg1UFbaYoNhI6uerMrdcUV0D+IEqkZ9fQQItXckSc5JR97R/VY/sIjYC61MP0v9QyS1CbhsF0gCJ3XJRo+QgnAXSdY7hMVyaSjvYH0qfkaT7AjQ5UA1yHntVHLLTlWNRDivOyWADGF0u+t4PdYfXbDFFpvNDZUdkITpRBJA00oBxQhZd+2LLkT8WSfYKBe6MgjEwujURVuv1UFZx6qcHqNwvOj9toltiFbVuywac88KT4+nr+9VfgDO5nHPet5JbNQqcJYCDGshBlqboGiRwthHsvXFldeZRmajBWQ2MvdJul50Qz3rrEDccM1ZcvcpBin5HW9ViM2je1RIl3pT2T5R4dKTmKBeCqP65XrAAopGoxlP1rdQJ0nnr7HETuM3/iiJ09WmVVN3yfHUDMrWXMTrv3glNl+6QX6Ce1CntBwhS4UFjjlE/aEl/6VghzKwGwkfrqHHzAwj4ovGuHGg6gIqBhIxNHoLD5KIQ0iKGBQeAmUrEPatRVWtDi8vO7TB4P8sVPD77GU9OCL14kCmZ8S5er/vaOnvxTOQ7Nsf0AIaQ36jkcXcSelx7TO7S0wlC+HHB6v8Ud/J9DlY0STpdVf1grdXdjco+WT48egB3Dntw== X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:VI1PR04MB5005.eurprd04.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230040)(366016)(376014)(52116014)(1800799024)(38350700014); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: x5lt/ebbq34h+K+lu1vpm5bJ3/bX9ynacLBQ/CLmS/p8q3rRf1zo7j3fLY20U1iBm4dHeYKB4ZycaL/tBvS2aC+7vS3iEae/ZwkqpLErh/FH9TNEH4ujoskyDFfNEbcKgk6hWvjWJsmWQA0Qc+XtjfKLdKs5y6GIqxBB7JEcM/ws+IYFrsb30kGXEtPQ5vAwsfMCNnK4lXpG4qQoH2BmoUUYVMGGgLDyGX6xSXNp/3+jKQqpPvkJDqgt2WqlcGkbcNCrmS0PG3SxviVZoP6tsf/cNlIhp8PaQuB9oTXygIY8SBRzLFaHa0+WYl+VLN6BjiuQ1BTgPaOo2Mvh0cf0wZZPExsPUMutC/qrC4b2M6JlRK+XTHzPU0cDziBQUG3AqQkRdnH1Hmb8u461VtCaYh7nt3f7q9jECtlb9a0uTr5uUaI4GE9zzIemBsxykH7bGoBT365EG3VDhtEsxfUYRWtuBryVNwo3FT17c+ycWfOwcZ1gPcU92onj31luZZNdybTWiB1hO7ZO71acCzkxGoLu6t2U3m+yreZtzKhXhrMHc0xFr5OChexs5w9cwX8KRkGCbIw2Ezn1pZ0YjFSrgxHPSpGh1OmNejnjVbLI5hSRQFESwsTg/phzM3OX3xyM7ONspjUWhRQ8igWc1h4RCFJB/GR7iqsu7eEJEfY0pozkOhrbLfVmJswXMzjPKufNtp6wI95Q38+lbH00gX030hmgKKv9KcikHF3fDu3l2IftnCb/DjNnLVLFh9BRtOXdjH996jOxW5RSxmBnKnisr1jrM6bQVW40AMc9mUGBg7cf3TcnyFyxJY6Q2W2rBb/5tKViJhu594Oi7RP34FZ7K19lpadKZBz9X07SpAnzgjtXRe2cFAh42/1Sb0MrD+dzhGoU+SOBpkm/aj4CXdHJspyC8k5Uf1DVu5DoBatoNesMXk5fmOFOX6mGXeeGQtbY/pnG86D0darZ3pQmpsr7135Y1K/ggfr9mVNH6MwxzAfAZQEL7ETENnPI3naWGqPRsGtDYJPCBaikW3rJEPmjl5ut7IvnHD4Aqh8esOzzXqEgUodn74yJ95voRNhEgYAUJclULkvy0an99Py7dG0Jmeq3QM8UJU3q9JBBZ/Nrxe4J+Se1rqpf9/B/C7a7VgIhYmuk7K+yh2DoAtIjMRSKESMCmpXg6hDcvxI+tLhArB5uw+1cvJkPV6iPtIO8K2gpexZxxNTv7W5nJS1NTDpS8Ue8yndsNUTcrhaYM34zYDCe0X3wi/tZ3ocNU9cl+f5WLSqKAXexXK1Z3CuVdzwcGVFJw5xLbcCippPVV8kO4lEucrLnqizf/cEq67WdVFjCDsdLloocGMX4XsibwJe4wl3fku8j8HrMXfatp+DzxVQqe7CGRZrK14LkgF7GRs9XYnHKwHchlPd+gJeyQyMDpLHV3FoOlTGyjI/O7DNlODlWVvAlTD0WexdvFE9U/W4JWmEEgUHoLldUVw++nb4evqVVQ8aWHp3HCS83SjNEC1EGR7+LLJH8W3nxX/xVAiSMC1xfZvJjHZhezcRZcsLZgTHOU/4mPetPV6NckahU9c+udf+33ACZPDE9yIntszst X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: 1eee1842-8981-4bc0-954f-08dcc80cf49d X-MS-Exchange-CrossTenant-AuthSource: VI1PR04MB5005.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 29 Aug 2024 09:28:35.3736 (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: KVEEQhC+lMGSXQQ0BiEy5SUQG4mEjsQysNiKGwEvg8+23eer/BBsauLHAIopBu59rj5AlHQcxqbMW9Uh2ktw4g== X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM7PR04MB6902 From: Carlos Song Request the parent interrupt during runtime resume and release the parent interrupt dunring runtime suspend for allowing the parent interrupt controller to enter runtime suspend if there are no active users. No visible impact if the parent controller is the GIC, but it has significant power savings for parent IRQ controllers like IRQSteer inside a subsystem on i.MX8 SoCs. Releasing the parent IRQ provides an opportunity for the subsystem to enter suspend states if there are no active users. Signed-off-by: Carlos Song Signed-off-by: Frank Li --- drivers/i2c/busses/i2c-imx-lpi2c.c | 40 ++++++++++++++++++++++++------ 1 file changed, 33 insertions(+), 7 deletions(-) diff --git a/drivers/i2c/busses/i2c-imx-lpi2c.c b/drivers/i2c/busses/i2c-imx-lpi2c.c index 23f83f10d5f6..0159ade235ef 100644 --- a/drivers/i2c/busses/i2c-imx-lpi2c.c +++ b/drivers/i2c/busses/i2c-imx-lpi2c.c @@ -168,6 +168,7 @@ struct lpi2c_imx_struct { struct i2c_adapter adapter; int num_clks; struct clk_bulk_data *clks; + int irq; void __iomem *base; __u8 *rx_buf; __u8 *tx_buf; @@ -1252,6 +1253,27 @@ static int lpi2c_dma_init(struct device *dev, dma_addr_t phy_addr) return ret; } +static int lpi2c_manage_irq_handler(struct lpi2c_imx_struct *lpi2c_imx, bool enable) +{ + int ret; + + if (enable) { + ret = devm_request_irq(lpi2c_imx->adapter.dev.parent, lpi2c_imx->irq, + lpi2c_imx_isr, IRQF_NO_SUSPEND, + dev_name(lpi2c_imx->adapter.dev.parent), + lpi2c_imx); + if (ret) { + dev_err(lpi2c_imx->adapter.dev.parent, "can't claim irq %d\n", + lpi2c_imx->irq); + return ret; + } + } else { + devm_free_irq(lpi2c_imx->adapter.dev.parent, lpi2c_imx->irq, lpi2c_imx); + } + + return 0; +} + static u32 lpi2c_imx_func(struct i2c_adapter *adapter) { return I2C_FUNC_I2C | I2C_FUNC_SMBUS_EMUL | @@ -1277,7 +1299,7 @@ static int lpi2c_imx_probe(struct platform_device *pdev) struct resource *res; dma_addr_t phy_addr; unsigned int temp; - int irq, ret; + int ret; lpi2c_imx = devm_kzalloc(&pdev->dev, sizeof(*lpi2c_imx), GFP_KERNEL); if (!lpi2c_imx) @@ -1287,9 +1309,9 @@ static int lpi2c_imx_probe(struct platform_device *pdev) if (IS_ERR(lpi2c_imx->base)) return PTR_ERR(lpi2c_imx->base); - irq = platform_get_irq(pdev, 0); - if (irq < 0) - return irq; + lpi2c_imx->irq = platform_get_irq(pdev, 0); + if (lpi2c_imx->irq < 0) + return lpi2c_imx->irq; lpi2c_imx->adapter.owner = THIS_MODULE; lpi2c_imx->adapter.algo = &lpi2c_imx_algo; @@ -1309,10 +1331,9 @@ static int lpi2c_imx_probe(struct platform_device *pdev) if (ret) lpi2c_imx->bitrate = I2C_MAX_STANDARD_MODE_FREQ; - ret = devm_request_irq(&pdev->dev, irq, lpi2c_imx_isr, IRQF_NO_SUSPEND, - pdev->name, lpi2c_imx); + ret = lpi2c_manage_irq_handler(lpi2c_imx, true); if (ret) - return dev_err_probe(&pdev->dev, ret, "can't claim irq %d\n", irq); + return ret; i2c_set_adapdata(&lpi2c_imx->adapter, lpi2c_imx); platform_set_drvdata(pdev, lpi2c_imx); @@ -1392,6 +1413,7 @@ static int __maybe_unused lpi2c_runtime_suspend(struct device *dev) { struct lpi2c_imx_struct *lpi2c_imx = dev_get_drvdata(dev); + lpi2c_manage_irq_handler(lpi2c_imx, false); clk_bulk_disable(lpi2c_imx->num_clks, lpi2c_imx->clks); pinctrl_pm_select_sleep_state(dev); @@ -1410,6 +1432,10 @@ static int __maybe_unused lpi2c_runtime_resume(struct device *dev) return ret; } + ret = lpi2c_manage_irq_handler(lpi2c_imx, true); + if (ret) + return ret; + return 0; } From patchwork Thu Aug 29 09:37:22 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Carlos Song X-Patchwork-Id: 823666 Received: from DB3PR0202CU003.outbound.protection.outlook.com (mail-northeuropeazon11011036.outbound.protection.outlook.com [52.101.65.36]) (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 8AB1E18FDA6; Thu, 29 Aug 2024 09:28:46 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=52.101.65.36 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1724923728; cv=fail; b=NCjsmX3rNyhfpHBmuce4w/1c7FlekZH3gRfYERcjSAqgeb1AhS4mCtk5J+NxB+GjDjzFpjXWoh0C7f+P9TiPZATcJokAzc1oZeIFz1aMsH6eIlxR1BBWkJMlBVL/BuKlZz+v5FMJnw7Tj9qGicHCfdzy5Pg3rgnDXdwQKLXooN4= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1724923728; c=relaxed/simple; bh=k1QqtE24uZOJyPWcfFnA1FVkrloawdKgeMGB3WDNXLo=; h=From:To:Cc:Subject:Date:Message-Id:In-Reply-To:References: Content-Type:MIME-Version; b=eTfS0tqXqrsXLPqXuOB4Z/lLR+G6nOYZ0c5R3bQyQ9D2eXhHlAmyTZqIbG0E668RMF1BY5enwyunBIxZUYVjhSGyl2pw6Msq3P/Ii3nkl7knwxibuM22LHoLlbfaBxxdAgGFDNvwSL+D4oD2so1yfhEP4yx9apk7kzpYe7Dr+Eg= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=nxp.com; spf=pass smtp.mailfrom=nxp.com; dkim=pass (2048-bit key) header.d=nxp.com header.i=@nxp.com header.b=W4Jzuh1o; arc=fail smtp.client-ip=52.101.65.36 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=nxp.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=nxp.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=nxp.com header.i=@nxp.com header.b="W4Jzuh1o" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=tX78+gwBXu5DF/p1QKBUIz4aMFoemJPxUjLjgvjD912Q39FYnkoxw7SnZWgEWGqGaoIwwCdE0IlRKTGIPkQNF0IKjYTaKGiPmqlCLrnQVzP90bddLqVnn+y+KLdzGNi/3NQv+tYdGPtqfXPpfE0wrnLMo6XL817wdgnG46NM+pFMkIzZm0bLgncCFNAJ3y/+uST80+suo5fXbIoTBmXgw4fapqTE/f6IfwGSNg2PQnTjE/gPa+Y/lu8x1O/laizfAUo8A4TvAqEoifWQPgs9TrKVmIp/enxIFf3POc5wXlHGutJQrMsEd1cX/ZBLgPnKy+3IeT60rfp3nLa6+B2Fsg== 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=9pOKWFsLShoUkFNjIDJxlro2YyKFvBdbovV7Gwq3uZY=; b=H6qJHKnWSR78EzNBgg6e997vGnvl1FQSndcrR7Q/4DLEcMz/Q8GAJ59j/QiBLemBIQ8iHC+EA8JOiA8+tvp2VA3Pf3euTiFanfZEjOkTUY/AQavUfJ26VLpQDDJoOxLpnP0uBFNTbdTPi4X2qjfFhB7EX8jLm2ZWtD61m+dUcLFARLYwykErUPn8ncwWyTG2geKEgIHGtSQxu3RFeFXonJTw87jhZQ7SyHn2ITLF2xHVDgABLALhpLhvXFc3CqUyrQS5veHtrDYGfbFUFHZhp6FO5QwcCdngamNo6DXN1afZFAAbhqtYr2l2K4y6STTxZV1fvghMkv4BuA2iWnq1fQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nxp.com; dmarc=pass action=none header.from=nxp.com; dkim=pass header.d=nxp.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=9pOKWFsLShoUkFNjIDJxlro2YyKFvBdbovV7Gwq3uZY=; b=W4Jzuh1oOvnAuItagQbuCbZZe/RV0mWYGRrOVWFcJPgCIHgTX0W+r72DpEudHR2tpOCPpCagHCz6iVBqpvyn900TzWkjL5Fx2U+oDjE7QqkTqaYNv62jJrjc7Wrr0mvwTH6TPnxzJqvZZHJNRkpQFTPhCkET4o4Qfocp6ht5yQ4KaVO6tumIwMuUCYFCK1OfDByCVPHOEhFg8RxfvW25/ws2C2c5jVgGGauMJlwv7V9kjEkWmoOmViKv+896g2gm7tBUQyFaQq+LLCWina2M3G+wbcqOIQCp6t36cbF5DwhIYxuLQELxNSK3y6uiz/8AYVYYp9tPO0vzJVheJsvQww== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=nxp.com; Received: from VI1PR04MB5005.eurprd04.prod.outlook.com (2603:10a6:803:57::30) by AM7PR04MB6902.eurprd04.prod.outlook.com (2603:10a6:20b:107::20) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7897.28; Thu, 29 Aug 2024 09:28:44 +0000 Received: from VI1PR04MB5005.eurprd04.prod.outlook.com ([fe80::3a57:67cf:6ee0:5ddb]) by VI1PR04MB5005.eurprd04.prod.outlook.com ([fe80::3a57:67cf:6ee0:5ddb%4]) with mapi id 15.20.7897.021; Thu, 29 Aug 2024 09:28:44 +0000 From: carlos.song@nxp.com To: aisheng.dong@nxp.com, andi.shyti@kernel.org, shawnguo@kernel.org, s.hauer@pengutronix.de, kernel@pengutronix.de, festevam@gmail.com Cc: linux-i2c@vger.kernel.org, imx@lists.linux.dev, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: [PATCH 5/5] i2c: imx-lpi2c: increase PM timeout to avoid operate clk frequently Date: Thu, 29 Aug 2024 17:37:22 +0800 Message-Id: <20240829093722.2714921-5-carlos.song@nxp.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20240829093722.2714921-1-carlos.song@nxp.com> References: <20240829093722.2714921-1-carlos.song@nxp.com> X-ClientProxiedBy: AS4PR10CA0023.EURPRD10.PROD.OUTLOOK.COM (2603:10a6:20b:5d8::15) To VI1PR04MB5005.eurprd04.prod.outlook.com (2603:10a6:803:57::30) Precedence: bulk X-Mailing-List: linux-i2c@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: VI1PR04MB5005:EE_|AM7PR04MB6902:EE_ X-MS-Office365-Filtering-Correlation-Id: b01af4f0-34a4-49fc-559a-08dcc80cf9a3 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; ARA:13230040|366016|376014|52116014|1800799024|38350700014; X-Microsoft-Antispam-Message-Info: Snr6dm1xVekDOO/ux7Q+U+TvaI/n24zpPVl+Pyvppw3ri13XoNiIRLF/SmaTzXtUZUS6gw0E1K4kFXRk6oiuBUgGqK6meyktqCeHvQ5tjjpXahsA2r0X5zKJmAAJ95oiTvXGzMDm9oPsLKVek8S+eAEkHrH6aGRxX9yT4S4nGhQtF6bMFzC+VPb7YITYNOX+568p6TfIUp+6vVwc48zWiSt0FWuvzWkJbfUu9wssJwVKmjcr50Zh7NB9VBhYkrw5XALyWA9mbuQzRb1flN263fXx+YP9bbVIoWE8NDf4IQxkZz5LgG48zSIhB8oe/UmKjGjzfiisufGIdx3Byel7Sty9JxOMMienRepcZ9an6fpntY1jZRIBKsdK7esntfvPxPS/QozPXgSecd9o9cyPyg/JjPDGt3wg0Ec+52CfBQDrIBABDtgnFjf0QWQP2BvtD98e/IwJi1Ry+raLM6E+AuOYo4G/WomxTykZSaE1aXLhazVmZud1zk1c6fbeAtvT1JpzmV5ynVPf9cW6FHoCfj6b7yeGic0r1kRF9XcctHOqDYU9zp1Ae8R6/Jg1YEhisZVMQ0LQRBrcCBSVBf3G/KVNETP4/iAeYh2lI/rgCZfSYkgkXdl2tfNcCvt/qmlB0qOKhUPQaFgvYAMZmSdI5G6coHewrpq/HIXg6RPoRoqwFR5Yl/SXPMlJhHFDMPScG+NKrgWEw43CpMnWqoFWhoUyeiT4Oz6QQseEZGZwRiUN4trssLliItmaXzMfsCMV0oqYoxspio6Ha+8itwzzvCjEH+umN5psVhQXcTRhCOqCSfv9EygFyLVUNnuK99+tEvmAvbEc0PKCc3Ent8fdouraNmppKhqMLeFBOjNYsOEW60AArJnTZn8PL61tgRDBqxx7/DexppN8E1GKLRn6UcT7go+/pTWWHvlthh4VypF7nhQFnQfJRLxDwfZwttAF99YdwEoIGncXhYdEUM8sKv+0SrBsdfSxtbiHRpSa7dovsjToTIaKh5qaKd//u6sBzNyFkjcpV70gAKnBglMIDjeE6I3wmMolpGjWXomfSJnIfjsz0a3XWoVzxBiJ+AhwU5pEeHLCx8LCLn1Soacf6T3QxDj1XnMsKbAqVXjbnNNkOHsEdonPSfdZjDsyE66Lf0z0KCSnIukorpgHj8T8Tqb/3clHAuB/bzONBKEpptxw6Bt+nbNlk6PD/PUuBUmPiuALR757YejWEdHrnF+7X3gKyPKSbZgBeG3Lde8rMgPRI8Z2RiHr2ogKIOPQ/r8glauQVGAPFIOrdcS4hsquV13mCejSdX+oWByyRfFfrRBQ18GD8Kv4zz+8vh0SxzLFxNs1Loji6Ui1DfZbjGQni3sOjVVn/E7JVULHU+apd8fpaAcmL73DgBhUoMg90HdV8XCLefufqbbHRSllWWMJ8A== X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:VI1PR04MB5005.eurprd04.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230040)(366016)(376014)(52116014)(1800799024)(38350700014); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: Y7ceSi4WujvjGB1gOdd4qc34CcB0ePh4+vIK6ISw7Mh5o1t2ecgiMvlDIHHqXIOtkK1ue36Sur2ad5hndblehW8oOtcfyoartRA6cegqNI/uGEmkDTKJ4eB7WYvc1mDeHN/DcZXDXXQNxJjAa3RYT3ySlfAsnwZACuug6H4jekVA/2cv+mwTXNGs1U2q4YVpx9JlBav/FOyKQa3W4arVlsfVk4z4oPOjQMzk2cHGR1Y3bgX6aNfjMeo5gHnlvIYHaJ9Yka9ga9w2XMDMmB6D8WN1sPdM2a4NwmB5fEMPGLgqhxVu34I2XsVHBgFUomApoHz37hYa2+9yHSfootVb0LtmKUnegai8qscUw3OOXsdbVGlordBS6/xw6lUVaRTHebWEW2WgH8VEJ64ghn/gDivJsBOorx/KNtqKULPziXKXyvR06dt23EiOhbR7wN8iAMfepr1Nqxtk2PDN1TxPpZegcUUmtBzWUpGlDJoJARqrxB8taizY8HjWC6MoRgzuLlQzkBd/rT8RO1t0jG2RArcIf0WAosfGu7Kaine+TRurZYkvpbGVJy192mqb+++BvdtdWLrqJ6Vn03Zs+0xmlyxJaguM6pwjaA0Ts9Y9mTQpcNPGagOxiVtmDLH8x3dvvq16n8+lvb0bVIa1fkjK2kJJAQoPaKkrmL7w5KvfIUGGNQb1sjS4Tt9zV9/v2i2bNiHpmQl1dMsleB3VqbpUwi3pkaePcwmfdcDoC95mmnnVI65I9NJMjNgGSsnWbqsg9qmkYo/tcnkUwXSSse4+zzUTtVY9MX6GAiTNBh8baPfiNgpxIcKypfEN7FfK6firr+7ir2ZgKsU3PZAgnZ1xZO+kDs4JYWWJmQggMWxVwEtJF5gltNLwbL95SbdXsanmSvJ4nKwpgw/M/HbnJHJkS51qcJn6pDq/o6TZySWVKTJgRTO5OspDjXt4n8yVk+sEl/5+lzDa3lk1P/GPv3G1aw4CfQkdTV3Z8CQZoL2IvVaC8kYBlpjkL5xsjXPfJdLUk5/8v1KwECln3OVeDg/HxTlJpP6aDj/S4sBrGiUPvvaUgwlqSvK0wGjAwKFUEVQrRRvvEOaz5nvG0bjEBxil+pVJwF0msmIbn4e2upfr3/2sXRA3fGb2uGmpTLhnVm/R8ccHCK+gTbBhFnJ1/UUzO8GUbwyCObwsqAEcuSvAhgHluMm9lBjUkBlKcXx8QHTgCMlLmAuEmKJoiQvzYYXzcHYALqA4J/8d17Atk3j6mbPjlk2x0OjlwLrxvaqO+9b8eQGvWhXFpU+dCLLHVceV+ZM+JeJnu67H16sqBptdkAF5Kr1ec6A8SEEPVuOVv9gVt+EUHkwCbC/gFjgoV+bQV5gSabd4btWCJ/wl5NB8nI5NzheqOpf7BriKWtxsTMHbE3DPzSlmr8xrw91ae/ffXQN6+T3KFcsXMJyXiuLlilfpnalsRHQ+Mcx6o25tpRvkVvfQafgJM0yTlvwikh8B3UAA6phtjHuiSHghRKd2n5fRjYMjlwG+A/Crqd0AwCfAo/AJiIwhCSpte4gXCoahkLEtOZ3Mds0pM8okrtlPRaZzI8PruvdlLBWfEUTqTf4R X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: b01af4f0-34a4-49fc-559a-08dcc80cf9a3 X-MS-Exchange-CrossTenant-AuthSource: VI1PR04MB5005.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 29 Aug 2024 09:28:44.0041 (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: tTPojjFw0XwkJUNmgdTZcRqMwEzrRdAO+T6VzUma/vRaG+PM2kIh64e7smY2nXIfF+G6vsqGfeapLUavrAQd4A== X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM7PR04MB6902 From: Clark Wang Switching the clock frequently will affect the data transmission efficiency, and prolong the timeout to reduce autosuspend times for lpi2c. Signed-off-by: Clark Wang Signed-off-by: Carlos Song --- drivers/i2c/busses/i2c-imx-lpi2c.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/i2c/busses/i2c-imx-lpi2c.c b/drivers/i2c/busses/i2c-imx-lpi2c.c index 210d505db76d..cc5e5d96aacd 100644 --- a/drivers/i2c/busses/i2c-imx-lpi2c.c +++ b/drivers/i2c/busses/i2c-imx-lpi2c.c @@ -129,7 +129,7 @@ #define I2C_CLK_RATIO 2 #define CHUNK_DATA 256 -#define I2C_PM_TIMEOUT 10 /* ms */ +#define I2C_PM_TIMEOUT 1000 /* ms */ #define I2C_DMA_THRESHOLD 8 /* bytes */ enum lpi2c_imx_mode {