Received-SPF: pass (google.com: domain of linux-kernel+bounces-72031-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1;
Message-ID:
 <SEZPR06MB6959B7D4949E1D81FB57052796512@SEZPR06MB6959.apcprd06.prod.outlook.com>
Date: Tue, 20 Feb 2024 05:50:44 +0800
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH RFC v3 1/5] dt-bindings: 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>, Shawn Guo
 <shawn.guo@linaro.org>, Philipp Zabel <p.zabel@pengutronix.de>
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: <20240220-inno-phy-v3-0-893cdf8633b4@outlook.com>
 <20240220-inno-phy-v3-1-893cdf8633b4@outlook.com>
 <1ed26e53-9f92-47a8-857d-777da512ee12@linaro.org>
 <SEZPR06MB69599A79CA066DAEC73BCEF796512@SEZPR06MB6959.apcprd06.prod.outlook.com>
 <5169eb03-a27f-4948-a175-50bba4e78f06@linaro.org>
From: Yang Xiwen <forbidden405@outlook.com>
In-Reply-To: <5169eb03-a27f-4948-a175-50bba4e78f06@linaro.org>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-MessageSentRepresentingType: 1
X-MS-PublicTrafficType: Email
X-MS-TrafficTypeDiagnostic: SEZPR06MB6959:EE_|KL1PR06MB5993:EE_
X-MS-Office365-Filtering-Correlation-Id: a7197071-411f-4c69-fe81-08dc3194d310
X-Microsoft-Antispam: BCL:0;
X-Microsoft-Antispam-Message-Info:
	Vsf3H3JZqv+h5jYFOcAT50Ix+XeUNkxcsVpjKNPAFBSjF3E612v53DKs5wfp3Dl5ho7KTILxnLIeZXEYce4WLja0Wk1MqSRkxmiO196W38sCJcRevYdKFi3KZZ1cf8uTC7KkILiQbZNo6yC0VP6Cza49ACfNPP+m1H0Vfu1sFiCmTW7TUYUhuaO2Y00SnXBdFjYgAnh9Lv3peUooII4fJQUiyU2kkB/cVCYPjcddwIUw2pK5FsrxfvfML2vyehKvIw3LKw3qfMFkzJZdYR4/dnCwTV6ebK+A2gYLFWa/WROP7JLwKz5HHQnJ5cqVd/O1Mug2sdDHe6ky6FKsGUJC+uzq4z5tmASO9/6SZfQVOT9nFyM8vyZBJSlwTiPgBg5hMN19RtwPqlpxh8JmYV9Sjm/jHGJAlIu5n7pMpm6m0Jmz6G7bjwx38gFLWu7mS6Pf7vuL4Do5cKKb0ccesxNxpD0fTsThPW1zgdWfDQo0Bip6UED2Z+op8wdAW3Q3OgwSd6pdGJJeI82ALIxeo9K0W81+v2wCihlPTXXKe4174kU04eDwKckhnI8doKYiFOCq
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?aHNHSm1qSjlPcWRxdGE5Y0ZYVUlNYUJrb0VTViszeGN4S3Vmdis1S0Fud0ZM?=
 =?utf-8?B?OGs0bUlSaWRsbFEyWmhTbXF5WEdVM0tBUmxRRld5aUhNMUtrRVYycUpwcnEz?=
 =?utf-8?B?ck1KRWNUV2JWc0NpWHBxUlVyOU4rVlh5b0lhOXFJMnVGNnpqbEV5WWxJMFJx?=
 =?utf-8?B?TWZ5ZW9aME4rN09iajJ0THpBZTJIa1lVMFd0MnNQZThRNnY1VE9hN09xbVkz?=
 =?utf-8?B?WDdIYmkzYjJwZlFESnMvSHVFdXRnUTc2bEwrb1M2dmJRU2VjcGg5TUo5WURs?=
 =?utf-8?B?dlo3TEFQVU1vTHl3UlJ3dmx3U3pJa3RDM0czczZNaHdnU1hoK0UwT1lacjBZ?=
 =?utf-8?B?UkVaZFBxZ1pJeUJ1c0F6RnU5d2hwc2dXaDVuN3JxSm9OQXl0M1VEWkt2N2lE?=
 =?utf-8?B?M0g5SDV2S3dwWlYvTHRzVEFvQ3VSa2k2a05obW5DQU9za0xLTm9UQm9RNng4?=
 =?utf-8?B?YlZsOVkzOTQxZkEwUUNQRUlFNUlOcTEwZXYyejVJYjdEMy9LU0plRGNsTmY1?=
 =?utf-8?B?ak16UUJRQWxVd0gyQ0V1Y0NXaE9NY2tjUWI5Mm1SL2ZkeWlldGo0OGtBS2Vh?=
 =?utf-8?B?aDZTREs5WkpCVDRWdjNjcExLVEpsK1kyNTcwQ2ZENzN0bzcrWGhlRUVuY2hX?=
 =?utf-8?B?eTB3UDJMVmdSZXBJQjd0SHhxK3puV2EvWmhMaitCR1FBM1BobzBGOEFOZnhR?=
 =?utf-8?B?MXhlVTJCWjYyRzdSVGZDLzhZZjdBeStRWWJ0S0t6UDV1M2htemswK3BuNDBr?=
 =?utf-8?B?dk1ac2M3enA5bGY1RmcxUTdTell0eG8zbE5vUE03VDN1TWtrTlVuNXlsZm4w?=
 =?utf-8?B?RVcxV1hUR3c4K0oyUFYyOHQyNExVbHV0KzlBTXh5d0x4UktjT3pPbXo5S2ht?=
 =?utf-8?B?K2IyYitISHZQRUM5cU1oYmVhN0h4dXVycTByT1cyOEVXa2xnalZwN1JWSmZO?=
 =?utf-8?B?a1JORFQ0Q2VBNW5GRTNGTDB6VlRIeEVGSGxSV3NhcG9NM3kyMm05NXBKQ3dt?=
 =?utf-8?B?WUJTaEhjaGVnS1RNYVdNb1pLSWZjcWhIR3ZFemlNeTR0VUhOQmxYMWszMk5u?=
 =?utf-8?B?bWRGL2FLdTlTakJHOURwUVRsaklJZGVuTTJpaHFWMnJJaGExTjkvNm9SYTVi?=
 =?utf-8?B?VzJPc3ZQVlk0ZzlnbmJIMEV0TDcrVURQYU9abW1Ga2RUc3lrTEFZQTJZbFV5?=
 =?utf-8?B?SEh0VldFY2U2MWFEbXVrODZ3NlNxZnJSR2U1Q2RpU29jZXF4THBEZWF2L1ly?=
 =?utf-8?B?S05raEk1TWVIRnkwU0pGNXcrSGVnbmgrajMzcjdBUlFEejNxam5Mc2trbzZo?=
 =?utf-8?B?NTlOaENiNGJ2Z25tWkhiaStmVzgwZ3dlQy9oTDFLSVp1cWQyK0pxZm9kM2di?=
 =?utf-8?B?NGs2QUQ1dGdlaXhoVm1GZ1NFcU5Vb0pkbktjdk94ZHpsRWRSQlMxcnR4Zzho?=
 =?utf-8?B?a2FRYmFuRk1jakZ6V3NwY3Z1eG9uOEpPN3FlQXRMellXOXFhNlcvd2NIdW94?=
 =?utf-8?B?SlZuYzRHMVEzWUVuMFdnYXB4bXZuam9ORWJ2N0dkOHM1M0MyRWNvK2J2TURQ?=
 =?utf-8?B?cWZhSURUZXBEZXUwS3RBMzViN0s2T2Y5L1kxY1dLZzZtbi8zbTNhaDBPdGhv?=
 =?utf-8?B?amVNdGZaL0dURzUrZFJoTm9nSFk4MmF3STZrUTltYTlQZGZjTXBUT2plZi92?=
 =?utf-8?B?bnZXQ0FjMGt5bGxobEdCVkRSKzNoUDhsOCtOVTBtMXF4T04rZjlNMHUreU9t?=
 =?utf-8?Q?hvT3IZ3F2cWyCjWyoCFozVkGXo78SyCT+//kayo?=
