search

heikomat / linux

Linux kernel source tree
Other
43 stars 8 forks source link

Use Takashis updated patches to build the kernel packages #10

Closed heikomat closed 5 years ago

heikomat commented 5 years ago

As described in https://github.com/heikomat/linux/issues/8#issuecomment-493794843, @tiwai seems to have updated his cx2072x patches (whose 4.13-variant are the base this fork was built upon).

This means, that his 5.1-patches are probably of higher quality than the ones in this repo, because he actually knows what he's doing (i'm just capable of reading code and error messages, and was lucky to make his 4.13 patches work in newer kernels)

This however doesn't make this repository fully obsolete, as people seem to like the one-command-install-script that configures pulseaudio and installs the required firmware, alsa-ucm-files and prebuild kernel packages.

Because of this, this issue is here to track the following work:

  1. build a debian linux package based solely on Takashis cx2072x-5.1 kernel found here
  2. apply the pulsaudio configuration and install the ucm files and the linux package just like the installer script would
  3. Test if everything works like with our current 5.1-cx2072x (or maybe even better)
  4. If so, merge Takashis cx2072x-5.1-branch into the master of this fork and re-release 5.1

@roethigj Thank you for making me aware, that Takashi has updated patches!

heikomat commented 5 years ago

@7twin to answer your question from https://github.com/heikomat/linux/issues/8#issuecomment-493796212: Until the release based on Takashis updated patches is proven to work and ready for release, the current 5.1 release i made today will remain where it is. There is no reason not to test the arch build, as the build method should idealy not change at all after switching to Takashis updated patches.

7twin commented 5 years ago

@heikomat Great, then I'll still find the time to check if it works and report back. Should I post in the previous issue or here once I do?

heikomat commented 5 years ago

@7twin in the previous one please, because this one is about takashis patches, the other one is about arch packages.

heikomat commented 5 years ago

Also: as discussed here we might eventually have cx2072x patches in the regular linux kernel. Maybe with Kernel 5.3? When this happens AND alsa gets updated to include the ucm files, then sound on the e200ha should work out of the box. Only time will tell :D

7twin commented 5 years ago

@heikomat Sure, I'll post it in the other issue then. Regarding the 5.3 merge, that'd be actually great, because then we could just keep pulling the kernel and not worry about having to manually patch things in anymore :+1:

heikomat commented 5 years ago

ok, the 5.1 kernel is now based on takashis updated patches. One nice sideeffect is, that it now only differs from the regular linux kernel by 5 commits! (3 with patches from @tiwai, two with the updated ucm files and manual)

Thanks again to @roethigj, the hint regarding the namechange (chtcx2072x -> bytchtcx2072x) saved me a lot of time.

5.1 is now re-released, based on the updated patches from @tiwai 🎉

heikomat commented 5 years ago

@7twin see https://github.com/heikomat/linux_cx2072x_build/commit/7b70a07db75227a5e2ef277bb99924c244523637 for the changes in the dockerized build (just some config changes). Would be super nice, if you could adjust and test your dockerized arch build!

7twin commented 5 years ago

@heikomat ah that's for the touchpad fix I assume. I'll check if my config contains that and test it with docker asap!

heikomat commented 5 years ago

@7twin yeah, but also the rename of cht -> bytcht

7twin commented 5 years ago

@heikomat not sure if those affect my build (they don't affect the linked docker build either?), except maybe that the zcat'd config I created has to be done again because of the first thing you mentioned?