-
Notifications
You must be signed in to change notification settings - Fork 33
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
10.1 updates #92
10.1 updates #92
Conversation
New warnings found with rstcheck:
|
Please check the commit format in the link in |
source/linux/Foundational_Components/Tools/Flash_via_Fastboot.rst
Outdated
Show resolved
Hide resolved
cec479c
to
83ea517
Compare
New warnings found with rstcheck:
|
83ea517
to
1748c1a
Compare
New warnings found with rstcheck:
|
source/linux/Foundational_Components/Tools/Flash_via_Fastboot.rst
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is linux/Foundational_Components/Tools/Flash_via_DFU
file used in any other platforms?
If not, please delete the rst files as well
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you also add a line in AM62X and AM62P device specific Release Notes highlighting Snagboot support, WIFI OOB and Terminal in TI Apps Launcher under New features in SDK 10.1 and eventually point to respective user guides.
source/linux/Foundational_Components/Tools/Flash_via_Fastboot.rst
Outdated
Show resolved
Hide resolved
source/linux/Foundational_Components/Tools/Flash_via_Fastboot.rst
Outdated
Show resolved
Hide resolved
source/linux/Foundational_Components/Tools/Flash_via_Fastboot.rst
Outdated
Show resolved
Hide resolved
New warnings found with rstcheck:
|
@cshilwant Yep it will not go in this release. But we may get a requirement in future for DFU based factory flashing tool. Is it okay to delete the doc? |
Yes, if the rst file is no longer needed we should delete it. |
1748c1a
to
a8e8309
Compare
New warnings found with rstcheck:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@paresh-bhagat12 please plan to handle these errors reported by rstcheck- #92 (comment)
a8e8309
to
0600f22
Compare
source/linux/Foundational_Components/Tools/Flash_via_Fastboot.rst
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are your commits following the guidelines?
https://github.com/TexasInstruments/processor-sdk-doc/blob/master/CONTRIBUTING.md#commit-formatting
I see only feat: XYZ, which doesn't seem to be in line with the right format. Please check once again and update accordingly.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@paresh-bhagat12 Can you resolve the merge conflicts
source/linux/Foundational_Components/Tools/Flash_via_Fastboot.rst
Outdated
Show resolved
Hide resolved
source/linux/Foundational_Components/Tools/Flash_via_Fastboot.rst
Outdated
Show resolved
Hide resolved
source/linux/Foundational_Components/Tools/Flash_via_Fastboot.rst
Outdated
Show resolved
Hide resolved
@DhruvaG2000 I am following this format https://www.conventionalcommits.org/en/v1.0.0/#commit-message-with-multi-paragraph-body-and-multiple-footers |
f1a2473
to
aabcfac
Compare
@cshilwant @StaticRocket @AashvijShenai I have updated the PR. please review |
aabcfac
to
b9c4ed5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just 2 little things
source/linux/Foundational_Components/Tools/Flash_via_Fastboot.rst
Outdated
Show resolved
Hide resolved
- Add info about Wi-Fi demo showing steps including how to enable Wi-Fi, scanning SSIDs and handling connections. - Add info about terminal demo for accessing terminal from the ti-apps-launcher. - Update build instruction for Linux and RT-Linux build. - Mention Wi-Fi and Terminal demo info in Release Notes for 10.1. Signed-off-by: Paresh Bhagat <[email protected]>
- Add a new doc for flashing via fastboot using Snagfactory tool. - Update Flash via Uart doc. This tool will not be packaged in Linux SDK and can be picked from MCU PLUS SDK installer. - DFU tool will not be supported in 10.1 SDK. Remove the doc. - Mention Snagfactory Tool in Release Notes for 10.1. Signed-off-by: Paresh Bhagat <[email protected]>
Jailhouse Image build needs some extra branding steps and cannot be build simply by specifying image name during build. So add a new section which mentions steps to build jailhouse image. Signed-off-by: Paresh Bhagat <[email protected]>
b9c4ed5
to
0d00bd3
Compare
@DhruvaG2000 @AashvijShenai Author has updated the PR. |
No description provided.