diff options
| author | David Vrabel <dv02@dv02pc01.europe.root.pri> | 2008-10-16 13:56:53 +0100 |
|---|---|---|
| committer | David Vrabel <dv02@dv02pc01.europe.root.pri> | 2008-10-16 13:56:53 +0100 |
| commit | 8092d7c9789581eea478c40d615a2632c3af17bb (patch) | |
| tree | a9fccedfcdbfd9a74220aa99d48fa8a003313deb /drivers/uwb/beacon.c | |
| parent | 92c4d9bd1648b3eaca6b8b8f8932eec390ba7327 (diff) | |
| download | lwn-8092d7c9789581eea478c40d615a2632c3af17bb.tar.gz lwn-8092d7c9789581eea478c40d615a2632c3af17bb.zip | |
uwb: don't use printk_ratelimit() so often
Avoid using printk_ratelimit() in many places because:
- many were error messages reporting broken hardware (it's useful to
get all of these).
- the message itself wasn't useful so the message has been removed.
Signed-off-by: David Vrabel <david.vrabel@csr.com>
Diffstat (limited to 'drivers/uwb/beacon.c')
| -rw-r--r-- | drivers/uwb/beacon.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/drivers/uwb/beacon.c b/drivers/uwb/beacon.c index 1ccf9eb3e086..c7f3c972f07d 100644 --- a/drivers/uwb/beacon.c +++ b/drivers/uwb/beacon.c @@ -410,7 +410,6 @@ int uwbd_evt_handle_rc_beacon(struct uwb_event *evt) struct uwb_rc_evt_beacon *be; struct uwb_beacon_frame *bf; struct uwb_beca_e *bce; - struct device *dev = &evt->rc->uwb_dev.dev; unsigned long last_ts; rc = evt->rc; @@ -419,14 +418,12 @@ int uwbd_evt_handle_rc_beacon(struct uwb_event *evt) if (result < 0) return result; - /* Ignore beacon if it is from an alien. */ + /* FIXME: handle alien beacons. */ if (be->bBeaconType == UWB_RC_BEACON_TYPE_OL_ALIEN || be->bBeaconType == UWB_RC_BEACON_TYPE_NOL_ALIEN) { - if (printk_ratelimit()) - dev_err(dev, "BEACON received from ALIEN. Action? \n"); - result = -ENOSYS; - return 0; + return -ENOSYS; } + bf = (struct uwb_beacon_frame *) be->BeaconInfo; /* |