Commit 53178d71 authored by Paul Mundt's avatar Paul Mundt

sh: Fix up SH7786 PCIe PHY initialization.

This brings the clocking and register setting in line with the somewhat
factually ambiguous specification.
Signed-off-by: default avatarPaul Mundt <lethal@linux-sh.org>
parent 7656e248
...@@ -148,16 +148,11 @@ static int pci_wait_for_irq(struct pci_channel *chan, unsigned int mask) ...@@ -148,16 +148,11 @@ static int pci_wait_for_irq(struct pci_channel *chan, unsigned int mask)
static void phy_write_reg(struct pci_channel *chan, unsigned int addr, static void phy_write_reg(struct pci_channel *chan, unsigned int addr,
unsigned int lane, unsigned int data) unsigned int lane, unsigned int data)
{ {
unsigned long phyaddr, ctrl; unsigned long phyaddr;
phyaddr = (1 << BITS_CMD) + ((lane & 0xf) << BITS_LANE) + phyaddr = (1 << BITS_CMD) + ((lane & 0xf) << BITS_LANE) +
((addr & 0xff) << BITS_ADR); ((addr & 0xff) << BITS_ADR);
/* Enable clock */
ctrl = pci_read_reg(chan, SH4A_PCIEPHYCTLR);
ctrl |= (1 << BITS_CKE);
pci_write_reg(chan, ctrl, SH4A_PCIEPHYCTLR);
/* Set write data */ /* Set write data */
pci_write_reg(chan, data, SH4A_PCIEPHYDOUTR); pci_write_reg(chan, data, SH4A_PCIEPHYDOUTR);
pci_write_reg(chan, phyaddr, SH4A_PCIEPHYADRR); pci_write_reg(chan, phyaddr, SH4A_PCIEPHYADRR);
...@@ -165,20 +160,22 @@ static void phy_write_reg(struct pci_channel *chan, unsigned int addr, ...@@ -165,20 +160,22 @@ static void phy_write_reg(struct pci_channel *chan, unsigned int addr,
phy_wait_for_ack(chan); phy_wait_for_ack(chan);
/* Clear command */ /* Clear command */
pci_write_reg(chan, 0, SH4A_PCIEPHYDOUTR);
pci_write_reg(chan, 0, SH4A_PCIEPHYADRR); pci_write_reg(chan, 0, SH4A_PCIEPHYADRR);
phy_wait_for_ack(chan); phy_wait_for_ack(chan);
/* Disable clock */
ctrl = pci_read_reg(chan, SH4A_PCIEPHYCTLR);
ctrl &= ~(1 << BITS_CKE);
pci_write_reg(chan, ctrl, SH4A_PCIEPHYCTLR);
} }
static int phy_init(struct pci_channel *chan) static int phy_init(struct pci_channel *chan)
{ {
unsigned long ctrl;
unsigned int timeout = 100; unsigned int timeout = 100;
/* Enable clock */
ctrl = pci_read_reg(chan, SH4A_PCIEPHYCTLR);
ctrl |= (1 << BITS_CKE);
pci_write_reg(chan, ctrl, SH4A_PCIEPHYCTLR);
/* Initialize the phy */ /* Initialize the phy */
phy_write_reg(chan, 0x60, 0xf, 0x004b008b); phy_write_reg(chan, 0x60, 0xf, 0x004b008b);
phy_write_reg(chan, 0x61, 0xf, 0x00007b41); phy_write_reg(chan, 0x61, 0xf, 0x00007b41);
...@@ -187,9 +184,15 @@ static int phy_init(struct pci_channel *chan) ...@@ -187,9 +184,15 @@ static int phy_init(struct pci_channel *chan)
phy_write_reg(chan, 0x66, 0xf, 0x00000010); phy_write_reg(chan, 0x66, 0xf, 0x00000010);
phy_write_reg(chan, 0x74, 0xf, 0x0007001c); phy_write_reg(chan, 0x74, 0xf, 0x0007001c);
phy_write_reg(chan, 0x79, 0xf, 0x01fc000d); phy_write_reg(chan, 0x79, 0xf, 0x01fc000d);
phy_write_reg(chan, 0xb0, 0xf, 0x00000610);
/* Deassert Standby */ /* Deassert Standby */
phy_write_reg(chan, 0x67, 0xf, 0x00000400); phy_write_reg(chan, 0x67, 0x1, 0x00000400);
/* Disable clock */
ctrl = pci_read_reg(chan, SH4A_PCIEPHYCTLR);
ctrl &= ~(1 << BITS_CKE);
pci_write_reg(chan, ctrl, SH4A_PCIEPHYCTLR);
while (timeout--) { while (timeout--) {
if (pci_read_reg(chan, SH4A_PCIEPHYSR)) if (pci_read_reg(chan, SH4A_PCIEPHYSR))
...@@ -287,6 +290,9 @@ static int pcie_init(struct sh7786_pcie_port *port) ...@@ -287,6 +290,9 @@ static int pcie_init(struct sh7786_pcie_port *port)
__raw_writel(memphys, chan->reg_base + SH4A_PCIELAR0); __raw_writel(memphys, chan->reg_base + SH4A_PCIELAR0);
__raw_writel((memsize - SZ_256) | 1, chan->reg_base + SH4A_PCIELAMR0); __raw_writel((memsize - SZ_256) | 1, chan->reg_base + SH4A_PCIELAMR0);
__raw_writel(memphys, chan->reg_base + SH4A_PCIEPCICONF4);
__raw_writel(0, chan->reg_base + SH4A_PCIEPCICONF5);
/* Finish initialization */ /* Finish initialization */
data = pci_read_reg(chan, SH4A_PCIETCTLR); data = pci_read_reg(chan, SH4A_PCIETCTLR);
data |= 0x1; data |= 0x1;
......
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