(nuttx) branch master updated (9f84695ef7 -> b3712797fa)

2025-06-22 Thread xiaoxiang
This is an automated email from the ASF dual-hosted git repository.

xiaoxiang pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/nuttx.git


from 9f84695ef7 CI checkpatch: fix cvt2utf not found and add check of all 
necessary tools
 new a98f7a1045 arch/risc-v: Fix debugging syscall info
 new 14af1c6ef7 boards/risc-v/qemu-rv: Add missing section on 
`ld-nuttsbi.script`
 new b3712797fa Documentation/rv-virt: Add entries for `nsbi[|64]` 
defconfigs

The 3 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 .../risc-v/qemu-rv/boards/rv-virt/index.rst| 33 ++
 arch/risc-v/src/common/riscv_swint.c   |  4 +--
 .../qemu-rv/rv-virt/scripts/ld-nuttsbi.script  |  6 
 3 files changed, 41 insertions(+), 2 deletions(-)



(nuttx) 03/03: Documentation/rv-virt: Add entries for `nsbi[|64]` defconfigs

2025-06-22 Thread xiaoxiang
This is an automated email from the ASF dual-hosted git repository.

xiaoxiang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/nuttx.git

commit b3712797faf5ea00d2146751d740503aac3b630a
Author: Tiago Medicci Serrano 
AuthorDate: Fri Jun 20 14:39:32 2025 -0300

Documentation/rv-virt: Add entries for `nsbi[|64]` defconfigs

This commit adds entries for the `rv-virt:nsbi[|64]` defconfigs in
the `rv-virt` board documentation page.
---
 .../risc-v/qemu-rv/boards/rv-virt/index.rst| 33 ++
 1 file changed, 33 insertions(+)

diff --git a/Documentation/platforms/risc-v/qemu-rv/boards/rv-virt/index.rst 
b/Documentation/platforms/risc-v/qemu-rv/boards/rv-virt/index.rst
index 7329f198a8..6cdbe6ae4e 100644
--- a/Documentation/platforms/risc-v/qemu-rv/boards/rv-virt/index.rst
+++ b/Documentation/platforms/risc-v/qemu-rv/boards/rv-virt/index.rst
@@ -371,6 +371,39 @@ nsh64
 
 Identical to the `nsh`_ configuration, but for 64-bit RISC-V.
 
+nsbi
+
+
+This is similar to the `knsh`, but using NuttX's native (minimalistic)
+SBI. It uses `hostfs` and QEMU in semi-hosting mode to load the
+user-space applications. This is intended for 32-bit RISC-V.
+
+To build it, use the following command::
+
+$ make V=1 -j$(nproc)
+$ make export V=1 -j$(nproc)
+$ pushd ../apps
+$ ./tools/mkimport.sh -z -x ../nuttx/nuttx-export-*.tar.gz
+$ make import V=1 -j$(nproc)
+$ popd
+
+Run it with QEMU using the default command for 32-bit RISC-V without
+the ``-bios none`` option. Please note that it still runs in S-mode,
+but bypasses QEMU's OpenSBI.
+
+In `nsh`, applications can be run directly::
+
+nsh> hello
+
+nsbi64
+--
+
+Identical to the `nsbi`_ configuration, but for 64-bit RISC-V.
+
+Run it with QEMU using the default command for 64-bit RISC-V without
+the ``-bios none`` option. Please note that it still runs in S-mode,
+but bypasses QEMU's OpenSBI.
+
 smp
 ---
 



Re: [PR] remove warnings [nuttx]

2025-06-22 Thread via GitHub


acassis commented on code in PR #16571:
URL: https://github.com/apache/nuttx/pull/16571#discussion_r2160382203


##
drivers/input/gt9xx.c:
##
@@ -196,7 +196,7 @@ static int gt9xx_i2c_read(FAR struct gt9xx_dev_s *dev,
 
   const int msgv_len = sizeof(msgv) / sizeof(msgv[0]);
 
-  iinfo("reg=0x%x, buflen=%ld\n", reg, buflen);
+  iinfo("reg=0x%x, buflen=%d\n", reg, buflen);

Review Comment:
   @halyssonJr maybe you should use PRId32 here, please look other drivers



-- 
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...@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] [utsname] libc: Link dependency on `.version` file [nuttx]

