Re: [PR] tools/Unix.mk: Add debug_info target to print nxdiag output [nuttx]

2025-01-06 Thread via GitHub
xiaoxiang781216 merged PR #15304: URL: https://github.com/apache/nuttx/pull/15304 -- 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: commits-unsubscr...@nu

Re: [PR] tools/Unix.mk: Add debug_info target to print nxdiag output [nuttx]

2025-01-06 Thread via GitHub
xiaoxiang781216 commented on code in PR #15304: URL: https://github.com/apache/nuttx/pull/15304#discussion_r1904253895 ## tools/Unix.mk: ## @@ -618,6 +619,35 @@ bootloader: clean_bootloader: $(Q) $(MAKE) -C $(ARCH_SRC) clean_bootloader +checkpython3: + @if [ -z

Re: [PR] tools/Unix.mk: Add debug_info target to print nxdiag output [nuttx]

2025-01-04 Thread via GitHub
cederom commented on PR #15304: URL: https://github.com/apache/nuttx/pull/15304#issuecomment-2571486061 > > @cederom: I would suggest renaming `debug_info` target to `sysinfo` or `dump_sysinfo` or `sysinfo_dump` (or similar) because we operate on **sysinfo** (i.e. `sysinfo.h` / `tools/parse

Re: [PR] tools/Unix.mk: Add debug_info target to print nxdiag output [nuttx]

2025-01-04 Thread via GitHub
patacongo commented on PR #15304: URL: https://github.com/apache/nuttx/pull/15304#issuecomment-2571480871 > I would suggest renaming `debug_info` target to `sysinfo` or `dump_sysinfo` or `sysinfo_dump` (or similar) because we operate on **sysinfo** (i.e. `sysinfo.h` / `tools/parse_sysinfo.p

Re: [PR] tools/Unix.mk: Add debug_info target to print nxdiag output [nuttx]

2025-01-04 Thread via GitHub
cederom commented on PR #15304: URL: https://github.com/apache/nuttx/pull/15304#issuecomment-2571467086 > > the problem is that it's not good to put the arch/chip specific code in a common script. > > My suggestion is to decouple the `host_sysinfo.py` from chip-specific. Let's split

Re: [PR] tools/Unix.mk: Add debug_info target to print nxdiag output [nuttx]

2025-01-04 Thread via GitHub
cederom commented on code in PR #15304: URL: https://github.com/apache/nuttx/pull/15304#discussion_r1903180161 ## tools/Unix.mk: ## @@ -618,6 +619,40 @@ bootloader: clean_bootloader: $(Q) $(MAKE) -C $(ARCH_SRC) clean_bootloader +checkpython3: + @if [ -z "$$(whic

Re: [PR] tools/Unix.mk: Add debug_info target to print nxdiag output [nuttx]

2025-01-03 Thread via GitHub
tmedicci commented on PR #15304: URL: https://github.com/apache/nuttx/pull/15304#issuecomment-2569178461 > the problem is that it's not good to put the arch/chip specific code in a common script. My suggestion is to decouple the `host_sysinfo.py` from chip-specific. Let's split Espre

Re: [PR] tools/Unix.mk: Add debug_info target to print nxdiag output [nuttx]

2025-01-03 Thread via GitHub
eren-terzioglu commented on code in PR #15304: URL: https://github.com/apache/nuttx/pull/15304#discussion_r1901561567 ## tools/Unix.mk: ## @@ -618,6 +619,40 @@ bootloader: clean_bootloader: $(Q) $(MAKE) -C $(ARCH_SRC) clean_bootloader +checkpython3: + @if [ -z "

Re: [PR] tools/Unix.mk: Add debug_info target to print nxdiag output [nuttx]

2025-01-03 Thread via GitHub
eren-terzioglu commented on PR #15304: URL: https://github.com/apache/nuttx/pull/15304#issuecomment-2568839601 > I would suggest renaming `debug_info` target to `sysinfo` or `dump_sysinfo` or `sysinfo_dump` (or similar) because we operate on **sysinfo** (i.e. `sysinfo.h` / `tools/parse_sysi

Re: [PR] tools/Unix.mk: Add debug_info target to print nxdiag output [nuttx]

