Hi Marek, On Fri, Dec 7, 2012 at 1:24 PM, Marek Vasut <marex@xxxxxxx> wrote: > This structure definition seems to be some kind of remnant from > previous development that accidentally made it mainline. Remove > it as it's unused. > > Signed-off-by: Marek Vasut <marex@xxxxxxx> > Cc: Fabio Estevam <fabio.estevam@xxxxxxxxxxxxx> > Cc: Jonathan Cameron <jic23@xxxxxxxxxx> > Cc: Shawn Guo <shawn.guo@xxxxxxxxxx> > --- > drivers/staging/iio/adc/mxs-lradc.c | 5 ----- > 1 file changed, 5 deletions(-) > > diff --git a/drivers/staging/iio/adc/mxs-lradc.c b/drivers/staging/iio/adc/mxs-lradc.c > index ca7c1fa..67abf50 100644 > --- a/drivers/staging/iio/adc/mxs-lradc.c > +++ b/drivers/staging/iio/adc/mxs-lradc.c > @@ -75,11 +75,6 @@ static const char * const mxs_lradc_irq_name[] = { > "mxs-lradc-button1", > }; > > -struct mxs_lradc_chan { > - uint8_t slot; > - uint8_t flags; > -}; > - > struct mxs_lradc { > struct device *dev; > void __iomem *base; Can you combine patches 1 and 2 into a single patch? Thanks for working on this. I will test the series next week on my mx28evk. Regards, Fabio Estevam -- 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