linux-toradex.git/drivers/extcon/extcon-max8997.c, branch v3.14.45

extcon: max8997: Fix NULL pointer exception on missing pdata

2014-07-01T03:11:52+00:00

commit dfee4111febf3d9ef3a640b2cd6205c75f4e7e3d upstream.

Fix NULL pointer exception when platform data is not supplied. The
driver dereferenced pdata pointer where it could be NULL.

Signed-off-by: Krzysztof Kozlowski 
Fixes: 810d601f07c
Signed-off-by: Chanwoo Choi 
Signed-off-by: Greg Kroah-Hartman

Merge tag 'extcon-next-for-3.13' of git://git.kernel.org/pub/scm/linux/kernel/git/chanwoo/extcon into char-misc-next

2013-09-27T03:47:25+00:00

Chanwoo writes:

Update extcon for 3.13

This patchset modify extcon core to remove unnecessary allocation sequence for
'dev' instance and change extcon_dev_register() interface. extcon-gpio use
gpiolib API to get debounce time and include small fix of extcon core/device
driver.

Detailed description for patchset:
1. Modify extcon core driver
- The extcon-gpio driver use gpio_set_debounce() API provided from gpiolib
if gpio driver for SoC support gpio_set_debounce() function and support 'gpio_
activ_low' filed to check whether gpio active state is 1(high) or 0(low).
- Change field type of 'dev' in structure extcon_dev and remove the sequence
of allocating memory of 'struct dev' on extcon_dev_register() function because
extcon device must need 'struct device.
- Change extcon_dev_register() prototype to simplify it and remove unnecessary
parameter as below:

2. Fix coding style and typo
- extcon core     : Fix indentation coding style and remove unnecessary casting
- extcon-max8997  : Fix checkpatch warning
- extcon-max77693 : Fix checkpatch warning
- extcon-arizona  : Fix typo of comment and modify minor issue
- extcon-palmas   : Use dev_get_platdata()

3. Modify extcon-arizona driver
- Modify minor issue about micbias and comparision statement

extcon: Simplify extcon_dev_register() prototype by removing unnecessary parameter

2013-09-27T00:37:01+00:00

This patch remove extcon_dev_register()'s second parameter which means
the pointer of parent device to simplify prototype of this function.
So, if extcon device has the parent device, it should set the pointer of
parent device to edev.dev.parent in extcon device driver instead of in
extcon_dev_register().

Cc: Graeme Gregory 
Cc: Kishon Vijay Abraham I 
Cc: Charles Keepax 
Cc: Mark Brown 
Signed-off-by: Chanwoo Choi 
Signed-off-by: Myungjoo Ham

extcon: max8997: Fix checkpatch warning

2013-09-27T00:36:59+00:00

Fixes the following warning:
WARNING: space prohibited before semicolon

Signed-off-by: Sachin Kamat 
Signed-off-by: Chanwoo Choi

extcon: Fix up 80 column coding style issues

2013-09-26T19:03:30+00:00

This patch fix 80 column coding sytle issues by using checkpatch script.

Cc: Charles Keepax 
Cc: Mark Brown 
Signed-off-by: Chanwoo Choi 
Signed-off-by: Myungjoo Ham 
Signed-off-by: Greg Kroah-Hartman

extcon: max8997: Fix return value

2013-04-08T22:53:46+00:00

Return the value obtained from the function instead of hardcoding.
Fixes the following warnings:
drivers/extcon/extcon-max8997.c:235 max8997_muic_set_path() info:
why not propagate 'ret' from max8997_update_reg() instead of (-11)?
drivers/extcon/extcon-max8997.c:248 max8997_muic_set_path() info:
why not propagate 'ret' from max8997_update_reg() instead of (-11)?

Signed-off-by: Sachin Kamat 
Signed-off-by: Chanwoo Choi 
Signed-off-by: Myungjoo Ham

extcon: max8997: use dev_err() instead of pr_err()

2013-04-08T22:53:46+00:00

dev_err() is more preferred than pr_err().

Signed-off-by: Jingoo Han 
Signed-off-by: Chanwoo Choi 
Signed-off-by: Myungjoo Ham

extcon: max8997: Check the pointer of platform data to protect null pointer error

2013-03-13T08:28:15+00:00

This patch check the pointer of platform data to protect
kernel panic when platform data is not used and code clean.

Signed-off-by: Chanwoo Choi 
Signed-off-by: Myungjoo Ham

extcon: max8997: Use workqueue to check cable state after completing boot of platform

2013-02-13T22:54:49+00:00

This patch use delayed workqueue to check cable state after a certain
time. If extcon-max8997 driver check cable state during booting of
platform, this couldn't send the correct notification of cable state
to extcon consumer. Alwasys, this driver should check cable state
after the completion of platform initialization

Signed-off-by: Chanwoo Choi 
Signed-off-by: Myungjoo Ham

extcon: max8997: Set default UART/USB path on probe

2013-02-13T22:54:36+00:00

This patch set default H/W line path according to platfomr data.
The MAX8997 MUIC device can possibly set UART/USB or UART_AUX
/USB_AUX to internal H/W line path of MUIC device. Namely, only
one H/W line is used for two operation.

For example,
if H/W line path of MAX8997 device set UART/USB, micro usb cable
is connected to AP(Application Processor) and if H/W line path
set UART_AUX/USB_AUX, micro usb cable is connected to CP(Coprocessor).

Signed-off-by: Chanwoo Choi 
Signed-off-by: Myungjoo Ham