[GitHub] [incubator-nuttx] mrivi opened a new pull request #1623: RTR extension

2020-08-21 Thread GitBox
mrivi opened a new pull request #1623: URL: https://github.com/apache/incubator-nuttx/pull/1623 ## Summary this PR extends the support for RTR also in cases where the extended id is used. ## Impact ## Testing

[GitHub] [incubator-nuttx] Ouss4 commented on a change in pull request #1616: Align the two configure programs (configure.sh and configure.c)

2020-08-21 Thread GitBox
Ouss4 commented on a change in pull request #1616: URL: https://github.com/apache/incubator-nuttx/pull/1616#discussion_r474737098 ## File path: tools/configure.c ## @@ -794,6 +835,8 @@ static void check_configdir(void) static void check_configured(void) { + char

[GitHub] [incubator-nuttx] davids5 commented on pull request #1577: drivers/rwbuffer: Avoid ftl driver allocate the temporary erase buffer

2020-08-21 Thread GitBox
davids5 commented on pull request #1577: URL: https://github.com/apache/incubator-nuttx/pull/1577#issuecomment-678328420 @xiaoxiang781216 By doing a force push, I can not see what you changed from what it was. So it forces me to re-review the whole thing. I will look at this again over

[GitHub] [incubator-nuttx] patacongo commented on pull request #1577: drivers/rwbuffer: Avoid ftl driver allocate the temporary erase buffer

2020-08-21 Thread GitBox
patacongo commented on pull request #1577: URL: https://github.com/apache/incubator-nuttx/pull/1577#issuecomment-678350047 > @acassis shall we ask Ken Pettit, to have a look? Ken is a great guy and a good person to review anything. But this is an odd request since Ken is not one of

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #1616: Align the two configure programs (configure.sh and configure.c)

2020-08-21 Thread GitBox
xiaoxiang781216 commented on a change in pull request #1616: URL: https://github.com/apache/incubator-nuttx/pull/1616#discussion_r474740592 ## File path: tools/configure.c ## @@ -794,6 +835,8 @@ static void check_configdir(void) static void check_configured(void) { + char

[GitHub] [incubator-nuttx] acassis commented on pull request #1577: drivers/rwbuffer: Avoid ftl driver allocate the temporary erase buffer

2020-08-21 Thread GitBox
acassis commented on pull request #1577: URL: https://github.com/apache/incubator-nuttx/pull/1577#issuecomment-678343987 @davids5 I pinged Ken and he told me that will take a look. This is an automated message from the

[GitHub] [incubator-nuttx] davids5 commented on issue #1568: STM32F4 core over-drive enable/disable sequence problem.

2020-08-21 Thread GitBox
davids5 commented on issue #1568: URL: https://github.com/apache/incubator-nuttx/issues/1568#issuecomment-678362575 @fjpanag stm32_pwr_enableoverdrive sounds like a landmine, we should probably delete it it if is not used.

[GitHub] [incubator-nuttx] davids5 commented on pull request #1623: RTR extension

2020-08-21 Thread GitBox
davids5 commented on pull request #1623: URL: https://github.com/apache/incubator-nuttx/pull/1623#issuecomment-678369922 @mrivi Please run tools nxstyle on the changed file. If the log lines MUST to be kept intact for readability you can extend the lines of , (it is the

[incubator-nuttx] branch master updated (3543950 -> 8617cd9)

2020-08-21 Thread liuhaitao
This is an automated email from the ASF dual-hosted git repository. liuhaitao pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git. from 3543950 arch: cxd56xx: Use spinlock API in cxd56_serial.c add 8ce0ff5 arch/sim: Make

[GitHub] [incubator-nuttx] liuguo09 commented on pull request #1461: arch/sim: Make up_internal.h includable in host environment

2020-08-21 Thread GitBox
liuguo09 commented on pull request #1461: URL: https://github.com/apache/incubator-nuttx/pull/1461#issuecomment-678173542 The mixed case letter nxstyle error could be ignored. So I'll help to merge it. This is an automated

