From nobody Sun Dec 14 21:37:41 2025 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id DB97A215179 for ; Wed, 5 Feb 2025 06:40:34 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1738737634; cv=none; b=Um0pBybvgLfqIh9kAJlHRW5T2z52dSmWnuW3Y1w8jfiIawFUX+h/Mm9Xr5YIpVy5KDbRjjcbm2vOaM87wU1K3dA6tToGBKnmci1MJrA9g5aYvew7rz5zPrELUG+9JhvgcZvTJHi2JQgeqFYoJPMDBBE2I6AX8cosoQOBGRQ/IDc= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1738737634; c=relaxed/simple; bh=gJttkzn6BJFiI8fPoTRjXq8kfmPVSjAVBm2+rRUyFm4=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=Ie3w0AZY5SqhDa76N7Y92UUrAMJYZwYSn25xVEV7iq9DDqF29dhexadPieb7Jumoqk7QllEkTTYMEAPSCZOrOLIIJXGb9SW8h6jML0OHaHQwUY6cMhXChvlvOAqGKWyD0doDTjMDjF6fVS0oiadRMDte8YB09+UJHZiB23r3u4c= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=BPyHYCx3; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="BPyHYCx3" Received: by smtp.kernel.org (Postfix) with ESMTPS id 63ED9C4CEE5; Wed, 5 Feb 2025 06:40:34 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1738737634; bh=gJttkzn6BJFiI8fPoTRjXq8kfmPVSjAVBm2+rRUyFm4=; h=From:Date:Subject:References:In-Reply-To:To:Cc:Reply-To:From; b=BPyHYCx3qkb3x2tF10WH39ZOpMy/A0Lemb6mWDjsJMJBaC7G/cYUQ+DVZztf5Ud6+ 9EKbas4LexMr6B4Zt9DQdPoOZNKew9PYDmphmcAZYu9fxNQhcJXOXYOy8xAd3TsWDt HkGivJxHhdTvhpK9Ym22pixmXCfliz/YJXYFiqy3TFupocBLI9pZStcivzxWIqV7ro FoqYW6vDGm+0LDf5jesfYIFaEMAyql7G5CiQs6TUjEA/AIabrt87ImSj193eXaMk8C uvdUIu4MDAZgUmqTucv4KRJIqONqoTrynjmGPi9KPqGclGJZQEopsUxq4KXrjzl0Sb olrBLwMr6qVNg== Received: from aws-us-west-2-korg-lkml-1.web.codeaurora.org (localhost.localdomain [127.0.0.1]) by smtp.lore.kernel.org (Postfix) with ESMTP id 50FA9C02197; Wed, 5 Feb 2025 06:40:34 +0000 (UTC) From: Hermes Wu via B4 Relay Date: Wed, 05 Feb 2025 14:41:03 +0800 Subject: [PATCH v3 1/2] drm/bridge: it6505: add support DRM_BRIDGE_OP_HDMI to drm_bridge Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20250205-add-audio-codec-v3-1-26cfadb2d81f@ite.com.tw> References: <20250205-add-audio-codec-v3-0-26cfadb2d81f@ite.com.tw> In-Reply-To: <20250205-add-audio-codec-v3-0-26cfadb2d81f@ite.com.tw> To: Andrzej Hajda , Neil Armstrong , Robert Foss , Laurent Pinchart , Jonas Karlman , Jernej Skrabec , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Simona Vetter Cc: treapking@chromium.org, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, Pet.Weng@ite.com.tw, Kenneth.Hung@ite.com.tw, Hermes Wu X-Mailer: b4 0.14.2 X-Developer-Signature: v=1; a=ed25519-sha256; t=1738737663; l=9180; i=Hermes.wu@ite.com.tw; s=20241230; h=from:subject:message-id; bh=HsvghXCd4JyDknaGsA3NNvvjMzjG9ZuIBRMO9tJETkE=; b=igPnzGn4a+UA7iXGKouqKmB8ravxdw3TuH1hogt7Jxy6U28L0PkOsFZYXDvi8f4KMBTm/DoQy 354DC/w75vsArtSHk3Xul6y96cU6enH49Vvs1QuCkNdSPl/Q0/oT+dx X-Developer-Key: i=Hermes.wu@ite.com.tw; a=ed25519; pk=qho5Dawp2WWj9CGyjtJ6/Y10xH8odjRdS6SXDaDAerU= X-Endpoint-Received: by B4 Relay for Hermes.wu@ite.com.tw/20241230 with auth_id=310 X-Original-From: Hermes Wu Reply-To: Hermes.wu@ite.com.tw From: Hermes Wu Add DRM_BRIDGE_OP_HDMI to bridge.ops and implement necessary callback functions. The native AVI and AUDIO infoframe configuration API are removed. In .atomic_enable use drm_atomic_helper_connector_hdmi_update_infoframes(). for infoframe updates. Signed-off-by: Hermes Wu --- drivers/gpu/drm/bridge/ite-it6505.c | 185 ++++++++++++++++++++++----------= ---- 1 file changed, 115 insertions(+), 70 deletions(-) diff --git a/drivers/gpu/drm/bridge/ite-it6505.c b/drivers/gpu/drm/bridge/i= te-it6505.c index 88ef76a37fe6accacdd343839ff2569b31b18ceb..4c766854de14093b80949bdb410= 488f504b24db8 100644 --- a/drivers/gpu/drm/bridge/ite-it6505.c +++ b/drivers/gpu/drm/bridge/ite-it6505.c @@ -25,6 +25,8 @@ =20 #include #include +#include +#include #include #include #include @@ -1414,37 +1416,6 @@ static void it6505_variable_config(struct it6505 *it= 6505) memset(it6505->bksvs, 0, sizeof(it6505->bksvs)); } =20 -static int it6505_send_video_infoframe(struct it6505 *it6505, - struct hdmi_avi_infoframe *frame) -{ - u8 buffer[HDMI_INFOFRAME_HEADER_SIZE + HDMI_AVI_INFOFRAME_SIZE]; - int err; - struct device *dev =3D it6505->dev; - - err =3D hdmi_avi_infoframe_pack(frame, buffer, sizeof(buffer)); - if (err < 0) { - dev_err(dev, "Failed to pack AVI infoframe: %d", err); - return err; - } - - err =3D it6505_set_bits(it6505, REG_INFOFRAME_CTRL, EN_AVI_PKT, 0x00); - if (err) - return err; - - err =3D regmap_bulk_write(it6505->regmap, REG_AVI_INFO_DB1, - buffer + HDMI_INFOFRAME_HEADER_SIZE, - frame->length); - if (err) - return err; - - err =3D it6505_set_bits(it6505, REG_INFOFRAME_CTRL, EN_AVI_PKT, - EN_AVI_PKT); - if (err) - return err; - - return 0; -} - static void it6505_get_extcon_property(struct it6505 *it6505) { int err; @@ -1604,27 +1575,6 @@ static void it6505_enable_audio_source(struct it6505= *it6505) it6505_write(it6505, REG_AUDIO_SRC_CTRL, audio_source_count); } =20 -static void it6505_enable_audio_infoframe(struct it6505 *it6505) -{ - struct device *dev =3D it6505->dev; - u8 audio_info_ca[] =3D { 0x00, 0x00, 0x01, 0x03, 0x07, 0x0B, 0x0F, 0x1F }; - - DRM_DEV_DEBUG_DRIVER(dev, "infoframe channel_allocation:0x%02x", - audio_info_ca[it6505->audio.channel_count - 1]); - - it6505_write(it6505, REG_AUD_INFOFRAM_DB1, it6505->audio.channel_count - - 1); - it6505_write(it6505, REG_AUD_INFOFRAM_DB2, 0x00); - it6505_write(it6505, REG_AUD_INFOFRAM_DB3, - audio_info_ca[it6505->audio.channel_count - 1]); - it6505_write(it6505, REG_AUD_INFOFRAM_DB4, 0x00); - it6505_write(it6505, REG_AUD_INFOFRAM_SUM, 0x00); - - /* Enable Audio InfoFrame */ - it6505_set_bits(it6505, REG_INFOFRAME_CTRL, EN_AUD_CTRL_PKT, - EN_AUD_CTRL_PKT); -} - static void it6505_disable_audio(struct it6505 *it6505) { it6505_set_bits(it6505, REG_DATA_MUTE_CTRL, EN_AUD_MUTE, EN_AUD_MUTE); @@ -1644,7 +1594,6 @@ static void it6505_enable_audio(struct it6505 *it6505) it6505_setup_audio_channel_status(it6505); it6505_setup_audio_format(it6505); it6505_enable_audio_source(it6505); - it6505_enable_audio_infoframe(it6505); =20 it6505_write(it6505, REG_AUDIO_N_0_7, 0x00); it6505_write(it6505, REG_AUDIO_N_8_15, 0x80); @@ -3168,17 +3117,13 @@ it6505_bridge_mode_valid(struct drm_bridge *bridge, const struct drm_display_info *info, const struct drm_display_mode *mode) { - struct it6505 *it6505 =3D bridge_to_it6505(bridge); - - if (mode->flags & DRM_MODE_FLAG_INTERLACE) - return MODE_NO_INTERLACE; - - if (mode->clock > it6505->max_dpi_pixel_clock) - return MODE_CLOCK_HIGH; + unsigned long long rate; =20 - it6505->video_info.clock =3D mode->clock; + rate =3D drm_hdmi_compute_mode_clock(mode, 8, HDMI_COLORSPACE_RGB); + if (rate =3D=3D 0) + return MODE_NOCLOCK; =20 - return MODE_OK; + return bridge->funcs->hdmi_tmds_char_rate_valid(bridge, mode, rate); } =20 static void it6505_bridge_atomic_enable(struct drm_bridge *bridge, @@ -3217,6 +3162,8 @@ static void it6505_bridge_atomic_enable(struct drm_br= idge *bridge, if (WARN_ON(!mode)) return; =20 + drm_atomic_helper_connector_hdmi_update_infoframes(connector, state); + ret =3D drm_hdmi_avi_infoframe_from_display_mode(&frame, connector, mode); @@ -3224,15 +3171,8 @@ static void it6505_bridge_atomic_enable(struct drm_b= ridge *bridge, dev_err(dev, "Failed to setup AVI infoframe: %d", ret); =20 it6505_update_video_parameter(it6505, mode); - - ret =3D it6505_send_video_infoframe(it6505, &frame); - - if (ret) - dev_err(dev, "Failed to send AVI infoframe: %d", ret); - it6505_int_mask_enable(it6505); it6505_video_reset(it6505); - it6505_drm_dp_link_set_power(&it6505->aux, &it6505->link, DP_SET_POWER_D0); } @@ -3302,6 +3242,106 @@ static const struct drm_edid *it6505_bridge_edid_re= ad(struct drm_bridge *bridge, return drm_edid_dup(it6505->cached_edid); } =20 +static void it6505_write_audio_infoframe(struct it6505 *it6505, + const u8 *buffer, size_t len) +{ + it6505_set_bits(it6505, REG_INFOFRAME_CTRL, EN_AUD_PKT, 0); + regmap_bulk_write(it6505->regmap, REG_AUD_INFOFRAM_DB1, + buffer + HDMI_INFOFRAME_HEADER_SIZE, + 4); + it6505_write(it6505, REG_AUD_INFOFRAM_SUM, buffer[3]); + it6505_set_bits(it6505, REG_INFOFRAME_CTRL, EN_AUD_PKT, + EN_AUD_PKT); +} + +static int it6505_write_avi_infoframe(struct it6505 *it6505, + const u8 *buffer, size_t len) +{ + struct device *dev =3D it6505->dev; + + if (len - HDMI_INFOFRAME_HEADER_SIZE > 13) { + DRM_DEV_DEBUG_DRIVER(dev, "AVI size fail %d", len); + return -EINVAL; + } + + it6505_set_bits(it6505, REG_INFOFRAME_CTRL, EN_AVI_PKT, 0); + regmap_bulk_write(it6505->regmap, REG_AVI_INFO_DB1, + buffer + HDMI_INFOFRAME_HEADER_SIZE, + len - HDMI_INFOFRAME_HEADER_SIZE); + + it6505_write(it6505, REG_AVI_INFO_SUM, buffer[3]); + it6505_set_bits(it6505, REG_INFOFRAME_CTRL, EN_AVI_PKT, + EN_AVI_PKT); + + return 0; +} + +static int it6505_bridge_hdmi_write_infoframe(struct drm_bridge *bridge, + enum hdmi_infoframe_type type, + const u8 *buffer, size_t len) +{ + struct it6505 *it6505 =3D bridge_to_it6505(bridge); + struct device *dev =3D it6505->dev; + + /*HW supports AVI and AUDIO infoframe only*/ + switch (type) { + case HDMI_INFOFRAME_TYPE_AUDIO: + it6505_write_audio_infoframe(it6505, buffer, len); + break; + + case HDMI_INFOFRAME_TYPE_AVI: + it6505_write_avi_infoframe(it6505, buffer, len); + break; + default: + dev_dbg(dev, "unsupported HDMI infoframe 0x%x\n", type); + break; + } + + return 0; +} + +static int it6505_bridge_hdmi_clear_infoframe(struct drm_bridge *bridge, + enum hdmi_infoframe_type type) +{ + struct it6505 *it6505 =3D bridge_to_it6505(bridge); + struct device *dev =3D it6505->dev; + + /*HW supports AVI and AUDIO infoframe only*/ + switch (type) { + case HDMI_INFOFRAME_TYPE_AUDIO: + it6505_set_bits(it6505, REG_INFOFRAME_CTRL, EN_AUD_PKT, 0); + break; + + case HDMI_INFOFRAME_TYPE_AVI: + it6505_set_bits(it6505, REG_INFOFRAME_CTRL, EN_AVI_PKT, 0); + break; + default: + dev_dbg(dev, "unsupported HDMI infoframe 0x%x\n", type); + break; + } + + return 0; +} + +static enum drm_mode_status +it6505_bridge_hdmi_tmds_char_rate_valid(const struct drm_bridge *bridge, + const struct drm_display_mode *mode, + unsigned long long tmds_rate) +{ + const struct it6505 *it6505 =3D container_of(bridge, + const struct it6505, + bridge); + + if (mode->flags & DRM_MODE_FLAG_INTERLACE) + return MODE_NO_INTERLACE; + + if (mode->clock > it6505->max_dpi_pixel_clock || + tmds_rate > 148500000) + return MODE_CLOCK_HIGH; + + return MODE_OK; +} + static const struct drm_bridge_funcs it6505_bridge_funcs =3D { .atomic_duplicate_state =3D drm_atomic_helper_bridge_duplicate_state, .atomic_destroy_state =3D drm_atomic_helper_bridge_destroy_state, @@ -3315,6 +3355,9 @@ static const struct drm_bridge_funcs it6505_bridge_fu= ncs =3D { .atomic_post_disable =3D it6505_bridge_atomic_post_disable, .detect =3D it6505_bridge_detect, .edid_read =3D it6505_bridge_edid_read, + .hdmi_clear_infoframe =3D it6505_bridge_hdmi_clear_infoframe, + .hdmi_write_infoframe =3D it6505_bridge_hdmi_write_infoframe, + .hdmi_tmds_char_rate_valid =3D it6505_bridge_hdmi_tmds_char_rate_valid, }; =20 static __maybe_unused int it6505_bridge_resume(struct device *dev) @@ -3700,7 +3743,9 @@ static int it6505_i2c_probe(struct i2c_client *client) it6505->bridge.funcs =3D &it6505_bridge_funcs; it6505->bridge.type =3D DRM_MODE_CONNECTOR_DisplayPort; it6505->bridge.ops =3D DRM_BRIDGE_OP_DETECT | DRM_BRIDGE_OP_EDID | - DRM_BRIDGE_OP_HPD; + DRM_BRIDGE_OP_HPD | DRM_BRIDGE_OP_HDMI; + it6505->bridge.vendor =3D "iTE"; + it6505->bridge.product =3D "IT6505"; drm_bridge_add(&it6505->bridge); =20 return 0; --=20 2.34.1 From nobody Sun Dec 14 21:37:41 2025 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id DB9E0215F7A for ; Wed, 5 Feb 2025 06:40:34 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1738737634; cv=none; b=ZkHiKSYnKOrHEyhdHn4fsmqw1r726CnXp5gT52t2GbQsWCM5uotBpcQIBIo/fPNa0yvzXgf2F441VqaYQexBDNj5A++4PoJL4pG1hAXD0PozH83XZBOwhzjJ8gyrn/7dKX3HN+ccAeZd6e3LHHlMhbkvv+UNh8TfmsHdKnu2uQ0= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1738737634; c=relaxed/simple; bh=h4uoZiDaDCnjp4xbn++cw3iPpGRIxoOC4t2uGuV8Lr8=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=gmFeoGj9xCT1WEaUgkwHBz7RVhTpPpeg3dwH9mo78IzCgelHQkPZTPc2Hq1F2xDgEhqq+7GQj5IeQq5Az/cTKEMXmLTI4TINNLi5d7GsburDmH27P2It0ybXZDABzsI9oP/PMUhdH/TDc1EMfYxwbE4M69l7mEq7scLjoafaJME= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=TIQSSz97; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="TIQSSz97" Received: by smtp.kernel.org (Postfix) with ESMTPS id 7467CC4CEE6; Wed, 5 Feb 2025 06:40:34 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1738737634; bh=h4uoZiDaDCnjp4xbn++cw3iPpGRIxoOC4t2uGuV8Lr8=; h=From:Date:Subject:References:In-Reply-To:To:Cc:Reply-To:From; b=TIQSSz97U8yWHdkz4Ovgfc11YSHdnxE+u8jlcx/5Bygf3DqvyTsI/dT1k1OP+tvZv 7ybCmiFfkz/xgIEQu+YQY4dsSY8FutA9ExDuJcLtXRbO31gAJr1fZxHqIr+Cdb3x2T slPZ70SI06vxCP6nCqSAB/b+bWsuy0cmyNzW7pEjKwixws2U7njQ5I3urkmUj56bfS IvXUXIEYDNvO9e4Hp8IIz4jACe9Eis1m0zEqh4EFJeSKlgPWZ7AEpYz2ZdJi7O8208 W0pE9d8RUIjGTpgN7lQSvcLe+yJ+VpOLxDWSO9TDXE4Vhjmuj7ajcBqrl+R9sz/kHi 6hXMZ4EtmLOng== Received: from aws-us-west-2-korg-lkml-1.web.codeaurora.org (localhost.localdomain [127.0.0.1]) by smtp.lore.kernel.org (Postfix) with ESMTP id 61866C02199; Wed, 5 Feb 2025 06:40:34 +0000 (UTC) From: Hermes Wu via B4 Relay Date: Wed, 05 Feb 2025 14:41:04 +0800 Subject: [PATCH v3 2/2] drm/bridge: it6505: add audio support to drm_bridge_funcs Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20250205-add-audio-codec-v3-2-26cfadb2d81f@ite.com.tw> References: <20250205-add-audio-codec-v3-0-26cfadb2d81f@ite.com.tw> In-Reply-To: <20250205-add-audio-codec-v3-0-26cfadb2d81f@ite.com.tw> To: Andrzej Hajda , Neil Armstrong , Robert Foss , Laurent Pinchart , Jonas Karlman , Jernej Skrabec , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Simona Vetter Cc: treapking@chromium.org, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, Pet.Weng@ite.com.tw, Kenneth.Hung@ite.com.tw, Hermes Wu X-Mailer: b4 0.14.2 X-Developer-Signature: v=1; a=ed25519-sha256; t=1738737663; l=2998; i=Hermes.wu@ite.com.tw; s=20241230; h=from:subject:message-id; bh=ALqN5ab0u4x0omcwFuCJ35/u/+RI1mf1aEJhjK4tFQs=; b=oSN8pAg0gEhe90kCJmKSJhBKKMKU0FbNorw4tsyh+YfjvjrGj/tgUFTW92DavYy8DbTeqw1+8 VkjM8QF1HRtBtw4mXS+aLQPsRrC5C4F+7z2KDJw6LVunN7+TDl/C7SW X-Developer-Key: i=Hermes.wu@ite.com.tw; a=ed25519; pk=qho5Dawp2WWj9CGyjtJ6/Y10xH8odjRdS6SXDaDAerU= X-Endpoint-Received: by B4 Relay for Hermes.wu@ite.com.tw/20241230 with auth_id=310 X-Original-From: Hermes Wu Reply-To: Hermes.wu@ite.com.tw From: Hermes Wu For supporting audio form I2S to DP audio data sub stream. Add hdmi_audio callbacks to drm_bridge_funcs for the HDMI codec framework. Signed-off-by: Hermes Wu --- drivers/gpu/drm/bridge/ite-it6505.c | 46 +++++++++++++++++++++++++++++++++= ++++ 1 file changed, 46 insertions(+) diff --git a/drivers/gpu/drm/bridge/ite-it6505.c b/drivers/gpu/drm/bridge/i= te-it6505.c index 4c766854de14093b80949bdb410488f504b24db8..832c4e3ff4ada31367788faed95= 83da676e36cf8 100644 --- a/drivers/gpu/drm/bridge/ite-it6505.c +++ b/drivers/gpu/drm/bridge/ite-it6505.c @@ -3342,6 +3342,46 @@ it6505_bridge_hdmi_tmds_char_rate_valid(const struct= drm_bridge *bridge, return MODE_OK; } =20 +static int it6505_bridge_hdmi_audio_startup(struct drm_connector *connecto= r, + struct drm_bridge *bridge) +{ + struct it6505 *it6505 =3D bridge_to_it6505(bridge); + struct device *dev =3D it6505->dev; + + if (!it6505->powered || it6505->enable_drv_hold) + return -EIO; + + DRM_DEV_DEBUG_DRIVER(dev, "Audio enable"); + it6505_enable_audio(it6505); + + return 0; +} + +static int it6505_bridge_hdmi_audio_prepare(struct drm_connector *connecto= r, + struct drm_bridge *bridge, + struct hdmi_codec_daifmt *fmt, + struct hdmi_codec_params *hparms) +{ + struct it6505 *it6505 =3D bridge_to_it6505(bridge); + int err; + + err =3D it6505_audio_setup_hw_params(it6505, hparms); + if (err < 0) + return err; + + return drm_atomic_helper_connector_hdmi_update_audio_infoframe(connector, + &hparms->cea); +} + +static void it6505_bridge_hdmi_audio_shutdown(struct drm_connector *connec= tor, + struct drm_bridge *bridge) +{ + struct it6505 *it6505 =3D bridge_to_it6505(bridge); + + if (it6505->powered && !it6505->enable_drv_hold) + it6505_disable_audio(it6505); +} + static const struct drm_bridge_funcs it6505_bridge_funcs =3D { .atomic_duplicate_state =3D drm_atomic_helper_bridge_duplicate_state, .atomic_destroy_state =3D drm_atomic_helper_bridge_destroy_state, @@ -3358,6 +3398,9 @@ static const struct drm_bridge_funcs it6505_bridge_fu= ncs =3D { .hdmi_clear_infoframe =3D it6505_bridge_hdmi_clear_infoframe, .hdmi_write_infoframe =3D it6505_bridge_hdmi_write_infoframe, .hdmi_tmds_char_rate_valid =3D it6505_bridge_hdmi_tmds_char_rate_valid, + .hdmi_audio_startup =3D it6505_bridge_hdmi_audio_startup, + .hdmi_audio_prepare =3D it6505_bridge_hdmi_audio_prepare, + .hdmi_audio_shutdown =3D it6505_bridge_hdmi_audio_shutdown, }; =20 static __maybe_unused int it6505_bridge_resume(struct device *dev) @@ -3746,6 +3789,9 @@ static int it6505_i2c_probe(struct i2c_client *client) DRM_BRIDGE_OP_HPD | DRM_BRIDGE_OP_HDMI; it6505->bridge.vendor =3D "iTE"; it6505->bridge.product =3D "IT6505"; + it6505->bridge.hdmi_audio_dev =3D dev; + it6505->bridge.hdmi_audio_max_i2s_playback_channels =3D 2; + it6505->bridge.hdmi_audio_dai_port =3D 1; drm_bridge_add(&it6505->bridge); =20 return 0; --=20 2.34.1