Received-SPF: pass (google.com: domain of linux-kernel+bounces-91750-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:
 <SEZPR06MB6959C96A0C09FBF47F1DDEF096222@SEZPR06MB6959.apcprd06.prod.outlook.com>
Date: Tue, 5 Mar 2024 14:51:24 +0800
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH net-next v7 9/9] net: hisi_femac: remove unused
 MODULE_ALIAS()
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>, 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: <20240301-net-v7-0-45823597d4d4@outlook.com>
 <20240301-net-v7-9-45823597d4d4@outlook.com>
 <724c1d96-6ec6-49c9-9714-502abd741252@linaro.org>
From: Yang Xiwen <forbidden405@outlook.com>
In-Reply-To: <724c1d96-6ec6-49c9-9714-502abd741252@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_|SI2PR06MB5164:EE_
X-MS-Office365-Filtering-Correlation-Id: 025f4197-80c1-4878-2af5-08dc3ce0aec3
X-MS-Exchange-SLBlob-MailProps:
	LVbdfIC7uFCGxI/zYMkvshTNNymXRuhs1NnaL5oKXsiSWiKElk5v9Hx9hfirO8h5M+ZpRsdOk9b/5jZybzYpks2iDHJbC/QnB2g116NZ8s1SSN4e0fvCzSM0IXZ0HGOXH+lr92Yv5QskyEciprXmPfO710WZpubX7cm4NxrHQeRA6KRoKQ5j7QbSCX/6syF8vd/bNPZNsjMDARkk8TCxC3SzOp9Lp74SNbOtMmYXaZjVIh5zOmPTJ/SbId6TZ8cRfljlKxbiI/4J3BfzcQ3/dWo1maHUlxTmsSfb2tCcTkbFH1JimflXQ6aCOWx977FCFRiCprx3X7zKjpdIjD+guDdi1dPBV9wxZxAuzUgbVqChAEwXafljOzMbqkX/BEPUQs7gmrxH6c9CLFwnqXj9kp8b68CYjl7doTkztxIS9/wOym17EYJ+AB4Yhn3taKvY1F+ljPYdL97ffwDb56gE6IifOBI4n2r1w/SgW4BK2qjEiL0O2WA6a6pvOcZTASIXdYuYJNZ0KFVA0uQj3IG6/rgIXshd4PSMR68tqNQJBmksH3qVp5aYH5lcZgrJY05zKykf4q8rwvFjZ+lerAiN2NUj5zQTv03QRj8hZQUeM5zfUpVzr5aP/H7RpbPqxWhbAVyy8KB4+5WNmXqLTsju9rQFhyaJbucDu1N+RMZXbYUDulHPrA8jQUTGDLG+UikBf1Hn9mg3pUCvtMClrc9mgP5Ru56PhND5gtOpNQgBiWUChyn9I9l4IQ==
