Re: [PATCH 1/3] acpi: video: trivial costmetic cleanups

2013-08-03 Thread Felipe Contreras
On Sat, Aug 3, 2013 at 6:38 AM, Rafael J. Wysocki wrote: > On Friday, August 02, 2013 08:34:29 PM Felipe Contreras wrote: >> On Fri, Aug 2, 2013 at 7:07 PM, Rafael J. Wysocki wrote: >> > On Friday, August 02, 2013 12:52:18 PM Felipe Contreras wrote: >> >> On Fri, Aug 2, 2013 at 9:05 AM, Rafael J.

Re: [PATCH 1/3] acpi: video: trivial costmetic cleanups

2013-08-03 Thread Rafael J. Wysocki
On Friday, August 02, 2013 08:34:29 PM Felipe Contreras wrote: > On Fri, Aug 2, 2013 at 7:07 PM, Rafael J. Wysocki wrote: > > On Friday, August 02, 2013 12:52:18 PM Felipe Contreras wrote: > >> On Fri, Aug 2, 2013 at 9:05 AM, Rafael J. Wysocki wrote: > >> > On Thursday, August 01, 2013 11:15:38 P

Re: [PATCH 1/3] acpi: video: trivial costmetic cleanups

2013-08-02 Thread Felipe Contreras
On Fri, Aug 2, 2013 at 7:07 PM, Rafael J. Wysocki wrote: > On Friday, August 02, 2013 12:52:18 PM Felipe Contreras wrote: >> On Fri, Aug 2, 2013 at 9:05 AM, Rafael J. Wysocki wrote: >> > On Thursday, August 01, 2013 11:15:38 PM Felipe Contreras wrote: >> >> On Thu, Aug 1, 2013 at 8:50 PM, Aaron L

Re: [PATCH 1/3] acpi: video: trivial costmetic cleanups

2013-08-02 Thread Rafael J. Wysocki
On Friday, August 02, 2013 12:52:18 PM Felipe Contreras wrote: > On Fri, Aug 2, 2013 at 9:05 AM, Rafael J. Wysocki wrote: > > On Thursday, August 01, 2013 11:15:38 PM Felipe Contreras wrote: > >> On Thu, Aug 1, 2013 at 8:50 PM, Aaron Lu wrote: > >> > On 08/02/2013 07:43 AM, Felipe Contreras wrote

Re: [PATCH 1/3] acpi: video: trivial costmetic cleanups

2013-08-02 Thread Felipe Contreras
On Fri, Aug 2, 2013 at 9:05 AM, Rafael J. Wysocki wrote: > On Thursday, August 01, 2013 11:15:38 PM Felipe Contreras wrote: >> On Thu, Aug 1, 2013 at 8:50 PM, Aaron Lu wrote: >> > On 08/02/2013 07:43 AM, Felipe Contreras wrote: >> >> Signed-off-by: Felipe Contreras >> > >> > Please add change lo

Re: [PATCH 1/3] acpi: video: trivial costmetic cleanups

2013-08-02 Thread Rafael J. Wysocki
On Thursday, August 01, 2013 11:15:38 PM Felipe Contreras wrote: > On Thu, Aug 1, 2013 at 8:50 PM, Aaron Lu wrote: > > On 08/02/2013 07:43 AM, Felipe Contreras wrote: > >> Signed-off-by: Felipe Contreras > > > > Please add change log explaining what you have changed. > > It seems that the patch m

Re: [PATCH 1/3] acpi: video: trivial costmetic cleanups

2013-08-01 Thread Felipe Contreras
On Thu, Aug 1, 2013 at 8:50 PM, Aaron Lu wrote: > On 08/02/2013 07:43 AM, Felipe Contreras wrote: >> Signed-off-by: Felipe Contreras > > Please add change log explaining what you have changed. > It seems that the patch modify comment style only, some add a space and > some change spaces to tab, i

Re: [PATCH 1/3] acpi: video: trivial costmetic cleanups

2013-08-01 Thread Aaron Lu
On 08/02/2013 07:43 AM, Felipe Contreras wrote: > Signed-off-by: Felipe Contreras Please add change log explaining what you have changed. It seems that the patch modify comment style only, some add a space and some change spaces to tab, is it the case? Thanks, Aaron > --- > drivers/acpi/video.

[PATCH 1/3] acpi: video: trivial costmetic cleanups

2013-08-01 Thread Felipe Contreras
Signed-off-by: Felipe Contreras --- drivers/acpi/video.c | 114 +++ 1 file changed, 61 insertions(+), 53 deletions(-) diff --git a/drivers/acpi/video.c b/drivers/acpi/video.c index 0ec434d..f021bf4 100644 --- a/drivers/acpi/video.c +++ b/drivers/ac