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 444FF23EA87 for ; Thu, 19 Feb 2026 05:38:13 +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=1771479494; cv=none; b=qnIeOLBFfSmZEzgXkTuW0FM7gus8yZkUMKLzAulQDyqHlCsrQ46kgRyW03IFF1wA5BkvXMZIJG9dtPBugUUMf+wQaMW1FJF29ElsdqypHzDmkmkcO2Nci1lGngohbUzX1SoO+nXPWgEhf8J4XGlnDaHhf4+wbTtzQTUHFgfgIFE= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1771479494; c=relaxed/simple; bh=JarSg4Iy2X5OeOrsbK3lxs8hqItkhx4v7l+xAx/pSn4=; h=Subject:To:Cc:References:From:Message-ID:Date:MIME-Version: In-Reply-To:Content-Type; b=kFdZVB9KZvSCNBW7dUEvq10NUasFIcknh6N68qhaHj0//C6Fn161jEgGmcXOs6es5a+4nFp6Wq8Xqbz3qGmFRGeSDFiWTJKBsdkqpvPTRLNQBmeUYNLZp8SidHfTZah5nurbRpGpWRl9V7MrfHAF/JGVhQxXI8+QVvgk70n+DH4= 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=iUOTJcxI; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b=RMr0o7Ja; 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="iUOTJcxI"; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b="RMr0o7Ja" Received: from pps.filterd (m0279870.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.11/8.18.1.11) with ESMTP id 61J3O4jv4071897 for ; Thu, 19 Feb 2026 05:38:12 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= bx9vef0HkXb6vOZHSEYOmtJceWicO+PSjEh/i3J41t0=; b=iUOTJcxI5tVwiqCa /KjjOGai9QMxIxMzTxQo24CwE4R5X+J5euxtXcTyG5V54g+jlWpmuzcZWu7kGgnu QuS4JYOO0AM/aW7WFKgN6s26+YV1H5JDNOdyRN+nQ0wUAnFZGkooU9e3UD8AUTHT 40RDqL0Ysp0smn5otmWsm6KazMA6ipifrd21H70hoTslUA1u5Yy0I4ewLPy9H15+ SM3h/gWAc6kGTSdryVgNLsEks5slUqZfireVfW2A7GamY2Yd6C0a/k7k5w/hT3Zn rc+9MhPjBfQ1vT4WdK2RCUFA5+M6DI9F7CcyHvCC0pceXhjd71rVzM2o/1SDajE9 cq28/A== Received: from mail-pf1-f199.google.com (mail-pf1-f199.google.com [209.85.210.199]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 4cdtjy07pc-1 (version=TLSv1.3 cipher=TLS_AES_128_GCM_SHA256 bits=128 verify=NOT) for ; Thu, 19 Feb 2026 05:38:11 +0000 (GMT) Received: by mail-pf1-f199.google.com with SMTP id d2e1a72fcca58-82436a49592so273711b3a.2 for ; Wed, 18 Feb 2026 21:38:11 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oss.qualcomm.com; s=google; t=1771479491; x=1772084291; darn=vger.kernel.org; h=content-transfer-encoding:content-language:in-reply-to:mime-version :user-agent:date:message-id:from:references:cc:to:subject:from:to:cc :subject:date:message-id:reply-to; bh=bx9vef0HkXb6vOZHSEYOmtJceWicO+PSjEh/i3J41t0=; b=RMr0o7Jaqfc2HTLMhvDZZdFTfs/0RnAg/JSAwp4JWkk88H3VW5M3xLi1Fmilamn1Vw 2RC9ID0+7HisfMNUq4mh7ZYhwMj2jPP5rS3G2GuaxuoL68I1rhEOEjf4ghZkWtp4W/QY 728NFMJhBCC6++Yrc2bgbbx253OuC8t6lpQ6QuWCx7f6uhASPlrabhOSSDvb/0FrAScu duou4v0gPf+YqHPzOIvsNB06DQ7lDcFAfTJdIaJ7SXwlMcYPj2kNSfTWzAkel8h4XjzN pY6qLJ4OXVSqmmqv5E+muqptkBt6e/hb3XiSPqI8+Lr8nHysn4SnOlLSHCIdf7GDsBwd ZmEQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1771479491; x=1772084291; h=content-transfer-encoding:content-language:in-reply-to:mime-version :user-agent:date:message-id:from:references:cc:to:subject:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=bx9vef0HkXb6vOZHSEYOmtJceWicO+PSjEh/i3J41t0=; b=m5iAJK8+2+kzpg3nWwod2R7ncCaMW2ZZEfds7ncqvVQjH5en/fUtF01k/mVx71fJsC 9rA5aJUfZt193kdUoYNtLdBRo9jDlpFW6lB7TJ+vMs8k4tC31ifbNtPEzP52UTMzCVzG 4YR68IGnBXrBrHIWAKT0VOVBFiRwVUp9HDSXcCIN3RACbd/soqd8dTaBIa3gazgaEqjU M8w//3dJ9q+aIbLlE8CGemTcPMNcXRmuc36sFNUSbm/vn/lad/ffofMhpB5UNecrkGz2 wj3LP4gKWI304B8G5jU9FHsU7nyOhiTWgbDeOesp4/TjPuTbhuU6alFCmyIism1ycgIT A9EQ== X-Forwarded-Encrypted: i=1; AJvYcCX+F9a2hmt7VpUlHOxhPnfzF9xtfcXD9qxLsM2wksfvGAC5kI0nnVfpVNwZOJgauhuy92dyOcIqq/wJmfU=@vger.kernel.org X-Gm-Message-State: AOJu0YxV+qN40A0WZlDVXhcRROvogl6uUYKH4JcarAPBFVB6UGu1oiSU 2gWjNevjJ+QTJPk2B2uYs/V4wzbFWY1GakrjpWNuwdaqVPfUlix2arVQC7fWKmfSWP1iWq/J5Fc aKL+ghGXsAl8aa3X93prZ/2HdTVg3raOnK/q2mdXq7gaFETeVvRIZPhCI4i7achGyX0k= X-Gm-Gg: AZuq6aL5y+WXyxafN3gTPFIUkIOtstIVNDxPp3ZOBKM4jv6/wWSFtAV6b1YjilOwhxp Ei+tyYEjLrCGMuLLgQg325kXU+czIYqlyNI+N89fL52q/jIKO4CtfI9niD9E+AHC8+UXdHbZXiS 0sm6g+gz6Z8AbrFfwEfdiLuvGRbeA8lLyCoKUya5AiwsIU/6QT1FNwPCBNm1zTcDo+L/FSfjwiF sqIsqkBKH38mrFGgUqHeOs26FuRv8IxvmdkPFaRD+GeGxQa5j3BArRKzZgn+39dw6WPt95rBL+v ZQaQiJwGBspG+rYNUhXRTZBKJLkUl0vS69MBDBRhitWnLxqFUm21Y00DbdVRvrAKU2MnOKD39D5 V758eyd+Y/wHdC2N9KYIIh/N9nWUXTjDBlHgEG1O3WsNFb2GkDw== X-Received: by 2002:a05:6a00:7586:b0:821:70e7:74ca with SMTP id d2e1a72fcca58-824c965d569mr15419705b3a.70.1771479490932; Wed, 18 Feb 2026 21:38:10 -0800 (PST) X-Received: by 2002:a05:6a00:7586:b0:821:70e7:74ca with SMTP id d2e1a72fcca58-824c965d569mr15419671b3a.70.1771479490448; Wed, 18 Feb 2026 21:38:10 -0800 (PST) Received: from [10.217.222.63] ([202.46.22.19]) by smtp.gmail.com with ESMTPSA id d2e1a72fcca58-824c6a409e6sm18351122b3a.18.2026.02.18.21.38.06 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 18 Feb 2026 21:38:10 -0800 (PST) Subject: Re: [PATCH v4 1/3] dt-bindings: mmc: sdhci-msm: Add ICE phandle To: Krzysztof Kozlowski Cc: ulf.hansson@linaro.org, robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org, andersson@kernel.org, konradybcio@kernel.org, linux-mmc@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Abel Vesa , Abhinaba Rakshit References: <20260217052526.2335759-1-neeraj.soni@oss.qualcomm.com> <20260217052526.2335759-2-neeraj.soni@oss.qualcomm.com> <20260217-berserk-puma-of-focus-bcbe82@quoll> From: Neeraj Soni Message-ID: Date: Thu, 19 Feb 2026 11:08:05 +0530 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:68.0) Gecko/20100101 Thunderbird/68.12.1 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 In-Reply-To: <20260217-berserk-puma-of-focus-bcbe82@quoll> Content-Type: text/plain; charset=utf-8 Content-Language: en-GB Content-Transfer-Encoding: 7bit X-Proofpoint-Spam-Details-Enc: AW1haW4tMjYwMjE5MDA0OCBTYWx0ZWRfX21EAkJSgr4X6 lPtlEZHFPF3qeuHCmYaMoHac9ucfKYqmF4R7S811NOg40mDyRnm/YienvpIEsnDzpwsyjFQ1+rq ZvguqsDqP2wMLcgALiewsyrobUC/ZFHt0eGkhM6Xx3z8O8+cYSsdfY3KTpwKXf3eXD/q3CDT/sj fvZeXjU44m26L1erVP6EgC4OzV1OjD/MDXYqyNGHVd3CAU29WyKWvHC4xbIkZHK9LTMOfbFIN2T ff0b4PCDZCBT/pF89Ez/U2bbIwkX4IBc8qjij0l1rGRwGLaYRwfFra0k7l6yjhUnNp5rgPh5hub t7Qtyz0gw0Jn9bVdkAc3xCNtwNIu9v+Z6sPEYckSGI3KQdcJ6RqqTX0bTiJb+/IgoT0FYdI/idp INxTzFGZvB/cHgzraawhzC71dAAIPPBaJZsqYFZ5vCeYbJUZ4A7eXBV/J5RQzMFQPUd8Wk16l/9 MGHaLqViR5l5vxafNiA== X-Proofpoint-GUID: BtcMpiUBMUjk4U2y-Df5fFUwGTGWy9Td X-Authority-Analysis: v=2.4 cv=FoUIPmrq c=1 sm=1 tr=0 ts=6996a1c3 cx=c_pps a=WW5sKcV1LcKqjgzy2JUPuA==:117 a=fChuTYTh2wq5r3m49p7fHw==:17 a=IkcTkHD0fZMA:10 a=HzLeVaNsDn8A:10 a=s4-Qcg_JpJYA:10 a=VkNPw1HP01LnGYTKEx00:22 a=Mpw57Om8IfrbqaoTuvik:22 a=GgsMoib0sEa3-_RKJdDe:22 a=P-IC7800AAAA:8 a=VwQbUJbxAAAA:8 a=KKAkSRfTAAAA:8 a=EUspDBNiAAAA:8 a=7CWuv0upaalPmBxTJ3QA:9 a=QEXdDO2ut3YA:10 a=OpyuDcXvxspvyRM73sMx:22 a=d3PnA9EDa4IxuAV0gXij:22 a=cvBusfyB2V15izCimMoJ:22 X-Proofpoint-ORIG-GUID: BtcMpiUBMUjk4U2y-Df5fFUwGTGWy9Td X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1121,Hydra:6.1.51,FMLib:17.12.100.49 definitions=2026-02-19_01,2026-02-18_01,2025-10-01_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 impostorscore=0 priorityscore=1501 suspectscore=0 phishscore=0 adultscore=0 lowpriorityscore=0 malwarescore=0 bulkscore=0 spamscore=0 clxscore=1015 classifier=typeunknown authscore=0 authtc= authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.22.0-2601150000 definitions=main-2602190048 On 2/18/2026 1:46 AM, Krzysztof Kozlowski wrote: > On Tue, Feb 17, 2026 at 10:55:24AM +0530, Neeraj Soni wrote: >> Starting with sc7280(kodiak), the ICE will have its own device-tree node. >> So add the qcom,ice property to reference it. >> >> To avoid double-modeling, when qcom,ice is present, disallow an embedded ICE > > Please wrap commit message according to Linux coding style / submission > process (neither too early nor over the limit): > https://elixir.bootlin.com/linux/v6.4-rc1/source/Documentation/process/submitting-patches.rst#L597 > Ack. Will fix in next patch. >> register region in the SDHCI node. Older SoCs without ICE remain valid as >> no additional requirement is imposed. >> >> Co-developed-by: Abel Vesa >> Signed-off-by: Abel Vesa >> Co-developed-by: Abhinaba Rakshit >> Signed-off-by: Abhinaba Rakshit >> Signed-off-by: Neeraj Soni >> >> --- >> >> Some initial work is done by Abel here: >> https://lore.kernel.org/all/ba3da82d-999b-b040-5230-36e60293e0fd@linaro.org/ >> and by Abhinaba here: >> https://lore.kernel.org/all/20251009-add-separate-ice-ufs-and-emmc-device-nodes-for-qcs615-platform-v1-1-2a34d8d03c72@oss.qualcomm.com/ >> >> This patch adds the purpose and usage for phandle in the description and encodes >> it properly in the schema. >> --- >> .../devicetree/bindings/mmc/sdhci-msm.yaml | 15 +++++++++++++++ >> 1 file changed, 15 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/mmc/sdhci-msm.yaml b/Documentation/devicetree/bindings/mmc/sdhci-msm.yaml >> index 938be8228d66..9b902e0c8d09 100644 >> --- a/Documentation/devicetree/bindings/mmc/sdhci-msm.yaml >> +++ b/Documentation/devicetree/bindings/mmc/sdhci-msm.yaml >> @@ -140,6 +140,11 @@ properties: >> $ref: /schemas/types.yaml#/definitions/uint32 >> description: platform specific settings for DLL_CONFIG reg. >> >> + qcom,ice: >> + $ref: /schemas/types.yaml#/definitions/phandle >> + description: >> + phandle to the Inline Crypto Engine (ICE) hardware block for this controller. > > > Srsly, I asked once and not much improved. What is explicitly written in > coding style which I asked you twice to read? > I have attempted to modifiy it as per the comment you gave in v3 here: https://lore.kernel.org/all/b6e510da-b369-4c43-b9a1-455478af4948@kernel.org/ I will rephrase it to explain the usage better in next patch. > And how long is this line? Why such trivialities cannot be fixed and I > need to remind this every time? > It is 83 character long but i understand now you expect it to be wrapped at 75 as per kernel coding style. I missed it and stuck to the limit dt_binding_check tool allowed. Will fix in next patch. >> + >> iommus: >> minItems: 1 >> maxItems: 8 >> @@ -223,6 +228,16 @@ allOf: >> - const: cqhci >> - const: ice >> >> + - if: >> + required: >> + - qcom,ice >> + then: >> + properties: >> + reg-names: >> + not: >> + contains: >> + const: ice > > And reg is still 4? This is not correct syntax. You need to define > proper and final constraints per each device. I would write example, but > why... more things you could just ignore. > I had included changes for reg in v3: https://lore.kernel.org/all/20260206112053.3287756-2-neeraj.soni@oss.qualcomm.com/ but those were not reviewed so i assume them to be incorrect and dropped it. Will fix this in next patch and post. > Best regards, > Krzysztof > Regards, Neeraj