From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mx0a-0031df01.pphosted.com (mx0a-0031df01.pphosted.com [205.220.168.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 50FDB313293 for ; Wed, 10 Sep 2025 10:20:47 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=205.220.168.131 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1757499648; cv=none; b=qNAWZxRGqDAEfkijUzluyMa6S8FCBPeDiABiNTtbbYgYaUP6MASfgmbp2rnIF0cYrvJp4uXZrwwWLStKkUOmAjevmNB1CiKkORTOnLqqHIr8OhqC2ZogYEZZRaPNasxKbxY0RVRD+muJmoER0FOFOboMtYm3cUjn+FOdjcxb2yU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1757499648; c=relaxed/simple; bh=dPyTE0l1phm6v0Rl0DSEcOvngS19JHw4+Quv+kDqtxQ=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=Jm45MGT6YXkXFLHy6PZdNouK8PZMGWlZoMaP/+42nkJe0SyQoGL732QMSP2rh7tZlP2O/M7Dpn3vPM/vS2d7SL7a4TCKaw4W/HEKa9Ib4lnK4K+9FyjWqbIOXV6P9CQJ6Q6x7nFuy2vgB5/2JREbYmwMbkuEM4AeVlfa1Z90P40= 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=KDKYBHeZ; arc=none smtp.client-ip=205.220.168.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="KDKYBHeZ" Received: from pps.filterd (m0279864.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.2/8.18.1.2) with ESMTP id 58AAFL3t032162 for ; Wed, 10 Sep 2025 10:20:46 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= i/d/gpEsTNH/PsmnaA/x5LfmNsy7Z1rG7KWqKHeh/ss=; b=KDKYBHeZXggbS5vp 9kmDOmGy3Ztjl8sOsXSc+KactyCsBTXGYzO5gm6Buov/5PfZt400snyCKF4vUTBZ rM7fxhixCTPt14Sepj1El1/vikXTffE6Ta5/GoC2R04RSEN02rIdHPMeIk6xYeYq YPq2D7Scm4x6RbRj6BECJvj85IDImR9zbhlKgN8SpOvgxr4ZtKzpieARwVSEqKKl Vk8ofQvCATJFZRe47qaZdpOCr32t2ayUN7IK4eTz93WbzV0WCxlk1j+JueHDXVoV oVxzuyLYS6rqBFq/AmHwBdhsaQK2jF2o0VsS2Sw+pslUVy+5IwoCa78hlAoy07pG kUgjnw== Received: from mail-qt1-f200.google.com (mail-qt1-f200.google.com [209.85.160.200]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 490e4m3bqu-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128 verify=NOT) for ; Wed, 10 Sep 2025 10:20:46 +0000 (GMT) Received: by mail-qt1-f200.google.com with SMTP id d75a77b69052e-4b5f76fe6f6so13250821cf.0 for ; Wed, 10 Sep 2025 03:20:46 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1757499645; x=1758104445; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=i/d/gpEsTNH/PsmnaA/x5LfmNsy7Z1rG7KWqKHeh/ss=; b=Kl/D8/yn3Z8yiDzMhC7+c3ML/0oPE1bDwABink7ZtHQFhBcUs4YQDetEAYMBBEDzig edWNpEp9TUz6UCWhiJhP0I/LAEacPI5QyQSPwufqZqNR45WKINK2rdy9HBQVOx5ohrMz JTgKEInpRDm1V27vTocuYNpBmB8SELTl00yJWloeJjiQQvlLH+lL7Y1t4xJ4DWYAPBUG EuqYAFwhjn1x6fN6zuuh2Kmc24bprFbOLzZTfDvUCM+ERKHHocTSUg1tfu2kiC2fDoUh QR0fCBDXBTx0cdZ64bEDQycAfSC/dI+0iubGXTYWxjL3iaAW841xrtJNRfT81Wj6xS67 nZHA== X-Forwarded-Encrypted: i=1; AJvYcCUp4ZJiE1z85U7Kk7QDRkj2KtRR0vED0874HOgAFRr3BEUe7afR+4TBDDIkTh4+WhgSYP+9Wjf3PiEY1Q==@vger.kernel.org X-Gm-Message-State: AOJu0YwOjUHpBvEXygCGHHAci56VGTp8micFy9BM0j4WzAX/fzHtaLMG wLp/kKwcfw3t8fiSw6bFH2ZmX368SE6ExUJosgV+v/JN43bFjwDOYwu/4kkkfJ8dGHa5tXRrfrM tB5h3JfOk6rzmo7Uth5hF8kDYQr6/xwWsq7f5PvVPaq4M2jeJnl7BaIlnA109ERRXZw== X-Gm-Gg: ASbGncuQzAv586+TyoSl1v+f4oGCe+DezoOBxdNqpYoR0UqwSU0douL2PzxrznA10MJ DMuCua9eyxQVR92kUH32Pe9vbXjL7ipJ9QZYEUXDYY4WEwc+Qz4X5HJENUrBOtMGU2PlsgtAbGG MXqOvOKJsnz+XM40dR3Rf5OAYk7OSjlAu8eJGJA1TIyzq6DGQroo5xbjmfjxUxA1vyQxyAfznG2 2abGQ6bU9eSTM5z5oF0abQKuo0PotCKq3IIbQJ3T9eksk48CPIR+LQjmrn8x5S8sAaQZ0NOJ478 4gHEXhT+rTTip5cVkhIDfsmvItY/ay2bISWIVzC0s3XiYwcObMBJC15fwRBDtTw1lZQCFbWVJ/C xedwpU0dSU0oLby/ut31L1w== X-Received: by 2002:a05:622a:2d0d:b0:4b6:2de0:10d3 with SMTP id d75a77b69052e-4b62de01819mr3370471cf.10.1757499644959; Wed, 10 Sep 2025 03:20:44 -0700 (PDT) X-Google-Smtp-Source: AGHT+IHn0gApGINKlGbJZ+b48iUyZqmQ9MAUTg97FcSCgm10scqcmE42igYJtWIFBYJDs8y0ApXJzQ== X-Received: by 2002:a05:622a:2d0d:b0:4b6:2de0:10d3 with SMTP id d75a77b69052e-4b62de01819mr3370171cf.10.1757499644207; Wed, 10 Sep 2025 03:20:44 -0700 (PDT) Received: from [192.168.149.223] (078088045245.garwolin.vectranet.pl. [78.88.45.245]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-b078334d74bsm143041766b.57.2025.09.10.03.20.42 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 10 Sep 2025 03:20:43 -0700 (PDT) Message-ID: <136ad3f0-c4a9-47dc-9c62-682a830da823@oss.qualcomm.com> Date: Wed, 10 Sep 2025 12:20:40 +0200 Precedence: bulk X-Mailing-List: linux-sound@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v1 1/2] ASoC: dt-bindings: qcom,sm8250: Add QCS8300 sound card To: Krzysztof Kozlowski , Mohammad Rafi Shaik , Srinivas Kandagatla , Liam Girdwood , Mark Brown , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Jaroslav Kysela , Takashi Iwai Cc: linux-arm-msm@vger.kernel.org, linux-sound@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, kernel@oss.qualcomm.com, prasad.kumpatla@oss.qualcomm.com, ajay.nandam@oss.qualcomm.com References: <20250905142647.2566951-1-mohammad.rafi.shaik@oss.qualcomm.com> <20250905142647.2566951-2-mohammad.rafi.shaik@oss.qualcomm.com> <43090acb-ea36-4015-b14f-78d44d789d42@kernel.org> Content-Language: en-US From: Konrad Dybcio In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-Proofpoint-Spam-Details-Enc: AW1haW4tMjUwOTA2MDAzOCBTYWx0ZWRfX7fGjIcd/p0G/ SO9iM9KnouS0sTcuy5zT4Lxxd/6logfsyNKYFIf1g0OMY8xmIh6udP9oy8Zl8Y+iEaPKBdVVd98 E2js3oM9U4pzPXzy6J48PcaNYgu1i2iTBF5ZSFrFhQPP+csjhrcV4S7sGNiCgDsqcCw17dy6vCW NmEFVljvz1C0mJbZAcn/SBvEf0UuqpvGYGHFdtc8RISnONc7I1G5Q7Tagggm9Q8ZWJA6zWRmw6z vb5rKLvcN4V7zcO6brHF0rh9at2tKGFfalzMs0NbqEiP6t7uxaM0ZaupAX1H7W1dmChDFILrFYv TiACJ0vW+PH9Ik9Kv28FY9Or5ao+4W9ByfDUrI6tzYtY6GB4gjYzWCoUzS4N9BCk9Y/kChqomEZ PVODpcWb X-Authority-Analysis: v=2.4 cv=J66q7BnS c=1 sm=1 tr=0 ts=68c150fe cx=c_pps a=JbAStetqSzwMeJznSMzCyw==:117 a=FpWmc02/iXfjRdCD7H54yg==:17 a=IkcTkHD0fZMA:10 a=yJojWOMRYYMA:10 a=VwQbUJbxAAAA:8 a=EUspDBNiAAAA:8 a=-LPFx5p5mU8rmwG2qkYA:9 a=QEXdDO2ut3YA:10 a=uxP6HrT_eTzRwkO_Te1X:22 X-Proofpoint-GUID: 9ZKNoUg7jKEYlSOPlzvG5nqh5XssScEb X-Proofpoint-ORIG-GUID: 9ZKNoUg7jKEYlSOPlzvG5nqh5XssScEb X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1117,Hydra:6.1.9,FMLib:17.12.80.40 definitions=2025-09-09_03,2025-09-10_01,2025-03-28_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 impostorscore=0 malwarescore=0 clxscore=1015 spamscore=0 phishscore=0 adultscore=0 priorityscore=1501 suspectscore=0 bulkscore=0 classifier=typeunknown authscore=0 authtc= authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.19.0-2507300000 definitions=main-2509060038 On 9/10/25 11:43 AM, 'Krzysztof Kozlowski' via kernel wrote: > On 10/09/2025 10:05, Mohammad Rafi Shaik wrote: >> >> >> On 9/10/2025 1:09 PM, Krzysztof Kozlowski wrote: >>> On 10/09/2025 09:26, Krzysztof Kozlowski wrote: >>>>> diff --git a/Documentation/devicetree/bindings/sound/qcom,sm8250.yaml b/Documentation/devicetree/bindings/sound/qcom,sm8250.yaml >>>>> index 8ac91625dce5..eebf80c1d79a 100644 >>>>> --- a/Documentation/devicetree/bindings/sound/qcom,sm8250.yaml >>>>> +++ b/Documentation/devicetree/bindings/sound/qcom,sm8250.yaml >>>>> @@ -35,6 +35,7 @@ properties: >>>>> - qcom,qcm6490-idp-sndcard >>>>> - qcom,qcs6490-rb3gen2-sndcard >>>>> - qcom,qcs8275-sndcard >>>> >>>> So what is the point of this compatible? There is no user of it and I >>>> think you added QCS8275 for this case exactly... >>>> >>>> Shall I start reverting commits from Qualcomm because you post patches >>>> "just in case" and turns out they are completely not needed? No single >>>> user of such code? >>> >>> >>> @Mark, >>> >>> In case it wasn't obvious, please do not merge the patch till we get >>> some clarification. For sure it is wrong one way or another: either >>> incomplete or just duplicated. >>> >> >> The device tree currently uses qcs8275 as the sound compatible, and >> the corresponding Device tree changes have already been applied and merged. >> >> Reverting this now would break the ABI. > > If reverting would break ABI then: > >> >> A new device tree patch with qcs8300 is currently under review: >> >> https://lore.kernel.org/linux-arm-msm/20250910044512.1369640-1-mohammad.rafi.shaik@oss.qualcomm.com/ > > This is ABI break thus NAK. > >> >> Once the machine driver and device tree patch with qcs8300 are accepted >> and merged, >> >> I will promptly submit a cleanup patch to remove of discontinued >> compatibles from the machine driver. > > So this is the same hardware? Then no, we do not rename compatibles. The QCS8275 compatible snuck in before we concluded the pathway of renaming the SoC to reflect its actual name.. In this case, I think we'll just have to eat the cost and go with the existing compatible, because it's simply cosmetic. That said.. I'm a little confused why we're accepting SoC-compatibles as well as board-compatibles for soundcards (e.g. rb2, rb3gen2, 6940idp) in parallel.. is it with an intention that the SoC one is present in reference designs and therefore will be common? Konrad