> -----Original Message----- > From: Darren Hart [mailto:[email protected]] > Sent: Monday, October 16, 2017 4:57 PM > To: Limonciello, Mario <[email protected]> > Cc: [email protected]; [email protected]; linux- > [email protected]; [email protected]; [email protected]; > [email protected]; [email protected]; [email protected]; [email protected]; > [email protected]; [email protected] > Subject: Re: [PATCH v8 00/15] Introduce support for Dell SMBIOS over WMI > > On Mon, Oct 16, 2017 at 01:48:05PM +0000, [email protected] wrote: > > > -----Original Message----- > > > From: Pali Rohár [mailto:[email protected]] > > > Sent: Monday, October 16, 2017 8:37 AM > > > To: Limonciello, Mario <[email protected]> > > > Cc: [email protected]; [email protected]; linux- > > > [email protected]; [email protected]; > [email protected]; > > > [email protected]; [email protected]; [email protected]; [email protected]; > > > [email protected]; [email protected] > > > Subject: Re: [PATCH v8 00/15] Introduce support for Dell SMBIOS over WMI > > > > > > On Monday 16 October 2017 13:23:04 [email protected] wrote: > > > > > -----Original Message----- > > > > > From: Pali Rohár [mailto:[email protected]] > > > > > Sent: Monday, October 16, 2017 2:33 AM > > > > > To: Limonciello, Mario <[email protected]> > > > > > Cc: [email protected]; Andy Shevchenko > <[email protected]>; > > > > > LKML <[email protected]>; platform-driver- > [email protected]; > > > Andy > > > > > Lutomirski <[email protected]>; [email protected]; [email protected]; > > > > > [email protected]; [email protected]; Greg KH <[email protected]>; Alan Cox > > > > > <[email protected]> > > > > > Subject: Re: [PATCH v8 00/15] Introduce support for Dell SMBIOS over > > > > > WMI > > > > > > > > > > Hi! Just a small idea, what about including example userspace > > > > > application which would use this new API? I think such application can > > > > > be useful for documentation purpose to see how this new API should be > > > > > used. And in review process other people can point if there would be > > > > > some problems with usage. > > > > > > > > > > I do not know if it is requirement or not for introducing new kernel > > > > > API, but still example code is very useful for developers when they > > > > > would start to use this new API and correctly. > > > > > > > > > > -- > > > > > Pali Rohár > > > > > [email protected] > > > > > > > > Pali, > > > > > > > > I've produced a small sample application available here: > > > > https://github.com/superm1/smbios-wmi-poc > > > > > > Simple one-file example application. Great! > > > > > > Darren, Greg, Mario, I think it would be better to have this example > > > application in tools/ or in Documentation/... What do you think? > > > > > > Looking for examples on random github repos is harder and less official > > > as part of linux source tree. > > > > > > > If others also agree, I'm fine with including it with the kernel. > > Please advise where it should be located. > > > > It was originally a POC made just for my own testing of this API. > > I will probably need to clean it up a little though for wider distribution > > to the kernel so I would prefer to send it in a follow up patch if no other > > feedback requiring changes to this series as is. > > > > One possible issue is that it currently uses glib. Is that OK? > > In my opinion, what is important is that there is an open-source > userspace consumer of the interface. I don't think it needs to be in the > kernel. If this was a more generic interface which could be tested on > any hardware, then I think a selftests target would be sensible. As it > is, this is a very specific use case, and a simple reference to the > smbios-wmi-poc project URL in documentation someplace (or even in the > dell-smbios driver comments) seems more than adequate to me. > To Pali's point I noticed stuff like this: https://github.com/torvalds/linux/blob/master/tools/laptop/freefall/freefall.c which shows how to use /dev/freefall.
I don't mind to switch my tool to drop glib and bring in kernel. I'll plan to bring it in here in a separate patch: tools/wmi/dell-smbios-example.c kbuild test robot caught a few other missing includes for certain situations, so I'll be making another change set for that anyway. Thanks,
