Merge branch 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/ieee1394/linux1394-2.6

* 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/ieee1394/linux1394-2.6: (23 commits)
  firewire: ohci: extend initialization log message
  firewire: ohci: fix IR/IT context mask mixup
  firewire: ohci: add module parameter to activate quirk fixes
  firewire: ohci: use an ID table for quirks detection
  firewire: ohci: reorder struct fw_ohci for better cache efficiency
  firewire: ohci: remove unused dualbuffer IR code
  firewire: core: combine a bit of repeated code
  firewire: core: change type of a data buffer
  firewire: cdev: increment ABI version number
  firewire: cdev: add more flexible cycle timer ioctl
  firewire: core: rename an internal function
  firewire: core: fix an information leak
  firewire: core: increase stack size of config ROM reader
  firewire: core: don't fail device creation in case of too large config ROM blocks
  firewire: core: fix "giving up on config rom" with Panasonic AG-DV2500
  firewire: remove incomplete Bus_Time CSR support
  firewire: get_cycle_timer optimization and cleanup
  firewire: ohci: enable cycle timer fix on ALi and NEC controllers
  firewire: ohci: work around cycle timer bugs on VIA controllers
  firewire: make PCI device id constant
  ...

1 files changed, 6 insertions, 11 deletions

diff --git a/drivers/firewire/core-transaction.c b/drivers/firewire/core-transaction.c
index 495849eb13cc..673b03f8b4ec 100644
--- a/drivers/firewire/core-transaction.c
+++ b/drivers/firewire/core-transaction.c
@@ -921,23 +921,15 @@ static void handle_registers(struct fw_card *card, struct fw_request *request,
                 void *payload, size_t length, void *callback_data)
 {
         int reg = offset & ~CSR_REGISTER_BASE;
-        unsigned long long bus_time;
         __be32 *data = payload;
         int rcode = RCODE_COMPLETE;
 
         switch (reg) {
         case CSR_CYCLE_TIME:
-        case CSR_BUS_TIME:
-                if (!TCODE_IS_READ_REQUEST(tcode) || length != 4) {
-                        rcode = RCODE_TYPE_ERROR;
-                        break;
-                }
-
-                bus_time = card->driver->get_bus_time(card);
-                if (reg == CSR_CYCLE_TIME)
-                        *data = cpu_to_be32(bus_time);
+                if (TCODE_IS_READ_REQUEST(tcode) && length == 4)
+                        *data = cpu_to_be32(card->driver->get_cycle_time(card));
                 else
-                        *data = cpu_to_be32(bus_time >> 25);
+                        rcode = RCODE_TYPE_ERROR;
                 break;
 
         case CSR_BROADCAST_CHANNEL:
@@ -968,6 +960,9 @@ static void handle_registers(struct fw_card *card, struct fw_request *request,
         case CSR_BUSY_TIMEOUT:
                 /* FIXME: Implement this. */
 
+        case CSR_BUS_TIME:
+                /* Useless without initialization by the bus manager. */
+
         default:
                 rcode = RCODE_ADDRESS_ERROR;
                 break;