On 4 March 2013 17:09, Andy Shevchenko <[email protected]> wrote: > We don't need to have them global and later we would like to protect access to > them as well. > > Signed-off-by: Andy Shevchenko <[email protected]> > --- > drivers/dma/dmatest.c | 29 ++++++++++++++++------------- > 1 file changed, 16 insertions(+), 13 deletions(-) > > diff --git a/drivers/dma/dmatest.c b/drivers/dma/dmatest.c > index 7f9e3cc..475a21a 100644 > --- a/drivers/dma/dmatest.c > +++ b/drivers/dma/dmatest.c > @@ -109,6 +109,7 @@ struct dmatest_chan { > * @timeout: transfer timeout in msec, -1 for infinite timeout > */ > struct dmatest_info { > + /* Test parameters */ > unsigned int buf_size; > char channel[20]; > char device[20]; > @@ -118,17 +119,14 @@ struct dmatest_info { > unsigned int xor_sources; > unsigned int pq_sources; > int timeout; > + > + /* Internal state */ > + struct list_head channels; > + unsigned int nr_channels; > };
Acked-by: Viresh Kumar <[email protected]> What about merging this patch with the patch which added this structure. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/

