On 07/22/2016 07:18 AM, Wu, Songjun wrote:

<snip>

quoted

quoted

+/*
+ * index(0~11):  raw formats.
+ * index(12~12): the formats which can be converted from raw format by ISC.
+ * index():      the formats which can only be provided by subdev.
+ */
+static struct isc_format isc_formats[] = {

static const

Some members in structure isc_format need be modified, so it can not be 
const.

OK. Please add a comment about that.

Regards,

	Hans