From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-eopbgr690053.outbound.protection.outlook.com ([40.107.69.53] helo=NAM04-CO1-obe.outbound.protection.outlook.com) by bombadil.infradead.org with esmtps (Exim 4.90_1 #2 (Red Hat Linux)) id 1fVb0Y-0002kH-C7 for linux-mtd@lists.infradead.org; Wed, 20 Jun 2018 11:10:47 +0000 From: Joakim Tjernlund To: "boris.brezillon@bootlin.com" CC: "stable@vger.kernel.org" , "linux-mtd@lists.infradead.org" Subject: Re: [PATCH v2 3/4] mtd: cfi_cmdset_0002: Avoid point less unlocking/locking Date: Wed, 20 Jun 2018 11:10:23 +0000 Message-ID: References: <20180605172607.50acb34f@bbrezillon> <20180606101330.11071-1-joakim.tjernlund@infinera.com> <20180606101330.11071-3-joakim.tjernlund@infinera.com> <20180620111430.0dd2ecde@bbrezillon> In-Reply-To: <20180620111430.0dd2ecde@bbrezillon> Content-Language: en-US Content-Type: text/plain; charset="utf-8" Content-ID: <9D7932E66D06D7468851C2BE10715BA7@namprd10.prod.outlook.com> Content-Transfer-Encoding: base64 MIME-Version: 1.0 List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , T24gV2VkLCAyMDE4LTA2LTIwIGF0IDExOjE0ICswMjAwLCBCb3JpcyBCcmV6aWxsb24gd3JvdGU6 DQo+IENBVVRJT046IFRoaXMgZW1haWwgb3JpZ2luYXRlZCBmcm9tIG91dHNpZGUgb2YgdGhlIG9y Z2FuaXphdGlvbi4gRG8gbm90IGNsaWNrIGxpbmtzIG9yIG9wZW4gYXR0YWNobWVudHMgdW5sZXNz IHlvdSByZWNvZ25pemUgdGhlIHNlbmRlciBhbmQga25vdyB0aGUgY29udGVudCBpcyBzYWZlLg0K PiANCj4gDQo+IE9uIFdlZCwgIDYgSnVuIDIwMTggMTI6MTM6MjkgKzAyMDANCj4gSm9ha2ltIFRq ZXJubHVuZCA8am9ha2ltLnRqZXJubHVuZEBpbmZpbmVyYS5jb20+IHdyb3RlOg0KPiANCj4gPiBj ZmlfcHBiX3VubG9jaygpIHdhbGtzIGFsbCBmbGFzaCBjaGlwcyB3aGVuIHVubG9ja2luZyBzZWN0 b3JzLg0KPiA+IHRlc3RpbmcgbG9jayBzdGF0dXMgb24gZWFjaCBjaGlwIHdoaWNoIGNhdXNlcyBy ZWxvY2tpbmcgb2YgYWxyZWFkeQ0KPiA+IGxvY2tlZCBzZWN0b3JzLiBUZXN0IGFnYWluc3Qgb2Zm c2V0IHRvIGF2aW9kIHRoaXMgYWxpYXNpbmcuDQo+IA0KPiAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgIF4gYXZvaWQNCj4gDQo+IEFzIEkgc2FpZCBiZWZvcmUsIEkgdGhp bmsgdGhlIGN1cnJlbnQgY29kZSBpcyBkb2luZyB3b3JzZSB0aGFuIGp1c3QNCj4gcmVsb2NraW5n IGFscmVhZHkgbG9ja2VkIHNlY3RvcnMuIEFzIHNvb24gYXMgeW91IGNyb3NzIGEgY2hpcCBib3Vu ZGFyeSwNCj4gYWRkciBpcyBzZXQgYmFjayB0byAwLCBhbmQgdGhlIChhZGRyIDwgb2ZmcyB8fCBh ZHIgPj0gKG9mcyArIGxlbikpIG1pZ2h0DQo+IGJlIHRydWUgd2hpbGUgaXQgc2hvdWxkbid0IGJl IChhYnNvbHV0ZSBvZmZzZXQgc3RpbGwgaW4gdGhlIHVubG9jaw0KPiByYW5nZSksIHdoaWNoIG1l YW5zIHlvdSdsbCBsb2NrIHNlY3RvcnMgdGhhdCB0aGUgY2FsbGVyIGV4cGVjdCB0byBiZQ0KPiB1 bmxvY2tlZC4NCg0KSSBkb24ndCBzZWUgaG93LCB0aGUgY29kZSBhc2tzIGZvciBpdHMgY3VycmVu dCBsb2NrIHN0YXR1cyBhbmQgd2lsbCByZWFwcGx5DQp0aG9zZSB0aGF0IGFyZSBsb2NrZWQgYWdh aW4uDQoNCj4gDQo+ID4gDQo+ID4gRml4ZXM6IDE2NDhlYWFhMTU3NSAoIm10ZDogY2ZpX2NtZHNl dF8wMDAyOiBTdXBwb3J0IFBlcnNpc3RlbnQgUHJvdGVjdGlvbiBCaXRzIChQUEIpIGxvY2tpbmci KQ0KPiA+IENjOiBzdGFibGVAdmdlci5rZXJuZWwub3JnDQo+ID4gU2lnbmVkLW9mZi1ieTogSm9h a2ltIFRqZXJubHVuZCA8am9ha2ltLnRqZXJubHVuZEBpbmZpbmVyYS5jb20+DQo+ID4gLS0tDQo+ ID4gDQo+ID4gIHYyIC0gU3BpbHQgaW50byBzZXZlcmFsIHBhdGNoZXMNCj4gPiANCj4gPiANCj4g PiAgZHJpdmVycy9tdGQvY2hpcHMvY2ZpX2NtZHNldF8wMDAyLmMgfCAyICstDQo+ID4gIDEgZmls ZSBjaGFuZ2VkLCAxIGluc2VydGlvbigrKSwgMSBkZWxldGlvbigtKQ0KPiA+IA0KPiA+IGRpZmYg LS1naXQgYS9kcml2ZXJzL210ZC9jaGlwcy9jZmlfY21kc2V0XzAwMDIuYyBiL2RyaXZlcnMvbXRk L2NoaXBzL2NmaV9jbWRzZXRfMDAwMi5jDQo+ID4gaW5kZXggY2I4NWNjY2M0OGMxLi5iNjI3M2Nl ODNkZTcgMTAwNjQ0DQo+ID4gLS0tIGEvZHJpdmVycy9tdGQvY2hpcHMvY2ZpX2NtZHNldF8wMDAy LmMNCj4gPiArKysgYi9kcml2ZXJzL210ZC9jaGlwcy9jZmlfY21kc2V0XzAwMDIuYw0KPiA+IEBA IC0yNjcwLDcgKzI2NzAsNyBAQCBzdGF0aWMgaW50IF9fbWF5YmVfdW51c2VkIGNmaV9wcGJfdW5s b2NrKHN0cnVjdCBtdGRfaW5mbyAqbXRkLCBsb2ZmX3Qgb2ZzLA0KPiA+ICAgICAgICAgICAgICAg ICogc2VjdG9ycyBzaGFsbCBiZSB1bmxvY2tlZCwgc28gbGV0cyBrZWVwIHRoZWlyIGxvY2tpbmcN Cj4gPiAgICAgICAgICAgICAgICAqIHN0YXR1cyBhdCAidW5sb2NrZWQiIChsb2NrZWQ9MCkgZm9y IHRoZSBmaW5hbCByZS1sb2NraW5nLg0KPiA+ICAgICAgICAgICAgICAgICovDQo+ID4gLSAgICAg ICAgICAgICBpZiAoKGFkciA8IG9mcykgfHwgKGFkciA+PSAob2ZzICsgbGVuKSkpIHsNCj4gPiAr ICAgICAgICAgICAgIGlmICgob2Zmc2V0IDwgb2ZzKSB8fCAob2Zmc2V0ID49IChvZnMgKyBsZW4p KSkgew0KPiA+ICAgICAgICAgICAgICAgICAgICAgICBzZWN0W3NlY3RvcnNdLmNoaXAgPSAmY2Zp LT5jaGlwc1tjaGlwbnVtXTsNCj4gPiAgICAgICAgICAgICAgICAgICAgICAgc2VjdFtzZWN0b3Jz XS5hZHIgPSBhZHI7DQo+ID4gICAgICAgICAgICAgICAgICAgICAgIHNlY3Rbc2VjdG9yc10ubG9j a2VkID0gZG9fcHBiX3h4bG9jaygNCj4gDQo+IA0K