2014-02-08 Alexander Mezin <mezin.alexander@xxxxxxxxx>:
> 2014-02-08 13:33 GMT+07:00 Lan Tianyu <lantianyu1986@xxxxxxxxx>:
>> 2014-02-08 Alexander Mezin <mezin.alexander@xxxxxxxxx>:
>>> Do the same thing as in acpi_ac_resume.
>>>
>>> BIOS on my laptop (HP Pavilion dv6-6179er) don't send proper
>>> notifications when AC adapter is plugged/unplugged.
>>> This change makes it possible to workaround the problem by using
>>> acpid and a shell script.
>>>
>>> This patch has the same effect as
>>> commit 50a2bc5429f07ec4d53df2d287b03bdbceb281bb
>>> "ACPI / AC: change notification handler type to ACPI_ALL_NOTIFY"
>>>
>>> Aforementioned commit could be reverted.
>>
>> Hi Alexander:
>> I don't think the previous commit can be reverted. AC driver
>> should get system notify events(E,G bus check event) and process them.
>> Some drivers(E,G radeon) also need to know AC event and AC driver's
>> notify handler sends the event to these drivers via block notifier call chain.
>> Further more, this patch only works when the related sysfs interfaces are
>> accessed. It's redundant with previous commit.
>
> OK, I didn't think about other drivers and so.
>
> Second patch works fine on its own (if old commit isn't reverted, of course),
> I've tested it just now. Should I resend it as single patch?
Hi:
What's the benefit of the second patch without this patch?
So far, bios sends the ACPI notify event of power change to AC driver and
the driver would change status and send uevent. That seems enough.
BTW, battery status update normally is also triggered by ACPI notify event.
--
Best regards
Tianyu Lan
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
