Message ID | 20200124200231.10517-4-ville.syrjala@linux.intel.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | [1/8] drm/edid: Check the number of detailed timing descriptors in the CEA ext block | expand |
Title should be s/i915/edid/ , with that fixed: Reviewed-by: Alex Deucher <alexander.deucher@amd.com> On Fri, Jan 24, 2020 at 3:03 PM Ville Syrjala <ville.syrjala@linux.intel.com> wrote: > > From: Ville Syrjälä <ville.syrjala@linux.intel.com> > > Nuke some whitespace that shouldn't be there. > > Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> > --- > drivers/gpu/drm/drm_edid.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c > index d6bce58b27ac..3df5744026b0 100644 > --- a/drivers/gpu/drm/drm_edid.c > +++ b/drivers/gpu/drm/drm_edid.c > @@ -2842,7 +2842,7 @@ do_inferred_modes(struct detailed_timing *timing, void *c) > closure->modes += drm_dmt_modes_for_range(closure->connector, > closure->edid, > timing); > - > + > if (!version_greater(closure->edid, 1, 1)) > return; /* GTF not defined yet */ > > @@ -3084,7 +3084,7 @@ do_cvt_mode(struct detailed_timing *timing, void *c) > > static int > add_cvt_modes(struct drm_connector *connector, struct edid *edid) > -{ > +{ > struct detailed_mode_closure closure = { > .connector = connector, > .edid = edid, > @@ -4342,7 +4342,7 @@ void drm_edid_get_monitor_name(struct edid *edid, char *name, int bufsize) > { > int name_length; > char buf[13]; > - > + > if (bufsize <= 0) > return; > > -- > 2.24.1 > > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel
> -----Original Message----- > From: Intel-gfx <intel-gfx-bounces@lists.freedesktop.org> On Behalf Of Alex > Deucher > Sent: Tuesday, January 28, 2020 3:58 AM > To: Ville Syrjala <ville.syrjala@linux.intel.com> > Cc: Intel Graphics Development <intel-gfx@lists.freedesktop.org>; Maling list - DRI > developers <dri-devel@lists.freedesktop.org> > Subject: Re: [Intel-gfx] [PATCH 4/8] drm/i915: Clear out spurious whitespace > > Title should be s/i915/edid/ , with that fixed: > Reviewed-by: Alex Deucher <alexander.deucher@amd.com> Yeah with the title fixed, this is Reviewed-by: Uma Shankar <uma.shankar@intel.com> > > On Fri, Jan 24, 2020 at 3:03 PM Ville Syrjala <ville.syrjala@linux.intel.com> wrote: > > > > From: Ville Syrjälä <ville.syrjala@linux.intel.com> > > > > Nuke some whitespace that shouldn't be there. > > > > Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> > > --- > > drivers/gpu/drm/drm_edid.c | 6 +++--- > > 1 file changed, 3 insertions(+), 3 deletions(-) > > > > diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c > > index d6bce58b27ac..3df5744026b0 100644 > > --- a/drivers/gpu/drm/drm_edid.c > > +++ b/drivers/gpu/drm/drm_edid.c > > @@ -2842,7 +2842,7 @@ do_inferred_modes(struct detailed_timing *timing, void > *c) > > closure->modes += drm_dmt_modes_for_range(closure->connector, > > closure->edid, > > timing); > > - > > + > > if (!version_greater(closure->edid, 1, 1)) > > return; /* GTF not defined yet */ > > > > @@ -3084,7 +3084,7 @@ do_cvt_mode(struct detailed_timing *timing, void > > *c) > > > > static int > > add_cvt_modes(struct drm_connector *connector, struct edid *edid) -{ > > +{ > > struct detailed_mode_closure closure = { > > .connector = connector, > > .edid = edid, > > @@ -4342,7 +4342,7 @@ void drm_edid_get_monitor_name(struct edid > > *edid, char *name, int bufsize) { > > int name_length; > > char buf[13]; > > - > > + > > if (bufsize <= 0) > > return; > > > > -- > > 2.24.1 > > > > _______________________________________________ > > dri-devel mailing list > > dri-devel@lists.freedesktop.org > > https://lists.freedesktop.org/mailman/listinfo/dri-devel > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/intel-gfx
diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c index d6bce58b27ac..3df5744026b0 100644 --- a/drivers/gpu/drm/drm_edid.c +++ b/drivers/gpu/drm/drm_edid.c @@ -2842,7 +2842,7 @@ do_inferred_modes(struct detailed_timing *timing, void *c) closure->modes += drm_dmt_modes_for_range(closure->connector, closure->edid, timing); - + if (!version_greater(closure->edid, 1, 1)) return; /* GTF not defined yet */ @@ -3084,7 +3084,7 @@ do_cvt_mode(struct detailed_timing *timing, void *c) static int add_cvt_modes(struct drm_connector *connector, struct edid *edid) -{ +{ struct detailed_mode_closure closure = { .connector = connector, .edid = edid, @@ -4342,7 +4342,7 @@ void drm_edid_get_monitor_name(struct edid *edid, char *name, int bufsize) { int name_length; char buf[13]; - + if (bufsize <= 0) return;