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 E3A38C7EE25 for ; Thu, 8 Jun 2023 08:28:54 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234046AbjFHI2x (ORCPT ); Thu, 8 Jun 2023 04:28:53 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:34412 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232262AbjFHI2v (ORCPT ); Thu, 8 Jun 2023 04:28:51 -0400 Received: from mail-ed1-x532.google.com (mail-ed1-x532.google.com [IPv6:2a00:1450:4864:20::532]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id E3B5D2700 for ; Thu, 8 Jun 2023 01:28:49 -0700 (PDT) Received: by mail-ed1-x532.google.com with SMTP id 4fb4d7f45d1cf-51480d3e161so455072a12.3 for ; Thu, 08 Jun 2023 01:28:49 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1686212928; x=1688804928; 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=mecQ34dk2V9y6dj5rR+/RnPx6Aef7iQYAh5UFotv1xo=; b=YOuXdQwTL0HiqsuDJ2OMmCSusM9E5OEW/ovbK+fy8/Ndz8cb5RyF1n1QlhilBzqeuW 3gh1zTvG7WUB5wCr1NkVXEkhHxJz0Trn77gbdHJJognIYkXNmesScC2h0fmtNIqjfXbx x49RK2FCpm0zpnOYjCZCX+484Cl6Pci7GslevrHsQaDcqjg1AlluAsffRhUhYsAlPvKx bDSg5N7ErAgUIy61k/pu9d1PUZmuyRcsvKmYRiTlCcht9U3ykifmVK67IUhQx8dMq/uC fhw8bo73cYfBu2SDg/LVekXM9dRtn6e+a2whScQPYNpfoMrma0HY+C21DJONkNdMlhb1 iy1Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1686212928; x=1688804928; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=mecQ34dk2V9y6dj5rR+/RnPx6Aef7iQYAh5UFotv1xo=; b=GaS0D73r6EfYUduR9Gcw3CWOg5VMGAMJb2LmS+lD8mkFeP2s9ReRTzXdfGx35Y3juB piTqH8hhbysBf5oNcLcb4Df9cyNWwIxD7ZriiDIhIAxwMCpQh8DvbaZvw2M8A1l/dfAi AZkpewyfVXs/g+Y9Lrq/0GlMTftR/Qg3emR/srpyc/MjO2bbOY9LZ/D2NLyjnkQa45rF 4LiuHTdfrb+UAwVHblO7Q0dguqpRmNO0rB8IN8OtNRYDUraTn6YhmPciGzvn6IYa4FdV 0f10EusDpSIOmyL0Hyd8Tii7Nk8LLkXwzPgaasCzRAHrl3bTGvVgVzQcMQ8RZzW0Nn7j bGYA== X-Gm-Message-State: AC+VfDzBZ9pUEayPtVunymEEu46PvfiP9lyU8wV4AllWPezJ7aStENXt PWspVTkBFvG0uDsZ41WGWEFc+dB3LDu357Rw7eQ= X-Google-Smtp-Source: ACHHUZ5QenQx/QeD/48u6FI2zqKtnWEaRxUQn7l0QY2Xhr1/LIy4nn8DwP/ooeoVFbjEmpIbtoTK5Q== X-Received: by 2002:a17:907:3f9a:b0:96f:8666:5fc4 with SMTP id hr26-20020a1709073f9a00b0096f86665fc4mr10730952ejc.50.1686212928223; Thu, 08 Jun 2023 01:28:48 -0700 (PDT) Received: from [192.168.1.20] ([178.197.219.26]) by smtp.gmail.com with ESMTPSA id h20-20020a170906855400b009787062d21csm386574ejy.77.2023.06.08.01.28.45 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 08 Jun 2023 01:28:47 -0700 (PDT) Message-ID: Date: Thu, 8 Jun 2023 10:28:44 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.11.2 Subject: Re: [PATCH v3 4/5] dt-bindings: phy: realtek: Add the doc about the Realtek SoC USB 2.0 PHY To: =?UTF-8?B?U3RhbmxleSBDaGFuZ1vmmIzogrLlvrdd?= Cc: Vinod Koul , Kishon Vijay Abraham I , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Alan Stern , Ray Chi , Matthias Kaehlcke , Douglas Anderson , Michael Grzeschik , Mathias Nyman , Flavio Suligoi , "linux-phy@lists.infradead.org" , "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "linux-usb@vger.kernel.org" References: <20230607062500.24669-1-stanley_chang@realtek.com> <20230607062500.24669-4-stanley_chang@realtek.com> <7cce1d72-6b4d-9fff-32bc-942193388134@linaro.org> <8a88cbee5c6245f2941c700b2bb30697@realtek.com> <7df8ffb6-a544-d10e-5273-fd6c4b368b20@linaro.org> <7d503e3028a7487a9a087cfa061fff9d@realtek.com> Content-Language: en-US From: Krzysztof Kozlowski In-Reply-To: <7d503e3028a7487a9a087cfa061fff9d@realtek.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org On 08/06/2023 10:21, Stanley Chang[昌育德] wrote: > >>> Maybe I use the word "control power domain" is not well, I just want to >> control the ldo of usb phy. >>> Revised: >>> The phandle of syscon used to control the ldo of USB PHY. >> >> Isn't this still a power domain? > > I only control a register, it is not needed a driver of power domain. Aren't many power domains just a registers? What about other drivers? Don't you want in other driver control LDO of something else? And in other something else? > > >>> >>>>> + >>>>> +patternProperties: >>>>> + "^phy@[0-3]+$": >>>>> + type: object >>>>> + description: >>>>> + Each sub-node is a PHY device for one XHCI controller. >>>> >>>> I don't think it is true. You claim above that you have 0 as >>>> phy-cells, means you have one phy. Here you say you can have up to 4 phys. >>> >>> I mean the driver can support up to 4 phys. >> >> What driver can or cannot do, does not matter. This is about hardware. >> >>> For RTD1295 has only one phy. >>> For RTD1395 has two phys. >> >> Two phys? So how do you reference them when cells=0? > > > About RTD1395 SoCs USB > XHCI controller#1 -- usb2phy -- phy#0 > |- phy#1 > One xhci controller map to one phy driver. > And one phy driver have two phys (phy@0 and phy@1). > > Maybe the "phy" name is confusing. > This "phy" not mean a phy driver. We do not talk about drivers, but DTS and hardware. > Would "port" be more appropriate? > > For example, > Using phy@0 and phy@1: > usb_port1_usb2phy: usb-phy@13c14 { > compatible = "realtek,rtd1395-usb2phy", "realtek,usb2phy"; > reg = <0x132c4 0x4>, <0x31280 0x8>; > #address-cells = <1>; > #size-cells = <0>; > #phy-cells = <0>; > realtek,usb-ctrl = <&usb_ctrl>; > > phy@0 { > reg = <0>; So such child is a NAK... you have nothing here. But it's unrelated topic. > }; > phy@1 { > reg = <1>; > }; > }; > > Change: port@0 and port@1 > usb_port1_usb2phy: usb-phy@13c14 { > compatible = "realtek,rtd1395-usb2phy", "realtek,usb2phy"; > reg = <0x132c4 0x4>, <0x31280 0x8>; > #address-cells = <1>; > #size-cells = <0>; > #phy-cells = <0>; > realtek,usb-ctrl = <&usb_ctrl>; > > prot@0 { > reg = <0>; > }; > port@1 { > reg = <1>; > }; > }; This is not the answer. This is the provider. How do you reference it from the consumer. Upstream your entire DTS. It's frustrating to try to understand your DTS from pieces of information you are sharing. Also very time consuming and you are not the only one sending patches for review... Best regards, Krzysztof