[lkml]   [2016]   [Feb]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
Subject[PATCH V6 0/6] rtc: max77686: make max77686 rtc driver as IP driver
Based on discussion on patch series of MAX77620 when adding separate
driver for max77620 RTC, it is discussed to reuse the max77686 driver
for all CHips MAX77802, MAX77686 and MAX77620. For this, the rtc-max77686
need to make as IP driver independent of their MFD parent driver.

This series makes the rtc-max77686 as independent driver from its
parent. Required information is passed through the device parent
which is generic and does not depends on any max77686 specific
header ifnromation.

CC: Krzysztof Kozlowski <>
CC: Javier Martinez Canillas <>

Changes from V1:
- Added reviewed/tested by tag which we got from V1.
- Remove changes from Kconfig.
- Maintain all register definition in max77686 private header and remove
the movement to rtc driver.
- Taken care of all comments on V1 from Krzysztof and Javier.

Changes from V2:
- Fix the issue of crash in unbind.

Changes from V3:
- Fix suspend-resume after unbind/bind by unmapping virq in remove callback.

Changes from V5:
- Abandon the addition of new API in regmap-irq and add dispose of virq
in regmap_del_irq_chip().

Laxman Dewangan (6):
regmap: irq: dispose all virtual irq before removing domain
rtc: max77686: fix checkpatch error
rtc: max77686: use rtc regmap to access RTC registers
rtc: max77686: avoid reference of parent device info multiple places
mfd: max77686: do not set i2c client data for rtc i2c client
rtc: max77686: move initialisation of rtc regmap, irq chip locally

drivers/base/regmap/regmap-irq.c | 21 ++++
drivers/mfd/max77686.c | 86 +-------------
drivers/rtc/rtc-max77686.c | 214 ++++++++++++++++++++++++++---------
include/linux/mfd/max77686-private.h | 3 -
4 files changed, 182 insertions(+), 142 deletions(-)


 \ /
  Last update: 2016-02-09 14:01    [W:0.093 / U:2.584 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site