You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by GitBox <gi...@apache.org> on 2021/07/19 08:46:15 UTC

[GitHub] [incubator-nuttx] SunJun8 opened a new pull request #4178: Add BL602 documentation

SunJun8 opened a new pull request #4178:
URL: https://github.com/apache/incubator-nuttx/pull/4178


   ## Summary
   Add BL602 documentation
   ## Impact
   
   ## Testing
   
   


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

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



[GitHub] [incubator-nuttx] btashton edited a comment on pull request #4178: Add BL602 documentation

Posted by GitBox <gi...@apache.org>.
btashton edited a comment on pull request #4178:
URL: https://github.com/apache/incubator-nuttx/pull/4178#issuecomment-882535493


   > BTW @btashton did you see @gustavonihei comment at https://github.com/apache/incubator-nuttx-apps/pull/566 ? 
   > What do you suggest? We have three options: 1) wait more few days for your patch, if you think it is feasible; 2) Merge it ignoring the missing configuration options; 3) Close the PR and use an mbedTLS port that @Ouss4 did sometime ago.
   
   This is not related to this PR, the ask here was to remove the duplicated documentation


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

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



[GitHub] [incubator-nuttx] acassis merged pull request #4178: Add BL602 documentation

Posted by GitBox <gi...@apache.org>.
acassis merged pull request #4178:
URL: https://github.com/apache/incubator-nuttx/pull/4178


   


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

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



[GitHub] [incubator-nuttx] btashton edited a comment on pull request #4178: Add BL602 documentation

Posted by GitBox <gi...@apache.org>.
btashton edited a comment on pull request #4178:
URL: https://github.com/apache/incubator-nuttx/pull/4178#issuecomment-882535493


   > BTW @btashton did you see @gustavonihei comment at https://github.com/apache/incubator-nuttx-apps/pull/566 ? 
   > What do you suggest? We have three options: 1) wait more few days for your patch, if you think it is feasible; 2) Merge it ignoring the missing configuration options; 3) Close the PR and use an mbedTLS port that @Ouss4 did sometime ago.
   
   This is not related to this PR, the ask here was to remove the duplicated documentation


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

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



[GitHub] [incubator-nuttx] btashton commented on pull request #4178: Add BL602 documentation

Posted by GitBox <gi...@apache.org>.
btashton commented on pull request #4178:
URL: https://github.com/apache/incubator-nuttx/pull/4178#issuecomment-882535493


   > BTW @btashton did you see @gustavonihei comment at https://github.com/apache/incubator-nuttx-apps/pull/566 ? 
   > What do you suggest? We have three options: 1) wait more few days for your patch, if you think it is feasible; 2) Merge it ignoring the missing configuration options; 3) Close the PR and use an mbedTLS port that @Ouss4 did sometime ago.
   
   This is not related. To this PR, the ask here was to remove the duplicated documentation


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

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



[GitHub] [incubator-nuttx] SunJun8 commented on pull request #4178: Add BL602 documentation

Posted by GitBox <gi...@apache.org>.
SunJun8 commented on pull request #4178:
URL: https://github.com/apache/incubator-nuttx/pull/4178#issuecomment-882980624


   @acassis well, noticed that it has been deleted in the new commit, thanks


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

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



[GitHub] [incubator-nuttx] btashton edited a comment on pull request #4178: Add BL602 documentation

Posted by GitBox <gi...@apache.org>.
btashton edited a comment on pull request #4178:
URL: https://github.com/apache/incubator-nuttx/pull/4178#issuecomment-882535493


   > BTW @btashton did you see @gustavonihei comment at https://github.com/apache/incubator-nuttx-apps/pull/566 ? 
   > What do you suggest? We have three options: 1) wait more few days for your patch, if you think it is feasible; 2) Merge it ignoring the missing configuration options; 3) Close the PR and use an mbedTLS port that @Ouss4 did sometime ago.
   
   This is not related to this PR, the ask here was to remove the duplicated documentation


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

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



[GitHub] [incubator-nuttx] btashton commented on pull request #4178: Add BL602 documentation

