[mediatek:v4.2-next/soc 3/3] warning: (MTK_SCPSYS) selects MTK_INFRACFG which has unmet direct dependencies (ARCH_MEDIATEK)

[Matthias Brugger]

2015-06-25 9:37 GMT+02:00 Daniel Kurtz <djkurtz at chromium.org>:
> Hi Matthias,
>
> On Thu, Jun 25, 2015 at 4:15 AM, kbuild test robot
> <fengguang.wu at intel.com> wrote:
>>
>> tree:   https://github.com/mbgg/linux-mediatek v4.2-next/soc
>> head:   86d8467ed3bb9243fdedc5376c2b07b93b776645
>> commit: 86d8467ed3bb9243fdedc5376c2b07b93b776645 [3/3] soc: Mediatek: Add SCPSYS power domain driver
>> config: xtensa-allyesconfig (attached as .config)
>> reproduce:
>>   wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross
>>   chmod +x ~/bin/make.cross
>>   git checkout 86d8467ed3bb9243fdedc5376c2b07b93b776645
>>   # save the attached .config to linux build tree
>>   make.cross ARCH=xtensa
>>
>> All warnings (new ones prefixed by >>):
>>
>> warning: (MTK_SCPSYS) selects MTK_INFRACFG which has unmet direct dependencies (ARCH_MEDIATEK)
>
> I guess this is because config MTK_INFRACFG  depends on ARCH_MEDIATEK
> [0], but config MTK_SCPSYS depends on (ARCH_MEDIATEK || COMPILE_TEST).
> So, in the (COMPILE_TEST && !ARCH_MEDIATEK) case
> (xtensa-allyesconfig), MTK_INFRACFG is disabled, but selected by
> MTK_SCPSYS.
>
> [0] [PATCH 1/4] soc: mediatek: Add infracfg misc driver support
>  .. drivers/soc/mediatek/Kconfig:
> +config MTK_INFRACFG
> +       bool "MediaTek INFRACFG Support"
> +       depends on ARCH_MEDIATEK
> +       select REGMAP
>
> [1] [PATCH 3/4] soc: Mediatek: Add SCPSYS power domain driver
> drivers/soc/mediatek/Kconfig
>
> +config MTK_SCPSYS
> +       bool "MediaTek SCPSYS Support"
> +       depends on ARCH_MEDIATEK || COMPILE_TEST
> +       select REGMAP
> +       select MTK_INFRACFG
> +       select PM_GENERIC_DOMAINS if PM
>
> Can you fix-up by adding:
> +       depends on ARCH_MEDIATEK || COMPILE_TEST
> to:
> +config MTK_INFRACFG
>
> And then re-apply to your tree?
>
> I guess people want to support "COMPILE_TEST" for these config
> options, but I am not really sure.

Ok let's give this a try (I'm not sure neither).
If this is not working, I will have a look into this on Monday (I will
be in a remote area for holidays).

Cheers,
Matthias

-- 
motzblog.wordpress.com