On Thu, 2016-01-21 at 11:26 +0200, Westerberg, Mika wrote: > On Wed, Jan 20, 2016 at 06:21:36PM +0200, Andy Shevchenko wrote: > > Even if we are not going to get a channel the message is issued. > > Check the > > result of acpi_dma_request_slave_chan_by_index() before issuing the > > message. > > > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> > > The debug message is there because it prints out the index of the > FixedDMA descriptor we are trying to get channel for -- even if we > cannot get the channel. Then it should sound a bit differently, shoudln't it? Something like dev_dbg(dev, "Looking for DMA channel \"%s\" at index %d...\n", name, index); > > Since it is just a debug message your reasoning works for me as well > ;-) > Acked-by: Mika Westerberg <mika.westerberg@xxxxxxxxxxxxxxx> Thanks! -- Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> Intel Finland Oy -- To unsubscribe from this list: send the line "unsubscribe dmaengine" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
