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.

Sign up for GitHub

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

Jump to bottom

Bump up kernel version, fw and ucode #4183

Merged
merged 3 commits into from
Sep 2, 2024
Merged

Bump up kernel version, fw and ucode #4183

merged 3 commits into from
Sep 2, 2024

Conversation

rouming
Copy link
Contributor

@rouming rouming commented Aug 29, 2024 •
edited
Loading

This just bumps up a kernel version in the kernel-commits.mk, kernel-version.mk and also increases fw and ucode to the latest available by today.

So these versions will be used:

amd64-generic: v6.1.106
amd64-rt: v6.1.105
arm64-generic: v6.1.106
riscv64-generic: v6.1.106

Update: this should be merged after this is merged: lf-edge/eve-kernel#133

@rouming rouming requested a review from rucoder August 29, 2024 13:44
@rouming rouming force-pushed the bump-up-kernel-fw-ucode branch from e403899 to 596e119 Compare August 29, 2024 13:52
OhmSpectator
OhmSpectator reviewed Aug 29, 2024
View reviewed changes
rucoder
rucoder suggested changes Aug 29, 2024
View reviewed changes
@rouming rouming force-pushed the bump-up-kernel-fw-ucode branch from 596e119 to 1ee77e0 Compare August 30, 2024 10:57
@rouming rouming marked this pull request as draft August 30, 2024 10:59
eriknordmark
eriknordmark approved these changes Sep 1, 2024
View reviewed changes
Copy link
Contributor

@eriknordmark eriknordmark left a comment

Choose a reason for hiding this comment

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

LGTM but the commit message linter is not happy

eriknordmark
eriknordmark approved these changes Sep 1, 2024
View reviewed changes
Copy link
Contributor

@eriknordmark eriknordmark left a comment

Choose a reason for hiding this comment

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

Once more

@christoph-zededa
Copy link
Contributor

Can we please have commit messages that are relevant?
F.e.:

e61cce864cfa: Dockerfile: add pahole dependency

is not only irrelevant for this repository, but it is even mentioned twice.

@rouming
pkg/fw: bump up firwmare and ucode to latest
508b231 
Just keep both up-to-date

Signed-off-by: Roman Penyaev <[email protected]>
@rouming
Kernel update - [amd64-generic, amd64-rt, amd64-generic, arm64-generi…
ce4a38a 
…c, arm64-generic, riscv64-generic]

eve-kernel-amd64-v6.1.38-generic
    ea5616553bc8: Disable KEMPLD_GPIO
    d0e0aa8e9a50: Use template instead of gpiochip_irqchip_add_nested
    98bd856cdfb8: MFD: Add Kontron PLD drivers (kempld)

eve-kernel-amd64-v6.1.105-rt
    b3b777f0e02a .github/workflows/publish.yml: switch to the v6.1.105 branch
    a59e2f1c04b2 Makefile.eve: bump up KERNEL_TAG to v6.1.105
    51c61f599176 eve_defconfig: Enable PSI (Pressure Stall Information).
    1f1a76f27b03 Dockerfile: ensure this option is not set
    e92c2b955b4a bpf: lockdown for security
    (full git log: lf-edge/eve-kernel@8fe1d58...b3b777f)

eve-kernel-amd64-v6.1.106-generic
    f99091b2865e: .github/workflows/publish.yml: switch to the v6.1.106 branch
    a8088585edf1: Makefile.eve: bump up KERNEL_TAG to v6.1.106
    188c83106027: Disable KEMPLD_GPIO
    4791ff80cc80: Use template instead of gpiochip_irqchip_add_nested
    e4e5b211d379: MFD: Add Kontron PLD drivers (kempld)
    e7a508161f48: eve_defconfig: Enable PSI (Pressure Stall Information).
    (full git log: lf-edge/eve-kernel@50a48e3...f99091b)

eve-kernel-arm64-v6.1.38-generic
    6dc8132a2e10: drivers: rtc: add rpi-rtc driver for Raspberry Pi 5
    dccb76edb094: dts: Add device tree for Raspberry Pi 5
    dcc640f25194: Add ethernet support for Raspberry Pi 5
    c6323f5c51f3: Add pci/usb support for Raspberry Pi 5
    1a0891be1657: Add sd card support for Raspberry Pi 5
    c892d29d4ab1: Add general support for Raspberry Pi 5

eve-kernel-arm64-v6.1.106-generic
    21c6311b7fa3: .github/workflows/publish.yml: switch to the v6.1.106 branch
    894d7cd7a8b2: Makefile.eve: bump up KERNEL_TAG to v6.1.106
    1bf372e24224: drivers: rtc: add rpi-rtc driver for Raspberry Pi 5
    38cf4baa867d: dts: Add device tree for Raspberry Pi 5
    2c33a083408a: Add ethernet support for Raspberry Pi 5
    (full git log: lf-edge/eve-kernel@bb04ab1...21c6311)

eve-kernel-riscv64-v6.1.106-generic
    f8f3bd1d26f6: .github/workflows/publish.yml: switch to the v6.1.106 branch
    c2fd6a863b25: Dockerfile: ensure this option is not set
    df6d7ada77bf: bpf: lockdown for security
    1838a44ab4c9: Dockerfile.clang: fix spelling errors
    44fa350a111c: Make sure we have correct buildx builder
    (full git log: lf-edge/eve-kernel@7e4bf6f...f8f3bd1)

Signed-off-by: Roman Penyaev <[email protected]>
@rouming
kernel-version.mk: switch to latest kernels
b93e6e8 
The following kernels are switched to latest eve-kernel branches:

amd64-generic:   v6.1.106
amd64-rt:        v6.1.105
arm64-generic:   v6.1.106
riscv64-generic: v6.1.106

Signed-off-by: Roman Penyaev <[email protected]>
@rouming rouming force-pushed the bump-up-kernel-fw-ucode branch from 1ee77e0 to b93e6e8 Compare September 2, 2024 10:13
@rouming rouming marked this pull request as ready for review September 2, 2024 10:14
@rouming
Copy link
Contributor Author

rouming commented Sep 2, 2024

Can we please have commit messages that are relevant? F.e.:

e61cce864cfa: Dockerfile: add pahole dependency

is not only irrelevant for this repository, but it is even mentioned twice.

I understand your concern to minimize the log for autogenerated commit, which described the difference in eve-kernel. Here in this PR I manually minimized the log by reducing to 5 commits for each branch and put the link on github for those who are thrilled to see the full log. Christoph, I suggest you to raise this concern to @rucoder and contribute to the tools/update_kernel_commits.py with a PR, then it makes more sense to change things rather to complain in each commit which aims to update kernel SHA.

@OhmSpectator
Copy link
Member

By the way, I want to fix the commit message linter to allow kernel update commits in the form they are done now. Ignore it until then.

eriknordmark
eriknordmark approved these changes Sep 2, 2024
View reviewed changes
Copy link
Contributor

@eriknordmark eriknordmark left a comment

Choose a reason for hiding this comment

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

LGTM - ignoring the commit linter.

@eriknordmark eriknordmark merged commit 6881580 into lf-edge:master Sep 2, 2024
37 of 38 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@OhmSpectator OhmSpectator OhmSpectator left review comments

@rucoder rucoder rucoder requested changes

@eriknordmark eriknordmark eriknordmark approved these changes

@rene rene Awaiting requested review from rene rene is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@rouming @christoph-zededa @OhmSpectator @rucoder @eriknordmark