mbox series

[v6,00/13] HID: new driver for PS5 'DualSense' controller

Message ID 20210207214908.79573-1-roderick@gaikai.com (mailing list archive)
Headers show
Series HID: new driver for PS5 'DualSense' controller | expand

Message

Roderick Colenbrander Feb. 7, 2021, 9:48 p.m. UTC
From: Roderick Colenbrander <roderick.colenbrander@sony.com>

Hi,

This is an updated version with a few small changes suggested by
by Barnabás and Benjamin. It is mostly a few additional checks and
a few checkpath related changes.

Changes since v5:
- Fixed a couple of checkpath warnings.
- Renamed firmware_/hardware_version show functions.
- Used DEVICE_ATTR_RO for firmware/hardware_version show functions.
- Changed player_ids array to static const.
- Added error check for power_supply_powers.
- Ensured devm_kasprintf return value is checked.

Thanks,

Roderick Colenbrander
Sony Interactive Entertainment, LLC

Roderick Colenbrander (13):
  HID: playstation: initial DualSense USB support.
  HID: playstation: use DualSense MAC address as unique identifier.
  HID: playstation: add DualSense battery support.
  HID: playstation: add DualSense touchpad support.
  HID: playstation: add DualSense accelerometer and gyroscope support.
  HID: playstation: track devices in list.
  HID: playstation: add DualSense Bluetooth support.
  HID: playstation: add DualSense classic rumble support.
  HID: playstation: add DualSense lightbar support
  HID: playstation: add microphone mute support for DualSense.
  HID: playstation: add DualSense player LEDs support.
  HID: playstation: DualSense set LEDs to default player id.
  HID: playstation: report DualSense hardware and firmware version.

 MAINTAINERS                   |    6 +
 drivers/hid/Kconfig           |   21 +
 drivers/hid/Makefile          |    1 +
 drivers/hid/hid-ids.h         |    1 +
 drivers/hid/hid-playstation.c | 1504 +++++++++++++++++++++++++++++++++
 5 files changed, 1533 insertions(+)
 create mode 100644 drivers/hid/hid-playstation.c

Comments

Benjamin Tissoires Feb. 11, 2021, 4:07 p.m. UTC | #1
On Sun, Feb 7, 2021 at 10:49 PM Roderick Colenbrander
<roderick@gaikai.com> wrote:
>
> From: Roderick Colenbrander <roderick.colenbrander@sony.com>
>
> Hi,
>
> This is an updated version with a few small changes suggested by
> by Barnabás and Benjamin. It is mostly a few additional checks and
> a few checkpath related changes.
>
> Changes since v5:
> - Fixed a couple of checkpath warnings.
> - Renamed firmware_/hardware_version show functions.
> - Used DEVICE_ATTR_RO for firmware/hardware_version show functions.
> - Changed player_ids array to static const.
> - Added error check for power_supply_powers.
> - Ensured devm_kasprintf return value is checked.

Thanks a lot for the quick resubmission (and for Barnabás final review).

I double checked that Baranabás' comments were addressed and added the
rev-b to the series.

The series is now pushed to for-5.12/playstation.

Cheers,
Benjamin

>
> Thanks,
>
> Roderick Colenbrander
> Sony Interactive Entertainment, LLC
>
> Roderick Colenbrander (13):
>   HID: playstation: initial DualSense USB support.
>   HID: playstation: use DualSense MAC address as unique identifier.
>   HID: playstation: add DualSense battery support.
>   HID: playstation: add DualSense touchpad support.
>   HID: playstation: add DualSense accelerometer and gyroscope support.
>   HID: playstation: track devices in list.
>   HID: playstation: add DualSense Bluetooth support.
>   HID: playstation: add DualSense classic rumble support.
>   HID: playstation: add DualSense lightbar support
>   HID: playstation: add microphone mute support for DualSense.
>   HID: playstation: add DualSense player LEDs support.
>   HID: playstation: DualSense set LEDs to default player id.
>   HID: playstation: report DualSense hardware and firmware version.
>
>  MAINTAINERS                   |    6 +
>  drivers/hid/Kconfig           |   21 +
>  drivers/hid/Makefile          |    1 +
>  drivers/hid/hid-ids.h         |    1 +
>  drivers/hid/hid-playstation.c | 1504 +++++++++++++++++++++++++++++++++
>  5 files changed, 1533 insertions(+)
>  create mode 100644 drivers/hid/hid-playstation.c
>
> --
> 2.26.2
>
Roderick Colenbrander Feb. 11, 2021, 5:24 p.m. UTC | #2
On Thu, Feb 11, 2021 at 8:07 AM Benjamin Tissoires
<benjamin.tissoires@redhat.com> wrote:
>
> On Sun, Feb 7, 2021 at 10:49 PM Roderick Colenbrander
> <roderick@gaikai.com> wrote:
> >
> > From: Roderick Colenbrander <roderick.colenbrander@sony.com>
> >
> > Hi,
> >
> > This is an updated version with a few small changes suggested by
> > by Barnabás and Benjamin. It is mostly a few additional checks and
> > a few checkpath related changes.
> >
> > Changes since v5:
> > - Fixed a couple of checkpath warnings.
> > - Renamed firmware_/hardware_version show functions.
> > - Used DEVICE_ATTR_RO for firmware/hardware_version show functions.
> > - Changed player_ids array to static const.
> > - Added error check for power_supply_powers.
> > - Ensured devm_kasprintf return value is checked.
>
> Thanks a lot for the quick resubmission (and for Barnabás final review).
>
> I double checked that Baranabás' comments were addressed and added the
> rev-b to the series.
>
> The series is now pushed to for-5.12/playstation.
>
> Cheers,
> Benjamin
>

Great news! Thanks everyone for their help.

Thanks,
Roderick