Re: [PATCH 00/32] clk: mediatek: Migrate to common probe/remove helpers and fix memory leaks
Jakub Kicinski <[email protected]> Mon, 3 Aug 2026 18:40:47 -0700
| Newsgroups | org.kernel.vger.linux-clk,org.infradead.lists.linux-arm-kernel,org.infradead.lists.linux-mediatek,org.kernel.vger.linux-kernel,org.kernel.vger.netdev,org.kernel.vger.stable |
|---|---|
| Message-ID | <[email protected]> |
On Mon, 3 Aug 2026 16:28:07 -0400 Brian Masney wrote:
> Hi Jakub,
>
> On Mon, Aug 03, 2026 at 12:50:34PM -0700, Jakub Kicinski wrote:
> > On Mon, 3 Aug 2026 23:16:27 +0900 Akari Tsuyukusa wrote:
> > > This series migrates most MediaTek clock drivers to the common
> > > probe/remove helpers provided by the MTK clock framework.
> >
> > Why are you CCing netdev on this? None of it touches networking
>
> I saw that message from you about two different mediatek patch series
> from different submitters. This series has 38 files, all under
> drivers/clk/mediatek/. If I run get_maintainers.pl on each file path
> individuall like this:
>
> $ cat filelist | awk '{print $1}' | \
> xargs -iblah echo ./scripts/get_maintainer.pl blah
>
> Then netdev list doesn't show up.
>
> However, if I look at it as a whole, then it shows up:
>
> $ b4 am [email protected]
> $ ./scripts/get_maintainer.pl \
> ./20260803_akkun11_open_clk_mediatek_migrate_to_common_probe_remove_helpers_and_fix_memory_leaks.mbx
>
> ...
> [email protected] (open list:PTP HARDWARE CLOCK SUPPORT:Keyword:(?:\b|_)ptp(?:\b|_))
>
> I think it's matching on this entry in MAINTAINERS:
>
> PTP HARDWARE CLOCK SUPPORT
> M: Richard Cochran <[email protected]>
> L: [email protected]
> S: Maintained
> W: http://linuxptp.sourceforge.net/
> F: Documentation/ABI/testing/sysfs-ptp
> F: Documentation/driver-api/ptp.rst
> F: drivers/net/phy/dp83640*
> F: drivers/ptp/*
> F: include/linux/ptp_cl*
> K: (?:\b|_)ptp(?:\b|_)
>
> I think it's the K: for the ptp that's matching.
>
> $ grep ptp 20260803_akkun11_open_clk_mediatek_migrate_to_common_probe_remove_helpers_and_fix_memory_leaks.mbx
> -static const struct mtk_pll_data ptp_plls[] = {
> - PLL(CLK_PTPPLL, "ptppll", PTPPLL_CON0, PTPPLL_CON0, 0, 0, PLL_AO,
> +static const struct mtk_clk_desc ptp_plls = {
> + PLL(CLK_PTPPLL, "ptppll", PTPPLL_CON0, PTPPLL_CON0, 0, 0, PLL_AO,
>
I see, thanks for digging thru this!
Patchwork matching rules must be missing the PTP side because none of
the patches end up assigned to netdev. No big deal but going forward
I think you can drop this extra CC.