Commit 153e3979 authored by Clemens Ladisch's avatar Clemens Ladisch

firewire: ohci: speed up PHY register accesses

Most PHY chips, when idle, can complete a register access in the time
needed for two or three PCI read transactions; bigger delays occur only
when data is currently being moved over the link/PHY interface.  So if
we busy-wait a few times when waiting for the register access to finish,
it is likely that we can finish without having to sleep.
Signed-off-by: default avatarClemens Ladisch <clemens@ladisch.de>
parent f9c70f91
...@@ -474,12 +474,17 @@ static int read_phy_reg(struct fw_ohci *ohci, int addr) ...@@ -474,12 +474,17 @@ static int read_phy_reg(struct fw_ohci *ohci, int addr)
int i; int i;
reg_write(ohci, OHCI1394_PhyControl, OHCI1394_PhyControl_Read(addr)); reg_write(ohci, OHCI1394_PhyControl, OHCI1394_PhyControl_Read(addr));
for (i = 0; i < 10; i++) { for (i = 0; i < 3 + 100; i++) {
val = reg_read(ohci, OHCI1394_PhyControl); val = reg_read(ohci, OHCI1394_PhyControl);
if (val & OHCI1394_PhyControl_ReadDone) if (val & OHCI1394_PhyControl_ReadDone)
return OHCI1394_PhyControl_ReadData(val); return OHCI1394_PhyControl_ReadData(val);
msleep(1); /*
* Try a few times without waiting. Sleeping is necessary
* only when the link/PHY interface is busy.
*/
if (i >= 3)
msleep(1);
} }
fw_error("failed to read phy reg\n"); fw_error("failed to read phy reg\n");
...@@ -492,12 +497,13 @@ static int write_phy_reg(const struct fw_ohci *ohci, int addr, u32 val) ...@@ -492,12 +497,13 @@ static int write_phy_reg(const struct fw_ohci *ohci, int addr, u32 val)
reg_write(ohci, OHCI1394_PhyControl, reg_write(ohci, OHCI1394_PhyControl,
OHCI1394_PhyControl_Write(addr, val)); OHCI1394_PhyControl_Write(addr, val));
for (i = 0; i < 100; i++) { for (i = 0; i < 3 + 100; i++) {
val = reg_read(ohci, OHCI1394_PhyControl); val = reg_read(ohci, OHCI1394_PhyControl);
if (!(val & OHCI1394_PhyControl_WritePending)) if (!(val & OHCI1394_PhyControl_WritePending))
return 0; return 0;
msleep(1); if (i >= 3)
msleep(1);
} }
fw_error("failed to write phy reg\n"); fw_error("failed to write phy reg\n");
......
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