(nuttx-apps) branch master updated: system/nxinit: Revert "Give the builtin list to the kernel"

[casaroli]

This is an automated email from the ASF dual-hosted git repository.

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


The following commit(s) were added to refs/heads/master by this push:
     new 303c33d18 system/nxinit: Revert "Give the builtin list to the kernel"
303c33d18 is described below

commit 303c33d18914f7b95782f464022cca268c28de85
Author: wangjianyu3 <wangjian...@xiaomi.com>
AuthorDate: Sun Oct 25 20:39:05 2026 +0800

    system/nxinit: Revert "Give the builtin list to the kernel"
    
    This reverts commit d9454d89d02.
    
    apache/nuttx#20964 removed BOARDIOC_BUILTINS and struct boardioc_builtin_s,
    so nxinit no longer builds in a protected build:
    
      init.c:214:29: error: storage size of 'builtin' isn't known
      init.c:218:12: error: 'BOARDIOC_BUILTINS' undeclared (first use in this 
function)
    
    Assisted-by: Kiro:claude-opus-5.5
    Signed-off-by: wangjianyu3 <wangjian...@xiaomi.com>
---
 system/nxinit/init.c | 27 ---------------------------
 1 file changed, 27 deletions(-)

diff --git a/system/nxinit/init.c b/system/nxinit/init.c
index 2400edb12..d0329a114 100644
--- a/system/nxinit/init.c
+++ b/system/nxinit/init.c
@@ -37,7 +37,6 @@
 #include <sys/param.h>
 #include <sys/wait.h>
 
-#include <nuttx/lib/builtin.h>
 #include <nuttx/usb/usbdev_trace.h>
 
 #include "action.h"
@@ -197,30 +196,6 @@ static int init_boot_reason(FAR struct action_manager_s 
*am)
 #  define init_boot_reason(am) (0)
 #endif
 
-/****************************************************************************
- * Name: init_builtins
- *
- * Description:
- *   In a protected build the kernel cannot read the list of builtin
- *   applications in user space, so posix_spawn() of a builtin fails with
- *   ENOENT until the list is given to the kernel.
- *
- ****************************************************************************/
-
-#if defined(CONFIG_BUILD_PROTECTED) && defined(CONFIG_BUILTIN) && \
-    defined(CONFIG_BOARDCTL)
-static void init_builtins(void)
-{
-  struct boardioc_builtin_s builtin;
-
-  builtin.builtins = g_builtins;
-  builtin.count    = g_builtin_count;
-  boardctl(BOARDIOC_BUILTINS, (uintptr_t)&builtin);
-}
-#else
-#  define init_builtins()
-#endif
-
 /****************************************************************************
  * Public Functions
  ****************************************************************************/
@@ -267,8 +242,6 @@ int main(int argc, FAR char *argv[])
   size_t i;
   int r;
 
-  init_builtins();
-
 #ifdef CONFIG_BMP
   snprintf(ACTION_TRIGGER_KEY(&am), sizeof(ACTION_TRIGGER_KEY(&am)),
            ACTION_PROP_KEY_TRIGGER ".core%d", sched_getcpu());