* [PATCH 1/1] v4l uapi: Make "Vertical Colour Bars" menu item a little more generic @ 2018-12-04 13:45 Sakari Ailus 2018-12-04 14:32 ` Mauro Carvalho Chehab 0 siblings, 1 reply; 3+ messages in thread From: Sakari Ailus @ 2018-12-04 13:45 UTC (permalink / raw) To: linux-media; +Cc: bingbu.cao The test pattern could contain a different number of colour bars than eight, make the entry more useful by removing "Eight " from the name. Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com> --- include/uapi/linux/v4l2-controls.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/uapi/linux/v4l2-controls.h b/include/uapi/linux/v4l2-controls.h index acb2a57fa5d6..88f2759c2ce4 100644 --- a/include/uapi/linux/v4l2-controls.h +++ b/include/uapi/linux/v4l2-controls.h @@ -1016,7 +1016,7 @@ enum v4l2_jpeg_chroma_subsampling { #define V4L2_CID_TEST_PATTERN (V4L2_CID_IMAGE_PROC_CLASS_BASE + 3) #define V4L2_TEST_PATTERN_DISABLED "Disabled" #define V4L2_TEST_PATTERN_SOLID_COLOUR "Solid Colour" -#define V4L2_TEST_PATTERN_VERT_COLOUR_BARS "Eight Vertical Colour Bars" +#define V4L2_TEST_PATTERN_VERT_COLOUR_BARS "Vertical Colour Bars" #define V4L2_TEST_PATTERN_VERT_COLOUR_BARS_FADE_TO_GREY "Colour Bars With Fade to Grey" #define V4L2_TEST_PATTERN_PN9 "Pseudorandom Sequence (PN9)" #define V4L2_CID_DEINTERLACING_MODE (V4L2_CID_IMAGE_PROC_CLASS_BASE + 4) -- 2.11.0 ^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH 1/1] v4l uapi: Make "Vertical Colour Bars" menu item a little more generic 2018-12-04 13:45 [PATCH 1/1] v4l uapi: Make "Vertical Colour Bars" menu item a little more generic Sakari Ailus @ 2018-12-04 14:32 ` Mauro Carvalho Chehab 2018-12-04 14:38 ` Mauro Carvalho Chehab 0 siblings, 1 reply; 3+ messages in thread From: Mauro Carvalho Chehab @ 2018-12-04 14:32 UTC (permalink / raw) To: Sakari Ailus; +Cc: linux-media, bingbu.cao Em Tue, 4 Dec 2018 15:45:06 +0200 Sakari Ailus <sakari.ailus@linux.intel.com> escreveu: > The test pattern could contain a different number of colour bars than > eight, make the entry more useful by removing "Eight " from the name. > > Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com> > --- > include/uapi/linux/v4l2-controls.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/include/uapi/linux/v4l2-controls.h b/include/uapi/linux/v4l2-controls.h > index acb2a57fa5d6..88f2759c2ce4 100644 > --- a/include/uapi/linux/v4l2-controls.h > +++ b/include/uapi/linux/v4l2-controls.h > @@ -1016,7 +1016,7 @@ enum v4l2_jpeg_chroma_subsampling { > #define V4L2_CID_TEST_PATTERN (V4L2_CID_IMAGE_PROC_CLASS_BASE + 3) > #define V4L2_TEST_PATTERN_DISABLED "Disabled" > #define V4L2_TEST_PATTERN_SOLID_COLOUR "Solid Colour" > -#define V4L2_TEST_PATTERN_VERT_COLOUR_BARS "Eight Vertical Colour Bars" > +#define V4L2_TEST_PATTERN_VERT_COLOUR_BARS "Vertical Colour Bars" No, we can't do that. This is on an uAPI file. We should, instead, create another #define for non-eight vertical color bars. Before you say, yeah, I agree that we messed this one, as the defined name doesn't mention 8 bars... I would, instead, do something like: -#define V4L2_TEST_PATTERN_VERT_COLOUR_BARS "Eight Vertical Colour Bars" +#define V4L2_TEST_PATTERN_VERT_8_COLOUR_BARS "Eight Vertical Colour Bars" +#define V4L2_TEST_PATTERN_VERT_N_COLOUR_BARS "Vertical Colour Bars" + + /* Kept for backward-compatibility */ +#define V4L2_TEST_PATTERN_VERT_COLOUR_BARS V4L2_TEST_PATTERN_VERT_8_COLOUR_BARS And, of course, update the documentation accordingly. > #define V4L2_TEST_PATTERN_VERT_COLOUR_BARS_FADE_TO_GREY "Colour Bars With Fade to Grey" > #define V4L2_TEST_PATTERN_PN9 "Pseudorandom Sequence (PN9)" > #define V4L2_CID_DEINTERLACING_MODE (V4L2_CID_IMAGE_PROC_CLASS_BASE + 4) Thanks, Mauro ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH 1/1] v4l uapi: Make "Vertical Colour Bars" menu item a little more generic 2018-12-04 14:32 ` Mauro Carvalho Chehab @ 2018-12-04 14:38 ` Mauro Carvalho Chehab 0 siblings, 0 replies; 3+ messages in thread From: Mauro Carvalho Chehab @ 2018-12-04 14:38 UTC (permalink / raw) To: Sakari Ailus; +Cc: linux-media, bingbu.cao Em Tue, 4 Dec 2018 12:32:32 -0200 Mauro Carvalho Chehab <mchehab+samsung@kernel.org> escreveu: > Em Tue, 4 Dec 2018 15:45:06 +0200 > Sakari Ailus <sakari.ailus@linux.intel.com> escreveu: > > > The test pattern could contain a different number of colour bars than > > eight, make the entry more useful by removing "Eight " from the name. > > > > Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com> > > --- > > include/uapi/linux/v4l2-controls.h | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/include/uapi/linux/v4l2-controls.h b/include/uapi/linux/v4l2-controls.h > > index acb2a57fa5d6..88f2759c2ce4 100644 > > --- a/include/uapi/linux/v4l2-controls.h > > +++ b/include/uapi/linux/v4l2-controls.h > > @@ -1016,7 +1016,7 @@ enum v4l2_jpeg_chroma_subsampling { > > #define V4L2_CID_TEST_PATTERN (V4L2_CID_IMAGE_PROC_CLASS_BASE + 3) > > #define V4L2_TEST_PATTERN_DISABLED "Disabled" > > #define V4L2_TEST_PATTERN_SOLID_COLOUR "Solid Colour" > > -#define V4L2_TEST_PATTERN_VERT_COLOUR_BARS "Eight Vertical Colour Bars" > > +#define V4L2_TEST_PATTERN_VERT_COLOUR_BARS "Vertical Colour Bars" > > No, we can't do that. This is on an uAPI file. > > We should, instead, create another #define for non-eight vertical > color bars. > > Before you say, yeah, I agree that we messed this one, as the defined > name doesn't mention 8 bars... > > I would, instead, do something like: > > -#define V4L2_TEST_PATTERN_VERT_COLOUR_BARS "Eight Vertical Colour Bars" > +#define V4L2_TEST_PATTERN_VERT_8_COLOUR_BARS "Eight Vertical Colour Bars" > +#define V4L2_TEST_PATTERN_VERT_N_COLOUR_BARS "Vertical Colour Bars" > + > + /* Kept for backward-compatibility */ > +#define V4L2_TEST_PATTERN_VERT_COLOUR_BARS V4L2_TEST_PATTERN_VERT_8_COLOUR_BARS > > And, of course, update the documentation accordingly. Please ignore this comment. I didn't realize that those definitions don't exist yet at the uAPI file, and that this is a follow up for another patch: Subject: [PATCH v2 1/1] media: Use common test pattern menu entries Next time, please put them into a patch series, as it makes easier for reviewers. Thanks, Mauro ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-12-04 14:38 UTC | newest] Thread overview: 3+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2018-12-04 13:45 [PATCH 1/1] v4l uapi: Make "Vertical Colour Bars" menu item a little more generic Sakari Ailus 2018-12-04 14:32 ` Mauro Carvalho Chehab 2018-12-04 14:38 ` Mauro Carvalho Chehab
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.