2025-06-22 Thread via GitHub


Javinator9889 closed pull request #16564: [utsname] libc: Link dependency on 
`.version` file
URL: https://github.com/apache/nuttx/pull/16564


-- 
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...@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] [utsname] libc: Link dependency on `.version` file [nuttx]

2025-06-22 Thread via GitHub


Javinator9889 commented on PR #16564:
URL: https://github.com/apache/nuttx/pull/16564#issuecomment-2994364394

   > > > I think this change is fine. We already have an explicit dependency on 
.config, so having an explicit dependency on .version makes sense to me.
   > > > Are there any other non-source-code files that should also trigger the 
full rebuild?
   > > > (Currently I'm only commenting; will approve once I can test the 
changes with an actual build.)
   > > 
   > > 
   > > A quick search shows that there are only a few usages of `version.h`:
   > > 
![image](https://private-user-images.githubusercontent.com/25952165/457498241-0a91d3f5-dd61-432f-85a2-a0cd57d2ba9b.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3NTA0NTA2MDIsIm5iZiI6MTc1MDQ1MDMwMiwicGF0aCI6Ii8yNTk1MjE2NS80NTc0OTgyNDEtMGE5MWQzZjUtZGQ2MS00MzJmLTg1YTItYTBjZDU3ZDJiYTliLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNTA2MjAlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjUwNjIwVDIwMTE0MlomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTkyOGE1Yjc0MmJkZjM1ZmE4NTRiYWU2ZmZhZjdiNGE4NDYyZjU4Y2UyYWQ1NTc4YzE5YmFkNDRmNzcwZGJkODUmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0In0.dnwToDMaIRuo7ZCF4y2b1HTOy86CNEtREIjbSmw9rUk)
   > > And the `procfs` one is already being re-compiled whenever `version.h` 
changes - but only that file. TBH, I don't exactly understand why the `procfs` 
one is being recompiled while the `libc` is not.
   > 
   > I think there's a bigger bug with the dependencies @xiaoxiang781216 
@hartmannathan. I've tried changing any other file declared in the `Make.deps` 
and **the recompilation is not triggering** either. I'll take a deeper look on 
Monday and post here anything I find.
   
   So, I've been digging a little bit into it and I've found the root cause. 
This problem was already fixed on https://github.com/apache/nuttx/pull/7572 by 
@zouboan. The problem lies on the double delimiter. `make` is quite literally 
looking for a `bin//` - with double dash - and it fails to find anything:
   
   
![image](https://github.com/user-attachments/assets/471438dc-bbe0-4368-abd5-10c68c9a9b29)
   
   As it turns out, `make` is completely unable to find any rule that matches 
the requirement, so it considers it's already up-to-date. I had this problem 
simply because we were using an older NuttX version that doesn't include the 
fix. Therefore, there's no need to always remake `libc` whenever the `.version` 
file changes, as the `version.h` is populated correctly and `make` identifies 
the changes.
   
   BR,


-- 
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...@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[PR] Refine vfs source file layout [nuttx]

2025-06-22 Thread via GitHub


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

   ## Summary
   
   - fs/vfs: Rename lock.h to vfs.h
   - fs: Move inotify.c from fs/notify/ to fs/vfs/
   
   ## Impact
   
   refine vfs source file layout
   
   ## Testing
   
   ci
   


-- 
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...@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [I] [BUG] CONFIG_FILE_STREAM needs to be enabled by default for some applications (i.e. can example) [nuttx]

2025-06-22 Thread via GitHub


acassis commented on issue #16570:
URL: https://github.com/apache/nuttx/issues/16570#issuecomment-2994425088

   @xiaoxiang781216 I think the issue is happening because of it:
   
   ```
   config FILE_STREAM
   bool "Enable FILE stream"
   default !DEFAULT_SMALL
   ---help--- 
   Enable the standard buffered input/output support
   ```
   By default FILE_STREAM is not enabled for DEFAULT_SMALL and the usbnsh 
config for Bluepill is using CONFIG_DEFAUL_SMALL


-- 
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...@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] libc: Move stdio/README to Documentation/components/libs/libc/stream.rst [nuttx]

2025-06-22 Thread via GitHub


acassis merged PR #16559:
URL: https://github.com/apache/nuttx/pull/16559


-- 
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...@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] [Fix] Esp32S3 LCD FB resolution. [nuttx]

