Received-SPF: pass (google.com: domain of linux-kernel+bounces-69810-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
Message-ID:
 <SEZPR06MB695983231D04A2E1947AF61896532@SEZPR06MB6959.apcprd06.prod.outlook.com>
Date: Sat, 17 Feb 2024 18:54:51 +0800
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH RFC 1/4] dt-binding: phy: hisi-inno-usb2: convert to YAML
Content-Language: en-US
To: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
 Vinod Koul <vkoul@kernel.org>, Kishon Vijay Abraham I <kishon@kernel.org>,
 Rob Herring <robh+dt@kernel.org>,
 Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
 Conor Dooley <conor+dt@kernel.org>,
 Jiancheng Xue <xuejiancheng@hisilicon.com>,
 Pengcheng Li <lpc.li@hisilicon.com>, Shawn Guo <shawn.guo@linaro.org>
Cc: linux-phy@lists.infradead.org, devicetree@vger.kernel.org,
 linux-kernel@vger.kernel.org, Kishon Vijay Abraham I <kishon@ti.com>,
 David Yang <mmyangfl@gmail.com>
References: <20240216-inno-phy-v1-0-1ab912f0533f@outlook.com>
 <20240216-inno-phy-v1-1-1ab912f0533f@outlook.com>
 <63b3eff6-49eb-46f3-a6d9-878eddf6de53@linaro.org>
 <SEZPR06MB69593B898A42192D134B01A896532@SEZPR06MB6959.apcprd06.prod.outlook.com>
 <dfecd473-dcc1-4977-901d-6c7bb615bc97@linaro.org>
