On August 6, 2026 thus sayeth Simon Glass: > Hi Bryan, > > On 2026-08-04T21:55:02, Bryan Brattlof <[email protected]> wrote: > > doc: board: ti: correct path to tcl scripts > > > > OpenOCD has moved where it has placed the tcl scripts for TI's reference > > boards. Update their paths to the new locations. > > Minor: 'has moved where it has placed' reads awkwardly. Something like > 'OpenOCD has reorganised the tcl scripts for TI's reference boards > into a ti/ subdirectory' would be clearer, and it would be worth > naming the OpenOCD release/commit that made the move so readers know > which version they need.
Sure. > > > > > Acked-by: Andrew Davis <[email protected]> > > Reviewed-by: Anshul Dalal <[email protected]> > > Tested-by: Anshul Dalal <[email protected]> > > Signed-off-by: Bryan Brattlof <[email protected]> > > > > doc/board/ti/am6254atl_sk.rst | 2 +- > > doc/board/ti/am62ax_sk.rst | 2 +- > > doc/board/ti/am62dx_evm.rst | 11 ++++++----- > > doc/board/ti/am62px_sk.rst | 12 +++++------- > > doc/board/ti/am62x_sk.rst | 2 +- > > doc/board/ti/am64x_evm.rst | 24 ++++++++++++++++++++++++ > > doc/board/ti/am65x_evm.rst | 2 +- > > doc/board/ti/j7200_evm.rst | 2 +- > > doc/board/ti/j721e_evm.rst | 2 +- > > doc/board/ti/j722s_evm.rst | 2 +- > > doc/board/ti/j784s4_evm.rst | 2 +- > > 11 files changed, 43 insertions(+), 20 deletions(-) > > > diff --git a/doc/board/ti/am62px_sk.rst b/doc/board/ti/am62px_sk.rst > > @@ -331,13 +331,11 @@ detailed setup information. > > + Until the next stable release of OpenOCD is available in your > > development > > + environment's distribution, it might be necessary to build OpenOCD > > `from the > > + source <https://github.com/openocd-org/open ocd>`_. > > The URL has a stray space - 'openocd-org/open ocd' should be > 'openocd-org/openocd'. This will render as a broken link. Thanks. > > > diff --git a/doc/board/ti/am64x_evm.rst b/doc/board/ti/am64x_evm.rst > > @@ -227,3 +227,27 @@ https://www.ti.com/lit/pdf/spruim2 under the `Boot > > Mode Pins` section. > > + Until the next stable release of OpenOCD is available in your > > development > > + environment's distribution, it might be necissary to build OpenOCD > > `from the > > + source <https://github.com/openocd-org/open ocd>`_. > > Two issues on these lines: 'necissary' should be 'necessary' (the v4 > changelog claims this was fixed, but it is still here), and the URL > again has a stray space. Apparently I misspelled it multiple times. > > > diff --git a/doc/board/ti/am64x_evm.rst b/doc/board/ti/am64x_evm.rst > > @@ -227,3 +227,27 @@ https://www.ti.com/lit/pdf/spruim2 under the `Boot > > Mode Pins` section. > > +Debugging U-Boot > > +---------------- > > This whole new section is adding OpenOCD debugging documentation for > AM64x, not correcting a tcl path. Please split it into a separate > patch so the subject matches the change. Sure > > > diff --git a/doc/board/ti/j721e_evm.rst b/doc/board/ti/j721e_evm.rst > > @@ -261,4 +261,4 @@ To start OpenOCD and connect to the board > > - openocd -f board/ti_j721eevm.cfg > > + openocd -f board/ti/j721evm.cfg > > Just to check - every other file in this patch keeps the trailing name > identical (e.g. ti_am625evm.cfg -> ti/am625evm.cfg), but here the > second 'e' is dropped (j721eevm -> j721evm). Is that really what > OpenOCD ships, or is this a typo? Yes this has been a bug no one has noticed for a while. https://github.com/openocd-org/openocd/blob/master/tcl/board/ti/j721evm.cfg Previously named: https://github.com/openocd-org/openocd/blob/v0.12.0/tcl/board/ti_j721evm.cfg ~Bryan
