[PATCH 2/8] ARM i.MX dma-mx1-mx2: use wrapper
Sascha Hauer
s.hauer at pengutronix.de
Mon Aug 9 17:42:30 EDT 2010
On Mon, Aug 09, 2010 at 08:51:01PM +0200, Magnus Lilja wrote:
> Hi Sascha,
>
> > diff --git a/arch/arm/mach-imx/dma-v1.c b/arch/arm/mach-imx/dma-v1.c
> > index 3e8c47c..d141458 100644
> > --- a/arch/arm/mach-imx/dma-v1.c
> > +++ b/arch/arm/mach-imx/dma-v1.c
> > @@ -35,6 +35,7 @@
> ...
> > @@ -503,7 +517,7 @@ void imx_dma_disable(int channel)
> > struct imx_dma_channel *imxdma = &imx_dma_channels[channel];
> > unsigned long flags;
> >
> > - pr_debug("imxdma%d: imx_dma_disable\n", channel);
> > + pr_info("imxdma%d: imx_dma_disable\n", channel);
>
> With this change imx_dma_enable() uses pr_debug() while
> imx_dma_disable uses pr_info(). Shouldn't both use the same pr_*()
> since the functions are paired?
They should both use pr_debug. Will fix.
Sascha
--
Pengutronix e.K. | |
Industrial Linux Solutions | http://www.pengutronix.de/ |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
More information about the linux-arm-kernel
mailing list