From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mx0b-0031df01.pphosted.com (mx0b-0031df01.pphosted.com [205.220.180.131]) (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 294133191D9 for ; Fri, 19 Dec 2025 12:59:55 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=205.220.180.131 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1766149199; cv=none; b=qwWe+rJ+oqJgtDMxBaqVvWrt63X1kSXmYcdmd0Z9RaZHNt2c7m191peuMYVX4+b5a0R7ZnkYd6uOwMCRszqHmUxrDMK9BYsdV9YLYheyZP3v6U4lbNBOSDWH9Y5SJKdMNFF0dRv6PYuRkSuBDRK/lZnA3P/lxC62V7XMEyzbzWQ= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1766149199; c=relaxed/simple; bh=EOL6PQ/u2G2IGnbiEjczbalUkxet7R3Cs0ByyQPY9VI=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=ZcE5rVmh99g8oUqvMw4fUlUJ6ffAlEEoL5wMDqF8OFMLfvdAP9/Ibf+dQ1cy6sfkuHdcfX+fG0rDLuZYcMwFksCsmS+i7BYM2863HW4c06mPhiPIFJE5riBYXDmXLRmKAI//TUDyK8DEkGHSFVfOZzoQCCKXQ2d0GXX2geNKWiM= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=oss.qualcomm.com; spf=pass smtp.mailfrom=oss.qualcomm.com; dkim=pass (2048-bit key) header.d=qualcomm.com header.i=@qualcomm.com header.b=XGnJGFFI; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b=QQIdgql0; arc=none smtp.client-ip=205.220.180.131 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=oss.qualcomm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=oss.qualcomm.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=qualcomm.com header.i=@qualcomm.com header.b="XGnJGFFI"; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b="QQIdgql0" Received: from pps.filterd (m0279872.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.11/8.18.1.11) with ESMTP id 5BJBBdbI3976261 for ; Fri, 19 Dec 2025 12:59:54 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=qualcomm.com; h= cc:content-transfer-encoding:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to; s=qcppdkim1; bh= uDatXU9sVuQNjMrgEpskfEj2cmOKOLi/Cc2zxJakF40=; b=XGnJGFFI6I3dFl9d /9drXzoH9V3S1c6bE63XRIYZ5N+V0VFjmEmEtNw4RvBGT2/PCPRiFr7e+xPpqjyj 3nVobtuOcct9JEuyWu9ljw6Y76fvB3yECtocRJfd0jQPdiMDzZPO3HkjpRji6iuA Ms9qza35JYlfxpnXo90yErUUtpq7ari+Az8H/WsTIy8eLZFMfyJ535kvIfM/Yv3A f8Vw/RrC7Z7pBqE56UXvVj2XBiPlWQ4HPAAhDLPamTCOPwYhpakBAfdvPzMXq4Jn G8odNBlIRKMKfsVdwttWJTubR504OL5Z97cAxGMFUOAaHI5aq8WDbxKnOp54cKJr Wh70Ow== Received: from mail-qv1-f69.google.com (mail-qv1-f69.google.com [209.85.219.69]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 4b4r2fjmf0-1 (version=TLSv1.3 cipher=TLS_AES_128_GCM_SHA256 bits=128 verify=NOT) for ; Fri, 19 Dec 2025 12:59:54 +0000 (GMT) Received: by mail-qv1-f69.google.com with SMTP id 6a1803df08f44-88880bc706cso3796136d6.2 for ; Fri, 19 Dec 2025 04:59:54 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oss.qualcomm.com; s=google; t=1766149194; x=1766753994; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=uDatXU9sVuQNjMrgEpskfEj2cmOKOLi/Cc2zxJakF40=; b=QQIdgql0EwiRjopReM5G/GFp/2IZFtHx62ki0QVcBOnOA9eJborzIuFze2I2YqfHkr 37/WVuYYYTIMxB0+tpjeUJ+PQ1iSqXgz2D6lb1OnF9hClG2WKh7ZGt5DmD8PIrfkOlPk VvPRD6KxIM72FjA2JtVXl1/P0mNQVHlK9KrQMG6EDwEce6np/vTYjGo8KcaET5HcqlS9 4lUfnZ4i5IBkTQCOTGorJaW7QcltMX/W9ab87EnoQeyUGlamnee3DxrahMEmJvdwBQLi Ar513PaOKr6PIL8KpI1PROZ6rwqdgiMOnN4+rXWrSC253Ft0zOo7sJU34UIfkUuxf94a XAXA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1766149194; x=1766753994; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-gg:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=uDatXU9sVuQNjMrgEpskfEj2cmOKOLi/Cc2zxJakF40=; b=eQffSyqWIXe2Wr2at1uR3WvkFokRRjIcSTIKBhZmdaftGKkMLQqmR7GjSkMM9hwU+s GnUaBUw2slJN3G3XirzAKllpN183/3HKGY3Ahl/m6Oukx3UXZAqahnmy/HiLr2fOv7Rx wxmy9O8UvGDu+ljkun7qAihjQ7JwlTKOugYFplVA6XmBMw2UiybANp5uNc3g9a7+6BOQ ZssAANl0tpOj9HTUDA2xJySPzEe4BDnRlcDydW8go2zHy2doyzPnEFx8Se8CbIcm8PGy Boq9ME75H/5Q+DljskXRuZrK85jt2mvPJ11zYoJ3MolCbQeNvTI8QBvty84vnPubBGcU e6KQ== X-Forwarded-Encrypted: i=1; AJvYcCWjoaVDau6akqeVwHfVPuyOY6WLP8ByiLfEoZW7x2/ssFyYXjPowojo9fwZ9OCt2OCzoktoh2QmFDjfEjo=@vger.kernel.org X-Gm-Message-State: AOJu0YyUZKvwegNYs+AiXCmjlNf38Xt/s34N6ezpM+z5An/XmCtlKmYq MLsDNSRfoN0MV95B2+Ls4s7Xm9gmu1MCrzDOENLnP0Jpw9/mndtl45ppmeOo9rBDKyyZNoRMo15 v5aZEGbyWtBxHfB3Sw68E+ZOmQlIvfhbdDipra3Pt+lOCfyFcQXmSdaRBJI3PJkoYj9E= X-Gm-Gg: AY/fxX7crsh2iYlCS9pIwyaXEIY2s4k4rZIBBtDhWG+UmWCQIYawtD8yXuPtPgitFIz bihqGH81JUHGq+ekjHpDH8GWQV5VHeE8gEkKepVMWVmLjFMLQ6dyhmLqg71ntq0vryHSwZcIx7G fX43WdqWyqtFqxeFuCLuc4tEyC4OHgF7sYJKswFObCAvsyq+r/YxZUYFv1TVYss205UV91hdV01 iJnAAjOd1OdqWsBPWP76RI7a/01PzEqR62uPD+CxWUZLK66Y0UPPyJoVQ+Qq+f2tuYwVAKEF8+m pD3nLCKXiH3IRWRqxoVC180NxVnNcd3IITf4T6oB0A6Kp5Qus5fOUCYXdXHyDIRadxgyyymTdYf 1ubsOxwr3GGuinwnlRNI3gCl5eHOubwM5zAIRLiBrrvASPdD3WRmMo2AaXKAuqNCXbA== X-Received: by 2002:ac8:5a81:0:b0:4f0:299b:c785 with SMTP id d75a77b69052e-4f4abe04451mr25245711cf.11.1766149193650; Fri, 19 Dec 2025 04:59:53 -0800 (PST) X-Google-Smtp-Source: AGHT+IGjcthfl3sxtdcKpBgFJkGHhbCyHkIuhfXhbLOo69LJeriInHWrUVeKj3LisrSYtdhhwb9hUw== X-Received: by 2002:ac8:5a81:0:b0:4f0:299b:c785 with SMTP id d75a77b69052e-4f4abe04451mr25245511cf.11.1766149193156; Fri, 19 Dec 2025 04:59:53 -0800 (PST) Received: from [192.168.119.72] (078088045245.garwolin.vectranet.pl. [78.88.45.245]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-b8037ae9556sm232741466b.28.2025.12.19.04.59.50 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 19 Dec 2025 04:59:52 -0800 (PST) Message-ID: Date: Fri, 19 Dec 2025 13:59:49 +0100 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v2 2/2] arm64: dts: qcom: x1e78100-t14s: Add audio playback over DisplayPort To: Krzysztof Kozlowski , Krzysztof Kozlowski , Bjorn Andersson , Konrad Dybcio , Rob Herring , Krzysztof Kozlowski , Conor Dooley , linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Cc: Abel Vesa References: <20251217120051.98198-3-krzysztof.kozlowski@oss.qualcomm.com> <20251217120051.98198-4-krzysztof.kozlowski@oss.qualcomm.com> <2d4953c6-184d-423b-80e9-871c6e00da35@oss.qualcomm.com> <564732e7-2c86-417d-8568-69f40ea7d4da@kernel.org> <47758ce9-5ec6-4ed6-9f84-13cbdd444d75@oss.qualcomm.com> <487ff592-e67c-41c8-a398-f79aa6e5f69c@kernel.org> <455edd55-132d-423d-b0c0-afc7fa40aa30@oss.qualcomm.com> <267dad52-0f22-4e5b-a929-a77cdc062490@kernel.org> Content-Language: en-US From: Konrad Dybcio In-Reply-To: <267dad52-0f22-4e5b-a929-a77cdc062490@kernel.org> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-Authority-Analysis: v=2.4 cv=NODYOk6g c=1 sm=1 tr=0 ts=69454c4a cx=c_pps a=wEM5vcRIz55oU/E2lInRtA==:117 a=FpWmc02/iXfjRdCD7H54yg==:17 a=IkcTkHD0fZMA:10 a=wP3pNCr1ah4A:10 a=s4-Qcg_JpJYA:10 a=VkNPw1HP01LnGYTKEx00:22 a=KKAkSRfTAAAA:8 a=EUspDBNiAAAA:8 a=dyGxTdY6081K_zEpinIA:9 a=QEXdDO2ut3YA:10 a=OIgjcC2v60KrkQgK7BGD:22 a=cvBusfyB2V15izCimMoJ:22 X-Proofpoint-ORIG-GUID: CPbU1iKTiBnSyLnHKbI-JQeGsMG0ka5l X-Proofpoint-GUID: CPbU1iKTiBnSyLnHKbI-JQeGsMG0ka5l X-Proofpoint-Spam-Details-Enc: AW1haW4tMjUxMjE5MDEwNiBTYWx0ZWRfX515+uINmxt+D 2YnJuSgxg6I79RDz+QYKY3KyyHH9s4HF3JYKRPq+IMN+xe7UafDJSw2C8WJ5X9j//NIWXk/Kw3a vFIUqR+bPN0K9/n1bWlNojxlNfo19Gy1s1MgDTEDjP6iFmo2FCCx2rizqUVVOKJuV0ViE5Xv8nJ u+qx1Ww4IejYSNPLGAjwkvGsiF+toIhbMs6fbvpWZ3dx4U/lhk+MzDQGeR4AIQBJDjpBNvGXxyv eFWVDX+uwQ0X8Fs3pVvQByaIRdv4uce7oH96UOk/1bbGXcwgnlGByyOlOH3o4jbgEOtWhbEC790 rqrHsz2MgjhbQD5tY16YK8thAil5Rz/rkm+CSp1HfwU/9/21rsU9FDK+nsZTxS4lPIvUQgXtWPV YcXXRwRcvhScXDomIiKLILxlIkxD+jPdCJ0+G3zdGB7ltg6CYxiF4/kO013qcqWx9lQISnsb2Z5 +VrNqzS716p/LnDajYg== X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1121,Hydra:6.1.9,FMLib:17.12.100.49 definitions=2025-12-19_04,2025-12-17_02,2025-10-01_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 clxscore=1015 phishscore=0 suspectscore=0 impostorscore=0 bulkscore=0 adultscore=0 malwarescore=0 priorityscore=1501 lowpriorityscore=0 spamscore=0 classifier=typeunknown authscore=0 authtc= authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.22.0-2512120000 definitions=main-2512190106 On 12/19/25 1:57 PM, Krzysztof Kozlowski wrote: > On 19/12/2025 13:53, Konrad Dybcio wrote: >> On 12/17/25 2:00 PM, Krzysztof Kozlowski wrote: >>> On 17/12/2025 13:51, Konrad Dybcio wrote: >>>> On 12/17/25 1:38 PM, Krzysztof Kozlowski wrote: >>>>> On 17/12/2025 13:33, Konrad Dybcio wrote: >>>>>> On 12/17/25 1:00 PM, Krzysztof Kozlowski wrote: >>>>>>> Add necessary DAI links and DAI name prefixes to enable audio playback >>>>>>> over USB/DisplayPort and HDMI. The HDMI port is not yet enabled, but it >>>>>>> should carry respective DAI name prefix regardless. >>>>>>> >>>>>>> Reviewed-by: Abel Vesa >>>>>>> Signed-off-by: Krzysztof Kozlowski >>>>>>> >>>>>>> --- >>>>>> >>>>>> (something's inserting a \n before --- in your latest patches but I >>>>> >>>>> No, it is not. It was always like that and only recently git started >>>>> dropping it, but it is irrelevant. >>>>> >>>>>> don't know if this is a problem) >>>>>> >>>>>> [...] >>>>>> >>>>>>> + displayport-2-dai-link { >>>>>>> + link-name = "DisplayPort2 Playback"; >>>>>>> + >>>>>>> + codec { >>>>>>> + sound-dai = <&mdss_dp2>; >>>>>> >>>>>> How does this work out with fw_devlink? >>>>> >>>>> How is this related to this patchset? Please stop nit-picking irrelevant >>>>> things. >>>> >>>> I'm asking whether this is going to break sync_state because you're not >>>> enabling mdss_dp2 - I believe that's a fair question.. >>> >>> DTS description is independent of driver, thus this is correct code >>> regardless whether there is interconnect involved anywhere or not. I >>> don't have the answer how this affects interconnect, but I see no reason >>> anyone would create such ICC path - between hardware and fake SW construct. >> >> sync_state isn't exclusive to the interconnect framework >> >> In any case, could you please confirm/deny that sound still works as >> expected with linux-next/master + this patch alone and we move on? > > It was working all the time - I was using it on my machine for 3 months > already. Other people are also using it, because it is part of our > laptop tree and no regressions or issues were reported. Thanks, all of my previous comments stemmed from the worries of side-effects of referencing a status=disabled node Reviewed-by: Konrad Dybcio Konrad