| Message ID | 1383279423-21189-1-git-send-email-mengdong.lin@intel.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
Hi Daniel, Could you help review this v4 patch? I'm sorry again this patch does not in-reply-to the previous review thread. I didn't generate a message id for my previous v2 patch by 'git format-patch'. So I took the message id generated by 'git send-email' from the my Outlook inbox, and generated this v4 patch. But it seems that message id does not make this mail in the proper review thread. I'll always use 'git format-patch' to generate the message id in the future to avoid such annoying issues. Thanks Mengdong > -----Original Message----- > From: Lin, Mengdong > Sent: Friday, November 01, 2013 12:17 PM > To: intel-gfx@lists.freedesktop.org > Cc: Lin, Mengdong > Subject: [PATCH v4] drm/i915/vlv: enable HDA display audio for Valleyview2 > > From: Mengdong Lin <mengdong.lin@intel.com> > > This patch defines HD-Audio configuration registers and enables display audio > from HDA controller for Valleyview2. > > v2: fix missing offset VLV_DISPLAY_BASE > v3: rename patch from 'enable HDMI audio' to 'enable HDA display audio', since > it's for both HDMI and DP audio > v4: use enc_to_dig_port() to get port number, instead of using Haswell specific > function intel_ddi_get_encoder_port() > > Signed-off-by: Mengdong Lin <mengdong.lin@intel.com> > Reviewed-by: Jesse Barnes <jbarnes@virtuousgeek.org> > > diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_reg.h > index 47de41f..7f68215 100644 > --- a/drivers/gpu/drm/i915/i915_reg.h > +++ b/drivers/gpu/drm/i915/i915_reg.h > @@ -4924,6 +4924,18 @@ > CPT_AUD_CNTL_ST_B) > #define CPT_AUD_CNTRL_ST2 0xE50C0 > > +#define VLV_HDMIW_HDMIEDID_A (VLV_DISPLAY_BASE + 0x62050) > +#define VLV_HDMIW_HDMIEDID_B (VLV_DISPLAY_BASE + 0x62150) > +#define VLV_HDMIW_HDMIEDID(pipe) _PIPE(pipe, \ > + VLV_HDMIW_HDMIEDID_A, \ > + VLV_HDMIW_HDMIEDID_B) > +#define VLV_AUD_CNTL_ST_A (VLV_DISPLAY_BASE + 0x620B4) > +#define VLV_AUD_CNTL_ST_B (VLV_DISPLAY_BASE + 0x621B4) > +#define VLV_AUD_CNTL_ST(pipe) _PIPE(pipe, \ > + VLV_AUD_CNTL_ST_A, \ > + VLV_AUD_CNTL_ST_B) > +#define VLV_AUD_CNTL_ST2 (VLV_DISPLAY_BASE + 0x620C0) > + > /* These are the 4 32-bit write offset registers for each stream > * output buffer. It determines the offset from the > * 3DSTATE_SO_BUFFERs that the next streamed vertex output goes to. > @@ -4940,6 +4952,12 @@ > #define CPT_AUD_CFG(pipe) _PIPE(pipe, \ > CPT_AUD_CONFIG_A, \ > CPT_AUD_CONFIG_B) > +#define VLV_AUD_CONFIG_A (VLV_DISPLAY_BASE + 0x62000) > +#define VLV_AUD_CONFIG_B (VLV_DISPLAY_BASE + 0x62100) > +#define VLV_AUD_CFG(pipe) _PIPE(pipe, \ > + VLV_AUD_CONFIG_A, \ > + VLV_AUD_CONFIG_B) > + > #define AUD_CONFIG_N_VALUE_INDEX (1 << 29) > #define AUD_CONFIG_N_PROG_ENABLE (1 << 28) > #define AUD_CONFIG_UPPER_N_SHIFT 20 > diff --git a/drivers/gpu/drm/i915/intel_display.c > b/drivers/gpu/drm/i915/intel_display.c > index c69a5b8..6328caf 100644 > --- a/drivers/gpu/drm/i915/intel_display.c > +++ b/drivers/gpu/drm/i915/intel_display.c > @@ -6992,6 +6992,11 @@ static void ironlake_write_eld(struct drm_connector > *connector, > aud_config = IBX_AUD_CFG(pipe); > aud_cntl_st = IBX_AUD_CNTL_ST(pipe); > aud_cntrl_st2 = IBX_AUD_CNTL_ST2; > + } else if (IS_VALLEYVIEW(connector->dev)) { > + hdmiw_hdmiedid = VLV_HDMIW_HDMIEDID(pipe); > + aud_config = VLV_AUD_CFG(pipe); > + aud_cntl_st = VLV_AUD_CNTL_ST(pipe); > + aud_cntrl_st2 = VLV_AUD_CNTL_ST2; > } else { > hdmiw_hdmiedid = CPT_HDMIW_HDMIEDID(pipe); > aud_config = CPT_AUD_CFG(pipe); > @@ -7001,8 +7006,19 @@ static void ironlake_write_eld(struct drm_connector > *connector, > > DRM_DEBUG_DRIVER("ELD on pipe %c\n", pipe_name(pipe)); > > - i = I915_READ(aud_cntl_st); > - i = (i >> 29) & DIP_PORT_SEL_MASK; /* DIP_Port_Select, 0x1 = > PortB */ > + if (IS_VALLEYVIEW(connector->dev)) { > + struct intel_encoder *intel_encoder; > + struct intel_digital_port *intel_dig_port; > + > + intel_encoder = intel_attached_encoder(connector); > + intel_dig_port = enc_to_dig_port(&intel_encoder->base); > + i = intel_dig_port->port; > + } else { > + i = I915_READ(aud_cntl_st); > + i = (i >> 29) & DIP_PORT_SEL_MASK; > + /* DIP_Port_Select, 0x1 = PortB */ > + } > + > if (!i) { > DRM_DEBUG_DRIVER("Audio directed to unknown port\n"); > /* operate blindly on all ports */ > @@ -10320,7 +10336,8 @@ static void intel_init_display(struct drm_device > *dev) > } > } else if (IS_G4X(dev)) { > dev_priv->display.write_eld = g4x_write_eld; > - } > + } else if (IS_VALLEYVIEW(dev)) > + dev_priv->display.write_eld = ironlake_write_eld; > > /* Default just returns -ENODEV to indicate unsupported */ > dev_priv->display.queue_flip = intel_default_queue_flip; > -- > 1.8.1.2
On Tue, Nov 05, 2013 at 05:48:20AM +0000, Lin, Mengdong wrote: > Hi Daniel, > > Could you help review this v4 patch? Oops, I wanted to merge it but forgot it. Merged to 3.13-fixes branch, thanks for the patch. > I'm sorry again this patch does not in-reply-to the previous review thread. > I didn't generate a message id for my previous v2 patch by 'git format-patch'. > So I took the message id generated by 'git send-email' from the my Outlook inbox, and generated this v4 patch. > But it seems that message id does not make this mail in the proper review thread. > I'll always use 'git format-patch' to generate the message id in the future to avoid such annoying issues. No worries ;-) Cheers, Daniel
diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_reg.h index 47de41f..7f68215 100644 --- a/drivers/gpu/drm/i915/i915_reg.h +++ b/drivers/gpu/drm/i915/i915_reg.h @@ -4924,6 +4924,18 @@ CPT_AUD_CNTL_ST_B) #define CPT_AUD_CNTRL_ST2 0xE50C0 +#define VLV_HDMIW_HDMIEDID_A (VLV_DISPLAY_BASE + 0x62050) +#define VLV_HDMIW_HDMIEDID_B (VLV_DISPLAY_BASE + 0x62150) +#define VLV_HDMIW_HDMIEDID(pipe) _PIPE(pipe, \ + VLV_HDMIW_HDMIEDID_A, \ + VLV_HDMIW_HDMIEDID_B) +#define VLV_AUD_CNTL_ST_A (VLV_DISPLAY_BASE + 0x620B4) +#define VLV_AUD_CNTL_ST_B (VLV_DISPLAY_BASE + 0x621B4) +#define VLV_AUD_CNTL_ST(pipe) _PIPE(pipe, \ + VLV_AUD_CNTL_ST_A, \ + VLV_AUD_CNTL_ST_B) +#define VLV_AUD_CNTL_ST2 (VLV_DISPLAY_BASE + 0x620C0) + /* These are the 4 32-bit write offset registers for each stream * output buffer. It determines the offset from the * 3DSTATE_SO_BUFFERs that the next streamed vertex output goes to. @@ -4940,6 +4952,12 @@ #define CPT_AUD_CFG(pipe) _PIPE(pipe, \ CPT_AUD_CONFIG_A, \ CPT_AUD_CONFIG_B) +#define VLV_AUD_CONFIG_A (VLV_DISPLAY_BASE + 0x62000) +#define VLV_AUD_CONFIG_B (VLV_DISPLAY_BASE + 0x62100) +#define VLV_AUD_CFG(pipe) _PIPE(pipe, \ + VLV_AUD_CONFIG_A, \ + VLV_AUD_CONFIG_B) + #define AUD_CONFIG_N_VALUE_INDEX (1 << 29) #define AUD_CONFIG_N_PROG_ENABLE (1 << 28) #define AUD_CONFIG_UPPER_N_SHIFT 20 diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c index c69a5b8..6328caf 100644 --- a/drivers/gpu/drm/i915/intel_display.c +++ b/drivers/gpu/drm/i915/intel_display.c @@ -6992,6 +6992,11 @@ static void ironlake_write_eld(struct drm_connector *connector, aud_config = IBX_AUD_CFG(pipe); aud_cntl_st = IBX_AUD_CNTL_ST(pipe); aud_cntrl_st2 = IBX_AUD_CNTL_ST2; + } else if (IS_VALLEYVIEW(connector->dev)) { + hdmiw_hdmiedid = VLV_HDMIW_HDMIEDID(pipe); + aud_config = VLV_AUD_CFG(pipe); + aud_cntl_st = VLV_AUD_CNTL_ST(pipe); + aud_cntrl_st2 = VLV_AUD_CNTL_ST2; } else { hdmiw_hdmiedid = CPT_HDMIW_HDMIEDID(pipe); aud_config = CPT_AUD_CFG(pipe); @@ -7001,8 +7006,19 @@ static void ironlake_write_eld(struct drm_connector *connector, DRM_DEBUG_DRIVER("ELD on pipe %c\n", pipe_name(pipe)); - i = I915_READ(aud_cntl_st); - i = (i >> 29) & DIP_PORT_SEL_MASK; /* DIP_Port_Select, 0x1 = PortB */ + if (IS_VALLEYVIEW(connector->dev)) { + struct intel_encoder *intel_encoder; + struct intel_digital_port *intel_dig_port; + + intel_encoder = intel_attached_encoder(connector); + intel_dig_port = enc_to_dig_port(&intel_encoder->base); + i = intel_dig_port->port; + } else { + i = I915_READ(aud_cntl_st); + i = (i >> 29) & DIP_PORT_SEL_MASK; + /* DIP_Port_Select, 0x1 = PortB */ + } + if (!i) { DRM_DEBUG_DRIVER("Audio directed to unknown port\n"); /* operate blindly on all ports */ @@ -10320,7 +10336,8 @@ static void intel_init_display(struct drm_device *dev) } } else if (IS_G4X(dev)) { dev_priv->display.write_eld = g4x_write_eld; - } + } else if (IS_VALLEYVIEW(dev)) + dev_priv->display.write_eld = ironlake_write_eld; /* Default just returns -ENODEV to indicate unsupported */ dev_priv->display.queue_flip = intel_default_queue_flip;