Re: [PATCH v3 2/3] smiapp: Add driver-specific test pattern menu item definitions

2014-06-11 Thread Sakari Ailus

Hi Prabhakar,

Thanks for the review! :-)

Prabhakar Lad wrote:

Hi Sakari,

On Thu, May 29, 2014 at 3:40 PM, Sakari Ailus
sakari.ai...@linux.intel.com wrote:

Add numeric definitions for menu items used in the smiapp driver's test
pattern menu.

Signed-off-by: Sakari Ailus sakari.ai...@linux.intel.com
---
  include/uapi/linux/smiapp.h | 29 +


Don't you need to add an entry in Kbuild file for this ?


Good poing. I'll send v3.1 for this one as well.

--
Sakari Ailus
sakari.ai...@linux.intel.com
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v3 2/3] smiapp: Add driver-specific test pattern menu item definitions

2014-06-10 Thread Hans Verkuil
On 05/29/14 16:40, Sakari Ailus wrote:
 Add numeric definitions for menu items used in the smiapp driver's test
 pattern menu.
 
 Signed-off-by: Sakari Ailus sakari.ai...@linux.intel.com

Acked-by: Hans Verkuil hans.verk...@cisco.com

 ---
  include/uapi/linux/smiapp.h | 29 +
  1 file changed, 29 insertions(+)
  create mode 100644 include/uapi/linux/smiapp.h
 
 diff --git a/include/uapi/linux/smiapp.h b/include/uapi/linux/smiapp.h
 new file mode 100644
 index 000..53938f4
 --- /dev/null
 +++ b/include/uapi/linux/smiapp.h
 @@ -0,0 +1,29 @@
 +/*
 + * include/uapi/linux/smiapp.h
 + *
 + * Generic driver for SMIA/SMIA++ compliant camera modules
 + *
 + * Copyright (C) 2014 Intel Corporation
 + * Contact: Sakari Ailus sakari.ai...@iki.fi
 + *
 + * This program is free software; you can redistribute it and/or
 + * modify it under the terms of the GNU General Public License
 + * version 2 as published by the Free Software Foundation.
 + *
 + * This program is distributed in the hope that it will be useful, but
 + * WITHOUT ANY WARRANTY; without even the implied warranty of
 + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
 + * General Public License for more details.
 + *
 + */
 +
 +#ifndef __UAPI_LINUX_SMIAPP_H_
 +#define __UAPI_LINUX_SMIAPP_H_
 +
 +#define V4L2_SMIAPP_TEST_PATTERN_MODE_DISABLED   0
 +#define V4L2_SMIAPP_TEST_PATTERN_MODE_SOLID_COLOUR   1
 +#define V4L2_SMIAPP_TEST_PATTERN_MODE_COLOUR_BARS2
 +#define V4L2_SMIAPP_TEST_PATTERN_MODE_COLOUR_BARS_GREY   3
 +#define V4L2_SMIAPP_TEST_PATTERN_MODE_PN94
 +
 +#endif /* __UAPI_LINUX_SMIAPP_H_ */
 

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


Re: [PATCH v3 2/3] smiapp: Add driver-specific test pattern menu item definitions

2014-06-10 Thread Prabhakar Lad
Hi Sakari,

On Thu, May 29, 2014 at 3:40 PM, Sakari Ailus
sakari.ai...@linux.intel.com wrote:
 Add numeric definitions for menu items used in the smiapp driver's test
 pattern menu.

 Signed-off-by: Sakari Ailus sakari.ai...@linux.intel.com
 ---
  include/uapi/linux/smiapp.h | 29 +

Don't you need to add an entry in Kbuild file for this ?

Regards,
--Prabhakar Lad

  1 file changed, 29 insertions(+)
  create mode 100644 include/uapi/linux/smiapp.h

 diff --git a/include/uapi/linux/smiapp.h b/include/uapi/linux/smiapp.h
 new file mode 100644
 index 000..53938f4
 --- /dev/null
 +++ b/include/uapi/linux/smiapp.h
 @@ -0,0 +1,29 @@
 +/*
 + * include/uapi/linux/smiapp.h
 + *
 + * Generic driver for SMIA/SMIA++ compliant camera modules
 + *
 + * Copyright (C) 2014 Intel Corporation
 + * Contact: Sakari Ailus sakari.ai...@iki.fi
 + *
 + * This program is free software; you can redistribute it and/or
 + * modify it under the terms of the GNU General Public License
 + * version 2 as published by the Free Software Foundation.
 + *
 + * This program is distributed in the hope that it will be useful, but
 + * WITHOUT ANY WARRANTY; without even the implied warranty of
 + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
 + * General Public License for more details.
 + *
 + */
 +
 +#ifndef __UAPI_LINUX_SMIAPP_H_
 +#define __UAPI_LINUX_SMIAPP_H_
 +
 +#define V4L2_SMIAPP_TEST_PATTERN_MODE_DISABLED 0
 +#define V4L2_SMIAPP_TEST_PATTERN_MODE_SOLID_COLOUR 1
 +#define V4L2_SMIAPP_TEST_PATTERN_MODE_COLOUR_BARS  2
 +#define V4L2_SMIAPP_TEST_PATTERN_MODE_COLOUR_BARS_GREY 3
 +#define V4L2_SMIAPP_TEST_PATTERN_MODE_PN9  4
 +
 +#endif /* __UAPI_LINUX_SMIAPP_H_ */
 --
 1.8.3.2

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


[PATCH v3 2/3] smiapp: Add driver-specific test pattern menu item definitions

2014-05-29 Thread Sakari Ailus
Add numeric definitions for menu items used in the smiapp driver's test
pattern menu.