2025-01-02 Thread via GitHub
cederom commented on code in PR #15304: URL: https://github.com/apache/nuttx/pull/15304#discussion_r1901424224 ## tools/Unix.mk: ## @@ -618,6 +619,40 @@ bootloader: clean_bootloader: $(Q) $(MAKE) -C $(ARCH_SRC) clean_bootloader +checkpython3: + @if [ -z "$$(whic

Re: [PR] tools/Unix.mk: Add debug_info target to print nxdiag output [nuttx]

2025-01-02 Thread via GitHub
xiaoxiang781216 commented on code in PR #15304: URL: https://github.com/apache/nuttx/pull/15304#discussion_r1901406462 ## tools/Unix.mk: ## @@ -618,6 +619,40 @@ bootloader: clean_bootloader: $(Q) $(MAKE) -C $(ARCH_SRC) clean_bootloader +checkpython3: + @if [ -z

Re: [PR] tools/Unix.mk: Add debug_info target to print nxdiag output [nuttx]

2025-01-02 Thread via GitHub
cederom commented on PR #15304: URL: https://github.com/apache/nuttx/pull/15304#issuecomment-2568465565 @eren-terzioglu this is really amazing and welcome job, big thank you for the contributions!! Let's invite the community to the discussion :-) Especially may come handy in the Indep

Re: [PR] tools/Unix.mk: Add debug_info target to print nxdiag output [nuttx]

2025-01-02 Thread via GitHub
cederom commented on code in PR #15304: URL: https://github.com/apache/nuttx/pull/15304#discussion_r1901303013 ## tools/Unix.mk: ## @@ -618,6 +619,40 @@ bootloader: clean_bootloader: $(Q) $(MAKE) -C $(ARCH_SRC) clean_bootloader +checkpython3: + @if [ -z "$$(whic

Re: [PR] tools/Unix.mk: Add debug_info target to print nxdiag output [nuttx]

2025-01-02 Thread via GitHub
cederom commented on code in PR #15304: URL: https://github.com/apache/nuttx/pull/15304#discussion_r1901303013 ## tools/Unix.mk: ## @@ -618,6 +619,40 @@ bootloader: clean_bootloader: $(Q) $(MAKE) -C $(ARCH_SRC) clean_bootloader +checkpython3: + @if [ -z "$$(whic

Re: [PR] tools/Unix.mk: Add debug_info target to print nxdiag output [nuttx]

2025-01-02 Thread via GitHub
cederom commented on PR #15304: URL: https://github.com/apache/nuttx/pull/15304#issuecomment-2568452614 I would suggest renaming `debug_info` target to `sysinfo` or `dump_sysinfo` or `sysinfo_dump` (or similar) because we operate on **sysinfo** (i.e. `sysinfo.h` / `tools/parse_sysinfo.py` /

Re: [PR] tools/Unix.mk: Add debug_info target to print nxdiag output [nuttx]

2025-01-02 Thread via GitHub
tmedicci commented on code in PR #15304: URL: https://github.com/apache/nuttx/pull/15304#discussion_r1900837806 ## tools/Unix.mk: ## @@ -618,6 +619,40 @@ bootloader: clean_bootloader: $(Q) $(MAKE) -C $(ARCH_SRC) clean_bootloader +checkpython3: + @if [ -z "$$(whi

Re: [PR] tools/Unix.mk: Add debug_info target to print nxdiag output [nuttx]

2024-12-31 Thread via GitHub
eren-terzioglu commented on code in PR #15304: URL: https://github.com/apache/nuttx/pull/15304#discussion_r1900089093 ## tools/Unix.mk: ## @@ -618,6 +619,40 @@ bootloader: clean_bootloader: $(Q) $(MAKE) -C $(ARCH_SRC) clean_bootloader +checkpython3: + @if [ -z "

Re: [PR] tools/Unix.mk: Add debug_info target to print nxdiag output [nuttx]

