Re: [PATCH v3 0/4] OMAP: McBSP: Use register cache

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

 



Hello,

On Thursday 03 December 2009 09:49:05 ext Jarkko Nikula wrote:
> On Tue, 1 Dec 2009 04:10:07 +0100
> 
> Janusz Krzysztofik <jkrzyszt@xxxxxxxxxxxx> wrote:
> > Change the way McBSP registers are maintained: store values written to
> > the device in a cache in order to  make use of those cached values when
> > convenient.
> >
> > This could help for developing the McBSP context save/restore features,
> > as well as solve the problem of possible register corruption experienced
> > on OMAP1510 based Amstrad Delta board, at least.
> >
> > Janusz Krzysztofik (4):
> > 	OMAP: McBSP: Use macros for all register read/write operations
> > 	OMAP: McBSP: Prepare register read/write macros API for caching
> > 	OMAP: McBSP: Introduce caching in register write operations
> > 	OMAP: McBSP: Use cache when modifying individual register bits
> >
> >  arch/arm/plat-omap/include/plat/mcbsp.h |    5
> >  arch/arm/plat-omap/mcbsp.c              |  397
> > ++++++++++++++++++++++++--------------------------- 2 files changed, 198
> > insertions(+), 204 deletions(-)
> 
> Looks good to me and audio works fine with the Beagle. One minor
> comment to the patch 3.

Sorry, but I have totally missed the discussion about this in l-o..
It looks good for me as well, the only comment I have is the same as Jarkko for 
patch 3.
I was about to give it a try, but since Jarkko already tested it on Beagle, I'm 
OK with this series (with the comment for patch 3).

-- 
Péter
--
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