Subject: Re: [PATCH V3 1/7] firmware: imx: Add stubs for !CONFIG_IMX_SCU
case

On 16/03/2020 09:40:52+0000, Anson Huang wrote:

quoted

quoted

Why is that an issue? If they don't have IMX_SCU selected, then the
other SCU driver are not selected either, having stubs doesn't
change that you will have to select at least one option. Please
explain what is the issue that is not solved here.

OK, what I thought is even without IMX_SCU selected, other SCU drivers
still can be selected for build test after adding "COMPILE_TEST" to
the kconfig, like below, if without IMX_SCU API stubs, the
"COMPILE_TEST" can NOT be added to SCU drivers to enable build test, so I

think the IMX_SCU API stubs should be added?

quoted

No they shouldn't because there is not point adding COMPILE_TEST to the
SCU drivers. We don't add COMPILE_TEST to all the drivers and add stubs to
all the subsystems. E.g there is no point trying to compile an I2C driver if the
I2C core is not enabled.

OK, make sense.

Thanks,
Anson