Hi, On 11/06/2018 03:42 AM, Chaohong guo wrote: > Interrupt vector 0 is valid on some platform. falcon driver wrongly > treated interrupt vector 0 as invalid. We fix it in libata, and fix it > in this driver as well. > > Signed-off-by: Chaohong guo <chaohong.guo@xxxxxxxxx> This patch should be folded into patch #1 (otherwise it breaks bisectability). There are also other libata drivers using polling mode that need updating: pata_palmld.c and pata_platform.c. > --- > drivers/ata/pata_falcon.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/ata/pata_falcon.c b/drivers/ata/pata_falcon.c > index 41e0d6a6cd05..a991053aee87 100644 > --- a/drivers/ata/pata_falcon.c > +++ b/drivers/ata/pata_falcon.c > @@ -171,7 +171,7 @@ static int pata_falcon_init_one(void) > (unsigned long)base + ATA_HD_CONTROL); > > /* activate */ > - return ata_host_activate(host, 0, NULL, 0, &pata_falcon_sht); > + return ata_host_activate(host, -1, NULL, 0, &pata_falcon_sht); > } > > module_init(pata_falcon_init_one); Best regards, -- Bartlomiej Zolnierkiewicz Samsung R&D Institute Poland Samsung Electronics
