From mboxrd@z Thu Jan 1 00:00:00 1970 From: Quentin Lambert Date: Fri, 19 Dec 2014 11:10:08 +0000 Subject: [PATCH 0/3] Minor clean up in mpt2sas Message-Id: <20141219111008.GA17968@sloth> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Nagalakshmi Nandigama , Praveen Krishnamoorthy , Sreekanth Reddy , Abhijit Mahajan , "James E.J. Bottomley" Cc: MPT-FusionLinux.pdl@avagotech.com, linux-scsi@vger.kernel.org, kernel-janitors@vger.kernel.org, linux-kernel@vger.kernel.org The first patch merges two similar functions. Since the resulting function is only called three times, it adds a parameter to differenciate between the two behaviours. The second patch proposes a simplification of the function code. The function previously used a variable to skip through the rest of the loop. It now uses a break statement to exit it. The third patch simplifies an unnecesary complicated use of a boolean variable. Quentin Lambert (3): [SCSI] mpt2sas: Merge two similar functions [SCSI] mpt2sas: Simplify the code of mpt2sas_scsih_tm_flag [SCSI] mpt2sas: Remove unnecessary use of a boolean variable drivers/scsi/mpt2sas/mpt2sas_base.c | 6 +---- drivers/scsi/mpt2sas/mpt2sas_scsih.c | 48 +++++++----------------------------- 2 files changed, 10 insertions(+), 44 deletions(-) -- 1.9.1