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 AEE2F40312F for ; Tue, 30 Jun 2026 11:43:51 +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=1782819833; cv=none; b=VvPQlipn+gydvl4SSCIDNTXwlblXE93DHl8+mnsuHAqVRqQ+fp4XgzgT6jPwKBlu1xwGFFNB2WgtmkLD4OVfwHdLctgGsLvjZzmlakhXSSRCI7u+dMJIuFDfjG83byyZQs3TG7xOXlelczn/+0yjko/Je6rNVnbK+psKJYSAQRk= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1782819833; c=relaxed/simple; bh=ydxso3tQzTPdXtcvGFdCqB1x6Jr03cYbvaMFGBrRqxY=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=qkfO7ghrNlCfAdeBM3ZbPYHLNCXY34sP+FkNP1KkNLt0u4knI/SNVD2Q3T++xSBbJ4lVGX66cEozPODH10ku5WNjVjdmlAldbGMP4QGJj9IS0jSfE2lzkzmUkTH/06L7KjYPh6sQ9TiPfeARkHwQ7YNYRcFTiKMmwVmM3qeV8jM= 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=d0Rq5OHc; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b=BNmdRs8/; 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="d0Rq5OHc"; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b="BNmdRs8/" Received: from pps.filterd (m0279864.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.11/8.18.1.11) with ESMTP id 65U9n4vh1508664 for ; Tue, 30 Jun 2026 11:43:51 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= dVMvkTVLGH+FcIS/vfSvK/ujPY+UerDGY/JcdPljsRg=; b=d0Rq5OHcxH34F+t2 luWaixD9UWj7vzhGYA6kxXL+JJBPzSmvjx7PMBdI3LttHL5XGZGliNh/m4Y2BuH4 tjQfbD8CjuqpcTy3oz2zRFnCcvryFpqDMtJ3jGsq2F5/kxNk/3fFGmJLJsbSTlD4 HHFCjdYQF6e8gvSVPdwKKVlrBl+Iv43I8jaPVFLSPKQc5Gpacd/F+e0cByxLxgya cMCCP6r8KgTjQiJB2UEXaJbyl8cWYsewsO9mizSdpbqrF0nw049GkXyb1lJ8A/pV 1Gwkr5qjs9VV3xBsJRnPbwpdKn30Mcj+B5Kvlcx9Nybyb6Hm1x2TMWnQ4Fl8eRAe htS8tQ== Received: from mail-qk1-f198.google.com (mail-qk1-f198.google.com [209.85.222.198]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 4f3y9j3abq-1 (version=TLSv1.3 cipher=TLS_AES_128_GCM_SHA256 bits=128 verify=NOT) for ; Tue, 30 Jun 2026 11:43:50 +0000 (GMT) Received: by mail-qk1-f198.google.com with SMTP id af79cd13be357-92e6cb57d25so5893885a.3 for ; Tue, 30 Jun 2026 04:43:50 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oss.qualcomm.com; s=google; t=1782819830; x=1783424630; darn=vger.kernel.org; h=content-transfer-encoding:content-type: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 :content-type; bh=dVMvkTVLGH+FcIS/vfSvK/ujPY+UerDGY/JcdPljsRg=; b=BNmdRs8/PyYa+oAZWRmtrp5PnQSCOkqcWArZ1b0ZnXakrM5Rj8sPDW0Bj5SVxwNXGa 9QXMXRckW5xdyMw5CtT+cSID6H2+iPM47Edr3KkkEhcJmHukekxzNwOTIfP9haQVgOZ6 0uHM9SvafC95StRAf4h1lfUS8MgP6/VNgjCnrrH9neBvcjrk5qBEntLqmjXMLv+g7fdg FqFB0i6HHOoWC7XOnEa8l67NaUVjUJdmvUCWx85WV15VQ85eM7c+FAySj/9WGq9caRWG 0W8AgB0BVDChzIZzW/g23aTVvk+InWYnRm+DRPmno3cZJtOKH/Ioi1OSKZ/aoLZ5aw1Q iaAw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1782819830; x=1783424630; h=content-transfer-encoding:content-type: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:content-type; bh=dVMvkTVLGH+FcIS/vfSvK/ujPY+UerDGY/JcdPljsRg=; b=gtU+MmrEMh95H1pXB/6CV6wX/p/jdtzbcPPNQJ3b36N8shJ4u/Py+mCYjNfRpvVQ46 c0uhQbPzpaa/c+KoxWUs826L97+YkMmo6Ygidhkz9AxIsq04Y2q0NKgaIV6j5T+YmGEF ZOsk0QjGucy3Bot98vV4DSimBASVFgJMis/Et9rWEcq1qgMEndh6MUtds8trlotH0z1F 94/5Nnqm2C9EhZUzev3/Z6CiT+0uKRFsjoGYUP3OTLKk8VYr9CiwBrvfRT0x4EV2VG0e xsjgccYiISKS9QibY6QqvY6FU455z+hG4vTmIobqGJwpgi0GDaJxe4ToLq/NulVFIsbS ULeg== X-Forwarded-Encrypted: i=1; AFNElJ+fv70yejcGQvckcE8RuHeXrxDTFBNxoYGtfVGeSQ0Awx+7P8sLkjSHpwWZ0clgQs+4PPQzOaSQ1Vrp@vger.kernel.org X-Gm-Message-State: AOJu0YwufAKTAMAGmKc10emO4KK4Ih3Ahcnm9MKw7qvakuO7DxPowLd5 0e+IZ/rf4K9gAXpw9u0/YyzIHjCAPwvXp1EoPuKGOvW8VAhCiu/H2jU24j66Xf9Dq4Ta79MJPcn YzDjGLNPn2y4kBzTa/tt10HOn58RQgmyOjaBFOyTuyAdtjvlR59HFM19cYmL4apBc X-Gm-Gg: AfdE7clweiEFjCez07Y9NWzypFjPdcsid7s9vwRS1tGMEHjU5bxg20xY2WTa3IC8eiq 1b0yaBxc4YOomCA6uYh3LBpQn8jYI2eUck5L6eXtCX9Y4suPAq93jEob0ve+RNis+hezOrl4eeP 1NMazpZ4LVGmXFu9f6DWtlh8ajj4Ktdw16TsR9EDGe67IOvOdY2RluVhZtkuiitqGFOMIXqM7Xh lzlwAj6uhrQtIUGFsHRe4MruReOG5AtG2gaWq4uAIKAJQ1W1Mh1XunoWxe0szbKf6AouOHOSXrg MzPRfbyT75ZmtXLOPokp4EIJJhV3DotIEefJfCyBLbxzMS5HS2Eg8q8R9UhCy3Ar7S51lL3KTST bM7OeneGMH2VZGW3iufYsMZUlwdHAwj+fXlg= X-Received: by 2002:a05:620a:1a13:b0:92a:d546:ba04 with SMTP id af79cd13be357-92e62620d05mr347161485a.1.1782819830021; Tue, 30 Jun 2026 04:43:50 -0700 (PDT) X-Received: by 2002:a05:620a:1a13:b0:92a:d546:ba04 with SMTP id af79cd13be357-92e62620d05mr347156685a.1.1782819829433; Tue, 30 Jun 2026 04:43:49 -0700 (PDT) Received: from [192.168.120.170] ([178.235.128.140]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-c1288d18735sm113653666b.6.2026.06.30.04.43.46 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 30 Jun 2026 04:43:48 -0700 (PDT) Message-ID: <1416ada5-ccdf-400e-b2b8-d2c5c7e335dc@oss.qualcomm.com> Date: Tue, 30 Jun 2026 13:43:45 +0200 Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v3 05/10] ARM: dts: qcom: msm8960: add RPM clock controller and fix USB clocks To: Antony Kurniawan Soemardi , Bjorn Andersson , Michael Turquette , Stephen Boyd , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Lee Jones , Konrad Dybcio Cc: Krzysztof Kozlowski , linux-arm-msm@vger.kernel.org, linux-clk@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, phone-devel@vger.kernel.org, Rudraksha Gupta , Alexandre MINETTE References: <20260601-msm8960-wifi-v3-0-fec6ac8dba02@smankusors.com> <20260601-msm8960-wifi-v3-5-fec6ac8dba02@smankusors.com> <1d15a420-7360-429e-a451-ec1f012a0346@oss.qualcomm.com> <417600e8-9be7-4c6e-a62c-f10a5fca7a9a@smankusors.com> Content-Language: en-US From: Konrad Dybcio In-Reply-To: <417600e8-9be7-4c6e-a62c-f10a5fca7a9a@smankusors.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-Proofpoint-ORIG-GUID: 3_DMx7v7mxz2XoW129mS4CZVktzUR-ut X-Proofpoint-Spam-Details-Enc: AW1haW4tMjYwNjMwMDEwOCBTYWx0ZWRfX7pFhxP0LCgGQ wqnzJwwIg1cxA+//eM/O4Dhw6lvq26DIC2USfMpYpzqZFDtGFdjExnrIpJCledhTCl40eQX85Xk 6ch670vXyF5B79rRoEd7PUI31WGADoy55adH9qlgqU319Yzm+009wwnT+SPnJXraihI9/u23IK+ Go1bAyAsYSq/eVKz7s7CmUvl3FNVSW0aNevpxZCX0AzehZw8XYgwvSsBHig6MRgpigsWJsDcJKv nFMUjrL7TGpsFuJQJBHZ76hjxCvto/vXrSYmbyZz8P0TpCIAL1RW5Svf0eydxIzm6VCV8zkSsXH NTweWMFNgkT+V/sY+WhBfQkirP7fL73T8nTsEPq1IkFEPaF7uL/4CJ2fhiAYpol7iet/iKpmx5f rHDwtbFAGDbuahJ/muN52PqynPR6g1Lar6e63Qx9WX5C3PdSH9YNWNqNgnGgr0rMh4UunumTQIj v8zIs/PXOS2f8B1ksdA== X-Proofpoint-GUID: 3_DMx7v7mxz2XoW129mS4CZVktzUR-ut X-Proofpoint-Spam-Info: AW1haW4tMjYwNjMwMDEwOCBTYWx0ZWRfX1khLDNzv/1LA xpFzJhO36CS7iX3X9SKBDJi1y649RIOxdll3bFFHYUiSmOrTX0lDyYT6iiauOHYHdpIUi6yLkFr DzIa/ZYCsiI8IZum3ZCM+343G9VeSKg= X-Authority-Analysis: v=2.4 cv=SINykuvH c=1 sm=1 tr=0 ts=6a43abf7 cx=c_pps a=qKBjSQ1v91RyAK45QCPf5w==:117 a=PRfkaYvzSr8QmIIGAkY2Sg==:17 a=IkcTkHD0fZMA:10 a=FelO9ux0wxsA:10 a=s4-Qcg_JpJYA:10 a=VkNPw1HP01LnGYTKEx00:22 a=u7WPNUs3qKkmUXheDGA7:22 a=DJpcGTmdVt4CTyJn9g5Z:22 a=wxLWbCv9AAAA:8 a=iVYy-9Xbl0-MiRC387cA:9 a=3ZKOabzyN94A:10 a=QEXdDO2ut3YA:10 a=NFOGd7dJGGMPyQGDc5-O:22 a=QJY96suAAestDpCc5Gi9:22 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-30_03,2026-06-26_01,2025-10-01_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 suspectscore=0 malwarescore=0 priorityscore=1501 spamscore=0 phishscore=0 clxscore=1015 bulkscore=0 lowpriorityscore=0 adultscore=0 impostorscore=0 classifier=typeunknown authscore=0 authtc= authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.22.0-2606150000 definitions=main-2606300108 On 6/29/26 9:41 PM, Antony Kurniawan Soemardi wrote: > On 6/16/2026 8:43 PM, Konrad Dybcio wrote: >> On 6/16/26 3:04 PM, Antony Kurniawan Soemardi wrote: >>> On 6/9/2026 7:21 PM, Konrad Dybcio wrote: >>>> On 6/1/26 10:51 AM, Antony Kurniawan Soemardi via B4 Relay wrote: >>>>> From: Antony Kurniawan Soemardi >>>>> @@ -507,8 +519,12 @@ usb1: usb@12500000 { >>>>>                reg = <0x12500000 0x200>, >>>>>                      <0x12500200 0x200>; >>>>>                interrupts = ; >>>>> -            clocks = <&gcc USB_HS1_XCVR_CLK>, <&gcc USB_HS1_H_CLK>; >>>>> -            clock-names = "core", "iface"; >>>>> +            clocks = <&gcc USB_HS1_H_CLK>, >>>>> +                 <&rpmcc RPM_DAYTONA_FABRIC_CLK>, >>>>> +                 <&gcc USB_HS1_XCVR_CLK>; >>>>> +            clock-names = "iface", >>>>> +                      "core", >>>>> +                      "fs"; >>>> >>>> The bindings change you sent changes the expectations - "core" used >>>> to be the first clock. And I would guesstimate that the >>>> DAYTONA_FABRIC clock is not really "core" - does downstream do any >>>> ratesetting on the other two? >>> >>> Looking at the downstream, I can only find HS1_XCVR being set to 60MHz, DAYTONA_FABRIC being set to the max rate (just for voting purposes?). I don't see any clk_set_rate for HS1_P though. >>> >>> Would you rather the other way around? Like "core", "iface", and "fs"? My concern is that such a change would result in a large number of warnings for newer SoC device trees. >> >> I didn't notice you're actually aligning the order with bindings. I was >> under the impression this was a random change. >> >> For the clock assignments themselves, I think the schema reflects a >> full-speed (i.e. usb 1.x) core.. I dug out some ancient doc that says >> that we should have: >> >> - ahb (bus clock - perhaps daytona in this case?) >> - system (core clock for the thing, >55 Mhz for compliant HS operation >>    or at least 35 MHz for any sort of operation) >> - ulpi_clk (60 MHz, coming from the USB PHY) (we can probably ignore this >>    in our description) >> - inactivity_timer since there's a BAM instance attached to this host >>    (possibly handled implicitly) >> >> for the record, there's 4 hosts: >> >> USB1_HS @ 0x12500000 (this one) >> USB2_HSIC @ 0x12520000 >> USB1_FS @ 0x18000000 >> USB2_FS @ 0x18100000 >> >> I don't know if they are all exposed and functional though > > Hmm, I'm not sure which option is technically more correct. > > I did try swapping the clocks, using daytona as the iface clock and > `hs1_h` as the core clock. USB still works as expected in Sony Xperia > SP. They aren't handled very differently in the driver, one's just enabled slightly ahead of the other, so I wouldn't expect it to surface any issues (although perhaps it would be good to know whether downstream e.g. does ratesetting on any of these clocks to save power when the full capability of the host isn't necessary) > If that's what you want, let me know. I'm also curious whether others > agree with this approach. Dmitry? Konrad