Re: [RFC v5,2/3] controls.xml : add addtional Flash fault bits

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Sakari
Thank you for your comments.
Hi Daniel,

Daniel Jeong wrote:
Added more comment about Input voltage flash monitor and external temp function.

Signed-off-by: Daniel Jeong <gshark.jeong@xxxxxxxxx>
---
  Documentation/DocBook/media/v4l/controls.xml |   18 ++++++++++++++++++
  1 file changed, 18 insertions(+)

diff --git a/Documentation/DocBook/media/v4l/controls.xml b/Documentation/DocBook/media/v4l/controls.xml
index a5a3188..145a127 100644
--- a/Documentation/DocBook/media/v4l/controls.xml
+++ b/Documentation/DocBook/media/v4l/controls.xml
@@ -4370,6 +4370,24 @@ interface and may change in the future.</para>
                <entry>The flash controller has detected a short or open
                circuit condition on the indicator LED.</entry>
              </row>
+            <row>
+ <entry><constant>V4L2_FLASH_FAULT_UNDER_VOLTAGE</constant></entry>
+              <entry>Flash controller voltage to the flash LED
+              has been below the minimum limit specific to the flash
+              controller.</entry>
+            </row>
+            <row>
+ <entry><constant>V4L2_FLASH_FAULT_INPUT_VOLTAGE</constant></entry>
+ <entry>The flash controller has detected adjustment by IVFM
+              (Input Voltage Flash Monitor) block.
+          If during the flash current turn-on, the input voltage falls
+          below the threshold input voltage, IVFM adjust level</entry>

Could you add an explanation what does this fault actually mean to the user?

"IVFM" at least doesn't say anything to me...

How about this ?
"The flash current can't reach to the target current due to the input voltage falling below lower limit and flash controller adjusted the flash current not to occur low voltage envent"
+            </row>
+            <row>
+ <entry><constant>V4L2_FLASH_FAULT_LED_OVER_TEMPERATURE</constant></entry> + <entry>The flash controller has detected that TEMP input has
+              crossed threshold by external temperature sensor.</entry>

How about this instead?

"The temperature of the LED has exceeded its allowed upper limit."

Ok.
--
To unsubscribe from this list: send the line "unsubscribe linux-doc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux