linux-stable.git/drivers/extcon, branch v5.10.35

extcon: Fix error handling in extcon_dev_register

2021-04-07T13:00:11+00:00

[ Upstream commit d3bdd1c3140724967ca4136755538fa7c05c2b4e ]

When devm_kcalloc() fails, we should execute device_unregister()
to unregister edev->dev from system.

Fixes: 046050f6e623e ("extcon: Update the prototype of extcon_register_notifier() with enum extcon")
Signed-off-by: Dinghao Liu 
Signed-off-by: Chanwoo Choi 
Signed-off-by: Sasha Levin

extcon: max77693: Fix modalias string

2020-12-30T10:53:49+00:00

[ Upstream commit e1efdb604f5c9903a5d92ef42244009d3c04880f ]

The platform device driver name is "max77693-muic", so advertise it
properly in the modalias string. This fixes automated module loading when
this driver is compiled as a module.

Fixes: db1b9037424b ("extcon: MAX77693: Add extcon-max77693 driver to support Maxim MAX77693 MUIC device")
Signed-off-by: Marek Szyprowski 
Signed-off-by: Chanwoo Choi 
Signed-off-by: Sasha Levin

extcon: axp288: Use module_platform_driver to simplify the code

2020-09-29T15:40:06+00:00

module_platform_driver() makes the code simpler by eliminating
boilerplate code.

Signed-off-by: Liu Shixin 
Reviewed-by: Hans de Goede 
Signed-off-by: Chanwoo Choi

extcon: ptn5150: Do not print error during probe if nothing is attached

2020-09-29T15:38:54+00:00

The commit 85256f611f66 ("extcon: ptn5150: Check current USB mode when
probing") reused code for checking CC status register in the probe path
to determine what is initially connected.  However if nothing is
connected, the CC status register will have 0x0 value and print an error
message:

    ptn5150 1-003d: Unknown Port status : 0

This is not an error.  Also any other unknown port status values are not
really errors but unhandled cases.

Fixes: 85256f611f66 ("extcon: ptn5150: Check current USB mode when probing")
Signed-off-by: Krzysztof Kozlowski 
Signed-off-by: Chanwoo Choi

extcon: ptn5150: Use defines for registers

2020-09-24T10:20:49+00:00

The register addresses are not continuous, so use simple defines for
them.  This also makes it easier to find the address for register.

No functional change.

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

extcon: palmas: Simplify with dev_err_probe()

2020-09-24T10:20:49+00:00

Common pattern of handling deferred probe can be simplified with
dev_err_probe().  Less code and the error value gets printed.

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

extcon: max8997: Return error code of extcon_dev_allocate()

2020-09-24T10:20:49+00:00

devm_extcon_dev_allocate() can fail of multiple reasons.  The call
returns proper error code on failure so pass it instead of fixed ENOMEM.

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

extcon: max77843: Return error code of extcon_dev_allocate()

2020-09-24T10:20:49+00:00

devm_extcon_dev_allocate() can fail of multiple reasons.  The call
returns proper error code on failure so pass it instead of fixed ENOMEM.

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

extcon: max77693: Return error code of extcon_dev_allocate()

2020-09-24T10:20:49+00:00

devm_extcon_dev_allocate() can fail of multiple reasons.  The call
returns proper error code on failure so pass it instead of fixed ENOMEM.

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

extcon: max14577: Return error code of extcon_dev_allocate()

2020-09-24T10:20:49+00:00

devm_extcon_dev_allocate() can fail of multiple reasons.  The call
returns proper error code on failure so pass it instead of fixed ENOMEM.

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