From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from wout2-smtp.messagingengine.com (wout2-smtp.messagingengine.com [64.147.123.25]) by mx.groups.io with SMTP id smtpd.web11.31474.1683758234623810303 for ; Wed, 10 May 2023 15:37:14 -0700 Authentication-Results: mx.groups.io; dkim=fail reason="signature has expired" header.i=@bsdio.com header.s=fm1 header.b=Uhb+/knX; spf=pass (domain: bsdio.com, ip: 64.147.123.25, mailfrom: rebecca@bsdio.com) Received: from compute3.internal (compute3.nyi.internal [10.202.2.43]) by mailout.west.internal (Postfix) with ESMTP id EF39D320095F; Wed, 10 May 2023 18:37:13 -0400 (EDT) Received: from mailfrontend1 ([10.202.2.162]) by compute3.internal (MEProxy); Wed, 10 May 2023 18:37:14 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bsdio.com; h=cc :cc:content-transfer-encoding:content-type:content-type:date :date:from:from:in-reply-to:in-reply-to:message-id:mime-version :references:reply-to:sender:subject:subject:to:to; s=fm1; t= 1683758233; x=1683844633; bh=+x0z7KOoxUIEVb60090u6mQJC43vuIbIkTA wp386lVo=; b=Uhb+/knXcQsIj4SgJh6m6EsjJ8a7Uw/QJcYDEmalEIX3QvqWVeK kROPXAYKZgsbHWInHITwAjEIk5Ck7E68x1rSWjCXc8ipFnndTQJzSUwLnGzGvqa5 hm0UrsAQMC2VRS0qc1BeSCXgFPeUcCJDPkMCHVGMceEZhTIyMYlT3sweJ6issx0r Gxqjpyu41A/4VNIsFwGD93Sfooa2QBwNvWXdunYnQ+t+IiSw/UPC35aXuc1DDCY+ Oxe9+WbyXVv/zeU/WqVSFB51VNUkKinDpuuSKrYCWXEPV2GeC4GwPUYhnnL9bCHz EvSIsUMVgu44fLY3rRh0f+JloN5VZB+uRCA== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-transfer-encoding :content-type:content-type:date:date:feedback-id:feedback-id :from:from:in-reply-to:in-reply-to:message-id:mime-version :references:reply-to:sender:subject:subject:to:to:x-me-proxy :x-me-proxy:x-me-sender:x-me-sender:x-sasl-enc; s=fm3; t= 1683758233; x=1683844633; bh=+x0z7KOoxUIEVb60090u6mQJC43vuIbIkTA wp386lVo=; b=PSvYHDgDJk3k0EKVcvDKoWJZps4YYJ16PniLTWbXe53ztcRvrt4 8sHXOlUApovG6tIFVyojiXoSAc8VieUyQ3YYVvDcSoWbsO9EkJrRnMPtS+dicOF/ iHogR8Z2o3bBzwnJewTOS30DzE8QJb5A37091Pu5XM/diJHIcRcLHzeTDWYJB8NU BwOwTFLd1Ug2lf9R7y26GIYF/lMsiTNVDg09k8tIrfQ04yP/S0U4W39dnRr1SZhD hRG75lHp/xoogQCjjVN4FXV5pkyJ5qQxVxSAa9GqeUfThht0ueiRvmjpf3GpOCQU UAotD8o48Uvt717Tbkfwsrtw7+YmZl9x9pw== X-ME-Sender: X-ME-Received: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvhedrfeegjedgudefucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhepkfffgggfuffvvehfhfgjtgfgsehtkeertddtfeejnecuhfhrohhmpeftvggs vggttggrucevrhgrnhcuoehrvggsvggttggrsegsshguihhordgtohhmqeenucggtffrrg htthgvrhhnpeegfeegveduheejgfduffefhfehleehiefghfetvdejvdelhfeukefhhfdv geehveenucevlhhushhtvghrufhiiigvpedtnecurfgrrhgrmhepmhgrihhlfhhrohhmpe hrvggsvggttggrsegsshguihhordgtohhm X-ME-Proxy: Feedback-ID: i5b994698:Fastmail Received: by mail.messagingengine.com (Postfix) with ESMTPA; Wed, 10 May 2023 18:37:13 -0400 (EDT) Message-ID: Date: Wed, 10 May 2023 16:37:12 -0600 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.10.0 Subject: Re: [PATCH edk2-platforms 1/1] Silicon/Marvell: Fix setting I2cStatus in MvI2cEnableConf To: Marcin Wojtas Cc: devel@edk2.groups.io, Leif Lindholm References: <20230510153756.1885992-1-rebecca@bsdio.com> From: "Rebecca Cran" In-Reply-To: Content-Language: en-US Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit Pushed as 1f8ff3b1c03fab1fc5af285c2d939904e560c1c2. On 5/10/23 15:56, Marcin Wojtas wrote: > śr., 10 maj 2023 o 17:38 Rebecca Cran napisał(a): >> Fix setting I2cStatus in MvI2cEnableConf. >> >> Signed-off-by: Rebecca Cran >> --- >> Silicon/Marvell/Drivers/I2c/MvI2cDxe/MvI2cDxe.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/Silicon/Marvell/Drivers/I2c/MvI2cDxe/MvI2cDxe.c b/Silicon/Marvell/Drivers/I2c/MvI2cDxe/MvI2cDxe.c >> index b4dda56bbd2f..a8ff722aab22 100755 >> --- a/Silicon/Marvell/Drivers/I2c/MvI2cDxe/MvI2cDxe.c >> +++ b/Silicon/Marvell/Drivers/I2c/MvI2cDxe/MvI2cDxe.c >> @@ -777,7 +777,7 @@ MvI2cEnableConf ( >> { >> /* do nothing */ >> if (I2cStatus != NULL) >> - I2cStatus = EFI_SUCCESS; >> + *I2cStatus = EFI_SUCCESS; >> if (Event != NULL) >> gBS->SignalEvent(Event); >> return EFI_SUCCESS; >> -- >> 2.39.2 >> > Reviewed-by: Marcin Wojtas > > Thanks! > Marcin