From mboxrd@z Thu Jan  1 00:00:00 1970
From: Daniel Vetter <daniel@ffwll.ch>
Subject: Re: [PATCH 01/12] video/hdmi: Replace the payload
 length by their defines
Date: Wed, 7 Aug 2013 10:13:16 +0200
Message-ID: <20130807081315.GS22035@phenom.ffwll.local>
References: <1375817544-14565-1-git-send-email-damien.lespiau@intel.com>
	<1375817544-14565-2-git-send-email-damien.lespiau@intel.com>
	<CADnq5_NgTUQ_wgbOkCCzQWzuJpfSTL50ALbpDN-Bfc25WVKJMw@mail.gmail.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
Return-path: <intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org>
Received: from mail-wi0-f177.google.com (mail-wi0-f177.google.com
	[209.85.212.177])
	by gabe.freedesktop.org (Postfix) with ESMTP id D3528E5CE5
	for <intel-gfx@lists.freedesktop.org>;
	Wed,  7 Aug 2013 01:13:09 -0700 (PDT)
Received: by mail-wi0-f177.google.com with SMTP id hq12so1365025wib.4
	for <intel-gfx@lists.freedesktop.org>;
	Wed, 07 Aug 2013 01:13:08 -0700 (PDT)
Content-Disposition: inline
In-Reply-To: <CADnq5_NgTUQ_wgbOkCCzQWzuJpfSTL50ALbpDN-Bfc25WVKJMw@mail.gmail.com>
List-Unsubscribe: <http://lists.freedesktop.org/mailman/options/intel-gfx>,
	<mailto:intel-gfx-request@lists.freedesktop.org?subject=unsubscribe>
List-Archive: <http://lists.freedesktop.org/archives/intel-gfx>
List-Post: <mailto:intel-gfx@lists.freedesktop.org>
List-Help: <mailto:intel-gfx-request@lists.freedesktop.org?subject=help>
List-Subscribe: <http://lists.freedesktop.org/mailman/listinfo/intel-gfx>,
	<mailto:intel-gfx-request@lists.freedesktop.org?subject=subscribe>
Sender: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org
Errors-To: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org
To: Alex Deucher <alexdeucher@gmail.com>
Cc: Thierry Reding <thierry.reding@avionic-design.de>, intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org
List-Id: intel-gfx@lists.freedesktop.org

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=E4l=E4 <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 =3D HDMI_INFOFRAME_TYPE_AVI;
> >         frame->version =3D 2;
> > -       frame->length =3D 13;
> > +       frame->length =3D HDMI_AVI_INFOFRAME_SIZE;
> >
> >         return 0;
> >  }
> > @@ -151,7 +151,7 @@ int hdmi_spd_infoframe_init(struct hdmi_spd_infofra=
me *frame,
> >
> >         frame->type =3D HDMI_INFOFRAME_TYPE_SPD;
> >         frame->version =3D 1;
> > -       frame->length =3D 25;
> > +       frame->length =3D 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_inf=
oframe *frame)
> >
> >         frame->type =3D HDMI_INFOFRAME_TYPE_AUDIO;
> >         frame->version =3D 1;
> > -       frame->length =3D 10;
> > +       frame->length =3D 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