2025-06-22 Thread via GitHub


xiaoxiang781216 merged PR #16566:
URL: https://github.com/apache/nuttx/pull/16566


-- 
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...@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



(nuttx) branch master updated: fix framebuffer config resolution.

2025-06-22 Thread xiaoxiang
This is an automated email from the ASF dual-hosted git repository.

xiaoxiang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/nuttx.git


The following commit(s) were added to refs/heads/master by this push:
 new 74ce16da5c fix framebuffer config resolution.
74ce16da5c is described below

commit 74ce16da5c838c98988ad6b27be4695428b6e981
Author: halyssonJr 
AuthorDate: Sat Jun 21 16:35:33 2025 -0300

fix framebuffer config resolution.
---
 arch/xtensa/src/esp32s3/esp32s3_lcd.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/xtensa/src/esp32s3/esp32s3_lcd.c 
b/arch/xtensa/src/esp32s3/esp32s3_lcd.c
index 5cf54210d8..8b863c79ad 100644
--- a/arch/xtensa/src/esp32s3/esp32s3_lcd.c
+++ b/arch/xtensa/src/esp32s3/esp32s3_lcd.c
@@ -312,8 +312,8 @@ static struct esp32s3_lcd_s g_lcd_priv;
 static const struct fb_videoinfo_s g_base_videoinfo =
 {
   .fmt  = ESP32S3_LCD_COLOR_FMT,
-  .xres = CONFIG_ESP32S3_LCD_VRES,
-  .yres = CONFIG_ESP32S3_LCD_HRES,
+  .xres = CONFIG_ESP32S3_LCD_HRES,
+  .yres = CONFIG_ESP32S3_LCD_VRES,
   .nplanes  = 1
 };
 



Re: [I] [BUG] CONFIG_FILE_STREAM needs to be enabled by default for some applications (i.e. can example) [nuttx]

2025-06-22 Thread via GitHub


xiaoxiang781216 commented on issue #16570:
URL: https://github.com/apache/nuttx/issues/16570#issuecomment-299446

   > > ...BTW, the better fix is switching fopen/fclose to open/close as much 
as we can.
   > 
   > I thought fopen/fclose was the preferred, portable, method? Since 
open/close are not ANSI C IIRC.
   
   open/close is specified by POSIX. So, if you don't plan to port your code to 
Windows, open/close is equal to fopen/fclose.


-- 
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...@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



(nuttx-website) branch asf-site updated: Publishing web: be40bd59e126a7ffe6ebacfb1b7682fcd8e89704 docs: bc4041db608e9a04fd811dd74bcc67d8bded0d25

2025-06-22 Thread github-bot
This is an automated email from the ASF dual-hosted git repository.

github-bot pushed a commit to branch asf-site
in repository https://gitbox.apache.org/repos/asf/nuttx-website.git


The following commit(s) were added to refs/heads/asf-site by this push:
 new 122106c36 Publishing web: be40bd59e126a7ffe6ebacfb1b7682fcd8e89704 
docs: bc4041db608e9a04fd811dd74bcc67d8bded0d25
122106c36 is described below

commit 122106c36568ca26da09645d4ca47247d8d31a72
Author: Alan Carvalho de Assis 
AuthorDate: Mon Jun 23 00:33:40 2025 +

