On Sun, Oct 01, 2017 at 02:25:00PM +0000, mario.limoncie...@dell.com wrote: > > -----Original Message----- > > From: Greg KH [mailto:gre...@linuxfoundation.org] > > Sent: Sunday, October 1, 2017 8:24 AM > > To: Limonciello, Mario <mario_limoncie...@dell.com> > > Cc: dvh...@infradead.org; andy.shevche...@gmail.com; linux- > > ker...@vger.kernel.org; platform-driver-...@vger.kernel.org; > > l...@kernel.org; > > quasi...@google.com; pali.ro...@gmail.com; r...@rjwysocki.net; > > mj...@google.com; h...@lst.de > > Subject: Re: [PATCH v3 4/8] platform/x86: wmi: create character devices when > > requested by drivers > > > > On Sat, Sep 30, 2017 at 07:26:57PM +0000, mario.limoncie...@dell.com wrote: > > > I think what's Darren's most interested in is the that conceptually this > > > is > > > an approach you can agree with. > > > > > > "A WMI vendor driver binds to the WMI bus and requests the WMI bus to > > create > > > a character device. The WMI bus creates a character device > > > /dev/wmi/$driver > > > which the WMI vendor driver will process all various file operations." > > > > We've been over this before, let's see that actual implementation which > > defines the "will process all..." implementation to see if that is > > actually an acceptable thing. > > > > thanks, > > > > greg k-h > > I'm working on changes still (that you'll be CC on), but I don't expect this > specific > concept to change in the future changes: > > Bus creating character device: > https://patchwork.kernel.org/patch/9975283/
A bit hard to comment on web pages, and I would like to point out your obvious memory leak, but it's hard to do that this way :( > Driver requesting character device: > https://patchwork.kernel.org/patch/9975263/ There's other issues with this patch as well, I'll wait for a real one in my inbox to be able to actually review it :( thanks, greg k-h
