Re: [PATCH v2 0/6] Move dell-led to drivers/platform/x86

[Alex Hung]

On Wed, Feb 15, 2017 at 9:54 PM, MichaÅ KÄpieÅ <kernel@xxxxxxxxxx> wrote:
> Alex,
>
>> I tested the patches with the following setup. Please note I tested
>> the microphone mute led by GUI, as the hotkey does not work on this
>> system.
>
> Thank you for taking the time to test this series, it is appreciated.  I
> would not expect the switching method (GUI/hotkey) to matter here, as
> long as we can compare behavior of two kernel versions by consistently
> using one of them.
>
>>
>> 1. Downloaded the below patches from patchwork
>>
>> 9518737 Awaiting Upstream [v2,1/6] dell-led: remove GUID check from
>> dell_micmute_led_set()
>> 9518761 Awaiting Upstream [v2,2/6] ALSA: hda - use
>> dell_micmute_led_set() instead of dell_app_wmi_led_set()
>> 9518741 Awaiting Upstream [v2,3/6] ALSA: hda - rename
>> dell_led_set_func to dell_micmute_led_set_func
>> 9518725 Awaiting Upstream [v2,4/6] platform/x86: dell-laptop: import
>> dell_micmute_led_set() from drivers/leds/dell-led.c
>> 9518727 Awaiting Upstream [v2,5/6] dell-led: remove code related to mic mute LED
>> 9518735 Awaiting Upstream [v2,6/6] dell-led: move driver to
>> drivers/platform/x86/dell-wmi-led.c
>> 9520053 Awaiting Upstream [v2,6+/6] platform/x86: dell-wmi-led: fix
>> coding style issues
>>
>> 2. Installed Ubuntu 16.10 (Linux kernel 4.8) on Dell Latitude 7180
>
> Just to be sure, I assume you meant 7280?

This is a typo, and it should be 7480, which should be the same as 7280.

>
>>
>>   -> Microphone mute led works as expected
>>
>> 3. Applied downloaded patches to kernel 4.8 without any conflicts, but
>> kernel will not compile successfully.
>
> I am confused.  This patch series should apply cleanly to 4.9 and its
> surroundings.  Between 4.8 and 4.9, commit 4875a5f72180 ("ALSA: hda -
> Fix a failure of micmute led when having multi adcs") happened.   This
> makes it impossible to apply this patch series cleanly on top of 4.8
> without reducing context to one line.
>
> Moreover, I applied this patch series (with reduced context) to 4.8 and
> successfully compiled the kernel.  Could you shed some light on where
> exactly did your build crash?

The error messages such as below, so it is probably not "fails to
compile". Since 4.9 works better, I probably will focus on 4.9 & 4.10,
though.

find /home/alexhung/src/kernel/ubuntu-yakkety/debian/build/build-generic/
-name \*.ko | \
    sed -e 's/.*\/\([^\/]*\)\.ko/\1/' | sort >
/home/alexhung/src/kernel/ubuntu-yakkety/debian.master/abi/4.8.0-37.39/amd64/generic.modules
II: Checking modules for generic...
   reading new modules...read 4821 modules.
   reading old modules...
      MISS: dell-led
      NEW : dell-wmi-led
      read 4821 modules : new(1)  missing(1)
EE: Missing modules (start begging for mercy)
debian/rules.d/4-checks.mk:12: recipe for target 'module-check-generic' failed
make: *** [module-check-generic] Error 1


>
>>
>> 4. Installed Ubuntu Zesty kernel (Linux kernel 4.9)
>>
>>   -> Microphone mute led works as expected
>>
>> 5. Applied and compiled downloaded patches to kernel 4.9
>>
>>   -> Microphone mute led does not work
>
> Any chance of a git bisect?

Certainly. Give me some time.

>
>>
>> 6. Compiled and installed Linux kernel 4.10 rc8
>>
>>   -> Microphone mute led does not work
>
> This is even more confusing.  Between 4.9, which you claim works fine,
> and 4.10-rc8, which you claim does not work, I can see no changes to
> either drivers/leds/dell-led.c or sound/pci/hda/dell_wmi_helper.c.
> Kernel configuration issue?

I was surprised too, and I compiled it twice and installed on both
Ubuntu 16.04 and 16.10.

I would guess it is config too. On 4.9, it was using Ubuntu default.
On 4.10rc8, I am pretty much using the instruction on
https://wiki.ubuntu.com/KernelTeam/GitKernelBuild. The .config is
available @ http://paste.ubuntu.com/24001113/


>
>>
>> 7. Applied and compiled downloaded patches to kernel 4.10 rc8
>>
>>   -> Microphone mute led does not work
>
> No surprises here if it really does not work with 4.10-rc8.
>
> --
> Best regards,
> MichaÅ KÄpieÅ



-- 
Cheers,
Alex Hung