From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753476Ab1AAXeR (ORCPT ); Sat, 1 Jan 2011 18:34:17 -0500 Received: from palinux.external.hp.com ([192.25.206.14]:41304 "EHLO mail.parisc-linux.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750973Ab1AAXeQ (ORCPT ); Sat, 1 Jan 2011 18:34:16 -0500 Date: Sat, 1 Jan 2011 16:34:14 -0700 From: Matthew Wilcox To: Torsten Kaiser Cc: roel kluin , nick.cheng@areca.com.tw, Andrew Morton , LKML , James.Bottomley@suse.de, linux-scsi@vger.kernel.org Subject: Re: [PATCH] [SCSI] arcmsr: &/| confusion in arcmsr_build_ccb() Message-ID: <20110101233414.GA13577@parisc-linux.org> References: <4D1F5A17.4010807@gmail.com> <20110101170854.GB32652@parisc-linux.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.18 (2008-05-17) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sat, Jan 01, 2011 at 06:28:34PM +0100, Torsten Kaiser wrote: > > - if (pcmd->sc_data_direction == DMA_TO_DEVICE ) { > + if (pcmd->cmnd[0]|WRITE_6 || pcmd->cmnd[0] | WRITE_10 || > pcmd->cmnd[0]|WRITE_12) { > > Until June 2010 it was that way... So it was. Nick, why did you make this change? Your commit message is less than informative, partly because you're making such huge changes in a single commit. -- Matthew Wilcox Intel Open Source Technology Centre "Bill, look, we understand that you're interested in selling us this operating system, but compare it to ours. We can't possibly take such a retrograde step."