2024-12-31 Thread via GitHub
eren-terzioglu commented on code in PR #15304: URL: https://github.com/apache/nuttx/pull/15304#discussion_r1900089093 ## tools/Unix.mk: ## @@ -618,6 +619,40 @@ bootloader: clean_bootloader: $(Q) $(MAKE) -C $(ARCH_SRC) clean_bootloader +checkpython3: + @if [ -z "

Re: [PR] tools/Unix.mk: Add debug_info target to print nxdiag output [nuttx]

2024-12-31 Thread via GitHub
xiaoxiang781216 commented on code in PR #15304: URL: https://github.com/apache/nuttx/pull/15304#discussion_r1900081865 ## tools/Unix.mk: ## @@ -618,6 +619,40 @@ bootloader: clean_bootloader: $(Q) $(MAKE) -C $(ARCH_SRC) clean_bootloader +checkpython3: + @if [ -z

Re: [PR] tools/Unix.mk: Add debug_info target to print nxdiag output [nuttx]

2024-12-31 Thread via GitHub
eren-terzioglu commented on code in PR #15304: URL: https://github.com/apache/nuttx/pull/15304#discussion_r1899986031 ## tools/Unix.mk: ## @@ -618,6 +619,40 @@ bootloader: clean_bootloader: $(Q) $(MAKE) -C $(ARCH_SRC) clean_bootloader +checkpython3: + @if [ -z "

Re: [PR] tools/Unix.mk: Add debug_info target to print nxdiag output [nuttx]

2024-12-31 Thread via GitHub
eren-terzioglu commented on code in PR #15304: URL: https://github.com/apache/nuttx/pull/15304#discussion_r1899986031 ## tools/Unix.mk: ## @@ -618,6 +619,40 @@ bootloader: clean_bootloader: $(Q) $(MAKE) -C $(ARCH_SRC) clean_bootloader +checkpython3: + @if [ -z "

Re: [PR] tools/Unix.mk: Add debug_info target to print nxdiag output [nuttx]

2024-12-31 Thread via GitHub
eren-terzioglu commented on code in PR #15304: URL: https://github.com/apache/nuttx/pull/15304#discussion_r1899986031 ## tools/Unix.mk: ## @@ -618,6 +619,40 @@ bootloader: clean_bootloader: $(Q) $(MAKE) -C $(ARCH_SRC) clean_bootloader +checkpython3: + @if [ -z "

Re: [PR] tools/Unix.mk: Add debug_info target to print nxdiag output [nuttx]

2024-12-31 Thread via GitHub
xiaoxiang781216 commented on code in PR #15304: URL: https://github.com/apache/nuttx/pull/15304#discussion_r1899984490 ## tools/Unix.mk: ## @@ -618,6 +619,40 @@ bootloader: clean_bootloader: $(Q) $(MAKE) -C $(ARCH_SRC) clean_bootloader +checkpython3: + @if [ -z

Re: [PR] tools/Unix.mk: Add debug_info target to print nxdiag output [nuttx]

2024-12-30 Thread via GitHub
eren-terzioglu commented on code in PR #15304: URL: https://github.com/apache/nuttx/pull/15304#discussion_r1899584805 ## tools/Unix.mk: ## @@ -618,6 +619,13 @@ bootloader: clean_bootloader: $(Q) $(MAKE) -C $(ARCH_SRC) clean_bootloader +# debug_info: Parse nxdiag examp

Re: [PR] tools/Unix.mk: Add debug_info target to print nxdiag output [nuttx]

2024-12-30 Thread via GitHub
eren-terzioglu commented on code in PR #15304: URL: https://github.com/apache/nuttx/pull/15304#discussion_r1899574887 ## tools/Unix.mk: ## @@ -618,6 +619,13 @@ bootloader: clean_bootloader: $(Q) $(MAKE) -C $(ARCH_SRC) clean_bootloader +# debug_info: Parse nxdiag examp

Re: [PR] tools/Unix.mk: Add debug_info target to print nxdiag output [nuttx]

2024-12-30 Thread via GitHub
xiaoxiang781216 commented on code in PR #15304: URL: https://github.com/apache/nuttx/pull/15304#discussion_r1899574098 ## tools/Unix.mk: ## @@ -618,6 +619,13 @@ bootloader: clean_bootloader: $(Q) $(MAKE) -C $(ARCH_SRC) clean_bootloader +# debug_info: Parse nxdiag exam

