soc: lowrisc: opentitan: Fix manifest sizing (c8e9da26) · Commits · Wenxi XU / zephyr

The actual manifest is 1024 bytes, but we previously had padding bytes due to the implementation of SECTION_FUNC(). The manifest is not executable code so SECTION_VAR() is appropriate and produces a section of the appropriate size. Fixes: #82822 Signed-off-by:

Shawn Nematbakhsh <shawn@rivosinc.com>

soc/lowrisc/opentitan/Kconfig.defconfig

+1 −1

Changes for soc/lowrisc/opentitan/Kconfig.defconfig: 1 added line, 1 removed line.
Original line number	Diff line number	Diff line
		@@ -21,6 +21,6 @@ config NUM_IRQS
		# The OpenTitan SoC requires a manifest in front of the
		# application binary.
		config ROM_START_OFFSET
		default 0x404
		default 0x400

		endif # SOC_OPENTITAN

soc/lowrisc/opentitan/rom_header.S

+1 −1

Changes for soc/lowrisc/opentitan/rom_header.S: 1 added line, 1 removed line.
Original line number	Diff line number	Diff line
		@@ -17,7 +17,7 @@ GTEXT(__rom_header)
		* 0x6c47 (minor). The manifest format is documented here:
		* https://github.com/lowRISC/opentitan/blob/689a163294e1791bd30cfe096decf7f9233abad4/sw/host/opentitanlib/src/image/manifest.rs#L205
		*/
		SECTION_FUNC(rom_header, __rom_header)
		SECTION_VAR(rom_header, __rom_header)
		.rept(225)
		.word 0
		.endr

Admin message