From mboxrd@z Thu Jan 1 00:00:00 1970 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=-5.5 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=no 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 0D3EDC433E0 for ; Wed, 6 Jan 2021 09:33:17 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (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 9716222DBF for ; Wed, 6 Jan 2021 09:33:16 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 9716222DBF Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=arm.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=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Message-ID:Subject:To:From: Date:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:References:List-Owner; bh=pRkKREtawg7EJaToN3WAiQBfdINfif8Ln2zKYhlRHM8=; b=Zajc5XGXEFifQ2lq5MJ+YRFKt Q8BzFlKkkigGOlOa+cDCGK6Yt7DXzhg19jtucuVJJBGgLEeM+0Y2e2bP3Hn9AhL+p+/iY14/BqBML Ny6jkjRObEjNyRmLTDqT2sDW/CiKWeIl6VaY4HN90XRowXpJRBoUTv4K5qGUpOpQgpQUgUNIzLIEu YqZVPTC9z8auJJWeg8W3B32Rf8wQnnnvicTj15DzNl/23QCoD5ToihqEDcXWoidT7QgAv0k7Sx/xC G7tonO6twirNIrOnOh4xjcTDoEflogmflxvedFtW/n6AXSqAP07MNm9JuSSXHLjJJFECAPNvu4d1h 09vtOzECg==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1kx59h-0001xb-St; Wed, 06 Jan 2021 09:31:01 +0000 Received: from foss.arm.com ([217.140.110.172]) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1kx59f-0001x7-4o for linux-arm-kernel@lists.infradead.org; Wed, 06 Jan 2021 09:30:59 +0000 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id A81D81FB; Wed, 6 Jan 2021 01:30:57 -0800 (PST) Received: from bogus (unknown [10.57.35.27]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id E261E3F719; Wed, 6 Jan 2021 01:30:55 -0800 (PST) Date: Wed, 6 Jan 2021 09:30:53 +0000 From: Sudeep Holla To: Jim Quinlan Subject: Re: [PATCH v4 2/2] firmware: arm_scmi: Augment SMC/HVC to allow optional interrupt Message-ID: <20210106093053.tnlncqvcba7qnwnk@bogus> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: User-Agent: NeoMutt/20171215 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210106_043059_250393_559C2F95 X-CRM114-Status: GOOD ( 20.74 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Florian Fainelli , open list , bcm-kernel-feedback-list , Jim Quinlan , Sudeep Holla , "open list:SYSTEM CONTROL & POWER/MANAGEMENT INTERFACE Mes..." Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Tue, Jan 05, 2021 at 01:32:49PM -0500, Jim Quinlan wrote: [...] > > I don't think that is the case; the bottom routine, > do_wait_for_common(), decrements the x->done after a completion (which > does an increment). Regardless, I think it is prudent to add the > reinit patch you've provided below. > Ah right, but I will add that. > BTW, one thing I could have done was incorporate the timeout value but > I thought that since this is the "fast" call we shouldn't be timing > out at all. > Agreed, we can add it later. However it is not related to fast call, it is more for protection against failure of delivery of interrupt from the platform or any firmware responsible IMO. > > This electronic communication and the information and any files > > transmitted with it, or attached to it, are confidential and are intended > > solely for the use of the individual or entity to whom it is addressed and > > may contain information that is confidential, legally privileged, > > protected by privacy laws, or otherwise restricted from disclosure to > > anyone else. If you are not the intended recipient or the person > > responsible for delivering the e-mail to the intended recipient, you are > > hereby notified that any use, copying, distributing, dissemination, > > forwarding, printing, or copying of this e-mail is strictly prohibited. If > > you received this e-mail in error, please return the e-mail to the sender, > > delete it from your computer, and destroy any printed copy of it. I am assuming this was unintentional and ignoring. If not disregard my response. Otherwise you may need to fix your mail server. -- Regards, Sudeep _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel