Skip to content

batman-adv: drop kmod-lib-crc32c on newer kernels#1162

Merged
simonwunderlich merged 1 commit intoopenwrt:masterfrom
namiltd:patch-1
Mar 30, 2026
Merged

batman-adv: drop kmod-lib-crc32c on newer kernels#1162
simonwunderlich merged 1 commit intoopenwrt:masterfrom
namiltd:patch-1

Conversation

@namiltd
Copy link
Copy Markdown
Contributor

@namiltd namiltd commented Mar 3, 2026

This patch makes the dependency on kmod-lib-crc32c for the batman-adv package conditional: it is only required for Linux kernel 6.12. For kernel 6.18 and newer, the dependency is no longer needed. This prevents installing unnecessary modules on newer kernels.

This patch makes the dependency on kmod-lib-crc32c for the batman-adv package conditional: it is only required for Linux kernel 6.12. For kernel 6.18 and newer, the dependency is no longer needed. This prevents installing unnecessary modules on newer kernels.

Signed-off-by: Mieczyslaw Nalewaj <namiltd@poczta.onet.pl>
@namiltd
Copy link
Copy Markdown
Contributor Author

namiltd commented Mar 25, 2026

Ping @PolynomialDivision

@PolynomialDivision
Copy link
Copy Markdown
Member

@namiltd the correct maintainer is @ecsv

@ecsv
Copy link
Copy Markdown
Contributor

ecsv commented Mar 30, 2026

@namiltd the correct maintainer is @ecsv

no, it is @simonwunderlich

Copy link
Copy Markdown
Contributor

@ecsv ecsv left a comment

Choose a reason for hiding this comment

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

it is only required for Linux kernel 6.12. For kernel 6.18 and newer, the dependency is no longer needed. This prevents installing unnecessary modules on newer kernels.

To be more factual correct: The CONFIG_LIBCRC32C dependency is gone since Linux v6.15-rc2. batman-adv is since then only depending on CONFIG_CRC32 - which is activated automatically by the generic kernel configuration.

@simonwunderlich simonwunderlich merged commit 4ec8a66 into openwrt:master Mar 30, 2026
11 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.

4 participants