Mauro,
On Sun, Aug 19, 2012 at 10:23 PM, Ezequiel Garcia <elezegarcia@xxxxxxxxx> wrote:
> This is just a cleaning patch to produce more useful
> debug messages.
>
> Signed-off-by: Ezequiel Garcia <elezegarcia@xxxxxxxxx>
> ---
> drivers/media/usb/stk1160/stk1160-video.c | 14 +++++++-------
> 1 files changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/drivers/media/usb/stk1160/stk1160-video.c b/drivers/media/usb/stk1160/stk1160-video.c
> index 3785269..022092a 100644
> --- a/drivers/media/usb/stk1160/stk1160-video.c
> +++ b/drivers/media/usb/stk1160/stk1160-video.c
> @@ -342,18 +342,18 @@ static void stk1160_isoc_irq(struct urb *urb)
> */
> void stk1160_cancel_isoc(struct stk1160 *dev)
> {
> - int i;
> + int i, num_bufs = dev->isoc_ctl.num_bufs;
>
> /*
> * This check is not necessary, but we add it
> * to avoid a spurious debug message
> */
> - if (!dev->isoc_ctl.num_bufs)
> + if (!num_bufs)
> return;
>
> - stk1160_dbg("killing urbs...\n");
> + stk1160_dbg("killing %d urbs...\n", num_bufs);
>
> - for (i = 0; i < dev->isoc_ctl.num_bufs; i++) {
> + for (i = 0; i < num_bufs; i++) {
>
> /*
> * To kill urbs we can't be in atomic context.
> @@ -373,11 +373,11 @@ void stk1160_cancel_isoc(struct stk1160 *dev)
> void stk1160_free_isoc(struct stk1160 *dev)
> {
> struct urb *urb;
> - int i;
> + int i, num_bufs = dev->isoc_ctl.num_bufs;
>
> - stk1160_dbg("freeing urb buffers...\n");
> + stk1160_dbg("freeing %d urb buffers...\n", num_bufs);
>
> - for (i = 0; i < dev->isoc_ctl.num_bufs; i++) {
> + for (i = 0; i < num_bufs; i++) {
>
> urb = dev->isoc_ctl.urb[i];
> if (urb) {
> --
> 1.7.8.6
>
Please don't forget these patches for your v3.7 pull request.
Unless you don't want to add them yet.
Thanks,
Ezequiel.
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
