Re: Review request for four patch submissions

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi there,

this is a gentle review request for these patch submissions
which still are open:


> [PATCH v3] HID: hid-bigbenff: driver for BigBen Interactive, PS3OFMINIPAD gamepad
> Message-ID: <196c9828-7d2e-de60-b984-9bb3bd80513a@xxxxxxxx>

The issues mentioned in the review of v2 have been resolved,
but you made two comments where I asked for feedback in
<7a8dac82-a935-a4d6-9c6a-9559dc36f35b@xxxxxxxx>
that haven't been responded to yet.


> [PATCH 0/5] HID: hid-sony.c: Use devm_ api to simplify Sony game device driver
> Message-ID: <76bacb70-f61e-6b00-233b-92c43aff1fca@xxxxxxxx>

The code has been tested by Roderick's team at Sony - see
<CAEc3jaCfpt1mCimsL2jkfMnLhLenuZAabJqiJRSsAhtKi08zQw@xxxxxxxxxxxxxx>
- and they appear to be happy with the change.


> [PATCH v2] HID: hid-sony: Brighter RGB values and add 8th color.
> Message-ID: <d379c448-e6b2-f858-bffa-5c3256c8030a@xxxxxxxx>

v2 fixes the maximum brightness that Roderick commented on in
<CAEc3jaBvQCYBm6PEPexE_fRn619uRZBdDi6rKie1RagjDU7sww@xxxxxxxxxxxxxx>
and I hope that it's now ready for acceptance.


> [PATCH 0/1] HID: hid-betopff.c: Use devm_kzalloc, move _init() code into _probe().
> Message-ID: <b992e649-f5f4-06c0-09a4-876c1b8a11ef@xxxxxxxx>

There has been no response from Huang, the driver's original
author, but he doesn't seem to contribute much to Linux mailing
lists, today. There have been no comments from the
linux-input mailing list, also.

The patch is just adding the use devm_ and it's also a small
code reshuffle to fix an error handling issue that the
original code forgot to deal with.


Thanks a lot,

Hanno
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Media Devel]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Linux Wireless Networking]     [Linux Omap]

  Powered by Linux