[GitHub] [incubator-nuttx] liuguo09 merged pull request #1461: arch/sim: Make up_internal.h includable in host environment

2020-08-21 Thread GitBox
liuguo09 merged pull request #1461: URL: https://github.com/apache/incubator-nuttx/pull/1461 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

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #1616: Align the two configure programs (configure.sh and configure.c)

2020-08-21 Thread GitBox
xiaoxiang781216 commented on pull request #1616: URL: https://github.com/apache/incubator-nuttx/pull/1616#issuecomment-678324024 LGTM, but nxstyle warning need be fixed first. This is an automated message from the Apache Git

[GitHub] [incubator-nuttx] davids5 commented on pull request #1577: drivers/rwbuffer: Avoid ftl driver allocate the temporary erase buffer

2020-08-21 Thread GitBox
davids5 commented on pull request #1577: URL: https://github.com/apache/incubator-nuttx/pull/1577#issuecomment-678348205 > @davids5 I pinged Ken and he told me that will take a look. @acassis, Ken - Thank you! This

[GitHub] [incubator-nuttx] Ouss4 commented on a change in pull request #1616: Align the two configure programs (configure.sh and configure.c)

2020-08-21 Thread GitBox
Ouss4 commented on a change in pull request #1616: URL: https://github.com/apache/incubator-nuttx/pull/1616#discussion_r474735419 ## File path: tools/configure.c ## @@ -808,12 +851,35 @@ static void check_configured(void) } else { -

[GitHub] [incubator-nuttx] Ouss4 commented on a change in pull request #1616: Align the two configure programs (configure.sh and configure.c)

2020-08-21 Thread GitBox
Ouss4 commented on a change in pull request #1616: URL: https://github.com/apache/incubator-nuttx/pull/1616#discussion_r474743103 ## File path: tools/configure.c ## @@ -808,12 +851,35 @@ static void check_configured(void) } else { -

[GitHub] [incubator-nuttx] davids5 commented on pull request #1577: drivers/rwbuffer: Avoid ftl driver allocate the temporary erase buffer

2020-08-21 Thread GitBox
davids5 commented on pull request #1577: URL: https://github.com/apache/incubator-nuttx/pull/1577#issuecomment-678172279 @xiaoxiang781216 - thank you for the additional information and link. More context always helps! What guarantees the buffer has the capacity for the expansion?

[GitHub] [incubator-nuttx] jerpelea merged pull request #1621: arch: cxd56xx: Use spinlock API in cxd56_serial.c

2020-08-21 Thread GitBox
jerpelea merged pull request #1621: URL: https://github.com/apache/incubator-nuttx/pull/1621 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

[incubator-nuttx] branch master updated (39539be -> 3543950)

2020-08-21 Thread jerpelea
This is an automated email from the ASF dual-hosted git repository. jerpelea pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git. from 39539be xtensa/esp32: Improve SPI transmission add 3543950 arch: cxd56xx: Use spinlock API in

[incubator-nuttx] branch master updated (8617cd9 -> 6670bc2)

2020-08-21 Thread acassis
This is an automated email from the ASF dual-hosted git repository. acassis pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git. from 8617cd9 Fix nxstyle warning add 6670bc2 libc/time: Implement tm::tm_gmtoff field No new revisions

[GitHub] [incubator-nuttx] acassis commented on pull request #1615: libc/time: Implement tm::tm_gmtoff field

2020-08-21 Thread GitBox
acassis commented on pull request #1615: URL: https://github.com/apache/incubator-nuttx/pull/1615#issuecomment-678226157 Thank you Xiang! This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [incubator-nuttx] acassis merged pull request #1615: libc/time: Implement tm::tm_gmtoff field

