RE: [OMAPZOOM][PATCH 5/5] DSPBRIDGE: Remove unnecessary checks in HW_MBOX_IsFull

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Felipe,

	This patch is base on your patch about increase the performance waiting less and checking more the mail box.

Regards,
Fernando.

-----Original Message-----
From: Felipe Contreras [mailto:felipe.contreras@xxxxxxxxx] 
Sent: Tuesday, March 03, 2009 1:22 AM
To: Guzman Lugo, Fernando
Cc: Pandita, Vikram; Kanigeri, Hari; Aguilar Pena, Leed; linux-omap@xxxxxxxxxxxxxxx
Subject: Re: [OMAPZOOM][PATCH 5/5] DSPBRIDGE: Remove unnecessary checks in HW_MBOX_IsFull

On Tue, Mar 3, 2009 at 3:35 AM, Guzman Lugo, Fernando <x0095840@xxxxxx> wrote:
> From db89674b7316d7490e71c131091758eb7ef0eca2 Mon Sep 17 00:00:00 2001
> From: Fernando Guzman Lugo <x0095840@xxxxxx>
> Date: Wed, 25 Feb 2009 19:27:41 -0600
> Subject: [PATCH] DSPBRIDGE: Remove unnecessary checks in HW_MBOX_IsFull function
>
> This patch removes some unnecesarry checks in HW_MBOX_IsFull
> function

Is that all this patch is doing?

> -       u16 cnt = 10;
> +       u16 cnt = 1000;

> -       while (--cnt) {
> -               hwStatus = HW_MBOX_IsFull(resources.dwMboxBase,
> -                                          MBOX_ARM2DSP, &mbxFull);
> -               if (mbxFull)
> -                       UTIL_Wait(1000);        /* wait for 1 ms)      */
> -               else
> -                       break;
> -       }
> +       while (--cnt && HW_MBOX_IsFull(resources.dwMboxBase, MBOX_ARM2DSP))
> +               udelay(1);
> +

-- 
Felipe Contreras

--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux