extcon: int3496: process id-pin first so that we start with the right status
authorHans de Goede <hdegoede@redhat.com>
Tue, 13 Feb 2018 19:25:50 +0000 (20:25 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 28 Feb 2018 09:19:39 +0000 (10:19 +0100)
commit606f74e3d01b39ab2b602102d2c56a1ef9c9b3b1
treeeb8ee178de46a063b82adeb0b57a001907b228ec
parent29e76b211e9231e1d565551de84d9abd6eff5658
extcon: int3496: process id-pin first so that we start with the right status

commit 0434352d3d2e950cf5e743f6062abd87de22f960 upstream.

Some other drivers may be waiting for our extcon to show-up, exiting their
probe methods with -EPROBE_DEFER until we show up.

These drivers will typically get the cable state directly after getting
the extcon, this commit changes the int3496 code to wait for the initial
processing of the id-pin to complete before exiting probe() with 0, which
will cause devices waiting on the defered probe to get reprobed.

This fixes a race where the initial work might still be running while other
drivers were already calling extcon_get_state().

Fixes: 2f556bdb9f2e ("extcon: int3496: Add Intel INT3496 ACPI ... driver")
Cc: stable@vger.kernel.org
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/extcon/extcon-intel-int3496.c