On Tue, 2013-02-12 at 04:27 -0800, Anand wrote: > >From edeb2b4b89ff6c7ddbb73366a2b4f6a86815d8eb Mon Sep 17 00:00:00 2001 > From: Sakthivel K <Sakthivel.SaravananKamalRaju@xxxxxxxx> > Date: Mon, 4 Feb 2013 12:15:28 +0530 > Subject: [PATCH 02/15] pm80xx: Added vendor and device ids for > SPCv/SPCve controllers > > Updated of pci id table with device, vendor, subdevice and subvendor > ids for 8081, 8088, 8089 SAS/SATA controllers > > Signed-off-by: Sakthivel K <DL.Team.PMC.SPC.Linux.open.source@xxxxxxxx> > Signed-off-by: Anand Kumar S <DL.Team.PMC.SPC.Linux.open.source@xxxxxxxx> This doesn't build: CC [M] drivers/scsi/pm8001/pm8001_init.o drivers/scsi/pm8001/pm8001_init.c:857:37: error: ‘chip_8008’ undeclared here (not in a function) drivers/scsi/pm8001/pm8001_init.c:859:37: error: ‘chip_8018’ undeclared here (not in a function) drivers/scsi/pm8001/pm8001_init.c:861:37: error: ‘chip_8009’ undeclared here (not in a function) drivers/scsi/pm8001/pm8001_init.c:863:37: error: ‘chip_8019’ undeclared here (not in a function) make[3]: *** [drivers/scsi/pm8001/pm8001_init.o] Error 1 That's because you don't define this chip_8019 until patch 03/15 ... that doesn't work. The series has to be bisectable. That means that each patch applied in series must build and actually function. James -- 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