rockchip: Allow dt overlays that disable nodes

[vidplace7]

Description

On armhf rockchip vendor, currently kernel-bundled devicetree overlays that set a node to disabled are not compiled correctly (compiled file does not include disabled directives).

CONFIG_ROCKCHIP_MINI_KERNEL implicitly sets

CONFIG_DTC_OMIT_DISABLED=y
CONFIG_DTC_OMIT_EMPTY=y

this causes bundled devicetree overlays which set nodes to "disabled" to become noop.

Disable these DTC_OMIT options to restore overlay disable functionality.

Relates to:

• RK3506: Add overlay support / add overlays to disable Ethernet ifaces linux-rockchip#447

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Please also note any relevant details for your test configuration.

• Compile/run tested against armsom-forge1 using kernel from RK3506: Add overlay support / add overlays to disable Ethernet ifaces linux-rockchip#447
• Can list overlays and apply them with armbian-config
• Confirmed kernel-bundled overlays which set disabled function correctly.

Checklist:

• My code follows the style guidelines of this project
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• My changes generate no new warnings
• Any dependent changes have been merged and published in downstream modules

Summary by CodeRabbit

• Chores
  • Updated kernel configuration file with clarification comments for device tree compiler settings. No functional changes.

[coderabbitai]

📝 Walkthrough

Walkthrough

Two comment lines indicating unset DTC options are added to the Linux Rockchip vendor kernel configuration file. These are informational annotations with no functional impact on kernel configuration or behavior.

Changes

Cohort / File(s)	Summary
Kernel Configuration Comments config/kernel/linux-rockchip-vendor.config	Added two comment lines documenting that DTC_OMIT_DISABLED and DTC_OMIT_EMPTY configuration options are not set.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 Hops through configs with glee,
Comments added, plain to see,
DTC options marked "not set,"
Documentation, no regret!

🚥 Pre-merge checks | ✅ 3 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Merge Conflict Detection	⚠️ Warning	❌ Merge conflicts detected (2 files): ⚔️ config/kernel/linux-rockchip-vendor.config (content) ⚔️ extensions/ti-debpkgs.sh (content) These conflicts must be resolved before merging into main.	Resolve conflicts locally and push changes to this branch.

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'rockchip: Allow dt overlays that disable nodes' directly and specifically describes the main functional change — enabling device-tree overlays to properly retain 'disabled' directives by disabling the DTC_OMIT options.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

⚔️ Resolve merge conflicts (beta)

• Auto-commit resolved conflicts to branch rockchip-fix-disable-dtbos
• Post resolved changes as copyable diffs in a comment

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[paolosabatino]

Weird feature 🤔
Merge at will!

[github-actions]

✅ This PR has been reviewed and approved — all set for merge!

[rpardini]

Vendor kernel never ceases to amaze. Nice find.