TimJTi commented on code in PR #9252:
URL: https://github.com/apache/nuttx/pull/9252#discussion_r1192472323
##########
arch/arm/src/sama5/Kconfig:
##########
@@ -799,6 +799,7 @@ config SAMA5_ARM
config SAMA5_FUSE
bool "Fuse Controller (FUSE)"
default n
+ depends on SAMA5_HAVE_FUSE
Review Comment:
> @TimJTi is there some SAMA5 MCU that doesn't have FUSE? Maybe this
"depends on" is not needed
@acassis - SAMA5D2 and D4 have "secure fuse controller" and SAMA5D3 has just
a "fuse controller", and they do all seem to have one or the other. What is
missing from Kconfig is selecting SAMA5_FUSE for the ARCH_CHIP_SAMA5D3 family
devices. So it is needed if anyone ever wants to add a non-secure Fuse
Controller driver for that device. It used to be set for A2 and A4, but that
was "wrong" in my mind as they have SFC instead so I add that instead/
I can add it for the SAMA3 completeness if wanted, even though it won't
actually enable anything useful?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
