Skip to content
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

kernel: 5.15: disable kmod-ptp for broken targets #11486

Closed
wants to merge 1 commit into from

Conversation

johnnyslee
Copy link
Contributor

Currently in linux 5.15, mv88e6xxx dsa driver cannot be built-in as described in commit 97c77ff

This disables kmod-ptp, which is dragging the driver to become a kmod, for the following affected targets in linux 5.15:

  • kirkwood
  • mvebu
  • qoriq

Currently in linux 5.15, mv88e6xxx dsa driver cannot be built-in as
described in commit 97c77ff

This disables kmod-ptp, which is dragging the driver to become a kmod,
for the following affected targets in linux 5.15:
- kirkwood
- mvebu
- qoriq

Signed-off-by: Johnny S. Lee <foss@jsl.io>
@github-actions github-actions bot added core packages pull request/issue for core (in-tree) packages kernel pull request/issue with Linux kernel related changes labels Dec 13, 2022
johnnyslee referenced this pull request Dec 13, 2022
This reverts commit 5429411 as upstream
in commit e5f31552674e ("ethernet: fix PTP_1588_CLOCK dependencies") has
changed `PTP_1588_CLOCK` dependency handling in 5.15 kernel.

That currently leads to `CONFIG_NET_DSA_MV88E6XXX=m` in images produced
by buildbots due to `CONFIG_ALL_KMODS=y` config option being used in
those builds, which leads to a broken LAN bridge network on several
devices.

References: https://lists.infradead.org/pipermail/openwrt-devel/2022-December/039950.html
Signed-off-by: Petr Štetiar <ynezz@true.cz>
@BKPepe
Copy link
Member

BKPepe commented Dec 14, 2022

Just sharing my opinion. What I am doing for almost 6 months in my environment is to have separate kmod package with the Marvell switch. You can look here https://gitlab.nic.cz/turris/os/build/-/blob/hbk/patches/openwrt/a-new-kernel-5.15/0012-separate-kmod-marvell-switch.patch this ensures that the Marvell switch is indeed there if you can not rely on kernel configs. It was also described here https://lists.openwrt.org/pipermail/openwrt-devel/2022-December/039972.html , so it was no need to disable kmod-ptp, but I think others should decide what is preferred.

@johnnyslee
Copy link
Contributor Author

See also: Discussion in 97c77ff

@johnnyslee
Copy link
Contributor Author

Better solution here: 97c77ff#commitcomment-93150734

@johnnyslee johnnyslee closed this Dec 15, 2022
@johnnyslee
Copy link
Contributor Author

Replaced by PR #11522

fengmushu pushed a commit to fengmushu/openwrt that referenced this pull request Nov 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core packages pull request/issue for core (in-tree) packages kernel pull request/issue with Linux kernel related changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants