libata: Change prototype of mode_filter to remove ata_port*
With Tejun having added adev->ap some time ago we can get rid of the almost unused port being passed to mode filters. And while we are doing filters, lets turn on the !IORDY filter as well. Signed-off-by: Alan Cox <alan@redhat.com> With some hand massaging from Signed-off-by: Jeff Garzik <jeff@garzik.org>
This commit is contained in:
parent
9d2c7c75f8
commit
a76b62ca70
|
@ -3545,7 +3545,7 @@ static void ata_dev_xfermask(struct ata_device *dev)
|
||||||
}
|
}
|
||||||
|
|
||||||
if (ap->ops->mode_filter)
|
if (ap->ops->mode_filter)
|
||||||
xfer_mask = ap->ops->mode_filter(ap, dev, xfer_mask);
|
xfer_mask = ap->ops->mode_filter(dev, xfer_mask);
|
||||||
|
|
||||||
/* Apply cable rule here. Don't apply it early because when
|
/* Apply cable rule here. Don't apply it early because when
|
||||||
* we handle hot plug the cable type can itself change.
|
* we handle hot plug the cable type can itself change.
|
||||||
|
|
|
@ -893,12 +893,12 @@ int ata_pci_clear_simplex(struct pci_dev *pdev)
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
unsigned long ata_pci_default_filter(const struct ata_port *ap, struct ata_device *adev, unsigned long xfer_mask)
|
unsigned long ata_pci_default_filter(struct ata_device *adev, unsigned long xfer_mask)
|
||||||
{
|
{
|
||||||
/* Filter out DMA modes if the device has been configured by
|
/* Filter out DMA modes if the device has been configured by
|
||||||
the BIOS as PIO only */
|
the BIOS as PIO only */
|
||||||
|
|
||||||
if (ap->ioaddr.bmdma_addr == 0)
|
if (adev->ap->ioaddr.bmdma_addr == 0)
|
||||||
xfer_mask &= ~(ATA_MASK_MWDMA | ATA_MASK_UDMA);
|
xfer_mask &= ~(ATA_MASK_MWDMA | ATA_MASK_UDMA);
|
||||||
return xfer_mask;
|
return xfer_mask;
|
||||||
}
|
}
|
||||||
|
|
|
@ -151,7 +151,7 @@ static void ali_early_error_handler(struct ata_port *ap)
|
||||||
* fix that later on. Also ensure we do not do UDMA on WDC drives
|
* fix that later on. Also ensure we do not do UDMA on WDC drives
|
||||||
*/
|
*/
|
||||||
|
|
||||||
static unsigned long ali_20_filter(const struct ata_port *ap, struct ata_device *adev, unsigned long mask)
|
static unsigned long ali_20_filter(struct ata_device *adev, unsigned long mask)
|
||||||
{
|
{
|
||||||
char model_num[ATA_ID_PROD_LEN + 1];
|
char model_num[ATA_ID_PROD_LEN + 1];
|
||||||
/* No DMA on anything but a disk for now */
|
/* No DMA on anything but a disk for now */
|
||||||
|
@ -160,7 +160,7 @@ static unsigned long ali_20_filter(const struct ata_port *ap, struct ata_device
|
||||||
ata_id_c_string(adev->id, model_num, ATA_ID_PROD, sizeof(model_num));
|
ata_id_c_string(adev->id, model_num, ATA_ID_PROD, sizeof(model_num));
|
||||||
if (strstr(model_num, "WDC"))
|
if (strstr(model_num, "WDC"))
|
||||||
return mask &= ~ATA_MASK_UDMA;
|
return mask &= ~ATA_MASK_UDMA;
|
||||||
return ata_pci_default_filter(ap, adev, mask);
|
return ata_pci_default_filter(adev, mask);
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
|
|
@ -175,7 +175,7 @@ static int hpt_dma_blacklisted(const struct ata_device *dev, char *modestr, cons
|
||||||
* Block UDMA on devices that cause trouble with this controller.
|
* Block UDMA on devices that cause trouble with this controller.
|
||||||
*/
|
*/
|
||||||
|
|
||||||
static unsigned long hpt366_filter(const struct ata_port *ap, struct ata_device *adev, unsigned long mask)
|
static unsigned long hpt366_filter(struct ata_device *adev, unsigned long mask)
|
||||||
{
|
{
|
||||||
if (adev->class == ATA_DEV_ATA) {
|
if (adev->class == ATA_DEV_ATA) {
|
||||||
if (hpt_dma_blacklisted(adev, "UDMA", bad_ata33))
|
if (hpt_dma_blacklisted(adev, "UDMA", bad_ata33))
|
||||||
|
@ -185,7 +185,7 @@ static unsigned long hpt366_filter(const struct ata_port *ap, struct ata_device
|
||||||
if (hpt_dma_blacklisted(adev, "UDMA4", bad_ata66_4))
|
if (hpt_dma_blacklisted(adev, "UDMA4", bad_ata66_4))
|
||||||
mask &= ~(0x0F << ATA_SHIFT_UDMA);
|
mask &= ~(0x0F << ATA_SHIFT_UDMA);
|
||||||
}
|
}
|
||||||
return ata_pci_default_filter(ap, adev, mask);
|
return ata_pci_default_filter(adev, mask);
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
|
|
@ -272,13 +272,12 @@ static const char *bad_ata100_5[] = {
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* hpt370_filter - mode selection filter
|
* hpt370_filter - mode selection filter
|
||||||
* @ap: ATA interface
|
|
||||||
* @adev: ATA device
|
* @adev: ATA device
|
||||||
*
|
*
|
||||||
* Block UDMA on devices that cause trouble with this controller.
|
* Block UDMA on devices that cause trouble with this controller.
|
||||||
*/
|
*/
|
||||||
|
|
||||||
static unsigned long hpt370_filter(const struct ata_port *ap, struct ata_device *adev, unsigned long mask)
|
static unsigned long hpt370_filter(struct ata_device *adev, unsigned long mask)
|
||||||
{
|
{
|
||||||
if (adev->class == ATA_DEV_ATA) {
|
if (adev->class == ATA_DEV_ATA) {
|
||||||
if (hpt_dma_blacklisted(adev, "UDMA", bad_ata33))
|
if (hpt_dma_blacklisted(adev, "UDMA", bad_ata33))
|
||||||
|
@ -286,24 +285,23 @@ static unsigned long hpt370_filter(const struct ata_port *ap, struct ata_device
|
||||||
if (hpt_dma_blacklisted(adev, "UDMA100", bad_ata100_5))
|
if (hpt_dma_blacklisted(adev, "UDMA100", bad_ata100_5))
|
||||||
mask &= ~(0x1F << ATA_SHIFT_UDMA);
|
mask &= ~(0x1F << ATA_SHIFT_UDMA);
|
||||||
}
|
}
|
||||||
return ata_pci_default_filter(ap, adev, mask);
|
return ata_pci_default_filter(adev, mask);
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* hpt370a_filter - mode selection filter
|
* hpt370a_filter - mode selection filter
|
||||||
* @ap: ATA interface
|
|
||||||
* @adev: ATA device
|
* @adev: ATA device
|
||||||
*
|
*
|
||||||
* Block UDMA on devices that cause trouble with this controller.
|
* Block UDMA on devices that cause trouble with this controller.
|
||||||
*/
|
*/
|
||||||
|
|
||||||
static unsigned long hpt370a_filter(const struct ata_port *ap, struct ata_device *adev, unsigned long mask)
|
static unsigned long hpt370a_filter(struct ata_device *adev, unsigned long mask)
|
||||||
{
|
{
|
||||||
if (adev->class != ATA_DEV_ATA) {
|
if (adev->class != ATA_DEV_ATA) {
|
||||||
if (hpt_dma_blacklisted(adev, "UDMA100", bad_ata100_5))
|
if (hpt_dma_blacklisted(adev, "UDMA100", bad_ata100_5))
|
||||||
mask &= ~ (0x1F << ATA_SHIFT_UDMA);
|
mask &= ~ (0x1F << ATA_SHIFT_UDMA);
|
||||||
}
|
}
|
||||||
return ata_pci_default_filter(ap, adev, mask);
|
return ata_pci_default_filter(adev, mask);
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
|
|
@ -185,31 +185,31 @@ static u8 serverworks_is_csb(struct pci_dev *pdev)
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* serverworks_osb4_filter - mode selection filter
|
* serverworks_osb4_filter - mode selection filter
|
||||||
* @ap: ATA interface
|
|
||||||
* @adev: ATA device
|
* @adev: ATA device
|
||||||
|
* @mask: Mask of proposed modes
|
||||||
*
|
*
|
||||||
* Filter the offered modes for the device to apply controller
|
* Filter the offered modes for the device to apply controller
|
||||||
* specific rules. OSB4 requires no UDMA for disks due to a FIFO
|
* specific rules. OSB4 requires no UDMA for disks due to a FIFO
|
||||||
* bug we hit.
|
* bug we hit.
|
||||||
*/
|
*/
|
||||||
|
|
||||||
static unsigned long serverworks_osb4_filter(const struct ata_port *ap, struct ata_device *adev, unsigned long mask)
|
static unsigned long serverworks_osb4_filter(struct ata_device *adev, unsigned long mask)
|
||||||
{
|
{
|
||||||
if (adev->class == ATA_DEV_ATA)
|
if (adev->class == ATA_DEV_ATA)
|
||||||
mask &= ~ATA_MASK_UDMA;
|
mask &= ~ATA_MASK_UDMA;
|
||||||
return ata_pci_default_filter(ap, adev, mask);
|
return ata_pci_default_filter(adev, mask);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* serverworks_csb_filter - mode selection filter
|
* serverworks_csb_filter - mode selection filter
|
||||||
* @ap: ATA interface
|
|
||||||
* @adev: ATA device
|
* @adev: ATA device
|
||||||
|
* @mask: Mask of proposed modes
|
||||||
*
|
*
|
||||||
* Check the blacklist and disable UDMA5 if matched
|
* Check the blacklist and disable UDMA5 if matched
|
||||||
*/
|
*/
|
||||||
|
|
||||||
static unsigned long serverworks_csb_filter(const struct ata_port *ap, struct ata_device *adev, unsigned long mask)
|
static unsigned long serverworks_csb_filter(struct ata_device *adev, unsigned long mask)
|
||||||
{
|
{
|
||||||
const char *p;
|
const char *p;
|
||||||
char model_num[ATA_ID_PROD_LEN + 1];
|
char model_num[ATA_ID_PROD_LEN + 1];
|
||||||
|
@ -217,7 +217,7 @@ static unsigned long serverworks_csb_filter(const struct ata_port *ap, struct at
|
||||||
|
|
||||||
/* Disk, UDMA */
|
/* Disk, UDMA */
|
||||||
if (adev->class != ATA_DEV_ATA)
|
if (adev->class != ATA_DEV_ATA)
|
||||||
return ata_pci_default_filter(ap, adev, mask);
|
return ata_pci_default_filter(adev, mask);
|
||||||
|
|
||||||
/* Actually do need to check */
|
/* Actually do need to check */
|
||||||
ata_id_c_string(adev->id, model_num, ATA_ID_PROD, sizeof(model_num));
|
ata_id_c_string(adev->id, model_num, ATA_ID_PROD, sizeof(model_num));
|
||||||
|
@ -226,7 +226,7 @@ static unsigned long serverworks_csb_filter(const struct ata_port *ap, struct at
|
||||||
if (!strcmp(p, model_num))
|
if (!strcmp(p, model_num))
|
||||||
mask &= ~(0x1F << ATA_SHIFT_UDMA);
|
mask &= ~(0x1F << ATA_SHIFT_UDMA);
|
||||||
}
|
}
|
||||||
return ata_pci_default_filter(ap, adev, mask);
|
return ata_pci_default_filter(adev, mask);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
|
|
|
@ -601,7 +601,7 @@ struct ata_port_operations {
|
||||||
|
|
||||||
void (*set_piomode) (struct ata_port *, struct ata_device *);
|
void (*set_piomode) (struct ata_port *, struct ata_device *);
|
||||||
void (*set_dmamode) (struct ata_port *, struct ata_device *);
|
void (*set_dmamode) (struct ata_port *, struct ata_device *);
|
||||||
unsigned long (*mode_filter) (const struct ata_port *, struct ata_device *, unsigned long);
|
unsigned long (*mode_filter) (struct ata_device *, unsigned long);
|
||||||
|
|
||||||
void (*tf_load) (struct ata_port *ap, const struct ata_taskfile *tf);
|
void (*tf_load) (struct ata_port *ap, const struct ata_taskfile *tf);
|
||||||
void (*tf_read) (struct ata_port *ap, struct ata_taskfile *tf);
|
void (*tf_read) (struct ata_port *ap, struct ata_taskfile *tf);
|
||||||
|
@ -882,7 +882,7 @@ struct pci_bits {
|
||||||
extern struct ata_probe_ent *
|
extern struct ata_probe_ent *
|
||||||
ata_pci_init_native_mode(struct pci_dev *pdev, struct ata_port_info **port, int portmask);
|
ata_pci_init_native_mode(struct pci_dev *pdev, struct ata_port_info **port, int portmask);
|
||||||
extern int pci_test_config_bits(struct pci_dev *pdev, const struct pci_bits *bits);
|
extern int pci_test_config_bits(struct pci_dev *pdev, const struct pci_bits *bits);
|
||||||
extern unsigned long ata_pci_default_filter(const struct ata_port *, struct ata_device *, unsigned long);
|
extern unsigned long ata_pci_default_filter(struct ata_device *, unsigned long);
|
||||||
#endif /* CONFIG_PCI */
|
#endif /* CONFIG_PCI */
|
||||||
|
|
||||||
/*
|
/*
|
||||||
|
|
Loading…
Reference in a new issue