Re: linux-next: Tree for Sep 1 [sound/soc/mediatek/mt8195/snd-soc-mt8195-afe.ko]

[Trevor Wu]

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