@@ -714,44 +714,30 @@ static int sata_fsl_port_start(struct ata_port *ap)
/*
* Now, we can bring the controller on-line & also initiate
* the COMINIT sequence, we simply return here and the boot-probing
* & device discovery process is re-initiated by libATA using a
* Softreset EH (dummy) session. Hence, boot probing and device
* discovey will be part of sata_fsl_softreset() callback.
*/
temp = ioread32(hcr_base + HCONTROL);
iowrite32((temp | HCONTROL_ONLINE_PHY_RST), hcr_base + HCONTROL);
VPRINTK("HStatus = 0x%x\n", ioread32(hcr_base + HSTATUS));
VPRINTK("HControl = 0x%x\n", ioread32(hcr_base + HCONTROL));
VPRINTK("CHBA = 0x%x\n", ioread32(hcr_base + CHBA));
-#ifdef CONFIG_MPC8315_DS
- /*
- * Workaround for 8315DS board 3gbps link-up issue,
- * currently limit SATA port to GEN1 speed
- */
- sata_fsl_scr_read(&ap->link, SCR_CONTROL, &temp);
- temp &= ~(0xF << 4);
- temp |= (0x1 << 4);
- sata_fsl_scr_write(&ap->link, SCR_CONTROL, temp);
-
- sata_fsl_scr_read(&ap->link, SCR_CONTROL, &temp);
- dev_warn(dev, "scr_control, speed limited to %x\n", temp);
-#endif
-
return 0;
}
static void sata_fsl_port_stop(struct ata_port *ap)
{
struct device *dev = ap->host->dev;
struct sata_fsl_port_priv *pp = ap->private_data;
struct sata_fsl_host_priv *host_priv = ap->host->private_data;
void __iomem *hcr_base = host_priv->hcr_base;
u32 temp;
/*
* Force host controller to go off-line, aborting current operations
*/
temp = ioread32(hcr_base + HCONTROL);
@@ -1432,30 +1418,39 @@ static int sata_fsl_probe(struct platform_device *ofdev)
}
host_priv->irq = irq;
if (of_device_is_compatible(ofdev->dev.of_node, "fsl,pq-sata-v2"))
host_priv->data_snoop = DATA_SNOOP_ENABLE_V2;
else
host_priv->data_snoop = DATA_SNOOP_ENABLE_V1;
/* allocate host structure */
host = ata_host_alloc_pinfo(&ofdev->dev, ppi, SATA_FSL_MAX_PORTS);
if (!host) {
retval = -ENOMEM;
goto error_exit_with_cleanup;
}
+ /* set speed limit if requested by device tree */
+ if (!of_property_read_u32(ofdev->dev.of_node, "sata-spd-limit",
+ &temp)) {
+ int i;
+ for (i = 0; i < SATA_FSL_MAX_PORTS; ++i)
+ host->ports[i]->link.hw_sata_spd_limit = temp;
+ dev_warn(&ofdev->dev, "speed limit set to gen %u\n", temp);
+ }
+
/* host->iomap is not used currently */
host->private_data = host_priv;
/* initialize host controller */
sata_fsl_init_controller(host);
/*
* Now, register with libATA core, this will also initiate the
* device discovery process, invoking our port_start() handler &
* error_handler() to execute a dummy Softreset EH session
*/
ata_host_activate(host, irq, sata_fsl_interrupt, SATA_FSL_IRQ_FLAG,
&sata_fsl_sht);
_______________________________________________
Linuxppc-dev mailing list
Jeff -- Thanks for the quick reply -- sorry that it took me a few days to get back to you. [Also, apologies if anyone gets a dupe -- I'm working on a new mail configuration, and while I did test it, something went sideways the first time I tried to send this.] Jeff Garzik writes: > Regarding this patch: Search for "sata_spd_limit" and xxx_spd* functions Ok, I see them, but it's not entirely clear how I'm supposed to use them. I think that maybe I want to set hw_sata_spd_limit in each port's link in the ata_host structure, after ata_host_alloc_pinfo, but before ata_host_activate. Something like the patch at the end of this message? It seems to have correctly set the spd_ values, but the negotiated link speed is still too high: # cd /sys/devices/e0000000.immr/e0019000.sata # find * -name '*_spd*' -print | xargs grep . ata2/link2/ata_link/link2/sata_spd:3.0 Gbps ata2/link2/ata_link/link2/hw_sata_spd_limit:1.5 Gbps ata2/link2/ata_link/link2/sata_spd_limit:1.5 Gbps Or am I misinterpreting those values? Maybe I need to call ata_set_sata_spd as well. Can I do that before discovery, or should it be a part of the port_start callback? And if the latter, shouldn't it be handled within the ata core, instead of expecting each host driver to do that call? With my original patch, I see the correct (throttled-down) value for "sata_spd": # cd /sys/devices/e0000000.immr/e0019000.sata # find . -name '*_spd*' -print | xargs grep . ./ata2/link2/ata_link/link2/sata_spd:1.5 Gbps ./ata2/link2/ata_link/link2/hw_sata_spd_limit:1.5 Gbps ./ata2/link2/ata_link/link2/sata_spd_limit:1.5 Gbps Thanks for any pointers. Patch below. Thanks again, Anthony Foiani (Pardon the extra context, but it seemed the easiest way to show how this call slotted in between the alloc and the init.) Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev