Received-SPF: pass (google.com: domain of linux-kernel+bounces-68229-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:
 <SEZPR06MB695938B228E762B9B53BAF2F964C2@SEZPR06MB6959.apcprd06.prod.outlook.com>
Date: Fri, 16 Feb 2024 16:21:24 +0800
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH 2/6] net: hisi_femac: remove unused compatible strings
Content-Language: en-US
To: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
 Yisen Zhuang <yisen.zhuang@huawei.com>, Salil Mehta
 <salil.mehta@huawei.com>, "David S. Miller" <davem@davemloft.net>,
 Eric Dumazet <edumazet@google.com>, Jakub Kicinski <kuba@kernel.org>,
 Paolo Abeni <pabeni@redhat.com>, Rob Herring <robh+dt@kernel.org>,
 Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
 Conor Dooley <conor+dt@kernel.org>, Yang Xiwen <forbidden405@foxmail.com>,
 Andrew Lunn <andrew@lunn.ch>, Heiner Kallweit <hkallweit1@gmail.com>,
 Russell King <linux@armlinux.org.uk>
Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org,
 devicetree@vger.kernel.org
References: <20240216-net-v1-0-e0ad972cda99@outlook.com>
 <20240216-net-v1-2-e0ad972cda99@outlook.com>
 <68c9477a-3940-4024-8c86-aa6106e8a210@linaro.org>
From: Yang Xiwen <forbidden405@outlook.com>
In-Reply-To: <68c9477a-3940-4024-8c86-aa6106e8a210@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_|TYSPR06MB6750:EE_
X-MS-Office365-Filtering-Correlation-Id: 755bf9f6-a5e7-4b5f-05f8-08dc2ec845dd
X-Microsoft-Antispam: BCL:0;
X-Microsoft-Antispam-Message-Info:
	9Sf/yegB8G8ViZbhyCvKJIRieJtXmGc82yL6DzBC7j441XcScmEJqdW/SE7LjtCVq2IcRAJwEcHxq40ay2CmKNdKuZSbITxpmecXlm9w7bd5sNaI/OujZwcJTJgBZMqsGCsR05xeQ+0CVsAMuNRaPtXxY8QNVtdf/uz/Jkeyzg2O9IQGeIrKwHtZn2fwt5Kwa222jmmzY16CBSq0CPaLT1UwN7I1D0UbEubUQkzZfJYLPdOSbg443SL4cMrtqiN2H0dZnAO79whhFH1d7pRSYJ4/tulLXTFZJlHEKPZk5ZtwPSNScC2aQce3KYybZ5wtDdrllE7sxNX6fVRiB1+X6vjzxdSFU6ugTug5dml+ihIuC8VIAYn9z9Sl4wU7yjevSFMSVBBtokqfgUpMwaJ2MyaDh4CaO1iPpVPlDu9/WjHLN7ClSoVfH0AVORFHFV+SEzJencSRDKZz3L6heLRR4FwLcDa6rz8Jt2AKX3MdI1KcHK0JutX4wZci06wTtXCQgy/6HrGfXgYO9mXonWHT6O2Trxs6yM8WyQu28VywlOgOaxzOg+A9ySPCDrVPfo/q
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?M0h2d1VMQzR3NElPUnkyT2JEcjBoOG04Um9JSWE0ZXZySGtMUG14K2hkS01Q?=
 =?utf-8?B?Z3NqUGIvV01FVTdwTnIzeGYxZWZSdGZTZFdhTnpSZUdqVkxIT3BvSlBIR2dI?=
 =?utf-8?B?SHE1RTBMc0drZDk5VmZObFBYbEx4VjBjNFpzMFpJdHlSZXpLalVUTHlFRC9P?=
 =?utf-8?B?V2VDMTkzeDJOOExBaVhad0lKRnlEcUsxTkY5TEU3RUZQN0ZnUDdzQ3F3cGN4?=
 =?utf-8?B?VEZiVVk3QnhxRCtwN0FDSkU2YWdFN2Nyc0t2aVpoZDNDeXp3NG1jZ1N1aFhP?=
 =?utf-8?B?R2lsMy9sSDRTTU9UQ01ZUmFCaEU2c3UwRWFKaFg3b1hQbll3MjJlMGxCaUpy?=
 =?utf-8?B?dGRzSnN6aS8wbVZjQUJPdm4vMlJqTFIrSXdwOGN0OE1NNW9JbWtwQy9CMmNF?=
 =?utf-8?B?aktJS3FpWWNEckErNHJsMysyZFd3MkFCbUVRTm96SS9tTlhHTE12TzV4OFMw?=
 =?utf-8?B?eC9OVmJBRmxuK2F0RlVIV2pYWXRPYUJEYkYvWURXRFJaRE9BUVg2YmdnNm5i?=
 =?utf-8?B?RzNLalA1RFFpZWY3M0tjYlpUUG9ON0dCUXIyQmJURmlXZ1ArUHViUFQ3YUJy?=
 =?utf-8?B?b0lqaDdrSmtvMGFhTUdCL20wSkltWVozWk1zc1NnOHhoQTRQOG05NDAyN3VF?=
 =?utf-8?B?M2NJRENUSFlCMzJCOGlFbk9Tbkp4dG91bVZwMnBsRVgxd1E0eE1xR0swUDBJ?=
 =?utf-8?B?alVvM3N4SHVtS2czeW5MSXR1OURSRUVvL2wxeEh5QkRxZHBLZkUzS1Y3andt?=
 =?utf-8?B?VXA0LzdpdXJPL0F4dTFZVmhiVWVsaDFwQ2FuMlBYUHdvaTl5ZUdsWFZFcERT?=
 =?utf-8?B?RTM1Znh4ejdXdDNJeUZFeUNEY0xkOUViUi9yQzFoOGNNZERDeXBUZEFlUDJY?=
 =?utf-8?B?NGpyZUFXTlNZODlHanhZeWFFVUZBNDNvclBtbUorQlQ4bi9VTzdyenlCNXZE?=
 =?utf-8?B?OEtDL0xtWU1JcHJ6ajVnb21ZZWNOb1U3dW1LTGc1SUJOM3VrbHZMN0Jaenpq?=
 =?utf-8?B?U0pDbWhWWVZ6aUhTWThJWHVkZVVLcDVuL1JLUGNtOHQ1RllWbDJXcTA5aEJC?=
 =?utf-8?B?UXh3QjBwVHZsUGlzU0JOaDh1SDlSdk1nelNvUDh6ZWdXcDIrQWVWMitQbThH?=
 =?utf-8?B?YWJYdWZFZ2RXaE03SE00YVdDNUd3dm91YWdTZEJOaW1qeVJxcnZaOVFpUnBP?=
 =?utf-8?B?MzZ0YitzU1FwL21CdjJVUnBtT1NmMWlvTVFXdW4rc1B3L0wrUTZQSzVLbHRH?=
 =?utf-8?B?Tm1XNDVmY0xOeloyc1ZkZzQ5ZWVUc3JmNG5UOCtyMmh2TUk3QVJmcFhzNWl3?=
 =?utf-8?B?VTdDMHR2NDBkd3Uxbll3TkE0VC9ZanZhSzVObXlPVFlNb2wzVmgzOTRIS21H?=
 =?utf-8?B?Nk85ZkYxQXlURkVseGd4dWVxWkhPREd6T1lFTFpXUUdUOEFUSHhoWEdGN3JK?=
 =?utf-8?B?N2RINVphTmx3SHdVRmlEY1hCNlNkSUUwa2daSnUvUEgvdllzK3BYYkhVSytY?=
 =?utf-8?B?TjF6MTZnd0orQ3FoQUloNkF4NkE1THh6Y2tpWlVQbksrQXNmSFMrM1M3Q3Fo?=
 =?utf-8?B?RFNrempZbDJQNnhLZXhNZGZVVTVjNGlDQjVhaFZINWFwdXhwQ1k5c1FPM3Zq?=
 =?utf-8?B?YW50VzVoVjlOSitBNDZ4bFBoMDE4T1RaMzZkUmcyMUtRb2tBNllrcjNSYUdj?=
 =?utf-8?B?RS96bnk4T3M1R3dxdnFVL0s1bXdObjFTaXBnVWdPMEdkQTIyWmlFd2dtTFNX?=
 =?utf-8?Q?RfQcQ4Ms20CmjQ4k2sj3ngiiuucaRb/CUpXDm7j?=
X-OriginatorOrg: outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 755bf9f6-a5e7-4b5f-05f8-08dc2ec845dd
X-MS-Exchange-CrossTenant-AuthSource: SEZPR06MB6959.apcprd06.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 16 Feb 2024 08:21:28.1590
 (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: TYSPR06MB6750

On 2/16/2024 3:20 PM, Krzysztof Kozlowski wrote:
> On 16/02/2024 00:48, Yang Xiwen via B4 Relay wrote:
>> From: Yang Xiwen <forbidden405@outlook.com>
>>
>> These compatible strings are not found in any mainline dts, remove them.
> That's not a real reason. What about all other users?
The people who want their devices being supported should post a working 
dts first. Having found the dts missing is strongly telling me that this 
SoC(Hi3516) is orphan and EOL already. I can't even find it in git 
commit logs. I'll argue that the old binding is simply wrong, and does 
not describe the hardware properly. Who knows? Could anyone tell me if 
the driver is still working for Hi3516 or not? I'm very willing to keep 
the backward compatibility if someone can tell me the effort i paid to 
maintain the old binding really makes sense. But the only things i found 
in mainline kernel about Hi3516 is an CRG(clock) driver and this femac 
driver. And it's been 8 years since last update for this SoC.
>
> Best regards,
> Krzysztof
>

-- 
Regards,
Yang Xiwen