Open ColorfulRhino opened 1 week ago
Jira ticket: AR-2394
I've one use case for the legacy / 5.10 kernel: being able to write MAC addresses to the Realtek 2.5gbit (r8125/r8169) ethernets. See instructions at Mario's -- the program only builds and works on 5.10, not 6.1.
Otherwise, I use legacy on "production" systems -- if it is removed from Armbian, I'll be forced to userpatch the hell out of it.
We can remove it from board configs - "legacy" is anyway EOL label. And later remove source defs.
Yes. I'd be happy if it's removed from KERNEL_TARGETS, since we can actually build BRANCH=legacy even if it isn't listed there.
But I'd keep the family code handling it, which amounts to a 10-line block (repeated twice, just like vendor
, due to the rk35xx
/rockchip-rk3588
(-which-is-actually-rk35xx-LINUXFAMILY) duplication crazy.
Okay we can do it like that if you prefer π
I'll keep this task open as a reminder to finally delete legacy
at some point later down the line (in some months/a year? we can see and re-evaluate in a few months).
I have updated the task list to reflect this approach. Feel free to edit it :)
With a soon-to-be-updated linux-rockchip
6.1-rkr3
kernel, it may be time to finally sunset the 5.10legacy
kernel for RK3588. The 6.1vendor
kernel is overall just a better/more up-to-date version of the Rockchip kernel and will receive updates down the line. With the recent addition of thecurrent
kernel branch for RK3588, we currently have 5 active kernel branches for this SOC: https://github.com/armbian/build/blob/main/config/sources/families/rockchip-rk3588.conf It's also easier for @amazingfate to maintain only one branch of the kernel.This task/issue is a place to discuss this topic before implementation. Let's hear your opinions π
Task List
For marking the
legacy
kernel as deprecated/EOL (now):KERNELPATCHDIR
to default forvendor
in the family configFor removing the
legacy
kernel as a whole (TO-DO at a later date):legacy
andvendor
point to the same variables in the family configrk3588-legacy
legacy
rk3588 kernel in the code