Re: [PATCH 4/5] Staging/iio/adc/touchscreen/MXS: add interrupt driven touch detection

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Dear Jürgen Beisert,

> 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.

You can put them into a separate patch which can be NAK'd separatelly :)

Best regards,
Marek Vasut
--
To unsubscribe from this list: send the line "unsubscribe linux-iio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux