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

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

 



Thursday 03 December 2009 10:35:37 Peter Ujfalusi napisał(a):
> 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).

Hi Peter,

That's probably my fault, I've dropped you from CC: at some point, sorry.

I'm still learning whom I should CC: and whom I shouldn't in order to rise my 
chances on patches being commented, acknowledged and applied in a reasonable 
time ;). Initial version of this one was waiting since August, that's why I 
tried to narrow the audience in hope it could help getting it rejected or 
accepted finally.

Thanks,
Janusz
--
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