[PATCH v4 07/11] cxl: Refactor AFU init/teardown
    Ian Munsie 
    imunsie at au1.ibm.com
       
    Fri Aug 14 16:22:42 AEST 2015
    
    
  
Excerpts from Daniel Axtens's message of 2015-08-13 14:11:25 +1000:
> +    rc = cxl_map_slice_regs(afu, adapter, dev);
> +    if (rc)
> +        return rc;
>  
> -    if ((rc = cxl_map_slice_regs(afu, adapter, dev)))
Like the previous patch, mixing this coding style change in with this
patch makes the diff harder to follow than necessary (though not as hard
as the last one). If you happen to do a v5, please put the coding style
changes in a separate patch, but otherwise it looks fine:
Acked-by: Ian Munsie <imunsie at au1.ibm.com>
    
    
More information about the Linuxppc-dev
mailing list