Re: [PATCH 08/13] powerpc: Add new _EXCEPTION_PROLOG_1 macro

2016-09-18 Thread Madhavan Srinivasan 



On Friday 16 September 2016 03:42 PM, Nicholas Piggin wrote:

On Thu, 15 Sep 2016 18:31:58 +0530
Madhavan Srinivasan  wrote:


To support addition of "bitmask" to MASKABLE_* macros,
factor out the EXCPETION_PROLOG_1 macro.

Signed-off-by: Madhavan Srinivasan 

Really minor nit, but as a matter of readability of the series,
would you consider moving this next to patch 10 where it's used,
if you submit again?

Yes sure. Make sense. Will do it.

Maddy



Reviewed-by: Nicholas Piggin

Re: [PATCH 08/13] powerpc: Add new _EXCEPTION_PROLOG_1 macro

2016-09-16 Thread Nicholas Piggin 
On Thu, 15 Sep 2016 18:31:58 +0530
Madhavan Srinivasan  wrote:

> To support addition of "bitmask" to MASKABLE_* macros,
> factor out the EXCPETION_PROLOG_1 macro.
> 
> Signed-off-by: Madhavan Srinivasan 

Really minor nit, but as a matter of readability of the series,
would you consider moving this next to patch 10 where it's used,
if you submit again?

Reviewed-by: Nicholas Piggin