From patchwork Wed Jun 16 03:10:44 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Quan Nguyen X-Patchwork-Id: 12323791 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-11.9 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH, MAILING_LIST_MULTI,MSGID_FROM_MTA_HEADER,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED, USER_AGENT_GIT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 8389FC48BE5 for ; Wed, 16 Jun 2021 03:12:55 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 362D461356 for ; Wed, 16 Jun 2021 03:12:55 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 362D461356 Authentication-Results: mail.kernel.org; dmarc=fail (p=quarantine dis=none) header.from=os.amperecomputing.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-Id:Date:Subject:Cc :To:From:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References: List-Owner; bh=P84rOM82ih+cX+UMA8Ml2ABeVykDrpWhz1Le/NLu7zE=; b=QrpDX8kvIXuDsc Z6BhYaEiVSrnyVi0xl+n/lH62jg5pzUrPKOcMeucbPi+Uv71ZZO5fWJCFDL8i8o6Nvt13Uk93HFTG D7Rz8fBHN4H9OaTeWBsJw850XZBww8pRnNC2w3+4i/kfMfJEl0UozsnN7NluzHLOu/1k9HXM0pYHW xflYP8U2WYrl31QNab8MaztuEBEDJGE3Huh8aAs1z4OH2BSC6U+p9Z69WB3oXHgWPteoEiU3smk/Q 3huB+xySVswPXYO+W+1RmKXt/uVq7Cl4dJubbobqnTL2l8/AB/wgOtey+HDwNk3kk0LNVLsUStcn0 zsiP2NoYEyWRwVNvBJWg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1ltLxR-004WV3-Rj; Wed, 16 Jun 2021 03:11:14 +0000 Received: from mail-bn8nam12on2094.outbound.protection.outlook.com ([40.107.237.94] helo=NAM12-BN8-obe.outbound.protection.outlook.com) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1ltLxN-004WUK-UF for linux-arm-kernel@lists.infradead.org; Wed, 16 Jun 2021 03:11:11 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=HdfvqzRDO65NR8BxDpeuEWhFBQEC9MzM28Qpup9sTtLo9uF6zWTrRikbSmCRcK5A6fktP8St/HceF5I2y/BiPOJncHchfanIzBB3GJj/dLeZah1EGCtd+5Exer6yxqn/b7wvr/XTzynxhP9rELEnkCA4WadH7ocweCQDh3Ah9MqIwnew5ib/8e9SkrH2NpBdt4B9DgnBu/a3apa2yxr4vq4iLJOD1qxF7WbLoKucmlzHW8X2kSw4Pq2pMPK9FKqcM8paW6PnSkIcsyj4/WncL3QIv4l5f2gMpDW1uZwnm3FbZghn9XEQ9KyqxdV29ZsfrKQJ5VHb/uM+8YplxprLhw== 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-SenderADCheck; bh=rB4IwdppHnDtjSly2R1vOI/yiyYMAekO7oDaZRSqtxI=; b=bc81QRwE5TeCSJcSTkmsZ3djxCYqc2RDwAeLNHtPuOIi2WCy2tNo5/2k9JtVgoJuuLapD528iCLWfXkW0l4lIgtxqsPGrFUfGKIJ/7Xfedk0IUfALNbr2Y4t/leIZER43NMLEo2kycqutXCOwNEwsJCA+pQhD7DHfqN3Az16UMSUbejh6pk+iTAs/JpifErP5y8Zcy62BnYzlEuwbHA95OfliOURBzRTqi3O63Rp9Q4izRxiaDq+Nx9UQvwmHyrQLx2BKoGdPBswku3OLeHW5Pa8MpSgcXBN0XKEHb4JsyHY1rgI+F3BpZILazKzPe68ro2x8TLsg90nZiKQIAdBeQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=os.amperecomputing.com; dmarc=pass action=none header.from=os.amperecomputing.com; dkim=pass header.d=os.amperecomputing.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=os.amperecomputing.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=rB4IwdppHnDtjSly2R1vOI/yiyYMAekO7oDaZRSqtxI=; b=OHVc6pOxVB7zLniy1IGif3dT3jg8T8lhEsqbbmv4STe+xO9yS5wUchJmGOYjFZau1IG9kTHzEruyAGkrUMYk+1S8aMk/clixzMqrTN1QTYc1aPRdDUVAglG1ol2te4e4vWTbDUaXgiU1ZMmwQu1bctuisKjYEnCbd/0Z+txs4lo= Authentication-Results: google.com; dkim=none (message not signed) header.d=none;google.com; dmarc=none action=none header.from=os.amperecomputing.com; Received: from MW2PR0102MB3482.prod.exchangelabs.com (2603:10b6:302:c::32) by CO2PR01MB2200.prod.exchangelabs.com (2603:10b6:102:7::26) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4219.24; Wed, 16 Jun 2021 03:11:03 +0000 Received: from MW2PR0102MB3482.prod.exchangelabs.com ([fe80::452a:24fb:12cb:9d7e]) by MW2PR0102MB3482.prod.exchangelabs.com ([fe80::452a:24fb:12cb:9d7e%5]) with mapi id 15.20.4219.026; Wed, 16 Jun 2021 03:11:03 +0000 From: Quan Nguyen To: Brendan Higgins , Benjamin Herrenschmidt , Joel Stanley , Andrew Jeffery , Wolfram Sang , Jae Hyun Yoo , Guenter Roeck , linux-i2c@vger.kernel.org, openbmc@lists.ozlabs.org, linux-arm-kernel@lists.infradead.org, linux-aspeed@lists.ozlabs.org, linux-kernel@vger.kernel.org Cc: Open Source Submission , Phong Vo , "Thang Q . Nguyen" Subject: [PATCH v2 0/2] i2c: aspeed: Late ack Tx done irqs and fix unhandled Tx done with NAK Date: Wed, 16 Jun 2021 10:10:44 +0700 Message-Id: <20210616031046.2317-1-quan@os.amperecomputing.com> X-Mailer: git-send-email 2.28.0 X-Originating-IP: [118.69.219.201] X-ClientProxiedBy: HKAPR03CA0001.apcprd03.prod.outlook.com (2603:1096:203:c8::6) To MW2PR0102MB3482.prod.exchangelabs.com (2603:10b6:302:c::32) MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 Received: from hcm-sw-17.amperecomputing.com (118.69.219.201) by HKAPR03CA0001.apcprd03.prod.outlook.com (2603:1096:203:c8::6) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4242.9 via Frontend Transport; Wed, 16 Jun 2021 03:11:00 +0000 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: fed0ce44-2bcb-48b3-32ac-08d930745fcf X-MS-TrafficTypeDiagnostic: CO2PR01MB2200: X-MS-Exchange-Transport-Forked: True X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:597; X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: vf0UeId2CY1xx5pyR3tbR1jGrKNpr9S8ZExEED9RsCSH0DrUzpTw6L49QZJhH/eAmGo7BrWfD3XJXR1ergFOcnoYFPkvy2kgXNCUxRuWZVhTmuvj5CiV+w+jGfwwzbX3n5N3VTDphwuOYKD4ap40nK/uXtqmOP4kGHlA46hp7jZobS7rLOfWuDPTBtFI16mF6ozopqwqMUzQEzkCVKIIE4Yh3YYhSvH0enkCbuJsgKH/31w/Z9jE8pHfHUKdHEV9V9soarDqNV8kcv8Epx/unegdRT4l/T0I+1NVbDdKp6VTi8nPgVlhFwxxT7JZ+dkN0dZzjcH/k1tc5RSm1T5gCCBrtuXToU34BEZsFkLgQzY0hKpVy8/E2nyRX3K+GgdEPneJsNJEAtmwQXwQUjKJRdGx9uxKUd5HwocAFz3WqiWhuqmPnrUcVU+FCiyjIEjkQ/5p7A0bK/l4y/lZRBFEyaCUDR4lWlEjiB40lZkhA2oZA/CYmcRM0fgDXWKA50vPHa9EEwHJpebi6iI9gwvD/2hEBmLnp8oyiIyK0C5ocV9DeN8E9M/rHh+V3ZWJivT+8FkXpIlatNQOmcvOkOdYmAvamAlJVSLMrtwlNmT/mfoYTfm4xPzpnXq4QqbFIOvrEK5+OIqKaN4e6YpUugDpDStSHSzx6opwnNJCO7ftL6xk0gsWx4WWkZ8z8doqpBjpanTrOPunXJPLizBs2GTusc/N4j/TBHmbpr2hmD27pWxnEV35TFWwhJyztaiVOhHGDW0yAj/66tTpcGzxm2XHZ75YVKVGLuz+ZoYR8OL8ris= X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:MW2PR0102MB3482.prod.exchangelabs.com; PTR:; CAT:NONE; SFS:(4636009)(39840400004)(396003)(366004)(346002)(376002)(136003)(66946007)(956004)(1076003)(186003)(16526019)(921005)(4744005)(2616005)(38100700002)(38350700002)(6512007)(5660300002)(7416002)(83380400001)(8676002)(2906002)(6506007)(107886003)(4326008)(110136005)(54906003)(26005)(6666004)(6486002)(478600001)(52116002)(966005)(8936002)(86362001)(316002)(66476007)(66556008); DIR:OUT; SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: tlY27AphRGIJ3nfurgBuxtCLZycnUbAqlOAQ2iUb0ol3aN/EHDdzmzl8cquOSIQtKfWSPAaizM8rBxXNiKy+Lean9FAq6da6B6JeU1i6Hy5fbenzADB4ZW4q/AaI/SU3aFsAvREGkwU9ED1eHeeeVreh56I17qLuH2ZZYbPssYLpSnRn9cHuzwLmELla1qpEbBFh5QURr181IqA5RZSkWojpbxvyIEOtR+Q00zgIRlYIcqW72IfaM2/O44SQzx/Ttcy++Bm3bg5Dhe67JZ+XQAh69ej/4YCMvWpT3ByW9qeXc5AghhUCDATd9cDHWFCaFenx4UInuXqNGpmK+xbzCw3vul7fdpRIOJTnJGMChW+en5wDlEHqi521LxuLptx4TyElClVfQrjAvCnrGhh0o8mxj+pVTKAT4lypr5vsDgyfy5faRID3vDkfSlUUBnjsfGhfuV1iTts0FyxLQ+yDFoIczTv9y02zhe/CqVkKVuJY9s6ecB+3iCaJfVexv9+cafgGzcxXj/QagD+osi51OPBmc7b599vfImgcOrJuYA+P+2BEBfYwbcEt+xuf3qMVB6pspiqgfviBx1kgbTZkmrVgjBY8xuVb+qMPaUBSns5cFl28CJ7ksye+rgCk0KPAp0alWpjdrHPlJtv+UV3QUI2UcF/7Y6VWQvNxIGNo6jqKVrOaYhFGQkuvAPRleSvoI2sejXzX8lUnz0el8vBxqz07GTn/cvyASesJjv3crFjqSO5oPj5Z+gzYd3yJuUSzLL/sYPvu4vGqawauNV7AZzQOoATtdO8M3+SSA0DMmUATlxbVc5+V8s4U7K27r5V2xYnPXL/etXosWecqK4ytha6ClNKW8O9ucllWeB4Vjsq84oVBOYGdMCI5jYQd/2LvKaysV+HMgvcKV1NeeBty/zBP9blvF6JVOW3aYtWaYcpEC6PRcIf3402tkaM73fX4p3tAokyDAmq7KQc/963sBIFIztBA7MpOaRv+t7sJM+FJt19WacK/+AhIrWSN/65qtWQy41c1DeNBo1ZnMEhuO8Ska+k9OslCqZ9y8OZQTEi16Z3DryICPkORTq2n6IkuTd3iRHl1g6ARDeeUGPQfjOBlKyGc1A++NVKh72pp4557JaN7D0gC6DqxI6yW5HSoUq+fcYiNlJUZAMqwkgwB1EUDmxEbl+Tqf6bVGIr0ycsP7hNnpMZ6s1wlYdvwArNIBi1Ztvs0SIBgKqr4M5hg23EKRfHByCKaQ4RKm8yO2Ui+obLCdK3yxA8btQ4qfc3FwvF9W8jOiaCievoXSgvV1uDRl7eeAIHO12sOvdR3vu+0Eqklwoyx+JQmzDol0Mnu X-OriginatorOrg: os.amperecomputing.com X-MS-Exchange-CrossTenant-Network-Message-Id: fed0ce44-2bcb-48b3-32ac-08d930745fcf X-MS-Exchange-CrossTenant-AuthSource: MW2PR0102MB3482.prod.exchangelabs.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 16 Jun 2021 03:11:03.6486 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 3bc2b170-fd94-476d-b0ce-4229bdc904a7 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: 4/T2clpQ3aS0jAx85eJcGqpdA+N7zPz6nvqtcQ6SEcc9AlEGdsZsf5cPEJMIMuAu1aT5szNIRAV0SM+UA+9m5UJ3yMmRgtptL+ZHFf82lv8= X-MS-Exchange-Transport-CrossTenantHeadersStamped: CO2PR01MB2200 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210615_201110_110569_6A186358 X-CRM114-Status: UNSURE ( 7.51 ) X-CRM114-Notice: Please train this message. X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org This series consists of two patches to fix the below issues observed when testing with slave mode: + Unhandled Tx done with NAK + Early ack'ed of Tx done (ACK and NAK) causing "Unexpected Ack on read request". v2: + Split these patches to separate series [Joel] + Added the Fixes lines [Joel] + Fixed multiline comment [Joel] + Refactor irq clearing code [Joel, Guenter] + Revised commit message [Joel, Quan] v1: + These patches are first introduced in https://lkml.org/lkml/2021/5/19/205 Quan Nguyen (2): i2c: aspeed: Fix unhandled Tx done with NAK i2c: aspeed: Acknowledge Tx done with and without ACK irq late drivers/i2c/busses/i2c-aspeed.c | 22 ++++++++++++++-------- 1 file changed, 14 insertions(+), 8 deletions(-)