On 08/31/2014 11:55 PM, Christoph Hellwig wrote:
>> -scsi_extd_sense_format(unsigned char asc, unsigned char ascq) {
>> +scsi_extd_sense_format(unsigned char asc, unsigned char ascq,
>> + const char **fmt) {
>
> please move the opening brace to a new line per normal Linux style.
> -scsi_extd_sense_format(unsigned char asc, unsigned char ascq) {
> +scsi_extd_sense_format(unsigned char asc, unsigned char ascq,
> +const char **fmt) {
please move the opening brace to a new line per normal Linux style.
Otherwise looks good,
Reviewed-by: Christoph Hellwig
2 matches
Mail list logo