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 892C0C001E0 for ; Mon, 23 Oct 2023 09:11:47 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229936AbjJWJLq (ORCPT ); Mon, 23 Oct 2023 05:11:46 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:40106 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230136AbjJWJLo (ORCPT ); Mon, 23 Oct 2023 05:11:44 -0400 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id CE943D7B; Mon, 23 Oct 2023 02:11:39 -0700 (PDT) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 6FA22C433C8; Mon, 23 Oct 2023 09:11:39 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1698052299; bh=4X70b+y5Rv1VPzOWwLZG3sDuDOPKJsxum64a06GpX2s=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=bnQk+4OxI/6+02PUkt2DxouJgD2tN/LJ+U0iSXfJjHydL4jTxJxALCeIQtOOh24wd 5Lag11TF2iApRxIyikbrXeQbRbsYURMXJbE5Tnx0hbX8945cY/NupCGzt8vgvlOCMp tdR2NLep23Ii3z0p+P7AkjmTwcixVRwho95fuDkAHzgpxI5mqY+Nx8d2KhkBe46Q42 AANDVKC5FotJN8fz7cEU0oRQblQUe2WUgzXfTO4Bq+xBwX45+ecSPn+Fx1CbqKJcmx E+GvDm3h2K5FaRHTYUH+YBBlGCD0QGfef1FAr3v1pPDhb0SyTK80QM7d3TProwUryT gKmv3KIwYGVZA== Received: from johan by xi.lan with local (Exim 4.96) (envelope-from ) id 1quqyW-0007rt-2E; Mon, 23 Oct 2023 11:11:53 +0200 Date: Mon, 23 Oct 2023 11:11:52 +0200 From: Johan Hovold To: Krishna Kurapati PSSNV Cc: Thinh Nguyen , Greg Kroah-Hartman , Philipp Zabel , Andy Gross , Bjorn Andersson , Konrad Dybcio , Rob Herring , Krzysztof Kozlowski , Felipe Balbi , Wesley Cheng , linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, quic_pkondeti@quicinc.com, quic_ppratap@quicinc.com, quic_jackp@quicinc.com, ahalaney@redhat.com, quic_shazhuss@quicinc.com, Harsh Agarwal , kernel test robot Subject: Re: [PATCH v13 03/10] usb: dwc3: core: Refactor PHY logic to support Multiport Controller Message-ID: References: <20231007154806.605-1-quic_kriskura@quicinc.com> <20231007154806.605-4-quic_kriskura@quicinc.com> <257716c4-7194-4d26-a34c-fff09234628f@quicinc.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <257716c4-7194-4d26-a34c-fff09234628f@quicinc.com> Precedence: bulk List-ID: X-Mailing-List: linux-arm-msm@vger.kernel.org On Sun, Oct 22, 2023 at 11:33:52PM +0530, Krishna Kurapati PSSNV wrote: > On 10/20/2023 3:27 PM, Johan Hovold wrote: > > On Sat, Oct 07, 2023 at 09:17:59PM +0530, Krishna Kurapati wrote: > >> From: Harsh Agarwal > >> > >> Currently the DWC3 driver supports only single port controller > >> which requires at most one HS and one SS PHY. > > > > Should that not be "at least one HS PHY and at most one SS PHY"? > > > > No, I think it would be appropriate to say "at least one phy (HS/SS)" as > even one phy is sufficient to get things working. No, that would be a violation if the binding (even if the driver may not currently enforce it for generic phys) as well as the USB spec. Also note that your implementation (i.e. this very patch) assumes that num_usb2_ports >= num_usb3_ports. > >> Add support for detecting, obtaining and configuring phy's supported > > > > "PHYs" for consistency, no apostrophe > > > >> by a multiport controller and. Limit the max number of ports > > > > "and." what? Looks like part of the sentence is missing? Or just drop > > " and"? > > > >> supported to 4 as only SC8280 which is a quad port controller supports > > > > s/4/four/ > > > > Just change this to > > > > Limit support to multiport controllers with up to four ports for > > now (e.g. as needed for SC8280XP). > > > >> Multiport currently. > > > > You use capitalised "Multiport" in several places it seems. Is this an > > established term for these controllers or should it just be "multiport" > > or "multiple ports"? > > > This is an established term AFAIK. So I've been using it here like this. Do you have a pointer? A google search seems to mostly come up with links to this patch series. Johan