Re: [PATCH v2] leds: Introduce userspace leds driver

2016-09-16 Thread Jacek Anaszewski
Hi David, On 09/15/2016 05:25 PM, David Lechner wrote: On 09/15/2016 09:54 AM, Jacek Anaszewski wrote: Hi Pavel, On 09/15/2016 03:01 PM, Pavel Machek wrote: Hi! @@ -0,0 +1,23 @@ +/* + * Userspace driver support for leds subsystem + * + * This program is free software; you can redistribute

Re: [PATCH v2] leds: Introduce userspace leds driver

2016-09-16 Thread Jacek Anaszewski
Hi David, On 09/15/2016 05:25 PM, David Lechner wrote: On 09/15/2016 09:54 AM, Jacek Anaszewski wrote: Hi Pavel, On 09/15/2016 03:01 PM, Pavel Machek wrote: Hi! @@ -0,0 +1,23 @@ +/* + * Userspace driver support for leds subsystem + * + * This program is free software; you can redistribute

Re: [PATCH v2] leds: Introduce userspace leds driver

2016-09-15 Thread David Lechner
On 09/15/2016 09:54 AM, Jacek Anaszewski wrote: Hi Pavel, On 09/15/2016 03:01 PM, Pavel Machek wrote: Hi! @@ -0,0 +1,23 @@ +/* + * Userspace driver support for leds subsystem + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU

Re: [PATCH v2] leds: Introduce userspace leds driver

2016-09-15 Thread David Lechner
On 09/15/2016 09:54 AM, Jacek Anaszewski wrote: Hi Pavel, On 09/15/2016 03:01 PM, Pavel Machek wrote: Hi! @@ -0,0 +1,23 @@ +/* + * Userspace driver support for leds subsystem + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU

Re: [PATCH v2] leds: Introduce userspace leds driver

2016-09-15 Thread David Lechner
On 09/15/2016 09:54 AM, Jacek Anaszewski wrote: Hi Pavel, On 09/15/2016 03:01 PM, Pavel Machek wrote: Hi! @@ -0,0 +1,23 @@ +/* + * Userspace driver support for leds subsystem + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU

Re: [PATCH v2] leds: Introduce userspace leds driver

2016-09-15 Thread David Lechner
On 09/15/2016 09:54 AM, Jacek Anaszewski wrote: Hi Pavel, On 09/15/2016 03:01 PM, Pavel Machek wrote: Hi! @@ -0,0 +1,23 @@ +/* + * Userspace driver support for leds subsystem + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU

Re: [PATCH v2] leds: Introduce userspace leds driver

2016-09-15 Thread Jacek Anaszewski
Hi Pavel, On 09/15/2016 03:01 PM, Pavel Machek wrote: Hi! @@ -0,0 +1,23 @@ +/* + * Userspace driver support for leds subsystem + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free

Re: [PATCH v2] leds: Introduce userspace leds driver

2016-09-15 Thread Jacek Anaszewski
Hi Pavel, On 09/15/2016 03:01 PM, Pavel Machek wrote: Hi! @@ -0,0 +1,23 @@ +/* + * Userspace driver support for leds subsystem + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free

Re: [PATCH v2] leds: Introduce userspace leds driver

2016-09-15 Thread Pavel Machek
Hi! > >@@ -0,0 +1,23 @@ > >+/* > >+ * Userspace driver support for leds subsystem > >+ * > >+ * This program is free software; you can redistribute it and/or modify > >+ * it under the terms of the GNU General Public License as published by > >+ * the Free Software Foundation; either version 2 of

Re: [PATCH v2] leds: Introduce userspace leds driver

2016-09-15 Thread Pavel Machek
Hi! > >@@ -0,0 +1,23 @@ > >+/* > >+ * Userspace driver support for leds subsystem > >+ * > >+ * This program is free software; you can redistribute it and/or modify > >+ * it under the terms of the GNU General Public License as published by > >+ * the Free Software Foundation; either version 2 of

Re: [PATCH v2] leds: Introduce userspace leds driver

2016-09-09 Thread Jacek Anaszewski
Hi David, Thanks for the update. Even more nitpicking below. Please also rebase the patch on top of current linux-leds.git, for-next branch. On 09/08/2016 09:04 PM, David Lechner wrote: This driver creates a userspace leds driver similar to uinput. New leds are created by opening /dev/uleds

Re: [PATCH v2] leds: Introduce userspace leds driver

2016-09-09 Thread Jacek Anaszewski
Hi David, Thanks for the update. Even more nitpicking below. Please also rebase the patch on top of current linux-leds.git, for-next branch. On 09/08/2016 09:04 PM, David Lechner wrote: This driver creates a userspace leds driver similar to uinput. New leds are created by opening /dev/uleds

Re: [PATCH v2] leds: Introduce userspace leds driver

2016-09-08 Thread Peter Meerwald-Stadler
> This driver creates a userspace leds driver similar to uinput. nitpicking below > New leds are created by opening /dev/uleds and writing a uleds_user_dev > struct. A new leds class device is registered with the name given in the > struct. Reading will return a single byte that is the current

Re: [PATCH v2] leds: Introduce userspace leds driver

2016-09-08 Thread Peter Meerwald-Stadler
> This driver creates a userspace leds driver similar to uinput. nitpicking below > New leds are created by opening /dev/uleds and writing a uleds_user_dev > struct. A new leds class device is registered with the name given in the > struct. Reading will return a single byte that is the current

[PATCH v2] leds: Introduce userspace leds driver

2016-09-08 Thread David Lechner
This driver creates a userspace leds driver similar to uinput. New leds are created by opening /dev/uleds and writing a uleds_user_dev struct. A new leds class device is registered with the name given in the struct. Reading will return a single byte that is the current brightness. The poll()

[PATCH v2] leds: Introduce userspace leds driver

2016-09-08 Thread David Lechner
This driver creates a userspace leds driver similar to uinput. New leds are created by opening /dev/uleds and writing a uleds_user_dev struct. A new leds class device is registered with the name given in the struct. Reading will return a single byte that is the current brightness. The poll()