From patchwork Mon May 29 22:34:02 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Vladimir Oltean X-Patchwork-Id: 100443 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:994d:0:b0:3d9:f83d:47d9 with SMTP id k13csp1803733vqr; Mon, 29 May 2023 15:41:33 -0700 (PDT) X-Google-Smtp-Source: ACHHUZ7vaAt6FUxw7xL4mCkVuwaYXe6pyYnZnjvazL6WRVvt4/RCxhIV+kpS+A0VGv/aEFmsBcfO X-Received: by 2002:a17:902:e5c9:b0:1ad:bccc:af77 with SMTP id u9-20020a170902e5c900b001adbcccaf77mr10095137plf.18.1685400092929; Mon, 29 May 2023 15:41:32 -0700 (PDT) ARC-Seal: i=2; a=rsa-sha256; t=1685400092; cv=pass; d=google.com; s=arc-20160816; b=YX46dkL9CnlskKS+88CgTm4UjdT+x9GDmI8yoWfa9WuNtmOGTPNcGUEr7jAZ5TOOfe 8qiy+nzQAnGsFw4WYhzx3jX/7+do76pH1z5CPLVoYzqIgyqcAcSImUh0dmDlt5pcTKZN 51df07XBsIzW9sApSUi6eXGY4hVCkfoyW0PMaUNa7OsjbSvMimJOEpOHbRP8OVX79YZX w3xn4Zz7hqxpGhRkUOMqLhx2t1l4XTEvOC+zrauwWrNWecpjrctSdPnhV3wSmLpKUjnF /4h3KlT+c3EOm6Gu/2OoZh52JWoyJaYvk2svh6lvtbqUJ8aIYq26Kea4JgmIgZFVQhis MYaA== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:mime-version:content-transfer-encoding :message-id:date:subject:cc:to:from:dkim-signature; bh=vQRoxtHHXhOUcOs3oxtW1ichj+Gt9HHxXjXfNIEUweo=; b=bxqdg9oVdQa7RjMikNXfSxvx2TjveOXtPOcM2hM+bSaW5/Ht+AypWRa4FM1QKtPexG piWXbw5BRRt86n2CIMjwBtmd7pz+YRbGWVk7E8sXx33vrz4QUVEeNfriV7Z8cD2Y9yWU +xgkzZCzBWjv83smAk/6z31cbEulyxdD9yDR7k9tT2DnQuDgbO5Wygvn2ZMX1WjeilhL RRDk7wvXaiMGmGCH71evY1hLAEYfdnv9NVvpfTnvWj8PdXPJj/b0i6etcrKc3HuMRpXb iUBR29o2PViMGXOE0yGsSJGFfoTtVl8ok4/o04s8xRGwpv7884gjGjR9pNoVn59jkPoT Tt4w== ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@nxp.com header.s=selector2 header.b=oqpHh3we; arc=pass (i=1 spf=pass spfdomain=nxp.com dkim=pass dkdomain=nxp.com dmarc=pass fromdomain=nxp.com); spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=nxp.com Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id q20-20020a170902e31400b001b0113e102dsi7259236plc.365.2023.05.29.15.41.18; Mon, 29 May 2023 15:41:32 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@nxp.com header.s=selector2 header.b=oqpHh3we; arc=pass (i=1 spf=pass spfdomain=nxp.com dkim=pass dkdomain=nxp.com dmarc=pass fromdomain=nxp.com); spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=nxp.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229956AbjE2We1 (ORCPT + 99 others); Mon, 29 May 2023 18:34:27 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:52092 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229496AbjE2WeZ (ORCPT ); Mon, 29 May 2023 18:34:25 -0400 Received: from EUR02-VI1-obe.outbound.protection.outlook.com (mail-vi1eur02on2082.outbound.protection.outlook.com [40.107.241.82]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 23FAA9F; Mon, 29 May 2023 15:34:24 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=duIcogVctI+xstN3VvVt5CAYGtzbfRRZCqX7EoD9KQibrtxosl1drbvv8w90GIl9LbRQZFauh8c9tLqWh2fdFlIBeu5qrnv6jtvyTAv90QEipZTxFmgy8a+fdp0r1TEspZGTXrVb0H2bFGCD/pel4SiDsIUVESrVI9+PlcDraJY3FB5pDOvyF37FHt7ZO5SzIi+jA+cp2qTvMG9Qh8iRGN6BOeal3H+V5IzZTQFYM8abFgH+J3oJ7A4QRPYZk6Dmxh6Nqsrfx8NkaKEkFbmrMbBa3AllRJlImM2TYuSzTbS+txcLZ3us1uhf3hVIUWiLGhiyd5DAWX53vJLHOXa5IQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; 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=vQRoxtHHXhOUcOs3oxtW1ichj+Gt9HHxXjXfNIEUweo=; b=ek8GzNN+y5whWOaxLI+6PE+bk4sxBEPCnWCy5I6j3cWZ8uRS+eIb5mOPKzbnX9mGQjXLscmZH3rYyp0BJt2nG+MFwLbg3mWEGrEuytTNiKcQbpXPh3Qo6lOuPD3btW2B2mnH0KFxjtncrrDZgSGII2Wr2srvJtqwvTxFuboN5HJHd5i2RvVCNt5waLxe1SZ6/WCB+d08ZV3nWitS7QURLQ1Lr5XM6RnW93ojwswjT2b1LJaSlReQ3LZ3Vnp671BsZq72w3GJhrqeYXqUQrirxhRIfqAv3vus3Od0FiDaVhjw/2ulIfyIl73Th7THfaBrDIflZXJM2EQnoRKqt9mf4g== 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=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=vQRoxtHHXhOUcOs3oxtW1ichj+Gt9HHxXjXfNIEUweo=; b=oqpHh3wef7Q98m6AuhXTeknx7DVsBA4XmMg4mtgjywFZOVvtOd5EalZya5n+CudKpuomwR/3Lo+QhtRSEiWTNZ/Q3JHHXAWzkEl2I6V/6KUf90iu9JJfggDK7A6ypzUbuuBzImByfTCRXQKGpBDvzQCnGRYfTEEUvpDZU7wwJG4= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=nxp.com; Received: from AM0PR04MB6452.eurprd04.prod.outlook.com (2603:10a6:208:16d::21) by DBBPR04MB7802.eurprd04.prod.outlook.com (2603:10a6:10:1f0::20) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6433.22; Mon, 29 May 2023 22:34:21 +0000 Received: from AM0PR04MB6452.eurprd04.prod.outlook.com ([fe80::b027:17aa:e5f5:4fea]) by AM0PR04MB6452.eurprd04.prod.outlook.com ([fe80::b027:17aa:e5f5:4fea%6]) with mapi id 15.20.6433.022; Mon, 29 May 2023 22:34:21 +0000 From: Vladimir Oltean To: linux-spi@vger.kernel.org Cc: Vladimir Oltean , Mark Brown , linux-kernel@vger.kernel.org, Lisa Chen Subject: [PATCH] spi: fsl-dspi: avoid SCK glitches with continuous transfers Date: Tue, 30 May 2023 01:34:02 +0300 Message-Id: <20230529223402.1199503-1-vladimir.oltean@nxp.com> X-Mailer: git-send-email 2.34.1 X-ClientProxiedBy: FR3P281CA0028.DEUP281.PROD.OUTLOOK.COM (2603:10a6:d10:1c::19) To AM0PR04MB6452.eurprd04.prod.outlook.com (2603:10a6:208:16d::21) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: AM0PR04MB6452:EE_|DBBPR04MB7802:EE_ X-MS-Office365-Filtering-Correlation-Id: 6ca219f3-6025-4b86-f576-08db6094d8c2 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: gVa8gaZgnQgzrlAXWRYX+PiT7QW7aX0icCHx7OznQdu2IfIjbSNHeeDLklWoQrKAa0ZoqA0oiwxmYxpYbuyMNRfzNLQQ+m7Gx105+RQU+SA9iG2ssdU7xYjbWjrqiJcuri4/AC2aFpe+PuhWYaZsrFZmlXk3wADu3yHNXCy+mRsU2yDeXwRBTWtsyxn/0/5Rid9M6v0WrO5gV1i3bSoiygqnVgPnLjD5vF7LeDmbVRCfhrzjrxDjy6q1d8WQe5ArMxBBkdEFVbzCcJEPRZhUcOTo3D6hdJMn69tasrAgtmz/rCJMYxoNIaHe+lYY+1odSA/hrAAs8esgZ9u6mqiejJoo2xxC65t3z/4eBVLC+tDUzMDz8JRc66dtjrhk174GprzYMCAGFhuKgmiLaNTOMtGOHu80cg3lvFBXqhcQVwS6Dhy7urh13zdwwrAy24ccAUMUpJLycdmJH4KKYEJjxk4JvWLQU0BJu7/9Rnu+2DgJpCmv49OM/kL7yKFpqYB9tUSbgxem8BTwZLNGbOPpuSXQGVKVAqCRq2du/sEiWk7jRya3EryeE5bVw0edSv/iPvt98jjJXIUQFWvBAgyRCcG6dTmjuMgmTJusszt5UzB8P0ZyYfeIuklVh9ZmgCvl X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:AM0PR04MB6452.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230028)(4636009)(396003)(136003)(346002)(39860400002)(366004)(376002)(451199021)(66946007)(4326008)(6916009)(36756003)(66476007)(66556008)(83380400001)(5660300002)(44832011)(52116002)(86362001)(38350700002)(41300700001)(38100700002)(186003)(2616005)(8936002)(8676002)(2906002)(6486002)(6666004)(6512007)(6506007)(316002)(26005)(1076003)(478600001)(54906003);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?q?Q9GrEWwNdRo4KJKHp+q4Fog4ATVH?= =?utf-8?q?HGQ+u7hrGadpMmh2KIK1Dxy5c9QRqMC6JRZHX10oMJLA/O9jeEWSYm3FG3qbYs97v?= =?utf-8?q?7LyqeHni1+Z+rV1f3mh+Vjn1eDxYbTZs0vFV1vhocNrA8yNCFCBV032WoignALlxT?= =?utf-8?q?l/vlZc27Wbt0RjxWBvMEwBTNW+By6fEgBKqbd58voGSTxL+P/Y13nrhq6FqC9Zxc6?= =?utf-8?q?iVqCQvfMOdQgQqBhKLW7+o51sH+J6c+zguknuSuVTqCJilHd1jfhw/pNQ7QrcN3ZV?= =?utf-8?q?sYFGqlUDqUe0OMsJU9rBouoZSlJ1AELS2CkcakZ9hLsAuBQe3DTM3fiYmfQKxFJPy?= =?utf-8?q?GEn6Kn8yTmc5gSp25wVuCIuxslQ0bUYdG5COUasN1q97qZzRqAUKNc/RFakvnVc3n?= =?utf-8?q?tURY1UEc2r30jmsDcza1sjeGa0d0d0AAtlhcaLZ6BmTejszyHUfpIZwkD018WmwA4?= =?utf-8?q?4yGIdWpSHTH57PocPL0qVDkpG53pDBgCztQ+Fr2K3Xlxytx4RLihQVXsQS7Y2jgGq?= =?utf-8?q?iGbcHbwxuao5D1RmtIEx0vyz6FMTtrZGhERk/HzNfNAPher0cyoL/kvhjyYjNVoWK?= =?utf-8?q?3rnNVJy9Y11WuzHKsnxh/vsEp3WwgGel4MwLQPmcnd/CXjgbswVEum0XmHOwnH7+K?= =?utf-8?q?7Bbcpg99LoQB/Cm5loxNctWajBzGllHCE9MNcbP2P0FyVusAfdzRKB08x8/oMb866?= =?utf-8?q?wPoyzijwk0KtpFOIVQw4M0+2cux6j8zzpGxA0PmOJ19hIfFU1ixdGb7uLvzciIbYk?= =?utf-8?q?R4kI8N9KNvCS+nDEHPAz/n/HRu2EG/P7lEJyFvQ1EvyThYUSaf2W1wpJXWCtIZ/1n?= =?utf-8?q?nGaPYYeCdCiac3htOC9QGfnAVDftc3Tcx0MBDAsFRyGFGNSnubbTme1cL6ggVpjXr?= =?utf-8?q?d6OW+1zf2L+MF64LOjd+2g88aH8PhqUYD4vAbnuQ9Tz8ZVQoEPACVEpeBHtE7Rm1z?= =?utf-8?q?fjyjdG+rx2MnEnBdw4paDhtaZ98u8YPf+NuzyTog+78Y9Z7guxUv9n5Ri/+vlFKHV?= =?utf-8?q?euwDpcoQZukhbirIJ5zDWJBFFqeWrZ+4lpgW5VyyrKwTTkHNeUad6WIYeusJ/+l6d?= =?utf-8?q?N2AcroS5xpg/fty69HtYnwS3fMk6fmC5nFjxLrr1I8UXWfxuug4XfRpG/uXJZjQsr?= =?utf-8?q?2woEZzy7z6e+5MtjKY0RpyqpTyBIu2poIUbF8kcpQKQ11R6URd/AwwjJOmD0H03bx?= =?utf-8?q?1vH/QrLy20oJ+xCsVpHEa4ra3s5gzUt1V77X3xpeyEIK5fT7/bK9pePn5Z4AjmSJz?= =?utf-8?q?fkOklGEBvK8KZ2X+4QA3WXz3pz0wMojSRp3PkEoVpTfsmzZwV5vAWVq//MMNIW8Eg?= =?utf-8?q?+hjfUoWeKZ7diR+XZ242Ky/nBy0i+D6KZoZ1Kvq3RzwMR+uMn4uGwJ+pjHRPEXjE8?= =?utf-8?q?p/FgCD3l5gehc11Td8Avk+2jfPf3m8PXirKwcH6XRa2Px6xTcjAt3pjAyi2h/Yfzw?= =?utf-8?q?Zt7W/MRsGQlsHCnvut9Iw4bkqjSb4F/ney7FnOKJSN+Abdq5T8bE4pL7Qho1qPhfJ?= =?utf-8?q?mJR/nhPawXngBTfx/LweesdV3iNrfXibbg=3D=3D?= X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: 6ca219f3-6025-4b86-f576-08db6094d8c2 X-MS-Exchange-CrossTenant-AuthSource: AM0PR04MB6452.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 29 May 2023 22:34:21.3431 (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: 4Lx+eb0S3PemfRc2LGILX+3mCS2qYes0fvaFBZ47IJ6s8Y2CqNHIrMbtZhgSYzyldE426lIo8EasrBgpry8KUg== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DBBPR04MB7802 X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE, RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1767270087798055188?= X-GMAIL-MSGID: =?utf-8?q?1767270087798055188?= The DSPI controller has configurable timing for (a) tCSC: the interval between the assertion of the chip select and the first clock edge (b) tASC: the interval between the last clock edge and the deassertion of the chip select What is a bit surprising, but is documented in the figure "Example of continuous transfer (CPHA=1, CONT=1)" in the datasheet, is that when the chip select stays asserted between multiple TX FIFO writes, the tCSC and tASC times still apply. With CONT=1, chip select remains asserted, but SCK takes a break and goes to the idle state for tASC + tCSC ns. In other words, the default values (of 0 and 0 ns) result in SCK glitches where the SCK transition to the idle state, as well as the SCK transition from the idle state, will have no delay in between, and it may appear that a SCK cycle has simply gone missing. The resulting timing violation might cause data corruption in many peripherals, as their chip select is asserted. The driver has device tree bindings for tCSC ("fsl,spi-cs-sck-delay") and tASC ("fsl,spi-sck-cs-delay"), but these are only specified to apply when the chip select toggles in the first place, and this timing characteristic depends on each peripheral. Many peripherals do not have explicit timing requirements, so many device trees do not have these properties present at all. Nonetheless, the lack of SCK glitches is a common sense requirement, and since the SCK stays in the idle state during transfers for tCSC+tASC ns, and that in itself should look like half a cycle, then let's ensure that tCSC and tASC are at least a quarter of a SCK period, such that their sum is at least half of one. Fixes: 95bf15f38641 ("spi: fsl-dspi: Add ~50ns delay between cs and sck") Reported-by: Lisa Chen (陈敏捷) Debugged-by: Lisa Chen (陈敏捷) Tested-by: Lisa Chen (陈敏捷) Signed-off-by: Vladimir Oltean --- drivers/spi/spi-fsl-dspi.c | 15 +++++++++++++++ 1 file changed, 15 insertions(+) diff --git a/drivers/spi/spi-fsl-dspi.c b/drivers/spi/spi-fsl-dspi.c index 4339485d202c..674cfe05f411 100644 --- a/drivers/spi/spi-fsl-dspi.c +++ b/drivers/spi/spi-fsl-dspi.c @@ -1002,7 +1002,9 @@ static int dspi_transfer_one_message(struct spi_controller *ctlr, static int dspi_setup(struct spi_device *spi) { struct fsl_dspi *dspi = spi_controller_get_devdata(spi->controller); + u32 period_ns = DIV_ROUND_UP(NSEC_PER_SEC, spi->max_speed_hz); unsigned char br = 0, pbr = 0, pcssck = 0, cssck = 0; + u32 quarter_period_ns = DIV_ROUND_UP(period_ns, 4); u32 cs_sck_delay = 0, sck_cs_delay = 0; struct fsl_dspi_platform_data *pdata; unsigned char pasc = 0, asc = 0; @@ -1031,6 +1033,19 @@ static int dspi_setup(struct spi_device *spi) sck_cs_delay = pdata->sck_cs_delay; } + /* Since tCSC and tASC apply to continuous transfers too, avoid SCK + * glitches of half a cycle by never allowing tCSC + tASC to go below + * half a SCK period. + */ + if (cs_sck_delay < quarter_period_ns) + cs_sck_delay = quarter_period_ns; + if (sck_cs_delay < quarter_period_ns) + sck_cs_delay = quarter_period_ns; + + dev_dbg(&spi->dev, + "DSPI controller timing params: CS-to-SCK delay %u ns, SCK-to-CS delay %u ns\n", + cs_sck_delay, sck_cs_delay); + clkrate = clk_get_rate(dspi->clk); hz_to_spi_baud(&pbr, &br, spi->max_speed_hz, clkrate);