On Tue, Aug 01, 2017 at 05:43:12PM +0200, Pali Rohár wrote:
> On Tuesday 01 August 2017 08:37:26 Andy Lutomirski wrote:
> > When I converted dell-wmi to the new bus infrastructure, I left the
> > call to dell_wmi_check_descriptor_buffer() in dell_wmi_init(). This
> > could cause two problems:
> >
> > - An error message when loading the driver on a system without
> > dell-wmi. We'd try to read the event descriptor even if the WMI
> > GUID wasn't there.
> >
> > - A possible race if dell-wmi was loaded manually before wmi was
> > fully initialized.
> >
> > Fix it by moving the call to the probe function where it belongs.
> >
> > Fixes: bff589be59c5 ("platform/x86: dell-wmi: Convert to the WMI bus infrastructure")
> > Signed-off-by: Andy Lutomirski <luto@xxxxxxxxxx>
> Hi! You should move also dell_wmi_events_set_enabled() into
> dell_wmi_probe() as there is no need to enable receiving events prior to
> creating input device.
That is a good idea, but it is a separate functional change. Especially as this
is addressing a specific bug, let's keep this as is, and add a second patch to
move the enable receiving events.
--
Darren Hart
VMware Open Source Technology Center
