linux-toradex.git/drivers/extcon, branch v4.9.96

extcon: palmas: Check the parent instance to prevent the NULL

2017-11-21T08:23:21+00:00

[ Upstream commit 9fe172b9be532acc23e35ba693700383ab775e66 ]

extcon-palmas must be child of palmas and expects parent's
drvdata to be valid. Check for non NULL parent drvdata and
fail if it is NULL. Not doing so will result in a NULL
pointer dereference later in the probe() parent drvdata
is NULL (e.g. misplaced extcon-palmas node in device tree).

Signed-off-by: Roger Quadros 
Signed-off-by: Chanwoo Choi 
Signed-off-by: Sasha Levin 
Signed-off-by: Greg Kroah-Hartman

extcon: Remove potential problem when calling extcon_register_notifier()

2017-11-21T08:23:21+00:00

[ Upstream commit 01b4c9a1ae07a25d208cad0da7dd288007a22984 ]

This patch removes the potential problem of extcon_register_notifier()
when edev parameter is NULL. When edev is NULL, this function returns
the first extcon device which includes the sepecific external connector
of second paramter. But, it don't guarantee the same operation in all cases.
To remove this confusion and potential problem, this patch fixes it.

Signed-off-by: Chanwoo Choi 
Signed-off-by: Sasha Levin 
Signed-off-by: Greg Kroah-Hartman

extcon: axp288: Use vbus-valid instead of -present to determine cable presence

2017-10-08T08:26:03+00:00

[ Upstream commit 5757aca10146061befd168dab37fb0db1ccd8f73 ]

The vbus-present bit in the power status register also gets set to 1
when a usb-host cable (id-pin shorted to ground) is plugged in and a 5v
boost converter is supplying 5v to the otg usb bus.

This causes a "disconnect or unknown or ID event" warning in dmesg as
well as the extcon device to report the last detected charger cable
type as being connected even though none is connected.

This commit switches to checking the vbus-valid bit instead, which is
only 1 when both vbus is present and the vbus-path is enabled in the
vbus-path control register (the vbus-path gets disabled when a usb-host
cable is detected, to avoid the pmic drawing power from the 5v boost
converter).

Signed-off-by: Hans de Goede 
Acked-by: Chanwoo Choi 
Signed-off-by: Chanwoo Choi 
Signed-off-by: Sasha Levin 
Signed-off-by: Greg Kroah-Hartman

extcon: return error code on failure

2017-01-19T19:18:03+00:00

commit 5b11ebedd6a8bb4271b796e498cd15c0fe1133b6 upstream.

Function get_zeroed_page() returns a NULL pointer if there is no enough
memory. In function extcon_sync(), it returns 0 if the call to
get_zeroed_page() fails. The return value 0 indicates success in the
context, which is incosistent with the execution status. This patch
fixes the bug by returning -ENOMEM.

Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=188611

Signed-off-by: Pan Bian 
Fixes: a580982f0836e
Acked-by: Chanwoo Choi 
Signed-off-by: Greg Kroah-Hartman

extcon: qcom-spmi-misc: Sync the extcon state on interrupt

2016-10-26T07:04:29+00:00

The driver was changed after submission to use the new style APIs
like extcon_set_state(). Unfortunately, that only sets the state,
and doesn't notify any consumers that the cable state has
changed. Use extcon_set_state_sync() here instead so that we
notify cable consumers of the state change. This fixes USB
host-device role switching on the db8074 platform.

Fixes: 38085c987f52 ("extcon: Add support for qcom SPMI PMIC USB id detection hardware")
Signed-off-by: Stephen Boyd 
Signed-off-by: Chanwoo Choi

Merge branch 'next' into resolution

2016-09-15T11:15:20+00:00

Conflicts:
	drivers/extcon/extcon-adc-jack.c
	drivers/extcon/extcon-arizona.c
	drivers/extcon/extcon-gpio.c
	include/linux/extcon.h

extcon: Add support for qcom SPMI PMIC USB id detection hardware

2016-09-13T02:26:26+00:00

Some Qualcomm PMICs have a misc device that performs USB id pin
detection via an interrupt. When the interrupt triggers, we
should read the interrupt line to see if it has gone high or low.
If the interrupt is low then the ID pin is grounded, and if the
interrupt is high then the ID pin is being held high.

Cc: Roger Quadros 
Signed-off-by: Stephen Boyd 
[cw00.choi: Edited the driver description and added the author information]
Signed-off-by: Chanwoo Choi

extcon: Use the extcon_set_state_sync() instead of deprecated functions

2016-09-13T02:26:26+00:00

This patch alters the renamed extcon API to set the state of the external
connectors instead of deprecated extcon_set_cable_state_().

Because the patch[1] modifies the function name to maintain the function
naming pattern.
- extcon_set_cable_state_() -> extcon_set_state_sync()
- extcon_get_cable_state_() -> extcon_get_state()

[1] https://lkml.org/lkml/2016/8/4/729
- extcon: Rename the extcon_set/get_state() to maintain the function naming pattern

Signed-off-by: Chanwoo Choi 
Acked-by: Charles Keepax 
Acked-by: Roger Quadros

extcon: axp288: Fix spelling mistake

2016-09-13T02:26:26+00:00

This patch fixes the spelling mistake in dev_dbg messages.

Signed-off-by: Colin Ian King 
[cw00.choi: Modify the patch title/description]
Signed-off-by: Chanwoo Choi

extcon: max14577: Change Krzysztof Kozlowski's email to kernel.org

2016-09-13T02:26:26+00:00

Change my email address to kernel.org instead of Samsung one for the
purpose of any future contact.  The copyrights remain untouched and are
attributed to Samsung.

Signed-off-by: Krzysztof Kozlowski 
Signed-off-by: Chanwoo Choi