X-Microsoft-Antispam: BCL:0;
X-Microsoft-Antispam-Message-Info:
	LgVrlyWkBopzHp/uNiqoXuSrC0L9SKl9dWY5xE8AUQdV3vuxwfkA7FSGYiG2DF3adu/VZztfV4Pyhc9JAhNB37SM1DJwD2ix+9+IwiG/7MIjS/GYsABq7oaZwRvKmSjpKf4B3tyA5b317JP6Htjns2Kqe0C3OR3NDcITtQAtmJIFFhR9Ed62gYeUJ18FpV3PrkZp4nOklHPKF/69IsSw7kn+f+iBQkhM9rvjocgIk6PqH36XVqA5sxAHUbMrqgU+F+ILFt3xT1bzPPEKzljO5vPVGOKRPybcrcIV7CkgPZpfZTu536i0Au+Z+IokwwmJPn+MN8Yah1hhH4Ql0eKyom2/4+LDfKeGHBbLM54yFDBga2t8vS4GieC/sg/9qem2jKMYrzaNGHMVAVbqO+ZAC8hOAMANDXaFvZLFXG8n9ELmPtp3BDKGkbzQhNqtI4ghMPqwRoezz2zbX/9cyAbV+T9ShjBhY9WuLcjGx2lFuAMGuVzGpG2/GWWYn2eb+l/MBsSA2Sl37dWzX8epw0sxMisygaxDSeJIsg/OaEmBDbIi075Izj9b+ARMFR3Rlb7u
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?a2NPdDlkVDU5eFduck92ZURETG96NHJYTzRoeFpMelA1TU1DUkZkOFlLNTIv?=
 =?utf-8?B?SXhkdEVzVkNpMXRKV0NMQWNYcExvY3ovTnFyTU5qZ2c3Mmo3Z2Z5YStNZWpB?=
 =?utf-8?B?Q1gyYnBlcEFySi9kNUhJaVpWeWF6WnFFTWsrUjA4RU9aT1ZSQ3FvMCt5UStv?=
 =?utf-8?B?TzVOL1VLL1VkR0x4Vk0zZDhxNE1rdVB6UjVHK3VBbW9iU2hZdGRaQlI1c3dj?=
 =?utf-8?B?TGlqZjFyb0duSHNCeVVqRG4yWWd5b0sxbjRIVGdpdndMMmZsMEFvRy9UbkNU?=
 =?utf-8?B?TUJHT1l0THR4SEFnMU93TUZKSVZnMTZCR3hja211akt1QVlaV0hjbUp6Rk5h?=
 =?utf-8?B?enNzWUZlMGx3T0hPN1NSYmlzOU1UZWdxR3VOYmRrQXVEL1F6aHI1WDM1R0J2?=
 =?utf-8?B?YS80VE1zbTNub2dtc3UySWxuUkgzUk1reEtJZ0ozdGd5NFZYbytXRU9vM25Q?=
 =?utf-8?B?eG1BUUtUSlpKS203eHd4MWtweVQ0VC9iUjNuWXNIQ2ZnU1BBMUtYeWs4TnRU?=
 =?utf-8?B?OXo1SnJYZUlyTkUzUUErckEra1pUMzMxRWFHeTRmVndCWlJ5cksvYTdYbDRG?=
 =?utf-8?B?RHpuMTNmZ1JYOHE3c2NCUFArVmZKQVBGT0J5MmhnUkUwV1FJcmgrOXM0SUQy?=
 =?utf-8?B?VHFrQnk5TWo1NUxlc3dHYzMram10WEdmM0dMTENyK2RtTVVpeW5wZk96eHg2?=
 =?utf-8?B?bjFSNGYrMDRkQ25YMUNpeThsRlJESnB3K01Ra2IzbUlYT3FPcjJBakpDbGNj?=
 =?utf-8?B?djNPUXFueGRtSVVFZUlHNG9wVGRrMExNRjJjRVdlSm5GNUlCK0hLQi9SLytK?=
 =?utf-8?B?bUpBa2ptK2VOdG91NWkxbHRMUm95NHBNWW9ZS0ZHdkp5QTQ5S1AzaHdkazJM?=
 =?utf-8?B?Z3ZuYmVwUGw0MXJzYUg5NXplRWEwZjA4MG0rT3hWRi94RjRZc1RWb3ZsYmp6?=
 =?utf-8?B?MVZmSDVNajJVYUozdytmb1VQQytFTFF3ZnZCdFlJRHFFYlFwV1RScy9VcGFM?=
 =?utf-8?B?enVQUWpYQXljSUlVLzY1SE1iS1loYXJSZGdmdGdtZTVmaVdOTzErek1GY05S?=
 =?utf-8?B?V3hUSHNPYVRFeXZwVWU1MHhLQjNod3pZRkFhS1oyVDBLZEZaOVBkcGFiVmE0?=
 =?utf-8?B?bWFuQlMzdlQvQkJ0RlYxQUkvVkd1SFJXWlFQbCs1ZHhpUStwVk0zZjFDWjNU?=
 =?utf-8?B?andVT2REZ2RWM0tPS0R4Zm5VNG15ODZhKzZlUXowbUczVVN2SGRMZm1vUVY0?=
 =?utf-8?B?UFJOUGRUY296UzV5bENBMmJNd3R6eGIrTGhRdkE1RHlaOGRRbXl5NW1ieHh0?=
 =?utf-8?B?NWpVV21JWGJtSFlrMG5ydk1ZNExCZ3F6L2MrdWJLRkU4MEhyZitnd3dvNE9V?=
 =?utf-8?B?UzdIcWVEMlBiQTc5NFVOSGp4T0tZTXN4bEN4Q3pqSnEwVnJaZlM4U3diMVFr?=
 =?utf-8?B?cE50VEhBK0orVyttN0p0ZUtPMEtaQ3dzLzdxQWFSd1NlQ2RhVkNSeWQwVFQ1?=
 =?utf-8?B?ajkrWTJRR05uQ0llYkVZZzYzNllEREcweUplT1oxd2FrMjNMcGJSV0g3dHNJ?=
 =?utf-8?B?TWxRTUxnOTAzaG8wSHRQNjZYRitUMERONG9tRlNKZy9hUDNpdzBlczBMNHd4?=
 =?utf-8?B?eWZTWU5nZjU4cEhUUFJURkZBYjAzNkhBOWlHbE1uMFJlTWpScUZ5MFhFcjVu?=
 =?utf-8?B?YlFZNkFBK1g5d0JnVFBSY0lJclM1YWpwR1dhS2pOZlN0UEFjM0V3VHRvdVN2?=
 =?utf-8?Q?tUSvBLhDSGKoQtDRGiDqrp0p+pHFuKsGCXzRE79?=
X-OriginatorOrg: outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 025f4197-80c1-4878-2af5-08dc3ce0aec3
X-MS-Exchange-CrossTenant-AuthSource: SEZPR06MB6959.apcprd06.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 05 Mar 2024 06:51:28.9005
 (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: SI2PR06MB5164

On 3/1/2024 2:50 PM, Krzysztof Kozlowski wrote:
> On 01/03/2024 04:36, Yang Xiwen via B4 Relay wrote:
>> From: Yang Xiwen <forbidden405@outlook.com>
>>
>> We already have MODULE_DEVICE_TABLE(), so this MODULE_ALIAS() has no use
>> and should be removed.
>>
> That's not accurate. You don't have for platform IDs, so you basically
> remove ability to autoload via platform. It's fine for me, but the
> commit msg is just misleading.


I've never heard of any distro making use of this custom alias. A 
generic OF based module alias is enough.


>
>
> Best regards,
> Krzysztof
>

-- 
Regards,
Yang Xiwen