Re: [PATCH v4 5/5] OMAP: mailbox: use runtime pm for clk and sysc handling

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

 



Hi,

On Thu, Dec 16, 2010 at 2:28 AM, Varadarajan, Charulatha <charu@xxxxxx> wrote:
> On Thu, Dec 16, 2010 at 12:17, Omar Ramirez Luna <omar.ramirez@xxxxxx> wrote:
>> Use runtime pm APIs to enable/disable mailbox clocks and
>> to configure SYSC register.
>>
>> Based on the patch sent by Felipe Contreras:
>> https://patchwork.kernel.org/patch/101662/
>>
>> Signed-off-by: Omar Ramirez Luna <omar.ramirez@xxxxxx>
>> ---
>>  arch/arm/mach-omap2/mailbox.c |   27 +++++----------------------
>>  1 files changed, 5 insertions(+), 22 deletions(-)
>>
>> diff --git a/arch/arm/mach-omap2/mailbox.c b/arch/arm/mach-omap2/mailbox.c
>> index 40ddeca..f5f72ba 100644
>> --- a/arch/arm/mach-omap2/mailbox.c
>> +++ b/arch/arm/mach-omap2/mailbox.c
>> @@ -14,6 +14,7 @@
>>  #include <linux/err.h>
>>  #include <linux/platform_device.h>
>>  #include <linux/io.h>
>> +#include <linux/pm_runtime.h>
>>  #include <plat/mailbox.h>
>>  #include <mach/irqs.h>
>>
>> @@ -34,12 +35,8 @@
>>  #define MAILBOX_IRQ_NOTFULL(m)         (1 << (2 * (m) + 1))
>>
>>  /* SYSCONFIG: register bit definition */
>> -#define AUTOIDLE       (1 << 0)
>>  #define SOFTRESET      (1 << 1)
>> -#define SMARTIDLE      (2 << 3)
>>  #define OMAP4_SOFTRESET        (1 << 0)
>> -#define OMAP4_NOIDLE   (1 << 2)
>> -#define OMAP4_SMARTIDLE        (2 << 2)
>>
>>  /* SYSSTATUS: register bit definition */
>>  #define RESETDONE      (1 << 0)
>
> Is this still required?

Yes, mailbox uses the softreset every time it is requested (and that
it has no current users at the time), it does it before configuring
sysc, as opossed to what this patch does (where clk is enabled, sysc
configured, and then softreset), but no harm was seen with this
sequence.

AFAIK, enabling/disabling hwmod handles prm reset but not softreset
bit, that's why I left it there.

Regards,

Omar
--
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