HID: Bluetooth: hidp: register HID devices async

While l2cap_user callbacks are running, the whole hci_dev is locked. Even
if we would add more fine-grained locking to HCI core, it would still be
called from the non-reentrant rx work-queue and thus block the event
processing.

However, if we want to perform synchronous I/O during HID device
registration (eg., to perform device-detection), we need the HCI core
to be able to dispatch incoming data.

Therefore, we now move device-registration to a separate worker. The HCI
core can continue running and we add devices asynchronously in another
kernel thread. Device removal is synchronized and waits for the worker
to exit before calling the usual device removal functions.

If l2cap_user->remove is called before the thread registered the devices,
we set "terminate" to true and the thread will skip it. If
l2cap_user->remove is called after it, we notice this as the device
is no longer in HIDP_SESSION_PREPARING state and simply unregister the
device as we did before.
There is no new deadlock as we now call hidp_session_add_dev() with
one lock less held (the HCI lock) and it cannot itself call back into
HCI as it was called with the HCI-lock held before.

One might wonder whether this can block during device unregistration.
But we set "terminate" to true and wake the HIDP thread up _before_
unregistering the HID/input devices. Therefore, all pending HID I/O
operations are canceled. All further I/O attempts will fail with ENODEV
or EIO. So all latency we can get are few context-switches, but no
timeouts or blocking I/O waits!

This change also prepares for a long standing HID bug. All HID devices
that register power_supply devices need to be able to handle callbacks
during registration (a power_supply oddity that cannot easily be fixed).
So with this patch available, we can allow HID I/O during registration
by calling the recently introduced hid_device_io_start/stop helpers,
which currently are a no-op for bluetooth due to this locking.

Note that we cannot do the same for input devices. input-core doesn't
allow us to call input_event() asynchronously to input_register_device(),
which HID-core kindly allows (for good reasons).
Fixing input-core to allow this isn't as easy as it sounds and is,
beside simplifying HIDP, not really an improvement. Hence, we still
register input devices synchronously as we did before. Only HID devices
are registered asynchronously.

Signed-off-by: David Herrmann <dh.herrmann@gmail.com>
Acked-by: Jiri Kosina <jkosina@suse.cz>
Acked-by: Gustavo Padovan <gustavo.padovan@collabora.co.uk>
Tested-by: Daniel Nicoletti <dantti12@gmail.com>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
Backport support for the Sony Sixaxis and DualShock 4 controllers.

This backports support for force-feedback on both the Sixaxis and
DualShock 4, full LED support for both controllers, battery reporting for
both controllers and touchpad support on the DS4.  All features are
functional under both USB and Bluetooth.

Additionally, this adds a workaround to stop the Sixaxis from blinking
after the PS button is pushed when connected via USB and it uses the same
trick as the xpad driver to set the default LED values using the joystick
device number.

Note that the DualShock 4 needs BlueZ 5.14 or newer to pair correctly.

Signed-off-by: Frank Praznik <frank.praznik@oh.rr.com>
Workaround for sending reports to the Sixaxis and DualShock 4.

The Sixaxis and DualShock 4 want output reports sent on the control
channel when running on Bluetooth, so a special case is needed for them.

This is a bit of an ugly hack, but it gets the job done without needing to
backport the rather extensive HID system changes in newer kernels that are
needed to make these controllers function via Bluetooth.

Signed-off-by: Frank Praznik <frank.praznik@oh.rr.com>
Correct DualShock 4 HID descriptors

 -A 6 bit value had a logical maximum of 255 when the largest it can be is 63.
 -Correct a logical max value that should be 255 instead of -1.
 -Clamp the gyroscope values to -8192/8191 as these are the largest true
  orientation values.  Anything larger or smaller is just noise.

Signed-off-by: Frank Praznik <frank.praznik@oh.rr.com>
Merge pull request #6 from ArtVandelae/alchemist-3.10

Backport support for the DualShock 3 and 4 controllers.