On Thu, Feb 28, 2013 at 04:43:06PM -0800, John Stultz wrote: > From: Erik Gilling <konk...@android.com> > > This is needed to allow modules to link against the sync subsystem > > Cc: Maarten Lankhorst <maarten.lankho...@canonical.com> > Cc: Erik Gilling <konk...@android.com> > Cc: Daniel Vetter <daniel.vet...@ffwll.ch> > Cc: Rob Clark <robcl...@gmail.com> > Cc: Sumit Semwal <sumit.sem...@linaro.org> > Cc: Greg KH <gre...@linuxfoundation.org> > Cc: dri-de...@lists.freedesktop.org > Cc: Android Kernel Team <kernel-t...@android.com> > Signed-off-by: Erik Gilling <konk...@android.com> > Signed-off-by: John Stultz <john.stu...@linaro.org> > --- > 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 majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
