On 05/25/2015 04:28 PM, Sylwester Nawrocki wrote:
On 25/05/15 14:50, Jacek Anaszewski wrote:
On 23/05/15 14:03, Sakari Ailus wrote:
On Thu, May 21, 2015 at 03:28:40PM +0200, Sylwester Nawrocki wrote:
flash-leds = <&flash_xx &image_sensor_x>, <...>;
One more matter to consider: xenon flash devices.
How about samsung,camera-flashes (and ti,camera-flashes)? After pondering
this awhile, I'm ok with removing the vendor prefix as well.
Let me know what you think.
I thought about it a bit more and I have some doubts about semantics
as above. I'm fine with 'camera-flashes' as far as name is concerned.
Perhaps we should put only phandles to leds or xenon flash devices
in the 'camera-flashes' property. I think it would be more future
proof in case there is more nodes needed to describe the camera flash
(or a camera module) than the above two. And phandles to corresponding
image sensor device nodes would be put in a separate property.
Could you give examples of the cases you are thinking of?
I don't have any examples in mind ATM, I just wanted to point out
the above convention might not be flexible enough. Especially since
we already know there is more sub-devices within the camera module
than just flashes and image sensors.
camera-flashes = <&flash_xx>, ...
camera-flash-masters = <&image_sensor_x>, ...
Then pairs at same index would describe a single flash, 0 would indicate
a null entry if needed.
When it should be needed?
Not sure if there is a real use case for null entries, it was just to
note we can skip any entry if needed - probably an irrelevant comment.
I could imagine 2 LEDs of which one is only triggered in software, so
it wouldn't have a 'camera-flash-masters' entry.
Similarly we could create properties for other sub-devices of a camera
module, like lenses, etc.
I have had the ninth version of the patch set ready to send today,
so in view of your doubts I made the property samsung specific
so as not to prevent us from going further while we will be
discussing the implementation of the generic property.
The 9th version of the patch set has been just sent.
--
Best Regards,
Jacek Anaszewski
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
