XPost: linux.debian.bugs.dist, linux.debian.maint.boot
From: kibi@debian.org

Hi,

Looping in debian-kernel@, context being missing touchpad support on an
amd64 machine, unless an extra module is added to the installer.

Feel free to steal the bug report away from us and adjust debian/installer/modules as you see fit!

Roland Clobus <rclobus@rclobus.nl> (2025-05-28):

On 27/05/2025 19:20, Pascal Hambourg wrote:

It looks like #1104057. Does lsmod show a pinctrl* module on a
system with working touchpad ?

I've built a custom installer and `pinctrl-meteorlake` does the trick.
With this additional kernel module the touchpad works in the graphical installer.

Checking linux.git, it seems pinctrl modules are only shipped in udebs
for a single arch:

kibi@tokyo:~/debian-kernel/linux.git (debian/6.12/trixie <)$ git grep pinctrl -- debian/installer/
debian/installer/modules/arm64/kernel-image:drivers/pinctrl/**

I'm not sure how many pinctrl modules would make sense to ship, or in
which udeb, for other architectures.

Looking at the amd64 config, the modular ones are:

CONFIG_PINCTRL_ALDERLAKE=m
CONFIG_PINCTRL_CS42L43=m
CONFIG_PINCTRL_ELKHARTLAKE=m
CONFIG_PINCTRL_EMMITSBURG=m
CONFIG_PINCTRL_INTEL_PLATFORM=m
CONFIG_PINCTRL_JASPERLAKE=m
CONFIG_PINCTRL_LAKEFIELD=m
CONFIG_PINCTRL_METEORLAKE=m
CONFIG_PINCTRL_METEORPOINT=m

while the built-in ones are:

CONFIG_PINCTRL_AMD=y
CONFIG_PINCTRL_BAYTRAIL=y
CONFIG_PINCTRL_CHERRYVIEW=y
CONFIG_PINCTRL_INTEL=y
CONFIG_PINCTRL_BROXTON=y
CONFIG_PINCTRL_CANNONLAKE=y
CONFIG_PINCTRL_CEDARFORK=y
CONFIG_PINCTRL_DENVERTON=y
CONFIG_PINCTRL_GEMINILAKE=y
CONFIG_PINCTRL_ICELAKE=y
CONFIG_PINCTRL_LEWISBURG=y
CONFIG_PINCTRL_SUNRISEPOINT=y
CONFIG_PINCTRL_TIGERLAKE=y


Cheers,
--
Cyril Brulebois (kibi@debian.org) <https://debamax.com/>
D-I release manager -- Release team member -- Freelance Consultant

-----BEGIN PGP SIGNATURE-----

iQIzBAABCgAdFiEEtg6/KYRFPHDXTPR4/5FK8MKzVSAFAmg3ZksACgkQ/5FK8MKz VSDhMBAAlbrW+VbE5/bL0oGpBJWlJgNoUHB2EntD07NVaU8puUleMV/76Uw5ltFB 6oK54E9YwSowl6XhYeu0GLoVEDbwsxp9vbj3RwndSNks9EkWZBpWOKbH+IhHAX3t YV5JdVFeABMEsVsIUbP8INbrkcGox6OwRbyy2e4LvRJ+h444hHPD+pvwluXgVhTH 0OGF8Qy5PTYusGyAlBwmwfjE/ZpxMAn9+32PEH7R9tQ4czASZ2tQdt3RB0N/Eve+ sS7TdqSgeGQuPBqAPE0XpRNyqGdHeVPwATUIMhOBV1Xio6KYrCaVocPolvTxz40I UUpwEi3CFQCV4b/kPM6iVJQlAsC1qJyaJr07g7fxEXwjjZkG3wxpkm0xKnViq1dt cErruBPDcbnBsM42S6X3DA9tfIJzywWpJb2mtpgUK8zAsV1ZjSqE15JEE3fUI1WZ NGgEUlNIgRRmp3N6QR2kqeByxIN613V2qzDCUHYk0kSWd3H4VKS43HbFxXbfVaMg 7fCRJqKr1NKBHUdI5CDx0zrQ0pjuuuwAx8KOOYhpisB24dvTYrhTzFu7GRaY8BPt pjLpn3y8gxek74NawhlIgzYf9roWSBktWXxbuEiTA4weOfL/rPvSGcph5MUH4wHK 1mMF7jQtYGKBHSrz8hZ/1cvwE3U/3XDdUWrJMBbplaJvlJpG+fc=
=jZLA
-----END PGP SIGNATURE-----

--- SoupGate-Win32 v1.05
*

XPost: linux.debian.bugs.dist, linux.debian.maint.boot
From: carnil@debian.org

Hi Cyril,

On Wed, May 28, 2025 at 09:38:51PM +0200, Cyril Brulebois wrote:

Hi,

Looping in debian-kernel@, context being missing touchpad support on an
amd64 machine, unless an extra module is added to the installer.

Feel free to steal the bug report away from us and adjust debian/installer/modules as you see fit!

Roland Clobus <rclobus@rclobus.nl> (2025-05-28):

On 27/05/2025 19:20, Pascal Hambourg wrote:

It looks like #1104057. Does lsmod show a pinctrl* module on a
system with working touchpad ?

I've built a custom installer and `pinctrl-meteorlake` does the trick.
With this additional kernel module the touchpad works in the graphical installer.

Checking linux.git, it seems pinctrl modules are only shipped in udebs
for a single arch:

kibi@tokyo:~/debian-kernel/linux.git (debian/6.12/trixie <)$ git grep pinctrl -- debian/installer/
debian/installer/modules/arm64/kernel-image:drivers/pinctrl/**

I'm not sure how many pinctrl modules would make sense to ship, or in
which udeb, for other architectures.

Looking at the amd64 config, the modular ones are:

CONFIG_PINCTRL_ALDERLAKE=m
CONFIG_PINCTRL_CS42L43=m
CONFIG_PINCTRL_ELKHARTLAKE=m
CONFIG_PINCTRL_EMMITSBURG=m
CONFIG_PINCTRL_INTEL_PLATFORM=m
CONFIG_PINCTRL_JASPERLAKE=m
CONFIG_PINCTRL_LAKEFIELD=m
CONFIG_PINCTRL_METEORLAKE=m
CONFIG_PINCTRL_METEORPOINT=m

while the built-in ones are:

CONFIG_PINCTRL_AMD=y
CONFIG_PINCTRL_BAYTRAIL=y
CONFIG_PINCTRL_CHERRYVIEW=y
CONFIG_PINCTRL_INTEL=y
CONFIG_PINCTRL_BROXTON=y
CONFIG_PINCTRL_CANNONLAKE=y
CONFIG_PINCTRL_CEDARFORK=y
CONFIG_PINCTRL_DENVERTON=y
CONFIG_PINCTRL_GEMINILAKE=y
CONFIG_PINCTRL_ICELAKE=y
CONFIG_PINCTRL_LEWISBURG=y
CONFIG_PINCTRL_SUNRISEPOINT=y
CONFIG_PINCTRL_TIGERLAKE=y

Can you double-check if https://salsa.debian.org/kernel-team/linux/-/merge_requests/1530 make
sense to you?

If yes, feel free to reassign it to src:linux. And then we should mark
it as well as to be backported for 6.12.y for trixie.

Regards,
Salvatore

--- SoupGate-Win32 v1.05
* Origin: you cannot sedate... all the things you hate (1:229/2)

XPost: linux.debian.bugs.dist, linux.debian.maint.boot
From: kibi@debian.org

Hi,

Roland Clobus <rclobus@rclobus.nl> (2025-05-29):

Looking at this MR, I'm wondering where the required kernel modules
for d-i should be listed.

The list in `debian/installer/modules/amd64/kernel-image`
(kernel-team/linux) is rather short.
Recently we discussed adding i915 and a few of its dependencies, which are not listed here, but somewhere else [1].

The kernel-image udeb is used everywhere. i915, *if* it is to be added somewhere, does *not* belong there. See other files in that directory, fb-modules might make sense, speakup-modules might as well.

I'd prefer not touching the kernel udebs at the moment (for the i915
thing, which really is orthogonal to this bug report and this MR). We can add/remove things autonomously through the debian-installer source
package, and I really don't want to have through linux source uploads to
adjust stuff, especially given the amount of blur currently surrounding firmware-sof-signed vs. firmware-intel-sound…

If yes, feel free to reassign it to src:linux. And then we should mark
it as well as to be backported for 6.12.y for trixie.

That would look good to me (at least given the current state of things on
the arm64 side). If pinctrl were only about touchpads and stuff, one
could argue they only make sense for the graphical installer, but I
suppose this could control other internal stuff, so adding the whole
series of pinctrl modules, in kernel-image, might be the easiest.


Cheers,
--
Cyril Brulebois (kibi@debian.org) <https://debamax.com/>
D-I release manager -- Release team member -- Freelance Consultant

-----BEGIN PGP SIGNATURE-----

iQIzBAABCgAdFiEEtg6/KYRFPHDXTPR4/5FK8MKzVSAFAmg4Zb8ACgkQ/5FK8MKz VSB5MxAAu+/ekVRpKtH6DVa5LTR82Fu/O1kkcVS4m3gpfeHq+knoMnYkasLu7dfH nLFnDzoHLMJcMeVIGOqRNAC9MwVQbWkEptBpgmKCxmmo1uXyxbVWKDPhozV0sDsa RCj3zlKC/LmENDpZhe9K//hqL1mob5NPtq5jrzbpfxqd88O54Ljcn7RSQOu4JxUV KBIg8pRroQpZcPI+YbKU+7NjO2b2e+onsz/9CeA00QNGmAMl8GcWVaI0IBAhIxv+ Bd4K6g4ziM2OCvcyrtCk7p4ysySYrgjjfMCwVbqhWf/3CLyGEapCU+eRWz2ibCDa oEGQkG98cXO4b8JE6bdMl9pYt7orfQ04HwOjg0awXHob7ukGPfZwOabLsP7G7qBy e9TyPTOISrl+ntfWwsEjfwXHLIJV7L7zFKOM+JyaC9zZRVIncPMbM7GTUFm+O0YP rxtkAznGGZzGf51WbzACQPz6K7S+MwAf8SXydUFhNKetdLjAbzz8L77/z0uYdSYN weXe0G+nB7CBqDdb0WIkxw62PRZBuU9kfPduLWpV6yigbzgAn+FJQmFamlsENbak DbkI9hmdskFvE0BKXxi5CQtFGLHpgTZAwofrxlCekA+i9z+n1iAgcU/mPpvJbYAW PQfptPDv6P+jJwXV+v8z0Z2+mkj7Uht4wU7qyJGvfTMW59m9pbE=
=KPiM
-----END PGP SIGNATURE-----

--- SoupGate-Win32 v1.05
*

XPost: linux.debian.bugs.dist, linux.debian.maint.boot
From: pascal@plouf.fr.eu.org

On 29/05/2025 at 09:43, Salvatore Bonaccorso wrote:

On Wed, May 28, 2025 at 09:38:51PM +0200, Cyril Brulebois wrote:

I'm not sure how many pinctrl modules would make sense to ship, or in
which udeb, for other architectures.

Looking at the amd64 config, the modular ones are:

CONFIG_PINCTRL_ALDERLAKE=m
CONFIG_PINCTRL_CS42L43=m

(...)> Can you double-check if

https://salsa.debian.org/kernel-team/linux/-/merge_requests/1530 make
sense to you?

The MR adds pinctrl/intel/* but what about
pinctrl/cirrus/pinctrl-cs42l43.ko.xz which is enabled by CONFIG_PINCTRL_CS42L43=m and was mentioned by Cyril in the duplicate bug #1104057 [1] ?

[1] <https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1104057#65>

--- SoupGate-Win32 v1.05
* Origin: you cannot sedate... all the things you hate (1:229/2)