From nobody Sat Nov 2 12:22:32 2024 Delivered-To: importer@patchew.org Received-SPF: none (zoho.com: 198.145.21.10 is neither permitted nor denied by domain of lists.01.org) client-ip=198.145.21.10; envelope-from=edk2-devel-bounces@lists.01.org; helo=ml01.01.org; Authentication-Results: mx.zoho.com; spf=none (zoho.com: 198.145.21.10 is neither permitted nor denied by domain of lists.01.org) smtp.mailfrom=edk2-devel-bounces@lists.01.org; Return-Path: Received: from ml01.01.org (ml01.01.org [198.145.21.10]) by mx.zohomail.com with SMTPS id 1489375451075530.5602581447943; Sun, 12 Mar 2017 20:24:11 -0700 (PDT) Received: from [127.0.0.1] (localhost [IPv6:::1]) by ml01.01.org (Postfix) with ESMTP id 9F9BB8038D; Sun, 12 Mar 2017 20:24:09 -0700 (PDT) Received: from mga06.intel.com (mga06.intel.com [134.134.136.31]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id D287080388 for ; Sun, 12 Mar 2017 20:24:07 -0700 (PDT) Received: from fmsmga006.fm.intel.com ([10.253.24.20]) by orsmga104.jf.intel.com with ESMTP; 12 Mar 2017 20:24:07 -0700 Received: from shwdeftian.ccr.corp.intel.com ([10.239.158.36]) by fmsmga006.fm.intel.com with ESMTP; 12 Mar 2017 20:24:06 -0700 X-Original-To: edk2-devel@lists.01.org X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.36,156,1486454400"; d="scan'208";a="75826315" From: Feng Tian To: edk2-devel@lists.01.org Date: Mon, 13 Mar 2017 11:23:57 +0800 Message-Id: X-Mailer: git-send-email 2.7.1.windows.2 Subject: [edk2] [patch] MdeModulePkg/SdMmc: Add break to avoid dead loop when polling OCR Reg X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Hao Wu MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Errors-To: edk2-devel-bounces@lists.01.org Sender: "edk2-devel" X-ZohoMail: RSF_4 Z_629925259 SPT_0 Content-Type: text/plain; charset="utf-8" At worst case, OCR register may always not set BIT31. It will cause original code enter to dead loop. Adding a break for such case. Cc: Hao Wu Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Feng Tian Reviewed-by: Hao Wu --- MdeModulePkg/Bus/Pci/SdMmcPciHcDxe/EmmcDevice.c | 9 ++++++++- MdeModulePkg/Bus/Pci/SdMmcPciHcDxe/SdDevice.c | 8 ++++++++ MdeModulePkg/Bus/Sd/EmmcBlockIoPei/EmmcHci.c | 9 ++++++++- MdeModulePkg/Bus/Sd/SdBlockIoPei/SdHci.c | 9 ++++++++- 4 files changed, 32 insertions(+), 3 deletions(-) diff --git a/MdeModulePkg/Bus/Pci/SdMmcPciHcDxe/EmmcDevice.c b/MdeModulePkg= /Bus/Pci/SdMmcPciHcDxe/EmmcDevice.c index 9dbec10..6653a87 100755 --- a/MdeModulePkg/Bus/Pci/SdMmcPciHcDxe/EmmcDevice.c +++ b/MdeModulePkg/Bus/Pci/SdMmcPciHcDxe/EmmcDevice.c @@ -1109,6 +1109,7 @@ EmmcIdentification ( EFI_SD_MMC_PASS_THRU_PROTOCOL *PassThru; UINT32 Ocr; UINT16 Rca; + UINTN Retry; =20 PciIo =3D Private->PciIo; PassThru =3D &Private->PassThru; @@ -1119,7 +1120,8 @@ EmmcIdentification ( return Status; } =20 - Ocr =3D 0; + Ocr =3D 0; + Retry =3D 0; do { Status =3D EmmcGetOcr (PassThru, Slot, &Ocr); if (EFI_ERROR (Status)) { @@ -1127,6 +1129,11 @@ EmmcIdentification ( return Status; } Ocr |=3D BIT30; + + if (Retry++ =3D=3D 100) { + return EFI_DEVICE_ERROR;=09 + } + gBS->Stall(10 * 1000); } while ((Ocr & BIT31) =3D=3D 0); =20 Status =3D EmmcGetAllCid (PassThru, Slot); diff --git a/MdeModulePkg/Bus/Pci/SdMmcPciHcDxe/SdDevice.c b/MdeModulePkg/B= us/Pci/SdMmcPciHcDxe/SdDevice.c index 9122848..1da0849 100644 --- a/MdeModulePkg/Bus/Pci/SdMmcPciHcDxe/SdDevice.c +++ b/MdeModulePkg/Bus/Pci/SdMmcPciHcDxe/SdDevice.c @@ -1017,6 +1017,7 @@ SdCardIdentification ( UINT8 PowerCtrl; UINT32 PresentState; UINT8 HostCtrl2; + UINTN Retry; =20 PciIo =3D Private->PciIo; PassThru =3D &Private->PassThru; @@ -1097,12 +1098,19 @@ SdCardIdentification ( // Note here we only support the cards complied with SD physical // layer simplified spec version 2.0 and version 3.0 and above. // + Ocr =3D 0; + Retry =3D 0; do { Status =3D SdCardSendOpCond (PassThru, Slot, 0, Ocr, S18r, Xpc, TRUE, = &Ocr); if (EFI_ERROR (Status)) { DEBUG ((DEBUG_ERROR, "SdCardIdentification: SdCardSendOpCond fails w= ith %r Ocr %x, S18r %x, Xpc %x\n", Status, Ocr, S18r, Xpc)); return EFI_DEVICE_ERROR; } + + if (Retry++ =3D=3D 100) { + return EFI_DEVICE_ERROR;=09 + } + gBS->Stall(10 * 1000); } while ((Ocr & BIT31) =3D=3D 0); =20 // diff --git a/MdeModulePkg/Bus/Sd/EmmcBlockIoPei/EmmcHci.c b/MdeModulePkg/Bu= s/Sd/EmmcBlockIoPei/EmmcHci.c index 2c0baca..5cb20a3 100644 --- a/MdeModulePkg/Bus/Sd/EmmcBlockIoPei/EmmcHci.c +++ b/MdeModulePkg/Bus/Sd/EmmcBlockIoPei/EmmcHci.c @@ -2827,6 +2827,7 @@ EmmcPeimIdentification ( EFI_STATUS Status; UINT32 Ocr; UINT32 Rca; + UINTN Retry; =20 Status =3D EmmcPeimReset (Slot); if (EFI_ERROR (Status)) { @@ -2834,13 +2835,19 @@ EmmcPeimIdentification ( return Status; } =20 - Ocr =3D 0; + Ocr =3D 0; + Retry =3D 0; do { Status =3D EmmcPeimGetOcr (Slot, &Ocr); if (EFI_ERROR (Status)) { DEBUG ((EFI_D_ERROR, "EmmcPeimIdentification: EmmcPeimGetOcr fails w= ith %r\n", Status)); return Status; } + + if (Retry++ =3D=3D 100) { + return EFI_DEVICE_ERROR;=09 + } + MicroSecondDelay (10 * 1000); } while ((Ocr & BIT31) =3D=3D 0); =20 Status =3D EmmcPeimGetAllCid (Slot); diff --git a/MdeModulePkg/Bus/Sd/SdBlockIoPei/SdHci.c b/MdeModulePkg/Bus/Sd= /SdBlockIoPei/SdHci.c index 23e6563..81076ba 100644 --- a/MdeModulePkg/Bus/Sd/SdBlockIoPei/SdHci.c +++ b/MdeModulePkg/Bus/Sd/SdBlockIoPei/SdHci.c @@ -2754,7 +2754,7 @@ SdPeimIdentification ( UINT32 PresentState; UINT8 HostCtrl2; SD_HC_SLOT_CAP Capability; - + UINTN Retry; // // 1. Send Cmd0 to the device // @@ -2842,12 +2842,19 @@ SdPeimIdentification ( // Note here we only support the cards complied with SD physical // layer simplified spec version 2.0 and version 3.0 and above. // + Ocr =3D 0; + Retry =3D 0; do { Status =3D SdPeimSendOpCond (Slot, 0, Ocr, S18r, Xpc, TRUE, &Ocr); if (EFI_ERROR (Status)) { DEBUG ((EFI_D_ERROR, "SdPeimIdentification: SdPeimSendOpCond fails w= ith %r Ocr %x, S18r %x, Xpc %x\n", Status, Ocr, S18r, Xpc)); return EFI_DEVICE_ERROR; } + + if (Retry++ =3D=3D 100) { + return EFI_DEVICE_ERROR;=09 + } + MicroSecondDelay (10 * 1000); } while ((Ocr & BIT31) =3D=3D 0); =20 // --=20 2.7.1.windows.2 _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel