Bug#1002460: linux-image-amd64: Missing kernel configuration for Microsoft Laptops and Surface devices

2021-12-23 Thread Salvatore Bonaccorso
Hi,

On Thu, Dec 23, 2021 at 02:37:50PM +0100, Maximilian Luz wrote:
> On Wed, 22 Dec 2021 23:47:10 +0100 Vincent Blut  
> wrote:
> > Le 2021-12-22 14:07, Matthias Brennwald a écrit :
> > > Full details (including a suggested fix) are available here:
> > > https://github.com/linux-surface/linux-surface/issues/683
> > 
> > I need a clarification about CONFIG_SURFACE_AGGREGATOR_CDEV. This
> > options' help text mentions that it is intended for debugging and
> > development only, and should not be used otherwise. Is it really needed?
> 
> It's not needed for normal support, no. It's quite useful for debugging
> and adding support for new devices, though.
> 
> You can compare it a bit with USB Raw Gadget or HIDRAW (except that we
> don't want anyone writing user-space drivers against that interface),
> i.e. it provides a direct channel from user-space to the EC. Setting
> this option to M provides a module that needs to be loaded in explicitly
> when someone wants to use it.
> 
> I've added it to the list because other distros (Ubuntu, Arch) have
> enabled it. Feel free to keep it disabled.

Thank you. I have merged Vincent's MR (keeping for now the other
discussion option disabled), and it will be in the next 5.16.y upload.

Regards,
Salvatore



Bug#1002460: linux-image-amd64: Missing kernel configuration for Microsoft Laptops and Surface devices

2021-12-23 Thread Maximilian Luz

On Wed, 22 Dec 2021 23:47:10 +0100 Vincent Blut  wrote:

Le 2021-12-22 14:07, Matthias Brennwald a écrit :
> Full details (including a suggested fix) are available here:
> https://github.com/linux-surface/linux-surface/issues/683

I need a clarification about CONFIG_SURFACE_AGGREGATOR_CDEV. This options' 
help text mentions that it is intended for debugging and development only, 
and should not be used otherwise. Is it really needed?


It's not needed for normal support, no. It's quite useful for debugging
and adding support for new devices, though.

You can compare it a bit with USB Raw Gadget or HIDRAW (except that we
don't want anyone writing user-space drivers against that interface),
i.e. it provides a direct channel from user-space to the EC. Setting
this option to M provides a module that needs to be loaded in explicitly
when someone wants to use it.

I've added it to the list because other distros (Ubuntu, Arch) have
enabled it. Feel free to keep it disabled.

Regards,
Max



Bug#1002460: linux-image-amd64: Missing kernel configuration for Microsoft Laptops and Surface devices

2021-12-22 Thread Vincent Blut
Le 2021-12-22 14:07, Matthias Brennwald a écrit :
> Full details (including a suggested fix) are available here:
> https://github.com/linux-surface/linux-surface/issues/683

I need a clarification about CONFIG_SURFACE_AGGREGATOR_CDEV. This options' 
help text mentions that it is intended for debugging and development only, 
and should not be used otherwise. Is it really needed?

Cheers,
Vincent


signature.asc
Description: PGP signature


Processed: Re: Bug#1002460: linux-image-amd64: Missing kernel configuration for Microsoft Laptops and Surface devices

2021-12-22 Thread Debian Bug Tracking System
Processing control commands:

> reassign -1 src:linux
Bug #1002460 [src:linux] linux-image-amd64: Missing kernel configuration for 
Microsoft Laptops and Surface devices
Ignoring request to reassign bug #1002460 to the same package

-- 
1002460: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1002460
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems



Bug#1002460: linux-image-amd64: Missing kernel configuration for Microsoft Laptops and Surface devices

2021-12-22 Thread Vincent Blut
Control: reassign -1 src:linux

Hi Matthias,

Le 2021-12-22 14:07, Matthias Brennwald a écrit :
> Package: linux-image-amd64
> Version: 5.15.5-1
> Severity: normal
> X-Debbugs-Cc: mbren...@gmail.com
> 
> Dear Maintainer,
> 
> Some features for Linux on Microsoft Laptops and Surface devices that were
> implemented in the 5.13+ Linux kernel are not configured with the kernels
> provided by Debian.
> 
> Full details (including a suggested fix) are available here:
> https://github.com/linux-surface/linux-surface/issues/683

I'll work on this later today.

Cheers,
Vincent


signature.asc
Description: PGP signature


Processed: Re: Bug#1002460: linux-image-amd64: Missing kernel configuration for Microsoft Laptops and Surface devices

2021-12-22 Thread Debian Bug Tracking System
Processing control commands:

> reassign -1 src:linux
Bug #1002460 [linux-image-amd64] linux-image-amd64: Missing kernel 
configuration for Microsoft Laptops and Surface devices
Bug reassigned from package 'linux-image-amd64' to 'src:linux'.
No longer marked as found in versions linux-signed-amd64/5.15.5+1.
Ignoring request to alter fixed versions of bug #1002460 to the same values 
previously set

-- 
1002460: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1002460
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems



Bug#1002460: linux-image-amd64: Missing kernel configuration for Microsoft Laptops and Surface devices

2021-12-22 Thread Matthias Brennwald
Package: linux-image-amd64
Version: 5.15.5-1
Severity: normal
X-Debbugs-Cc: mbren...@gmail.com

Dear Maintainer,

Some features for Linux on Microsoft Laptops and Surface devices that were
implemented in the 5.13+ Linux kernel are not configured with the kernels
provided by Debian.

Full details (including a suggested fix) are available here:
https://github.com/linux-surface/linux-surface/issues/683


-- System Information:
Debian Release: bookworm/sid
  APT prefers unstable
  APT policy: (500, 'unstable'), (500, 'stable'), (1, 'experimental')
Architecture: amd64 (x86_64)

Kernel: Linux 5.15.0-2-amd64 (SMP w/8 CPU threads)
Kernel taint flags: TAINT_USER, TAINT_OOT_MODULE
Locale: LANG=en_CH.UTF-8, LC_CTYPE=en_CH.UTF-8 (charmap=UTF-8), LANGUAGE not set
Shell: /bin/sh linked to /usr/bin/dash
Init: systemd (via /run/systemd/system)
LSM: AppArmor: enabled

Versions of packages linux-image-amd64 depends on:
ii  linux-image-5.15.0-2-amd64  5.15.5-1

linux-image-amd64 recommends no packages.

linux-image-amd64 suggests no packages.

-- no debconf information