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
prev parent 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