Publishing web: be40bd59e126a7ffe6ebacfb1b7682fcd8e89704 docs: 
bc4041db608e9a04fd811dd74bcc67d8bded0d25
---
 content/docs/10.0.0/index.html |   2 +-
 content/docs/10.0.1/index.html |   2 +-
 content/docs/10.1.0/index.html |   2 +-
 content/docs/10.2.0/index.html |   2 +-
 content/docs/10.3.0/index.html |   2 +-
 content/docs/11.0.0/index.html |   2 +-
 content/docs/12.0.0/index.html |   2 +-
 content/docs/12.0.0/searchindex.js |   2 +-
 content/docs/12.1.0/index.html |   2 +-
 content/docs/12.1.0/searchindex.js |   2 +-
 content/docs/12.2.0/index.html |   2 +-
 .../xtensa/esp32/boards/esp32-audio-kit/index.html |   2 +-
 .../xtensa/esp32/boards/esp32-devkitc/index.html   |   2 +-
 content/docs/12.2.0/searchindex.js |   2 +-
 content/docs/12.2.1/index.html |   2 +-
 content/docs/12.2.1/searchindex.js |   2 +-
 content/docs/12.3.0/index.html |   2 +-
 content/docs/12.3.0/searchindex.js |   2 +-
 content/docs/12.4.0/index.html |   2 +-
 content/docs/12.4.0/searchindex.js |   2 +-
 content/docs/12.5.0/index.html |   2 +-
 content/docs/12.5.0/searchindex.js |   2 +-
 content/docs/12.5.1/index.html |   2 +-
 content/docs/12.5.1/searchindex.js |   2 +-
 content/docs/12.6.0/index.html |   2 +-
 content/docs/12.6.0/searchindex.js |   2 +-
 content/docs/12.7.0/index.html |   2 +-
 content/docs/12.7.0/searchindex.js |   2 +-
 content/docs/12.8.0/index.html |   2 +-
 content/docs/12.8.0/searchindex.js |   2 +-
 content/docs/12.9.0/index.html |   2 +-
 content/docs/12.9.0/searchindex.js |   2 +-
 .../_sources/components/libs/libc/index.rst.txt|   1 +
 .../_sources/components/libs/libc/stream.rst.txt   | 181 +++
 .../risc-v/qemu-rv/boards/rv-virt/index.rst.txt|  33 ++
 .../docs/latest/components/libs/libc/index.html|   8 +-
 .../libs/libc/{index.html => stream.html}  | 335 -
 .../docs/latest/components/libs/libc/zoneinfo.html |   6 +-
 content/docs/latest/guides/fully_linked_elf.html   |   2 +-
 .../docs/latest/guides/partially_linked_elf.html   |   2 +-
 content/docs/latest/index.html |   2 +-
 content/docs/latest/objects.inv| Bin 156301 -> 156373 bytes
 .../risc-v/qemu-rv/boards/rv-virt/index.html   |  29 ++
 content/docs/latest/searchindex.js |   2 +-
 content/feed.xml   |   4 +-
 45 files changed, 481 insertions(+), 188 deletions(-)

diff --git a/content/docs/10.0.0/index.html b/content/docs/10.0.0/index.html
index 1db25331f..7dd9ba2fd 100644
--- a/content/docs/10.0.0/index.html
+++ b/content/docs/10.0.0/index.html
@@ -207,7 +207,7 @@ by following these 
 NuttX Documentation¶
 NuttX is a real-time operating system (RTOS) with an emphasis on standards 
compliance and small footprint. Scalable from 8-bit to 32-bit microcontroller 
environments, the primary governing standards in NuttX are Posix and ANSI 
standards. Additional standard APIs from Unix and other common RTOS’s (such as 
VxWorks) are adopted for functionality not available under these standards, or 
for functionality that is not appropriate for deeply-embedded environments 
(such as fork()).
-Last Updated: 22 June 25 at 00:16
+Last Updated: 23 June 25 at 00:16
 
 Table of Contents
 
