From: Randy Dunlap <rdunlap@infradead.org> To: Tim Harvey <tharvey@gateworks.com>, linux-media@vger.kernel.org, alsa-devel@alsa-project.org Cc: devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, shawnguo@kernel.org, Steve Longerbeam <slongerbeam@gmail.com>, Philipp Zabel <p.zabel@pengutronix.de>, Hans Verkuil <hansverk@cisco.com>, Mauro Carvalho Chehab <mchehab@s-opensource.com>, Hans Verkuil <hverkuil@xs4all.nl>, Hans Verkuil <hans.verkuil@cisco.com> Subject: Re: [PATCH v8 1/7] v4l2-dv-timings: add v4l2_hdmi_colorimetry() Date: Tue, 6 Feb 2018 17:22:40 -0800 [thread overview] Message-ID: <c1dc5de9-e81d-b9bd-b587-1a94de96f97c@infradead.org> (raw) In-Reply-To: <1517948874-21681-2-git-send-email-tharvey@gateworks.com> On 02/06/2018 12:27 PM, Tim Harvey wrote: > From: Hans Verkuil <hverkuil@xs4all.nl> > > Add the v4l2_hdmi_colorimetry() function so we have a single function > that determines the colorspace, YCbCr encoding, quantization range and > transfer function from the InfoFrame data. > > Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com> > Signed-off-by: Tim Harvey <tharvey@gateworks.com> > --- > drivers/media/v4l2-core/v4l2-dv-timings.c | 141 ++++++++++++++++++++++++++++++ > include/media/v4l2-dv-timings.h | 21 +++++ > 2 files changed, 162 insertions(+) > > diff --git a/drivers/media/v4l2-core/v4l2-dv-timings.c b/drivers/media/v4l2-core/v4l2-dv-timings.c > index 930f9c5..0182d3d 100644 > --- a/drivers/media/v4l2-core/v4l2-dv-timings.c > +++ b/drivers/media/v4l2-core/v4l2-dv-timings.c > @@ -27,6 +27,7 @@ > #include <linux/v4l2-dv-timings.h> > #include <media/v4l2-dv-timings.h> > #include <linux/math64.h> > +#include <linux/hdmi.h> > > MODULE_AUTHOR("Hans Verkuil"); > MODULE_DESCRIPTION("V4L2 DV Timings Helper Functions"); > @@ -814,3 +815,143 @@ struct v4l2_fract v4l2_calc_aspect_ratio(u8 hor_landscape, u8 vert_portrait) > return aspect; > } > EXPORT_SYMBOL_GPL(v4l2_calc_aspect_ratio); > + > +/** v4l2_hdmi_rx_colorimetry - determine HDMI colorimetry information > + * based on various InfoFrames. > + * @avi - the AVI InfoFrame > + * @hdmi - the HDMI Vendor InfoFrame, may be NULL > + * @height - the frame height kernel-doc format for function parameters is like: * @avi: the AVI InfoFrame etc. > + * > + * Determines the HDMI colorimetry information, i.e. how the HDMI > + * pixel color data should be interpreted. > + * > + * Note that some of the newer features (DCI-P3, HDR) are not yet > + * implemented: the hdmi.h header needs to be updated to the HDMI 2.0 > + * and CTA-861-G standards. > + */ > +struct v4l2_hdmi_colorimetry > +v4l2_hdmi_rx_colorimetry(const struct hdmi_avi_infoframe *avi, > + const struct hdmi_vendor_infoframe *hdmi, > + unsigned int height) > +{ thanks, -- ~Randy
WARNING: multiple messages have this Message-ID (diff)
From: Randy Dunlap <rdunlap@infradead.org> To: Tim Harvey <tharvey@gateworks.com>, linux-media@vger.kernel.org, alsa-devel@alsa-project.org Cc: devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Hans Verkuil <hansverk@cisco.com>, Mauro Carvalho Chehab <mchehab@s-opensource.com>, Hans Verkuil <hans.verkuil@cisco.com>, Philipp Zabel <p.zabel@pengutronix.de>, Steve Longerbeam <slongerbeam@gmail.com>, Hans Verkuil <hverkuil@xs4all.nl>, shawnguo@kernel.org Subject: Re: [PATCH v8 1/7] v4l2-dv-timings: add v4l2_hdmi_colorimetry() Date: Tue, 6 Feb 2018 17:22:40 -0800 [thread overview] Message-ID: <c1dc5de9-e81d-b9bd-b587-1a94de96f97c@infradead.org> (raw) In-Reply-To: <1517948874-21681-2-git-send-email-tharvey@gateworks.com> On 02/06/2018 12:27 PM, Tim Harvey wrote: > From: Hans Verkuil <hverkuil@xs4all.nl> > > Add the v4l2_hdmi_colorimetry() function so we have a single function > that determines the colorspace, YCbCr encoding, quantization range and > transfer function from the InfoFrame data. > > Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com> > Signed-off-by: Tim Harvey <tharvey@gateworks.com> > --- > drivers/media/v4l2-core/v4l2-dv-timings.c | 141 ++++++++++++++++++++++++++++++ > include/media/v4l2-dv-timings.h | 21 +++++ > 2 files changed, 162 insertions(+) > > diff --git a/drivers/media/v4l2-core/v4l2-dv-timings.c b/drivers/media/v4l2-core/v4l2-dv-timings.c > index 930f9c5..0182d3d 100644 > --- a/drivers/media/v4l2-core/v4l2-dv-timings.c > +++ b/drivers/media/v4l2-core/v4l2-dv-timings.c > @@ -27,6 +27,7 @@ > #include <linux/v4l2-dv-timings.h> > #include <media/v4l2-dv-timings.h> > #include <linux/math64.h> > +#include <linux/hdmi.h> > > MODULE_AUTHOR("Hans Verkuil"); > MODULE_DESCRIPTION("V4L2 DV Timings Helper Functions"); > @@ -814,3 +815,143 @@ struct v4l2_fract v4l2_calc_aspect_ratio(u8 hor_landscape, u8 vert_portrait) > return aspect; > } > EXPORT_SYMBOL_GPL(v4l2_calc_aspect_ratio); > + > +/** v4l2_hdmi_rx_colorimetry - determine HDMI colorimetry information > + * based on various InfoFrames. > + * @avi - the AVI InfoFrame > + * @hdmi - the HDMI Vendor InfoFrame, may be NULL > + * @height - the frame height kernel-doc format for function parameters is like: * @avi: the AVI InfoFrame etc. > + * > + * Determines the HDMI colorimetry information, i.e. how the HDMI > + * pixel color data should be interpreted. > + * > + * Note that some of the newer features (DCI-P3, HDR) are not yet > + * implemented: the hdmi.h header needs to be updated to the HDMI 2.0 > + * and CTA-861-G standards. > + */ > +struct v4l2_hdmi_colorimetry > +v4l2_hdmi_rx_colorimetry(const struct hdmi_avi_infoframe *avi, > + const struct hdmi_vendor_infoframe *hdmi, > + unsigned int height) > +{ thanks, -- ~Randy
next prev parent reply other threads:[~2018-02-07 1:22 UTC|newest] Thread overview: 38+ messages / expand[flat|nested] mbox.gz Atom feed top 2018-02-06 20:27 [PATCH v8 0/7] TDA1997x HDMI video reciver Tim Harvey 2018-02-06 20:27 ` Tim Harvey 2018-02-06 20:27 ` [PATCH v8 1/7] v4l2-dv-timings: add v4l2_hdmi_colorimetry() Tim Harvey 2018-02-06 20:27 ` Tim Harvey 2018-02-07 1:22 ` Randy Dunlap [this message] 2018-02-07 1:22 ` Randy Dunlap 2018-02-06 20:27 ` [PATCH v8 2/7] media: v4l-ioctl: fix pad for VIDIOC_DV_TIMIGNS_CAP Tim Harvey 2018-02-06 20:27 ` Tim Harvey 2018-02-06 20:27 ` [PATCH v8 3/7] MAINTAINERS: add entry for NXP TDA1997x driver Tim Harvey 2018-02-06 20:27 ` [PATCH v8 4/7] media: dt-bindings: Add bindings for TDA1997X Tim Harvey 2018-02-06 20:27 ` Tim Harvey 2018-02-06 20:27 ` [PATCH v8 5/7] media: i2c: Add TDA1997x HDMI receiver driver Tim Harvey 2018-02-06 20:27 ` Tim Harvey 2018-02-06 20:38 ` Hans Verkuil 2018-02-06 20:38 ` Hans Verkuil 2018-02-06 21:03 ` Tim Harvey 2018-02-06 21:03 ` Tim Harvey 2018-02-06 21:09 ` Hans Verkuil 2018-02-06 20:27 ` [PATCH v8 6/7] ARM: dts: imx: Add TDA19971 HDMI Receiver to GW54xx Tim Harvey 2018-02-06 20:27 ` [PATCH v8 7/7] ARM: dts: imx: Add TDA19971 HDMI Receiver to GW551x Tim Harvey 2018-02-06 21:21 ` [PATCH v8 0/7] TDA1997x HDMI video reciver Hans Verkuil 2018-02-06 21:21 ` Hans Verkuil 2018-02-06 23:29 ` Tim Harvey 2018-02-06 23:29 ` Tim Harvey 2018-02-07 8:22 ` Hans Verkuil 2018-02-07 8:22 ` Hans Verkuil 2018-02-07 9:09 ` Hans Verkuil 2018-02-07 22:05 ` Tim Harvey 2018-02-07 22:05 ` Tim Harvey 2018-02-07 22:19 ` Hans Verkuil 2018-02-08 10:46 ` Philipp Zabel 2018-02-08 10:46 ` Philipp Zabel 2018-02-08 11:56 ` Hans Verkuil 2018-02-08 12:01 ` Hans Verkuil 2018-02-08 12:01 ` Hans Verkuil 2018-02-08 15:13 ` Philipp Zabel 2018-02-08 15:13 ` Philipp Zabel 2018-02-08 15:29 ` Hans Verkuil
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=c1dc5de9-e81d-b9bd-b587-1a94de96f97c@infradead.org \ --to=rdunlap@infradead.org \ --cc=alsa-devel@alsa-project.org \ --cc=devicetree@vger.kernel.org \ --cc=hans.verkuil@cisco.com \ --cc=hansverk@cisco.com \ --cc=hverkuil@xs4all.nl \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-media@vger.kernel.org \ --cc=mchehab@s-opensource.com \ --cc=p.zabel@pengutronix.de \ --cc=shawnguo@kernel.org \ --cc=slongerbeam@gmail.com \ --cc=tharvey@gateworks.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
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.