At Fri, 19 Dec 2008 16:35:01 +0200, Jarkko Nikula wrote: > > To: "ext Stanley.Miao" <stanley.miao@xxxxxxxxxxxxx> > Cc: alsa-devel@xxxxxxxxxxxxxxxx, linux-omap@xxxxxxxxxxxxxxx > Subject: Re: [alsa-devel] [PATCHv2 1/1] Fix a Oops bug in omap soc driver. > From: Jarkko Nikula <jarkko.nikula@xxxxxxxxx> > Delivered-To: alsa-devel@xxxxxxxxxxxxxxxx > Date: Fri, 19 Dec 2008 16:35:01 +0200 > Message-Id: <20081219163501.0e235884.jarkko.nikula@xxxxxxxxx> > In-Reply-To: <4edcd8463d981e4babef82167aa58131539c367f.1229601269.git.stanley.miao@xxxxxxxxxxxxx> > Organization: Nokia-M/Helsinki > Mime-Version: 1.0 > List-Unsubscribe: <http://mailman.alsa-project.org/mailman/listinfo/alsa-devel>, <mailto:alsa-devel-request@xxxxxxxxxxxxxxxx?subject=unsubscribe> > List-Archive: <http://mailman.alsa-project.org/pipermail/alsa-devel> > List-Post: <mailto:alsa-devel@xxxxxxxxxxxxxxxx> > List-Help: <mailto:alsa-devel-request@xxxxxxxxxxxxxxxx?subject=help> > List-Subscribe: <http://mailman.alsa-project.org/mailman/listinfo/alsa-devel>, <mailto:alsa-devel-request@xxxxxxxxxxxxxxxx?subject=subscribe> > Content-Transfer-Encoding: 7bit > > On Fri, 19 Dec 2008 22:08:22 +0800 > "ext Stanley.Miao" <stanley.miao@xxxxxxxxxxxxx> wrote: > > > There will be a Oops or frequent underrun messages when playing music with > > omap soc driver, this is because a data region is incorretly sized, other data > > region will be overwriten when writing to this data region. > > > > Signed-off-by: Stanley.Miao <stanley.miao@xxxxxxxxxxxxx> > > --- > > sound/soc/omap/omap-pcm.c | 2 +- > > 1 files changed, 1 insertions(+), 1 deletions(-) > > > > diff --git a/sound/soc/omap/omap-pcm.c b/sound/soc/omap/omap-pcm.c > > index ce580a9..2adfbce 100644 > > --- a/sound/soc/omap/omap-pcm.c > > +++ b/sound/soc/omap/omap-pcm.c > > @@ -233,7 +233,7 @@ static int omap_pcm_open(struct snd_pcm_substream *substream) > > if (ret < 0) > > goto out; > > > > - prtd = kzalloc(sizeof(prtd), GFP_KERNEL); > > + prtd = kzalloc(sizeof(struct omap_runtime_data), GFP_KERNEL); > > Oh my... Can you change this as > > prtd = kzalloc(sizeof(*prtd), GFP_KERNEL); > > like suggested by the CodingStyle and then you can have my ack. > > Acked-by: Jarkko Nikula <jarkko.nikula@xxxxxxxxx> Thanks, merged now. I'm going to push the fixes to the upstream soon. Takashi -- 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