From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id ACD2AC4332F for ; Wed, 1 Nov 2023 07:57:31 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231311AbjKAH5c (ORCPT ); Wed, 1 Nov 2023 03:57:32 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:39952 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231269AbjKAH5b (ORCPT ); Wed, 1 Nov 2023 03:57:31 -0400 Received: from mx0b-0031df01.pphosted.com (mx0b-0031df01.pphosted.com [205.220.180.131]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 20A32DB; Wed, 1 Nov 2023 00:57:25 -0700 (PDT) Received: from pps.filterd (m0279868.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.17.1.19/8.17.1.19) with ESMTP id 3A15gWo7007920; Wed, 1 Nov 2023 07:57:14 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=quicinc.com; h=message-id : date : mime-version : subject : to : references : from : in-reply-to : content-type : content-transfer-encoding; s=qcppdkim1; bh=+Uoqj1rcN7c5r6Vl2OVzUHE5X0Lv4psbwP7FskDGH94=; b=BQq499aXYY36MYq+LDOyByDrHGdYfBOMPaB9BLypyWPfm1By5rGjEhHuhzgeqoEnFumf oEnB4URv3+laFTWC4bDqRm4vZb0q5nA8tHmTVE/7KD0FKMKd7qeRBZJA366BQi4LFB5P +8JVNrmUzEJ32ATkJr7MYJio0rnxOkzT4zBY4ZQwyuLjB5/gR4zXfWhoKDTj3Ygxb0fs 3xI+BvCwdpZikE4WZiryFCk8QBggjS2FCckBVYDrcZtrZUJN6AyZhdsLh40DGRqOsew8 ih2IC748au+9lkyL/r7eP1ubkdXU84T1qhx0Y+P3hOI7YtQ5j8yGOCp6MR93KM3QuAGw ww== Received: from nalasppmta03.qualcomm.com (Global_NAT1.qualcomm.com [129.46.96.20]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 3u3e06rwaj-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Wed, 01 Nov 2023 07:57:13 +0000 Received: from nalasex01a.na.qualcomm.com (nalasex01a.na.qualcomm.com [10.47.209.196]) by NALASPPMTA03.qualcomm.com (8.17.1.5/8.17.1.5) with ESMTPS id 3A17vCs5007667 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Wed, 1 Nov 2023 07:57:12 GMT Received: from [10.201.2.96] (10.80.80.8) by nalasex01a.na.qualcomm.com (10.47.209.196) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.1118.39; Wed, 1 Nov 2023 00:57:07 -0700 Message-ID: Date: Wed, 1 Nov 2023 13:27:04 +0530 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH 2/3] clk: qcom: ipq6018: add USB GDSCs Content-Language: en-US To: Robert Marko , , , , , , , , , , , , References: <20231025104457.628109-1-robimarko@gmail.com> <20231025104457.628109-2-robimarko@gmail.com> From: Kathiravan Thirumoorthy In-Reply-To: Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: 7bit X-Originating-IP: [10.80.80.8] X-ClientProxiedBy: nasanex01b.na.qualcomm.com (10.46.141.250) To nalasex01a.na.qualcomm.com (10.47.209.196) X-QCInternal: smtphost X-Proofpoint-Virus-Version: vendor=nai engine=6200 definitions=5800 signatures=585085 X-Proofpoint-ORIG-GUID: 8fGxbklvQLSbcOUKQmpKnESOJM_BBjVB X-Proofpoint-GUID: 8fGxbklvQLSbcOUKQmpKnESOJM_BBjVB X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.272,Aquarius:18.0.987,Hydra:6.0.619,FMLib:17.11.176.26 definitions=2023-11-01_05,2023-10-31_03,2023-05-22_02 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 malwarescore=0 spamscore=0 impostorscore=0 mlxscore=0 clxscore=1015 phishscore=0 adultscore=0 lowpriorityscore=0 suspectscore=0 priorityscore=1501 bulkscore=0 mlxlogscore=999 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2310240000 definitions=main-2311010066 Precedence: bulk List-ID: X-Mailing-List: linux-clk@vger.kernel.org On 10/29/2023 4:34 PM, Robert Marko wrote: > On Wed, 25 Oct 2023 at 12:45, Robert Marko wrote: >> >> IPQ6018 has GDSC-s for each of the USB ports, so lets define them as such >> and drop the curent code that is de-asserting the USB GDSC-s as part of >> the GCC probe. >> >> Signed-off-by: Robert Marko > > Unfortunately, after testing on multiple devices I hit the same GDSC > issue I had a long time ago > that was the reason I did not send this upstream. > It seems that USB3 port GDSC (USB0 GDSC in code) works just fine, > however the USB2 one > (USB1 GDSC in code) it is stuck off and USB2 port will fail due to this: > 1.607531] ------------[ cut here ]------------ > [ 1.607559] usb1_gdsc status stuck at 'off' > [ 1.607592] WARNING: CPU: 0 PID: 35 at gdsc_toggle_logic+0x16c/0x174 > [ 1.615120] Modules linked in: > [ 1.621712] CPU: 0 PID: 35 Comm: kworker/u8:1 Tainted: G W > 6.6.0-rc7-next-20231026 #5 > [ 1.624586] Hardware name: Wallys DR6018 v4 (DT) > [ 1.633867] Workqueue: events_unbound deferred_probe_work_func > [ 1.638556] pstate: 60400005 (nZCv daif +PAN -UAO -TCO -DIT -SSBS BTYPE=--) > [ 1.644200] pc : gdsc_toggle_logic+0x16c/0x174 > [ 1.651052] lr : gdsc_toggle_logic+0x16c/0x174 > [ 1.655566] sp : ffffffc081a03a40 > [ 1.659990] x29: ffffffc081a03a40 x28: 0000000000000000 x27: 0000000000000000 > [ 1.663384] x26: ffffff8000156ac0 x25: 0000000000000000 x24: ffffffc081824c90 > [ 1.670502] x23: ffffffc081824840 x22: 0000000000000000 x21: 0000000000000001 > [ 1.677620] x20: 00000000ffffff92 x19: ffffffc081824840 x18: ffffffffffffffff > [ 1.684738] x17: 00000000cdb32eee x16: 00000000d648a142 x15: fffffffffffc38ff > [ 1.691855] x14: ffffffc0817994c0 x13: 00000000000000e0 x12: 00000000000000e0 > [ 1.698974] x11: 00000000ffffffea x10: ffffffc0817f14c0 x9 : 0000000000000001 > [ 1.706094] x8 : 0000000000000001 x7 : 0000000000017fe8 x6 : c0000000ffffefff > [ 1.713211] x5 : 0000000000057fa8 x4 : 0000000000000000 x3 : ffffffc081a03840 > [ 1.720328] x2 : ffffffc081799400 x1 : ffffffc081799400 x0 : 000000000000001f > [ 1.727448] Call trace: > [ 1.734556] gdsc_toggle_logic+0x16c/0x174 > [ 1.736815] gdsc_enable+0x60/0x27c > [ 1.740980] genpd_power_on+0x180/0x22c > [ 1.744367] __genpd_dev_pm_attach+0x140/0x238 > [ 1.748188] genpd_dev_pm_attach+0x60/0x70 > [ 1.752701] dev_pm_domain_attach+0x20/0x34 > [ 1.756780] platform_probe+0x50/0xc0 > [ 1.760859] really_probe+0x148/0x2b8 > [ 1.764679] __driver_probe_device+0x78/0x12c > [ 1.768327] driver_probe_device+0xdc/0x160 > [ 1.772667] __device_attach_driver+0xb8/0x134 > [ 1.776660] bus_for_each_drv+0x70/0xb8 > [ 1.781173] __device_attach+0xa0/0x184 > [ 1.784905] device_initial_probe+0x14/0x20 > [ 1.788725] bus_probe_device+0xac/0xb0 > [ 1.792891] deferred_probe_work_func+0x88/0xc0 > [ 1.796712] process_one_work+0x158/0x2bc > [ 1.801226] worker_thread+0x2a0/0x4bc > [ 1.805391] kthread+0xe4/0xf0 > [ 1.809035] ret_from_fork+0x10/0x20 > [ 1.812075] ---[ end trace 0000000000000000 ]--- > > Kathiravan, do you happen to have any docs or info if the USB1 GDSC is special > and its status bits are broken or? Robert, sorry for the late response. Unfortunately, we didn't handle the GDSC via the framework, so not sure if something is broken at HW level. I can try to get some information on this, but immediately is not quite possible. Thanks for the understanding. Thanks, > > Maybe the offset is even wrong as I based it on the current driver. > > Regards, > Robert > >> --- >> drivers/clk/qcom/Kconfig | 1 + >> drivers/clk/qcom/gcc-ipq6018.c | 33 ++++++++++++++++++++++++--------- >> 2 files changed, 25 insertions(+), 9 deletions(-) >> >> diff --git a/drivers/clk/qcom/Kconfig b/drivers/clk/qcom/Kconfig >> index ad1acd9b7426b..2aefa2231b51b 100644 >> --- a/drivers/clk/qcom/Kconfig >> +++ b/drivers/clk/qcom/Kconfig >> @@ -164,6 +164,7 @@ config IPQ_GCC_5332 >> >> config IPQ_GCC_6018 >> tristate "IPQ6018 Global Clock Controller" >> + select QCOM_GDSC >> help >> Support for global clock controller on ipq6018 devices. >> Say Y if you want to use peripheral devices such as UART, SPI, >> diff --git a/drivers/clk/qcom/gcc-ipq6018.c b/drivers/clk/qcom/gcc-ipq6018.c >> index cc20a16d8973c..9d5ee2ac012a1 100644 >> --- a/drivers/clk/qcom/gcc-ipq6018.c >> +++ b/drivers/clk/qcom/gcc-ipq6018.c >> @@ -23,6 +23,7 @@ >> #include "clk-alpha-pll.h" >> #include "clk-regmap-divider.h" >> #include "clk-regmap-mux.h" >> +#include "gdsc.h" >> #include "reset.h" >> >> enum { >> @@ -4691,6 +4692,22 @@ static struct clk_branch gcc_dcc_clk = { >> }, >> }; >> >> +static struct gdsc usb0_gdsc = { >> + .gdscr = 0x3e078, >> + .pd = { >> + .name = "usb0_gdsc", >> + }, >> + .pwrsts = PWRSTS_OFF_ON, >> +}; >> + >> +static struct gdsc usb1_gdsc = { >> + .gdscr = 0x3f078, >> + .pd = { >> + .name = "usb1_gdsc", >> + }, >> + .pwrsts = PWRSTS_OFF_ON, >> +}; >> + >> static const struct alpha_pll_config ubi32_pll_config = { >> .l = 0x3e, >> .alpha = 0x6667, >> @@ -5138,6 +5155,11 @@ static const struct qcom_reset_map gcc_ipq6018_resets[] = { >> [GCC_Q6_AXIM_ARES] = {0x59110, 4}, >> }; >> >> +static struct gdsc *gcc_ipq6018_gdscs[] = { >> + [USB0_GDSC] = &usb0_gdsc, >> + [USB1_GDSC] = &usb1_gdsc, >> +}; >> + >> static const struct of_device_id gcc_ipq6018_match_table[] = { >> { .compatible = "qcom,gcc-ipq6018" }, >> { } >> @@ -5160,6 +5182,8 @@ static const struct qcom_cc_desc gcc_ipq6018_desc = { >> .num_resets = ARRAY_SIZE(gcc_ipq6018_resets), >> .clk_hws = gcc_ipq6018_hws, >> .num_clk_hws = ARRAY_SIZE(gcc_ipq6018_hws), >> + .gdscs = gcc_ipq6018_gdscs, >> + .num_gdscs = ARRAY_SIZE(gcc_ipq6018_gdscs), >> }; >> >> static int gcc_ipq6018_probe(struct platform_device *pdev) >> @@ -5170,15 +5194,6 @@ static int gcc_ipq6018_probe(struct platform_device *pdev) >> if (IS_ERR(regmap)) >> return PTR_ERR(regmap); >> >> - /* Disable SW_COLLAPSE for USB0 GDSCR */ >> - regmap_update_bits(regmap, 0x3e078, BIT(0), 0x0); >> - /* Enable SW_OVERRIDE for USB0 GDSCR */ >> - regmap_update_bits(regmap, 0x3e078, BIT(2), BIT(2)); >> - /* Disable SW_COLLAPSE for USB1 GDSCR */ >> - regmap_update_bits(regmap, 0x3f078, BIT(0), 0x0); >> - /* Enable SW_OVERRIDE for USB1 GDSCR */ >> - regmap_update_bits(regmap, 0x3f078, BIT(2), BIT(2)); >> - >> /* SW Workaround for UBI Huyara PLL */ >> regmap_update_bits(regmap, 0x2501c, BIT(26), BIT(26)); >> >> -- >> 2.41.0 >>