Re: [PR] tools/Unix.mk: Add debug_info target to print nxdiag output [nuttx]

2024-12-30 Thread via GitHub
eren-terzioglu commented on code in PR #15304: URL: https://github.com/apache/nuttx/pull/15304#discussion_r1899565737 ## tools/Unix.mk: ## @@ -618,6 +619,13 @@ bootloader: clean_bootloader: $(Q) $(MAKE) -C $(ARCH_SRC) clean_bootloader +# debug_info: Parse nxdiag examp

Re: [PR] tools/Unix.mk: Add debug_info target to print nxdiag output [nuttx]

2024-12-30 Thread via GitHub
xiaoxiang781216 commented on code in PR #15304: URL: https://github.com/apache/nuttx/pull/15304#discussion_r1899563286 ## tools/Unix.mk: ## @@ -618,6 +619,13 @@ bootloader: clean_bootloader: $(Q) $(MAKE) -C $(ARCH_SRC) clean_bootloader +# debug_info: Parse nxdiag exam

Re: [PR] tools/Unix.mk: Add debug_info target to print nxdiag output [nuttx]

2024-12-30 Thread via GitHub
tmedicci commented on code in PR #15304: URL: https://github.com/apache/nuttx/pull/15304#discussion_r1899530499 ## tools/Unix.mk: ## @@ -618,6 +619,13 @@ bootloader: clean_bootloader: $(Q) $(MAKE) -C $(ARCH_SRC) clean_bootloader +# debug_info: Parse nxdiag example out

Re: [PR] tools/Unix.mk: Add debug_info target to print nxdiag output [nuttx]

2024-12-30 Thread via GitHub
eren-terzioglu commented on code in PR #15304: URL: https://github.com/apache/nuttx/pull/15304#discussion_r1899438836 ## tools/Unix.mk: ## @@ -618,6 +619,13 @@ bootloader: clean_bootloader: $(Q) $(MAKE) -C $(ARCH_SRC) clean_bootloader +# debug_info: Parse nxdiag examp

Re: [PR] tools/Unix.mk: Add debug_info target to print nxdiag output [nuttx]

2024-12-30 Thread via GitHub
eren-terzioglu commented on code in PR #15304: URL: https://github.com/apache/nuttx/pull/15304#discussion_r1899438836 ## tools/Unix.mk: ## @@ -618,6 +619,13 @@ bootloader: clean_bootloader: $(Q) $(MAKE) -C $(ARCH_SRC) clean_bootloader +# debug_info: Parse nxdiag examp

Re: [PR] tools/Unix.mk: Add debug_info target to print nxdiag output [nuttx]

2024-12-30 Thread via GitHub
xiaoxiang781216 commented on code in PR #15304: URL: https://github.com/apache/nuttx/pull/15304#discussion_r1899435040 ## tools/Unix.mk: ## @@ -618,6 +619,13 @@ bootloader: clean_bootloader: $(Q) $(MAKE) -C $(ARCH_SRC) clean_bootloader +# debug_info: Parse nxdiag exam

Re: [PR] tools/Unix.mk: Add debug_info target to print nxdiag output [nuttx]

2024-12-30 Thread via GitHub
eren-terzioglu commented on code in PR #15304: URL: https://github.com/apache/nuttx/pull/15304#discussion_r1899372419 ## tools/Unix.mk: ## @@ -618,6 +619,40 @@ bootloader: clean_bootloader: $(Q) $(MAKE) -C $(ARCH_SRC) clean_bootloader +# checkpython3: Checks python is

Re: [PR] tools/Unix.mk: Add debug_info target to print nxdiag output [nuttx]

2024-12-30 Thread via GitHub
eren-terzioglu commented on PR #15304: URL: https://github.com/apache/nuttx/pull/15304#issuecomment-2565180293 Thanks @xiaoxiang781216, updated. -- 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 th

Re: [PR] tools/Unix.mk: Add debug_info target to print nxdiag output [nuttx]

