diff mbox

[01/12] video/hdmi: Replace the payload length by their defines

Message ID 1375817544-14565-2-git-send-email-damien.lespiau@intel.com (mailing list archive)
State New, archived
Headers show

Commit Message

Lespiau, Damien Aug. 6, 2013, 7:32 p.m. UTC 
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(-)

Comments

Alex Deucher Aug. 7, 2013, 2:06 a.m. UTC | #1 
Patches 1-5, 10, 11 are:

Reviewed-by: Alex Deucher <alexander.deucher@amd.com>

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
Daniel Vetter Aug. 7, 2013, 8:13 a.m. UTC | #2 
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
diff mbox

Patch

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;
 }