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

2014-06-16 Thread Johan Hovold
On Sun, Jun 15, 2014 at 02:23:25AM +0300, Janne Kanniainen wrote: > > Hi! > > Hi. > > >> --- /dev/null > >> +++ b/Documentation/ABI/testing/sysfs-class-hid-driver-gt683r > >> @@ -0,0 +1,10 @@ > >> +What:/sys/class/hidraw//device/state > >> +Date:Jun 2014 > >> +Kern

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

2014-06-16 Thread Johan Hovold
On Sun, Jun 15, 2014 at 05:59:40PM +0300, Janne Kanniainen wrote: > >> Ok, so you decided to continue setting mode on every LED brightness > >> update. That should be fine, but you never answered my question about > >> whether it is necessary? > > > > I decided to do it that way because official dr

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

2014-06-15 Thread Janne Kanniainen
>> Ok, so you decided to continue setting mode on every LED brightness >> update. That should be fine, but you never answered my question about >> whether it is necessary? > > I decided to do it that way because official driver did it as well. I > can check if it is necessary. Ok, I checked this o

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

2014-06-14 Thread Janne Kanniainen
> Hi! Hi. >> --- /dev/null >> +++ b/Documentation/ABI/testing/sysfs-class-hid-driver-gt683r >> @@ -0,0 +1,10 @@ >> +What:/sys/class/hidraw//device/state >> +Date:Jun 2014 >> +KernelVersion: 3.15 >> +Contact: Janne Kanniainen >> +Description: >> +

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

2014-06-14 Thread Pavel Machek
On Thu 2014-06-12 23:34:12, Janne Kanniainen wrote: > This driver adds support for USB controlled led panels that exists in > MSI GT683R laptop Hi! > --- /dev/null > +++ b/Documentation/ABI/testing/sysfs-class-hid-driver-gt683r > @@ -0,0 +1,10 @@ > +What:/sys/class/hidraw//device

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

2014-06-13 Thread Janne Kanniainen
> Ok, so you decided to continue setting mode on every LED brightness > update. That should be fine, but you never answered my question about > whether it is necessary? I decided to do it that way because official driver did it as well. I can check if it is necessary. > You're almost done. One la

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

2014-06-13 Thread Johan Hovold
On Thu, Jun 12, 2014 at 11:34:12PM +0300, Janne Kanniainen wrote: > This driver adds support for USB controlled led panels that exists in > MSI GT683R laptop > > Signed-off-by: Janne Kanniainen > --- > Changes in v2: > - sorted headers to alphabetic order > - using devm_kzalloc >

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

2014-06-12 Thread Janne Kanniainen
This driver adds support for USB controlled led panels that exists in MSI GT683R laptop Signed-off-by: Janne Kanniainen --- Changes in v2: - sorted headers to alphabetic order - using devm_kzalloc - using BIT(n) - using usb_control_msg instead of usb_submit_urb