From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from pidgin.makrotopia.org (pidgin.makrotopia.org [185.142.180.65]) (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 6BD324B488E for ; Thu, 24 Sep 2026 18:42:44 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=185.142.180.65 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1790275366; cv=none; b=ZxY18sAqxZ3475pixgdKMLSNXymjY5DvIYhjVbTjBfJJsf+p8TR0VIVL2dIuDkrLdmr1yaiupWbgU6fprS23Cl6Vbw3Pbs0H7Ayg5Wi+rj2TCH71VrbjS/dKolCuppcJM/leqH/GWDMFNxZp/XcNri7rlGxDFJHYGynQDvDoLok= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1790275366; c=relaxed/simple; bh=doBQdgJUeFvEO4yzuA+5yfElbS/4J1IfbSI34d6AdSc=; h=Date:From:To:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=txAGEqJeHsD7ZWDeS0U539IOpvU0bwILK1hPVWNe6jTJYgXNO6mfEfvqWgqsePOCaUQ/ZzHitJhlBBy5O/iNg046PXCWVDb3LKn8Unr5B+EFLBGxRYJ4DkFNY0xu3Kx/6ct4otQFYqKq2ltNvmJGf1fkdoFxNBleDhT3VJtIhV4= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=makrotopia.org; spf=pass smtp.mailfrom=makrotopia.org; arc=none smtp.client-ip=185.142.180.65 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=makrotopia.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=makrotopia.org Received: from local by pidgin.makrotopia.org with esmtpsa (TLS1.3:TLS_AES_256_GCM_SHA384:256:X25519MLKEM768) (Exim 4.100) (envelope-from ) id 1x9oP0-000000007SV-3yLQ; Thu, 24 Sep 2026 18:42:39 +0000 Date: Thu, 24 Sep 2026 19:42:35 +0100 From: Daniel Golle To: Andrzej Hajda , Neil Armstrong , Robert Foss , Laurent Pinchart , Jonas Karlman , Jernej Skrabec , Luca Ceresoli , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Simona Vetter , Matthias Brugger , AngeloGioacchino Del Regno , Allen Chen , Hermes Wu , Pin-yen Lin , dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org Subject: [PATCH v7 14/14] drm/bridge: it6505: Don't reject audio hw_params without an encoder Message-ID: <2a125d71fcbae7dd393a3ea21f452caa63c2cfda.1790275151.git.daniel@makrotopia.org> References: 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=us-ascii Content-Disposition: inline In-Reply-To: it6505_audio_setup_hw_params() returns -ENODEV when no encoder is attached. With it6505 registering an hdmi-audio-codec this runs for every stream on the I2S DAI, and on mt8186-corsola, where the rt1019 speaker amplifier shares I2S3, the error tears down the whole DPCM backend and breaks speaker playback whenever no display is attached. The function only caches stream parameters, which needs no encoder. Drop the check, and apply the audio defaults once at probe instead of in it6505_variable_config(), which would clobber the cached parameters again on every hotplug. Actual audio output remains gated by it6505->powered. Fixes: b5c84a9edcd4 ("drm/bridge: add it6505 driver") Signed-off-by: Daniel Golle --- v7: no changes v6: no changes v5: no changes v4: apply audio defaults at probe time instead of in it6505_variable_config() v3: no changes v2: drop the encoder check entirely instead of returning 0 early --- drivers/gpu/drm/bridge/ite-it6505.c | 23 ++++++++++------------- 1 file changed, 10 insertions(+), 13 deletions(-) diff --git a/drivers/gpu/drm/bridge/ite-it6505.c b/drivers/gpu/drm/bridge/ite-it6505.c index 00eaac183130..079129925f8a 100644 --- a/drivers/gpu/drm/bridge/ite-it6505.c +++ b/drivers/gpu/drm/bridge/ite-it6505.c @@ -1371,16 +1371,6 @@ static void it6505_variable_config(struct it6505 *it6505) it6505->link_state = LINK_IDLE; it6505->hdcp_desired = HDCP_DESIRED; it6505->auto_train_retry = AUTO_TRAIN_RETRY; - it6505->audio.select = AUDIO_SELECT; - it6505->audio.sample_rate = AUDIO_SAMPLE_RATE; - it6505->audio.channel_count = AUDIO_CHANNEL_COUNT; - it6505->audio.type = AUDIO_TYPE; - it6505->audio.i2s_input_format = I2S_INPUT_FORMAT; - it6505->audio.i2s_justified = I2S_JUSTIFIED; - it6505->audio.i2s_data_delay = I2S_DATA_DELAY; - it6505->audio.i2s_ws_channel = I2S_WS_CHANNEL; - it6505->audio.i2s_data_sequence = I2S_DATA_SEQUENCE; - it6505->audio.word_length = AUDIO_WORD_LENGTH; memset(it6505->sha1_input, 0, sizeof(it6505->sha1_input)); memset(it6505->bksvs, 0, sizeof(it6505->bksvs)); } @@ -3025,9 +3015,6 @@ static int it6505_audio_setup_hw_params(struct it6505 *it6505, params->sample_rate, params->sample_width, params->cea.channels); - if (!it6505->bridge.encoder) - return -ENODEV; - if (params->cea.channels <= 1 || params->cea.channels > 8) { DRM_DEV_DEBUG_DRIVER(dev, "channel number: %d not support", params->cea.channels); @@ -3159,6 +3146,16 @@ static int it6505_register_audio_driver(struct device *dev) }; struct platform_device *pdev; + it6505->audio.select = AUDIO_SELECT; + it6505->audio.sample_rate = AUDIO_SAMPLE_RATE; + it6505->audio.channel_count = AUDIO_CHANNEL_COUNT; + it6505->audio.type = AUDIO_TYPE; + it6505->audio.i2s_input_format = I2S_INPUT_FORMAT; + it6505->audio.i2s_justified = I2S_JUSTIFIED; + it6505->audio.i2s_data_delay = I2S_DATA_DELAY; + it6505->audio.i2s_ws_channel = I2S_WS_CHANNEL; + it6505->audio.i2s_data_sequence = I2S_DATA_SEQUENCE; + it6505->audio.word_length = AUDIO_WORD_LENGTH; it6505->audio.mute = true; INIT_DELAYED_WORK(&it6505->delayed_audio, it6505_delayed_audio); -- 2.55.0