On 19. 04. 19 13:27, Robert P. J. Day wrote:
> 
>   just noticed the oddness regarding those two Kbuild symbols:
> 
> $ git grep ARM_DCC
> arch/x86/include/asm/acpi_table.h:#define ACPI_DBG2_ARM_DCC   0x000F
> configs/zynq_cse_qspi_defconfig:CONFIG_DEBUG_UART_ARM_DCC=y
> drivers/serial/Kconfig:config DEBUG_UART_ARM_DCC
> drivers/serial/Makefile:obj-$(CONFIG_ARM_DCC) += arm_dcc.o
> drivers/serial/arm_dcc.c:#ifdef CONFIG_DEBUG_UART_ARM_DCC
> include/configs/xilinx_versal.h:#define CONFIG_ARM_DCC
> include/configs/xilinx_zynqmp.h:#define CONFIG_ARM_DCC
> include/configs/zynq-common.h:#define CONFIG_ARM_DCC
> scripts/config_whitelist.txt:CONFIG_ARM_DCC
> $
> 
>   strangely(?), there is a Kbuild config directive for the latter:
> 
>   drivers/serial/Kconfig:config DEBUG_UART_ARM_DCC
> 
> but not the former, which is just hardcoded in some xilinx-associated
> header files. is this deliberate? it just seems ... weird.

Simply none has had time to convert ARM_DCC to Kconfig that's why it is
enabled via base platform.
And because debug symbol is in Kconfig it is enabled for platform which
needs it.
Feel free to send a patch to move symbol there.

Thanks,
Michal



-- 
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Xilinx Microblaze
Maintainer of Linux kernel - Xilinx Zynq ARM and ZynqMP ARM64 SoCs
U-Boot custodian - Xilinx Microblaze/Zynq/ZynqMP/Versal SoCs


Attachment: signature.asc
Description: OpenPGP digital signature

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to