Devicetree
 help / color / mirror / Atom feed
From: sashiko-bot@kernel.org
To: "Richard Genoud (TI)" <richard.genoud@bootlin.com>
Cc: robh@kernel.org, devicetree@vger.kernel.org, conor+dt@kernel.org
Subject: Re: [PATCH v4 7/7] arm64: dts: ti: k3-j742s2-ti-ipc-firmware: Reserve memory for LPM metadata
Date: Mon, 21 Sep 2026 14:59:37 +0000	[thread overview]
Message-ID: <20260921145937.DD5D61F000FF@smtp.kernel.org> (raw)
In-Reply-To: <20260921145325.300025-8-richard.genoud@bootlin.com>

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Low] dts: typo in SoC name in header comment
--

commit 4a69ff2c69c45956aad19749ee19fa4b33a413cf
Author: Richard Genoud (TI) <richard.genoud@bootlin.com>

arm64: dts: ti: k3-j742s2-ti-ipc-firmware: Reserve memory for LPM metadata

This commit declares a carveout memory region to store LPM metadata on the
J742S2 platform. The reserved memory is used to save and restore ATF, OPTEE,
and DM image context during suspend and resume cycles.

> diff --git a/arch/arm64/boot/dts/ti/k3-j742s2-ti-ipc-firmware.dtsi b/arch/arm64/boot/dts/ti/k3-j742s2-ti-ipc-firmware.dtsi
> new file mode 100644
> index 0000000000000..d66bcd2f4e194
> --- /dev/null
> +++ b/arch/arm64/boot/dts/ti/k3-j742s2-ti-ipc-firmware.dtsi
> @@ -0,0 +1,24 @@
> +// SPDX-License-Identifier: GPL-2.0-only OR MIT
> +/**
> + * Device Tree Source for enabling IPC using TI SDK firmware on J42S2 SoCs

[Severity: Low]
There appears to be a typo in the SoC name in the header comment of
arch/arm64/boot/dts/ti/k3-j742s2-ti-ipc-firmware.dtsi. Should J42S2 be
corrected to J742S2?

> + *
> + * Copyright (C) 2026 Texas Instruments Incorporated - https://www.ti.com/
> + * Copyright (C) 2026 Bootlin
> + */

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260921145325.300025-1-richard.genoud@bootlin.com?part=7

      reply	other threads:[~2026-09-21 14:59 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-09-21 14:53 [PATCH v4 0/7] dts: ti: k3-j7: Reserve memory for LPM metadata Richard Genoud (TI)
2026-09-21 14:53 ` [PATCH v4 1/7] dt-bindings: remoteproc: k3-r5f: Add Jacinto LPM memory region Richard Genoud (TI)
2026-09-22 16:02   ` Rob Herring (Arm)
2026-09-21 14:53 ` [PATCH v4 2/7] remoteproc: k3: skip mapping firewalled lpm-metadata " Richard Genoud (TI)
2026-09-21 14:53 ` [PATCH v4 3/7] arm64: dts: ti: k3-j7200-ti-ipc-firmware: Reserve memory for LPM metadata Richard Genoud (TI)
2026-09-21 15:03   ` sashiko-bot
2026-09-21 14:53 ` [PATCH v4 4/7] arm64: dts: ti: k3-j721s2-ti-ipc-firmware: " Richard Genoud (TI)
2026-09-21 14:53 ` [PATCH v4 5/7] arm64: dts: ti: k3-j784s4-ti-ipc-firmware: " Richard Genoud (TI)
2026-09-21 14:53 ` [PATCH v4 6/7] arm64: dts: ti: k3-j722s-ti-ipc-firmware: " Richard Genoud (TI)
2026-09-21 14:53 ` [PATCH v4 7/7] arm64: dts: ti: k3-j742s2-ti-ipc-firmware: " Richard Genoud (TI)
2026-09-21 14:59   ` sashiko-bot [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20260921145937.DD5D61F000FF@smtp.kernel.org \
    --to=sashiko-bot@kernel.org \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=richard.genoud@bootlin.com \
    --cc=robh@kernel.org \
    --cc=sashiko-reviews@lists.linux.dev \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox