Skip to content

Conversation

@jsuhaas22
Copy link
Collaborator

Instead of building TI Debian images with ti-bdebstrap, we are now using Armbian.

In e571245 ("feat(debian): AM62LX: Armbian-ize Debian docs for AM62L"), changes were made to migrate AM62L docs to Armbian from ti-bdebstrap. At the time of that commit, changes were specific to AM62L only since the other boards were still at Debian.

With 11.01 release, all platforms will use Debian built from the Armbian build framework. Therefore, generalize that information.

Specifically, do the following:

* Remove references to ti-bdebstrap.
* Explain the difference between Armbian and Debian briefly.
* Move sources/devices/AM62LX/debian/Building_Debian_Image.rst to sources/debian/, and delete the file of the same name from the latter path. The new file was originally written for AM62L, and thus was placed in that directory, but is now (after a few changes also made in this commit) applicable to all platforms.
* Do some formatting.

@github-actions
Copy link

github-actions bot commented Jul 2, 2025

New issues found with check_toc_txt.py:

WARNING:root:Invalid slug: configs/AM62LX/AM62LX_debian_toc.txt:5 devices/AM62LX/debian/Building_Debian_Image

@jsuhaas22 jsuhaas22 force-pushed the debian-to-armbian branch 3 times, most recently from e8e58b0 to 0d34130 Compare July 2, 2025 09:39
@jsuhaas22 jsuhaas22 force-pushed the debian-to-armbian branch 3 times, most recently from ac9ebc9 to d84d6ea Compare July 7, 2025 08:01
Copy link
Member

@StaticRocket StaticRocket left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Vale comments about passive voice and capitalization still need to be addressed.

@jsuhaas22 jsuhaas22 force-pushed the debian-to-armbian branch 3 times, most recently from 220c988 to 279a3a6 Compare July 8, 2025 06:06
@jsuhaas22
Copy link
Collaborator Author

Vale comments about passive voice and capitalization still need to be addressed.

@StaticRocket Fixed passive voice ones. The remaining capitalization ones are false alerts (its for titles and paths (.../linux/...)).

@jsuhaas22 jsuhaas22 force-pushed the debian-to-armbian branch from 279a3a6 to c377f24 Compare July 8, 2025 06:35
Copy link
Member

@cshilwant cshilwant left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sentence-style capitalization warnings from vale are still valid & must be fixed

@jsuhaas22
Copy link
Collaborator Author

sentence-style capitalization warnings from vale are still valid & must be fixed

@cshilwant Which ones? There are only 2 types of sentence style warnings there:

the first which recommends capitalizing headings like sentences (1st alphabet of 1st word in upper-case and everything else lower-case). This isn't how we capitalize headings in other pages though.

the second which asks me to capitalize "linux". This is false flag as well, since linux in this context is used inside a path, so its all lower-case.

@cshilwant cshilwant requested a review from StaticRocket July 9, 2025 20:11
@jsuhaas22 jsuhaas22 force-pushed the debian-to-armbian branch from c377f24 to 9ecf62b Compare July 10, 2025 10:32
@jsuhaas22 jsuhaas22 force-pushed the debian-to-armbian branch from 9ecf62b to 88025f5 Compare July 11, 2025 09:36
Copy link
Member

@cshilwant cshilwant left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Instead of building TI Debian images with ti-bdebstrap, we are now using
Armbian.

In e571245 ("feat(debian): AM62LX: Armbian-ize Debian docs for
AM62L"), changes were made to migrate AM62L docs to Armbian from
ti-bdebstrap. At the time of that commit, changes were specific to AM62L
only since the other boards were still at Debian.

With 11.01 release, all platforms will use Debian built from the Armbian
build framework. Therefore, generalize that information.

Specifically, do the following:

    * Remove references to ti-bdebstrap.
    * Explain the difference between Armbian and Debian briefly.
    * Move sources/devices/AM62LX/debian/Building_Debian_Image.rst to
      sources/debian/, and delete the file of the same name from the
      latter path. The new file was originally written for AM62L, and
      thus was placed in that directory, but is now (after a few changes
      also made in this commit) applicable to all platforms.
    * Do some formatting.

Signed-off-by: Suhaas Joshi <[email protected]>
@jsuhaas22 jsuhaas22 force-pushed the debian-to-armbian branch from 88025f5 to df5e787 Compare July 14, 2025 05:28
@cshilwant cshilwant dismissed StaticRocket’s stale review July 14, 2025 17:22

Author has updated the PR

@cshilwant
Copy link
Member

@StaticRocket to re-review

@StaticRocket StaticRocket merged commit 2cf3746 into TexasInstruments:master Jul 14, 2025
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants