summaryrefslogtreecommitdiff
path: root/drivers/firewire/core-transaction.c
diff options
context:
space:
mode:
authorStefan Richter <stefanr@s5r6.in-berlin.de>2010-06-12 22:34:50 +0400
committerStefan Richter <stefanr@s5r6.in-berlin.de>2010-06-19 15:01:41 +0400
commitc8a94ded57e9cc2498d401b2f5c856213a3e19fb (patch)
tree8fd6a196ff953270c03700dd682108baf3ee369b /drivers/firewire/core-transaction.c
parentdb3c9cc105ee844f6cd7a1beb9926fb8e9a093ae (diff)
downloadlinux-c8a94ded57e9cc2498d401b2f5c856213a3e19fb.tar.xz
firewire: normalize STATE_CLEAR/SET CSR access interface
Push the maintenance of STATE_CLEAR/SET.abdicate down into the card driver. This way, the read/write_csr_reg driver method works uniformly across all CSR offsets. Signed-off-by: Stefan Richter <stefanr@s5r6.in-berlin.de>
Diffstat (limited to 'drivers/firewire/core-transaction.c')
-rw-r--r--drivers/firewire/core-transaction.c41
1 files changed, 12 insertions, 29 deletions
diff --git a/drivers/firewire/core-transaction.c b/drivers/firewire/core-transaction.c
index 62bf30560a3e..87d69cddb231 100644
--- a/drivers/firewire/core-transaction.c
+++ b/drivers/firewire/core-transaction.c
@@ -982,20 +982,6 @@ static const struct fw_address_region registers_region =
{ .start = CSR_REGISTER_BASE,
.end = CSR_REGISTER_BASE | CSR_CONFIG_ROM, };
-static u32 read_state_register(struct fw_card *card)
-{
- u32 value;
-
- /* Bit 8 (cmstr): */
- value = card->driver->read_csr_reg(card, CSR_STATE_CLEAR);
-
- /* Bit 10 (abdicate): */
- if (card->csr_abdicate)
- value |= CSR_STATE_BIT_ABDICATE;
-
- return value;
-}
-
static void update_split_timeout(struct fw_card *card)
{
unsigned int cycles;
@@ -1021,29 +1007,25 @@ static void handle_registers(struct fw_card *card, struct fw_request *request,
switch (reg) {
case CSR_STATE_CLEAR:
- if (tcode == TCODE_READ_QUADLET_REQUEST) {
- *data = cpu_to_be32(read_state_register(card));
- } else if (tcode == TCODE_WRITE_QUADLET_REQUEST) {
+ if (tcode == TCODE_READ_QUADLET_REQUEST)
+ *data = cpu_to_be32(card->driver->
+ read_csr_reg(card, CSR_STATE_CLEAR));
+ else if (tcode == TCODE_WRITE_QUADLET_REQUEST)
card->driver->write_csr_reg(card, CSR_STATE_CLEAR,
be32_to_cpu(*data));
- if (*data & cpu_to_be32(CSR_STATE_BIT_ABDICATE))
- card->csr_abdicate = false;
- } else {
+ else
rcode = RCODE_TYPE_ERROR;
- }
break;
case CSR_STATE_SET:
- if (tcode == TCODE_READ_QUADLET_REQUEST) {
- *data = cpu_to_be32(read_state_register(card));
- } else if (tcode == TCODE_WRITE_QUADLET_REQUEST) {
+ if (tcode == TCODE_READ_QUADLET_REQUEST)
+ *data = cpu_to_be32(card->driver->
+ read_csr_reg(card, CSR_STATE_SET));
+ else if (tcode == TCODE_WRITE_QUADLET_REQUEST)
card->driver->write_csr_reg(card, CSR_STATE_SET,
be32_to_cpu(*data));
- if (*data & cpu_to_be32(CSR_STATE_BIT_ABDICATE))
- card->csr_abdicate = true;
- } else {
+ else
rcode = RCODE_TYPE_ERROR;
- }
break;
case CSR_NODE_IDS:
@@ -1063,7 +1045,8 @@ static void handle_registers(struct fw_card *card, struct fw_request *request,
case CSR_RESET_START:
if (tcode == TCODE_WRITE_QUADLET_REQUEST)
- card->csr_abdicate = false;
+ card->driver->write_csr_reg(card, CSR_STATE_CLEAR,
+ CSR_STATE_BIT_ABDICATE);
else
rcode = RCODE_TYPE_ERROR;
break;