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 8B397233721 for ; Thu, 18 Sep 2025 15:38:19 +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=1758209901; cv=none; b=aUJjjNi8CjndeLfTt7ob6wENst9/gYYMC4GDgVUX+iMCfS9Ofm7AxTDGRTR6tV1dywSJyCkm8q1Oohs9ZLvQ0mxeSWLjhfAy6j1Up8yaj0Psy1V8n62vGDbZIfvizD+WlQB+ncP4EzqCAUEnDpE0EaqtG66Qa4AYLpyN/BzMg6c= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1758209901; c=relaxed/simple; bh=2gRUPxfRjSbYhF1p+cW2cxTtaSknQ7GgjWsCTIxXxHo=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=dhkuzKu9D1w0vmtlOR0TXFdq+CunmA/nWK/YYZJxw6MXSXzFjJXUIwUcMK+vfcVUO1zkkBi3eUoMJVkRMIeI2S/AmzY9BAUv6Bsnf6bMFvvpWsgRM1CDsdRB0A6Aq54MrJbWKSEQo2uUxTZ2XZA41hGJZM1x8s86nlwXIcH/uzM= 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=UAaNBnv1; 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="UAaNBnv1" Received: from pps.filterd (m0279862.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.2/8.18.1.2) with ESMTP id 58IBjJmd029495 for ; Thu, 18 Sep 2025 15:38:18 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= 1k9ZRVsktbtLAiEK5sR9687tf7948D4xuFAEiteLjTQ=; b=UAaNBnv1ErvIrs9x vNWWU/TlijYTn1fE5kArVZoLSG3R95Bi5E3QmDVq+XRd34/IKYtCbWR6iLp+Ake1 6JSXY9+jZHeBBtJ5Fs5+dXKTFtKKr4JHOJH4G4x+5TYjPpSw1NF3kHdGG3hsWbmO te1rbAClSAg1g02uH8A5ShOhympZcbAYJMdOQmDqDSOj1yeKfl4h6b6bpZ7iSCLz phcXMpufCu68r9BduZ8sYdWyQ1PNBC+cMVwWW0j0bNwDq4XPo7PlQCTk3p9x3Lja 9MT6y3M0OwARinvnf14iyesHv0i1JORbSWaSgkNWEwLXWNkq6ofQd9SRzp7WU08p kzesNw== Received: from mail-qt1-f198.google.com (mail-qt1-f198.google.com [209.85.160.198]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 497fy0xx5q-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128 verify=NOT) for ; Thu, 18 Sep 2025 15:38:18 +0000 (GMT) Received: by mail-qt1-f198.google.com with SMTP id d75a77b69052e-4b604c02383so4112701cf.2 for ; Thu, 18 Sep 2025 08:38:18 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1758209897; x=1758814697; 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=1k9ZRVsktbtLAiEK5sR9687tf7948D4xuFAEiteLjTQ=; b=eu2ippoVWNndqoR4mGnoayQwbbun2urXrcDVTxSmpEXBHQEeXTz5YJtmRxHqskTcDV NyLbEVAGkgeuyruz4NL2q9YfpMg5clF4mcAQvmCQ1BR2WMa8I5FE39wt/QyNV76uSNMA U2aU4e0ZtLAa9fsyJRjWi1prf89iOLbOhN7rAfAN7Nwy9as2/2kaL3jTYm+UKX2qCga8 NnYoXGZjdYicl14MeO1dxzw0iKnlbUcdKwwZguCiwRejRa4EAxMxjr2EAmmsKlSuVlyH SjYxSQtL+7Tlnq18fGC4xD6glE4oRN0zJQJ1krNao4nfEES2Gt89D+ldoCN4VQ5d2pPX 4OBA== X-Forwarded-Encrypted: i=1; AJvYcCW/2w+PCAUp4losGGbR9y+MOaouVeSis5uuVIqB4pRheMNHPrel+HB7s6YP8H4Hf/DgChfev0MqHnU=@vger.kernel.org X-Gm-Message-State: AOJu0YyZQxygsdAa+rs9TJwM4CGU7ouYQqa2Z994jP0BMklfYbfTYyp0 hPHsbem2+sZEA325MCrBhY0g1GKjb9qtXuKVLIkyFRMYRLQzjYF4fQdpym0lDQHlXdtoVXJWogi eZzxJIgryNlOPnLBjVZ18pnbaVe4IdBrzrEbgThIHPdqgG+W7qSVOgsNeKop3lS8= X-Gm-Gg: ASbGncsKNkoiutG/v2v2Ul1zuwUGqmcMEZ/5s+yjCVHsv/Jv9Y7rkRWul0TS/YPv3vY djXD6f6w92i7rslNOgqk/vsAPf+VhhS0ZXpZpyu5cbPu0smoYQs5kFk6sFcTKZt0qOD5DwNDMDX DXOST6dhEQsMRR8wS28wjF5xogJBXyYDE0QRa4bw5RiQNlPnyCJGMX96gUP4EAjxSJdSNPrdweB zTV2TQdYNkcapCfC2u0fpV7EZAhoensb/ZFZlWVXy7XK/AMYhnQu+TxJYUjpgntqeV1g0iPpebs 3NrnYngXJGpkgXpDjnd11rS7C0zVtiFhA3V4eC4EpitKeOXaqsstlZQjPipnToZy6osSyTtn+D+ AgHGTeNEORx0DDQP0PfzXDg== X-Received: by 2002:a05:622a:1342:b0:4b5:e606:dc13 with SMTP id d75a77b69052e-4ba6aaa82e9mr45331201cf.10.1758209897210; Thu, 18 Sep 2025 08:38:17 -0700 (PDT) X-Google-Smtp-Source: AGHT+IGBfdAGIe8TQF/ReStc/B5aiHhufdTh5lJgSiTbdgmj+ayT1z4IZtsaBYVybC06t0pmBAwlgA== X-Received: by 2002:a05:622a:1342:b0:4b5:e606:dc13 with SMTP id d75a77b69052e-4ba6aaa82e9mr45330881cf.10.1758209896681; Thu, 18 Sep 2025 08:38:16 -0700 (PDT) Received: from [192.168.149.223] (078088045245.garwolin.vectranet.pl. [78.88.45.245]) by smtp.gmail.com with ESMTPSA id 4fb4d7f45d1cf-62fa5d4189bsm1616785a12.19.2025.09.18.08.38.14 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 18 Sep 2025 08:38:16 -0700 (PDT) Message-ID: Date: Thu, 18 Sep 2025 17:38:13 +0200 Precedence: bulk X-Mailing-List: linux-usb@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH RFC] dt-bindings: thunderbolt: Add Qualcomm USB4 Host Router To: Mika Westerberg Cc: Konrad Dybcio , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Bjorn Andersson , Wesley Cheng , Jack Pham , Raghavendra Thoorpu , Dmitry Baryshkov , Mayank Rana , Krishna Kurapati , Manivannan Sadhasivam , linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Andreas Noever , Mika Westerberg , Yehezkel Bernat , linux-usb@vger.kernel.org References: <20250916-topic-qcom_usb4_bindings-v1-1-943ecb2c0fa7@oss.qualcomm.com> <20250917061236.GF2912318@black.igk.intel.com> <20250918051244.GJ2912318@black.igk.intel.com> <035c0d66-bddd-495c-bd23-e1d98570ba7f@oss.qualcomm.com> <20250918152322.GK2912318@black.igk.intel.com> Content-Language: en-US From: Konrad Dybcio In-Reply-To: <20250918152322.GK2912318@black.igk.intel.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-Proofpoint-ORIG-GUID: lRiTpGULNQb310B_pA9VNqiDuBv7e2QS X-Authority-Analysis: v=2.4 cv=btZMBFai c=1 sm=1 tr=0 ts=68cc276a cx=c_pps a=mPf7EqFMSY9/WdsSgAYMbA==:117 a=FpWmc02/iXfjRdCD7H54yg==:17 a=IkcTkHD0fZMA:10 a=yJojWOMRYYMA:10 a=Zu-_-z5cN0bD-gc8aucA:9 a=QEXdDO2ut3YA:10 a=dawVfQjAaf238kedN5IG:22 X-Proofpoint-GUID: lRiTpGULNQb310B_pA9VNqiDuBv7e2QS X-Proofpoint-Spam-Details-Enc: AW1haW4tMjUwOTE2MDIwMiBTYWx0ZWRfX7fdN8vgyR4Cf DEsNox8V7pQIvAfagun6iu5Y14DC4gqOj12wa+5dfTp4IGEhU5DvZ+OuVXg7Knxx0Y8sYBdIQTo 22mriEDax/7l6sLAHihgmoMR0MRZASWBEBL2/ZFpCk7H0vVxJ8wIncMyjRZG1OrJoQ83MEL/O5/ ZXQCyRHft72VGkekrXuTOR0NDkVzPCX13JeXP0V2KE9UrPPZXVdg14xIeMWkwTX7JwInK93m/Ms ewfbfyaMNCrlFmBgZ6ir+K9HQS35kQwxisG9/HQ61xV0J+84Nk+tyUyVLILS+9hhENw3XsR0rt0 x26oRLg4H9wkpmTn//CNzhe8A5VoOtk1I+Nx+NcB9PeOKSvFN1dwr4qYRXB4mbyTQjhoSQ0xhfS N1h26tbm 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-18_01,2025-09-18_02,2025-03-28_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 bulkscore=0 priorityscore=1501 impostorscore=0 clxscore=1015 malwarescore=0 spamscore=0 adultscore=0 phishscore=0 suspectscore=0 classifier=typeunknown authscore=0 authtc= authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.19.0-2507300000 definitions=main-2509160202 On 9/18/25 5:23 PM, Mika Westerberg wrote: > Hi, > > On Thu, Sep 18, 2025 at 11:05:42AM +0200, Konrad Dybcio wrote: >>> I it hard to change these DT bindings later on? If yes then I would >>> definitely think forward and make this support MSI from the get-go. >> >> dt-bindings (attempt to) promise an ABI-like interface, so bindings >> for *a given IP block* ("dt-bindings describe the hardware") should >> not change, unless there's something critically wrong (e.g. "this >> could have never really worked"). > > Then I think it is good to think few steps forward and make sure when > Qualcomm adds MSI to their IP it can be easily desribed in the DT bindings. Your reply makes me believe I glossed over an important detail.. If qcom,x1e80100-hr doesn't have MSIs, qcom,nextsoc-hr can add them without any issues too, just that the X1E80100 binding can't be altered, e.g. "binding" refers to the (compatible, allowed_props, required_props) tuple Konrad > >> Adding new properties is always OK, marking the new properties as >> 'required' is not (unless it falls into the aforementioned case). >> >> It's also totally OK to add MSI properties to e.g. Apple Host Router >> bindings specifically when they come around, as it's simply a different >> piece of hardware. It's also OK to create a usb4-host-router.yaml down >> the line, which will act as a common include and perform any >> maintenance/code churn, so long as it doesn't end up in the bindings >> for any specific hw block (e.g. this QC one) becoming more strict >> than they were on HEAD^. > > Okay thanks for the explanation.