firewire: remove incomplete Bus_Time CSR support
authorStefan Richter <stefanr@s5r6.in-berlin.de>
Sun, 14 Feb 2010 17:49:18 +0000 (18:49 +0100)
committerStefan Richter <stefanr@s5r6.in-berlin.de>
Sat, 20 Feb 2010 21:33:14 +0000 (22:33 +0100)
commit168cf9af699e87d5a6f44b684583714ecabb8e71
tree2f491d28a4a129caf7c2be306aec33c437a5cb30
parent4a9bde9b8ab55a2bb51b57cad215a97bcf80bae2
firewire: remove incomplete Bus_Time CSR support

The current implementation of Bus_Time read access was buggy since it
did not ensure that Bus_Time.second_count_hi and second_count_lo came
from the same 128 seconds period.

Reported-by: HÃ¥kan Johansson <f96hajo@chalmers.se>
Instead of a fix, remove Bus_Time register support altogether.  The spec
requires all cycle master capable nodes to implement this (all Linux
nodes are cycle master capable) while it also says that it "may" be
initialized by the bus manager or by the IRM standing in for a bus
manager.  (Neither Linux' firewire-core nor ieee1394 nodemgr implement
this.)

Since we cannot rely on Bus_Time having been initialized by a bus
manager, it is better to return an error instead of a nonsensical value
on a read request to Bus_Time.

Alternatively, we could fix the Bus_Time read integrity bug _and_
implement (a) cycle master's write support of the register as well as
(b) bus manager's Bus_Time initialization service, i.e. preservation of
the Bus_Time when the cycle master node of a bus changes.  However, that
would be quite some code for a feature that is unreliable to begin with
and very likely unused in practice.

Signed-off-by: Stefan Richter <stefanr@s5r6.in-berlin.de>
drivers/firewire/core-cdev.c
drivers/firewire/core-transaction.c
drivers/firewire/core.h
drivers/firewire/ohci.c