2024-12-22 Thread via GitHub
xiaoxiang781216 commented on code in PR #15304: URL: https://github.com/apache/nuttx/pull/15304#discussion_r1895341358 ## tools/Unix.mk: ## @@ -618,6 +619,40 @@ bootloader: clean_bootloader: $(Q) $(MAKE) -C $(ARCH_SRC) clean_bootloader +# checkpython3: Checks python i

Re: [PR] tools/Unix.mk: Add debug_info target to print nxdiag output [nuttx]

2024-12-22 Thread via GitHub
eren-terzioglu commented on code in PR #15304: URL: https://github.com/apache/nuttx/pull/15304#discussion_r1895040146 ## tools/Unix.mk: ## @@ -618,6 +619,40 @@ bootloader: clean_bootloader: $(Q) $(MAKE) -C $(ARCH_SRC) clean_bootloader +# checkpython3: Checks python is

Re: [PR] tools/Unix.mk: Add debug_info target to print nxdiag output [nuttx]

2024-12-22 Thread via GitHub
eren-terzioglu commented on code in PR #15304: URL: https://github.com/apache/nuttx/pull/15304#discussion_r1895040146 ## tools/Unix.mk: ## @@ -618,6 +619,40 @@ bootloader: clean_bootloader: $(Q) $(MAKE) -C $(ARCH_SRC) clean_bootloader +# checkpython3: Checks python is

Re: [PR] tools/Unix.mk: Add debug_info target to print nxdiag output [nuttx]

2024-12-22 Thread via GitHub
eren-terzioglu commented on code in PR #15304: URL: https://github.com/apache/nuttx/pull/15304#discussion_r1895040146 ## tools/Unix.mk: ## @@ -618,6 +619,40 @@ bootloader: clean_bootloader: $(Q) $(MAKE) -C $(ARCH_SRC) clean_bootloader +# checkpython3: Checks python is

Re: [PR] tools/Unix.mk: Add debug_info target to print nxdiag output [nuttx]

2024-12-22 Thread via GitHub
eren-terzioglu commented on code in PR #15304: URL: https://github.com/apache/nuttx/pull/15304#discussion_r1895040146 ## tools/Unix.mk: ## @@ -618,6 +619,40 @@ bootloader: clean_bootloader: $(Q) $(MAKE) -C $(ARCH_SRC) clean_bootloader +# checkpython3: Checks python is

Re: [PR] tools/Unix.mk: Add debug_info target to print nxdiag output [nuttx]

2024-12-22 Thread via GitHub
eren-terzioglu commented on code in PR #15304: URL: https://github.com/apache/nuttx/pull/15304#discussion_r1895040146 ## tools/Unix.mk: ## @@ -618,6 +619,40 @@ bootloader: clean_bootloader: $(Q) $(MAKE) -C $(ARCH_SRC) clean_bootloader +# checkpython3: Checks python is

Re: [PR] tools/Unix.mk: Add debug_info target to print nxdiag output [nuttx]

2024-12-20 Thread via GitHub
xiaoxiang781216 commented on code in PR #15304: URL: https://github.com/apache/nuttx/pull/15304#discussion_r1894567114 ## tools/Unix.mk: ## @@ -618,6 +619,40 @@ bootloader: clean_bootloader: $(Q) $(MAKE) -C $(ARCH_SRC) clean_bootloader +# checkpython3: Checks python i

Re: [PR] tools/Unix.mk: Add debug_info target to print nxdiag output [nuttx]

2024-12-20 Thread via GitHub
nuttxpr commented on PR #15304: URL: https://github.com/apache/nuttx/pull/15304#issuecomment-2557512557 [**\[Experimental Bot, please feedback here\]**](https://github.com/search?q=repo%3Aapache%2Fnuttx+13552&type=issues) __Fill In The Commit Message:__ This PR contains a Commit with

[PR] tools/Unix.mk: Add debug_info target to print nxdiag output [nuttx]

2024-12-20 Thread via GitHub
eren-terzioglu opened a new pull request, #15304: URL: https://github.com/apache/nuttx/pull/15304 ## Summary Add debug_info target support to parse and print `sysinfo.h` file which is output file of nxdiag application without flashing it to the device or even enabling `SYSTEM_NXD