David Cohen wrote: > On Mon, Mar 28, 2011 at 3:55 PM, Sakari Ailus > <sakari.ailus@xxxxxxxxxxxxxxxxxxxxxxxxxx> wrote: >> Hi, > > Hi Sakari, Thanks for the comments, David! > [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. I'll change this to the next version of the RFC. Regards, -- Sakari Ailus sakari.ailus@xxxxxxxxxxxxxxxxxxxxxxxxxx -- 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