From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Permerror (SPF Permanent Error: Two or more type TXT spf records found.) identity=mailfrom; client-ip=134.134.136.20; helo=mga02.intel.com; envelope-from=star.zeng@intel.com; receiver=edk2-devel@lists.01.org Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) (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 CA031210FC0BD for ; Mon, 3 Sep 2018 18:30:49 -0700 (PDT) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga006.jf.intel.com ([10.7.209.51]) by orsmga101.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 03 Sep 2018 18:30:49 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.53,327,1531810800"; d="scan'208";a="71351145" Received: from fmsmsx104.amr.corp.intel.com ([10.18.124.202]) by orsmga006.jf.intel.com with ESMTP; 03 Sep 2018 18:30:44 -0700 Received: from fmsmsx116.amr.corp.intel.com (10.18.116.20) by fmsmsx104.amr.corp.intel.com (10.18.124.202) with Microsoft SMTP Server (TLS) id 14.3.319.2; Mon, 3 Sep 2018 18:30:44 -0700 Received: from shsmsx104.ccr.corp.intel.com (10.239.4.70) by fmsmsx116.amr.corp.intel.com (10.18.116.20) with Microsoft SMTP Server (TLS) id 14.3.319.2; Mon, 3 Sep 2018 18:30:43 -0700 Received: from shsmsx102.ccr.corp.intel.com ([169.254.2.226]) by SHSMSX104.ccr.corp.intel.com ([169.254.5.143]) with mapi id 14.03.0319.002; Tue, 4 Sep 2018 09:27:44 +0800 From: "Zeng, Star" To: Laszlo Ersek , edk2-devel-01 CC: Gerd Hoffmann , "Wang, Jian J" , "Ni, Ruiyu" , "Shi, Steven" , "Zeng, Star" Thread-Topic: [PATCH] MdeModulePkg/EhciDxe: fix host controller reset condition in BindingStart Thread-Index: AQHUQTNuAy95qhbdEkexZhGyLikIpqTeP02A//+vewCAAWrUYA== Date: Tue, 4 Sep 2018 01:27:43 +0000 Message-ID: <0C09AFA07DD0434D9E2A0C6AEB0483103BBB4E60@shsmsx102.ccr.corp.intel.com> References: <20180831140330.26888-1-lersek@redhat.com> <0C09AFA07DD0434D9E2A0C6AEB0483103BBB4C09@shsmsx102.ccr.corp.intel.com> In-Reply-To: Accept-Language: zh-CN, en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-ctpclassification: CTP_NT x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiOTc1OWQ0YTMtNTExYi00NWQzLTllZmQtOGQ3MTRlNjdlOTY0IiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE3LjEwLjE4MDQuNDkiLCJUcnVzdGVkTGFiZWxIYXNoIjoiOW44QWFlR3lcL3hkdXd5eXFTRDBURmpjTElybGRzRjB3aFc1RVpzTGVjRkFvd29CbTlTOTFOOEZrSnBrZWRUcE4ifQ== dlp-product: dlpe-windows dlp-version: 11.0.400.15 dlp-reaction: no-action x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Subject: Re: [PATCH] MdeModulePkg/EhciDxe: fix host controller reset condition in BindingStart X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 04 Sep 2018 01:30:50 -0000 Content-Language: en-US Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 T2gsIHllcy4gR290IHRoZSBwb2ludCwgdGhlIHBhdGNoIGlzIGNvcnJlY3QgaW5kZWVkLiA6KQ0K VGhhbmtzIGZvciB0aGUgZXhwbGFuYXRpb24uDQoNCk1heWJlLCBhbiBhYnN0cmFjdGVkIGZ1bmN0 aW9uIHdpbGwgbWFrZSB0aGUgbG9naWMgbW9yZSBjbGVhciwgbGlrZSBFaGNJc0RlYnVnUG9ydElu VXNlKCkuDQpXaGF0IGRvIHlvdSB0aGluaz8NCg0KDQpUaGFua3MsDQpTdGFyDQotLS0tLU9yaWdp bmFsIE1lc3NhZ2UtLS0tLQ0KRnJvbTogTGFzemxvIEVyc2VrIFttYWlsdG86bGVyc2VrQHJlZGhh dC5jb21dIA0KU2VudDogTW9uZGF5LCBTZXB0ZW1iZXIgMywgMjAxOCA3OjQ2IFBNDQpUbzogWmVu ZywgU3RhciA8c3Rhci56ZW5nQGludGVsLmNvbT47IGVkazItZGV2ZWwtMDEgPGVkazItZGV2ZWxA bGlzdHMuMDEub3JnPg0KQ2M6IEdlcmQgSG9mZm1hbm4gPGtyYXhlbEByZWRoYXQuY29tPjsgV2Fu ZywgSmlhbiBKIDxqaWFuLmoud2FuZ0BpbnRlbC5jb20+OyBOaSwgUnVpeXUgPHJ1aXl1Lm5pQGlu dGVsLmNvbT47IFNoaSwgU3RldmVuIDxzdGV2ZW4uc2hpQGludGVsLmNvbT4NClN1YmplY3Q6IFJl OiBbUEFUQ0hdIE1kZU1vZHVsZVBrZy9FaGNpRHhlOiBmaXggaG9zdCBjb250cm9sbGVyIHJlc2V0 IGNvbmRpdGlvbiBpbiBCaW5kaW5nU3RhcnQNCg0KSGkgU3RhciwNCg0KT24gMDkvMDMvMTggMTA6 MzUsIFplbmcsIFN0YXIgd3JvdGU6DQo+IEdvb2QgZmluZGluZy4NCj4NCj4gVGhlcmUgaXMgYW5v dGhlciBwbGFjZSBhdA0KPiBodHRwczovL2dpdGh1Yi5jb20vdGlhbm9jb3JlL2VkazIvYmxvYi9t YXN0ZXIvTWRlTW9kdWxlUGtnL0J1cy9QY2kvRWhjDQo+IGlEeGUvRWhjaS5jI0wxNTAgd2hpY2gg aGFzIHNpbWlsYXIgbG9naWMsIHBsZWFzZSB1cGRhdGUgaXQgYWxzby4NCg0KSSBhdWRpdGVkIGFs bCB0aGUgIkRlYnVnUG9ydE51bSIgY2hlY2tzIHRoYXQgd2VyZSBhZGRlZCBieSBjb21taXQgMDk5 NDNmNWVjYzBmLCBoZXJlOg0KDQogIGh0dHBzOi8vYnVnemlsbGEudGlhbm9jb3JlLm9yZy9zaG93 X2J1Zy5jZ2k/aWQ9MTEyOSNjMTYNCg0KVGhlIGxvY2F0aW9uIHRoYXQgeW91IG1lbnRpb24gaXMg aW4gdGhlIEVoY1Jlc2V0KCkgZnVuY3Rpb24uIEl0IGlzIGNvdmVyZWQgaW4gbXkgYW5hbHlzaXMg bGlua2VkIGFib3ZlLCBhbmQgaXQgbmVlZHMgbm8gdXBkYXRlLCBiZWNhdXNlIGl0IGlzIGNvcnJl Y3QuDQoNCk5hbWVseSwgaW4gRWhjUmVzZXQoKSwgdGhlIGNvbmRpdGlvbiBjYXB0dXJlcyB3aGVu IHRoZSByZXNldCBtdXN0IGJlICpza2lwcGVkKi4gRm9yIHRoYXQsIHRoZSBjb25kaXRpb24gaXMs DQoNCiAgb25lIG9mIHRoZSBVU0IgcG9ydHMgb24gdGhlIGhvc3QgY29udHJvbGxlciBpcyBhIGRl YnVnIHBvcnQsIEFORA0KICB0aGUgZGVidWcgcG9ydCBpcyBpbiB1c2UNCg0KV2hlbiB0aGlzIGNv bmRpdGlvbiBob2xkcywgd2Ugc2V0IFN0YXR1cyB0byBFRklfU1VDQ0VTUywgYW5kIGp1bXAgdG8g T05fRVhJVCAodGhhdCBpcywgd2Ugc2tpcCB0aGUgcmUtc2V0IGFuZCByZXR1cm4gc3VjY2VzcyB3 aXRob3V0IGRvaW5nIGFueXRoaW5nKS4gVGhpcyBpcyB3cml0dGVuIGluIEMgYXMNCg0KPiAgICAg aWYgKEVoYy0+RGVidWdQb3J0TnVtICE9IDApIHsNCj4gICAgICAgRGJnQ3RybFN0YXR1cyA9IEVo Y1JlYWREYmdSZWdpc3RlcihFaGMsIDApOw0KPiAgICAgICBpZiAoKERiZ0N0cmxTdGF0dXMgJiAo VVNCX0RFQlVHX1BPUlRfSU5fVVNFIHwgVVNCX0RFQlVHX1BPUlRfT1dORVIpKSA9PSAoVVNCX0RF QlVHX1BPUlRfSU5fVVNFIHwgVVNCX0RFQlVHX1BPUlRfT1dORVIpKSB7DQo+ICAgICAgICAgU3Rh dHVzID0gRUZJX1NVQ0NFU1M7DQo+ICAgICAgICAgZ290byBPTl9FWElUOw0KPiAgICAgICB9DQo+ ICAgICB9DQoNCkhvd2V2ZXIsIGF0IHRoZSBsb2NhdGlvbiB0aGF0IHRoZSBwYXRjaCBtb2RpZmll cywgdGhlIGNvbmRpdGlvbiByZWZlcnMgdG8gdGhlICpvcHBvc2l0ZSogY2FzZSwgdGhhdCBpcywg d2hlbiB0aGUgcmVzZXQgbXVzdCAqbm90KiBiZSBza2lwcGVkLg0KRm9yIHRoYXQsIHdlIGhhdmUg dG8gbmVnYXRlIHRoZSBjb25kaXRpb24gc2VlbiBhYm92ZS4NCg0KRm9yIHRoZSBuZWdhdGlvbiwg d2UgdXNlIERlIE1vcmdhbidzIExhd3MsIHRoYXQgaXM6DQoNCiAgIShBICYmIEIpID09ICghQSB8 fCAhQikNCg0KVGhhdCBpczoNCg0KICAqbm8qIFVTQiBwb3J0IG9uIHRoZSBob3N0IGNvbnRyb2xs ZXIgaXMgYSBkZWJ1ZyBwb3J0LCAqT1IqDQogIHRoZSBkZWJ1ZyBwb3J0IGlzICpub3QqIGluIHVz ZQ0KDQpBbmQgdGhpcyBpcyB3cml0dGVuLCBpbiBDLCBhcw0KDQo+ICAgaWYgKEVoYy0+RGVidWdQ b3J0TnVtID09IDApIHsNCj4gICAgIEVoY1Jlc2V0SEMgKEVoYywgRUhDX1JFU0VUX1RJTUVPVVQp Ow0KPiAgIH0gZWxzZSB7DQo+ICAgICBTdGF0ZSA9IEVoY1JlYWREYmdSZWdpc3RlcihFaGMsIDAp Ow0KPiAgICAgaWYgKChTdGF0ZSAmIChVU0JfREVCVUdfUE9SVF9JTl9VU0UgfCBVU0JfREVCVUdf UE9SVF9PV05FUikpICE9IChVU0JfREVCVUdfUE9SVF9JTl9VU0UgfCBVU0JfREVCVUdfUE9SVF9P V05FUikpIHsNCj4gICAgICAgRWhjUmVzZXRIQyAoRWhjLCBFSENfUkVTRVRfVElNRU9VVCk7DQo+ ICAgICB9DQo+ICAgfQ0KDQpUaGUgb3JpZ2luYWwgY29tbWl0ICgwOTk0M2Y1ZWNjMGYpIGdvdCB0 aGUgRWhjUmVzZXQoKSBtb2RpZmljYXRpb24gcmlnaHQgKHRoYXQgaXMsIHRoZSBjb25kaXRpb24g aXMgYWxyZWFkeSBjb3JyZWN0KTsgd2hhdCBpdCBnb3Qgd3Jvbmcgd2FzIHRoZSBuZWdhdGlvbiBv ZiB0aGUgY29uZGl0aW9uLCBmb3IgRWhjRHJpdmVyQmluZGluZ1N0YXJ0KCkuIFRoYXQncyB0aGUg b25seSB0aGluZyB3ZSBoYXZlIHRvIGNvcnJlY3QgaW4gdGhpcyBwYXRjaC4NCg0KVGhhbmtzLA0K TGFzemxvDQoNCg0KPg0KPg0KPiBUaGFua3MsDQo+IFN0YXINCj4gLS0tLS1PcmlnaW5hbCBNZXNz YWdlLS0tLS0NCj4gRnJvbTogTGFzemxvIEVyc2VrIFttYWlsdG86bGVyc2VrQHJlZGhhdC5jb21d DQo+IFNlbnQ6IEZyaWRheSwgQXVndXN0IDMxLCAyMDE4IDEwOjA0IFBNDQo+IFRvOiBlZGsyLWRl dmVsLTAxIDxlZGsyLWRldmVsQGxpc3RzLjAxLm9yZz4NCj4gQ2M6IEdlcmQgSG9mZm1hbm4gPGty YXhlbEByZWRoYXQuY29tPjsgV2FuZywgSmlhbiBKIA0KPiA8amlhbi5qLndhbmdAaW50ZWwuY29t PjsgTmksIFJ1aXl1IDxydWl5dS5uaUBpbnRlbC5jb20+OyBaZW5nLCBTdGFyIA0KPiA8c3Rhci56 ZW5nQGludGVsLmNvbT47IFNoaSwgU3RldmVuIDxzdGV2ZW4uc2hpQGludGVsLmNvbT4NCj4gU3Vi amVjdDogW1BBVENIXSBNZGVNb2R1bGVQa2cvRWhjaUR4ZTogZml4IGhvc3QgY29udHJvbGxlciBy ZXNldCANCj4gY29uZGl0aW9uIGluIEJpbmRpbmdTdGFydA0KPg0KPiBDb21taXQgMDk5NDNmNWVj YzBmICgiTWRlTW9kdWxlUGtnOiBTa2lwIHRvIG1hbmFnZSB1c2IgZGVidWcgcG9ydCBpbiANCj4g RURLSUkgRUhDSSBkcml2ZXIgaWYgaXQncyB1c2VkIGJ5IHVzYiBkZWJ1ZyBwb3J0IGRyaXZlciIs IDIwMTItMDQtMjgpIA0KPiBtYWRlIHRoZSBob3N0IGNvbnRyb2xsZXIgcmVzZXQgaW4gRWhjRHJp dmVyQmluZGluZ1N0YXJ0KCkgY29uZGl0aW9uYWwuIA0KPiBUaGUgaW50ZW50IHdhcyB0byBhdm9p ZCB0aGUgcmVzZXQgd2hlbg0KPiAtIG9uZSBvZiB0aGUgVVNCIHBvcnRzIG9uIHRoZSBob3N0IGNv bnRyb2xsZXIgd2FzIGEgRGVidWcgUG9ydCwgQU5EDQo+IC0gdGhlIERlYnVnIFBvcnQgd2FzIGlu IHVzZS4NCj4NCj4gVGhpcyB0cmFuc2xhdGVzIHRvIHRoZSBmb2xsb3dpbmcgcG9zaXRpdmUgY29u ZGl0aW9uOiByZXNldCB0aGUgY29udHJvbGxlciBvbmx5IGlmOg0KPiAtIG5vIFVTQiBwb3J0IG9u IHRoZSBob3N0IGNvbnRyb2xsZXIgaXMgYSBEZWJ1ZyBQb3J0LCBPUg0KPiAtIHRoZSBEZWJ1ZyBQ b3J0IGlzIG5vdCBpbiB1c2UuDQo+DQo+IENvbW1pdCAwOTk0M2Y1ZWNjMGYgZmFpbGVkIHRvIGlt cGxlbWVudCB0aGUgZmlyc3Qgc3ViY29uZGl0aW9uLCBhbmQgdGh1cyB0aGUgcmVzdWx0IGlzIHRv byBzdHJpY3Qgbm93IChmb3IgcmVzZXR0aW5nIHRoZSBob3N0IGNvbnRyb2xsZXIpLiBSZWxheCBp dCB0byB0aGUgY29ycmVjdCBjb25kaXRpb24uDQo+DQo+IFRoaXMgaXNzdWUgd2FzIGZvdW5kIGJ5 IFN0ZXZlbiBTaGkgb24gUUVNVS4gUUVNVSdzIEVIQ0kgZGV2aWNlIG1vZGVsIGRvZXMgbm90IGhh dmUgYSBEZWJ1ZyBQb3J0LiBJbiByZXBlYXRlZCBkaXNjb25uZWN0IC8gY29ubmVjdCBjeWNsZXMs IHRoZSBjb250cm9sbGVyIGlzIG5ldmVyIHJlc2V0IGluIEVoY0RyaXZlckJpbmRpbmdTdGFydCgp LCB0aGVyZWZvcmUgdGhlIGRldmljZXMgb24gdGhlIGNvbnRyb2xsZXIgYXJlIG5ldmVyIHJlLWVu dW1lcmF0ZWQgYWZ0ZXIgYSBkaXNjb25uZWN0Lg0KPg0KPiBDYzogR2VyZCBIb2ZmbWFubiA8a3Jh eGVsQHJlZGhhdC5jb20+DQo+IENjOiBKaWFuIEogV2FuZyA8amlhbi5qLndhbmdAaW50ZWwuY29t Pg0KPiBDYzogUnVpeXUgTmkgPHJ1aXl1Lm5pQGludGVsLmNvbT4NCj4gQ2M6IFN0YXIgWmVuZyA8 c3Rhci56ZW5nQGludGVsLmNvbT4NCj4gQ2M6IFN0ZXZlbiBTaGkgPHN0ZXZlbi5zaGlAaW50ZWwu Y29tPg0KPiBSZXBvcnRlZC1ieTogU3RldmVuIFNoaSA8c3RldmVuLnNoaUBpbnRlbC5jb20+DQo+ IFJlZjogaHR0cHM6Ly9idWd6aWxsYS50aWFub2NvcmUub3JnL3Nob3dfYnVnLmNnaT9pZD0xMTI5 DQo+IEZpeGVzOiAwOTk0M2Y1ZWNjMGZiYzBjOThjNTExYzgyNzAzYTBiYTNiMmI1ODE5DQo+IENv bnRyaWJ1dGVkLXVuZGVyOiBUaWFub0NvcmUgQ29udHJpYnV0aW9uIEFncmVlbWVudCAxLjENCj4g U2lnbmVkLW9mZi1ieTogTGFzemxvIEVyc2VrIDxsZXJzZWtAcmVkaGF0LmNvbT4NCj4gLS0tDQo+ DQo+IE5vdGVzOg0KPiAgICAgUmVwbzogICBodHRwczovL2dpdGh1Yi5jb20vbGVyc2VrL2VkazIu Z2l0DQo+ICAgICBCcmFuY2g6IGVoY2lfc3RhcnRfcmVzZXRfMTEyOQ0KPg0KPiAgTWRlTW9kdWxl UGtnL0J1cy9QY2kvRWhjaUR4ZS9FaGNpLmMgfCA0ICsrKy0NCj4gIDEgZmlsZSBjaGFuZ2VkLCAz IGluc2VydGlvbnMoKyksIDEgZGVsZXRpb24oLSkNCj4NCj4gZGlmZiAtLWdpdCBhL01kZU1vZHVs ZVBrZy9CdXMvUGNpL0VoY2lEeGUvRWhjaS5jIA0KPiBiL01kZU1vZHVsZVBrZy9CdXMvUGNpL0Vo Y2lEeGUvRWhjaS5jDQo+IGluZGV4IDMwYWQyYjJmZmVlZi4uODllZDAzNGI5MDkzIDEwMDY0NA0K PiAtLS0gYS9NZGVNb2R1bGVQa2cvQnVzL1BjaS9FaGNpRHhlL0VoY2kuYw0KPiArKysgYi9NZGVN b2R1bGVQa2cvQnVzL1BjaS9FaGNpRHhlL0VoY2kuYw0KPiBAQCAtMTkxNiwxMSArMTkxNiwxMyBA QCBFaGNEcml2ZXJCaW5kaW5nU3RhcnQgKA0KPiAgICAvLw0KPiAgICBpZiAoRmVhdHVyZVBjZEdl dCAoUGNkVHVybk9mZlVzYkxlZ2FjeVN1cHBvcnQpKSB7DQo+ICAgICAgRWhjQ2xlYXJMZWdhY3lT dXBwb3J0IChFaGMpOw0KPiAgICB9DQo+DQo+IC0gIGlmIChFaGMtPkRlYnVnUG9ydE51bSAhPSAw KSB7DQo+ICsgIGlmIChFaGMtPkRlYnVnUG9ydE51bSA9PSAwKSB7DQo+ICsgICAgRWhjUmVzZXRI QyAoRWhjLCBFSENfUkVTRVRfVElNRU9VVCk7ICB9IGVsc2Ugew0KPiAgICAgIFN0YXRlID0gRWhj UmVhZERiZ1JlZ2lzdGVyKEVoYywgMCk7DQo+ICAgICAgaWYgKChTdGF0ZSAmIChVU0JfREVCVUdf UE9SVF9JTl9VU0UgfCBVU0JfREVCVUdfUE9SVF9PV05FUikpICE9IChVU0JfREVCVUdfUE9SVF9J Tl9VU0UgfCBVU0JfREVCVUdfUE9SVF9PV05FUikpIHsNCj4gICAgICAgIEVoY1Jlc2V0SEMgKEVo YywgRUhDX1JFU0VUX1RJTUVPVVQpOw0KPiAgICAgIH0NCj4gICAgfQ0KPiAtLQ0KPiAyLjE0LjEu My5nYjdjZjZlMDI0MDFiDQo+DQoNCg==