2011-02-25 06:16:59

by Justin P. Mattock

[permalink] [raw]
Subject: [PATCH 21/21]sound:soc:samsung:dma.c remove one to many l's in the word.

The patch below removes an extra "l" in the word.

Signed-off-by: Justin P. Mattock <[email protected]>

---
sound/soc/samsung/dma.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/sound/soc/samsung/dma.c b/sound/soc/samsung/dma.c
index 2124019..807efa5 100644
--- a/sound/soc/samsung/dma.c
+++ b/sound/soc/samsung/dma.c
@@ -313,7 +313,7 @@ dma_pointer(struct snd_pcm_substream *substream)
/* we seem to be getting the odd error from the pcm library due
* to out-of-bounds pointers. this is maybe due to the dma engine
* not having loaded the new values for the channel before being
- * callled... (todo - fix )
+ * called... (todo - fix )
*/

if (res >= snd_pcm_lib_buffer_bytes(substream)) {
--
1.7.4.1


2011-02-25 07:00:06

by Jassi Brar

[permalink] [raw]
Subject: Re: [alsa-devel] [PATCH 21/21]sound:soc:samsung:dma.c remove one to many l's in the word.

On Fri, Feb 25, 2011 at 3:16 PM, Justin P. Mattock
<[email protected]> wrote:
> The patch below removes an extra "l" in the word.
>
> Signed-off-by: Justin P. Mattock <[email protected]>

Acked-by: Jassi Brar <[email protected]>

2011-02-25 09:01:45

by Liam Girdwood

[permalink] [raw]
Subject: Re: [alsa-devel] [PATCH 21/21]sound:soc:samsung:dma.c remove one to many l's in the word.

On Fri, 2011-02-25 at 16:00 +0900, Jassi Brar wrote:
> On Fri, Feb 25, 2011 at 3:16 PM, Justin P. Mattock
> <[email protected]> wrote:
> > The patch below removes an extra "l" in the word.
> >
> > Signed-off-by: Justin P. Mattock <[email protected]>
>
> Acked-by: Jassi Brar <[email protected]>

Acked-by: Liam Girdwood <[email protected]>

2011-02-25 11:51:18

by Mark Brown

[permalink] [raw]
Subject: Re: [alsa-devel] [PATCH 21/21]sound:soc:samsung:dma.c remove one to many l's in the word.

On Thu, Feb 24, 2011 at 10:16:37PM -0800, Justin P. Mattock wrote:
> The patch below removes an extra "l" in the word.
>
> Signed-off-by: Justin P. Mattock <[email protected]>

Applied.

Please remember to CC maintainers on patches as covered in
Documentation/SubmittingPatches, even trivial patches should be made
visible to maintainers. Please also try to use changelogs for your
patch which resemble other changelogs for the code area you're working
on.

2011-03-01 14:26:20

by Jiri Kosina

[permalink] [raw]
Subject: Re: [alsa-devel] [PATCH 21/21]sound:soc:samsung:dma.c remove one to many l's in the word.

On Fri, 25 Feb 2011, Mark Brown wrote:

> On Thu, Feb 24, 2011 at 10:16:37PM -0800, Justin P. Mattock wrote:
> > The patch below removes an extra "l" in the word.
> >
> > Signed-off-by: Justin P. Mattock <[email protected]>
>
> Applied.

I don't see it in linux-next as of today though ... ?

>
> Please remember to CC maintainers on patches as covered in
> Documentation/SubmittingPatches, even trivial patches should be made
> visible to maintainers. Please also try to use changelogs for your
> patch which resemble other changelogs for the code area you're working
> on.
>

--
Jiri Kosina
SUSE Labs, Novell Inc.

2011-03-01 14:28:57

by Mark Brown

[permalink] [raw]
Subject: Re: [alsa-devel] [PATCH 21/21]sound:soc:samsung:dma.c remove one to many l's in the word.

On Tue, Mar 01, 2011 at 03:26:16PM +0100, Jiri Kosina wrote:
> On Fri, 25 Feb 2011, Mark Brown wrote:

> > Applied.

> I don't see it in linux-next as of today though ... ?

I do - it's ceade6c81bd913b23bffb3d80909ea430850c79b. I rewrote the
subject line because:

> > Please remember to CC maintainers on patches as covered in
> > Documentation/SubmittingPatches, even trivial patches should be made
> > visible to maintainers. Please also try to use changelogs for your
> > patch which resemble other changelogs for the code area you're working
> > on.

2011-03-01 14:43:12

by Jiri Kosina

[permalink] [raw]
Subject: Re: [alsa-devel] [PATCH 21/21]sound:soc:samsung:dma.c remove one to many l's in the word.

On Tue, 1 Mar 2011, Mark Brown wrote:

> On Tue, Mar 01, 2011 at 03:26:16PM +0100, Jiri Kosina wrote:
> > On Fri, 25 Feb 2011, Mark Brown wrote:
>
> > > Applied.
>
> > I don't see it in linux-next as of today though ... ?
>
> I do - it's ceade6c81bd913b23bffb3d80909ea430850c79b.

Bah, I had bogus clone of linux-next, sorry for the noise.

--
Jiri Kosina
SUSE Labs, Novell Inc.