Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752110Ab3FXVIO (ORCPT ); Mon, 24 Jun 2013 17:08:14 -0400 Received: from mx1.redhat.com ([209.132.183.28]:46151 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751060Ab3FXVIM (ORCPT ); Mon, 24 Jun 2013 17:08:12 -0400 Date: Mon, 24 Jun 2013 17:07:47 -0400 From: Dave Jones To: Khalid Aziz Cc: James.Bottomley@HansenPartnership.com, linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org, Khalid Aziz Subject: Re: [PATCH RESEND v2 2/2] scsi: 64-bit port of buslogic driver Message-ID: <20130624210747.GA19721@redhat.com> Mail-Followup-To: Dave Jones , Khalid Aziz , James.Bottomley@HansenPartnership.com, linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org, Khalid Aziz References: <97daf2d29ceb78bdef3ddf6358dc634de35d0345.1368754556.git.khalid.aziz@oracle.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <97daf2d29ceb78bdef3ddf6358dc634de35d0345.1368754556.git.khalid.aziz@oracle.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 972 Lines: 27 On Mon, Jun 24, 2013 at 02:26:00PM -0600, Khalid Aziz wrote: > @@ -821,7 +821,7 @@ struct blogic_ccb { > unsigned char cdblen; /* Byte 2 */ > unsigned char sense_datalen; /* Byte 3 */ > u32 datalen; /* Bytes 4-7 */ > - u32 data; /* Bytes 8-11 */ > + void *data; /* Bytes 8-11 */ The comment (and subsequent ones for this struct) are now wrong on 64-bit. More of a concern though is that above this struct is this... /* Define the 32 Bit Mode Command Control Block (CCB) structure. The first 40 bytes are defined by and common to both the MultiMaster Firmware and the FlashPoint SCCB Manager. Is it safe to change this if the firmware/3rd party sw is expecting it to be a u32 ? Dave -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/