[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <Pine.LNX.4.64.0710162121330.29643@anakin>
Date: Tue, 16 Oct 2007 21:21:48 +0200 (CEST)
From: Geert Uytterhoeven <geert@...ux-m68k.org>
To: Boaz Harrosh <bharrosh@...asas.com>
Cc: James Bottomley <James.Bottomley@...elEye.com>,
Andrew Morton <akpm@...ux-foundation.org>,
Linus Torvalds <torvalds@...ux-foundation.org>,
linux-scsi <linux-scsi@...r.kernel.org>,
linux-kernel <linux-kernel@...r.kernel.org>,
Linux/m68k <linux-m68k@...r.kernel.org>
Subject: Re: [GIT PATCH] SCSI updates for 2.6.24
On Tue, 16 Oct 2007, Boaz Harrosh wrote:
> On Tue, Oct 16 2007 at 8:49 +0200, Geert Uytterhoeven <geert@...ux-m68k.org> wrote:
> > On Mon, 15 Oct 2007, James Bottomley wrote:
> >> This is the accumulated updates queued for 2.6.24. It contains the
> >> usual slew of driver updates, plus some gdth and advansys rewrites. We
> >> still have some outstanding bugs in gdth and fc4 for which I'm hoping to
> >> sweep fixes into the next update.
> >
> >> Boaz Harrosh (12):
> >> NCR5380: Use scsi_eh API for REQUEST_SENSE invocation
> >
> > This change broke compilation of the mac_scsi.c driver:
> >
> > | linux/drivers/scsi/NCR5380.c: In function 'NCR5380_information_transfer':
> > | linux/drivers/scsi/NCR5380.c:2282: error: 'struct NCR5380_hostdata' has no member named 'ses'
> > | linux/drivers/scsi/NCR5380.c:2283: error: 'struct NCR5380_hostdata' has no member named 'ses'
> > | linux/drivers/scsi/NCR5380.c:2284: error: 'struct NCR5380_hostdata' has no member named 'ses'
> > | linux/drivers/scsi/NCR5380.c:2288: error: 'struct NCR5380_hostdata' has no member named 'ses'
> >
> fix below, Sorry.
> ----
> >From 1e10cf1a39ff4909696a5c7950367261b38b3345 Mon Sep 17 00:00:00 2001
> From: Boaz Harrosh <bharrosh@...asas.com>
> Date: Tue, 16 Oct 2007 10:25:01 +0200
> Subject: [PATCH] scsi_mac.h: Define AUTOSENSE before include of NCR5380.h
>
> - Previese patch to NCR5380 broke scsi_mac because
> AUTOSENSE was defined after the inclusion of
> NCR5380.h. Fix it
>
> Signed-off-by: Boaz Harrosh <bharrosh@...asas.com>
Acked-by: Geert Uytterhoeven <geert@...ux-m68k.org>
> ---
> drivers/scsi/mac_scsi.c | 9 +++++----
> 1 files changed, 5 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/scsi/mac_scsi.c b/drivers/scsi/mac_scsi.c
> index cdbcaa5..abe2bda 100644
> --- a/drivers/scsi/mac_scsi.c
> +++ b/drivers/scsi/mac_scsi.c
> @@ -53,6 +53,11 @@
> #include "scsi.h"
> #include <scsi/scsi_host.h>
> #include "mac_scsi.h"
> +
> +/* These control the behaviour of the generic 5380 core */
> +#define AUTOSENSE
> +#define PSEUDO_DMA
> +
> #include "NCR5380.h"
>
> #if 0
> @@ -571,10 +576,6 @@ static int macscsi_pwrite (struct Scsi_Host *instance,
> }
>
>
> -/* These control the behaviour of the generic 5380 core */
> -#define AUTOSENSE
> -#define PSEUDO_DMA
> -
> #include "NCR5380.c"
>
> static struct scsi_host_template driver_template = {
> --
> 1.5.3.1
>
>
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@...ux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists