[GitHub] [incubator-nuttx] pussuw commented on pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-03-24 Thread GitBox
pussuw commented on pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#issuecomment-1077299391 > Ok, I get your point. U-mode thread never directly initiative the context switch through riscv_switch_context, instead through other normal syscall or the hardware int

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-03-24 Thread GitBox
xiaoxiang781216 commented on pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#issuecomment-1077307022 Why not implement OpenSBI API in M-mode required by S-mode NuttX directly? I think is more simple and general than a private MiniSBI. -- This is an automated

[GitHub] [incubator-nuttx] xiaoxiang781216 edited a comment on pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-03-24 Thread GitBox
xiaoxiang781216 edited a comment on pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#issuecomment-1077307022 Why not implement OpenSBI API in M-mode required by S-mode NuttX directly? I think is more simple and general than a private MiniSBI. We can write from s

[GitHub] [incubator-nuttx] xiaoxiang781216 edited a comment on pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-03-24 Thread GitBox
xiaoxiang781216 edited a comment on pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#issuecomment-1077307022 Why not implement OpenSBI API in M-mode required by S-mode NuttX directly? I think is more simple and general than a private MiniSBI. We can write from s

[GitHub] [incubator-nuttx] xiaoxiang781216 edited a comment on pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-03-24 Thread GitBox
xiaoxiang781216 edited a comment on pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#issuecomment-1077307022 Why not implement OpenSBI API in M-mode required by S-mode NuttX directly? I think is more simple and general than a private MiniSBI. We can write from s

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #5835: LITESDCARD peripheral driver support

2022-03-24 Thread GitBox
xiaoxiang781216 commented on pull request #5835: URL: https://github.com/apache/incubator-nuttx/pull/5835#issuecomment-1077319163 Please fix the warning report here: https://github.com/apache/incubator-nuttx/runs/5671271324?check_suite_focus=true -- This is an automated message from t

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5835: LITESDCARD peripheral driver support

2022-03-24 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5835: URL: https://github.com/apache/incubator-nuttx/pull/5835#discussion_r833996279 ## File path: arch/risc-v/src/litex/litex_cache.c ## @@ -0,0 +1,76 @@ +/**

[GitHub] [incubator-nuttx] anjiahao1 opened a new pull request #5836: tools:use dump log creat a gdbsever to debug

2022-03-24 Thread GitBox
anjiahao1 opened a new pull request #5836: URL: https://github.com/apache/incubator-nuttx/pull/5836 this is a tool,when crash help you to debug. Signed-off-by: anjiahao ## introduce This tool can help you analyze the call stack information when the system crashes ##

[GitHub] [incubator-nuttx] pussuw commented on pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-03-24 Thread GitBox
pussuw commented on pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#issuecomment-1077331325 > Why not implement OpenSBI API in M-mode required by S-mode NuttX directly? I think is more simple and general than a private MiniSBI. We can write from scratch in M-mo

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-03-24 Thread GitBox
xiaoxiang781216 commented on pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#issuecomment-1077337012 Yes, I understand, that is why I emphasize that M-mode only need implement the minimal API(subset of function and argument) which is actually used by S-mode Nut

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-03-24 Thread GitBox
xiaoxiang781216 commented on pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#issuecomment-1077339652 Maybe we can change CONFIG_OPENSBI at https://github.com/apache/incubator-nuttx/tree/master/arch/risc-v/src/opensbi from bool to choice and add a new minimal im

[GitHub] [incubator-nuttx] lupyuen commented on issue #5810: GPIO issues on BL602 + suggestions for RISC-V, maybe more.

2022-03-24 Thread GitBox
lupyuen commented on issue #5810: URL: https://github.com/apache/incubator-nuttx/issues/5810#issuecomment-1077355494 Cool thanks! BL602 MISO / MOSI Swap is in the master branch here: [bl602_spi.c](https://github.com/apache/incubator-nuttx/blob/master/arch/risc-v/src/bl602/bl602_spi.c#L1137

[GitHub] [incubator-nuttx] jlaitine commented on pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-03-24 Thread GitBox
jlaitine commented on pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#issuecomment-1077368297 I think that we are responsible for the SBI confusion, so let me explain a bit where we are. The OpenSBI library, which we added (by @eenurkka) is not really use

[GitHub] [incubator-nuttx] AndrewD commented on a change in pull request #5835: LITESDCARD peripheral driver support

