Re: [PATCH] scsi: aacraid: Fix missing break in switch statement

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: "Gustavo A. R. Silva" <gustavo@xxxxxxxxxxxxxx>
Subject: Re: [PATCH] scsi: aacraid: Fix missing break in switch statement
From: "Martin K. Petersen" <martin.petersen@xxxxxxxxxx>
Date: Wed, 27 Feb 2019 09:33:47 -0500
Cc: Adaptec OEM Raid Solutions <aacraid@xxxxxxxxxxxxx>, "James E.J. Bottomley" <jejb@xxxxxxxxxxxxx>, "Martin K. Petersen" <martin.petersen@xxxxxxxxxx>, Raghava Aditya Renukunta <RaghavaAditya.Renukunta@xxxxxxxxxxxxx>, linux-scsi@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, Kees Cook <keescook@xxxxxxxxxxxx>
In-reply-to: <20190215214242.GA8393@embeddedor> (Gustavo A. R. Silva's message of "Fri, 15 Feb 2019 15:42:42 -0600")
Organization: Oracle Corporation
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/26.1 (gnu/linux)

Gustavo,

> Add missing break statement and fix identation issue.

So much bad indentation in this driver. Tried various tab widths and it
is still all over the place.

Anyway. After staring at the firmware header I have convinced myself
that your fix is correct. Applied to 5.1/scsi-queue.

-- 
Martin K. Petersen	Oracle Linux Engineering

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Follow-Ups:
- Re: [PATCH] scsi: aacraid: Fix missing break in switch statement
  - From: Gustavo A. R. Silva

References:
- [PATCH] scsi: aacraid: Fix missing break in switch statement
  - From: Gustavo A. R. Silva

Prev by Date: Re: [PATCHv2 0/4] scsi: scsi: kill scsi command serial number
Next by Date: Re: [PATCH] scsi: mpt3sas: Add missing breaks in switch statements
Previous by thread: [PATCH] scsi: aacraid: Fix missing break in switch statement
Next by thread: Re: [PATCH] scsi: aacraid: Fix missing break in switch statement
Index(es):
- Date
- Thread

[Index of Archives] [SCSI Target Devel] [Linux SCSI Target Infrastructure] [Kernel Newbies] [IDE] [Security] [Git] [Netfilter] [Bugtraq] [Yosemite News] [MIPS Linux] [ARM Linux] [Linux Security] [Linux RAID] [Linux ATA RAID] [Linux IIO] [Samba] [Device Mapper]