On Mon, Oct 10, 2016 at 12:46:53AM -0400, Finn Thain wrote: > Avoid the call to NCR5380_poll_politely2() when possible. The call is > easily short-circuited on the PIO fast path, using the inline wrapper. > This requires that the NCR5380_read macro be made available before > any #include "NCR5380.h" so a few declarations have to be moved too. > > Signed-off-by: Finn Thain <fthain@xxxxxxxxxxxxxxxxxxx> > Reviewed-by: Hannes Reinecke <hare@xxxxxxxx> > Tested-by: Ondrej Zary <linux@xxxxxxxxxxxxxxxxxxxx> > Tested-by: Michael Schmitz <schmitzmic@xxxxxxxxx> > --- > diff --git a/drivers/scsi/arm/cumana_1.c b/drivers/scsi/arm/cumana_1.c > index ae1d4c6..fb7600d 100644 > --- a/drivers/scsi/arm/cumana_1.c > +++ b/drivers/scsi/arm/cumana_1.c > @@ -29,6 +29,10 @@ > #define NCR5380_implementation_fields \ > unsigned ctrl > > +struct NCR5380_hostdata; > +static u8 cumanascsi_read(struct NCR5380_hostdata *, unsigned int); > +static void cumanascsi_write(struct NCR5380_hostdata *, unsigned int, u8); > + > #include "../NCR5380.h" > > #define CTRL 0x16fc This seems to be non-obviously unrelated to this commit - should it be in some other commit? -- RMK's Patch system: http://www.armlinux.org.uk/developer/patches/ FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up according to speedtest.net. -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html