2022-03-24 Thread GitBox
AndrewD commented on a change in pull request #5835: URL: https://github.com/apache/incubator-nuttx/pull/5835#discussion_r834056629 ## File path: include/nuttx/sdio.h ## @@ -477,6 +477,7 @@ #define SDIO_CAPS_DMABEFOREWRITE 0x04 /* Bit 2=1: Executes DMA before write command *

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-03-24 Thread GitBox
xiaoxiang781216 commented on pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#issuecomment-1077383465 Since OpenSBI is the standard way to expose M-mode functionality to S-mode, what I suggest it's that we use OpenSBI API definition, but write our own minimal im

[GitHub] [incubator-nuttx] anjiahao1 commented on pull request #5836: tools:use dump log creat a gdbsever to debug

2022-03-24 Thread GitBox
anjiahao1 commented on pull request #5836: URL: https://github.com/apache/incubator-nuttx/pull/5836#issuecomment-1077384838 maybe Lint CI have conflict,about `:` 🐛 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [incubator-nuttx] xiaoxiang781216 edited a comment on pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-03-24 Thread GitBox
xiaoxiang781216 edited a comment on pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#issuecomment-1077383465 > I think that we are responsible for the SBI confusion, so let me explain a bit where we are. > > The OpenSBI library, which we added (by @eenurkk

[GitHub] [incubator-nuttx] xiaoxiang781216 edited a comment on pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-03-24 Thread GitBox
xiaoxiang781216 edited a comment on pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#issuecomment-1077383465 > I think that we are responsible for the SBI confusion, so let me explain a bit where we are. > > The OpenSBI library, which we added (by @eenurkk

[GitHub] [incubator-nuttx] jlaitine commented on pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-03-24 Thread GitBox
jlaitine commented on pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#issuecomment-1077406182 > Since OpenSBI is the standard way to expose M-mode functionality to S-mode, what I suggest it's that we use OpenSBI API definition, but write our own minimal imp

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-03-24 Thread GitBox
xiaoxiang781216 commented on pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#issuecomment-1077456944 > > Since OpenSBI is the standard way to expose M-mode functionality to S-mode, what I suggest it's that we use OpenSBI API definition, but write our own minima

[GitHub] [incubator-nuttx] xiaoxiang781216 edited a comment on pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-03-24 Thread GitBox
xiaoxiang781216 edited a comment on pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#issuecomment-1077456944 > > Since OpenSBI is the standard way to expose M-mode functionality to S-mode, what I suggest it's that we use OpenSBI API definition, but write our own

[GitHub] [incubator-nuttx] pussuw commented on pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-03-24 Thread GitBox
pussuw commented on pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#issuecomment-1077474453 I moved the SBI parts out of this review. I will publish that part later and the discussion about the SBI can be moved there. For now there are stub wrappers that

[GitHub] [incubator-nuttx] acassis commented on pull request #5835: LITESDCARD peripheral driver support

2022-03-24 Thread GitBox
acassis commented on pull request #5835: URL: https://github.com/apache/incubator-nuttx/pull/5835#issuecomment-1077562013 Hi @rtucker85 could you please fix these issues: ``` ../nuttx/tools/checkpatch.sh -g c92c4af304dcc09254dc4023ca822fef11d98233..HEAD Error: /home/runner/work/i

[GitHub] [incubator-nuttx] gustavonihei commented on a change in pull request #5836: tools:use dump log creat a gdbsever to debug

2022-03-24 Thread GitBox
gustavonihei commented on a change in pull request #5836: URL: https://github.com/apache/incubator-nuttx/pull/5836#discussion_r834392204 ## File path: tools/minidumpserver.py ## @@ -0,0 +1,530 @@ +#!/usr/bin/env python3 +# tools/minidumpserver.py +# +# Licensed to the Apache So

[GitHub] [incubator-nuttx] xiaoxiang781216 opened a new pull request #5837: libc: Implement getrandom on top of "/dev/[u]random"

2022-03-24 Thread GitBox
xiaoxiang781216 opened a new pull request #5837: URL: https://github.com/apache/incubator-nuttx/pull/5837 ## Summary - libc: Implement getrandom on top of "/dev/[u]random" - libc/uuid: Call getrandom instaed arc4random_buf ## Impact New API ## Testing Pass CI

[GitHub] [incubator-nuttx] gustavonihei commented on a change in pull request #5836: tools:use dump log creat a gdbsever to debug

2022-03-24 Thread GitBox
gustavonihei commented on a change in pull request #5836: URL: https://github.com/apache/incubator-nuttx/pull/5836#discussion_r834398608 ## File path: tools/minidumpserver.py ## @@ -0,0 +1,530 @@ +#!/usr/bin/env python3 +# tools/minidumpserver.py +# +# Licensed to the Apache So

[GitHub] [incubator-nuttx] gustavonihei commented on pull request #5836: tools:use dump log creat a gdbsever to debug

2022-03-24 Thread GitBox
gustavonihei commented on pull request #5836: URL: https://github.com/apache/incubator-nuttx/pull/5836#issuecomment-1077752838 > maybe Lint CI have conflict,about `:` bug Aren't we able to fix the linker issues? Something like: `line[tmp.span()[1] :]` into `line[tmp.span()[1] : ]`

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5837: libc: Implement getrandom on top of "/dev/[u]random"

2022-03-24 Thread GitBox
pkarashchenko commented on a change in pull request #5837: URL: https://github.com/apache/incubator-nuttx/pull/5837#discussion_r834434602 ## File path: libs/libc/misc/lib_getrandom.c ## @@ -0,0 +1,91 @@ +/*

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5836: tools:use dump log creat a gdbsever to debug

2022-03-24 Thread GitBox
pkarashchenko commented on a change in pull request #5836: URL: https://github.com/apache/incubator-nuttx/pull/5836#discussion_r83659 ## File path: tools/minidumpserver.py ## @@ -0,0 +1,530 @@ +#!/usr/bin/env python3 +# tools/minidumpserver.py +# +# Licensed to the Apache S

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5836: tools:use dump log creat a gdbsever to debug

2022-03-24 Thread GitBox
pkarashchenko commented on a change in pull request #5836: URL: https://github.com/apache/incubator-nuttx/pull/5836#discussion_r834446920 ## File path: tools/minidumpserver.py ## @@ -0,0 +1,530 @@ +#!/usr/bin/env python3 +# tools/minidumpserver.py +# +# Licensed to the Apache S

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5837: libc: Implement getrandom on top of "/dev/[u]random"

2022-03-24 Thread GitBox
pkarashchenko commented on a change in pull request #5837: URL: https://github.com/apache/incubator-nuttx/pull/5837#discussion_r834450589 ## File path: libs/libc/misc/lib_getrandom.c ## @@ -0,0 +1,91 @@ +/*

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-03-24 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#discussion_r834451712 ## File path: arch/risc-v/src/common/riscv_assert.c ## @@ -338,6 +338,10 @@ static void riscv_dumpstate(void) else { riscv_

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5837: libc: Implement getrandom on top of "/dev/[u]random"

2022-03-24 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5837: URL: https://github.com/apache/incubator-nuttx/pull/5837#discussion_r834479860 ## File path: libs/libc/misc/lib_getrandom.c ## @@ -0,0 +1,91 @@ +/***

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5837: libc: Implement getrandom on top of "/dev/[u]random"

2022-03-24 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5837: URL: https://github.com/apache/incubator-nuttx/pull/5837#discussion_r834480277 ## File path: libs/libc/misc/lib_getrandom.c ## @@ -0,0 +1,91 @@ +/***

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5837: libc: Implement getrandom on top of "/dev/[u]random"

2022-03-24 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5837: URL: https://github.com/apache/incubator-nuttx/pull/5837#discussion_r834482992 ## File path: libs/libc/uuid/lib_uuid_create.c ## @@ -42,25 +43,31 @@ void uuid_create(uuid_t *u, uint32_t *status) { -#ifdef CONFIG_C

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5837: libc: Implement getrandom on top of "/dev/[u]random"

2022-03-24 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5837: URL: https://github.com/apache/incubator-nuttx/pull/5837#discussion_r834483165 ## File path: libs/libc/uuid/lib_uuid_create.c ## @@ -42,25 +43,31 @@ void uuid_create(uuid_t *u, uint32_t *status) { -#ifdef CONFIG_C

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5837: libc: Implement getrandom on top of "/dev/[u]random"

2022-03-24 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5837: URL: https://github.com/apache/incubator-nuttx/pull/5837#discussion_r834485253 ## File path: include/sys/random.h ## @@ -0,0 +1,79 @@ +/ + *

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5837: libc: Implement getrandom on top of "/dev/[u]random"

2022-03-24 Thread GitBox
pkarashchenko commented on a change in pull request #5837: URL: https://github.com/apache/incubator-nuttx/pull/5837#discussion_r834489067 ## File path: libs/libc/uuid/lib_uuid_create.c ## @@ -42,20 +43,27 @@ void uuid_create(uuid_t *u, uint32_t *status) { -#ifdef CONFIG_CRY

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5837: libc: Implement getrandom on top of "/dev/[u]random"

2022-03-24 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5837: URL: https://github.com/apache/incubator-nuttx/pull/5837#discussion_r834498581 ## File path: libs/libc/uuid/lib_uuid_create.c ## @@ -42,20 +43,27 @@ void uuid_create(uuid_t *u, uint32_t *status) { -#ifdef CONFIG_C

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5837: libc: Implement getrandom on top of "/dev/[u]random"

2022-03-24 Thread GitBox
pkarashchenko commented on a change in pull request #5837: URL: https://github.com/apache/incubator-nuttx/pull/5837#discussion_r834502554 ## File path: include/sys/random.h ## @@ -0,0 +1,79 @@ +/ + * in

[GitHub] [incubator-nuttx-apps] pkarashchenko opened a new pull request #1088: fsutils/passwd: fix compilation

2022-03-24 Thread GitBox
pkarashchenko opened a new pull request #1088: URL: https://github.com/apache/incubator-nuttx-apps/pull/1088 ## Summary `passwd` compilation fails with ``` ./passwd_verify.c:29:10: fatal error: passwd.h: No such file or directory 29 | #include ``` ## Impact None.

[GitHub] [incubator-nuttx] gustavonihei commented on a change in pull request #5837: libc: Implement getrandom on top of "/dev/[u]random"

2022-03-24 Thread GitBox
gustavonihei commented on a change in pull request #5837: URL: https://github.com/apache/incubator-nuttx/pull/5837#discussion_r834558642 ## File path: include/sys/random.h ## @@ -0,0 +1,79 @@ +/ + * inc

[GitHub] [incubator-nuttx] gustavonihei commented on a change in pull request #5837: libc: Implement getrandom on top of "/dev/[u]random"

2022-03-24 Thread GitBox
gustavonihei commented on a change in pull request #5837: URL: https://github.com/apache/incubator-nuttx/pull/5837#discussion_r834558642 ## File path: include/sys/random.h ## @@ -0,0 +1,79 @@ +/ + * inc

[GitHub] [incubator-nuttx] gustavonihei commented on a change in pull request #5837: libc: Implement getrandom on top of "/dev/[u]random"

2022-03-24 Thread GitBox
gustavonihei commented on a change in pull request #5837: URL: https://github.com/apache/incubator-nuttx/pull/5837#discussion_r834560415 ## File path: libs/libc/misc/lib_getrandom.c ## @@ -0,0 +1,91 @@ +/**

[GitHub] [incubator-nuttx] pussuw commented on a change in pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-03-24 Thread GitBox
pussuw commented on a change in pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#discussion_r834587806 ## File path: arch/risc-v/src/common/riscv_assert.c ## @@ -338,6 +338,10 @@ static void riscv_dumpstate(void) else { riscv_saveuserc

[GitHub] [incubator-nuttx] pussuw commented on a change in pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-03-24 Thread GitBox
pussuw commented on a change in pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#discussion_r834588116 ## File path: arch/risc-v/src/mpfs/Make.defs ## @@ -34,6 +34,14 @@ CMN_CSRCS += riscv_mdelay.c riscv_udelay.c riscv_copyfullstate.c CMN_CSRCS +=

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5837: libc: Implement getrandom on top of "/dev/[u]random"

2022-03-24 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5837: URL: https://github.com/apache/incubator-nuttx/pull/5837#discussion_r834590146 ## File path: include/sys/random.h ## @@ -0,0 +1,79 @@ +/ + *

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5837: libc: Implement getrandom on top of "/dev/[u]random"

2022-03-24 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5837: URL: https://github.com/apache/incubator-nuttx/pull/5837#discussion_r834590574 ## File path: libs/libc/misc/lib_getrandom.c ## @@ -0,0 +1,91 @@ +/***

[GitHub] [incubator-nuttx] pussuw commented on a change in pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-03-24 Thread GitBox
pussuw commented on a change in pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#discussion_r834588405 ## File path: arch/risc-v/src/common/riscv_swint.c ## @@ -196,12 +229,14 @@ int riscv_swint(int irq, void *context, void *arg) * save regis

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5837: libc: Implement getrandom on top of "/dev/[u]random"

2022-03-24 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5837: URL: https://github.com/apache/incubator-nuttx/pull/5837#discussion_r834605056 ## File path: include/sys/random.h ## @@ -0,0 +1,79 @@ +/ + *

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-03-24 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#discussion_r834611239 ## File path: arch/risc-v/src/common/riscv_assert.c ## @@ -338,6 +338,10 @@ static void riscv_dumpstate(void) else { riscv_

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-03-24 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#discussion_r834612123 ## File path: arch/risc-v/src/mpfs/Make.defs ## @@ -34,6 +34,14 @@ CMN_CSRCS += riscv_mdelay.c riscv_udelay.c riscv_copyfullstate.c CMN_

[GitHub] [incubator-nuttx] pussuw commented on a change in pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-03-24 Thread GitBox
pussuw commented on a change in pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#discussion_r834612920 ## File path: arch/risc-v/src/mpfs/Make.defs ## @@ -34,6 +34,14 @@ CMN_CSRCS += riscv_mdelay.c riscv_udelay.c riscv_copyfullstate.c CMN_CSRCS +=

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-03-24 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#discussion_r834614792 ## File path: arch/risc-v/include/syscall.h ## @@ -391,6 +396,64 @@ static inline uintptr_t sys_call6(unsigned int nbr, uintptr_t parm1,

[GitHub] [incubator-nuttx] pussuw commented on a change in pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-03-24 Thread GitBox
pussuw commented on a change in pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#discussion_r834615075 ## File path: arch/risc-v/src/common/riscv_assert.c ## @@ -338,6 +338,10 @@ static void riscv_dumpstate(void) else { riscv_saveuserc

[GitHub] [incubator-nuttx] pussuw commented on a change in pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-03-24 Thread GitBox
pussuw commented on a change in pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#discussion_r834617061 ## File path: arch/risc-v/include/syscall.h ## @@ -391,6 +396,64 @@ static inline uintptr_t sys_call6(unsigned int nbr, uintptr_t parm1, return

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-03-24 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#discussion_r834618522 ## File path: arch/risc-v/include/mode.h ## @@ -0,0 +1,91 @@ +/***

[GitHub] [incubator-nuttx] pussuw commented on a change in pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-03-24 Thread GitBox
pussuw commented on a change in pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#discussion_r834620526 ## File path: arch/risc-v/src/common/riscv_assert.c ## @@ -338,6 +338,10 @@ static void riscv_dumpstate(void) else { riscv_saveuserc

[GitHub] [incubator-nuttx] pussuw commented on a change in pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-03-24 Thread GitBox
pussuw commented on a change in pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#discussion_r834627145 ## File path: arch/risc-v/include/mode.h ## @@ -0,0 +1,91 @@ +/ + * arc

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-03-24 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#discussion_r834637806 ## File path: arch/risc-v/include/syscall.h ## @@ -391,6 +396,64 @@ static inline uintptr_t sys_call6(unsigned int nbr, uintptr_t parm1,

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-03-24 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#discussion_r834638548 ## File path: arch/risc-v/src/mpfs/Make.defs ## @@ -34,6 +34,14 @@ CMN_CSRCS += riscv_mdelay.c riscv_udelay.c riscv_copyfullstate.c CMN_

[GitHub] [incubator-nuttx] pussuw commented on a change in pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-03-24 Thread GitBox
pussuw commented on a change in pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#discussion_r834641431 ## File path: arch/risc-v/include/syscall.h ## @@ -391,6 +396,64 @@ static inline uintptr_t sys_call6(unsigned int nbr, uintptr_t parm1, return

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-03-24 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#discussion_r834643637 ## File path: arch/risc-v/src/common/riscv_assert.c ## @@ -338,6 +338,10 @@ static void riscv_dumpstate(void) else { riscv_

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-03-24 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#discussion_r834643637 ## File path: arch/risc-v/src/common/riscv_assert.c ## @@ -338,6 +338,10 @@ static void riscv_dumpstate(void) else { riscv_

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-03-24 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#discussion_r834643637 ## File path: arch/risc-v/src/common/riscv_assert.c ## @@ -338,6 +338,10 @@ static void riscv_dumpstate(void) else { riscv_

[GitHub] [incubator-nuttx] pussuw commented on a change in pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-03-24 Thread GitBox
pussuw commented on a change in pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#discussion_r834645556 ## File path: arch/risc-v/src/mpfs/Make.defs ## @@ -34,6 +34,14 @@ CMN_CSRCS += riscv_mdelay.c riscv_udelay.c riscv_copyfullstate.c CMN_CSRCS +=

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-03-24 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#discussion_r834647332 ## File path: arch/risc-v/include/mode.h ## @@ -0,0 +1,91 @@ +/***

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-03-24 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#discussion_r834649820 ## File path: arch/risc-v/src/mpfs/Make.defs ## @@ -34,6 +34,14 @@ CMN_CSRCS += riscv_mdelay.c riscv_udelay.c riscv_copyfullstate.c CMN_

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-03-24 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#discussion_r834649820 ## File path: arch/risc-v/src/mpfs/Make.defs ## @@ -34,6 +34,14 @@ CMN_CSRCS += riscv_mdelay.c riscv_udelay.c riscv_copyfullstate.c CMN_

[GitHub] [incubator-nuttx] pussuw commented on a change in pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-03-24 Thread GitBox
pussuw commented on a change in pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#discussion_r834651377 ## File path: arch/risc-v/include/mode.h ## @@ -0,0 +1,91 @@ +/ + * arc

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-03-24 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#discussion_r834649820 ## File path: arch/risc-v/src/mpfs/Make.defs ## @@ -34,6 +34,14 @@ CMN_CSRCS += riscv_mdelay.c riscv_udelay.c riscv_copyfullstate.c CMN_

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-03-24 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#discussion_r834649820 ## File path: arch/risc-v/src/mpfs/Make.defs ## @@ -34,6 +34,14 @@ CMN_CSRCS += riscv_mdelay.c riscv_udelay.c riscv_copyfullstate.c CMN_

[GitHub] [incubator-nuttx] pussuw commented on a change in pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-03-24 Thread GitBox
pussuw commented on a change in pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#discussion_r834653649 ## File path: arch/risc-v/src/common/riscv_assert.c ## @@ -338,6 +338,10 @@ static void riscv_dumpstate(void) else { riscv_saveuserc

[GitHub] [incubator-nuttx] pussuw commented on a change in pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-03-24 Thread GitBox
pussuw commented on a change in pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#discussion_r834653649 ## File path: arch/risc-v/src/common/riscv_assert.c ## @@ -338,6 +338,10 @@ static void riscv_dumpstate(void) else { riscv_saveuserc

[GitHub] [incubator-nuttx] pussuw commented on a change in pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-03-24 Thread GitBox
pussuw commented on a change in pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#discussion_r834653649 ## File path: arch/risc-v/src/common/riscv_assert.c ## @@ -338,6 +338,10 @@ static void riscv_dumpstate(void) else { riscv_saveuserc

[GitHub] [incubator-nuttx] patacongo commented on issue #5832: STM3240-G Build is Broken

2022-03-24 Thread GitBox
patacongo commented on issue #5832: URL: https://github.com/apache/incubator-nuttx/issues/5832#issuecomment-1078204479 > @patacongo - what is the configure line used? It was a custom configuration based on stm3240g-evel:nsh -- This is an automated message from the Apache Git Servi

[GitHub] [incubator-nuttx] patacongo commented on issue #5832: STM3240-G Build is Broken

2022-03-24 Thread GitBox
patacongo commented on issue #5832: URL: https://github.com/apache/incubator-nuttx/issues/5832#issuecomment-1078208072 Also, after getting a clean build by disabling buttons and SDIO mux, the resulting binary would not run. It hardfaulted very early. -- This is an automated message fro

[GitHub] [incubator-nuttx] Ouss4 opened a new pull request #5838: esp32/Make.defs: Organise common arch files and chip files appropriately.

2022-03-24 Thread GitBox
Ouss4 opened a new pull request #5838: URL: https://github.com/apache/incubator-nuttx/pull/5838 ## Summary Some chip files where grouped with the common files variable. ## Impact ESP32. ## Testing All configs still build error free. -- This is an automated message from

[GitHub] [incubator-nuttx] rtucker85 commented on a change in pull request #5835: LITESDCARD peripheral driver support

2022-03-24 Thread GitBox
rtucker85 commented on a change in pull request #5835: URL: https://github.com/apache/incubator-nuttx/pull/5835#discussion_r834786849 ## File path: include/nuttx/sdio.h ## @@ -477,6 +477,7 @@ #define SDIO_CAPS_DMABEFOREWRITE 0x04 /* Bit 2=1: Executes DMA before write command

[incubator-nuttx-website] branch asf-site updated: Publishing web: 1b39f27b512cbb1fc294b70e0e70a67d0092b98d docs: 97fa9b849c5c9433c94776967d9c4805e59e89a9

2022-03-24 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/incubator-nuttx-website.git The following commit(s) were added to refs/heads/asf-site by this push: new b30a906 Publishing web

[GitHub] [incubator-nuttx] rtucker85 commented on a change in pull request #5835: LITESDCARD peripheral driver support

2022-03-24 Thread GitBox
rtucker85 commented on a change in pull request #5835: URL: https://github.com/apache/incubator-nuttx/pull/5835#discussion_r834861334 ## File path: arch/risc-v/src/litex/litex_cache.c ## @@ -0,0 +1,76 @@ +/

[GitHub] [incubator-nuttx] lupyuen edited a comment on issue #5810: GPIO issues on BL602 + suggestions for RISC-V, maybe more.

2022-03-24 Thread GitBox
lupyuen edited a comment on issue #5810: URL: https://github.com/apache/incubator-nuttx/issues/5810#issuecomment-1077001344 ST7789 Driver runs in Kernel Space so I don't think it can access `/dev/gpio*`? There is a generic LCD Driver that wraps up the ST7789 Driver as `/dev/lcd*`:

[GitHub] [incubator-nuttx] yamt commented on pull request #5837: libc: Implement getrandom on top of "/dev/[u]random"

2022-03-24 Thread GitBox
yamt commented on pull request #5837: URL: https://github.com/apache/incubator-nuttx/pull/5837#issuecomment-1078600313 * is this meant for linux compat? why not getentropy, which is accepted by posix? * it seems dangerous to ignore !GRND_INSECURE. -- This is an automated message from

[GitHub] [incubator-nuttx-apps] PeterBee97 opened a new pull request #1089: apps/mlearning: tweak libcmsisdsp makefile

2022-03-24 Thread GitBox
PeterBee97 opened a new pull request #1089: URL: https://github.com/apache/incubator-nuttx-apps/pull/1089 only one .c needed for each function group add -flax-vector-conversions to avoid build error on gcc && M55 Signed-off-by: Peter Bee ## Summary ## Impact #

[GitHub] [incubator-nuttx] no1wudi commented on pull request #5731: arch/risc-v: Improve speed of context switch

2022-03-24 Thread GitBox
no1wudi commented on pull request #5731: URL: https://github.com/apache/incubator-nuttx/pull/5731#issuecomment-1078604901 @xiaoxiang781216 @masayuki2009 This patch is ready to review now. -- This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #5837: libc: Implement getrandom on top of "/dev/[u]random"

2022-03-24 Thread GitBox
xiaoxiang781216 commented on pull request #5837: URL: https://github.com/apache/incubator-nuttx/pull/5837#issuecomment-1078607419 > * is this meant for linux compat? why not getentropy, which is accepted by posix? I thought getentropy also Linux specific before. Anyway, I prefer to s

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #5837: libc: Implement getrandom on top of "/dev/[u]random"

2022-03-24 Thread GitBox
pkarashchenko commented on pull request #5837: URL: https://github.com/apache/incubator-nuttx/pull/5837#issuecomment-1078609547 I think the good way is to support both as `getentropy` has a limitation of no more than 256 bytes buffer length, so may be still good for uuid, but we might avoi

[incubator-nuttx] branch master updated (97fa9b8 -> 914e958)

2022-03-24 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/incubator-nuttx.git. from 97fa9b8 defconfig: add Lua configs for sim and teensy-4.x add 914e958 esp32/Make.defs: Organise co

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #5838: esp32/Make.defs: Organise common arch files and chip files appropriately.

2022-03-24 Thread GitBox
xiaoxiang781216 merged pull request #5838: URL: https://github.com/apache/incubator-nuttx/pull/5838 -- 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

[GitHub] [incubator-nuttx] lupyuen edited a comment on issue #5810: GPIO issues on BL602 + suggestions for RISC-V, maybe more.

2022-03-24 Thread GitBox
lupyuen edited a comment on issue #5810: URL: https://github.com/apache/incubator-nuttx/issues/5810#issuecomment-1077355494 Cool thanks! BL602 MISO / MOSI Swap is in the master branch here: [bl602_spi.c](https://github.com/apache/incubator-nuttx/blob/master/arch/risc-v/src/bl602/bl602_spi.

[GitHub] [incubator-nuttx] anjiahao1 commented on a change in pull request #5836: tools:use dump log creat a gdbsever to debug

