Re: [PATCH] ARM: OMAP4: wakeupgen: remove duplicate AUXCOREBOOT* read/write

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

 



Tony,

On Thu, Sep 6, 2012 at 12:05 PM, Shilimkar, Santosh
<santosh.shilimkar@xxxxxx> wrote:
>
> On Thu, Sep 6, 2012 at 5:04 AM, Paul Walmsley <paul@xxxxxxxxx> wrote:
> >
> > The wakeupgen context-save code reads and stores the AUXCOREBOOT0 and
> > AUXCOREBOOT1 register contents twice.  This seems like a waste of
> > time, so, remove the duplicates.
> >
> > Signed-off-by: Paul Walmsley <paul@xxxxxxxxx>
> > Cc: Santosh Shilimkar <santosh.shilimkar@xxxxxx>
> > ---
> Looks fine to me. Thanks Paul.
> Reviewed-by: Santosh Shilimkar ,santosh.shilimkar@xxxxxx>
>
> While reviewing your patch, i noticed a typo in the code.
> Since the patch fixes nearby code as the subject patch, I generated
> that patch on top of yours. Same is end of the email as well as
> attachment. The patch applies on top of your patch.
>
> Regards
> Santosh
>
> From 5fcd24e5782e502aa607b73de78be68b394f9639 Mon Sep 17 00:00:00 2001
> From: Santosh Shilimkar <santosh.shilimkar@xxxxxx>
> Date: Thu, 6 Sep 2012 11:51:34 +0530
> Subject: [PATCH] ARM: OMAP4: wakeupgen: Fix the typo in AUXCOREBOOT
> register
>  save
>
> The wakeupgen context-save code reads AUXCOREBOOT0 register
> instead of AUXCOREBOOT1 register contents while saving AUXCOREBOOT1.
>
> Fix the same.
>
> Signed-off-by: Santosh Shilimkar <santosh.shilimkar@xxxxxx>
> ---

Will you able to pick up these couple of wakeupgen fixes from  here or
do you want me to send you a pull request for 3.6-rc5/6

Regards
Santosh
--
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