gemstonedevteam opened a new pull request, #20182:
URL: https://github.com/apache/nuttx/pull/20182

   ### Summary
   
     * The 16550 console provides `up_putc()`, polled character output that 
works
       from panic context with interrupts disabled, but the symbol was never
       declared for this arch, so `SYSLOG_DEFAULT` and the assert dump could not
       select it.
     * The interrupt-driven console cannot flush once an assertion stops the 
world,
       so without `up_putc()` selected there is no path for a panic dump to 
reach
       the console.
   
   ### Impact
   
     * Is new feature added? Is existing feature changed? **YES** — makes 
existing
       low-level console output selectable.
     * Impact on user? **NO**.
     * Impact on build? **NO**.
     * Impact on hardware? **YES** — arch/arm/am67 only.
     * Impact on documentation? **NO**.
     * Impact on security? **NO**.
     * Impact on compatibility? **NO** — additive Kconfig declaration.
     * Anything else? Depends on PRs 3–6.
   
   ### Testing
   
   ```
   Build Host(s): Pardus GNU/Linux 25 (Debian 13 base), x86_64, host gcc 14
   Toolchain:     arm-none-eabi-gcc 14.2.1 20241119 (15:14.2.rel1-1)
   Target:        arm / TI AM67 (J722S) Cortex-R5F, board t3-gem-o1:nsh
   nuttx-apps:    5a7ab4200
   Loaded on the main-domain R5F by Linux remoteproc; console on UART-MAIN1.
   ```
   
   Testing logs before change: not captured. Without `ARCH_LOWPUTC` declared,
   `SYSLOG_DEFAULT` has no polled output to select, so there is no path for a
   panic dump to reach the console — there is nothing to record.
   
   The version string in the logs is from the tested build; the commits were
   GPG-signed afterwards, which rewrites hashes. The code is unchanged.
   
   Testing logs after change:
   
   ```
   make distclean && ./tools/configure.sh t3-gem-o1:nsh && make -j
     -> nuttx ELF 543332 bytes, 0 compiler warnings
   
   Test plan result:
   console_alive            PASS  prompt responsive
   version                  PASS  NuttX version 0.0.0 0038417a4d Sep 10 2026 
13:49:33 t3-gem-o1:nsh
   procfs                   PASS  procfs readable
   tasks                    PASS  2 task lines
   crash_dump               PASS  panic dump reached the console, 
DFSR=0x00000008 (board halted; power-c
   
   PASS=5
   ```
   
   ### PR verification Self-Check
   
     * [x] This PR introduces only one functional change.
     * [x] I have updated all required description fields above.
     * [x] My PR adheres to Contributing Guidelines and Documentation.
     * [ ] My PR is still work in progress (not ready for review).
     * [x] My PR is ready for review and can be safely merged into a codebase.
   


-- 
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]

Reply via email to