Hi,
On 8/30/2016 5:11 AM, John Stultz wrote:
From: Andy Green <andy.gr...@linaro.org>
Set the initial audio packet settings to allow the audio
driver to work.
Cc: David Airlie <airl...@linux.ie>
Cc: Archit Taneja <arch...@codeaurora.org>
Cc: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
Cc: Wolfram Sang <wsa+rene...@sang-engineering.com>
Cc: Srinivas Kandagatla <srinivas.kandaga...@linaro.org>
Cc: "Ville Syrjälä" <ville.syrj...@linux.intel.com>
Cc: Boris Brezillon <boris.brezil...@free-electrons.com>
Cc: Andy Green <a...@warmcat.com>
Cc: Dave Long <dave.l...@linaro.org>
Cc: Guodong Xu <guodong...@linaro.org>
Cc: Zhangfei Gao <zhangfei....@linaro.org>
Cc: Mark Brown <broo...@kernel.org>
Cc: Lars-Peter Clausen <l...@metafoo.de>
Cc: Jose Abreu <joab...@synopsys.com>
Cc: dri-de...@lists.freedesktop.org
Signed-off-by: Andy Green <andy.gr...@linaro.org>
[jstultz: Forward ported to mainline, changed to use register
names rather then hex values, and removed config values set by
audio driver.]
Signed-off-by: John Stultz <john.stu...@linaro.org>
---
drivers/gpu/drm/bridge/adv7511/adv7533.c | 22 ++++++++++++++++++++++
1 file changed, 22 insertions(+)
diff --git a/drivers/gpu/drm/bridge/adv7511/adv7533.c
b/drivers/gpu/drm/bridge/adv7511/adv7533.c
index 6798ecf..cced7c9 100644
--- a/drivers/gpu/drm/bridge/adv7511/adv7533.c
+++ b/drivers/gpu/drm/bridge/adv7511/adv7533.c
@@ -104,6 +104,28 @@ void adv7533_dsi_power_on(struct adv7511 *adv)
/* disable test mode */
regmap_write(adv->regmap_cec, 0x55, 0x00);
+ /* hide Audio infoframe updates */
+ regmap_update_bits(adv->regmap, ADV7511_REG_INFOFRAME_UPDATE,
+ BIT(5), BIT(5));
+ /* enable N/CTS, enable Audio sample packets */
+ regmap_update_bits(adv->regmap, ADV7511_REG_PACKET_ENABLE1,
+ BIT(5), BIT(5));
+ /* enable N/CTS */
+ regmap_update_bits(adv->regmap, ADV7511_REG_PACKET_ENABLE1,
+ BIT(6), BIT(6));
+ /* not copyrighted */
+ regmap_update_bits(adv->regmap, ADV7511_REG_AUDIO_CFG1,
+ BIT(5), BIT(5));
+ /* enable audio infoframes */
+ regmap_update_bits(adv->regmap, ADV7511_REG_PACKET_ENABLE1,
+ BIT(3), BIT(3));
+ /* AV mute disable */
+ regmap_update_bits(adv->regmap, ADV7511_REG_GC(0),
+ BIT(7) | BIT(6), BIT(7));
+ /* use Audio infoframe updated info */
+ regmap_update_bits(adv->regmap, ADV7511_REG_GC(1),
+ BIT(5), 0);
+
Wouldn't these writes be needed by ADV751x chips too? These seem to
belong to the main ADV75xx regmap. These should probably be a separate
func in adv7511_audio.c or adv7511_drv.c, and it should be called in
adv7511_power_on, so that ADV751x chips can utilize this too.
Thanks,
Archit
regmap_register_patch(adv->regmap_cec, adv7533_cec_fixed_registers,
ARRAY_SIZE(adv7533_cec_fixed_registers));
}
--
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project