From: Yang Xiwen <forbidden405@outlook.com>
In-Reply-To: <dfecd473-dcc1-4977-901d-6c7bb615bc97@linaro.org>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-MessageSentRepresentingType: 1
X-MS-PublicTrafficType: Email
X-MS-TrafficTypeDiagnostic: SEZPR06MB6959:EE_|TYZPR06MB4015:EE_
X-MS-Office365-Filtering-Correlation-Id: b170a110-85f4-404d-fc6f-08dc2fa6e16c
X-Microsoft-Antispam: BCL:0;
X-Microsoft-Antispam-Message-Info:
	mnC3LzyfC8RN6UcZpQih9Oac3G8F+bKOaXAqZqTrdsvFbnzrWuPe8Pmf0DwW56X4Z3ccKqJVb4C1JvoAz1tx3JeVhK8VBHD6BWrf5lQEj9Ls2I7AMglYHxCFgwAPk9qXOZUQps0urXQiZBE0tAbFwS6QuvKXBuauV8tGslKFccjWokOWpQHhztOxFV2vajrJn66LzQFoF1DVkikzkEUM59EDAfHxW03udOLC3bwu52VBvIIemS4fk2YUIPjg0Fvu6myVFjZDD/jC8Sxv1jVoSZsMbUVEr2Mw+L6GLVhxMGsij90fU4FSsmBXHz4pElXd/qUIn8X0TENSAUIcGZD3tIpAiVaevTOEJaD9D3ZUfIGx7ERXOTrAExr6bOKKTakBHO91ebd1rQy6o9Pw4Xx+kqDZ35u2XU/XLXpHYqnzLkEpWZYKCNuupFpv7GMIH5o28F9S+KY1uvmsXUC/9CUqHUMmnkkYKblfk6pgUtdXxCvbsM2EVUX1TkMQg0AfcB7zBmnSANPlEUvUyCvGDO2/g5238yozXxRkzwqB5487kc611WhqIgm8KuA0kaQSUl+I
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?K2doS1hqU2t5bnR4eWNXalBtMGlKWmVFVjVZdkJUbkg3T2huZU9nWTBORHNq?=
 =?utf-8?B?aU9ZZGFTYmlhTTdsTzhqQ0RxT1RxWFVxd3VGQTRpOVFydC8zaXZIVWRiN1ow?=
 =?utf-8?B?WnhCbVpvU0duNGhMVVVDdFJpOHVGblhKWmpRZ2VybHlKYkc1V1Y3U2FtUlRS?=
 =?utf-8?B?SUNsYXJsU2UxbDEyTzA1OXcrdFZqZ2c3SGdTK3VCYThuTXFsMzU3SDhFUWQ1?=
 =?utf-8?B?K3pON0dmeWlzc3ZrVklYblVJcjBuS3gxNjBEYTBXSE9XazFYK01oYW0rKzA5?=
 =?utf-8?B?SjlKNGFBUVlsWVMyWE1vTlZvaXZjd3ZBb1FlUTVPaEM4dHU1d3hvSnNkQ2ds?=
 =?utf-8?B?RmJiY3pyK0FoTjZmT1A4MzlWbi9Tc3lOZldrSWJyNXhXU09IbTdWeXlHNUN5?=
 =?utf-8?B?R1k1WC9CbWFSQ1ZVd0VDTlRUTEFlSDIxblRvTlJuU2RuaTI3d0RyL20rbHh4?=
 =?utf-8?B?TVZqd3Mzcy8xYWY0dUR0eCtOc2ZHTE9zaTI5RDF4eW05ZFRIWWUrTHova2E2?=
 =?utf-8?B?dlk4bWdLQStvUkNEMkkwaU0vaXRiTjQ4ZlU3ZkRpR3BPV0pYeU1YQzVIU1o2?=
 =?utf-8?B?R096SWpYSGM4THRHMTlMUWNwVGZHR0FOTjNSMENQY2Nqc3Nxd3Z2L1A0aUxJ?=
 =?utf-8?B?R2hPbEp3UlUyc0RpeGt1RFZaYm93azF3aGNPRmlEZFlIM1RiaXA2Mjg2WkQr?=
 =?utf-8?B?NXM2ZVZwNkZTYVB5UENmcGpHT3Iyb2VseWx2b0VaOFhOdWMwWFZockxVbUJp?=
 =?utf-8?B?U05EKzdFcXRnNjNZTHdUc2tYdUxmejZ6SnFpdTJjV1YrSHR0NDR2QXk2SXVQ?=
 =?utf-8?B?TmV6dUs5djBOaEtXamUzVWwrcE04eHV5R1YvLzVadHJYQXA2SDdZQVg0enQr?=
 =?utf-8?B?eUZSOU03dVZTSkk1bGpRMXh6a2VHZTRxZzUyQS9ER1p0NWJVVmVoWG5HWkNR?=
 =?utf-8?B?dkFzQlRSOFZoVDNXSUFIeS9DV3BOVksySHFNVHBMamVGUGFacTI3ckxTSU8v?=
 =?utf-8?B?d0JGd0lNU0QrMW5McC9KWjYveG95M2lHanJUb3NKYitxb1lYZkRpM29RRlpy?=
 =?utf-8?B?YmJkU1hlSUdqZEppRjhXdUl2ejI1SmRidzN1THlRaXV1OEFXSmhwdmFGaWs1?=
 =?utf-8?B?UFJmUjlGOHZMNWxyeTV6SGV5VENVVjVQajNGRE1Tc0t3N2Z2dGo1UjJwdlBp?=
 =?utf-8?B?QnpqZEo4em5vR0tDb0p0eFJkdTlrZG50QWNMU1RXQmJab2hka3JUcjVOZEd4?=
 =?utf-8?B?QU1YakQycWdrUzNZYXl2cElqdmdNMFBJNjgvVlJrUnFiZzJkaUtDNnhPNWRC?=
 =?utf-8?B?MUNOZ2UvR3QrRWZLN0V4amRjRjJiWTZZOG9SZlBLMDk2RTlJZ09WWkZzTmt6?=
 =?utf-8?B?T0szR0k0SmFIYjdLZkx6dkU5TzkwUmFaQnNnb3FDRDg3bTJGdisxbU5qZ2Jv?=
 =?utf-8?B?L040UkJ4WWZQNXllbjRFZ1kvYmg4TlhGY3dUWHlpQTdOTjlDY05YdERiNUxT?=
 =?utf-8?B?TytpaUNxU2FhR0laemRtTXRwWTgzS1hYUFc5ejh4SFNuVlQ1YSs4RWd4Tk9Q?=
 =?utf-8?B?eG1SSUhtQUxPR1hpclM1OGdRb3VOR0MxRzlrc2xIZkM4VzRBQmMzVTJuaERE?=
 =?utf-8?B?VHd5ZFVmV1FOVktJdXF2ajg3bTBKSUlPWnNDMDRtZjZGUS9wa3N4UkdPT1NF?=
 =?utf-8?B?WklmanlQQndyanlIbjMvS0NldHlwUG50WkphbGhaVGFYQ2VzanhFSXdiTkIz?=
 =?utf-8?Q?jUiCErj9dUDk+FGeySDmHKnjiSwUWSoRa1Wu49m?=
