Received: by 10.213.65.68 with SMTP id h4csp24179imn; Mon, 12 Mar 2018 05:36:00 -0700 (PDT) X-Google-Smtp-Source: AG47ELtXAukTqOmv7soqaM98xkJVMQFxnafUiFF51wHtvWWyNqalW1Cd0DK91qZMGc0b0eo/fqF2 X-Received: by 2002:a17:902:858c:: with SMTP id e12-v6mr8137487plo.39.1520858160890; Mon, 12 Mar 2018 05:36:00 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1520858160; cv=none; d=google.com; s=arc-20160816; b=TyITE/wvapRvbXTFMFOXE/12QIqAk5oxly6Ho+/qixz5FYrNlLhwv8qq+1B+Vg7d3J hq2dUq+Oit1zKJiJMt1L7bb2jC5pByQMUBkEl27inucnlq0nL2x8RxFQTVJ6Y8dNkXQ+ rvhGWUlMvSIEadaO68Dd9VIw4M5RZtMIT6DMvnXl6QOe6xL5fhyLaMpC7yh/E8522Y8z QtopdQorKm+MIvpN0QWS6nf71O569jCvIpFDERKDu/DzHzoMkmiunJS+uvooHAXPypgo Zgj14+q1YA1ouGc5mA2RJLR6kAW66uyg9oHo2MQN2opHMwUMYZRkSPGGdqodjF8XEMv8 QJbw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:message-id:references :in-reply-to:subject:cc:to:from:date:content-transfer-encoding :mime-version:dkim-signature:dkim-signature :arc-authentication-results; bh=4yJqM+Tlw/Hr/Ufe0m8CmAvjYcPhOhzJOdvf+it1q/c=; b=DTG5zIpqUj5Q62M+DbGuUxwhdWHzqj8xeujgXWdNaSh+SGD3M6d2ggceEcAXKzLr1M dUshYQVjPvBa46qut0tN/b9fs3tGpNHNHPuZDYZcfx1QhTbKSIqdE+pxsAnqSrAieq0N r6vtghw8jqklOAc+qmTftV6yyC460kGkVWLiMXQQskUPPvpc3cttHPl3ek4dZhugJtik DLCZs6R36NBsNPDgcWXeOJB7twkur/rYTsOOZCNJjQmVyBCyCP1q51vXVk80Lav7jo69 IFoIWMO9KsT3Rintik+YakFBchKYKCXO/daPb64FxRPuwgXrflxAhpDMF6Us0H2Ljvsj rhyw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@codeaurora.org header.s=default header.b=WU8D0tHX; dkim=pass header.i=@codeaurora.org header.s=default header.b=j0JDFWRq; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id s7si2200264pgr.357.2018.03.12.05.35.39; Mon, 12 Mar 2018 05:36:00 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@codeaurora.org header.s=default header.b=WU8D0tHX; dkim=pass header.i=@codeaurora.org header.s=default header.b=j0JDFWRq; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751336AbeCLMen (ORCPT + 99 others); Mon, 12 Mar 2018 08:34:43 -0400 Received: from smtp.codeaurora.org ([198.145.29.96]:51458 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751104AbeCLMel (ORCPT ); Mon, 12 Mar 2018 08:34:41 -0400 Received: by smtp.codeaurora.org (Postfix, from userid 1000) id 26AA360452; Mon, 12 Mar 2018 12:34:41 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=codeaurora.org; s=default; t=1520858081; bh=XEdbFEOZG2d85ZQx95lR2PI2yxuijwAxJxxB4C1itV4=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=WU8D0tHXGVEn75aM0pSXEDzC3qqZAg62wd4LI0qlQxJ4JWbrrD/8BZsTVce1WlulC fSZs4liZ7XfM3B4sPdBmFEt8qO8AEEyWEPiXDVvSSflqSUeO6z3kDaECwLNFrNP3jx ElFWTB5bU/suOuqMr8fuAbqrN4IOlh8VAs8zfhhs= X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on pdx-caf-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-2.8 required=2.0 tests=ALL_TRUSTED,BAYES_00, DKIM_SIGNED,T_DKIM_INVALID autolearn=no autolearn_force=no version=3.4.0 Received: from mail.codeaurora.org (localhost.localdomain [127.0.0.1]) by smtp.codeaurora.org (Postfix) with ESMTP id 9376A60452; Mon, 12 Mar 2018 12:34:40 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=codeaurora.org; s=default; t=1520858080; bh=XEdbFEOZG2d85ZQx95lR2PI2yxuijwAxJxxB4C1itV4=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=j0JDFWRqLwxWepmcgYlSp+7ckKB3v7aKc+GB1xDp/Bmzj8ZQ8e8eA6W2KP7BomojT JXo5FG9uRr8dZJUepjQqNbMIlnburgVct2vyVEFofkEayS+yI5AiQHcQY/WN7AUNrK 09z8qsQ38BPC4/Wbgbe1Y/nIsKXlrYymFiK+YgAE= MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII; format=flowed Content-Transfer-Encoding: 7bit Date: Mon, 12 Mar 2018 18:04:40 +0530 From: Abhishek Sahu To: Andy Gross Cc: Wolfram Sang , David Brown , Sricharan R , linux-arm-msm@vger.kernel.org, linux-soc@vger.kernel.org, linux-i2c@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 06/12] i2c: qup: proper error handling for i2c error in BAM mode In-Reply-To: <20180227225856.GG20901@hector.attlocal.net> References: <1517644697-30806-1-git-send-email-absahu@codeaurora.org> <1517644697-30806-7-git-send-email-absahu@codeaurora.org> <20180227225856.GG20901@hector.attlocal.net> Message-ID: X-Sender: absahu@codeaurora.org User-Agent: Roundcube Webmail/1.2.5 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 2018-02-28 04:28, Andy Gross wrote: > On Sat, Feb 03, 2018 at 01:28:11PM +0530, Abhishek Sahu wrote: > > > >> @@ -841,20 +856,12 @@ static int qup_i2c_bam_do_xfer(struct >> qup_i2c_dev *qup, struct i2c_msg *msg, >> goto desc_err; >> } >> >> - if (rx_buf) >> - writel(QUP_BAM_INPUT_EOT, >> - qup->base + QUP_OUT_FIFO_BASE); >> - >> - writel(QUP_BAM_FLUSH_STOP, qup->base + QUP_OUT_FIFO_BASE); >> - >> qup_i2c_flush(qup); >> >> /* wait for remaining interrupts to occur */ >> if (!wait_for_completion_timeout(&qup->xfer, HZ)) >> dev_err(qup->dev, "flush timed out\n"); >> >> - qup_i2c_rel_dma(qup); >> - > > So this really only works due to the previous patch that adds the > flush/eot tags > to all of the read messages. If the answer to the previous question is > that > only the last read message gets the eot/flush, then this code needs to > remain in > place. Otherwise, it's fine. > > > Andy Thanks Andy, We need to schedule EOT/FLUSH after last message. For following transfer READ, READ, WRITE (FLUSH + EOT tags after that) In this case, FLUSH will clear all the descriptors till WRITE and trigger TX completion. EOT will be copied in RX FIFO and trigger RX completion. Thanks, Abhishek