[PATCH] misc: Fix always compiling MISC even for SPL/TPL
Sean Anderson
sean.anderson at seco.com
Fri Apr 22 22:00:10 CEST 2022
Hi Tom,
On 4/22/22 3:56 PM, Tom Rini wrote:
> On Tue, Apr 19, 2022 at 03:12:45PM -0400, Sean Anderson wrote:
>
>> We should only build support for misc if the appropriate SPL/TPL symbol
>> is defined.
>>
>> Fixes: aaba703fd0 ("spl: misc: Allow misc drivers in SPL and TPL")
>> Signed-off-by: Sean Anderson <sean.anderson at seco.com>
>> ---
>>
>> drivers/misc/Makefile | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/misc/Makefile b/drivers/misc/Makefile
>> index f22eff601a..b7a8ef68ab 100644
>> --- a/drivers/misc/Makefile
>> +++ b/drivers/misc/Makefile
>> @@ -3,7 +3,7 @@
>> # (C) Copyright 2000-2007
>> # Wolfgang Denk, DENX Software Engineering, wd at denx.de.
>>
>> -obj-$(CONFIG_MISC) += misc-uclass.o
>> +obj-$(CONFIG_$(SPL_TPL_)MISC) += misc-uclass.o
>>
>> obj-$(CONFIG_$(SPL_TPL_)CROS_EC) += cros_ec.o
>> obj-$(CONFIG_$(SPL_TPL_)CROS_EC_SANDBOX) += cros_ec_sandbox.o
>
> So, I know we talked on IRC and I thought it was all good, but, this
> causes problems. Today, large chunks of drivers/misc/Kconfig have SPL
> options that don't depend on SPL_MISC (and TPL is the same story). It's
> I think a straight forward effort to audit them, but it's also more than
> I'm comfortable doing as a fixup during testing. I hit a problem
> starting with sandbox_spl_defconfig but given all of the other options
> in there, I think it's a fair number to fix the dependencies on. Making
> SPL_MISC/TPL_MISC "default MISC" should keep things from suddenly
> breaking, I think.
>
OK, I'll send a v2 with that added.
--Sean
More information about the U-Boot
mailing list