2020-08-21 Thread GitBox
acassis merged pull request #1615: URL: https://github.com/apache/incubator-nuttx/pull/1615 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

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #1577: drivers/rwbuffer: Avoid ftl driver allocate the temporary erase buffer

2020-08-21 Thread GitBox
xiaoxiang781216 commented on pull request #1577: URL: https://github.com/apache/incubator-nuttx/pull/1577#issuecomment-678250774 @davids5 the new patch address your concern. This is an automated message from the Apache Git

[GitHub] [incubator-nuttx] davids5 commented on pull request #1577: drivers/rwbuffer: Avoid ftl driver allocate the temporary erase buffer

2020-08-21 Thread GitBox
davids5 commented on pull request #1577: URL: https://github.com/apache/incubator-nuttx/pull/1577#issuecomment-678203388 @xiaoxiang781216 Forgive all the question, I have not dug into this code before. Does this support lazy allocation? I ask because of line 832. If it does, does the the

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #1577: drivers/rwbuffer: Avoid ftl driver allocate the temporary erase buffer

2020-08-21 Thread GitBox
xiaoxiang781216 commented on pull request #1577: URL: https://github.com/apache/incubator-nuttx/pull/1577#issuecomment-678245898 > @xiaoxiang781216 Forgive all the question, I have not dug into this code before. Does this support lazy allocation? I ask because of line 832. If it does,

[GitHub] [incubator-nuttx] Ouss4 commented on pull request #1611: libc/stdio: Allocate file_struct dynamically

2020-08-21 Thread GitBox
Ouss4 commented on pull request #1611: URL: https://github.com/apache/incubator-nuttx/pull/1611#issuecomment-678134119 > It should work in theory. It's already in place for branches, @btashton set it up, we use it for releases. An extra effort is needed to make it work with PRs,

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #1577: drivers/rwbuffer: Avoid ftl driver allocate the temporary erase buffer

2020-08-21 Thread GitBox
xiaoxiang781216 commented on pull request #1577: URL: https://github.com/apache/incubator-nuttx/pull/1577#issuecomment-678145968 > @xiaoxiang781216 would you please provide more information on how this works and mechanism by which it saves memory? > I already describe the case

[GitHub] [incubator-nuttx] xiaoxiang781216 edited a comment on pull request #1611: libc/stdio: Allocate file_struct dynamically

2020-08-21 Thread GitBox
xiaoxiang781216 edited a comment on pull request #1611: URL: https://github.com/apache/incubator-nuttx/pull/1611#issuecomment-678150846 > > > It should work in theory. > > > > > > It's already in place for branches, @btashton set it up, we use it for releases. > > An extra

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #1577: drivers/rwbuffer: Avoid ftl driver allocate the temporary erase buffer

2020-08-21 Thread GitBox
xiaoxiang781216 commented on pull request #1577: URL: https://github.com/apache/incubator-nuttx/pull/1577#issuecomment-678176411 > @xiaoxiang781216 - thank you for the additional information and link. More context always helps! > > What guarantees the buffer has the capacity for the

[GitHub] [incubator-nuttx] leomar95 edited a comment on pull request #1617: Samd5e5 - progmem

2020-08-21 Thread GitBox
leomar95 edited a comment on pull request #1617: URL: https://github.com/apache/incubator-nuttx/pull/1617#issuecomment-677906930 > > I tried these steps, but it didn't work for #1618 > > It looks like your also did some 'git merge' . Merging and rebasing are incompatible because

[GitHub] [incubator-nuttx] davids5 commented on pull request #1611: libc/stdio: Allocate file_struct dynamically

2020-08-21 Thread GitBox
davids5 commented on pull request #1611: URL: https://github.com/apache/incubator-nuttx/pull/1611#issuecomment-678137091 > > It should work in theory. > > It's already in place for branches, @btashton set it up, we use it for releases. > An extra effort is needed to make it work

[GitHub] [incubator-nuttx] acassis commented on pull request #1577: drivers/rwbuffer: Avoid ftl driver allocate the temporary erase buffer

2020-08-21 Thread GitBox
acassis commented on pull request #1577: URL: https://github.com/apache/incubator-nuttx/pull/1577#issuecomment-678247667 > @xiaoxiang781216 would you please provide more information on how this works and mechanism by which it saves memory? > > @acassis shall we ask Ken Pettit, to

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #1616: Align the two configure programs (configure.sh and configure.c)

2020-08-21 Thread GitBox
xiaoxiang781216 commented on a change in pull request #1616: URL: https://github.com/apache/incubator-nuttx/pull/1616#discussion_r474697965 ## File path: tools/configure.c ## @@ -808,12 +851,35 @@ static void check_configured(void) } else { -

[GitHub] [incubator-nuttx] RB-tel opened a new pull request #1622: I2C(RIIC) support for RX65N

2020-08-21 Thread GitBox
RB-tel opened a new pull request #1622: URL: https://github.com/apache/incubator-nuttx/pull/1622 ## Summary RIIC suport for RX65N ## Impact ## Testing This is an automated message from the Apache Git

[GitHub] [incubator-nuttx] donghengqaz commented on pull request #1610: xtensa/esp32: Improve SPI transmission

2020-08-21 Thread GitBox
donghengqaz commented on pull request #1610: URL: https://github.com/apache/incubator-nuttx/pull/1610#issuecomment-678147136 > LGTM, thanks @donghengqaz. @Ouss4 Thanks for your deeply reviewing. This is an

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #1577: drivers/rwbuffer: Avoid ftl driver allocate the temporary erase buffer

2020-08-21 Thread GitBox
xiaoxiang781216 commented on pull request #1577: URL: https://github.com/apache/incubator-nuttx/pull/1577#issuecomment-678180637 @davids5 done. This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-nuttx] btashton commented on pull request #1611: libc/stdio: Allocate file_struct dynamically

2020-08-21 Thread GitBox
btashton commented on pull request #1611: URL: https://github.com/apache/incubator-nuttx/pull/1611#issuecomment-678236018 > > It should work in theory. > > It's already in place for branches, @btashton set it up, we use it for releases. > An extra effort is needed to make it work

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #1611: libc/stdio: Allocate file_struct dynamically

2020-08-21 Thread GitBox
xiaoxiang781216 commented on pull request #1611: URL: https://github.com/apache/incubator-nuttx/pull/1611#issuecomment-678150846 > > > It should work in theory. > > > > > > It's already in place for branches, @btashton set it up, we use it for releases. > > An extra effort is

[GitHub] [incubator-nuttx] Ouss4 commented on pull request #1616: Align the two configure programs (configure.sh and configure.c)

2020-08-21 Thread GitBox
Ouss4 commented on pull request #1616: URL: https://github.com/apache/incubator-nuttx/pull/1616#issuecomment-678565445 > but nxstyle warning need be fixed first. Done. This is an automated message from the Apache Git

[incubator-nuttx] branch master updated (6670bc2 -> f2af90d)

2020-08-21 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 6670bc2 libc/time: Implement tm::tm_gmtoff field add 1c295af tools/configure.sh: Don't distclean

[incubator-nuttx] branch master updated (6670bc2 -> f2af90d)

2020-08-21 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 6670bc2 libc/time: Implement tm::tm_gmtoff field add 1c295af tools/configure.sh: Don't distclean

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #1577: drivers/rwbuffer: Avoid ftl driver allocate the temporary erase buffer

2020-08-21 Thread GitBox
xiaoxiang781216 commented on pull request #1577: URL: https://github.com/apache/incubator-nuttx/pull/1577#issuecomment-678581436 > @xiaoxiang781216 Forgive all the question, I have not dug into this code before. Does this support lazy allocation? I ask because of line 832. If it does,

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #1616: Align the two configure programs (configure.sh and configure.c)

2020-08-21 Thread GitBox
xiaoxiang781216 merged pull request #1616: URL: https://github.com/apache/incubator-nuttx/pull/1616 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] masayuki2009 commented on pull request #1620: arch: cxd56xx: Fix IRQ control in cxd56_dmac.c

2020-08-21 Thread GitBox
masayuki2009 commented on pull request #1620: URL: https://github.com/apache/incubator-nuttx/pull/1620#issuecomment-678118898 @jerpelea This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-nuttx] jerpelea merged pull request #1620: arch: cxd56xx: Fix IRQ control in cxd56_dmac.c

2020-08-21 Thread GitBox
jerpelea merged pull request #1620: URL: https://github.com/apache/incubator-nuttx/pull/1620 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

[incubator-nuttx] branch master updated (8beb1ed -> 0d971d4)

2020-08-21 Thread jerpelea
This is an automated email from the ASF dual-hosted git repository. jerpelea pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git. from 8beb1ed Fix inverted errno in mq_open add 0d971d4 arch: cxd56xx: Fix IRQ control in cxd56_dmac.c

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #1577: drivers/rwbuffer: Avoid ftl driver allocate the temporary erase buffer

2020-08-21 Thread GitBox
xiaoxiang781216 commented on pull request #1577: URL: https://github.com/apache/incubator-nuttx/pull/1577#issuecomment-678122303 @acassis could you merge this patch? This is an automated message from the Apache Git Service.

[incubator-nuttx] branch master updated (0d971d4 -> 297c294)

2020-08-21 Thread liuhaitao
This is an automated email from the ASF dual-hosted git repository. liuhaitao pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git. from 0d971d4 arch: cxd56xx: Fix IRQ control in cxd56_dmac.c add 297c294 libc: Change ctype macro to

[GitHub] [incubator-nuttx] liuguo09 merged pull request #1496: libc: Change ctype macro to normal function

2020-08-21 Thread GitBox
liuguo09 merged pull request #1496: URL: https://github.com/apache/incubator-nuttx/pull/1496 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

[GitHub] [incubator-nuttx] masayuki2009 opened a new pull request #1621: arch: cxd56xx: Use spinlock API in cxd56_serial.c

2020-08-21 Thread GitBox
masayuki2009 opened a new pull request #1621: URL: https://github.com/apache/incubator-nuttx/pull/1621 ## Summary - This commit improves cxd56_serial performance in SMP mode. ## Impact - This commit affects SMP mode only. ## Testing - Tested with

[GitHub] [incubator-nuttx] Ouss4 merged pull request #1608: sched/signal: Add sighandler_t typedef

2020-08-21 Thread GitBox
Ouss4 merged pull request #1608: URL: https://github.com/apache/incubator-nuttx/pull/1608 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

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #1621: arch: cxd56xx: Use spinlock API in cxd56_serial.c

2020-08-21 Thread GitBox
masayuki2009 commented on pull request #1621: URL: https://github.com/apache/incubator-nuttx/pull/1621#issuecomment-678128070 @jerpelea This is an automated message from the Apache Git Service. To respond to the

[incubator-nuttx] branch master updated (297c294 -> b3e0116)

2020-08-21 Thread aguettouche
This is an automated email from the ASF dual-hosted git repository. aguettouche pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git. from 297c294 libc: Change ctype macro to normal function add b3e0116 sched/signal: Add sighandler_t

[GitHub] [incubator-nuttx] Ouss4 merged pull request #1610: xtensa/esp32: Improve SPI transmission

2020-08-21 Thread GitBox
Ouss4 merged pull request #1610: URL: https://github.com/apache/incubator-nuttx/pull/1610 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

[incubator-nuttx] branch master updated (b3e0116 -> 39539be)

2020-08-21 Thread aguettouche
This is an automated email from the ASF dual-hosted git repository. aguettouche pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git. from b3e0116 sched/signal: Add sighandler_t typedef add 39539be xtensa/esp32: Improve SPI