RE: [PATCH] drivers/power/sbs-battery.c: don't trigger falsesupply_changed event

From: Rhyland Klein
Date: Wed May 09 2012 - 10:51:12 EST


On Wed, 2012-05-09 at 2:31, Nikolaus Voss wrote:
>power_supply_changed() events are triggerd based on the return value
>of a get_property() call. However the property TECHNOLOGY is
>hard-coded to LION in this driver, thus always succeeds.
>
>So, with the battery removed, this triggers a false battery present
>uevent. This uevent triggers a new query via power_supply_uevent()
>which again starts to query all known properties and thus leads to
>an infinite loop of battery present/not-present uevents.
>
>This patch skips the battery presence detection for the hard-coded
>property TECHNOLOGY.
>
>Signed-off-by: Nikolaus Voss <n.voss@xxxxxxxxxxx>

Acked-by Rhyland Klein <rklein@xxxxxxxxxx>

--
nvpublic
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/