From: Daniel Vetter <daniel@ffwll.ch>
To: Alex Deucher <alexdeucher@gmail.com>
Cc: Thierry Reding <thierry.reding@avionic-design.de>,
intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org
Subject: Re: [PATCH 01/12] video/hdmi: Replace the payload length by their defines
Date: Wed, 7 Aug 2013 10:13:16 +0200 [thread overview]
Message-ID: <20130807081315.GS22035@phenom.ffwll.local> (raw)
In-Reply-To: <CADnq5_NgTUQ_wgbOkCCzQWzuJpfSTL50ALbpDN-Bfc25WVKJMw@mail.gmail.com>
On Tue, Aug 06, 2013 at 10:06:16PM -0400, Alex Deucher wrote:
> Patches 1-5, 10, 11 are:
>
> Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
Entire series merged to drm-intel-next-queue with Dave's irc ack. Thanks
for the patches and review.
-Daniel
>
> On Tue, Aug 6, 2013 at 3:32 PM, Damien Lespiau <damien.lespiau@intel.com> wrote:
> > Cc: Thierry Reding <thierry.reding@avionic-design.de>
> > Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
> > Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
> > ---
> > drivers/video/hdmi.c | 6 +++---
> > 1 file changed, 3 insertions(+), 3 deletions(-)
> >
> > diff --git a/drivers/video/hdmi.c b/drivers/video/hdmi.c
> > index 4017833..dbd882f 100644
> > --- a/drivers/video/hdmi.c
> > +++ b/drivers/video/hdmi.c
> > @@ -52,7 +52,7 @@ int hdmi_avi_infoframe_init(struct hdmi_avi_infoframe *frame)
> >
> > frame->type = HDMI_INFOFRAME_TYPE_AVI;
> > frame->version = 2;
> > - frame->length = 13;
> > + frame->length = HDMI_AVI_INFOFRAME_SIZE;
> >
> > return 0;
> > }
> > @@ -151,7 +151,7 @@ int hdmi_spd_infoframe_init(struct hdmi_spd_infoframe *frame,
> >
> > frame->type = HDMI_INFOFRAME_TYPE_SPD;
> > frame->version = 1;
> > - frame->length = 25;
> > + frame->length = HDMI_SPD_INFOFRAME_SIZE;
> >
> > strncpy(frame->vendor, vendor, sizeof(frame->vendor));
> > strncpy(frame->product, product, sizeof(frame->product));
> > @@ -218,7 +218,7 @@ int hdmi_audio_infoframe_init(struct hdmi_audio_infoframe *frame)
> >
> > frame->type = HDMI_INFOFRAME_TYPE_AUDIO;
> > frame->version = 1;
> > - frame->length = 10;
> > + frame->length = HDMI_AUDIO_INFOFRAME_SIZE;
> >
> > return 0;
> > }
> > --
> > 1.8.3.1
> >
> > _______________________________________________
> > dri-devel mailing list
> > dri-devel@lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/dri-devel
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel
--
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
next prev parent reply other threads:[~2013-08-07 8:13 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-08-06 19:32 Port the i915 HDMI infoframe code to the common infrastructure v2 Damien Lespiau
2013-08-06 19:32 ` [PATCH 01/12] video/hdmi: Replace the payload length by their defines Damien Lespiau
2013-08-07 2:06 ` Alex Deucher
2013-08-07 8:13 ` Daniel Vetter [this message]
2013-08-06 19:32 ` [PATCH 02/12] video/hdmi: Introduce a generic hdmi_infoframe union Damien Lespiau
2013-08-06 19:32 ` [PATCH 03/12] video/hdmi: Add a macro to return the size of a full infoframe Damien Lespiau
2013-08-06 19:32 ` [PATCH 04/12] video/hmdi: Clear the whole incoming buffer, not just the infoframe size Damien Lespiau
2013-08-07 10:56 ` [Intel-gfx] " Ville Syrjälä
2013-08-07 11:02 ` Damien Lespiau
2013-08-06 19:32 ` [PATCH 05/12] drm: Don't generate invalid AVI infoframes for CEA modes Damien Lespiau
2013-08-06 19:32 ` [PATCH 06/12] drm/i915/hdmi: Change the write_infoframe vfunc to take a buffer and a type Damien Lespiau
2013-08-07 10:58 ` Ville Syrjälä
2013-08-06 19:32 ` [PATCH 07/12] drm/i915/hdmi: Port the infoframe code to the common hdmi helpers Damien Lespiau
2013-08-07 11:00 ` Ville Syrjälä
2013-08-06 19:32 ` [PATCH 08/12] drm/i915/sdvo: Port the infoframe code to the shared infrastructure Damien Lespiau
2013-08-06 19:32 ` [PATCH 09/12] drm/i915: Remove the now obsolete infoframe definitions Damien Lespiau
2013-08-06 19:32 ` [PATCH 10/12] drm: Handle the DBLCLK flag in the common infoframe helper Damien Lespiau
2013-08-06 19:32 ` [PATCH 11/12] drm: Set aspect ratio fields in the AVI infoframe even for non CEA modes Damien Lespiau
2013-08-06 19:32 ` [PATCH 12/12] drm/i915/hmdi: Rename set_infoframe() to write_infoframe() Damien Lespiau
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=20130807081315.GS22035@phenom.ffwll.local \
--to=daniel@ffwll.ch \
--cc=alexdeucher@gmail.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=intel-gfx@lists.freedesktop.org \
--cc=thierry.reding@avionic-design.de \
/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: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox