From: "Pali Rohár" <pali.rohar@gmail.com>
To: Mario.Limonciello@dell.com
Cc: dvhart@infradead.org, andy.shevchenko@gmail.com,
linux-kernel@vger.kernel.org,
platform-driver-x86@vger.kernel.org, luto@kernel.org,
quasisec@google.com, rjw@rjwysocki.net, mjg59@google.com,
hch@lst.de, greg@kroah.com, gnomes@lxorguk.ukuu.org.uk
Subject: Re: [PATCH v8 00/15] Introduce support for Dell SMBIOS over WMI
Date: Mon, 16 Oct 2017 15:37:05 +0200 [thread overview]
Message-ID: <20171016133705.cf4f5jqgq4ebhfcj@pali> (raw)
In-Reply-To: <0aebeec94bad4ca9baae335ef9e591b8@ausx13mpc120.AMER.DELL.COM>
On Monday 16 October 2017 13:23:04 Mario.Limonciello@dell.com wrote:
> > -----Original Message-----
> > From: Pali Rohár [mailto:pali.rohar@gmail.com]
> > Sent: Monday, October 16, 2017 2:33 AM
> > To: Limonciello, Mario <Mario_Limonciello@Dell.com>
> > Cc: dvhart@infradead.org; Andy Shevchenko <andy.shevchenko@gmail.com>;
> > LKML <linux-kernel@vger.kernel.org>; platform-driver-x86@vger.kernel.org; Andy
> > Lutomirski <luto@kernel.org>; quasisec@google.com; rjw@rjwysocki.net;
> > mjg59@google.com; hch@lst.de; Greg KH <greg@kroah.com>; Alan Cox
> > <gnomes@lxorguk.ukuu.org.uk>
> > 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
> > pali.rohar@gmail.com
>
> 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.
--
Pali Rohár
pali.rohar@gmail.com
next prev parent reply other threads:[~2017-10-16 13:37 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-10-14 5:32 [PATCH v8 00/15] Introduce support for Dell SMBIOS over WMI Mario Limonciello
2017-10-14 5:32 ` [PATCH v8 01/15] platform/x86: wmi: Add new method wmidev_evaluate_method Mario Limonciello
2017-10-14 5:32 ` [PATCH v8 02/15] platform/x86: dell-wmi: increase severity of some failures Mario Limonciello
2017-10-14 5:32 ` [PATCH v8 03/15] platform/x86: dell-wmi: clean up wmi descriptor check Mario Limonciello
2017-10-14 5:32 ` [PATCH v8 04/15] platform/x86: dell-wmi: allow 32k return size in the descriptor Mario Limonciello
2017-10-14 5:32 ` [PATCH v8 05/15] platform/x86: dell-wmi-descriptor: split WMI descriptor into it's own driver Mario Limonciello
2017-10-14 5:32 ` [PATCH v8 06/15] platform/x86: wmi: Don't allow drivers to get each other's GUIDs Mario Limonciello
2017-10-14 5:32 ` [PATCH v8 07/15] platform/x86: dell-smbios: only run if proper oem string is detected Mario Limonciello
2017-10-14 5:32 ` [PATCH v8 08/15] platform/x86: dell-smbios: Add a sysfs interface for SMBIOS tokens Mario Limonciello
2017-10-17 6:37 ` kbuild test robot
2017-10-14 5:32 ` [PATCH v8 09/15] platform/x86: dell-smbios: Introduce dispatcher for SMM calls Mario Limonciello
2017-10-17 3:49 ` kbuild test robot
2017-10-17 4:14 ` [PATCH] platform/x86: dell-smbios: fix semicolon.cocci warnings kbuild test robot
2017-10-17 4:14 ` [PATCH v8 09/15] platform/x86: dell-smbios: Introduce dispatcher for SMM calls kbuild test robot
2017-10-17 4:20 ` kbuild test robot
2017-10-14 5:32 ` [PATCH v8 10/15] platform/x86: dell-smbios-wmi: Add new WMI dispatcher driver Mario Limonciello
2017-10-14 5:32 ` [PATCH v8 11/15] platform/x86: dell-smbios-smm: test for WSMT Mario Limonciello
2017-10-14 5:32 ` [PATCH v8 12/15] platform/x86: dell-smbios: Add filtering support Mario Limonciello
2017-10-17 4:11 ` kbuild test robot
2017-10-14 5:32 ` [PATCH v8 13/15] platform/x86: wmi: Add sysfs attribute for required_buffer_size Mario Limonciello
2017-10-14 5:32 ` [PATCH v8 14/15] platform/x86: wmi: create userspace interface for drivers Mario Limonciello
2017-10-17 3:58 ` kbuild test robot
2017-10-14 5:32 ` [PATCH v8 15/15] platform/x86: dell-smbios-wmi: introduce userspace interface Mario Limonciello
2017-10-16 2:45 ` [PATCH v8 00/15] Introduce support for Dell SMBIOS over WMI Edward O'Callaghan
2017-10-16 7:33 ` Pali Rohár
2017-10-16 13:23 ` Mario.Limonciello
2017-10-16 13:37 ` Pali Rohár [this message]
2017-10-16 13:48 ` Mario.Limonciello
2017-10-16 21:56 ` Darren Hart
2017-10-17 14:22 ` Mario.Limonciello
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20171016133705.cf4f5jqgq4ebhfcj@pali \
--to=pali.rohar@gmail.com \
--cc=Mario.Limonciello@dell.com \
--cc=andy.shevchenko@gmail.com \
--cc=dvhart@infradead.org \
--cc=gnomes@lxorguk.ukuu.org.uk \
--cc=greg@kroah.com \
--cc=hch@lst.de \
--cc=linux-kernel@vger.kernel.org \
--cc=luto@kernel.org \
--cc=mjg59@google.com \
--cc=platform-driver-x86@vger.kernel.org \
--cc=quasisec@google.com \
--cc=rjw@rjwysocki.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox