davids5 commented on PR #6815: URL: https://github.com/apache/incubator-nuttx/pull/6815#issuecomment-1211639776
@rm5248 Please have a look at https://github.com/apache/incubator-nuttx/blob/2335b69120a3e36f1aa0cecc19c95208806cae54/arch/mips/src/pic32mx/pic32mx_head.S#L638 and https://github.com/apache/incubator-nuttx/blob/c55085c0d8864c9401de97872ab3463d474f262c/arch/mips/src/pic32mx/Kconfig#L1109 ``` DEBUG<1:0>: Background Debugger Enable bits (forced to ‘11’ if code-protect is enabled) 11 = Debugger disabled 10 = Debugger enabled 01 = Reserved (same as ‘11’ setting) 00 = Reserved (same as ‘11’ setting) ``` It would be the correct place to manage security in the boot sequences AND allow JTAG to be useful from boot. Anything else is a security risk and may allow debugging when it is needed I would would strongly advise the PR be closed. -- 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]
