linux.git/drivers/mfd/max14577.c, branch v6.0

mfd: max14577: Do not enforce (incorrect) interrupt trigger type

2021-11-05T14:40:06+00:00

Interrupt line can be configured on different hardware in different way,
even inverted.  Therefore driver should not enforce specific trigger
type - edge falling - but instead rely on Devicetree to configure it.

The Maxim 14577/77836 datasheets describe the interrupt line as active
low with a requirement of acknowledge from the CPU therefore the edge
falling is not correct.

The interrupt line is shared between PMIC and charger driver, so using
level sensitive interrupt is here especially important to avoid races.
With an edge configuration in case if first PMIC signals interrupt
followed shortly after by the RTC, the interrupt might not be yet
cleared/acked thus the second one would not be noticed.

Signed-off-by: Krzysztof Kozlowski 
Acked-by: Rob Herring 
Signed-off-by: Lee Jones 
Link: https://lore.kernel.org/r/20210602110445.33536-4-krzysztof.kozlowski@canonical.com

mfd: max14577: Remove redundant initialization of variable current_bits

2020-08-13T06:50:04+00:00

The variable current_bits is being initialized with a value that is
never read and it is being updated later with a new value. The
initialization is redundant and can be removed.

Addresses-Coverity: ("Unused value")
Signed-off-by: Colin Ian King 
Reviewed-by: Krzysztof Kozlowski 
Signed-off-by: Lee Jones

mfd: max14577: Convert to i2c_new_dummy_device

2019-08-12T07:53:58+00:00

Move from i2c_new_dummy() to i2c_new_dummy_device(), so we now get an
ERRPTR which we use in error handling.

Signed-off-by: Wolfram Sang 
Reviewed-by: Krzysztof Kozlowski 
Signed-off-by: Lee Jones

mfd: maxim: Add SPDX license identifiers

2018-10-23T07:58:34+00:00

Replace GPL v2.0+ license statements with SPDX license identifiers.

Signed-off-by: Krzysztof Kozlowski 
Signed-off-by: Lee Jones

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

2016-10-04T14:48:02+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: Lee Jones

mfd: max14577: Use module_init() instead of subsys_initcall()

2016-06-29T09:14:43+00:00

The driver's init function is called at subsys init call level but the
dependencies provided by the driver are looked up by drivers that have
probe deferral support, so manual ordering of init calls isn't needed.

Suggested-by: Krzysztof Kozlowski 
Signed-off-by: Javier Martinez Canillas 
Signed-off-by: Lee Jones

mfd: Use to_i2c_client() instead of open-coding it

2016-01-14T08:43:50+00:00

Signed-off-by: Geliang Tang 
Signed-off-by: Lee Jones

mfd: Drop owner assignment from i2c_drivers

2015-08-11T14:08:48+00:00

i2c_driver does not need to set an owner because i2c_register_driver()
will set it.

Signed-off-by: Krzysztof Kozlowski 
Signed-off-by: Lee Jones

mfd: max14577: Fix obvious typo in company name in copyright

2014-11-25T16:19:24+00:00

Fix a typo in name of company in copyright comment.

Signed-off-by: Krzysztof Kozlowski 
Signed-off-by: Lee Jones

mfd: max14577: Don't pass IRQ domain to mfd_add_devices

2014-09-26T07:23:54+00:00

The max14577 MFD cells do not have any resources so the IRQ domain
passed to mfd_add_devices is not used.

Signed-off-by: Krzysztof Kozlowski 
Signed-off-by: Lee Jones