Hi Jacek, On Tue, Mar 24, 2015 at 09:35:05AM +0100, Jacek Anaszewski wrote: ... > >>>>diff --git a/drivers/media/v4l2-core/v4l2-flash.c > >>>>b/drivers/media/v4l2-core/v4l2-flash.c > >>>>new file mode 100644 > >>>>index 0000000..804c2e4 > >>>>--- /dev/null > >>>>+++ b/drivers/media/v4l2-core/v4l2-flash.c > >>>>@@ -0,0 +1,607 @@ > >>>>+/* > >>>>+ * V4L2 Flash LED sub-device registration helpers. > >>>>+ * > >>>>+ * Copyright (C) 2015 Samsung Electronics Co., Ltd > >>>>+ * Author: Jacek Anaszewski <[email protected]> > >>>>+ * > >>>>+ * This program is free software; you can redistribute it and/or modify > >>>>+ * it under the terms of the GNU General Public License version 2 as > >>>>+ * published by the Free Software Foundation. > >>>>+ */ > >>>>+ > >>>>+#include <linux/led-class-flash.h> > >>>>+#include <linux/module.h> > >>>>+#include <linux/mutex.h> > >>>>+#include <linux/of.h> > >>>>+#include <linux/slab.h> > >>>>+#include <linux/types.h> > >>>>+#include <media/v4l2-flash.h> > >>>>+#include "../../leds/leds.h" > >>> > >>>What do you need from leds.h? Shouldn't this be e.g. under include/linux > >>>instead? > > I need led_trigger_remove function.
It's exported but defined in what is obviously a private header file to the framework. Could it be moved to include/linux/leds.h instead? -- Regards, Sakari Ailus e-mail: [email protected] XMPP: [email protected] -- To unsubscribe from this list: send the line "unsubscribe linux-leds" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
