Re: [PATCH V2 4/6] dmaengine: tegra-apb: Only save channel state for those in use

From: Andy Shevchenko
Date: Mon Nov 09 2015 - 08:59:00 EST


On Mon, Nov 9, 2015 at 3:23 PM, Jon Hunter <jonathanh@xxxxxxxxxx> wrote:
> Currently the tegra-apb DMA driver suspend/resume helpers, save and
> restore the registers for all channels regardless of whether they are
> in use or not. Change this so that only channels that have been
> allocated and configured are saved and restored.

Nitpick

> + /*
> + * Only save the state of DMA channels that are in use.
> + */

One line?

> + /*
> + * Only restore the state of DMA channels that are in use.
> + */

Same.

--
With Best Regards,
Andy Shevchenko
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/