ASoC: txx9aclc: dynamically allocate dmaengine devname
authorAtsushi Nemoto <anemo@mba.ocn.ne.jp>
Thu, 25 Jun 2009 13:36:58 +0000 (22:36 +0900)
committerMark Brown <broonie@opensource.wolfsonmicro.com>
Thu, 25 Jun 2009 20:06:30 +0000 (21:06 +0100)
Use kasprintf to allocate temporary devname string instead of a
fixed size string.
This fixes "FIXME" introduced on removal of BUS_ID_SIZE.

Signed-off-by: Atsushi Nemoto <anemo@mba.ocn.ne.jp>
Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
sound/soc/txx9/txx9aclc.c

index 938a58a5a244c1d75c43ed969e3c14b64fb08bb1..efed64b8b026c18117b6b58a49d8cfb1dd467955 100644 (file)
@@ -297,15 +297,17 @@ static int txx9aclc_pcm_new(struct snd_card *card, struct snd_soc_dai *dai,
 static bool filter(struct dma_chan *chan, void *param)
 {
        struct txx9aclc_dmadata *dmadata = param;
-       char devname[20 + 2]; /* FIXME: old BUS_ID_SIZE + 2 */
+       char *devname;
+       bool found = false;
 
-       snprintf(devname, sizeof(devname), "%s.%d", dmadata->dma_res->name,
+       devname = kasprintf(GFP_KERNEL, "%s.%d", dmadata->dma_res->name,
                (int)dmadata->dma_res->start);
        if (strcmp(dev_name(chan->device->dev), devname) == 0) {
                chan->private = &dmadata->dma_slave;
-               return true;
+               found = true;
        }
-       return false;
+       kfree(devname);
+       return found;
 }
 
 static int txx9aclc_dma_init(struct txx9aclc_soc_device *dev,