Enable iptables comments for all targets

[monosoul]

Atm some targets don't have comments support enabled for iptables. There's no reason to limit that to just arm64-rockhip target, so this change addresses that.

Summary by CodeRabbit

• Chores
  • Adjusted kernel configuration settings to optimize network filtering support across different platform builds.

[home-assistant]

Hi @monosoul

It seems you haven't yet signed a CLA. Please do so here.

Once you do that we will be able to review and accept this pull request.

Thanks!

[home-assistant]

Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍

Learn more about our pull request process.

[home-assistant]

Hi @monosoul

It seems you haven't yet signed a CLA. Please do so here.

Once you do that we will be able to review and accept this pull request.

Thanks!

Stale

[coderabbitai]

📝 Walkthrough

Walkthrough

A single kernel configuration option for netfilter comment matching (CONFIG_NETFILTER_XT_MATCH_COMMENT) is added to the generic hassos configuration and removed from the ARM64 Rockchip-specific configuration.

Changes

Cohort / File(s)	Summary
Kernel Configuration - NETFILTER_XT_MATCH_COMMENT buildroot-external/kernel/v6.12.y/hassos.config	Added CONFIG_NETFILTER_XT_MATCH_COMMENT=m to enable netfilter comment match target as a loadable module.
Kernel Configuration - NETFILTER_XT_MATCH_COMMENT buildroot-external/kernel/v6.12.y/kernel-arm64-rockchip.config	Removed CONFIG_NETFILTER_XT_MATCH_COMMENT=m configuration option from ARM64 Rockchip-specific kernel config.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check	✅ Passed	The title accurately describes the main change: enabling iptables comments for all targets by adding CONFIG_NETFILTER_XT_MATCH_COMMENT support to hassos.config.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests 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.}

[monosoul]

Tbh, I think it prolly makes sense to enable other iptables-related modules for non arm64-rockhip targets too. I'm up for updating the PR in case the maintainers wouldn't mind that.

[sairon]

Thank you! Sure there is no reason to enable it only for some targets.

Tbh, I think it prolly makes sense to enable other iptables-related modules for non arm64-rockhip targets too. I'm up for updating the PR in case the maintainers wouldn't mind that.

I'd rather say the Rockchip fragment deserves a cleanup and enable modules on a justified basis.