From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=192.55.52.88; helo=mga01.intel.com; envelope-from=benjamin.you@intel.com; receiver=edk2-devel@lists.01.org Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) (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 D89622222C224 for ; Sun, 28 Jan 2018 00:43:38 -0800 (PST) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga004.jf.intel.com ([10.7.209.38]) by fmsmga101.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 28 Jan 2018 00:49:10 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.46,426,1511856000"; d="scan'208";a="169730745" Received: from fmsmsx104.amr.corp.intel.com ([10.18.124.202]) by orsmga004.jf.intel.com with ESMTP; 28 Jan 2018 00:49:09 -0800 Received: from fmsmsx115.amr.corp.intel.com (10.18.116.19) by fmsmsx104.amr.corp.intel.com (10.18.124.202) with Microsoft SMTP Server (TLS) id 14.3.319.2; Sun, 28 Jan 2018 00:49:09 -0800 Received: from shsmsx101.ccr.corp.intel.com (10.239.4.153) by fmsmsx115.amr.corp.intel.com (10.18.116.19) with Microsoft SMTP Server (TLS) id 14.3.319.2; Sun, 28 Jan 2018 00:49:09 -0800 Received: from shsmsx102.ccr.corp.intel.com ([169.254.2.124]) by SHSMSX101.ccr.corp.intel.com ([169.254.1.192]) with mapi id 14.03.0319.002; Sun, 28 Jan 2018 16:49:07 +0800 From: "You, Benjamin" To: Nico Huber , "edk2-devel@lists.01.org" CC: Arthur Heymans Thread-Topic: [edk2] [PATCH] CorebootPayloadPkg: Use correct BytesPerScanLine Thread-Index: AQHTloVEbO9yM5GE4kKb+ZrPyeBPYKOHEbGAgAAtWwCAAbLkYA== Date: Sun, 28 Jan 2018 08:49:07 +0000 Message-ID: References: <20180124105736.14877-1-arthur@aheymans.xyz> <87mv12gvjc.fsf@aheymans.xyz> <87efmdyokj.fsf@aheymans.xyz> In-Reply-To: Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-ctpclassification: CTP_NT x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiN2ExNmJkM2QtNDM2NC00ODgwLWEwZDMtYzI3NzdlYWZmYzNjIiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE3LjIuNS4xOCIsIlRydXN0ZWRMYWJlbEhhc2giOiJlSk5sNFNGQWxNd0VQSjhOdXdUM3FxejI5WG0ycnphK1NmOXc1VExpREE4YkZXMjVOQWpya3hDOEUrclhZRWNwIn0= dlp-product: dlpe-windows dlp-version: 11.0.0.116 dlp-reaction: no-action x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Subject: Re: [PATCH] CorebootPayloadPkg: Use correct BytesPerScanLine X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 28 Jan 2018 08:43:39 -0000 Content-Language: en-US Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 SGkgTmljbywNCg0KVGhhbmtzIGZvciB0aGUgZGV0YWlsZWQgaW5mb3JtYXRpb24uIEl0IG1ha2Vz IHNlbnNlLiBJIGRvIGxpa2UgdGhlIGlkZWEgb2YNCmRvY3VtZW50aW5nIHRoZSBsYl9mcmFtZWJ1 ZmZlci4NCg0KPiBUaGUgb25seSBndWFyYW50ZWUgZm9yIGBieXRlc19wZXJfcGl4ZWxgICh0eXBv PyAnYnl0ZXNfcGVyX3NjYW5saW5lJykgDQo+IGFuZCBgeF9yZXNvbHV0aW9uYCB5b3UgZ2V0IGFz IGEgY29uc3VtZXIsIGlzIHRoYXQgdGhlIGZvcm1lciBpcyBiaWcgDQo+IGVub3VnaCB0byBob2xk IGB4X3Jlc29sdXRpb25gIHBpeGVscy4NCg0KSSB0aGluayBpdCB3b3VsZCBiZSBnb29kIHRvIGFs c28gZG9jdW1lbnQgdGhhdCB0aGUgY29uc3VtZXIgaXMgYXNzdXJlZCB0aGF0DQppbiBmcmFtZWJ1 ZmZlciwgYWxsIHRoZSAneF9yZXNvbHV0aW9uJyBwaXhlbHMgYXJlIGFsaWduZWQgYXQgdGhlIGJl Z2lubmluZw0Kb2YgZWFjaCBzY2FubGluZSwgYW5kIHRoZSBleHRyYSBieXRlcyBhcmUgYWx3YXlz IHBhZGRlZCBhZnRlciB0aGUgDQoneF9yZXNvbHV0aW9uJyBwaXhlbHMgaW4gdGhlIHNjYW5saW5l LiBXb3VsZCB0aGlzIGJlIHRydWUgd2l0aCBleGlzdGluZw0KZ3JhcGhpY3MgZGV2aWNlcz8gKEkg YW0gbm90IGV4cGVydCBpbiB0aGlzIGFyZWEgc28gSSdkIGxpa2UgdG8gY29uZmlybS4pDQoNCj4g LS0tLS1PcmlnaW5hbCBNZXNzYWdlLS0tLS0NCj4gRnJvbTogTmljbyBIdWJlciBbbWFpbHRvOm5p Y28uaEBnbXguZGVdDQo+IFNlbnQ6IFNhdHVyZGF5LCBKYW51YXJ5IDI3LCAyMDE4IDEwOjE0IFBN DQo+IFRvOiBZb3UsIEJlbmphbWluIDxiZW5qYW1pbi55b3VAaW50ZWwuY29tPg0KPiBDYzogQXJ0 aHVyIEhleW1hbnMgPGFydGh1ckBhaGV5bWFucy54eXo+OyBlZGsyLWRldmVsQGxpc3RzLjAxLm9y Zw0KPiBTdWJqZWN0OiBSZTogW2VkazJdIFtQQVRDSF0gQ29yZWJvb3RQYXlsb2FkUGtnOiBVc2Ug Y29ycmVjdCBCeXRlc1BlclNjYW5MaW5lDQo+IA0KPiBIZWxsbyBCZW4sDQo+IA0KPiBPbiAyNy4w MS4yMDE4IDA1OjExLCBiZW5qYW1pbi55b3UgYXQgaW50ZWwuY29tIChZb3UsIEJlbmphbWluKSB3 cm90ZToNCj4gPiBUaGUgZnVuY3Rpb25pbmcgd2lsbCBkZXBlbmQgb24gQ29yZWJvb3QgaW50ZXJw cmV0aW5nIHByb3Blcmx5IHRvby4gSG93ZXZlcg0KPiA+IGZpeGluZyB0aGUgUGF5bG9hZCB3aWxs IG5vdCBjYXVzZSBhbnkgcmVncmVzc2lvbiBhbnl3YXkuDQo+IA0KPiBJJ20gbm90IHN1cmUgd2hh dCB5b3UgbWVhbiBieSAiaW50ZXJwcmV0aW5nIHByb3Blcmx5Ii4gVGhlDQo+IGBieXRlc19wZXJf bGluZWAgZmllbGQgaGFzIHRvIG1hdGNoIHRoZSBoYXJkd2FyZSBjb25maWd1cmF0aW9uIG9mYy4N Cj4gQnV0IGl0IGlzIG5ldmVyIGludGVycHJldGVkIGJ5IGNvcmVib290LCBjb3JlYm9vdCBpcyB0 aGUgcHJvZHVjZXINCj4gb2YgdGhhdCB2YWx1ZSwgbmV2ZXIgdGhlIGNvbnN1bWVyLg0KPiANCj4g UGxlYXNlIGRvbid0IG1ha2UgdGhpcyBtb3JlIGNvbXBsaWNhdGVkIHRoYW4gaXQgaXMuIFRoZSBj b25zdW1lciBvZiB0aGUNCj4gZnJhbWVidWZmZXIgY29uZmlndXJhdGlvbiAoaW4gdGhpcyBjYXNl IFRpYW5vQ29yZSkgc2ltcGx5IGhhcyB0byBob25vcg0KPiB0aGUgYGJ5dGVzX3Blcl9saW5lYCBm aWVsZCBhbmQgc2hvdWxkbid0IG1ha2UgYW55IGFzc3VtcHRpb25zIGFib3V0IGl0DQo+ICh0aGVy ZSBpcyBubyBnZW5lcmljIHJ1bGUgaG93IHRvIGdldCBmcm9tIGFuIGB4X3Jlc29sdXRpb25gIHRv IGENCj4gYGJ5dGVzX3Blcl9saW5lYCB2YWx1ZSwgdGhlcmUgYXJlIG9ubHkgY29uc3RyYWludHMu IG90aGVyd2lzZSwgdGhlcmUNCj4gd291bGQgYmUgbm8gcmVhc29uIHRvIHBhc3MgYGJ5dGVzX3Bl cl9saW5lYCBhdCBhbGwpLg0KPiANCj4gSWYgeW91J2QgbGlrZSwgSSBjYW4gYWRkIHNvbWUgZG9j dW1lbnRhdGlvbiB0byBgc3RydWN0IGxiX2ZyYW1lYnVmZmVyYA0KPiAodGhhdCdzIGhvdyBpdCdz IGNhbGxlZCBpbiBjb3JlYm9vdCwgZG9uJ3Qga25vdyB0aGUgbmFtZSBpbiBUaWFub0NvcmUpLg0K PiBUaGUgY29uc3VtZXIgc2hvdWxkIGJlIG9ubHkgd3JpdHRlbiB3aXRoIHRoYXQgc3RydWN0IChh bmQgaXQncyBkb2N1bWVuLQ0KPiB0YXRpb24pIGluIG1pbmQgKGkuZS4gbm90IGJ5IGxvb2tpbmcg YXQgY29yZWJvb3QgaW50ZXJuYWwgY29kZSB0aGF0IG1heQ0KPiBjaGFuZ2UgYXQgYW55IHRpbWUp Lg0KPiANCj4gPiBJIGFtIHN0aWxsIG5vdCB2ZXJ5IGNsZWFyIGFib3V0IHNvbWUgY2FzZXMgaW4g Q29yZWJvb3QgYXMgYmVsb3c6DQo+ID4NCj4gPj4gVGhpcyBpcyBob3cgeF9yZXNvbHV0aW9uIGlu aXRpYWxseSBnZXRzIHNldCBhZnRlciB0aGUgRURJRCBpcyByZWFkLCBidXQNCj4gPj4gaXQgaXMg ZnVydGhlciBtb2RpZmllZCB0byBzYXRpc2Z5IHRoZSBkaXNwbGF5IGNvbnRyb2xsZXJzIG5lZWRz LA0KPiA+PiBlLmcuIHNyYy9ub3J0aGJyaWRnZS9pbnRlbC9nbTQ1L2dtYS5jOg0KPiA+Pg0KPiA+ PiBlZGlkLT5ieXRlc19wZXJfbGluZSA9IChlZGlkLT5ieXRlc19wZXJfbGluZSArIDYzKSAmIH42 MzsNCj4gPg0KPiA+IFRoaXMgbGluZSBkb2VzIG5vdCBjaGFuZ2UgdGhlIHZhbHVlIG9mIGVkaWQt PmJ5dGVzX3Blcl9saW5lIHNpbmNlIGl0IGlzDQo+ID4gYWxyZWFkeSByb3VuZGVkIHVwIHRvIDY0 IGJ5IHByZXZpb3VzIGNhbGN1bGF0aW9uIGluIGVkaWQuYzoNCj4gPg0KPiA+ICAgZWRpZC0+Ynl0 ZXNfcGVyX2xpbmUgPSBBTElHTl9VUChlZGlkLT5tb2RlLmhhICoNCj4gPiAgCQlkaXZfcm91bmRf dXAoZmJfYnBwLCA4KSwgcm93X2J5dGVfYWxpZ25tZW50KTsNCj4gDQo+IFRoZSBwaWVjZSBvZiBj b2RlIGluIGdtNDUvZ21hLmMgdXNlcyBgc3RydWN0IGVkaWRgIGFzIGFuIGludGVyZmFjZS4NCj4g VGhlcmUgaXMgbm8gZ3VhcmFudGVlIHRoYXQgdGhpcyBzdHJ1Y3QgcnVuIHRocm91Z2ggdGhlIGNv ZGUgaW4gZWRpZC5jLA0KPiBpdCBjb3VsZCBhbHNvIGhhdmUgYmVlbiBnZW5lcmF0ZWQgc29tZXdo ZXJlIGVsc2UuIEJ1dCB0aGUgaGFyZHdhcmUNCj4gY292ZXJlZCBieSBnbTQ1L2dtYS5jIGhhcyB0 aGlzIHJlcXVpcmVtZW50LCBzbyBpdCBoYXMgdG8gYmUgZG9uZSB0aGVyZS4NCj4gDQo+IE1vcmVv dmVyIHRoZSBjb2RlIGluIGVkaWQuYyBpcyBicm9rZW4gYW55d2F5OiBpdCBhbGlnbnMgeF9yZXNv bHV0aW9uDQo+IHVwIGFzIHdlbGwsIHdoaWNoIGlzIHdyb25nIChnZW5lcmFsbHkuIGl0IG1pZ2h0 IGJlIGNvcnJlY3QgZm9yIHNvbWUNCj4gaGFyZHdhcmUgYnV0IHRoYXQgc2hvdWxkIGJlIGhhbmRs ZWQgaW4gdGhlIHJlc3BlY3RpdmUgZHJpdmVyKS4NCj4gDQo+IFBsZWFzZSBpZ25vcmUgYW55IGNv ZGUgc3Vycm91bmRpbmcgYHN0cnVjdCBlZGlkYCBpbiBjb3JlYm9vdCwgaXQncw0KPiB2ZXJ5IGJh ZGx5IGRlc2lnbmVkIGFuZCBpbXBsZW1lbnRlZCB3b3JzZS4NCj4gDQo+ID4+IFRoZXJlIGFyZSBh bHNvIG90aGVyIGNvZGUgcGF0aHMgdGhhdCBkb24ndCB1c2Ugc3JjL2xpYi9lZGlkLmMgdG8gc2V0 IHVwDQo+ID4+IHRoZSBmcmFtZWJ1ZmZlci4NCj4gPj4NCj4gPj4gSW4gc3JjL2RyaXZlcnMvaW50 ZWwvZ21hL2hpcmVzX2ZiL2dtYS5hZGIgd2UgaGF2ZToNCj4gPj4gICAgICAgICAgICAgeF9yZXNv bHV0aW9uICAgICAgICAgPT4gd29yZDMyIChmYi5XaWR0aCksDQo+ID4+ICAgICAgICAgICAgIHlf cmVzb2x1dGlvbiAgICAgICAgID0+IHdvcmQzMiAoZmIuSGVpZ2h0KSwNCj4gPj4gICAgICAgICAg ICAgYnl0ZXNfcGVyX2xpbmUgICAgICAgPT4gNCAqIHdvcmQzMiAoZmIuU3RyaWRlKSwNCj4gPj4N Cj4gPj5Gcm9tIHRoZSBzYW1lIGZpbGUsIEkgZm91bmQ6DQo+ID4gICAgIFN0cmlkZSAgPT4gKChX aWR0aF9UeXBlIChtaW5faCkgKyA2MykgLyA2NCkgKiA2NA0KPiA+DQo+ID4gVGhpcyBsaW5lIHNl ZW1zIHRvIGV4cGFuZCBTdHJpZGUgdG8gNjQgYWxpZ25tZW50IGluIHRoZSB1bml0IG9mIFBpeGVs LCBub3QNCj4gPiBCeXRlLiBJIHRob3VnaHQgbGluZSBwYWRkaW5nIGlzIG9uIDY0IGJ5dGUgYWxp Z25tZW50LCBub3Qgb24gNjQgcGl4ZWwNCj4gPiBhbGlnbm1lbnQuDQo+IA0KPiBZZXMsIHlvdSBh cmUgcmlnaHQuIFRoaXMgIm92ZXItYWxpZ25tZW50IiB3YXMgaW50cm9kdWNlZCBieSBhY2NpZGVu dCwgYnkNCj4gbWUuIEknZCBsaWtlIHRvIGtlZXAgaXQsIHRob3VnaC4gSXQncyBub3Qgd3Jvbmcg YnkgYW55IG1lYW5zLCBhbmQgb25seQ0KPiB3YXN0ZXMgYSBsaXR0bGUgbWVtb3J5LiBPVE9ILCBh cyBpdCdzIHVzaW5nIHVudXN1YWwgdmFsdWVzIHdoaWxlIHN0aWxsDQo+IGJlaW5nIGNvbXBsaWFu dCB3aXRoIGhhcmR3YXJlIGFuZCB0aGUgY29yZWJvb3QgZnJhbWVidWZmZXIgaW50ZXJmYWNlLA0K PiBpdCdzIGEgZ3JlYXQgbWVhc3VyZSB0byBkaXNjb3ZlciBidWdzIGluIGNvbnN1bWVycyBvZiBv dXIgZnJhbWVidWZmZXINCj4gaW50ZXJmYWNlLg0KPiANCj4gQnR3LiB0aGVyZSBpcyBtdWNoIG1v cmUgZ2Z4IGluaXQgY29kZSBpbiBjb3JlYm9vdCBmb3Igbm9uLUludGVsIGhhcmQtDQo+IHdhcmUu IFdoaWNoIG1heSBoYXZlIGl0J3Mgb3duIGNvbnN0cmFpbnRzIHJlZ2FyZGluZyBgYnl0ZXNfcGVy X3BpeGVsYC4NCj4gVGhlIG9ubHkgZ3VhcmFudGVlIGZvciBgYnl0ZXNfcGVyX3BpeGVsYCBhbmQg YHhfcmVzb2x1dGlvbmAgeW91IGdldCBhcw0KPiBhIGNvbnN1bWVyLCBpcyB0aGF0IHRoZSBmb3Jt ZXIgaXMgYmlnIGVub3VnaCB0byBob2xkIGB4X3Jlc29sdXRpb25gDQo+IHBpeGVscy4NCj4gDQo+ IE5pY28NCg0KVGhhbmtzLA0KDQotIGJlbg0K