X-OriginatorOrg: outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: b170a110-85f4-404d-fc6f-08dc2fa6e16c
X-MS-Exchange-CrossTenant-AuthSource: SEZPR06MB6959.apcprd06.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 17 Feb 2024 10:54:57.4350
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 84df9e7f-e9f6-40af-b435-aaaaaaaaaaaa
X-MS-Exchange-CrossTenant-RMS-PersistedConsumerOrg:
	00000000-0000-0000-0000-000000000000
X-MS-Exchange-Transport-CrossTenantHeadersStamped: TYZPR06MB4015

On 2/17/2024 6:29 PM, Krzysztof Kozlowski wrote:
> On 17/02/2024 11:24, Yang Xiwen wrote:
>
>>>> +
>>>> +examples:
>>>> +  - |
>>>> +    #include <dt-bindings/clock/histb-clock.h>
>>>> +
>>>> +    peripheral-controller@8a20000 {
>>>> +        compatible = "hisilicon,hi3798cv200-perictrl", "syscon", "simple-mfd";
>>>> +        reg = <0x8a20000 0x1000>;
>>>> +        #address-cells = <1>;
>>>> +        #size-cells = <1>;
>>>> +        ranges = <0x0 0x8a20000 0x1000>;
>>> Drop the node, not related to this binding. If this binding is supposed
>>> to be part of other device in case of MFD devices or some tightly
>>> coupled ones, then could be included in the example there.
>> For CV200, this binding is supposed to be always inside the perictrl
>> device. The PHY address space are accessed from a bus implemented by
>> perictrl.
> Every node is supposes to be somewhere in something, so with this logic
> you would start from soc@. What's wrong in putting it in parent schema?

When the ports reg property only has an dummy address (no size), it must 
be inside the perictrl node, accessed from the bus implemented by perictrl.

But when the ports has its own MMIO address space (mv200), it should be 
located under a simple-bus node instead.

So it's either:

perictrl@8a20000 {

     usb2-phy@120: {

         reg = <0x120 0x4>; // this is the register that controls writes 
and reads to the phy, implemented by perictrl. (just like SPI/I2C)

         phy@0: {

             reg = <0>; // the reg is used as an index

         };

     };

};

or:

soc@0 {

     usb2-phy@f9865000 { // MMIO

         reg = <0xf9865000 0x1000>

         port0@0 {

             reg = <0x0 0x400>; // used as MMIO address space

         };

     };

};

So here is why i include perictrl node in the example. If the ports are 
not mmio, the phy must be under a perictrl node. Or if the ports has its 
own address space, it should not be under a perictrl node, but rather an 
simple-bus node.

>
> Best regards,
> Krzysztof
>

-- 
Regards,
Yang Xiwen