Posted by GitBox <gi...@apache.org>.
btashton commented on pull request #4178:
URL: https://github.com/apache/incubator-nuttx/pull/4178#issuecomment-882528237


   @acassis there was an outstanding comment


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

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



[GitHub] [incubator-nuttx] acassis commented on pull request #4178: Add BL602 documentation

Posted by GitBox <gi...@apache.org>.
acassis commented on pull request #4178:
URL: https://github.com/apache/incubator-nuttx/pull/4178#issuecomment-882542489


   Ok, @SunJun8 could you please submit a PR to remove the documentation from boards/risc-v/bl602/bl602evb/README.txt ?


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

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



[GitHub] [incubator-nuttx] acassis merged pull request #4178: Add BL602 documentation

Posted by GitBox <gi...@apache.org>.
acassis merged pull request #4178:
URL: https://github.com/apache/incubator-nuttx/pull/4178


   


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

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



[GitHub] [incubator-nuttx] btashton commented on pull request #4178: Add BL602 documentation

Posted by GitBox <gi...@apache.org>.
btashton commented on pull request #4178:
URL: https://github.com/apache/incubator-nuttx/pull/4178#issuecomment-882528237






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

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



[GitHub] [incubator-nuttx] acassis commented on pull request #4178: Add BL602 documentation

Posted by GitBox <gi...@apache.org>.
acassis commented on pull request #4178:
URL: https://github.com/apache/incubator-nuttx/pull/4178#issuecomment-882534374


   BTW @btashton did you see @gustavonihei comment at https://github.com/apache/incubator-nuttx-apps/pull/566 ? 
   What do you suggest? We have three options: 1) wait more few days for your patch, if you think it is feasible; 2) Merge it ignoring the missing configuration options; 3) Close the PR and use an mbedTLS port that @Ouss4 did sometime ago.


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

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



[GitHub] [incubator-nuttx] btashton commented on pull request #4178: Add BL602 documentation

Posted by GitBox <gi...@apache.org>.
btashton commented on pull request #4178:
URL: https://github.com/apache/incubator-nuttx/pull/4178#issuecomment-882528237






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

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



[GitHub] [incubator-nuttx] acassis commented on pull request #4178: Add BL602 documentation

Posted by GitBox <gi...@apache.org>.
acassis commented on pull request #4178:
URL: https://github.com/apache/incubator-nuttx/pull/4178#issuecomment-882530711






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

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



[GitHub] [incubator-nuttx] SunJun8 commented on pull request #4178: Add BL602 documentation

Posted by GitBox <gi...@apache.org>.
SunJun8 commented on pull request #4178:
URL: https://github.com/apache/incubator-nuttx/pull/4178#issuecomment-882980624


   @acassis well, noticed that it has been deleted in the new commit, thanks


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

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



[GitHub] [incubator-nuttx] acassis merged pull request #4178: Add BL602 documentation

Posted by GitBox <gi...@apache.org>.
acassis merged pull request #4178:
URL: https://github.com/apache/incubator-nuttx/pull/4178


   


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

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



[GitHub] [incubator-nuttx] SunJun8 commented on pull request #4178: Add BL602 documentation

Posted by GitBox <gi...@apache.org>.
SunJun8 commented on pull request #4178:
URL: https://github.com/apache/incubator-nuttx/pull/4178#issuecomment-882980624


   @acassis well, noticed that it has been deleted in the new commit, thanks


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

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



[GitHub] [incubator-nuttx] acassis commented on pull request #4178: Add BL602 documentation

Posted by GitBox <gi...@apache.org>.
acassis commented on pull request #4178:
URL: https://github.com/apache/incubator-nuttx/pull/4178#issuecomment-882530711


   Oops, sorry I missed it!


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

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



[GitHub] [incubator-nuttx] acassis commented on pull request #4178: Add BL602 documentation

Posted by GitBox <gi...@apache.org>.
acassis commented on pull request #4178:
URL: https://github.com/apache/incubator-nuttx/pull/4178#issuecomment-882530711






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

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