On Friday 06 September 2013 14:26:11 Dan Carpenter wrote:
> On Fri, Sep 06, 2013 at 12:08:42PM +0200, Juergen Beisert wrote:
> > +static int mxs_lradc_read_ts_channel(struct mxs_lradc *lradc)
> > +{
> > + u32 reg;
> > + int val;
> > +
> > + reg = readl(lradc->base + LRADC_CTRL1);
> > +
> > + /* only channels 3 to 5 are of interest here */
> > + if (reg & LRADC_CTRL1_LRADC_IRQ(TS_CH_YP)) {
> > + pr_devel("!3");
>
> Gar. I meant to comment on this the first time this patch was sent.
>
> Please remove the pr_devel() calls. They are nonsense.
They are very helpful for development to monitor the statemachine... :) But you
are right, I will remove them.
Regards,
Juergen
--
Pengutronix e.K. | Juergen Beisert |
Linux Solutions for Science and Industry | http://www.pengutronix.de/ |
_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