X-OriginatorOrg: outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: a7197071-411f-4c69-fe81-08dc3194d310
X-MS-Exchange-CrossTenant-AuthSource: SEZPR06MB6959.apcprd06.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 19 Feb 2024 21:50:44.7007
 (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: KL1PR06MB5993

On 2/20/2024 5:36 AM, Krzysztof Kozlowski wrote:
> On 19/02/2024 22:33, Yang Xiwen wrote:
>> On 2/20/2024 5:30 AM, Krzysztof Kozlowski wrote:
>>> On 19/02/2024 22:27, Yang Xiwen via B4 Relay wrote:
>>>> From: Yang Xiwen <forbidden405@outlook.com>
>>>>
>>>> Add missing compatible "hisilicon,hi3798mv100-usb2-phy" to compatible
>>>> list due to prior driver change.
>>>>
>>>> Also rename to hisilicon,inno-usb2-phy.yaml and add this name to
>>>> compatible lists.
>>>>
>>>> Fixes: 3940ffc65492 ("phy: hisilicon: Add inno-usb2-phy driver for Hi3798MV100")
>>> I don't understand what is the bug being fixed here. Binding being in
>>> TXT is not a bug.
>> This commit adds the compatible "hisilicon,hi3798mv100-usb2-phy" to the
>> driver, but binding change is left alone.
> Then this patch is not a correct fix. It does something else then adding
> missing compatible. Remember that one commit is one logical change.
Okay, that's fine. I'll break them into 2 patches.
>
> Best regards,
> Krzysztof
>

-- 
Regards,
Yang Xiwen