From: Jacek Anaszewski <jacek.anaszewski-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
To: Pavel Machek <pavel-+ZI9xUNit7I@public.gmane.org>,
Sakari Ailus
<sakari.ailus-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>,
linux-leds-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Cc: linux-media-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
niklas.soderlund-1zkq55x86MTxsAP9Fp7wbw@public.gmane.org,
robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org,
hverkuil-qWit8jRvyhVmR6Xm/wNWPw@public.gmane.org,
laurent.pinchart-ryLnwIuWjnjg/C1BVhZhaw@public.gmane.org,
devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
sre-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org
Subject: Re: as3645a flash userland interface
Date: Tue, 12 Sep 2017 20:53:33 +0200 [thread overview]
Message-ID: <7b679cb3-ce58-e1d1-60bf-995896bf46eb@gmail.com> (raw)
In-Reply-To: <20170912103628.GB27117@amd>
Hi Pavel,
On 09/12/2017 12:36 PM, Pavel Machek wrote:
> Hi!
>
> There were some changes to as3645a flash controller. Before we have
> stable interface we have to keep forever I want to ask:
Note that we have already two LED flash class drivers - leds-max77693
and leds-aat1290. They have been present in mainline for over two years
now.
> What directory are the flash controls in?
>
> /sys/class/leds/led-controller:flash ?
>
> Could we arrange for something less generic, like
>
> /sys/class/leds/main-camera:flash ?
I'd rather avoid overcomplicating this. LED class device name pattern
is well defined to devicename:colour:function
(see Documentation/leds/leds-class.txt, "LED Device Naming" section).
In this case "flash" in place of the "function" segment makes the
things clear enough I suppose.
--
Best regards,
Jacek Anaszewski
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
WARNING: multiple messages have this Message-ID (diff)
From: Jacek Anaszewski <jacek.anaszewski@gmail.com>
To: Pavel Machek <pavel@ucw.cz>,
Sakari Ailus <sakari.ailus@linux.intel.com>,
linux-leds@vger.kernel.org
Cc: linux-media@vger.kernel.org, niklas.soderlund@ragnatech.se,
robh@kernel.org, hverkuil@xs4all.nl,
laurent.pinchart@ideasonboard.com, devicetree@vger.kernel.org,
sre@kernel.org
Subject: Re: as3645a flash userland interface
Date: Tue, 12 Sep 2017 20:53:33 +0200 [thread overview]
Message-ID: <7b679cb3-ce58-e1d1-60bf-995896bf46eb@gmail.com> (raw)
In-Reply-To: <20170912103628.GB27117@amd>
Hi Pavel,
On 09/12/2017 12:36 PM, Pavel Machek wrote:
> Hi!
>
> There were some changes to as3645a flash controller. Before we have
> stable interface we have to keep forever I want to ask:
Note that we have already two LED flash class drivers - leds-max77693
and leds-aat1290. They have been present in mainline for over two years
now.
> What directory are the flash controls in?
>
> /sys/class/leds/led-controller:flash ?
>
> Could we arrange for something less generic, like
>
> /sys/class/leds/main-camera:flash ?
I'd rather avoid overcomplicating this. LED class device name pattern
is well defined to devicename:colour:function
(see Documentation/leds/leds-class.txt, "LED Device Naming" section).
In this case "flash" in place of the "function" segment makes the
things clear enough I suppose.
--
Best regards,
Jacek Anaszewski
next prev parent reply other threads:[~2017-09-12 18:53 UTC|newest]
Thread overview: 62+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-09-12 8:42 [PATCH v11 00/24] Unified fwnode endpoint parser, async sub-device notifier support, N9 flash DTS Sakari Ailus
2017-09-12 8:42 ` [PATCH v11 03/24] v4l: async: Use more intuitive names for internal functions Sakari Ailus
2017-09-12 8:42 ` [PATCH v11 06/24] omap3isp: Use generic parser for parsing fwnode endpoints Sakari Ailus
2017-09-12 8:42 ` [PATCH v11 07/24] rcar-vin: " Sakari Ailus
2017-09-12 8:42 ` [PATCH v11 08/24] omap3isp: Fix check for our own sub-devices Sakari Ailus
2017-09-12 8:42 ` [PATCH v11 10/24] v4l: async: Move async subdev notifier operations to a separate structure Sakari Ailus
2017-09-12 8:42 ` [PATCH v11 11/24] v4l: async: Introduce helpers for calling async ops callbacks Sakari Ailus
[not found] ` <20170912084236.1154-12-sakari.ailus-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2017-09-12 9:39 ` Hans Verkuil
2017-09-12 9:39 ` Hans Verkuil
[not found] ` <20170912084236.1154-1-sakari.ailus-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2017-09-12 8:42 ` [PATCH v11 01/24] v4l: fwnode: Move KernelDoc documentation to the header Sakari Ailus
2017-09-12 8:42 ` Sakari Ailus
2017-09-12 8:42 ` [PATCH v11 02/24] v4l: async: Remove re-probing support Sakari Ailus
2017-09-12 8:42 ` Sakari Ailus
2017-09-12 8:42 ` [PATCH v11 04/24] v4l: async: Add V4L2 async documentation to the documentation build Sakari Ailus
2017-09-12 8:42 ` Sakari Ailus
2017-09-12 8:42 ` [PATCH v11 05/24] v4l: fwnode: Support generic parsing of graph endpoints in a device Sakari Ailus
2017-09-12 8:42 ` Sakari Ailus
2017-09-12 8:42 ` [PATCH v11 09/24] omap3isp: Print the name of the entity where no source pads could be found Sakari Ailus
2017-09-12 8:42 ` Sakari Ailus
2017-09-12 8:42 ` [PATCH v11 12/24] v4l: async: Register sub-devices before calling bound callback Sakari Ailus
2017-09-12 8:42 ` Sakari Ailus
2017-09-12 8:42 ` [PATCH v11 13/24] v4l: async: Allow async notifier register call succeed with no subdevs Sakari Ailus
2017-09-12 8:42 ` Sakari Ailus
2017-09-12 9:38 ` Hans Verkuil
2017-09-12 8:42 ` [PATCH v11 14/24] v4l: async: Allow binding notifiers to sub-devices Sakari Ailus
2017-09-12 8:42 ` Sakari Ailus
2017-09-12 8:42 ` [PATCH v11 20/24] dt: bindings: smiapp: Document lens-focus and flash-leds properties Sakari Ailus
2017-09-12 8:42 ` Sakari Ailus
2017-09-12 8:42 ` [PATCH v11 21/24] smiapp: Add support for flash and lens devices Sakari Ailus
2017-09-12 8:42 ` Sakari Ailus
[not found] ` <20170912084236.1154-22-sakari.ailus-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2017-09-12 10:31 ` Pavel Machek
2017-09-12 10:31 ` Pavel Machek
2017-09-12 8:42 ` [PATCH v11 24/24] arm: dts: omap3: N9/N950: Add flash references to the camera Sakari Ailus
2017-09-12 8:42 ` Sakari Ailus
[not found] ` <20170912084236.1154-25-sakari.ailus-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2017-09-12 10:36 ` as3645a flash userland interface Pavel Machek
2017-09-12 10:36 ` Pavel Machek
2017-09-12 10:47 ` Sakari Ailus
2017-09-12 11:40 ` Pavel Machek
2017-09-12 12:04 ` Sakari Ailus
2017-09-12 18:53 ` Jacek Anaszewski [this message]
2017-09-12 18:53 ` Jacek Anaszewski
2017-09-12 21:55 ` Pavel Machek
2017-09-13 17:53 ` Jacek Anaszewski
[not found] ` <CGME20170914092415epcas2p26c049a698851778673034c16afb290b9@epcas2p2.samsung.com>
[not found] ` <21824758-28a1-7007-6db5-86a900025d14-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2017-09-14 9:24 ` Sylwester Nawrocki
2017-09-14 9:24 ` Sylwester Nawrocki
[not found] ` <4bf12e8e-beff-0199-cdee-4a52ebe7cdaf-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
2017-09-14 10:07 ` Pavel Machek
2017-09-14 10:07 ` Pavel Machek
2017-09-14 10:28 ` Sakari Ailus
2017-09-14 10:28 ` Sakari Ailus
2017-09-14 11:01 ` Sylwester Nawrocki
[not found] ` <1f34a891-edb1-251c-86a8-ba4a90c485d3-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
2017-09-14 11:53 ` Pavel Machek
2017-09-14 11:53 ` Pavel Machek
2017-09-14 13:16 ` Sylwester Nawrocki
[not found] ` <7b679cb3-ce58-e1d1-60bf-995896bf46eb-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2017-09-18 9:12 ` Sakari Ailus
2017-09-18 9:12 ` Sakari Ailus
2017-09-12 8:42 ` [PATCH v11 15/24] dt: bindings: Add a binding for flash LED devices associated to a sensor Sakari Ailus
2017-09-12 8:42 ` [PATCH v11 16/24] dt: bindings: Add lens-focus binding for image sensors Sakari Ailus
2017-09-12 8:42 ` [PATCH v11 17/24] v4l: fwnode: Add a helper function for parsing generic references Sakari Ailus
2017-09-12 8:42 ` [PATCH v11 18/24] v4l: fwnode: Add a helper function to obtain device / interger references Sakari Ailus
2017-09-12 8:42 ` [PATCH v11 19/24] v4l: fwnode: Add convenience function for parsing common external refs Sakari Ailus
2017-09-12 8:42 ` [PATCH v11 22/24] ov5670: Add support for flash and lens devices Sakari Ailus
2017-09-12 8:42 ` [PATCH v11 23/24] ov13858: " Sakari Ailus
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=7b679cb3-ce58-e1d1-60bf-995896bf46eb@gmail.com \
--to=jacek.anaszewski-re5jqeeqqe8avxtiumwx3w@public.gmane.org \
--cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=hverkuil-qWit8jRvyhVmR6Xm/wNWPw@public.gmane.org \
--cc=laurent.pinchart-ryLnwIuWjnjg/C1BVhZhaw@public.gmane.org \
--cc=linux-leds-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linux-media-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=niklas.soderlund-1zkq55x86MTxsAP9Fp7wbw@public.gmane.org \
--cc=pavel-+ZI9xUNit7I@public.gmane.org \
--cc=robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
--cc=sakari.ailus-VuQAYsv1563Yd54FQh9/CA@public.gmane.org \
--cc=sre-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.