Bug#1009858: linux: please consider building the modules necessary for the MNT Reform laptop

[Johannes Schauer Marin Rodrigues]

Source: linux
Version: 5.10.106-1
Severity: wishlist
X-Debbugs-Cc: jo...@debian.org

Hi,

currently, we add the following to the arm64 kernel config provided by
the Debian kernel packaging to build a kernel that powers the MNT Reform
2 open source laptop:

CONFIG_DRM_LVDS_CODEC=m
CONFIG_DRM_SIMPLE_BRIDGE=m
CONFIG_DRM_TI_SN65DSI86=m
CONFIG_DRM_CDNS_MHDP8546=m
CONFIG_DRM_CDNS_HDMI_CEC=m
CONFIG_DRM_IMX_CDNS_MHDP=m
CONFIG_DRM_IMX_DCSS=m
CONFIG_DRM_PANEL_LVDS=m
CONFIG_I2C_IMX_LPI2C=m
CONFIG_I2C_MUX_REG=m
CONFIG_INTERCONNECT_IMX=m
CONFIG_INTERCONNECT_IMX8MQ=m
CONFIG_MFD_WM8994=m
CONFIG_MUX_GPIO=m
CONFIG_MUX_MMIO=m
CONFIG_RTC_DRV_PCF8523=m
CONFIG_USB_EHCI_FSL=m
CONFIG_BACKLIGHT_GPIO=m
CONFIG_BACKLIGHT_LED=m
CONFIG_NO_HZ_IDLE=y
CONFIG_SND_SOC_WM8960=m
CONFIG_SND_SOC_FSL_MICFIL=m
CONFIG_SND_IMX_SOC=m
CONFIG_SND_SOC_FSL_ASOC_CARD=m
CONFIG_SND_SOC_IMX_AUDMIX=m
CONFIG_SND_SOC_IMX_HDMI=m

Would you consider enabling these modules in the official Debian
packaging?

We still carry some patches that are not upstreamed yet but I wanted to
bring up this topic with you already to discuss this in parallel.

Thanks!

cheers, josch

[Vagrant Cascadian]

On 2022-04-19, Johannes Schauer Marin Rodrigues wrote:
> currently, we add the following to the arm64 kernel config provided by
> the Debian kernel packaging to build a kernel that powers the MNT Reform
> 2 open source laptop:

Thanks!

Merged most of the changes so far except...


> CONFIG_DRM_CDNS_HDMI_CEC=m
> CONFIG_DRM_IMX_CDNS_MHDP=m

These are not yet in upstream 5.17.x, so I left them out.


> CONFIG_NO_HZ_IDLE=y

It seems that the default defined in debian/config/config is
NO_HZ_FULL=y should default to the same behavior as NO_HZ_IDLE unless a
boot parameter is passed, according to kernel/time/Kconfig.


live well,
vagrant

[Vagrant Cascadian]

Control: tags 1009858 pending

On 2022-04-24, Vagrant Cascadian wrote:
> On 2022-04-19, Johannes Schauer Marin Rodrigues wrote:
>> currently, we add the following to the arm64 kernel config provided by
>> the Debian kernel packaging to build a kernel that powers the MNT Reform
>> 2 open source laptop:
>
> Thanks!
>
> Merged most of the changes so far except...

https://salsa.debian.org/kernel-team/linux/-/commit/aa9f2cfbd99e598ad02043d9238bf7886fc21d1a


live well,
vagrant

[Vagrant Cascadian]

On 2022-04-24, Vagrant Cascadian wrote:

> On 2022-04-19, Johannes Schauer Marin Rodrigues wrote:
>> currently, we add the following to the arm64 kernel config provided by
>> the Debian kernel packaging to build a kernel that powers the MNT Reform
>> 2 open source laptop:
>
> Thanks!
>
> Merged most of the changes so far except...

Which I can confirm boots! Although no output on the LCD, just serial
console...

>> CONFIG_DRM_CDNS_HDMI_CEC=m
>> CONFIG_DRM_IMX_CDNS_MHDP=m
>
> These are not yet in upstream 5.17.x, so I left them out.

Presumably LCD and HDMI don't work due to missing those, and/or other
not-yet-upstream patches...


live well,
vagrant

[Johannes Schauer Marin Rodrigues]

Hi Vagrant,

Quoting Vagrant Cascadian (2022-04-26 02:07:51)

amazing!! :D Having serial console work with the stock kernel from Debian will
make it so much easier to test and fix other stuff like debian-installer early
on. Thank you! :)

Yes, LCD support is part of the remaining patches:

https://source.mnt.re/reform/reform-debian-packages/-/tree/main/linux/patches

I also saw your message in #mnt-reform yesterday and you are right, my script
sets console=ttymxc1 instead of console=ttymxc0. This is due to a wrong commit
from my end because my S1 (ttymxc0) port has a non-functional TX pin so I was
always using S2 (ttymxc1) to talk with my reform. Fixed here:

https://source.mnt.re/reform/reform-system-image/-/merge_requests/46

Thanks again!

cheers, josch