From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rodrigo Vivi Subject: Re: [PATCH 1/6] drm/i915: add attached connector to hdmi container Date: Wed, 09 Sep 2015 18:54:17 +0000 Message-ID: References: <1441373176-22302-1-git-send-email-sonika.jindal@intel.com> <1441373176-22302-2-git-send-email-sonika.jindal@intel.com> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============1483564187==" Return-path: Received: from mail-ig0-f178.google.com (mail-ig0-f178.google.com [209.85.213.178]) by gabe.freedesktop.org (Postfix) with ESMTPS id 23C5C6EC98 for ; Wed, 9 Sep 2015 11:54:27 -0700 (PDT) Received: by igbkq10 with SMTP id kq10so1707264igb.0 for ; Wed, 09 Sep 2015 11:54:26 -0700 (PDT) In-Reply-To: <1441373176-22302-2-git-send-email-sonika.jindal@intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Sonika Jindal , intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org --===============1483564187== Content-Type: multipart/alternative; boundary=001a1134b964fdfaf6051f550201 --001a1134b964fdfaf6051f550201 Content-Type: text/plain; charset=UTF-8 Reviewed-by: Rodrigo Vivi On Fri, Sep 4, 2015 at 6:37 AM Sonika Jindal wrote: > From: Shashank Sharma > > This patch adds the intel_connector initialized to intel_hdmi > display, during the init phase, just like the other encoders do. > This attachment is very useful when we need to extract the connector > pointer during the hotplug handler function > > Signed-off-by: Shashank Sharma > --- > drivers/gpu/drm/i915/intel_drv.h | 1 + > drivers/gpu/drm/i915/intel_hdmi.c | 1 + > 2 files changed, 2 insertions(+) > > diff --git a/drivers/gpu/drm/i915/intel_drv.h > b/drivers/gpu/drm/i915/intel_drv.h > index c61ba47..b6c2c20 100644 > --- a/drivers/gpu/drm/i915/intel_drv.h > +++ b/drivers/gpu/drm/i915/intel_drv.h > @@ -667,6 +667,7 @@ struct intel_hdmi { > enum hdmi_force_audio force_audio; > bool rgb_quant_range_selectable; > enum hdmi_picture_aspect aspect_ratio; > + struct intel_connector *attached_connector; > void (*write_infoframe)(struct drm_encoder *encoder, > enum hdmi_infoframe_type type, > const void *frame, ssize_t len); > diff --git a/drivers/gpu/drm/i915/intel_hdmi.c > b/drivers/gpu/drm/i915/intel_hdmi.c > index ed25f64..5bdb386 100644 > --- a/drivers/gpu/drm/i915/intel_hdmi.c > +++ b/drivers/gpu/drm/i915/intel_hdmi.c > @@ -2107,6 +2107,7 @@ void intel_hdmi_init_connector(struct > intel_digital_port *intel_dig_port, > > intel_connector_attach_encoder(intel_connector, intel_encoder); > drm_connector_register(connector); > + intel_hdmi->attached_connector = intel_connector; > > /* For G4X desktop chip, PEG_BAND_GAP_DATA 3:0 must first be > written > * 0xd. Failure to do so will result in spurious interrupts being > -- > 1.7.10.4 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/intel-gfx > --001a1134b964fdfaf6051f550201 Content-Type: text/html; charset=UTF-8 Content-Transfer-Encoding: quoted-printable
Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>

On Fri, Sep 4, 2015 at 6:37 AM Sonika Jindal <sonika.jindal@intel.com> wro= te:
From: Shashank Sharma <shashank.sharma@i= ntel.com>

This patch adds the intel_connector initialized to intel_hdmi
display, during the init phase, just like the other encoders do.
This attachment is very useful when we need to extract the connector
pointer during the hotplug handler function

Signed-off-by: Shashank Sharma <shashank.sharma@intel.com>
---
=C2=A0drivers/gpu/drm/i915/intel_drv.h=C2=A0 |=C2=A0 =C2=A0 1 +
=C2=A0drivers/gpu/drm/i915/intel_hdmi.c |=C2=A0 =C2=A0 1 +
=C2=A02 files changed, 2 insertions(+)

diff --git a/drivers/gpu/drm/i915/intel_drv.h b/drivers/gpu/drm/i915/intel_= drv.h
index c61ba47..b6c2c20 100644
--- a/drivers/gpu/drm/i915/intel_drv.h
+++ b/drivers/gpu/drm/i915/intel_drv.h
@@ -667,6 +667,7 @@ struct intel_hdmi {
=C2=A0 =C2=A0 =C2=A0 =C2=A0 enum hdmi_force_audio force_audio;
=C2=A0 =C2=A0 =C2=A0 =C2=A0 bool rgb_quant_range_selectable;
=C2=A0 =C2=A0 =C2=A0 =C2=A0 enum hdmi_picture_aspect aspect_ratio;
+=C2=A0 =C2=A0 =C2=A0 =C2=A0struct intel_connector *attached_connector;
=C2=A0 =C2=A0 =C2=A0 =C2=A0 void (*write_infoframe)(struct drm_encoder *enc= oder,
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 enum hdmi_infoframe_type type,
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 const void *frame, ssize_t len);
diff --git a/drivers/gpu/drm/i915/intel_hdmi.c b/drivers/gpu/drm/i915/intel= _hdmi.c
index ed25f64..5bdb386 100644
--- a/drivers/gpu/drm/i915/intel_hdmi.c
+++ b/drivers/gpu/drm/i915/intel_hdmi.c
@@ -2107,6 +2107,7 @@ void intel_hdmi_init_connector(struct intel_digital_p= ort *intel_dig_port,

=C2=A0 =C2=A0 =C2=A0 =C2=A0 intel_connector_attach_encoder(intel_connector,= intel_encoder);
=C2=A0 =C2=A0 =C2=A0 =C2=A0 drm_connector_register(connector);
+=C2=A0 =C2=A0 =C2=A0 =C2=A0intel_hdmi->attached_connector =3D intel_con= nector;

=C2=A0 =C2=A0 =C2=A0 =C2=A0 /* For G4X desktop chip, PEG_BAND_GAP_DATA 3:0 = must first be written
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0* 0xd.=C2=A0 Failure to do so will result= in spurious interrupts being
--
1.7.10.4

_______________________________________________
Intel-gfx mailing list
Intel-= gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo= /intel-gfx
--001a1134b964fdfaf6051f550201-- --===============1483564187== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KSW50ZWwtZ2Z4 IG1haWxpbmcgbGlzdApJbnRlbC1nZnhAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHA6Ly9saXN0 cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9pbnRlbC1nZngK --===============1483564187==--