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

From: Pali RohÃr
Date: Wed Jun 07 2017 - 13:39:34 EST


On Tuesday 06 June 2017 15:56:21 Pali RohÃr wrote:
> On Tuesday 06 June 2017 13:46:16 Mario.Limonciello@xxxxxxxx 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.

> > 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@xxxxxxxxx

Attachment: signature.asc
Description: This is a digitally signed message part.