Hi Bryan, On 2026-08-04T21:55:02, Bryan Brattlof <[email protected]> wrote: > doc: ti: k3: fix pygments warning parsing device-tree > > A warning is emitted when generating the FIT image examples because of > white-space and invalid node names. Fix them
Please mention what the fix actually is (converting the placeholder brackets into valid DTS node syntax). > > Tested-by: Anshul Dalal <[email protected]> > Reviewed-by: Anshul Dalal <[email protected]> > Signed-off-by: Bryan Brattlof <[email protected]> > > doc/board/ti/k3.rst | 42 +++++++++++++++++++++++------------------- > 1 file changed, 23 insertions(+), 19 deletions(-) > diff --git a/doc/board/ti/k3.rst b/doc/board/ti/k3.rst > @@ -466,25 +466,29 @@ choose any configuration at runtime to boot from. > + configurations { > + default = <conf-1> > + conf-1 { This line is still not valid DTS - it is missing the trailing semicolon, and for FIT the 'default' property is a string, so it should be: default = 'conf-1'; > diff --git a/doc/board/ti/k3.rst b/doc/board/ti/k3.rst > @@ -466,25 +466,29 @@ choose any configuration at runtime to boot from. > + conf-2 { > + image = "image-2"; > + fdt = "fdt-1"; > + }; The two properties inside conf-2 are indented one extra space compared with those in conf-1 above. Please make them consistent. > diff --git a/doc/board/ti/k3.rst b/doc/board/ti/k3.rst > @@ -466,25 +466,29 @@ choose any configuration at runtime to boot from. > - /dts-v1/; > - > - / { Any reason to drop the /dts-v1/; header? The block is tagged '.. code-block:: dts' so keeping it makes the snippet a complete, valid DTS fragment. While you are here, the surrounding examples use 4-space indentation; changing this one to 3 spaces is a bit jarring. Regards, Simon
