Commit 1d513358 authored by Anton Vorontsov's avatar Anton Vorontsov Committed by Jeff Garzik

ahci: Get rid of pci_dev argument in ahci_save_initial_config()

To make the function generic we have to get rid of "struct pci_dev *",
so let's pass just a "struct devce *".
Signed-off-by: default avatarAnton Vorontsov <avorontsov@ru.mvista.com>
Signed-off-by: default avatarJeff Garzik <jgarzik@redhat.com>
parent 394d6e53
...@@ -840,7 +840,7 @@ static ssize_t ahci_show_port_cmd(struct device *dev, ...@@ -840,7 +840,7 @@ static ssize_t ahci_show_port_cmd(struct device *dev,
/** /**
* ahci_save_initial_config - Save and fixup initial config values * ahci_save_initial_config - Save and fixup initial config values
* @pdev: target PCI device * @dev: target AHCI device
* @hpriv: host private area to store config values * @hpriv: host private area to store config values
* @force_port_map: force port map to a specified value * @force_port_map: force port map to a specified value
* @mask_port_map: mask out particular bits from port map * @mask_port_map: mask out particular bits from port map
...@@ -855,7 +855,7 @@ static ssize_t ahci_show_port_cmd(struct device *dev, ...@@ -855,7 +855,7 @@ static ssize_t ahci_show_port_cmd(struct device *dev,
* LOCKING: * LOCKING:
* None. * None.
*/ */
static void ahci_save_initial_config(struct pci_dev *pdev, static void ahci_save_initial_config(struct device *dev,
struct ahci_host_priv *hpriv, struct ahci_host_priv *hpriv,
unsigned int force_port_map, unsigned int force_port_map,
unsigned int mask_port_map) unsigned int mask_port_map)
...@@ -883,45 +883,43 @@ static void ahci_save_initial_config(struct pci_dev *pdev, ...@@ -883,45 +883,43 @@ static void ahci_save_initial_config(struct pci_dev *pdev,
/* some chips have errata preventing 64bit use */ /* some chips have errata preventing 64bit use */
if ((cap & HOST_CAP_64) && (hpriv->flags & AHCI_HFLAG_32BIT_ONLY)) { if ((cap & HOST_CAP_64) && (hpriv->flags & AHCI_HFLAG_32BIT_ONLY)) {
dev_printk(KERN_INFO, &pdev->dev, dev_printk(KERN_INFO, dev,
"controller can't do 64bit DMA, forcing 32bit\n"); "controller can't do 64bit DMA, forcing 32bit\n");
cap &= ~HOST_CAP_64; cap &= ~HOST_CAP_64;
} }
if ((cap & HOST_CAP_NCQ) && (hpriv->flags & AHCI_HFLAG_NO_NCQ)) { if ((cap & HOST_CAP_NCQ) && (hpriv->flags & AHCI_HFLAG_NO_NCQ)) {
dev_printk(KERN_INFO, &pdev->dev, dev_printk(KERN_INFO, dev,
"controller can't do NCQ, turning off CAP_NCQ\n"); "controller can't do NCQ, turning off CAP_NCQ\n");
cap &= ~HOST_CAP_NCQ; cap &= ~HOST_CAP_NCQ;
} }
if (!(cap & HOST_CAP_NCQ) && (hpriv->flags & AHCI_HFLAG_YES_NCQ)) { if (!(cap & HOST_CAP_NCQ) && (hpriv->flags & AHCI_HFLAG_YES_NCQ)) {
dev_printk(KERN_INFO, &pdev->dev, dev_printk(KERN_INFO, dev,
"controller can do NCQ, turning on CAP_NCQ\n"); "controller can do NCQ, turning on CAP_NCQ\n");
cap |= HOST_CAP_NCQ; cap |= HOST_CAP_NCQ;
} }
if ((cap & HOST_CAP_PMP) && (hpriv->flags & AHCI_HFLAG_NO_PMP)) { if ((cap & HOST_CAP_PMP) && (hpriv->flags & AHCI_HFLAG_NO_PMP)) {
dev_printk(KERN_INFO, &pdev->dev, dev_printk(KERN_INFO, dev,
"controller can't do PMP, turning off CAP_PMP\n"); "controller can't do PMP, turning off CAP_PMP\n");
cap &= ~HOST_CAP_PMP; cap &= ~HOST_CAP_PMP;
} }
if ((cap & HOST_CAP_SNTF) && (hpriv->flags & AHCI_HFLAG_NO_SNTF)) { if ((cap & HOST_CAP_SNTF) && (hpriv->flags & AHCI_HFLAG_NO_SNTF)) {
dev_printk(KERN_INFO, &pdev->dev, dev_printk(KERN_INFO, dev,
"controller can't do SNTF, turning off CAP_SNTF\n"); "controller can't do SNTF, turning off CAP_SNTF\n");
cap &= ~HOST_CAP_SNTF; cap &= ~HOST_CAP_SNTF;
} }
if (force_port_map && port_map != force_port_map) { if (force_port_map && port_map != force_port_map) {
dev_printk(KERN_INFO, &pdev->dev, dev_printk(KERN_INFO, dev, "forcing port_map 0x%x -> 0x%x\n",
"forcing port_map 0x%x -> 0x%x\n",
port_map, force_port_map); port_map, force_port_map);
port_map = force_port_map; port_map = force_port_map;
} }
if (mask_port_map) { if (mask_port_map) {
dev_printk(KERN_ERR, &pdev->dev, dev_printk(KERN_ERR, dev, "masking port_map 0x%x -> 0x%x\n",
"masking port_map 0x%x -> 0x%x\n",
port_map, port_map,
port_map & mask_port_map); port_map & mask_port_map);
port_map &= mask_port_map; port_map &= mask_port_map;
...@@ -939,7 +937,7 @@ static void ahci_save_initial_config(struct pci_dev *pdev, ...@@ -939,7 +937,7 @@ static void ahci_save_initial_config(struct pci_dev *pdev,
* port_map and let it be generated from n_ports. * port_map and let it be generated from n_ports.
*/ */
if (map_ports > ahci_nr_ports(cap)) { if (map_ports > ahci_nr_ports(cap)) {
dev_printk(KERN_WARNING, &pdev->dev, dev_printk(KERN_WARNING, dev,
"implemented port map (0x%x) contains more " "implemented port map (0x%x) contains more "
"ports than nr_ports (%u), using nr_ports\n", "ports than nr_ports (%u), using nr_ports\n",
port_map, ahci_nr_ports(cap)); port_map, ahci_nr_ports(cap));
...@@ -950,7 +948,7 @@ static void ahci_save_initial_config(struct pci_dev *pdev, ...@@ -950,7 +948,7 @@ static void ahci_save_initial_config(struct pci_dev *pdev,
/* fabricate port_map from cap.nr_ports */ /* fabricate port_map from cap.nr_ports */
if (!port_map) { if (!port_map) {
port_map = (1 << ahci_nr_ports(cap)) - 1; port_map = (1 << ahci_nr_ports(cap)) - 1;
dev_printk(KERN_WARNING, &pdev->dev, dev_printk(KERN_WARNING, dev,
"forcing PORTS_IMPL to 0x%x\n", port_map); "forcing PORTS_IMPL to 0x%x\n", port_map);
/* write the fixed up value to the PI register */ /* write the fixed up value to the PI register */
...@@ -988,7 +986,8 @@ static void ahci_pci_save_initial_config(struct pci_dev *pdev, ...@@ -988,7 +986,8 @@ static void ahci_pci_save_initial_config(struct pci_dev *pdev,
"Disabling your PATA port. Use the boot option 'ahci.marvell_enable=0' to avoid this.\n"); "Disabling your PATA port. Use the boot option 'ahci.marvell_enable=0' to avoid this.\n");
} }
ahci_save_initial_config(pdev, hpriv, force_port_map, mask_port_map); ahci_save_initial_config(&pdev->dev, hpriv, force_port_map,
mask_port_map);
} }
/** /**
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment