summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorClemens Ladisch <clemens@ladisch.de>2010-04-04 15:19:52 +0200
committerStefan Richter <stefanr@s5r6.in-berlin.de>2010-04-10 16:51:14 +0200
commit4a96b4fcd6b35e9233df07b3c9ab38091edcfe7e (patch)
tree9a1e7500b64d98845607266b8f15b7a5384aa60f
parentca658b1e29d6be939207532e337fb640eb697f71 (diff)
downloadlinux-3.10-4a96b4fcd6b35e9233df07b3c9ab38091edcfe7e.tar.gz
linux-3.10-4a96b4fcd6b35e9233df07b3c9ab38091edcfe7e.tar.bz2
linux-3.10-4a96b4fcd6b35e9233df07b3c9ab38091edcfe7e.zip
firewire: ohci: add a function for reading PHY registers
Move the register reading code from ohci_update_phy_reg() into a function which can be used separately. Signed-off-by: Clemens Ladisch <clemens@ladisch.de> Signed-off-by: Stefan Richter <stefanr@s5r6.in-berlin.de>
-rw-r--r--drivers/firewire/ohci.c24
1 files changed, 19 insertions, 5 deletions
diff --git a/drivers/firewire/ohci.c b/drivers/firewire/ohci.c
index e33917bf97d..8ebccda94df 100644
--- a/drivers/firewire/ohci.c
+++ b/drivers/firewire/ohci.c
@@ -460,22 +460,36 @@ static inline void flush_writes(const struct fw_ohci *ohci)
reg_read(ohci, OHCI1394_Version);
}
-static int ohci_update_phy_reg(struct fw_card *card, int addr,
- int clear_bits, int set_bits)
+static int read_phy_reg(struct fw_card *card, int addr, u32 *value)
{
struct fw_ohci *ohci = fw_ohci(card);
- u32 val, old;
+ u32 val;
reg_write(ohci, OHCI1394_PhyControl, OHCI1394_PhyControl_Read(addr));
flush_writes(ohci);
msleep(2);
val = reg_read(ohci, OHCI1394_PhyControl);
if ((val & OHCI1394_PhyControl_ReadDone) == 0) {
- fw_error("failed to set phy reg bits.\n");
+ fw_error("failed to read phy reg bits\n");
return -EBUSY;
}
- old = OHCI1394_PhyControl_ReadData(val);
+ *value = OHCI1394_PhyControl_ReadData(val);
+
+ return 0;
+}
+
+static int ohci_update_phy_reg(struct fw_card *card, int addr,
+ int clear_bits, int set_bits)
+{
+ struct fw_ohci *ohci = fw_ohci(card);
+ u32 old;
+ int err;
+
+ err = read_phy_reg(card, addr, &old);
+ if (err < 0)
+ return err;
+
old = (old & ~clear_bits) | set_bits;
reg_write(ohci, OHCI1394_PhyControl,
OHCI1394_PhyControl_Write(addr, old));