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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 74FABEB64DD for ; Thu, 13 Jul 2023 04:16:00 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:CC:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=79fqF3UerBEkg8zL9EYxBO5zXBRYogD06n/2nXK2Viw=; b=uyHUK9NX5B1BN3 PpMNoBvSR8p8152bG9MxyQBxQqlC1IYlO9ETEwDmewGWMMK8I+1w/SvJ+Zgq7w+3d52b18Mie42ai xi+TguhnoNrHJbNF7PYUWX2XhCZaieLAES3SxRzgqxlJmzekf6bULGsn49gT9grER6rsTROmHmazI B4WAnZ4wMGF3LsPItHF5tmQ87m8gYHPUqXcnfaUBRD6s1b4vf4NMwybXsupTX/7fxPeJPT0j+UXk7 3bU/myr1meeHbYdU1Rp1NdNxiDV90rm5e64Z+W7Ejkll6QvA1Vgojrgo1QStc1/GD2gyHDko5tMid 1zYi3KcCU1y7+WEzzZ/Q==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qJnje-001tHP-1B; Thu, 13 Jul 2023 04:15:22 +0000 Received: from mx0b-0031df01.pphosted.com ([205.220.180.131]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qJnjb-001tGX-0c; Thu, 13 Jul 2023 04:15:20 +0000 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 36D2vfb6014648; Thu, 13 Jul 2023 04:14:54 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=quicinc.com; h=date : from : to : cc : subject : message-id : references : mime-version : content-type : in-reply-to; s=qcppdkim1; bh=lo5w9wU2QVl/tyJfc7XjBLXmBUMX7AYxuEXj6BLmjOw=; b=RfJI/uHotS5VcQYzMhAvjsVGoNiIK8lHB5pLA6Qg+J2uCLSvE94HUSv/FD2rCBBIgTyx bjzCVzpI8e8seTTIe+KUrPKzItIXxwf2ezh+wtgmHqDlsO9/utLqFeSV7rWtmTAtFVfk tyiTqK+nu+fA8fToeTuDstoQZxNgj/w/cs6HWg6XyfLA9/bvpxPFDMrsCDZlrEOP3RQs QoNCifMWsVgEPd/BwHdrIaWw0RWXFyTkARM1DuWXLTIGDdK0BhsGSxhzmZSIv5TF4jHb 3pZpk7Ee/JajV6jGPtFa3iP1nmKn3MfbYFB7hDiMAv8l9OzQGc+whcWSp3FKuCRDztIW 5A== Received: from nasanppmta05.qualcomm.com (i-global254.qualcomm.com [199.106.103.254]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 3rse45kama-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Thu, 13 Jul 2023 04:14:54 +0000 Received: from nasanex01a.na.qualcomm.com (nasanex01a.na.qualcomm.com [10.52.223.231]) by NASANPPMTA05.qualcomm.com (8.17.1.5/8.17.1.5) with ESMTPS id 36D4Eqjv023263 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Thu, 13 Jul 2023 04:14:52 GMT Received: from varda-linux.qualcomm.com (10.80.80.8) by nasanex01a.na.qualcomm.com (10.52.223.231) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.1118.30; Wed, 12 Jul 2023 21:14:44 -0700 Date: Thu, 13 Jul 2023 09:44:40 +0530 From: Varadarajan Narayanan To: Dmitry Baryshkov CC: Krzysztof Kozlowski , , , , , , , , , , , , , , , , , , , , , , , , , , Subject: Re: [PATCH v4 4/6] arm64: dts: qcom: ipq5332: Add USB related nodes Message-ID: <20230713041439.GA24906@varda-linux.qualcomm.com> References: <1f99805b6437aa8d6eaa4663e8d27b98ee595f00.1689160067.git.quic_varada@quicinc.com> <13555184-1708-befd-1f2c-5e6f7e04a6ce@linaro.org> <0f98360a-6d88-9a8b-5d60-3d6120e0640f@linaro.org> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <0f98360a-6d88-9a8b-5d60-3d6120e0640f@linaro.org> User-Agent: Mutt/1.5.24 (2015-08-30) X-Originating-IP: [10.80.80.8] X-ClientProxiedBy: nasanex01a.na.qualcomm.com (10.52.223.231) To nasanex01a.na.qualcomm.com (10.52.223.231) X-QCInternal: smtphost X-Proofpoint-Virus-Version: vendor=nai engine=6200 definitions=5800 signatures=585085 X-Proofpoint-GUID: AhuY4QKQugtt-AIl-fWparknQC9pZlTV X-Proofpoint-ORIG-GUID: AhuY4QKQugtt-AIl-fWparknQC9pZlTV X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.254,Aquarius:18.0.957,Hydra:6.0.591,FMLib:17.11.176.26 definitions=2023-07-13_01,2023-07-11_01,2023-05-22_02 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 adultscore=0 impostorscore=0 phishscore=0 priorityscore=1501 lowpriorityscore=0 bulkscore=0 suspectscore=0 mlxscore=0 spamscore=0 malwarescore=0 mlxlogscore=762 clxscore=1015 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2305260000 definitions=main-2307130036 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230712_211519_354972_B92698AA X-CRM114-Status: GOOD ( 22.77 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Wed, Jul 12, 2023 at 03:28:43PM +0300, Dmitry Baryshkov wrote: > On 12/07/2023 15:04, Krzysztof Kozlowski wrote: > >On 12/07/2023 13:38, Varadarajan Narayanan wrote: > >>Add USB phy and controller nodes. > >> > >>Signed-off-by: Varadarajan Narayanan > >>--- > >>v4: > >> Change node name > >> Remove blank line > >> 'make CHECK_DTBS=y DT_SCHEMA_FILES=qcom qcom/ipq5332-rdp441.dtb' passed > >>v1: > >> Rename phy node > >> Change compatible from m31,ipq5332-usb-hsphy -> qcom,ipq5332-usb-hsphy > >> Remove 'qscratch' from phy node > >> Fix alignment and upper-case hex no.s > >> Add clock definition for the phy > >> Remove snps,ref-clock-period-ns as it is not used. dwc3_ref_clk_period() > >> in dwc3/core.c takes the frequency from ref clock and calculates fladj > >> as appropriate. > >>--- > >> arch/arm64/boot/dts/qcom/ipq5332.dtsi | 53 +++++++++++++++++++++++++++++++++++ > >> 1 file changed, 53 insertions(+) > >> > >>diff --git a/arch/arm64/boot/dts/qcom/ipq5332.dtsi b/arch/arm64/boot/dts/qcom/ipq5332.dtsi > >>index 8bfc2db..8118356 100644 > >>--- a/arch/arm64/boot/dts/qcom/ipq5332.dtsi > >>+++ b/arch/arm64/boot/dts/qcom/ipq5332.dtsi > >>@@ -405,6 +405,59 @@ > >> status = "disabled"; > >> }; > >> }; > >>+ > >>+ usbphy0: usb-phy@7b000 { > >>+ compatible = "qcom,ipq5332-usb-hsphy"; > >>+ reg = <0x0007b000 0x12c>; > >>+ > >>+ clocks = <&gcc GCC_USB0_PHY_CFG_AHB_CLK>; > >>+ clock-names = "cfg_ahb"; > >>+ > >>+ resets = <&gcc GCC_QUSB2_0_PHY_BCR>; > >>+ > >>+ status = "disabled"; > >>+ }; > >>+ > >>+ usb2: usb2@8a00000 { > > > >So you responded to my comments, wait ten minutes and send v2? No need > >to wait for my feedback, right? > > > >No, it's not ok. This is "usb", not "usb2". Are you saying you have > >second device with the same address? > > Just to emphasise, it's the node name `usb2', which is not fine. DT label > `usb2' is (hopefully) fine. Thanks for the clarification. Will post a new patch. -Varada _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel