Skip to content

Support remoteproc boot arguments in job definitions#43

Open
anankulk wants to merge 1 commit intoqualcomm-linux:mainfrom
anankulk:cmdline_args
Open

Support remoteproc boot arguments in job definitions#43
anankulk wants to merge 1 commit intoqualcomm-linux:mainfrom
anankulk:cmdline_args

Conversation

@anankulk
Copy link
Copy Markdown

@anankulk anankulk commented Apr 28, 2026

Summary

Add support for dynamic kernel boot arguments in LAVA job definitions by introducing the default_boot_args parameter.

Changes

  • lava_Job_definition_generator.py: Read DEFAULT_BOOT_ARGS from environment and pass to template renderer
  • templates/boot/fastboot.jinja2: Append boot arguments to kernel cmdline using Jinja2 conditional

Example output:

--cmdline "console=ttyMSM0,115200n8 ... video=efifb:off pd_ignore_unused clk_ignore_unused"

Dependencies

Requires corresponding changes in the fastrpc repository to pass DEFAULT_BOOT_ARGS environment variable.


Related PR:
https://github.com/qualcomm-linux-stg/fastrpc/pull/114

- Add default_boot_args parameter to template renderer.
- Update fastboot.jinja2 to include boot args in kernel cmdline
- Handle empty/undefined values gracefully with Jinja2 conditional

This enables passing custom kernel arguments (e.g., earlycon, pd_ignore_unused,
clk_ignore_unused) to the boot image for remoteproc configuration.

Signed-off-by: Anand Kulkarni <anankulk@qti.qualcomm.com>
target_dtb = os.environ.get("TARGET_DTB")
flash_image = os.environ.get("FLASH_IMAGE")
flash_port = os.environ.get("FLASH_PORT", "0")
default_boot_args = os.environ.get("DEFAULT_BOOT_ARGS", "")
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we can't have BOOT ARGS from env

@yogeshlal
Copy link
Copy Markdown
Collaborator

We can use "pd_ignore_unused clk_ignore_unused" , coordinate with @taniyadas20 if you need any help

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants