On Thu, Feb 28, 2013 at 04:43:06PM -0800, John Stultz wrote: > From: Erik Gilling <[email protected]> > > This is needed to allow modules to link against the sync subsystem > > Cc: Maarten Lankhorst <[email protected]> > Cc: Erik Gilling <[email protected]> > Cc: Daniel Vetter <[email protected]> > Cc: Rob Clark <[email protected]> > Cc: Sumit Semwal <[email protected]> > Cc: Greg KH <[email protected]> > Cc: [email protected] > Cc: Android Kernel Team <[email protected]> > Signed-off-by: Erik Gilling <[email protected]> > Signed-off-by: John Stultz <[email protected]> > --- > drivers/staging/android/sync.c | 14 ++++++++++++++ > 1 file changed, 14 insertions(+) > > diff --git a/drivers/staging/android/sync.c b/drivers/staging/android/sync.c > index 54f84d9..6739a84 100644 > --- a/drivers/staging/android/sync.c > +++ b/drivers/staging/android/sync.c > @@ -15,6 +15,7 @@ > */ > > #include <linux/debugfs.h> > +#include <linux/export.h> > #include <linux/file.h> > #include <linux/fs.h> > #include <linux/kernel.h> > @@ -64,6 +65,7 @@ struct sync_timeline *sync_timeline_create(const struct > sync_timeline_ops *ops, > > return obj; > } > +EXPORT_SYMBOL(sync_timeline_create);
As these are now global, should they be a bit more "specific"? "sync_" seems pretty broad. Also, EXPORT_SYMBOL_GPL() perhaps? And who is using these exports? thanks, greg k-h -- 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/
