On Thu, 2021-09-02 at 10:22 +0200, Geert Uytterhoeven wrote: > Hi Trevor, > > On Thu, Sep 2, 2021 at 4:37 AM Trevor Wu <trevor.wu@xxxxxxxxxxxx> > wrote: > > On Wed, 2021-09-01 at 13:55 -0700, Randy Dunlap wrote: > > > On 9/1/21 1:17 AM, Stephen Rothwell wrote: > > > > Please do not add any v5.16 related code to your linux-next > > > > included > > > > branches until after v5.15-rc1 has been released. > > > > > > > > Changes since 20210831: > > > > > > > > > > > > > on x86_64: > > > > > > ERROR: modpost: "clkdev_add" [sound/soc/mediatek/mt8195/snd-soc- > > > mt8195-afe.ko] undefined! > > > ERROR: modpost: "clkdev_drop" [sound/soc/mediatek/mt8195/snd-soc- > > > mt8195-afe.ko] undefined! > > > ERROR: modpost: "clk_unregister_gate" > > > [sound/soc/mediatek/mt8195/snd- > > > soc-mt8195-afe.ko] undefined! > > > ERROR: modpost: "clk_register_gate" > > > [sound/soc/mediatek/mt8195/snd- > > > soc-mt8195-afe.ko] undefined! > > > > > > Full randconfig file is attached. > > > > > > > Hi Randy, > > > > The problem is caused by the dependency declaration, because it's > > not a > > driver for x86_64. > > The dependency declaration has been added in the following patch. > > > > https://urldefense.com/v3/__https://patchwork.kernel.org/project/alsa-devel/patch/7e628e359bde04ceb9ddd74a45931059b4a4623c.1630415860.git.geert*renesas@xxxxxxxxx/__;Kw!!CTRNKA9wMg0ARbw!wMq130mAo-s45pP6ShQ1S8UIRuJLhwOnCbQNAQHIE2zvNhjAd67h1rlqkIDxJvC5_g$ > > > > That is not sufficient, if COMPILE_TEST is enabled. > > Looks like it needs a dependency on COMMON_CLK, too. > > Gr{oetje,eeting}s, > > Geert > Hi Geert, Because it's a ARM64 driver, ARM64 will select COMMON_CLK. It seems that some dependency should be checked if COMPILE_TEST is enabled and the driver is compiled on non-ARM64 environment. We don't expect the driver can be used on non-ARM64 environment,may I remove COMPILE_TEST to solve the problem? If the driver only depends on ARCH_MEDIATEK, it must be compiled on ARM64. Thanks, Trevor
