On Mon, Mar 28, 2011 at 3:55 PM, Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxxxxxxxxxxxxx> wrote: > Hi, Hi Sakari, [snip] > This is a bitmask containing the fault information for the flash. This > assumes the proposed V4L2 bit mask controls [5]; otherwise this would > likely need to be a set of controls. > > #define V4L2_FLASH_FAULT_OVER_VOLTAGE Â Â Â Â Â 0x00000001 > #define V4L2_FLASH_FAULT_TIMEOUT Â Â Â Â Â Â Â Â0x00000002 > #define V4L2_FLASH_FAULT_OVER_TEMPERATURE Â Â Â 0x00000004 > #define V4L2_FLASH_FAULT_SHORT_CIRCUIT Â Â Â Â Â0x00000008 Sorry for bringing this a bit late. As we already talked directly, IMO (1 << 0), (1 << 1), ... could have a better readability to expose how you want to define an expand these macros. Br, David Cohen -- 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
