From mboxrd@z Thu Jan 1 00:00:00 1970 From: Subject: RE: [PATCH 15/16] platform/x86: wmi-mof: New driver to expose embedded WMI MOF metadata Date: Wed, 7 Jun 2017 20:23:08 +0000 Message-ID: References: <8aa135f430b441eb82614b22d662d1ff@ausx13mpc120.AMER.DELL.COM> <20170606135621.GC4690@pali> <201706071939.25681@pali> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from esa3.dell-outbound.iphmx.com ([68.232.153.94]:63083 "EHLO esa3.dell-outbound.iphmx.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751405AbdFGUXL (ORCPT ); Wed, 7 Jun 2017 16:23:11 -0400 In-Reply-To: <201706071939.25681@pali> Content-Language: en-US Sender: linux-acpi-owner@vger.kernel.org List-Id: linux-acpi@vger.kernel.org To: pali.rohar@gmail.com Cc: luto@amacapital.net, dvhart@infradead.org, platform-driver-x86@vger.kernel.org, andriy.shevchenko@linux.intel.com, luto@kernel.org, rjw@rjwysocki.net, linux-kernel@vger.kernel.org, linux-acpi@vger.kernel.org PiAtLS0tLU9yaWdpbmFsIE1lc3NhZ2UtLS0tLQ0KPiBGcm9tOiBQYWxpIFJvaMOhciBbbWFpbHRv OnBhbGkucm9oYXJAZ21haWwuY29tXQ0KPiBTZW50OiBXZWRuZXNkYXksIEp1bmUgNywgMjAxNyAx MjozOSBQTQ0KPiBUbzogTGltb25jaWVsbG8sIE1hcmlvIDxNYXJpb19MaW1vbmNpZWxsb0BEZWxs LmNvbT4NCj4gQ2M6IGx1dG9AYW1hY2FwaXRhbC5uZXQ7IGR2aGFydEBpbmZyYWRlYWQub3JnOyBw bGF0Zm9ybS1kcml2ZXItDQo+IHg4NkB2Z2VyLmtlcm5lbC5vcmc7IGFuZHJpeS5zaGV2Y2hlbmtv QGxpbnV4LmludGVsLmNvbTsgbHV0b0BrZXJuZWwub3JnOw0KPiByandAcmp3eXNvY2tpLm5ldDsg bGludXgta2VybmVsQHZnZXIua2VybmVsLm9yZzsgbGludXgtYWNwaUB2Z2VyLmtlcm5lbC5vcmcN Cj4gU3ViamVjdDogUmU6IFtQQVRDSCAxNS8xNl0gcGxhdGZvcm0veDg2OiB3bWktbW9mOiBOZXcg ZHJpdmVyIHRvIGV4cG9zZQ0KPiBlbWJlZGRlZCBXTUkgTU9GIG1ldGFkYXRhDQo+IA0KPiBPbiBU dWVzZGF5IDA2IEp1bmUgMjAxNyAxNTo1NjoyMSBQYWxpIFJvaMOhciB3cm90ZToNCj4gPiBPbiBU dWVzZGF5IDA2IEp1bmUgMjAxNyAxMzo0NjoxNiBNYXJpby5MaW1vbmNpZWxsb0BkZWxsLmNvbSB3 cm90ZToNCj4gPiA+IDIpIE9uIG15IHN5c3RlbSB3aGVuIHlvdSBleHBhbmQgdGhlIGFyZ3VtZW50 cyBmb3IgInZvaWQgRG9CRm4iIHRoZQ0KPiA+ID4gc291cmNlIGRvZXNuJ3QgZGVzY3JpYmUgaW5k aXZpZHVhbCBhcmd1bWVudHMgbGlrZSB5b3UgZG8uIEFnYWluDQo+ID4gPiB0aGlzIG1pZ2h0IG5v dCBtYXR0ZXIgdG8gTU9GIHBhcnNpbmcgdG9vbHMgYnV0IHdhbnRlZCB0byBsZXQgeW91DQo+ID4g PiBrbm93IGluIGNhc2UgaXQgZG9lcy4NCj4gPg0KPiA+IEkga25vdywgdGhpcyBwYXJ0IGlzIG1p c3NpbmcuIE9yZGVyIG9mIGFyZ3VtZW50cyBhcmUgb25seSBpbiBJRA0KPiA+IHF1YWxpZmllciBh bmQgbm90IHNvcnRlZCArIGluL291dCBkZS1kdXBsaWNhdGVkLg0KPiANCj4gSW1wbGVtZW50ZWQh IE5vdyBhcmd1bWVudHMgYXJlIGNvcnJlY3RseSBwbGFjZWQgYmFzZWQgb24gSUQgcXVhbGlmaWVy Lg0KSSB0aGluayBpdCdzIHN0aWxsIG9mZiBhIGxpdHRsZSB0aG91Z2guDQoNCldoYXQgSSdtIGdl dHRpbmcgYmFjayBub3cgZnJvbSBibWYybW9mIGlzOg0KCXZvaWQgRG9CRm4oW2luLCBEZXNjcmlw dGlvbigiRm4gYnVmIiksIG91dF0gQkRhdCBEYXRhKTsNCg0KV2hlcmVhcyBzb3VyY2UgcHV0cyBE ZXNjcmlwdGlvbiBhcyB0aGUgbGFzdCBhcmd1bWVudDogDQoJdm9pZCBEb0JGbihbaW4sIG91dCwg RGVzY3JpcHRpb24oIkZuIGJ1ZiIpXSBCRGF0IERhdGEpOw0KDQo+IA0KPiA+ID4gc291cmNlOg0K PiA+ID4gCXZvaWQgRG9CRm4oW2luLCBvdXQsIERlc2NyaXB0aW9uKCJGbiBidWYiKV0gQkRhdCBE YXRhKTsNCj4gPiA+DQo+ID4gPiBibWYybW9mOg0KPiA+ID4gCXZvaWQgZG9CRm4oW2luLCBEZXNj cmlwdGlvbigiRm4gYnVmIiksIElEKDApXSBCRGF0IERhdGEsIFtvdXQsDQo+ID4gPiAJRGVzY3Jp cHRpb24oIkZuIGJ1ZiIpLCBJRCgwKV0gQkRhdCBEYXRhKTsNCj4gDQo+IC0tDQo+IFBhbGkgUm9o w6FyDQo+IHBhbGkucm9oYXJAZ21haWwuY29tDQo= From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751534AbdFGUXN (ORCPT ); Wed, 7 Jun 2017 16:23:13 -0400 Received: from esa3.dell-outbound.iphmx.com ([68.232.153.94]:63083 "EHLO esa3.dell-outbound.iphmx.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751405AbdFGUXL (ORCPT ); Wed, 7 Jun 2017 16:23:11 -0400 From: X-LoopCount0: from 10.170.28.40 X-IronPort-AV: E=Sophos;i="5.39,311,1493701200"; d="scan'208";a="515266793" To: CC: , , , , , , , Subject: RE: [PATCH 15/16] platform/x86: wmi-mof: New driver to expose embedded WMI MOF metadata Thread-Topic: [PATCH 15/16] platform/x86: wmi-mof: New driver to expose embedded WMI MOF metadata Thread-Index: AQHS3kk7xMJsawcrI0WF/5NgAk77dKIXKtQAgAAFrACAANB6AP//0/SQgAHYjGyAAC0fcA== Date: Wed, 7 Jun 2017 20:23:08 +0000 Message-ID: References: <8aa135f430b441eb82614b22d662d1ff@ausx13mpc120.AMER.DELL.COM> <20170606135621.GC4690@pali> <201706071939.25681@pali> In-Reply-To: <201706071939.25681@pali> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-ms-exchange-transport-fromentityheader: Hosted x-originating-ip: [10.143.18.86] Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from base64 to 8bit by mail.home.local id v57KNMoU012173 > -----Original Message----- > From: Pali Rohár [mailto:pali.rohar@gmail.com] > Sent: Wednesday, June 7, 2017 12:39 PM > To: Limonciello, Mario > Cc: luto@amacapital.net; dvhart@infradead.org; platform-driver- > x86@vger.kernel.org; andriy.shevchenko@linux.intel.com; luto@kernel.org; > rjw@rjwysocki.net; linux-kernel@vger.kernel.org; linux-acpi@vger.kernel.org > Subject: Re: [PATCH 15/16] platform/x86: wmi-mof: New driver to expose > embedded WMI MOF metadata > > On Tuesday 06 June 2017 15:56:21 Pali Rohár wrote: > > On Tuesday 06 June 2017 13:46:16 Mario.Limonciello@dell.com wrote: > > > 2) On my system when you expand the arguments for "void DoBFn" the > > > source doesn't describe individual arguments like you do. Again > > > this might not matter to MOF parsing tools but wanted to let you > > > know in case it does. > > > > I know, this part is missing. Order of arguments are only in ID > > qualifier and not sorted + in/out de-duplicated. > > Implemented! Now arguments are correctly placed based on ID qualifier. I think it's still off a little though. What I'm getting back now from bmf2mof is: void DoBFn([in, Description("Fn buf"), out] BDat Data); Whereas source puts Description as the last argument: void DoBFn([in, out, Description("Fn buf")] BDat Data); > > > > source: > > > void DoBFn([in, out, Description("Fn buf")] BDat Data); > > > > > > bmf2mof: > > > void doBFn([in, Description("Fn buf"), ID(0)] BDat Data, [out, > > > Description("Fn buf"), ID(0)] BDat Data); > > -- > Pali Rohár > pali.rohar@gmail.com