[PATCH v4 1/2] platform/x86: dell-privacy: Add support for Dell hardware privacy
Pierre-Louis Bossart
pierre-louis.bossart at linux.intel.com
Mon Mar 8 16:59:15 CET 2021
>>> module_param(force_rfkill, bool, 0444);
>>> MODULE_PARM_DESC(force_rfkill, "enable rfkill on non whitelisted
>>> models");
>>>
>>> +
>>
>> spurious line change
> I just want to make them separate with more space .
> If it cause concern, I will remote the line in V5.
it's fine to improve spaces/alignment, just do it in a separate patch.
>
>>
>>> static const struct dmi_system_id dell_device_table[] __initconst = {
>>> {
>>> .ident = "Dell laptop",
>>> @@ -2205,11 +2209,17 @@ static int __init dell_init(void)
>>> dell_laptop_register_notifier(&dell_laptop_notifier);
>>>
>>> if (dell_smbios_find_token(GLOBAL_MIC_MUTE_DISABLE) &&
>>> - dell_smbios_find_token(GLOBAL_MIC_MUTE_ENABLE)) {
>>> - micmute_led_cdev.brightness =
>> ledtrig_audio_get(LED_AUDIO_MICMUTE);
>>> - ret = led_classdev_register(&platform_device->dev,
>> &micmute_led_cdev);
>>> - if (ret < 0)
>>> - goto fail_led;
>>> +
>> dell_smbios_find_token(GLOBAL_MIC_MUTE_ENABLE)) {
>>
>> not sure why you changed the alignment?
> The previous alignment is a little not correct.
> So I adjust it
> If it cause concern, will restore it to original shape.
same here, use a different patch.
More information about the Alsa-devel
mailing list