Commit a7bfb27b authored by Lee Jones's avatar Lee Jones Committed by Mika Westerberg

thunderbolt: nhi: Demote some non-conformant kernel-doc headers

Fixes the following W=1 kernel build warning(s):

 drivers/thunderbolt/nhi.c:53: warning: Function parameter or member 'ring' not described in 'ring_interrupt_active'
 drivers/thunderbolt/nhi.c:53: warning: Function parameter or member 'active' not described in 'ring_interrupt_active'
 drivers/thunderbolt/nhi.c:114: warning: Function parameter or member 'nhi' not described in 'nhi_disable_interrupts'
 drivers/thunderbolt/nhi.c:191: warning: Function parameter or member 'ring' not described in 'ring_write_descriptors'
 drivers/thunderbolt/nhi.c:225: warning: Function parameter or member 'work' not described in 'ring_work'

Cc: Andreas Noever <andreas.noever@gmail.com>
Cc: Michael Jamet <michael.jamet@intel.com>
Cc: Yehezkel Bernat <YehezkelShB@gmail.com>
Cc: linux-usb@vger.kernel.org
Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
[ mw: Demote only static functions ]
Signed-off-by: default avatarMika Westerberg <mika.westerberg@linux.intel.com>
parent 925a2af9
...@@ -44,7 +44,7 @@ static int ring_interrupt_index(struct tb_ring *ring) ...@@ -44,7 +44,7 @@ static int ring_interrupt_index(struct tb_ring *ring)
return bit; return bit;
} }
/** /*
* ring_interrupt_active() - activate/deactivate interrupts for a single ring * ring_interrupt_active() - activate/deactivate interrupts for a single ring
* *
* ring->nhi->lock must be held. * ring->nhi->lock must be held.
...@@ -105,7 +105,7 @@ static void ring_interrupt_active(struct tb_ring *ring, bool active) ...@@ -105,7 +105,7 @@ static void ring_interrupt_active(struct tb_ring *ring, bool active)
iowrite32(new, ring->nhi->iobase + reg); iowrite32(new, ring->nhi->iobase + reg);
} }
/** /*
* nhi_disable_interrupts() - disable interrupts for all rings * nhi_disable_interrupts() - disable interrupts for all rings
* *
* Use only during init and shutdown. * Use only during init and shutdown.
...@@ -182,7 +182,7 @@ static bool ring_empty(struct tb_ring *ring) ...@@ -182,7 +182,7 @@ static bool ring_empty(struct tb_ring *ring)
return ring->head == ring->tail; return ring->head == ring->tail;
} }
/** /*
* ring_write_descriptors() - post frames from ring->queue to the controller * ring_write_descriptors() - post frames from ring->queue to the controller
* *
* ring->lock is held. * ring->lock is held.
...@@ -212,7 +212,7 @@ static void ring_write_descriptors(struct tb_ring *ring) ...@@ -212,7 +212,7 @@ static void ring_write_descriptors(struct tb_ring *ring)
} }
} }
/** /*
* ring_work() - progress completed frames * ring_work() - progress completed frames
* *
* If the ring is shutting down then all frames are marked as canceled and * If the ring is shutting down then all frames are marked as canceled and
...@@ -754,7 +754,7 @@ void tb_ring_free(struct tb_ring *ring) ...@@ -754,7 +754,7 @@ void tb_ring_free(struct tb_ring *ring)
dev_dbg(&ring->nhi->pdev->dev, "freeing %s %d\n", RING_TYPE(ring), dev_dbg(&ring->nhi->pdev->dev, "freeing %s %d\n", RING_TYPE(ring),
ring->hop); ring->hop);
/** /*
* ring->work can no longer be scheduled (it is scheduled only * ring->work can no longer be scheduled (it is scheduled only
* by nhi_interrupt_work, ring_stop and ring_msix). Wait for it * by nhi_interrupt_work, ring_stop and ring_msix). Wait for it
* to finish before freeing the ring. * to finish before freeing the ring.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment