Skip to content

arm64:configs: updated zynqmp_defconfig to enable GPIO_SYSFS and EXPERT#3210

Merged
nunojsa merged 1 commit intoanalogdevicesinc:mainfrom
Alexandru-Sarbu:stingray_update
Mar 27, 2026
Merged

arm64:configs: updated zynqmp_defconfig to enable GPIO_SYSFS and EXPERT#3210
nunojsa merged 1 commit intoanalogdevicesinc:mainfrom
Alexandru-Sarbu:stingray_update

Conversation

@Alexandru-Sarbu
Copy link
Copy Markdown
Contributor

PR Description

  • CONFIG_GPIO_SYSFS and CONFIG_EXPERT were enabled in adi_zynqmp_defconfig to restore functionality of the stingray_power python script for the X-Band Phased Array Platform project.

PR Type

  • Bug fix (a change that fixes an issue)
  • New feature (a change that adds new functionality)
  • Breaking change (a change that affects other repos or cause CIs to fail)

PR Checklist

  • I have conducted a self-review of my own code changes
  • I have compiled my changes, including the documentation
  • I have tested the changes on the relevant hardware
  • I have updated the documentation outside this repo accordingly
  • I have provided links for the relevant upstream lore

Enable CONFIG_GPIO_SYSFS and CONFIG_EXPERT to restore the functionality
of the linux_image_ADI-scripts/power-service/stingray_power.py script
for the X-Band Phased Array project. Also added CONFIG_NAMESPACES to
ensure that CONFIG_USER_NS does not get disabled when running make
savedefconfig. Bothe NAMESPACES and USER_NS were added in commit
492b6c7 and NAMESPACES was later removed in commit d31fa31.

Signed-off-by: Alex Sarbu <alex.sarbu@analog.com>
Copy link
Copy Markdown
Collaborator

@nunojsa nunojsa left a comment

Choose a reason for hiding this comment

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

One minor question (more out of curiosity). Also, on a process thing. Commit subject should be arm64:configs: adi_zynqmp:... . Also note the space after :

CONFIG_IKCONFIG_PROC=y
CONFIG_LOG_BUF_SHIFT=16
CONFIG_CGROUPS=y
CONFIG_NAMESPACES=y
Copy link
Copy Markdown
Collaborator

@nunojsa nunojsa Mar 26, 2026

Choose a reason for hiding this comment

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

Curious, why do we need this? Do we have any sandbox/container thing relying on it?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Noted. It actually was initially like that, but I decided to try and adhere to the 50/72 rule. Will use the proper formatting in the future. Regarding the NAMESPACES, as per the initial commit that added this (492b6c7), it is needed for Scopy and Chromium. Wrote a more detailed history in the commit body

@nunojsa nunojsa merged commit acce9f0 into analogdevicesinc:main Mar 27, 2026
29 checks passed
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.

2 participants