Re: [PATCH v9] leds: USB: HID: Add support for MSI GT683R led panels

2014-06-23 Thread Oliver Neukum
On Wed, 2014-06-18 at 19:05 +0300, Janne Kanniainen wrote: This driver adds support for USB controlled led panels that exists in MSI GT683R laptop +static int gt683r_led_probe(struct hid_device *hdev, + const struct hid_device_id *id) +{ + int i; + int ret; +

Re: [PATCH v9] leds: USB: HID: Add support for MSI GT683R led panels

2014-06-23 Thread Johan Hovold
On Mon, Jun 23, 2014 at 04:35:13PM +0200, Oliver Neukum wrote: On Wed, 2014-06-18 at 19:05 +0300, Janne Kanniainen wrote: This driver adds support for USB controlled led panels that exists in MSI GT683R laptop +static int gt683r_led_probe(struct hid_device *hdev, +

Re: [PATCH v9] leds: USB: HID: Add support for MSI GT683R led panels

2014-06-23 Thread Greg KH
On Mon, Jun 23, 2014 at 04:42:55PM +0200, Johan Hovold wrote: On Mon, Jun 23, 2014 at 04:35:13PM +0200, Oliver Neukum wrote: On Wed, 2014-06-18 at 19:05 +0300, Janne Kanniainen wrote: This driver adds support for USB controlled led panels that exists in MSI GT683R laptop +static

Re: [PATCH v9] leds: USB: HID: Add support for MSI GT683R led panels

2014-06-23 Thread Janne Kanniainen
Sorry for noticing this thread late. No problem. Good that you noticed it now! Thank you. Just move the initialisation of the lock and work to the other private data initialisations directly after it's allocated. Can you send a follow up patch, Janne? Yes I can. Just a moment. Janne -- To

[PATCH v9] leds: USB: HID: Add support for MSI GT683R led panels

2014-06-18 Thread Janne Kanniainen
This driver adds support for USB controlled led panels that exists in MSI GT683R laptop Signed-off-by: Janne Kanniainen janne.kanniai...@gmail.com --- Changes in v2: - sorted headers to alphabetic order - using devm_kzalloc - using BIT(n) - using usb_control_msg

Re: [PATCH v9] leds: USB: HID: Add support for MSI GT683R led panels

2014-06-18 Thread Johan Hovold
On Wed, Jun 18, 2014 at 07:05:02PM +0300, Janne Kanniainen wrote: This driver adds support for USB controlled led panels that exists in MSI GT683R laptop Signed-off-by: Janne Kanniainen janne.kanniai...@gmail.com Reviewed-by: Johan Hovold jo...@kernel.org Thanks, Janne! Johan ---

Re: [PATCH v9] leds: USB: HID: Add support for MSI GT683R led panels

2014-06-18 Thread Janne Kanniainen
This driver adds support for USB controlled led panels that exists in MSI GT683R laptop Signed-off-by: Janne Kanniainen janne.kanniai...@gmail.com Reviewed-by: Johan Hovold jo...@kernel.org Thanks, Janne! Johan Thank you for reviewing my patch :) I sure learnt a lot from you and if I

Re: [PATCH v9] leds: USB: HID: Add support for MSI GT683R led panels

2014-06-18 Thread Johan Hovold
On Wed, Jun 18, 2014 at 09:41:35PM +0300, Janne Kanniainen wrote: This driver adds support for USB controlled led panels that exists in MSI GT683R laptop Signed-off-by: Janne Kanniainen janne.kanniai...@gmail.com Reviewed-by: Johan Hovold jo...@kernel.org Thanks, Janne! Johan

Re: [PATCH v9] leds: USB: HID: Add support for MSI GT683R led panels

2014-06-18 Thread Jiri Kosina
On Wed, 18 Jun 2014, Johan Hovold wrote: This driver adds support for USB controlled led panels that exists in MSI GT683R laptop Signed-off-by: Janne Kanniainen janne.kanniai...@gmail.com Reviewed-by: Johan Hovold jo...@kernel.org Thanks, Janne! Now applied. Thanks Janne for the