diff --git a/content/docs/10.0.1/index.html b/content/docs/10.0.1/index.html
index 997255fed..190732d4a 100644
--- a/content/docs/10.0.1/index.html
+++ b/content/docs/10.0.1/index.html
@@ -243,7 +243,7 @@ by following these 
 NuttX Documentation¶
 NuttX is a real-time operating system (RTOS) with an emphasis on standards 
compliance and small footprint. Scalable from 8-bit to 32-bit microcontroller 
environments, the primary governing standards in NuttX are Posix and ANSI 
standards. Additional standard APIs from Unix and other common RTOS’s (such as 
VxWorks) are adopted for functionality not available under these standards, or 
for functionality that is not appropriat

(nuttx) branch master updated: libc: Move stdio/README to Documentation/components/libs/libc/stream.rst

2025-06-22 Thread acassis
This is an automated email from the ASF dual-hosted git repository.

acassis pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/nuttx.git


The following commit(s) were added to refs/heads/master by this push:
 new bc4041db60 libc: Move stdio/README to 
Documentation/components/libs/libc/stream.rst
bc4041db60 is described below

commit bc4041db608e9a04fd811dd74bcc67d8bded0d25
Author: Xiang Xiao 
AuthorDate: Fri Jun 20 09:50:00 2025 +0800

libc: Move stdio/README to Documentation/components/libs/libc/stream.rst

follow up this change:
commit 84dc88730cd27ec7ab8ce7cefac89e3763dfd2d5
Author: Xiang Xiao 
Date:   Mon Jun 9 14:39:03 2025 +0800

libc: Move stream printf/scanf from libc/stdio to libc/stream

to keep all related code in one place

Signed-off-by: Xiang Xiao 
---
 Documentation/components/libs/libc/index.rst  |   1 +
 Documentation/components/libs/libc/stream.rst | 181 ++
 libs/libc/stdio/README.md | 155 --
 3 files changed, 182 insertions(+), 155 deletions(-)

diff --git a/Documentation/components/libs/libc/index.rst 
b/Documentation/components/libs/libc/index.rst
index fac61e12bf..0394d43dfa 100644
--- a/Documentation/components/libs/libc/index.rst
+++ b/Documentation/components/libs/libc/index.rst
@@ -153,4 +153,5 @@ Implementation Details
:maxdepth: 1
:caption: Contents:

+   stream.rst
zoneinfo.rst
diff --git a/Documentation/components/libs/libc/stream.rst 
b/Documentation/components/libs/libc/stream.rst
new file mode 100644
index 00..fbb446060e
--- /dev/null
+++ b/Documentation/components/libs/libc/stream.rst
@@ -0,0 +1,181 @@
+lib_bsprintf
+
+
+This function is mainly used to output the contents of the input
+structure. Supports standard formats for printf and scanf. For detailed
+parameters, see: 1. https://en.cppreference.com/w/c/io/fprintf 2.
+https://en.cppreference.com/w/c/io/fscanf
+
+-  **special**:
+
+   1. Float use %hf, “%f” or “%lf” is double, “%Lf” is long double.
+   2. The char array is specified with %.xs. for example: “char t[30]”
+  is specified with “%.30s”, char a [20] - " %.20s "
+   3. “%u” is unsigned int.
+   4. “%d” is int.
+   5. When using %f to format a double data type, the double is
+  truncated to 6 decimal places by default.
+   6. It is recommended that the “char[]” array be placed at the end of
+  the structure to prevent parameter configuration errors such as
+  “%.20s” from causing problems in parsing the entire buffer.
+
+-  **demo**
+
+   1. **struct**:
+
+   ::
+
+  begin_packed_struct
+  struct test
+  {
+uint8_t a;
+uint16_t b;
+uint32_t c;
+int8_t d;
+int16_t e;
+int32_t f;
+float g;
+double h;
+char i[32];
+uint64_t j;
+int64_t k;
+char l;
+unsigned char m;
+short int n;
+unsigned short int o;
+int p;
+unsigned int q;
+long r;
+unsigned long s;
+long long t;
+unsigned long long u;
+size_t v;
+long double w;
+  }end_packed_struct;
+
+   1. **format string**:
+
+   ::
+
+  const char* sg = "   uint8_t:[%hhu]\n" \
+   "  uint16_t:[%hu]\n" \
+   "  uint32_t:[%u]\n" \
+   "int8_t:[%hhd]\n" \
+   "   int16_t:[%hd]\n" \
+   "   int32_t:[%d]\n" \
+   " float:[%hf]\n" \
+   "double:[%f]\n" \
+   "char[]:[%.32s]\n" \
+   "  uint64_t:[%lu]\n" \
+   "   int64_t:[%ld]\n" \
+   "  char:[%hhd]\n" \
+   " unsigned char:[%hhu]\n" \
+   " short int:[%hd]\n" \
+   "unsigned short int:[%hu]\n" \
+   "   int:[%d]\n" \
+   "  unsigned int:[%u]\n" \
+   "  long:[%ld]\n" \
+   " unsigned long:[%lu]\n" \
+   " long long:[%lld]\n" \
+   "unsigned long long:[%llu]\n" \
+   "size_t:[%uz]\n" \
+   "   long double:[%Lf]\n";
+
+   1. **use**:
+
+   -  output to terminal:
+
+   ::
+
+  #ifdef CONFIG_FILE_STREAM
+   struct lib_stdoutstream_s stdoutstream;
+
+   lib_stdoutstream(&stdoutstream, stdout);
+
+   flockfile(stdout);
+   lib_bsprintf(&stdoutstream.common, sv, &test_v);
+   lib_bsprintf(&stdoutstream.common, sg, &test_g);
+   funlockfile(stdout);
+  #else
+   struct lib_rawoutstream_s rawoutstream;
+   struct lib_buffered

