Re: [PATCH 0/2] Staging: unisys: visorbus: style fix, using octal file permissions

2017-01-09 Thread Andy Shevchenko 
On Sat, Jan 7, 2017 at 5:48 AM, Derek Robson  wrote:
> Two files change in style fix, changes are octal file permissions.

Does it make any sense? Or did I miss something?

-- 
With Best Regards,
Andy Shevchenko

Re: [PATCH 0/2] Staging: unisys: visorbus: style fix, using octal file permissions

2017-01-09 Thread Andy Shevchenko 
On Sat, Jan 7, 2017 at 5:48 AM, Derek Robson  wrote:
> Two files change in style fix, changes are octal file permissions.

Does it make any sense? Or did I miss something?

-- 
With Best Regards,
Andy Shevchenko

Re: [PATCH 0/2] Staging: unisys: visorbus: style fix, using octal file permissions

2017-01-06 Thread Greg KH 
On Sat, Jan 07, 2017 at 04:48:01PM +1300, Derek Robson wrote:
> Two files change in style fix, changes are octal file permissions.
> 
> Derek Robson (2):
>   Staging: unisys: visorbus: style fix, using octal file permissions
>   Staging: unisys: visorbus: style fix, using octal file permissions

You can't send different patches that do different things with identical
subject lines, as that doesn't make sense, right?

Please resend making these subject unique, and add David's ack to them
as well on the resend.

thanks,

greg k-h

Re: [PATCH 0/2] Staging: unisys: visorbus: style fix, using octal file permissions

2017-01-06 Thread Greg KH 
On Sat, Jan 07, 2017 at 04:48:01PM +1300, Derek Robson wrote:
> Two files change in style fix, changes are octal file permissions.
> 
> Derek Robson (2):
>   Staging: unisys: visorbus: style fix, using octal file permissions
>   Staging: unisys: visorbus: style fix, using octal file permissions

You can't send different patches that do different things with identical
subject lines, as that doesn't make sense, right?

Please resend making these subject unique, and add David's ack to them
as well on the resend.

thanks,

greg k-h

RE: [PATCH 0/2] Staging: unisys: visorbus: style fix, using octal file permissions

2017-01-06 Thread Kershner, David A 
> -Original Message-
> From: Derek Robson [mailto:robso...@gmail.com]
> Sent: Friday, January 6, 2017 10:48 PM
> To: Kershner, David A ;
> gre...@linuxfoundation.org; Sell, Timothy C 
> Cc: *S-Par-Maintainer ;
> de...@driverdev.osuosl.org; linux-kernel@vger.kernel.org; Derek Robson
> 
> Subject: [PATCH 0/2] Staging: unisys: visorbus: style fix, using octal file
> permissions
> 
> Two files change in style fix, changes are octal file permissions.
> 

Series looks fine to me and runs on top of the s-Par firmware.

Acked-by: David Kershner 

> Derek Robson (2):
>   Staging: unisys: visorbus: style fix, using octal file permissions
>   Staging: unisys: visorbus: style fix, using octal file permissions
> 
>  drivers/staging/unisys/visorbus/visorbus_main.c | 6 +++---
>  drivers/staging/unisys/visorbus/visorchipset.c  | 2 +-
>  2 files changed, 4 insertions(+), 4 deletions(-)
> 
> --
> 2.11.0

RE: [PATCH 0/2] Staging: unisys: visorbus: style fix, using octal file permissions

2017-01-06 Thread Kershner, David A 
> -Original Message-
> From: Derek Robson [mailto:robso...@gmail.com]
> Sent: Friday, January 6, 2017 10:48 PM
> To: Kershner, David A ;
> gre...@linuxfoundation.org; Sell, Timothy C 
> Cc: *S-Par-Maintainer ;
> de...@driverdev.osuosl.org; linux-kernel@vger.kernel.org; Derek Robson
> 
> Subject: [PATCH 0/2] Staging: unisys: visorbus: style fix, using octal file
> permissions
> 
> Two files change in style fix, changes are octal file permissions.
> 

Series looks fine to me and runs on top of the s-Par firmware.

Acked-by: David Kershner 

> Derek Robson (2):
>   Staging: unisys: visorbus: style fix, using octal file permissions
>   Staging: unisys: visorbus: style fix, using octal file permissions
> 
>  drivers/staging/unisys/visorbus/visorbus_main.c | 6 +++---
>  drivers/staging/unisys/visorbus/visorchipset.c  | 2 +-
>  2 files changed, 4 insertions(+), 4 deletions(-)
> 
> --
> 2.11.0