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 9D3543F9F58 for ; Thu, 11 Jun 2026 14:14:15 +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=1781187256; cv=none; b=g1kNoP+/4NDBGL8DCPnqW6H1Fx320k8p0qRs6vjhwELWVPe/6pRNE9C194xvk8yhMARVCK1oKtJ/3ZdfJt184vboiRUH0qVh7saMoP0g6bUNcCUlNKk2EeNmIY71EyLXnWKzLquvzpykrF+4rmDdHnZKIRwA475lxoISIwv4BCY= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781187256; c=relaxed/simple; bh=/KXfgHqvKtIKL7qU20XZzM/WnBHXM4OFw2OoYJ39uDU=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=nJpqIz5n4BAMg/fQ05PIwJa/rR/77IWAzVwx3b4z/FBfmfDizt4sJbkmjT3ByvlWXaSxn22D+uVB/yaZDx/dnooJPtGjVEoTckYCHK7gjtXhRt9eoIJl4FdmgiAsb0qqXVi309Dnqbx9xgc7kyK6rrMomynBOJJ/+yFayds6Kjo= 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=R/Qln5nu; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b=YI0XGX1H; 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="R/Qln5nu"; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b="YI0XGX1H" 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 65BCSpeA546411 for ; Thu, 11 Jun 2026 14:14:14 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= BT1cmZgyC5OEU1NGiHjMPSjXyj8whKVB9HRG3NFstnE=; b=R/Qln5nuVBgPV919 sfIX6c1x5AAUsW+SqhU8GbRqOn5FZxTnLGF5MMhTjop9k1CUzkBfGHVUnlWWwm1X 1Ng8RG2z9BRw4UZI/gmpL9ZUyah+dbAMp3/7yoS4MCRk+h6YXmrIgx5LFtcRGtAq FACCU7+UcuEKjp+4DJVN9iyxc4xpc9q0qNH5tZkaGER5fiwr5qk1pr+BDX/3rVIe G45pLBwkU5lLfN/gtJIjtsuopbAxGvhvZH44xYl70gphNzjymbz+1RRKnQawi3Yd mj8E/DyNSYI9M+rp0CI2owsN3PhGB0qPST7FnV7pIyJQMJcgSbCXSkKn1sLa1AKm fF+CRw== Received: from mail-pg1-f199.google.com (mail-pg1-f199.google.com [209.85.215.199]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 4eqe6skw9f-1 (version=TLSv1.3 cipher=TLS_AES_128_GCM_SHA256 bits=128 verify=NOT) for ; Thu, 11 Jun 2026 14:14:14 +0000 (GMT) Received: by mail-pg1-f199.google.com with SMTP id 41be03b00d2f7-c8620ee0971so5734371a12.0 for ; Thu, 11 Jun 2026 07:14:14 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oss.qualcomm.com; s=google; t=1781187253; x=1781792053; 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=BT1cmZgyC5OEU1NGiHjMPSjXyj8whKVB9HRG3NFstnE=; b=YI0XGX1HAgDkT5L1E5TaLTgMTs8U42xWNdiq29xvjKSiz29KFmXI5pt49QnB4Y31hp w/K/iNwQ66L4Kib3ldHpU0uXFpdnLQqc3iY2Ppt82VAuhcg06zPxnyqxSMIaa7tbBthn BbzanQ06y/amr7TkeUH/IaGMDKaj3ey9QhCVPdli3GyK0wIFoREkKNNg8yEUQyMIWCXK rdzuW/pu8Wiegk1wABo7cgACmngqn4JBsH79C209nTRC8KZks5bDhMdkJKVB7He9qoSL wPzJs88oSXZ6AZIBONg6ZE+o+z1DHtx1dZyy/BIfkoGIua8oU7ETI5/CiqnKyXX1Ior1 ZY7A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1781187253; x=1781792053; 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=BT1cmZgyC5OEU1NGiHjMPSjXyj8whKVB9HRG3NFstnE=; b=Ai6pV/vaOkDCmkYNWnkq5iCbge1bgMpQ/m/3BotqiNND3gZWfv5p4Msc3JgXiTCQEu Z5/x9Akxd8YhnMaSeHUj+lot+/1c8I6goE8t/ba9bSTQ8xHAlLhs5Y4hCdrm44goutkl BBUAxTsivi/Btq3FGrT+t0LzNl4zng+IaSQFBg8C4yfQtbOJ/YG+viNiYsll2v4G20z4 wUsyTFMN/8OPV0BCig+UhGw+2ELsBmoNjqsQ48sRJRuKEm0yBQ4w/MR45DEdXbpVe6VP U/Rs2QXiwh175d+PdahXa/2eh7qCvIiA7zDVc02pWk6eN5n1IZKne5eTLp2OI/DpmMi7 vFTg== X-Forwarded-Encrypted: i=1; AFNElJ+Vzjl7hwiNnxN6yWYrVZGf5GQ5L6mmTu9IQC5HWjHxW7j96qImMWjTzhTghdIXiaFgRqZfFJykR9cmMfA=@vger.kernel.org X-Gm-Message-State: AOJu0Yyo/SLlOTL84z6MA76gMLKV8iiJgpqhVaX/85p8rkqKiJ/ETBdJ +rW2gQWQ8z6SCzXLV7Z2oUivKGdo57poxdp+2j1NLLSjJxKCSRJ0bNo1/zoNq1eU+6gq5tPvtIT +vqrw7oH7jauGeAzYZl1uMecjYaDOEuIT3Hh8YHENYUR7ie45GdnRN5wCyeYZ0eswAKmQKk1gh6 A= X-Gm-Gg: Acq92OFP1tiYazgukyf2JhBGa+jOiNm+8cj8SVYyaXSb4r6nz9PvVNpaC6n9hcBcOyu a3grWFmboQP995+z3kDw0ili8IlQXvswvGREynf/19LpDOsJpM1RD2LBsZVxt6EvY1L1QZp9XCn s99ncA0MdfTo+HzmwBqWiivs5c64fHTTI4lGPU/pagRzxs1HwF7s+xaRBANqQ3zrzx+c7z4EOro RSrcVn6SAv33LqPcjSJ4at4l5ROJuu031G5Bw2FXoxQTutpy6CK3cqve11SNSyyeNFDuUDcau7U asFzXp2k6GXcE38vGXin+6KLt6/sNfBIc+H990Zda1a6m3CFd8Vrb1TSkbRudSouuI2Z4OEK+ym e2KhLLn6xHVrIeZsJBkWFyL9KSGn8+XNp2xD3fD0duW78BI65lSeERbqn3HLWzQELi1VKpr2HkM dC5vviUGdqhSv/Qt/GQTJuuA== X-Received: by 2002:a05:6a00:8d8d:b0:842:6004:3fcf with SMTP id d2e1a72fcca58-84336a8b12cmr3445496b3a.29.1781187253306; Thu, 11 Jun 2026 07:14:13 -0700 (PDT) X-Received: by 2002:a05:6a00:8d8d:b0:842:6004:3fcf with SMTP id d2e1a72fcca58-84336a8b12cmr3445454b3a.29.1781187252798; Thu, 11 Jun 2026 07:14:12 -0700 (PDT) Received: from [10.133.33.231] (tpe-colo-wan-fw-bordernet.qualcomm.com. [103.229.16.4]) by smtp.gmail.com with ESMTPSA id d2e1a72fcca58-84337be7097sm2391884b3a.24.2026.06.11.07.14.08 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 11 Jun 2026 07:14:12 -0700 (PDT) Message-ID: Date: Thu, 11 Jun 2026 22:14:06 +0800 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 v7 1/2] dt-bindings: ufs: Document static TX Equalization settings properties To: Krzysztof Kozlowski , bvanassche@acm.org, beanhuo@micron.com, peter.wang@mediatek.com, martin.petersen@oracle.com, mani@kernel.org Cc: linux-scsi@vger.kernel.org, Alim Akhtar , Avri Altman , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Zhaoming Luo , Ram Kumar Dwivedi , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , open list References: <20260610071516.3763916-1-can.guo@oss.qualcomm.com> <20260610071516.3763916-2-can.guo@oss.qualcomm.com> Content-Language: en-US From: Can Guo In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Authority-Analysis: v=2.4 cv=GbMnWwXL c=1 sm=1 tr=0 ts=6a2ac2b6 cx=c_pps a=Oh5Dbbf/trHjhBongsHeRQ==:117 a=nuhDOHQX5FNHPW3J6Bj6AA==:17 a=IkcTkHD0fZMA:10 a=FelO9ux0wxsA:10 a=s4-Qcg_JpJYA:10 a=VkNPw1HP01LnGYTKEx00:22 a=u7WPNUs3qKkmUXheDGA7:22 a=yx91gb_oNiZeI1HMLzn7:22 a=qkbM5XvOScw6ZQYEvzUA:9 a=QEXdDO2ut3YA:10 a=_Vgx9l1VpLgwpw_dHYaR:22 X-Proofpoint-ORIG-GUID: BRoWKiVKx6pT5ZeVHCaRdNIVUq0wy1wk X-Proofpoint-GUID: BRoWKiVKx6pT5ZeVHCaRdNIVUq0wy1wk X-Proofpoint-Spam-Details-Enc: AW1haW4tMjYwNjExMDE0MyBTYWx0ZWRfX+WSqA4oCHRuy ifiyOhQ5jNi1Kg3r1OO5wBqqcZKoRw5A9f+W8dMlUTd8h+/43U6EFWhxCVQfdhXH69iRKBsXOL4 3cTonnXqb+SAElk1DMM5QLP5khwYA9f8YsZwP1CalkIkYqMq/5Y9BKhOYUlkEIby7/Dr6JKl4jY U33YAjgCXLNGtV4fQps2J0XBdjeLHbdDee0VT99/0U1BgoMAVgQibkSJnbV/8fVfkrErSH0HAnO f9/CgzZqP+R3Qjo2V2/MKb9hImtLkrweb9HppPSxJT8vnyia9LnHzvf3mqpnw0PY8YeGDljYgqV RAvOD2EAC/JpGGGG15MargMZ6rNWRMFfKURV3m91WX6JMz+q45YzubFnALm3scegQ4051ZciF8w Y0bqyro185EV1xIxGa4Ho5PXrF6GRkr/3Us00EgIWutPdUesR4KgYqtCh6Er/sIupDVb5VeSFaQ Wt56C9EkpwRJ7obc+RQ== X-Proofpoint-Spam-Info: AW1haW4tMjYwNjExMDE0MyBTYWx0ZWRfXym1/ytw0Q8jD 3EpPHQdGMvOBT9jzp9PCCxOc2zRPfiEypVV58SCrEQBOfMAK14GGSNwnSTxfikUZWjNabdCQoK/ 65j0fETfu3hR6LqrKEXSiuLVBnFN5s8= X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1143,Hydra:6.1.125,FMLib:17.12.100.49 definitions=2026-06-11_02,2026-06-11_01,2025-10-01_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 phishscore=0 lowpriorityscore=0 malwarescore=0 suspectscore=0 priorityscore=1501 bulkscore=0 adultscore=0 spamscore=0 impostorscore=0 clxscore=1015 classifier=typeunknown authscore=0 authtc= authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.22.0-2606040000 definitions=main-2606110143 On 6/11/2026 9:32 PM, Krzysztof Kozlowski wrote: > On 10/06/2026 09:15, Can Guo wrote: >> UFS v5.0/UFSHCI v5.0 add HS-G6 support via UniPro v3.0 and M-PHY v6.0. >> These specs define TX Equalization for all High Speed Gears, and HS-G6 may >> also require TX precode depending on channel characteristics. >> >> Document vendor-neutral DT properties in ufs-common.yaml: >> >> - patternProperties for txeq-preshoot-g[1-6] and txeq-deemphasis-g[1-6] >> - tx-precode-g6-host-lanes >> - tx-precode-g6-device-lanes >> >> txeq-preshoot-g[1-6] and txeq-deemphasis-g[1-6] accept per-lane tuples: >> , [] > Instead of repeating the diff, you should explain why these properties > are needed. Insufficient explanation was also pointed out at v1. > > Why this cannot be deduced from the IP compatible? Does it depend on the > device memory? Who determines the values here and what do they depend > on? Also here you explain lack of auto tuning for example. I will give more explanations in next version. > >> PreShoot and DeEmphasis values are 0..7 and accept 2 or 4 values for x1/x2 >> lane configurations. >> > ... > > >> + Lane indices for static Host-side TX precode enable settings for HS-G6 >> + only. Listed lanes have precode enabled; unlisted lanes are disabled. >> + >> + tx-precode-g6-device-lanes: >> + $ref: /schemas/types.yaml#/definitions/uint32-array >> + minItems: 1 >> + maxItems: 2 >> + uniqueItems: true >> + items: >> + minimum: 0 >> + maximum: 1 >> + description: | >> + Lane indices for static Device-side TX precode enable settings for HS-G6 >> + only. Listed lanes have precode enabled; unlisted lanes are disabled. > > I need to reverse my opinion and let's go to v6 implementation. These > properties look more consistent in v6 with respect to preshoot and > deepmhasis properties. > > You want actually matrix, so: > > tx-precode-enable-g6: > $ref: /schemas/types.yaml#/definitions/uint32-matrix > oneOf: > - items: > - description: Host_Lane0 precode > - description: Device_Lane0 precode > - items: > - description: Host_Lane0 precode > - description: Device_Lane0 precode > - description: Host_Lane1 precode > - description: Device_Lane1 precode > items: > enum: [0, 1] > description: > Static TX Precode enable values for HS-G6 only. > > And similar style with items also for preshoot and deepmhasis. Thanks for the suggestions. Will update in next version and come back. Best Regards, Can Guo. > > > Best regards, > Krzysztof