[PATCH v1 33/45] clk: mediatek: Switch to module_platform_driver() where possible

AngeloGioacchino Del Regno angelogioacchino.delregno at collabora.com
Tue Feb 7 01:03:09 PST 2023


Il 07/02/23 07:37, Chen-Yu Tsai ha scritto:
> On Mon, Feb 6, 2023 at 11:30 PM AngeloGioacchino Del Regno
> <angelogioacchino.delregno at collabora.com> wrote:
>>
>> Lots of clock drivers have got both .probe() and a .remove() callbacks:
>> switch from builtin_platform_driver() to module_platform_driver() so
>> that we actually register the .remove() callback.
>>
>> Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno at collabora.com>
>> ---
> 
> [...]
> 
>>   drivers/clk/mediatek/clk-mt8195-vpp0.c         | 2 +-
>>   drivers/clk/mediatek/clk-mt8195-vpp1.c         | 2 +-
> 
> These two conflict with "clk: mediatek: remove MT8195 vppsys/0/1 simple_probe".
> Since the rest of the vppsys stuff has been queued up, I'd like to see
> that one get merged first so we don't have two drivers matching the same
> compatible string.
> 
> This conflict also causes the last "clk: mediatek: Add MODULE_DEVICE_TABLE()
> where appropriate" to not apply.
> 
> ChenYu

Yes that makes a lot of sense - this series is really big, so let's get the
8195 vppsys stuff merged in first.

I'll rebase on top of that for v2.

Angelo



More information about the Linux-mediatek mailing list