Re: [PR] [New Board] ESP32S3-8048s043 [nuttx]

2025-06-22 Thread via GitHub


xiaoxiang781216 commented on PR #16558:
URL: https://github.com/apache/nuttx/pull/16558#issuecomment-2994114297

   still not fix all problem:
   ```
   < # CONFIG_ARCH_LEDS is not set
   Saving the new configuration file
   HEAD detached at pull/16558/merge
   Changes not staged for commit:
 (use "git add ..." to update what will be committed)
 (use "git restore ..." to discard changes in working directory)
modified:   boards/xtensa/esp32s3/esp32s3-8048S043/configs/nsh/defconfig
   ```


-- 
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...@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [I] [BUG] CONFIG_FILE_STREAM needs to be enabled by default for some applications (i.e. can example) [nuttx]

2025-06-22 Thread via GitHub


xiaoxiang781216 commented on issue #16570:
URL: https://github.com/apache/nuttx/issues/16570#issuecomment-2994258263

   > [@xiaoxiang781216](https://github.com/xiaoxiang781216) I think we need to 
verify which applications will need to select CONFIG_FILE_STREAM automatically 
to avoid issues like this
   
   not select, but depend on CONFIG_FILE_STREAM.
   BTW, the better fix is switching fopen/fclose to open/close as much as we 
can.


-- 
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...@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] [New Board] ESP32S3-8048s043 [nuttx]

2025-06-22 Thread via GitHub


halyssonJr commented on PR #16558:
URL: https://github.com/apache/nuttx/pull/16558#issuecomment-2994273647

   @xiaoxiang781216 , I opened another 