2022-03-24 Thread GitBox
anjiahao1 commented on a change in pull request #5836: URL: https://github.com/apache/incubator-nuttx/pull/5836#discussion_r834929431 ## File path: tools/minidumpserver.py ## @@ -0,0 +1,530 @@ +#!/usr/bin/env python3 +# tools/minidumpserver.py +# +# Licensed to the Apache Softw

[GitHub] [incubator-nuttx] no1wudi commented on a change in pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-03-24 Thread GitBox
no1wudi commented on a change in pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#discussion_r834936825 ## File path: arch/risc-v/Kconfig ## @@ -188,6 +189,30 @@ config ARCH_MMU_TYPE_SV39 bool default n +config ARCH_HAVE_S_MODE Rev

[GitHub] [incubator-nuttx] yamt commented on pull request #5837: libc: Implement getrandom on top of "/dev/[u]random"

2022-03-24 Thread GitBox
yamt commented on pull request #5837: URL: https://github.com/apache/incubator-nuttx/pull/5837#issuecomment-1078651938 > > * is this meant for linux compat? why not getentropy, which is accepted by posix? > > I thought getentropy also Linux specific before. Anyway, I prefer to suppo

[GitHub] [incubator-nuttx] yamt commented on pull request #5837: libc: Implement getrandom on top of "/dev/[u]random"

2022-03-24 Thread GitBox
yamt commented on pull request #5837: URL: https://github.com/apache/incubator-nuttx/pull/5837#issuecomment-1078658264 why do you replace arc4random_buf with getrandom? it seems awkward as one of the purpose of arc4random_buf (renamed from getrandom) was to avoid descrptor shortag

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5837: libc: Implement getrandom on top of "/dev/[u]random"

2022-03-24 Thread GitBox
pkarashchenko commented on a change in pull request #5837: URL: https://github.com/apache/incubator-nuttx/pull/5837#discussion_r834959115 ## File path: libs/libc/uuid/lib_uuid_create.c ## @@ -42,17 +43,24 @@ void uuid_create(uuid_t *u, uint32_t *status) { -#ifdef CONFIG_CRY

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #5837: libc: Implement getrandom on top of "/dev/[u]random"

2022-03-24 Thread GitBox
xiaoxiang781216 commented on pull request #5837: URL: https://github.com/apache/incubator-nuttx/pull/5837#issuecomment-1078688420 > why do you replace arc4random_buf with getrandom? One problem is that arc4random_buf couple with the random pool and only exist when CONFIG_DEV_URANDOM_

[GitHub] [incubator-nuttx] xiaoxiang781216 edited a comment on pull request #5837: libc: Implement getrandom on top of "/dev/[u]random"

2022-03-24 Thread GitBox
xiaoxiang781216 edited a comment on pull request #5837: URL: https://github.com/apache/incubator-nuttx/pull/5837#issuecomment-1078688420 > why do you replace arc4random_buf with getrandom? One problem is that arc4random_buf couple with the random pool and only exist when CONFIG_DEV_U

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #5837: libc: Implement getrandom on top of "/dev/[u]random"

2022-03-24 Thread GitBox
xiaoxiang781216 commented on pull request #5837: URL: https://github.com/apache/incubator-nuttx/pull/5837#issuecomment-1078693797 BTW, from the manual: https://www.freebsd.org/cgi/man.cgi?query=arc4random_buf&sektion=3&n=1 https://man7.org/linux/man-pages/man2/getrandom.2.html https

[GitHub] [incubator-nuttx] xiaoxiang781216 edited a comment on pull request #5837: libc: Implement getrandom on top of "/dev/[u]random"

2022-03-24 Thread GitBox
xiaoxiang781216 edited a comment on pull request #5837: URL: https://github.com/apache/incubator-nuttx/pull/5837#issuecomment-1078693797 BTW, from the manual: https://www.freebsd.org/cgi/man.cgi?query=arc4random_buf&sektion=3&n=1 https://man7.org/linux/man-pages/man2/getrandom.2.html

[GitHub] [incubator-nuttx] xiaoxiang781216 edited a comment on pull request #5837: libc: Implement getrandom on top of "/dev/[u]random"

2022-03-24 Thread GitBox
xiaoxiang781216 edited a comment on pull request #5837: URL: https://github.com/apache/incubator-nuttx/pull/5837#issuecomment-1078693797 BTW, from the manual: https://www.freebsd.org/cgi/man.cgi?query=arc4random_buf&sektion=3&n=1 https://man7.org/linux/man-pages/man2/getrandom.2.html

  1   2   >