source: linux: Add documentation of Ethernet boot for J722S #226
Conversation
chintan013
requested review from
cshilwant,
StaticRocket,
praneethbajjuri,
uditkumarti,
aniket-l and
VeeruPrudhvi
as code owners
github-actions
bot
assigned
bryanbrattlof,
devarsht,
jmenti,
praneethbajjuri,
r-vignesh and
uditkumarti
github-actions
bot
requested review from
bryanbrattlof,
devarsht,
jmenti and
r-vignesh
|
|
||
| .. code-block:: text | ||
|
|
||
| subnet 10.0.0.0 netmask 255.0.0.0 |
There was a problem hiding this comment.
Something tells me this infinite scrolling block of device specific config snippets should be templated at this point.
chintan013
force-pushed
the
j722s_doc_ethboot
branch
from
0d1392f to
9ec988f
Compare
| +------------------+--------------------------------------+-----------------------+------------------------+---------------------------+----------------------------+ | ||
|
|
||
| Build tiboot3.bin using ``j722s_evm_r5_ethboot_defconfig``. Build tispl.bin and u-boot.img using ``j722s_evm_a53_ethboot_defconfig``. For instructions to build the bootloader images please refer to :ref:`Build-U-Boot-label`. | ||
|
|
There was a problem hiding this comment.
why you want to duplicate information of
source/linux/Foundational_Components/U-Boot/UG-General-Info.rst
There was a problem hiding this comment.
Hello Udit, It is not duplicate information but it is pointing to the steps on how to build U-Boot images to boot via Ethernet using config files mentioned for Ethernet boot.
There was a problem hiding this comment.
Every column other than Supported interfaces seems to be redundant information already provided under UG-General-Info.
There was a problem hiding this comment.
Support for Ethernet boot for J722S is added recently, and this is a documentation for that. I believe that we do not have a documentation for thatin UG-General-info page. In the table, columns represent- config files to be used by J722S (specific to Ethernet boot), VCI string information which would be used to configure DHCP server during boot stage. We can find the same documentation already present for AM62 and AM64.
| @@ -26,6 +26,8 @@ binaries files to be sent over TFTP are listed in the table below. | |||
| | | ``am64x_evm_a53_defconfig`` | | :file:`tiboot3.bin` | :file:`tispl.bin` | :file:`u-boot.img` | | |||
| +------------------+------------------------------+------------------------+------------------------+---------------------------+---------------------------+ | |||
|
|
|||
| For instructions to build the bootloader images please refer to :ref:`Build-U-Boot-label`. | |||
There was a problem hiding this comment.
This line and the table above it, need to be normalized to 3 spaces.
There was a problem hiding this comment.
This line was added by mistake, I have removed it.
| +==================+======================================+=======================+========================+===========================+============================+ | ||
| | J722s EVM | ``j722s_evm_r5_ethboot_defconfig`` | CPSW NUSS Ethernet | VCI: TI K3 Bootp Boot | VCI: J722S U-Boot R5 SPL | VCI: J722S U-Boot A53 SPL | | ||
| | | ``j722s_evm_a53_ethboot_defconfig`` | | :file:`tiboot3.bin` | :file:`tispl.bin` | :file:`u-boot.img` | | ||
| +------------------+--------------------------------------+-----------------------+------------------------+---------------------------+----------------------------+ |
There was a problem hiding this comment.
This table should align with the directive and the below section. Remove a space.
| If using ISC dhcpd an example host entry would look like this: | ||
|
|
||
| .. code-block:: text |
There was a problem hiding this comment.
This section also needs to be normalized.
Ethernet boot support has been added to J722s SoC, add documentation for the same. Signed-off-by: Chintan Vankar <c-vankar@ti.com>
chintan013
force-pushed
the
j722s_doc_ethboot
branch
from
9ec988f to
0c3922c
Compare
Ethernet boot support has been added to J722s SoC, add documentation for the same.