Signed-off-by: Sakari Ailus sakari.ai...@linux.intel.com
---
 include/uapi/linux/smiapp.h | 29 +
 1 file changed, 29 insertions(+)
 create mode 100644 include/uapi/linux/smiapp.h

diff --git a/include/uapi/linux/smiapp.h b/include/uapi/linux/smiapp.h
new file mode 100644
index 000..53938f4
--- /dev/null
+++ b/include/uapi/linux/smiapp.h
@@ -0,0 +1,29 @@
+/*
+ * include/uapi/linux/smiapp.h
+ *
+ * Generic driver for SMIA/SMIA++ compliant camera modules
+ *
+ * Copyright (C) 2014 Intel Corporation
+ * Contact: Sakari Ailus sakari.ai...@iki.fi
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License
+ * version 2 as published by the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it will be useful, but
+ * WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * General Public License for more details.
+ *
+ */
+
+#ifndef __UAPI_LINUX_SMIAPP_H_
+#define __UAPI_LINUX_SMIAPP_H_
+
+#define V4L2_SMIAPP_TEST_PATTERN_MODE_DISABLED 0
+#define V4L2_SMIAPP_TEST_PATTERN_MODE_SOLID_COLOUR 1
+#define V4L2_SMIAPP_TEST_PATTERN_MODE_COLOUR_BARS  2
+#define V4L2_SMIAPP_TEST_PATTERN_MODE_COLOUR_BARS_GREY 3
+#define V4L2_SMIAPP_TEST_PATTERN_MODE_PN9  4
+
+#endif /* __UAPI_LINUX_SMIAPP_H_ */
-- 
1.8.3.2

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


Re: [PATCH v3 2/3] smiapp: Add driver-specific test pattern menu item definitions

2014-05-29 Thread Laurent Pinchart
On Thursday 29 May 2014 17:40:47 Sakari Ailus wrote:
 Add numeric definitions for menu items used in the smiapp driver's test
 pattern menu.
 
 Signed-off-by: Sakari Ailus sakari.ai...@linux.intel.com

Acked-by: Laurent Pinchart laurent.pinch...@ideasonboard.com

 ---
  include/uapi/linux/smiapp.h | 29 +
  1 file changed, 29 insertions(+)
  create mode 100644 include/uapi/linux/smiapp.h
 
 diff --git a/include/uapi/linux/smiapp.h b/include/uapi/linux/smiapp.h
 new file mode 100644
 index 000..53938f4
 --- /dev/null
 +++ b/include/uapi/linux/smiapp.h
 @@ -0,0 +1,29 @@
 +/*
 + * include/uapi/linux/smiapp.h
 + *
 + * Generic driver for SMIA/SMIA++ compliant camera modules
 + *
 + * Copyright (C) 2014 Intel Corporation
 + * Contact: Sakari Ailus sakari.ai...@iki.fi
 + *
 + * This program is free software; you can redistribute it and/or
 + * modify it under the terms of the GNU General Public License
 + * version 2 as published by the Free Software Foundation.
 + *
 + * This program is distributed in the hope that it will be useful, but
 + * WITHOUT ANY WARRANTY; without even the implied warranty of
 + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
 + * General Public License for more details.
 + *
 + */
 +
 +#ifndef __UAPI_LINUX_SMIAPP_H_
 +#define __UAPI_LINUX_SMIAPP_H_
 +
 +#define V4L2_SMIAPP_TEST_PATTERN_MODE_DISABLED   0
 +#define V4L2_SMIAPP_TEST_PATTERN_MODE_SOLID_COLOUR   1
 +#define V4L2_SMIAPP_TEST_PATTERN_MODE_COLOUR_BARS2
 +#define V4L2_SMIAPP_TEST_PATTERN_MODE_COLOUR_BARS_GREY   3
 +#define V4L2_SMIAPP_TEST_PATTERN_MODE_PN94

Out of curiosity, what's PN9 ?

 +
 +#endif /* __UAPI_LINUX_SMIAPP_H_ */

-- 
Regards,

Laurent Pinchart

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


Re: [PATCH v3 2/3] smiapp: Add driver-specific test pattern menu item definitions

2014-05-29 Thread Sakari Ailus

Laurent Pinchart wrote:

diff --git a/include/uapi/linux/smiapp.h b/include/uapi/linux/smiapp.h
new file mode 100644
index 000..53938f4
--- /dev/null
+++ b/include/uapi/linux/smiapp.h
@@ -0,0 +1,29 @@
+/*
+ * include/uapi/linux/smiapp.h
+ *
+ * Generic driver for SMIA/SMIA++ compliant camera modules
+ *
+ * Copyright (C) 2014 Intel Corporation
+ * Contact: Sakari Ailus sakari.ai...@iki.fi
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License
+ * version 2 as published by the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it will be useful, but
+ * WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * General Public License for more details.
+ *
+ */
+
+#ifndef __UAPI_LINUX_SMIAPP_H_
+#define __UAPI_LINUX_SMIAPP_H_
+
+#define V4L2_SMIAPP_TEST_PATTERN_MODE_DISABLED 0
+#define V4L2_SMIAPP_TEST_PATTERN_MODE_SOLID_COLOUR 1
+#define V4L2_SMIAPP_TEST_PATTERN_MODE_COLOUR_BARS  2
+#define V4L2_SMIAPP_TEST_PATTERN_MODE_COLOUR_BARS_GREY 3
+#define V4L2_SMIAPP_TEST_PATTERN_MODE_PN9  4


Out of curiosity, what's PN9 ?


It's a sequence of pseudo-random binary numbers, e.g.:

URL:http://en.wikipedia.org/wiki/Pseudorandom_binary_sequence

9 is the order of the polynomial.

--
Sakari Ailus
sakari.ai...@linux.intel.com
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html