Skip to content

doc: Add vendor filter for hw feature generation #89344

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

asmellby
Copy link
Collaborator

The hw feature generation takes a long time. The HW_FEATURES_TURBO_MODE option completely disables hw feature generation. Add a new option HW_FEATURES_VENDOR_FILTER to be able to selectively enable hw feature generation only for a given list of vendors. This option is useful when working on board documentation pages.

The hw feature generation takes a long time. The HW_FEATURES_TURBO_MODE
option completely disables hw feature generation. Add a new option
HW_FEATURES_VENDOR_FILTER to be able to selectively enable hw feature
generation only for a given list of vendors. This option is useful
when working on board documentation pages.

Signed-off-by: Aksel Skauge Mellbye <[email protected]>
@asmellby asmellby force-pushed the feature/doc-hw-feature-vendor-filter branch from 047f6da to b0b2311 Compare April 30, 2025 16:58
Copy link
Collaborator

@kartben kartben left a comment

Choose a reason for hiding this comment

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

will need to do a full review so just a random comment for now. Looks neat and very complete - thanks much for this!

You basically beat me to opening this PR as I had this on my backlog pretty much since the feature shipped and have been playing with hardcoded --vendor filters ever since :)

Comment on lines +207 to +209
if vendor_filter:
for vendor in vendor_filter:
twister_cmd += ["--vendor", vendor]
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think I'd rather have it be used in lieu of --all, when a filter has been provided, as it supersedes it anyway

@kartben kartben requested a review from Copilot April 30, 2025 20:47
Copilot

This comment was marked as outdated.

@kartben kartben requested a review from Copilot May 2, 2025 00:25
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a new configuration option for selectively enabling hardware feature generation for a specified list of vendors, which improves performance when generating board documentation.

  • Introduces the zephyr_hw_features_vendor_filter configuration in conf.py.
  • Updates run_twister_cmake_only and get_catalog to utilize the vendor filter during board catalog generation.
  • Registers the new configuration option in the Sphinx domain extension.

Reviewed Changes

Copilot reviewed 3 out of 7 changed files in this pull request and generated no comments.

File Description
doc/conf.py Adds a new configuration variable to control vendor-based filtering
doc/_scripts/gen_boards_catalog.py Modifies functions to accept and pass the vendor_filter parameter
doc/_extensions/zephyr/domain/init.py Updates Sphinx configuration to include the new vendor filter option
Files not reviewed (4)
  • doc/CMakeLists.txt: Language not supported
  • doc/Makefile: Language not supported
  • doc/contribute/documentation/generation.rst: Language not supported
  • doc/contribute/documentation/guidelines.rst: Language not supported

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

Successfully merging this pull request may close these issues.

2 participants