Re: [PATCH v6 3/3] leds/powernv: Add driver for PowerNV platform

2015-07-21 Thread Jacek Anaszewski
Vasant, On 21.07.2015 08:55, Vasant Hegde wrote: On 07/21/2015 11:24 AM, Vasant Hegde wrote: On 07/20/2015 03:10 AM, Jacek Anaszewski wrote: Hi Vasant, Jacek, I've revised your patch and found few more issues. Please refer to my comments below. Thanks. .../... Please don't exceed 75

Re: [PATCH v6 3/3] leds/powernv: Add driver for PowerNV platform

2015-07-21 Thread Jacek Anaszewski
Hi Vasant, On 21.07.2015 07:54, Vasant Hegde wrote: On 07/20/2015 03:10 AM, Jacek Anaszewski wrote: Hi Vasant, Jacek, I've revised your patch and found few more issues. Please refer to my comments below. Thanks. .../... Please don't exceed 75 character line length limit. Ok. I will

Re: [PATCH v6 3/3] leds/powernv: Add driver for PowerNV platform

2015-07-20 Thread Vasant Hegde
On 07/21/2015 11:24 AM, Vasant Hegde wrote: > On 07/20/2015 03:10 AM, Jacek Anaszewski wrote: >> Hi Vasant, >> > > Jacek, > >> I've revised your patch and found few more issues. >> Please refer to my comments below. > > Thanks. > > .../... > Please don't exceed 75 character line leng

Re: [PATCH v6 3/3] leds/powernv: Add driver for PowerNV platform

2015-07-20 Thread Vasant Hegde
On 07/20/2015 03:10 AM, Jacek Anaszewski wrote: > Hi Vasant, > Jacek, > I've revised your patch and found few more issues. > Please refer to my comments below. Thanks. .../... >>> >>> Please don't exceed 75 character line length limit. >> >> Ok. I will fix it.. But I thought 80 character is t

Re: [PATCH v6 3/3] leds/powernv: Add driver for PowerNV platform

2015-07-20 Thread Vasant Hegde
On 07/20/2015 11:46 AM, Jacek Anaszewski wrote: > On 19.07.2015 23:40, Jacek Anaszewski wrote: > [...] > +/* Platform driver probe */ > +static int powernv_led_probe(struct platform_device *pdev) > +{ > +int num_leds; > +struct device_node *led_node; > +struct po

Re: [PATCH v6 3/3] leds/powernv: Add driver for PowerNV platform

2015-07-19 Thread Jacek Anaszewski
On 19.07.2015 23:40, Jacek Anaszewski wrote: [...] +/* Platform driver probe */ +static int powernv_led_probe(struct platform_device *pdev) +{ +int num_leds; +struct device_node *led_node; +struct powernv_leds_priv *priv; + +led_node = of_find_node_by_path("/ibm,opal/leds"); +

Re: [PATCH v6 3/3] leds/powernv: Add driver for PowerNV platform

2015-07-19 Thread Jacek Anaszewski
Hi Vasant, I've revised your patch and found few more issues. Please refer to my comments below. On 17.07.2015 18:20, Vasant Hegde wrote: On 07/17/2015 08:55 PM, Jacek Anaszewski wrote: Hi Vasant, Hi Jacek, .../... As per the LED class framework, the 'brightness_set' function should no

Re: [PATCH v6 3/3] leds/powernv: Add driver for PowerNV platform

2015-07-17 Thread Vasant Hegde
On 07/17/2015 08:55 PM, Jacek Anaszewski wrote: > Hi Vasant, Hi Jacek, .../... >> As per the LED class framework, the 'brightness_set' function should not >> sleep. Hence these functions have been implemented through global work >> queue tasks which might sleep on OPAL async call completion. >

Re: [PATCH v6 3/3] leds/powernv: Add driver for PowerNV platform

2015-07-17 Thread Jacek Anaszewski
Hi Vasant, On 07/17/2015 12:41 PM, Vasant Hegde wrote: This patch implements LED driver for PowerNV platform using the existing generic LED class framework. PowerNV platform has below type of LEDs: - System attention Indicates there is a problem with the system that needs attention.

[PATCH v6 3/3] leds/powernv: Add driver for PowerNV platform

2015-07-17 Thread Vasant Hegde
This patch implements LED driver for PowerNV platform using the existing generic LED class framework. PowerNV platform has below type of LEDs: - System attention Indicates there is a problem with the system that needs attention. - Identify Helps the user locate/identify a particula