Re: [PATCH 15/16] platform/x86: wmi-mof: New driver to expose embedded WMI MOF metadata

From	Pali Rohár <pali.rohar@gmail.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH 15/16] platform/x86: wmi-mof: New driver to expose embedded WMI MOF metadata
Date	2017-06-06 16:00 +0200
Message-ID	<tPnbA-160-9@gated-at.bofh.it> (permalink)
References	(2 earlier) <tP8vT-e0-5@gated-at.bofh.it> <tP8vT-e0-15@gated-at.bofh.it> <tP8YV-nD-15@gated-at.bofh.it> <tPkx4-7Uz-29@gated-at.bofh.it> <tPnbA-160-11@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On Tuesday 06 June 2017 13:46:16 Mario.Limonciello@dell.com wrote:
> Pali,
> 
> Amazing job with what you've done so far.  A few comments I wanted to share from my taking look at your bmf2mof and comparing to "source" MOF.
> 1) At least in source the case used for String and Boolean is lower case.  I'm unsure if that actually matters for any MOF parsing tools, but I wanted to FYI in case it does.

In MS documentation is String, Boolean and Datetime with first char
uppercase. But looks like mofcomp accept both upper case and lower case
variants.

> 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.

> 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

Thread

[PATCH 15/16] platform/x86: wmi-mof: New driver to expose embedded WMI MOF metadata Darren Hart <dvhart@infradead.org> - 2017-05-27 07:40 +0200
  Re: [PATCH 15/16] platform/x86: wmi-mof: New driver to expose embedded WMI MOF metadata Pali Rohár <pali.rohar@gmail.com> - 2017-05-27 13:20 +0200
    Re: [PATCH 15/16] platform/x86: wmi-mof: New driver to expose  embedded WMI MOF metadata Andy Lutomirski <luto@kernel.org> - 2017-05-27 23:10 +0200
      Re: [PATCH 15/16] platform/x86: wmi-mof: New driver to expose  embedded WMI MOF metadata Andy Shevchenko <andriy.shevchenko@linux.intel.com> - 2017-05-30 17:30 +0200
        Re: [PATCH 15/16] platform/x86: wmi-mof: New driver to expose  embedded WMI MOF metadata Darren Hart <dvhart@infradead.org> - 2017-05-30 18:50 +0200
        Re: [PATCH 15/16] platform/x86: wmi-mof: New driver to expose embedded WMI MOF metadata Pali Rohár <pali.rohar@gmail.com> - 2017-05-30 19:10 +0200
          Re: [PATCH 15/16] platform/x86: wmi-mof: New driver to expose  embedded WMI MOF metadata Andy Shevchenko <andy.shevchenko@gmail.com> - 2017-05-30 19:30 +0200
    Re: [PATCH 15/16] platform/x86: wmi-mof: New driver to expose  embedded WMI MOF metadata Darren Hart <dvhart@infradead.org> - 2017-06-06 00:20 +0200
      Re: [PATCH 15/16] platform/x86: wmi-mof: New driver to expose embedded WMI MOF metadata Pali Rohár <pali.rohar@gmail.com> - 2017-06-06 00:20 +0200
        Re: [PATCH 15/16] platform/x86: wmi-mof: New driver to expose  embedded WMI MOF metadata Andy Lutomirski <luto@amacapital.net> - 2017-06-06 00:50 +0200
          Re: [PATCH 15/16] platform/x86: wmi-mof: New driver to expose  embedded WMI MOF metadata Pali Rohár <pali.rohar@gmail.com> - 2017-06-06 13:10 +0200
            Re: [PATCH 15/16] platform/x86: wmi-mof: New driver to expose  embedded WMI MOF metadata Pali Rohár <pali.rohar@gmail.com> - 2017-06-06 16:00 +0200
            RE: [PATCH 15/16] platform/x86: wmi-mof: New driver to expose  embedded WMI MOF metadata <Mario.Limonciello@dell.com> - 2017-06-06 16:00 +0200
        Re: [PATCH 15/16] platform/x86: wmi-mof: New driver to expose  embedded WMI MOF metadata Darren Hart <dvhart@infradead.org> - 2017-06-06 04:40 +0200

csiph-web