Change in ...coreboot[master]: arch/arm64: Make ARM64 specific options depend on select ARCH_ARM64 - coreboot-gerrit

11 Feb 2019


      Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31315 )
Change subject: arch/arm64: Make ARM64 specific options depend on select ARCH_ARM64
......................................................................
Patch Set 2:
I'm not really a fan of the extra boilerplate in all SoC Kconfig files. Can you explain why this is necessary? If it's just about removing some unused lines from .config, that doesn't really seem worth it. (I'd be okay with guarding all the really arch-specific options like ARM64_USE_ARCH_TIMER, but I'd prefer the stage architecture selectors like ARCH_ROMSTAGE_ARM64 to stay available without having to add more select lines to SoCs.)
-- 
To view, visit https://review.coreboot.org/c/coreboot/+/31315
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Iaa9cd902281e51f823717f6ea4c72e5736fefb31
Gerrit-Change-Number: 31315
Gerrit-PatchSet: 2
Gerrit-Owner: Arthur Heymans arthur@aheymans.xyz
Gerrit-Reviewer: Arthur Heymans arthur@aheymans.xyz
Gerrit-Reviewer: Julius Werner jwerner@chromium.org
Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org
Gerrit-Comment-Date: Mon, 11 Feb 2019 19:20:10 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment