From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753243AbcGUSqJ (ORCPT ); Thu, 21 Jul 2016 14:46:09 -0400 Received: from mail-lf0-f45.google.com ([209.85.215.45]:34659 "EHLO mail-lf0-f45.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752089AbcGUSqE (ORCPT ); Thu, 21 Jul 2016 14:46:04 -0400 Subject: Re: [PATCH resend 1/5] libata-scsi: minor cleanup in ata_mselect_*() To: tom.ty89@gmail.com, tj@kernel.org, hare@suse.de, arnd@arndb.de References: Cc: sfr@canb.auug.org.au, linux-ide@vger.kernel.org, linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org, linux-next@vger.kernel.org From: Sergei Shtylyov Organization: Cogent Embedded Message-ID: Date: Thu, 21 Jul 2016 21:45:59 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hello. On 07/21/2016 09:41 PM, tom.ty89@gmail.com wrote: > From: Tom Yan > > 1. Removed a repeated bit masking in ata_mselect_control() > 2. Moved `wce`/`d_sense` assignment below the page validity checks > 3. Added/Removed empty lines where appropriate > > Signed-off-by: Tom Yan Perhaps Tejun is OK with that, but if you ask me doing 1 task per patch is a better practice. [...] MBR, Sergei