PR(https://github.com/apache/nuttx/pull/16571) to fix this issue:
   
   ```
   input/gt9xx.c: In function 'gt9xx_i2c_read':
   Error: input/gt9xx.c:199:9: error: format '%ld' expects argument of type 
'long int', but argument 4 has type 'size_t' {aka 'unsigned int'} 
[-Werror=format=]
 199 |   iinfo("reg=0x%x, buflen=%ld\n", reg, buflen);
 | ^~~~   ~~
 ||
 |size_t {aka unsigned int}
   input/gt9xx.c:199:29: note: format string is defined here
 199 |   iinfo("reg=0x%x, buflen=%ld\n", reg, buflen);
 |   ~~^
 | |
 | long int
 |   %d
   input/gt9xx.c: In function 'gt9xx_read':
   Error: input/gt9xx.c:485:9: error: format '%ld' expects argument of type 
'long int', but argument 3 has type 'size_t' {aka 'unsigned int'} 
[-Werror=format=]
 485 |   iinfo("buflen=%ld\n", buflen);
 | ^~  ~~
 | |
 | size_t {aka unsigned int}
   input/gt9xx.c:485:19: note: format string is defined here
 485 |   iinfo("buflen=%ld\n", buflen);
 | ~~^
 |   |
 |   long int
 | %d
   Error: input/gt9xx.c:488:12: error: format '%ld' expects argument of type 
'long int', but argument 3 has type 'size_t' {aka 'unsigned int'} 
[-Werror=format=]
 488 |   ierr("Buffer should be at least %ld bytes, got %ld bytes\n",
 |^~
 489 |outlen, buflen);
 |~~
 ||
 |size_t {aka unsigned int}
   input/gt9xx.c:488:41: note: format string is defined here
 488 |   ierr("Buffer should be at least %ld bytes, got %ld bytes\n",
 |   ~~^
 | |
 | long int
 |   %d
   Error: input/gt9xx.c:488:12: error: format '%ld' expects argument of type 
'long int', but argument 4 has type 'size_t' {aka 'unsigned int'} 
[-Werror=format=]
 488 |   ierr("Buffer should be at least %ld bytes, got %ld bytes\n",
 |^~
 489 |outlen, buflen);
 |~~
 ||
 |size_t {aka unsigned int}
   input/gt9xx.c:488:56: note: format string is defined here
 488 |   ierr("Buffer should be at least %ld bytes, got %ld bytes\n",
 |  ~~^
 ||
 |long int
 |  %d
   cc1: all warnings being treated as errors
   make[1]: *** [Makefile:109: gt9xx.o] Error 1
   make[1]: Target 'libdrivers.a' not remade because of errors.
   ``` 


-- 
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...@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[PR] remove warnings [nuttx]

2025-06-22 Thread via GitHub


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

   ## Summary
   The PR aims to adjust the format according to what was expected. Also, 
remove some compilation warnings and stop breaking the pipeline (Linux 
Xtensa-02)
   
   ## Impact
   None
   ## Testing
   After multiple attempts to submit PR #16558, the CI/CD displays this error.
   
   


-- 
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...@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [I] [BUG] CONFIG_FILE_STREAM needs to be enabled by default for some applications (i.e. can example) [nuttx]

2025-06-22 Thread via GitHub


acassis commented on issue #16570:
URL: https://github.com/apache/nuttx/issues/16570#issuecomment-2994234837

   @xiaoxiang781216 I think we need to verify which applications will need to 
select CONFIG_FILE_STREAM automatically to avoid issues like this


-- 
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...@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[I] [BUG] CONFIG_FILE_STREAM needs to be enabled by default for some applications (i.e. can example) [nuttx]

2025-06-22 Thread via GitHub


acassis opened a new issue, #16570:
URL: https://github.com/apache/nuttx/issues/16570

   ### Description / Steps to reproduce the issue
   
   I was getting this error after enabling CAN example:
   
   LD: nuttx
   arm-none-eabi-ld: 
/home/alan/nuttxspace/nuttx/staging/libapps.a(can_main.c.home.alan.nuttxspace.apps.examples.can_1.o):
 in function `show_usage':
   can_main.c:(.text.show_usage+0x6): undefined reference to `lib_get_stream'
   arm-none-eabi-ld: can_main.c:(.text.show_usage+0xe): undefined reference to 
`fprintf'
   arm-none-eabi-ld: can_main.c:(.text.show_usage+0x14): undefined reference to 
`lib_get_stream'
   arm-none-eabi-ld: can_main.c:(.text.show_usage+0x1c): undefined reference to 
`fprintf'
   arm-none-eabi-ld: can_main.c:(.text.show_usage+0x22): undefined reference to 
`lib_get_stream'
   arm-none-eabi-ld: can_main.c:(.text.show_usage+0x2a): undefined reference to 
`fputs'
   arm-none-eabi-ld: can_main.c:(.text.show_usage+0x30): undefined reference to 
`lib_get_stream'
   arm-none-eabi-ld: can_main.c:(.text.show_usage+0x38): undefined reference to 
`fputs'
   arm-none-eabi-ld: can_main.c:(.text.show_usage+0x3e): undefined reference to 
`lib_get_stream'
   arm-none-eabi-ld: can_main.c:(.text.show_usage+0x46): undefined reference to 
`fputs'
   arm-none-eabi-ld: can_main.c:(.text.show_usage+0x4c): undefined reference to 
`lib_get_stream'
   arm-none-eabi-ld: can_main.c:(.text.show_usage+0x54): undefined reference to 
`fputs'
   arm-none-eabi-ld: can_main.c:(.text.show_usage+0x5a): undefined reference to 
`lib_get_stream'
   arm-none-eabi-ld: can_main.c:(.text.show_usage+0x64): undefined reference to 
`fprintf'
   arm-none-eabi-ld: can_main.c:(.text.show_usage+0x6a): undefined reference to 
`lib_get_stream'
   arm-none-eabi-ld: can_main.c:(.text.show_usage+0x76): undefined reference to 
`fputs'
   arm-none-eabi-ld: 
/home/alan/nuttxspace/nuttx/staging/libapps.a(can_main.c.home.alan.nuttxspace.apps.examples.can_1.o):
 in function `can_main':
   can_main.c:(.text.can_main+0x54): undefined reference to `lib_get_stream'
   arm-none-eabi-ld: can_main.c:(.text.can_main+0x84): undefined reference to 
`lib_get_stream'
   arm-none-eabi-ld: can_main.c:(.text.can_main+0x8c): undefined reference to 
`fputs'
   arm-none-eabi-ld: can_main.c:(.text.can_main+0xb0): undefined reference to 
`lib_get_stream'
   arm-none-eabi-ld: can_main.c:(.text.can_main+0xce): undefined reference to 
`lib_get_stream'
   arm-none-eabi-ld: can_main.c:(.text.can_main+0xda): undefined reference to 
`lib_get_stream'
   arm-none-eabi-ld: can_main.c:(.text.can_main+0x122): undefined reference to 
`lib_get_stream'
   arm-none-eabi-ld: can_main.c:(.text.can_main+0x12a): undefined reference to 
`fputs'
   arm-none-eabi-ld: can_main.c:(.text.can_main+0x16e): undefined reference to 
`lib_get_stream'
   arm-none-eabi-ld: can_main.c:(.text.can_main+0x172): undefined reference to 
`fflush'
   arm-none-eabi-ld: can_main.c:(.text.can_main+0x1da): undefined reference to 
`lib_get_stream'
   arm-none-eabi-ld: can_main.c:(.text.can_main+0x1de): undefined reference to 
`fflush'
   
   It was fixed after enabling CONFIG_FILE_STREAM=y
   
   ### On which OS does this issue occur?
   
   [OS: Other], [OS: Linux]
   
   ### What is the version of your OS?
   
   Ubuntu 24.04
   
   ### NuttX Version
   
   mainstream
   
   ### Issue Architecture
   
   [Arch: all]
   
   ### Issue Area
   
   [Area: Applications]
   
   ### Host information
   
   _No response_
   
   ### Verification
   
   - [x] I have verified before submitting the report.


-- 
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...@nuttx.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [I] [BUG] CONFIG_FILE_STREAM needs to be enabled by default for some applications (i.e. can example) [nuttx]

2025-06-22 Thread via GitHub


TimJTi commented on issue #16570:
URL: https://github.com/apache/nuttx/issues/16570#issuecomment-2994322674

   
   >...BTW, the better fix is switching fopen/fclose to open/close as much as 
we can.
   
   I thought fopen/fclose was the preferred, portable, method? Since open/close 
are not ANSI C IIRC.


-- 
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...@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org