RTC: DA9055 RTC driver
Subsystems (2)
PATCH v3 3/7 RTC: DA9055 RTC driver
This is the RTC patch for the DA9055 PMIC. This patch has got dependency on the DA9055 MFD core.
This patch is functionally tested on Samsung SMDKV6410.
---
changes since version v3:
- use of module_platform_driver macro
- add the regmap virtual irq map API.
changes since version v2:
- Use of devm_request_threaded_irq API
--- drivers/rtc/Kconfig | 10 + drivers/rtc/Makefile | 1 + drivers/rtc/rtc-da9055.c | 413 ++++++++++++++++++++++++++++++++++++++++++++++ 3 files changed, 424 insertions(+), 0 deletions(-) mode change 100644 => 100755 drivers/rtc/Kconfig create mode 100644 drivers/rtc/rtc-da9055.c
drivers/rtc/Kconfig +10 -0
@@ -571,6 +571,16 @@ config RTC_DRV_DA9052 Say y here to support the RTC driver for Dialog Semiconductor DA9052-BC and DA9053-AA/Bx PMICs. +config RTC_DRV_DA9055 + tristate "Dialog Semiconductor DA9055 RTC" + depends on MFD_DA9055 + help + If you say yes here you will get support for the + RTC of the Dialog DA9055 PMIC. + + This driver can also be built as a module. If so, the module + will be called rtc-da9055 + config RTC_DRV_EFI tristate "EFI RTC" depends on IA64
drivers/rtc/Makefile +1 -0
@@ -29,6 +29,7 @@ obj-$(CONFIG_RTC_DRV_BQ4802) += rtc-bq4802.o obj-$(CONFIG_RTC_DRV_CMOS) += rtc-cmos.o obj-$(CONFIG_RTC_DRV_COH901331) += rtc-coh901331.o obj-$(CONFIG_RTC_DRV_DA9052) += rtc-da9052.o +obj-$(CONFIG_RTC_DRV_DA9055) += rtc-da9055.o obj-$(CONFIG_RTC_DRV_DAVINCI) += rtc-davinci.o obj-$(CONFIG_RTC_DRV_DM355EVM) += rtc-dm355evm.o obj-$(CONFIG_RTC_DRV_VRTC) += rtc-mrst.o
drivers/rtc/rtc-da9055.c new file +413 -0
413 lines not on this page · raw message
Signature · 1 line
1.7.0.4
drivers/rtc/rtc-da9055.c line 396
Quoted 13 lines · Ashish Jangam
drivers/rtc/rtc-da9055.c new file +13
-0+#define da9055_rtc_freeze NULL +#endif + +static const struct dev_pm_ops da9055_rtc_pm_ops = { + .suspend = da9055_rtc_suspend, + .resume = da9055_rtc_resume, + + .freeze = da9055_rtc_freeze, + .thaw = da9055_rtc_resume, + .restore = da9055_rtc_resume, + + .poweroff = da9055_rtc_suspend, +};
You may want to use simple dev pm ops
Does this patch looks good?
Quoted 475 lines · Ashish Jangam
This is the RTC patch for the DA9055 PMIC. This patch has got dependency on the DA9055 MFD core.
This patch is functionally tested on Samsung SMDKV6410.
Signed-off-by David Dajun Chen <[email protected]>Signed-off-by Ashish Jangam <[email protected]>---
changes since version v3:
- use ofmodule_platform_drivermacro
- add the regmap virtual irq map API.
changes since version v2:
- Use ofdevm_request_threaded_irqAPI--- drivers/rtc/Kconfig | 10 + drivers/rtc/Makefile | 1 + drivers/rtc/rtc-da9055.c | 413 ++++++++++++++++++++++++++++++++++++++++++++++ 3 files changed, 424 insertions(+), 0 deletions(-) mode change 100644 => 100755 drivers/rtc/Kconfig create mode 100644 drivers/rtc/rtc-da9055.cdrivers/rtc/Kconfig +10
-0@@ -571,6 +571,16 @@ config RTC_DRV_DA9052 Say y here to support the RTC driver for Dialog Semiconductor DA9052-BC and DA9053-AA/Bx PMICs. +config RTC_DRV_DA9055 + tristate "Dialog Semiconductor DA9055 RTC" + depends on MFD_DA9055 + help + If you say yes here you will get support for the + RTC of the Dialog DA9055 PMIC. + + This driver can also be built as a module. If so, the module + will be called rtc-da9055 + config RTC_DRV_EFI tristate "EFI RTC" depends on IA64drivers/rtc/Makefile +1
-0@@ -29,6 +29,7 @@ obj-$(CONFIG_RTC_DRV_BQ4802) += rtc-bq4802.o obj-$(CONFIG_RTC_DRV_CMOS) += rtc-cmos.o obj-$(CONFIG_RTC_DRV_COH901331) += rtc-coh901331.o obj-$(CONFIG_RTC_DRV_DA9052) += rtc-da9052.o +obj-$(CONFIG_RTC_DRV_DA9055) += rtc-da9055.o obj-$(CONFIG_RTC_DRV_DAVINCI) += rtc-davinci.o obj-$(CONFIG_RTC_DRV_DM355EVM) += rtc-dm355evm.o obj-$(CONFIG_RTC_DRV_VRTC) += rtc-mrst.odrivers/rtc/rtc-da9055.c new file +413
-0@@ -0,0 +1,413 @@ +/* +* Real time clock driver for DA9055 +* +* Copyright(c) 2012 Dialog Semiconductor Ltd. +* +* Author: Dajun Dajun Chen <[email protected]> +* +* This program is free software; you can redistribute it and/or modify +* it under the terms of the GNU General Public License as published by +* the Free Software Foundation; either version 2 of the License, or +* (at your option) any later version. +* +*/ + +#include <linux/module.h> +#include <linux/platform_device.h> +#include <linux/rtc.h> + +#include <linux/mfd/da9055/core.h> +#include <linux/mfd/da9055/reg.h> +#include <linux/mfd/da9055/pdata.h> + +struct da9055_rtc { +struct rtc_device *rtc; +struct da9055 *da9055; +int alarm_enable; +}; + +static int da9055_rtc_enable_alarm (struct da9055_rtc *rtc,bool enable) +{ +int ret; +if (enable) { + ret =da9055_reg_update (rtc->da9055,DA9055_REG_ALARM_Y , +DA9055_RTC_ALM_EN , +DA9055_RTC_ALM_EN ); +if (ret !=0 ) +dev_err (rtc->da9055->dev,"Failed to enable ALM: %d\n" , + ret); + rtc->alarm_enable =1 ; + }else { + ret =da9055_reg_update (rtc->da9055,DA9055_REG_ALARM_Y , +DA9055_RTC_ALM_EN ,0 ); +if (ret !=0 ) +dev_err (rtc->da9055->dev, +"Failed to disable ALM: %d\n" , ret); + rtc->alarm_enable =0 ; + } +return ret; +} + +static irqreturn_t da9055_rtc_alm_irq (int irq,void *data) +{ +struct da9055_rtc *rtc = data; + +da9055_rtc_enable_alarm (rtc,0 ); +rtc_update_irq (rtc->rtc,1 ,RTC_IRQF |RTC_AF ); + +return IRQ_HANDLED ; +} + +static int da9055_read_alarm (struct da9055 *da9055,struct rtc_time *rtc_tm) +{ +int ret; +uint8_t v[5 ]; + + ret =da9055_group_read (da9055,DA9055_REG_ALARM_MI ,5 , v); +if (ret !=0 ) { +dev_err (da9055->dev,"Failed to group read ALM: %d\n" , ret); +return ret; + } + + rtc_tm->tm_year = (v[4 ] &DA9055_RTC_ALM_YEAR ) +100 ; + rtc_tm->tm_mon = (v[3 ] &DA9055_RTC_ALM_MONTH ) -1 ; + rtc_tm->tm_mday = v[2 ] &DA9055_RTC_ALM_DAY ; + rtc_tm->tm_hour = v[1 ] &DA9055_RTC_ALM_HOUR ; + rtc_tm->tm_min = v[0 ] &DA9055_RTC_ALM_MIN ; + +return rtc_valid_tm (rtc_tm); +} + +static int da9055_set_alarm (struct da9055 *da9055,struct rtc_time *rtc_tm) +{ +int ret; +uint8_t v[2 ]; + + rtc_tm->tm_year -=100 ; + rtc_tm->tm_mon +=1 ; + + ret =da9055_reg_update (da9055,DA9055_REG_ALARM_MI , +DA9055_RTC_ALM_MIN , rtc_tm->tm_min); +if (ret !=0 ) { +dev_err (da9055->dev,"Failed to write ALRM MIN: %d\n" , ret); +return ret; + } + + v[0 ] = rtc_tm->tm_hour; + v[1 ] = rtc_tm->tm_mday; + + ret =da9055_group_write (da9055,DA9055_REG_ALARM_H ,2 , v); +if (ret <0 ) +return ret; + + ret =da9055_reg_update (da9055,DA9055_REG_ALARM_MO , +DA9055_RTC_ALM_MONTH , rtc_tm->tm_mon); +if (ret <0 ) +dev_err (da9055->dev,"Failed to write ALM Month:%d\n" , ret); + + ret =da9055_reg_update (da9055,DA9055_REG_ALARM_Y , +DA9055_RTC_ALM_YEAR , rtc_tm->tm_year); +if (ret <0 ) +dev_err (da9055->dev,"Failed to write ALM Year:%d\n" , ret); + +return ret; +} + +static int da9055_rtc_get_alarm_status (struct da9055 *da9055) +{ +int ret; + + ret =da9055_reg_read (da9055,DA9055_REG_ALARM_Y ); +if (ret <0 ) { +dev_err (da9055->dev,"Failed to read ALM: %d\n" , ret); +return ret; + } + ret &=DA9055_RTC_ALM_EN ; +return (ret >0 ) ?1 :0 ; +} + +static int da9055_rtc_read_time (struct device *dev,struct rtc_time *rtc_tm) +{ +struct da9055_rtc *rtc =dev_get_drvdata (dev); +uint8_t v[6 ]; +int ret; + + ret =da9055_reg_read (rtc->da9055,DA9055_REG_COUNT_S ); +if (ret <0 ) +return ret; + +/* +* Registers are only valid when RTC_READ +* status bit is asserted +*/ +if (!(ret &DA9055_RTC_READ )) +return -EBUSY ; + + ret =da9055_group_read (rtc->da9055,DA9055_REG_COUNT_S ,6 , v); +if (ret <0 ) { +dev_err (rtc->da9055->dev,"Failed to read RTC time : %d\n" , + ret); +return ret; + } + + rtc_tm->tm_year = (v[5 ] &DA9055_RTC_YEAR ) +100 ; + rtc_tm->tm_mon = (v[4 ] &DA9055_RTC_MONTH ) -1 ; + rtc_tm->tm_mday = v[3 ] &DA9055_RTC_DAY ; + rtc_tm->tm_hour = v[2 ] &DA9055_RTC_HOUR ; + rtc_tm->tm_min = v[1 ] &DA9055_RTC_MIN ; + rtc_tm->tm_sec = v[0 ] &DA9055_RTC_SEC ; + +return rtc_valid_tm (rtc_tm); +} + +static int da9055_rtc_set_time (struct device *dev,struct rtc_time *tm) +{ +struct da9055_rtc *rtc; +uint8_t v[6 ]; + + rtc =dev_get_drvdata (dev); + + v[0 ] = tm->tm_sec; + v[1 ] = tm->tm_min; + v[2 ] = tm->tm_hour; + v[3 ] = tm->tm_mday; + v[4 ] = tm->tm_mon +1 ; + v[5 ] = tm->tm_year -100 ; + +return da9055_group_write (rtc->da9055,DA9055_REG_COUNT_S ,6 , v); +} + +static int da9055_rtc_read_alarm (struct device *dev,struct rtc_wkalrm *alrm) +{ +int ret; +struct rtc_time *tm = &alrm->time; +struct da9055_rtc *rtc =dev_get_drvdata (dev); + + ret =da9055_read_alarm (rtc->da9055, tm); + +if (ret) +return ret; + + alrm->enabled =da9055_rtc_get_alarm_status (rtc->da9055); + +return 0 ; +} + +static int da9055_rtc_set_alarm (struct device *dev,struct rtc_wkalrm *alrm) +{ +int ret; +struct rtc_time *tm = &alrm->time; +struct da9055_rtc *rtc =dev_get_drvdata (dev); + + ret =da9055_rtc_enable_alarm (rtc,0 ); +if (ret <0 ) +return ret; + + ret =da9055_set_alarm (rtc->da9055, tm); +if (ret) +return ret; + + ret =da9055_rtc_enable_alarm (rtc,1 ); + +return ret; +} + +static int da9055_rtc_alarm_irq_enable (struct device *dev,unsigned int enabled) +{ +struct da9055_rtc *rtc =dev_get_drvdata (dev); + +return da9055_rtc_enable_alarm (rtc, enabled); +} + +static const struct rtc_class_ops da9055_rtc_ops = { + .read_time = da9055_rtc_read_time, + .set_time = da9055_rtc_set_time, + .read_alarm = da9055_rtc_read_alarm, + .set_alarm = da9055_rtc_set_alarm, + .alarm_irq_enable = da9055_rtc_alarm_irq_enable, +}; + +static int __init da9055_rtc_device_init (struct da9055 *da9055, +struct da9055_pdata *pdata) +{ +int ret; + +/* Enable RTC and the internal Crystal */ + ret =da9055_reg_update (da9055,DA9055_REG_CONTROL_B , +DA9055_RTC_EN ,DA9055_RTC_EN ); +if (ret <0 ) +return ret; + ret =da9055_reg_update (da9055,DA9055_REG_EN_32K , +DA9055_CRYSTAL_EN ,DA9055_CRYSTAL_EN ); +if (ret <0 ) +return ret; + +/* Enable RTC in Power Down mode */ + ret =da9055_reg_update (da9055,DA9055_REG_CONTROL_B , +DA9055_RTC_MODE_PD ,DA9055_RTC_MODE_PD ); +if (ret <0 ) +return ret; + +/* Enable RTC in Reset mode */ +if (pdata && pdata->reset_enable) { + ret =da9055_reg_update (da9055,DA9055_REG_CONTROL_B , +DA9055_RTC_MODE_SD , +DA9055_RTC_MODE_SD << +DA9055_RTC_MODE_SD_SHIFT ); +if (ret <0 ) +return ret; + } + +/* Disable the RTC TICK ALM */ + ret =da9055_reg_update (da9055,DA9055_REG_ALARM_MO , +DA9055_RTC_TICK_WAKE_MASK ,0 ); +if (ret <0 ) +return ret; + +return 0 ; +} + +static int __devinit da9055_rtc_probe (struct platform_device *pdev) +{ +struct da9055_rtc *rtc; +struct da9055_pdata *pdata =NULL ; +int ret, alm_irq; + + rtc =devm_kzalloc (&pdev->dev,sizeof (struct da9055_rtc ),GFP_KERNEL ); +if (!rtc) +return -ENOMEM ; + + rtc->da9055 =dev_get_drvdata (pdev->dev.parent); + pdata = rtc->da9055->dev->platform_data; +platform_set_drvdata (pdev, rtc); + + ret =da9055_rtc_device_init (rtc->da9055, pdata); +if (ret <0 ) + goto err_rtc; + + ret =da9055_reg_read (rtc->da9055,DA9055_REG_ALARM_Y ); +if (ret <0 ) + goto err_rtc; + +if (ret &DA9055_RTC_ALM_EN ) + rtc->alarm_enable =1 ; + +device_init_wakeup (&pdev->dev,1 ); + + rtc->rtc =rtc_device_register (pdev->name, &pdev->dev, + &da9055_rtc_ops,THIS_MODULE ); +if (IS_ERR (rtc->rtc)) { + ret =PTR_ERR (rtc->rtc); + goto err_rtc; + } + + alm_irq =platform_get_irq_byname (pdev,"ALM" ); + alm_irq =regmap_irq_get_virq (rtc->da9055->irq_data, alm_irq); + ret =devm_request_threaded_irq (&pdev->dev, alm_irq,NULL , + da9055_rtc_alm_irq, +IRQF_TRIGGER_HIGH |IRQF_ONESHOT , +"ALM" , rtc); +if (ret !=0 ) +dev_err (rtc->da9055->dev,"irq registration failed: %d\n" , ret); + +err_rtc: +return ret; + +} + +static int __devexit da9055_rtc_remove(struct platform_device *pdev) +{ +struct da9055_rtc *rtc = pdev->dev.platform_data; + +rtc_device_unregister (rtc->rtc); +platform_set_drvdata (pdev,NULL ); + +return 0 ; +} + +#ifdef CONFIG_PM +/* Turn off the alarm if it should not be a wake source. */ +static int da9055_rtc_suspend (struct device *dev) +{ +struct platform_device *pdev =to_platform_device (dev); +struct da9055_rtc *rtc =dev_get_drvdata (&pdev->dev); +int ret; + +if (!device_may_wakeup (&pdev->dev)) { +/* Disable the ALM IRQ */ + ret =da9055_rtc_enable_alarm (rtc,0 ); +if (ret <0 ) +dev_err (&pdev->dev,"Failed to disable RTC ALM\n" ); + } + +return 0 ; +} + +/* Enable the alarm if it should be enabled (in case it was disabled to +* prevent use as a wake source). +*/ +static int da9055_rtc_resume (struct device *dev) +{ +struct platform_device *pdev =to_platform_device (dev); +struct da9055_rtc *rtc =dev_get_drvdata (&pdev->dev); +int ret; + +if (!device_may_wakeup (&pdev->dev)) { +if (rtc->alarm_enable) { + ret =da9055_rtc_enable_alarm (rtc,1 ); +if (ret <0 ) +dev_err (&pdev->dev, +"Failed to restart RTC ALM\n" ); + } + } + +return 0 ; +} + +/* Unconditionally disable the alarm */ +static int da9055_rtc_freeze (struct device *dev) +{ +struct platform_device *pdev =to_platform_device (dev); +struct da9055_rtc *rtc =dev_get_drvdata (&pdev->dev); +int ret; + + ret =da9055_rtc_enable_alarm (rtc,0 ); +if (ret <0 ) +dev_err (&pdev->dev,"Failed to freeze RTC ALMs\n" ); + +return 0 ; + +} +#else +#define da9055_rtc_suspend NULL +#define da9055_rtc_resume NULL +#define da9055_rtc_freeze NULL +#endif + +static const struct dev_pm_ops da9055_rtc_pm_ops = { + .suspend = da9055_rtc_suspend, + .resume = da9055_rtc_resume, + + .freeze = da9055_rtc_freeze, + .thaw = da9055_rtc_resume, + .restore = da9055_rtc_resume, + + .poweroff = da9055_rtc_suspend, +}; + +static struct platform_driver da9055_rtc_driver = { + .probe = da9055_rtc_probe, + .remove =__devexit_p (da9055_rtc_remove), + .driver = { + .name ="da9055-rtc" , + .owner =THIS_MODULE , + .pm = &da9055_rtc_pm_ops, + }, +}; + +module_platform_driver (da9055_rtc_driver); + +MODULE_AUTHOR ("David Dajun Chen <[email protected]>" ); +MODULE_DESCRIPTION ("RTC driver for Dialog DA9055 PMIC" ); +MODULE_LICENSE ("GPL" ); +MODULE_ALIAS ("platform:da9055-rtc" );
Re: [Patch v3 3/7] rtc: DA9055 RTC driver
Any comments on this patch.
Quoted 478 lines · Ashish Jangam
Does this patch looks good?
Quoted 475 lines · Ashish Jangam
This is the RTC patch for the DA9055 PMIC. This patch has got dependency on the DA9055 MFD core.
This patch is functionally tested on Samsung SMDKV6410.
Signed-off-by David Dajun Chen <[email protected]>Signed-off-by Ashish Jangam <[email protected]>---
changes since version v3:
- use ofmodule_platform_drivermacro
- add the regmap virtual irq map API.
changes since version v2:
- Use ofdevm_request_threaded_irqAPI--- drivers/rtc/Kconfig | 10 + drivers/rtc/Makefile | 1 + drivers/rtc/rtc-da9055.c | 413 ++++++++++++++++++++++++++++++++++++++++++++++ 3 files changed, 424 insertions(+), 0 deletions(-) mode change 100644 => 100755 drivers/rtc/Kconfig create mode 100644 drivers/rtc/rtc-da9055.cdrivers/rtc/Kconfig +10
-0@@ -571,6 +571,16 @@ config RTC_DRV_DA9052 Say y here to support the RTC driver for Dialog Semiconductor DA9052-BC and DA9053-AA/Bx PMICs. +config RTC_DRV_DA9055 + tristate "Dialog Semiconductor DA9055 RTC" + depends on MFD_DA9055 + help + If you say yes here you will get support for the + RTC of the Dialog DA9055 PMIC. + + This driver can also be built as a module. If so, the module + will be called rtc-da9055 + config RTC_DRV_EFI tristate "EFI RTC" depends on IA64drivers/rtc/Makefile +1
-0@@ -29,6 +29,7 @@ obj-$(CONFIG_RTC_DRV_BQ4802) += rtc-bq4802.o obj-$(CONFIG_RTC_DRV_CMOS) += rtc-cmos.o obj-$(CONFIG_RTC_DRV_COH901331) += rtc-coh901331.o obj-$(CONFIG_RTC_DRV_DA9052) += rtc-da9052.o +obj-$(CONFIG_RTC_DRV_DA9055) += rtc-da9055.o obj-$(CONFIG_RTC_DRV_DAVINCI) += rtc-davinci.o obj-$(CONFIG_RTC_DRV_DM355EVM) += rtc-dm355evm.o obj-$(CONFIG_RTC_DRV_VRTC) += rtc-mrst.odrivers/rtc/rtc-da9055.c new file +413
-0@@ -0,0 +1,413 @@ +/* +* Real time clock driver for DA9055 +* +* Copyright(c) 2012 Dialog Semiconductor Ltd. +* +* Author: Dajun Dajun Chen <[email protected]> +* +* This program is free software; you can redistribute it and/or modify +* it under the terms of the GNU General Public License as published by +* the Free Software Foundation; either version 2 of the License, or +* (at your option) any later version. +* +*/ + +#include <linux/module.h> +#include <linux/platform_device.h> +#include <linux/rtc.h> + +#include <linux/mfd/da9055/core.h> +#include <linux/mfd/da9055/reg.h> +#include <linux/mfd/da9055/pdata.h> + +struct da9055_rtc { +struct rtc_device *rtc; +struct da9055 *da9055; +int alarm_enable; +}; + +static int da9055_rtc_enable_alarm (struct da9055_rtc *rtc,bool enable) +{ +int ret; +if (enable) { + ret =da9055_reg_update (rtc->da9055,DA9055_REG_ALARM_Y , +DA9055_RTC_ALM_EN , +DA9055_RTC_ALM_EN ); +if (ret !=0 ) +dev_err (rtc->da9055->dev,"Failed to enable ALM: %d\n" , + ret); + rtc->alarm_enable =1 ; + }else { + ret =da9055_reg_update (rtc->da9055,DA9055_REG_ALARM_Y , +DA9055_RTC_ALM_EN ,0 ); +if (ret !=0 ) +dev_err (rtc->da9055->dev, +"Failed to disable ALM: %d\n" , ret); + rtc->alarm_enable =0 ; + } +return ret; +} + +static irqreturn_t da9055_rtc_alm_irq (int irq,void *data) +{ +struct da9055_rtc *rtc = data; + +da9055_rtc_enable_alarm (rtc,0 ); +rtc_update_irq (rtc->rtc,1 ,RTC_IRQF |RTC_AF ); + +return IRQ_HANDLED ; +} + +static int da9055_read_alarm (struct da9055 *da9055,struct rtc_time *rtc_tm) +{ +int ret; +uint8_t v[5 ]; + + ret =da9055_group_read (da9055,DA9055_REG_ALARM_MI ,5 , v); +if (ret !=0 ) { +dev_err (da9055->dev,"Failed to group read ALM: %d\n" , ret); +return ret; + } + + rtc_tm->tm_year = (v[4 ] &DA9055_RTC_ALM_YEAR ) +100 ; + rtc_tm->tm_mon = (v[3 ] &DA9055_RTC_ALM_MONTH ) -1 ; + rtc_tm->tm_mday = v[2 ] &DA9055_RTC_ALM_DAY ; + rtc_tm->tm_hour = v[1 ] &DA9055_RTC_ALM_HOUR ; + rtc_tm->tm_min = v[0 ] &DA9055_RTC_ALM_MIN ; + +return rtc_valid_tm (rtc_tm); +} + +static int da9055_set_alarm (struct da9055 *da9055,struct rtc_time *rtc_tm) +{ +int ret; +uint8_t v[2 ]; + + rtc_tm->tm_year -=100 ; + rtc_tm->tm_mon +=1 ; + + ret =da9055_reg_update (da9055,DA9055_REG_ALARM_MI , +DA9055_RTC_ALM_MIN , rtc_tm->tm_min); +if (ret !=0 ) { +dev_err (da9055->dev,"Failed to write ALRM MIN: %d\n" , ret); +return ret; + } + + v[0 ] = rtc_tm->tm_hour; + v[1 ] = rtc_tm->tm_mday; + + ret =da9055_group_write (da9055,DA9055_REG_ALARM_H ,2 , v); +if (ret <0 ) +return ret; + + ret =da9055_reg_update (da9055,DA9055_REG_ALARM_MO , +DA9055_RTC_ALM_MONTH , rtc_tm->tm_mon); +if (ret <0 ) +dev_err (da9055->dev,"Failed to write ALM Month:%d\n" , ret); + + ret =da9055_reg_update (da9055,DA9055_REG_ALARM_Y , +DA9055_RTC_ALM_YEAR , rtc_tm->tm_year); +if (ret <0 ) +dev_err (da9055->dev,"Failed to write ALM Year:%d\n" , ret); + +return ret; +} + +static int da9055_rtc_get_alarm_status (struct da9055 *da9055) +{ +int ret; + + ret =da9055_reg_read (da9055,DA9055_REG_ALARM_Y ); +if (ret <0 ) { +dev_err (da9055->dev,"Failed to read ALM: %d\n" , ret); +return ret; + } + ret &=DA9055_RTC_ALM_EN ; +return (ret >0 ) ?1 :0 ; +} + +static int da9055_rtc_read_time (struct device *dev,struct rtc_time *rtc_tm) +{ +struct da9055_rtc *rtc =dev_get_drvdata (dev); +uint8_t v[6 ]; +int ret; + + ret =da9055_reg_read (rtc->da9055,DA9055_REG_COUNT_S ); +if (ret <0 ) +return ret; + +/* +* Registers are only valid when RTC_READ +* status bit is asserted +*/ +if (!(ret &DA9055_RTC_READ )) +return -EBUSY ; + + ret =da9055_group_read (rtc->da9055,DA9055_REG_COUNT_S ,6 , v); +if (ret <0 ) { +dev_err (rtc->da9055->dev,"Failed to read RTC time : %d\n" , + ret); +return ret; + } + + rtc_tm->tm_year = (v[5 ] &DA9055_RTC_YEAR ) +100 ; + rtc_tm->tm_mon = (v[4 ] &DA9055_RTC_MONTH ) -1 ; + rtc_tm->tm_mday = v[3 ] &DA9055_RTC_DAY ; + rtc_tm->tm_hour = v[2 ] &DA9055_RTC_HOUR ; + rtc_tm->tm_min = v[1 ] &DA9055_RTC_MIN ; + rtc_tm->tm_sec = v[0 ] &DA9055_RTC_SEC ; + +return rtc_valid_tm (rtc_tm); +} + +static int da9055_rtc_set_time (struct device *dev,struct rtc_time *tm) +{ +struct da9055_rtc *rtc; +uint8_t v[6 ]; + + rtc =dev_get_drvdata (dev); + + v[0 ] = tm->tm_sec; + v[1 ] = tm->tm_min; + v[2 ] = tm->tm_hour; + v[3 ] = tm->tm_mday; + v[4 ] = tm->tm_mon +1 ; + v[5 ] = tm->tm_year -100 ; + +return da9055_group_write (rtc->da9055,DA9055_REG_COUNT_S ,6 , v); +} + +static int da9055_rtc_read_alarm (struct device *dev,struct rtc_wkalrm *alrm) +{ +int ret; +struct rtc_time *tm = &alrm->time; +struct da9055_rtc *rtc =dev_get_drvdata (dev); + + ret =da9055_read_alarm (rtc->da9055, tm); + +if (ret) +return ret; + + alrm->enabled =da9055_rtc_get_alarm_status (rtc->da9055); + +return 0 ; +} + +static int da9055_rtc_set_alarm (struct device *dev,struct rtc_wkalrm *alrm) +{ +int ret; +struct rtc_time *tm = &alrm->time; +struct da9055_rtc *rtc =dev_get_drvdata (dev); + + ret =da9055_rtc_enable_alarm (rtc,0 ); +if (ret <0 ) +return ret; + + ret =da9055_set_alarm (rtc->da9055, tm); +if (ret) +return ret; + + ret =da9055_rtc_enable_alarm (rtc,1 ); + +return ret; +} + +static int da9055_rtc_alarm_irq_enable (struct device *dev,unsigned int enabled) +{ +struct da9055_rtc *rtc =dev_get_drvdata (dev); + +return da9055_rtc_enable_alarm (rtc, enabled); +} + +static const struct rtc_class_ops da9055_rtc_ops = { + .read_time = da9055_rtc_read_time, + .set_time = da9055_rtc_set_time, + .read_alarm = da9055_rtc_read_alarm, + .set_alarm = da9055_rtc_set_alarm, + .alarm_irq_enable = da9055_rtc_alarm_irq_enable, +}; + +static int __init da9055_rtc_device_init (struct da9055 *da9055, +struct da9055_pdata *pdata) +{ +int ret; + +/* Enable RTC and the internal Crystal */ + ret =da9055_reg_update (da9055,DA9055_REG_CONTROL_B , +DA9055_RTC_EN ,DA9055_RTC_EN ); +if (ret <0 ) +return ret; + ret =da9055_reg_update (da9055,DA9055_REG_EN_32K , +DA9055_CRYSTAL_EN ,DA9055_CRYSTAL_EN ); +if (ret <0 ) +return ret; + +/* Enable RTC in Power Down mode */ + ret =da9055_reg_update (da9055,DA9055_REG_CONTROL_B , +DA9055_RTC_MODE_PD ,DA9055_RTC_MODE_PD ); +if (ret <0 ) +return ret; + +/* Enable RTC in Reset mode */ +if (pdata && pdata->reset_enable) { + ret =da9055_reg_update (da9055,DA9055_REG_CONTROL_B , +DA9055_RTC_MODE_SD , +DA9055_RTC_MODE_SD << +DA9055_RTC_MODE_SD_SHIFT ); +if (ret <0 ) +return ret; + } + +/* Disable the RTC TICK ALM */ + ret =da9055_reg_update (da9055,DA9055_REG_ALARM_MO , +DA9055_RTC_TICK_WAKE_MASK ,0 ); +if (ret <0 ) +return ret; + +return 0 ; +} + +static int __devinit da9055_rtc_probe (struct platform_device *pdev) +{ +struct da9055_rtc *rtc; +struct da9055_pdata *pdata =NULL ; +int ret, alm_irq; + + rtc =devm_kzalloc (&pdev->dev,sizeof (struct da9055_rtc ),GFP_KERNEL ); +if (!rtc) +return -ENOMEM ; + + rtc->da9055 =dev_get_drvdata (pdev->dev.parent); + pdata = rtc->da9055->dev->platform_data; +platform_set_drvdata (pdev, rtc); + + ret =da9055_rtc_device_init (rtc->da9055, pdata); +if (ret <0 ) + goto err_rtc; + + ret =da9055_reg_read (rtc->da9055,DA9055_REG_ALARM_Y ); +if (ret <0 ) + goto err_rtc; + +if (ret &DA9055_RTC_ALM_EN ) + rtc->alarm_enable =1 ; + +device_init_wakeup (&pdev->dev,1 ); + + rtc->rtc =rtc_device_register (pdev->name, &pdev->dev, + &da9055_rtc_ops,THIS_MODULE ); +if (IS_ERR (rtc->rtc)) { + ret =PTR_ERR (rtc->rtc); + goto err_rtc; + } + + alm_irq =platform_get_irq_byname (pdev,"ALM" ); + alm_irq =regmap_irq_get_virq (rtc->da9055->irq_data, alm_irq); + ret =devm_request_threaded_irq (&pdev->dev, alm_irq,NULL , + da9055_rtc_alm_irq, +IRQF_TRIGGER_HIGH |IRQF_ONESHOT , +"ALM" , rtc); +if (ret !=0 ) +dev_err (rtc->da9055->dev,"irq registration failed: %d\n" , ret); + +err_rtc: +return ret; + +} + +static int __devexit da9055_rtc_remove(struct platform_device *pdev) +{ +struct da9055_rtc *rtc = pdev->dev.platform_data; + +rtc_device_unregister (rtc->rtc); +platform_set_drvdata (pdev,NULL ); + +return 0 ; +} + +#ifdef CONFIG_PM +/* Turn off the alarm if it should not be a wake source. */ +static int da9055_rtc_suspend (struct device *dev) +{ +struct platform_device *pdev =to_platform_device (dev); +struct da9055_rtc *rtc =dev_get_drvdata (&pdev->dev); +int ret; + +if (!device_may_wakeup (&pdev->dev)) { +/* Disable the ALM IRQ */ + ret =da9055_rtc_enable_alarm (rtc,0 ); +if (ret <0 ) +dev_err (&pdev->dev,"Failed to disable RTC ALM\n" ); + } + +return 0 ; +} + +/* Enable the alarm if it should be enabled (in case it was disabled to +* prevent use as a wake source). +*/ +static int da9055_rtc_resume (struct device *dev) +{ +struct platform_device *pdev =to_platform_device (dev); +struct da9055_rtc *rtc =dev_get_drvdata (&pdev->dev); +int ret; + +if (!device_may_wakeup (&pdev->dev)) { +if (rtc->alarm_enable) { + ret =da9055_rtc_enable_alarm (rtc,1 ); +if (ret <0 ) +dev_err (&pdev->dev, +"Failed to restart RTC ALM\n" ); + } + } + +return 0 ; +} + +/* Unconditionally disable the alarm */ +static int da9055_rtc_freeze (struct device *dev) +{ +struct platform_device *pdev =to_platform_device (dev); +struct da9055_rtc *rtc =dev_get_drvdata (&pdev->dev); +int ret; + + ret =da9055_rtc_enable_alarm (rtc,0 ); +if (ret <0 ) +dev_err (&pdev->dev,"Failed to freeze RTC ALMs\n" ); + +return 0 ; + +} +#else +#define da9055_rtc_suspend NULL +#define da9055_rtc_resume NULL +#define da9055_rtc_freeze NULL +#endif + +static const struct dev_pm_ops da9055_rtc_pm_ops = { + .suspend = da9055_rtc_suspend, + .resume = da9055_rtc_resume, + + .freeze = da9055_rtc_freeze, + .thaw = da9055_rtc_resume, + .restore = da9055_rtc_resume, + + .poweroff = da9055_rtc_suspend, +}; + +static struct platform_driver da9055_rtc_driver = { + .probe = da9055_rtc_probe, + .remove =__devexit_p (da9055_rtc_remove), + .driver = { + .name ="da9055-rtc" , + .owner =THIS_MODULE , + .pm = &da9055_rtc_pm_ops, + }, +}; + +module_platform_driver (da9055_rtc_driver); + +MODULE_AUTHOR ("David Dajun Chen <[email protected]>" ); +MODULE_DESCRIPTION ("RTC driver for Dialog DA9055 PMIC" ); +MODULE_LICENSE ("GPL" ); +MODULE_ALIAS ("platform:da9055-rtc" );
RE: [Patch v3 3/7] rtc: DA9055 RTC driver
Quoted 52 lines
-----Original Message-----
From: [email protected] [mailto:linux-kernel- [email protected]] On Behalf Of Ashish Jangam
Sent: Friday, November 23, 2012 3:41 PM
To: [email protected]
Cc: [email protected]; [email protected]; [email protected]; [email protected]; rtc- [email protected]; Alessandro Zummo
Subject: Re: [Patch v3 3/7] rtc: DA9055 RTC driverAny comments on this patch.
Quoted 3 lines · Ashish Jangam
Does this patch looks good?
Quoted 1 line · Ashish Jangam
This is the RTC patch for the DA9055 PMIC. This patch has got
dependency on
Quoted 4 lines
Quoted 4 lines
the DA9055 MFD core.
This patch is functionally tested on Samsung SMDKV6410.
Quoted 12 lines
Quoted 8 lines
Signed-off-by David Dajun Chen <[email protected]>Signed-off-by Ashish Jangam <[email protected]>---
changes since version v3:
- use ofmodule_platform_drivermacro
- add the regmap virtual irq map API.
changes since version v2:
- Use ofdevm_request_threaded_irqAPIQuoted 4 lines
--- drivers/rtc/Kconfig | 10 + drivers/rtc/Makefile | 1 + drivers/rtc/rtc-da9055.c | 413++++++++++++++++++++++++++++++++++++++++++++++
Quoted 6 lines
Quoted 6 lines
3 files changed, 424 insertions(+), 0 deletions(-) mode change 100644 => 100755 drivers/rtc/Kconfig create mode 100644 drivers/rtc/rtc-da9055.c+static int da9055_set_alarm(struct da9055 *da9055, struct
rtc_time
*rtc_tm)Quoted 11 lines
drivers/rtc/rtc-da9055.c
Quoted 11 lines
drivers/rtc/rtc-da9055.c new file +11
-0+{ +int ret; +uint8_t v[2 ]; + + rtc_tm->tm_year -=100 ; + rtc_tm->tm_mon +=1 ; + + ret =da9055_reg_update (da9055,DA9055_REG_ALARM_MI , +DA9055_RTC_ALM_MIN , rtc_tm->tm_min); +if (ret !=0 ) { + dev_err(da9055->dev, "Failed to write ALRM MIN: %d\n",ret);
Quoted 7 lines
Quoted 7 lines
drivers/rtc/rtc-da9055.c line 99
Quoted 7 lines
drivers/rtc/rtc-da9055.c new file +7
-0+return ret; + } + + v[0 ] = rtc_tm->tm_hour; + v[1 ] = rtc_tm->tm_mday; + + ret =da9055_group_write (da9055,DA9055_REG_ALARM_H ,2 , v);
Why don't you write all registers at once using single multi byte write command as you already used in da9055_rtc_set_time() ?
Quoted 27 lines
Quoted 8 lines
drivers/rtc/rtc-da9055.c
Quoted 5 lines
drivers/rtc/rtc-da9055.c new file +5
-0+if (ret <0 ) +return ret; + + ret = da9055_reg_update(da9055, DA9055_REG_ALARM_MO, + DA9055_RTC_ALM_MONTH, rtc_tm-
tm_mon);Review
Quoted 2 lines
(quoted diff) +2
-0+ if (ret < 0) + dev_err(da9055->dev, "Failed to write ALM Month:%d\n",ret);
Quoted 8 lines
Quoted 8 lines
drivers/rtc/rtc-da9055.c new file +8
-0+ +static int da9055_rtc_set_time (struct device *dev,struct rtc_time *tm) +{ +struct da9055_rtc *rtc; +uint8_t v[6 ]; + + rtc =dev_get_drvdata (dev); +Quoted 8 lines
drivers/rtc/rtc-da9055.c
Quoted 8 lines
drivers/rtc/rtc-da9055.c new file +8
-0+ v[0 ] = tm->tm_sec; + v[1 ] = tm->tm_min; + v[2 ] = tm->tm_hour; + v[3 ] = tm->tm_mday; + v[4 ] = tm->tm_mon +1 ; + v[5 ] = tm->tm_year -100 ; + + return da9055_group_write(rtc->da9055, DA9055_REG_COUNT_S, 6,v);
Quoted 2 lines
Quoted 2 lines
Review
Quoted 2 lines
(quoted diff) +2
-0+} +
���{.n�+�������+%�����ݶ��w��{.n�+����{��G�����{ay�ʇڙ�,j��f���h��������z_�(�階�ݢj"���m������G������?����&���~��iO��z��v�^�m����������?�I�
RE: [Patch v3 3/7] rtc: DA9055 RTC driver
Quoted 52 lines · Venu Byravarasu
Quoted 52 lines
-----Original Message-----
From: [email protected] [mailto:linux-kernel- [email protected]] On Behalf Of Ashish Jangam
Sent: Friday, November 23, 2012 3:41 PM
To: [email protected]
Cc: [email protected]; [email protected]; [email protected]; [email protected]; rtc- [email protected]; Alessandro Zummo
Subject: Re: [Patch v3 3/7] rtc: DA9055 RTC driverAny comments on this patch.
Quoted 3 lines · Ashish Jangam
Does this patch looks good?
Quoted 1 line · Ashish Jangam
This is the RTC patch for the DA9055 PMIC. This patch has got
dependency on
Quoted 4 lines
Quoted 4 lines
the DA9055 MFD core.
This patch is functionally tested on Samsung SMDKV6410.
Quoted 12 lines
Quoted 8 lines
Signed-off-by David Dajun Chen <[email protected]>Signed-off-by Ashish Jangam <[email protected]>---
changes since version v3:
- use ofmodule_platform_drivermacro
- add the regmap virtual irq map API.
changes since version v2:
- Use ofdevm_request_threaded_irqAPIQuoted 4 lines
--- drivers/rtc/Kconfig | 10 + drivers/rtc/Makefile | 1 + drivers/rtc/rtc-da9055.c | 413++++++++++++++++++++++++++++++++++++++++++++++
Quoted 6 lines
Quoted 6 lines
3 files changed, 424 insertions(+), 0 deletions(-) mode change 100644 => 100755 drivers/rtc/Kconfig create mode 100644 drivers/rtc/rtc-da9055.c+static int da9055_set_alarm(struct da9055 *da9055, struct
rtc_time
*rtc_tm)Quoted 11 lines
drivers/rtc/rtc-da9055.c
Quoted 11 lines
drivers/rtc/rtc-da9055.c new file +11
-0+{ +int ret; +uint8_t v[2 ]; + + rtc_tm->tm_year -=100 ; + rtc_tm->tm_mon +=1 ; + + ret =da9055_reg_update (da9055,DA9055_REG_ALARM_MI , +DA9055_RTC_ALM_MIN , rtc_tm->tm_min); +if (ret !=0 ) { + dev_err(da9055->dev, "Failed to write ALRM MIN: %d\n",ret);
Quoted 10 lines
Quoted 7 lines
Quoted 7 lines
drivers/rtc/rtc-da9055.c line 99
Quoted 7 lines
drivers/rtc/rtc-da9055.c new file +7
-0+return ret; + } + + v[0 ] = rtc_tm->tm_hour; + v[1 ] = rtc_tm->tm_mday; + + ret =da9055_group_write (da9055,DA9055_REG_ALARM_H ,2 , v);Why don't you write all registers at once using single multi byte write command as you already used in
da9055_rtc_set_time()?
Unlike TIME/Date registers some of the ALARM register bits has got few other bits which should not get modified during the setting of ALARM therefore reg_update was used for those ALARM register.
Re: [Patch v3 3/7] rtc: DA9055 RTC driver
In reply to Ashish JangamQuoted 9 lines · On Fri, 23 Nov 2012 15:41:03 +0530 Ashish Jangam…
Quoted 8 lines · Ashish Jangam
Does this patch looks good?
Quoted 5 lines · Ashish Jangam
This is the RTC patch for the DA9055 PMIC. This patch has got dependency on the DA9055 MFD core.
This patch is functionally tested on Samsung SMDKV6410.
(Top-posting repaired. Please don't top-post).
Quoted 1 line
Any comments on this patch.
"shubhro <[email protected]>" made a comment, but it was ignored:
: On Thu, 11 Oct 2012 16:53:34 +0530
: shubhro <[email protected]> wrote:
:
: > On Thursday 11 October 2012 04:10 PM, Ashish Jangam wrote:
: > > +#define da9055_rtc_freeze NULL
: > > +#endif
: > > +
: > > +static const struct dev_pm_ops da9055_rtc_pm_ops = {
: > > + .suspend = da9055_rtc_suspend,
: > > + .resume = da9055_rtc_resume,
: > > +
: > > + .freeze = da9055_rtc_freeze,
: > > + .thaw = da9055_rtc_resume,
: > > + .restore = da9055_rtc_resume,
: > > +
: > > + .poweroff = da9055_rtc_suspend,
: > > +};
: > You may want to use simple dev pm ops
Re: [Patch v3 3/7] rtc: DA9055 RTC driver
Quoted 29 lines · Andrew Morton
Quoted 9 lines · On Fri, 23 Nov 2012 15:41:03 +0530 Ashish Jangam…
Quoted 8 lines · Ashish Jangam
Does this patch looks good?
Quoted 5 lines · Ashish Jangam
This is the RTC patch for the DA9055 PMIC. This patch has got dependency on the DA9055 MFD core.
This patch is functionally tested on Samsung SMDKV6410.
(Top-posting repaired. Please don't top-post).
Quoted 1 line
Any comments on this patch.
"shubhro <[email protected]>" made a comment, but it was ignored:
: On Thu, 11 Oct 2012 16:53:34 +0530
: shubhro <[email protected]> wrote:
:
: > On Thursday 11 October 2012 04:10 PM, Ashish Jangam wrote:
: > > +#defineda9055_rtc_freezeNULL
: > > +#endif
: > > +
: > > +static const structdev_pm_opsda9055_rtc_pm_ops= {
: > > + .suspend =da9055_rtc_suspend,
: > > + .resume =da9055_rtc_resume,
Quoted 8 lines
: > > +
: > > + .freeze =da9055_rtc_freeze,
: > > + .thaw =da9055_rtc_resume,
: > > + .restore =da9055_rtc_resume,
: > > +
: > > + .poweroff =da9055_rtc_suspend,
: > > +};
: > You may want to use simple dev pm ops
To support generic PM ops these additional ops were supported.
Re: [Patch v3 3/7] rtc: DA9055 RTC driver
In reply to Andrew MortonQuoted 40 lines · Andrew Morton
Quoted 9 lines · On Fri, 23 Nov 2012 15:41:03 +0530 Ashish Jangam…
Quoted 8 lines · Ashish Jangam
Does this patch looks good?
Quoted 5 lines · Ashish Jangam
This is the RTC patch for the DA9055 PMIC. This patch has got dependency on the DA9055 MFD core.
This patch is functionally tested on Samsung SMDKV6410.
(Top-posting repaired. Please don't top-post).
Quoted 1 line
Any comments on this patch.
"shubhro <[email protected]>" made a comment, but it was ignored:
: On Thu, 11 Oct 2012 16:53:34 +0530
: shubhro <[email protected]> wrote:
:
: > On Thursday 11 October 2012 04:10 PM, Ashish Jangam wrote:
: > > +#defineda9055_rtc_freezeNULL
: > > +#endif
: > > +
: > > +static const structdev_pm_opsda9055_rtc_pm_ops= {
: > > + .suspend =da9055_rtc_suspend,
: > > + .resume =da9055_rtc_resume,
: > > +
: > > + .freeze =da9055_rtc_freeze,
: > > + .thaw =da9055_rtc_resume,
: > > + .restore =da9055_rtc_resume,
: > > +
: > > + .poweroff =da9055_rtc_suspend,
: > > +};
: > You may want to use simple dev pm ops
Re: [Patch v3 3/7] rtc: DA9055 RTC driver
In reply to Ashish JangamQuoted 32 lines · Ashish Jangam
Quoted 23 lines · Andrew Morton
Quoted 7 lines · On Fri, 23 Nov 2012 15:41:03 +0530 Ashish Jangam…
Quoted 6 lines · Ashish Jangam
Does this patch looks good?
Quoted 4 lines · Ashish Jangam
This is the RTC patch for the DA9055 PMIC. This patch has got dependency on the DA9055 MFD core.
This patch is functionally tested on Samsung SMDKV6410.
"shubhro <[email protected]>" made a comment, but it was ignored:
: On Thu, 11 Oct 2012 16:53:34 +0530
: shubhro <[email protected]> wrote:
:
: > On Thursday 11 October 2012 04:10 PM, Ashish Jangam wrote:
: > > +#defineda9055_rtc_freezeNULL
: > > +#endif
: > > +
: > > +static const structdev_pm_opsda9055_rtc_pm_ops= {
: > > + .suspend =da9055_rtc_suspend,
: > > + .resume =da9055_rtc_resume,Quoted 8 lines
: > > +
: > > + .freeze =da9055_rtc_freeze,
: > > + .thaw =da9055_rtc_resume,
: > > + .restore =da9055_rtc_resume,
: > > +
: > > + .poweroff =da9055_rtc_suspend,
: > > +};
: > You may want to use simple dev pm ops
Quoted 1 line
To support generic PM ops these additional ops were supported.
Are there any further comments on this patch.
Re: [Patch v3 3/7] rtc: DA9055 RTC driver
Quoted 33 lines · On Mon, 10 Dec 2012 11:12:29 +0530 Ashish Jangam…
Quoted 32 lines · Ashish Jangam
Quoted 31 lines · Andrew Morton
Quoted 7 lines · On Fri, 23 Nov 2012 15:41:03 +0530 Ashish Jangam…
Quoted 6 lines · Ashish Jangam
Does this patch looks good?
Quoted 4 lines · Ashish Jangam
This is the RTC patch for the DA9055 PMIC. This patch has got dependency on the DA9055 MFD core.
This patch is functionally tested on Samsung SMDKV6410.
"shubhro <[email protected]>" made a comment, but it was ignored:
: On Thu, 11 Oct 2012 16:53:34 +0530
: shubhro <[email protected]> wrote:
:
: > On Thursday 11 October 2012 04:10 PM, Ashish Jangam wrote:
: > > +#defineda9055_rtc_freezeNULL
: > > +#endif
: > > +
: > > +static const structdev_pm_opsda9055_rtc_pm_ops= {
: > > + .suspend =da9055_rtc_suspend,
: > > + .resume =da9055_rtc_resume,
: > > +
: > > + .freeze =da9055_rtc_freeze,
: > > + .thaw =da9055_rtc_resume,
: > > + .restore =da9055_rtc_resume,
: > > +
: > > + .poweroff =da9055_rtc_suspend,
: > > +};
: > You may want to use simple dev pm ops
Quoted 2 lines
Quoted 1 line
To support generic PM ops these additional ops were supported.
Are there any further comments on this patch.
Is the patch usable against current mainline? "This patch has got dependency on the DA9055 MFD core". What is the status of that?
If the patch is ready to go then please refresh it, retest and resend?
Re: [Patch v3 3/7] rtc: DA9055 RTC driver
Quoted 36 lines · Andrew Morton
Quoted 33 lines · On Mon, 10 Dec 2012 11:12:29 +0530 Ashish Jangam…
Quoted 32 lines · Ashish Jangam
Quoted 31 lines · Andrew Morton
Quoted 7 lines · On Fri, 23 Nov 2012 15:41:03 +0530 Ashish Jangam…
Quoted 6 lines · Ashish Jangam
Does this patch looks good?
Quoted 4 lines · Ashish Jangam
This is the RTC patch for the DA9055 PMIC. This patch has got dependency on the DA9055 MFD core.
This patch is functionally tested on Samsung SMDKV6410.
"shubhro <[email protected]>" made a comment, but it was ignored:
: On Thu, 11 Oct 2012 16:53:34 +0530
: shubhro <[email protected]> wrote:
:
: > On Thursday 11 October 2012 04:10 PM, Ashish Jangam wrote:
: > > +#defineda9055_rtc_freezeNULL
: > > +#endif
: > > +
: > > +static const structdev_pm_opsda9055_rtc_pm_ops= {
: > > + .suspend =da9055_rtc_suspend,
: > > + .resume =da9055_rtc_resume,
: > > +
: > > + .freeze =da9055_rtc_freeze,
: > > + .thaw =da9055_rtc_resume,
: > > + .restore =da9055_rtc_resume,
: > > +
: > > + .poweroff =da9055_rtc_suspend,
: > > +};
: > You may want to use simple dev pm ops
Quoted 5 lines
Quoted 2 lines
Quoted 1 line
To support generic PM ops these additional ops were supported.
Are there any further comments on this patch.
Is the patch usable against current mainline? "This patch has got dependency on the DA9055 MFD core". What is the status of that?
The dependent DA9055 MFD core is merged.
Quoted 3 lines
If the patch is ready to go then please refresh it, retest and resend?
Ok, I can test and resend it.
Re: [Patch v3 3/7] rtc: DA9055 RTC driver
In reply to Andrew Morton(This rtc patch is resend for merge)
---
This is the RTC patch for the DA9055 PMIC. This patch has got dependency on the DA9055 MFD core.
This patch is functionally tested on Samsung SMDKV6410.
---
changes since version v3:
- use of module_platform_driver macro
- add the regmap virtual irq map API
changes since version v2:
- Use of devm_request_threaded_irq API
--- drivers/rtc/Kconfig | 10 + drivers/rtc/Makefile | 1 + drivers/rtc/rtc-da9055.c | 413 ++++++++++++++++++++++++++++++++++++++++++++++ 3 files changed, 424 insertions(+), 0 deletions(-) create mode 100644 drivers/rtc/rtc-da9055.c
drivers/rtc/Kconfig +10 -0
@@ -609,6 +609,16 @@ config RTC_DRV_DA9052 Say y here to support the RTC driver for Dialog Semiconductor DA9052-BC and DA9053-AA/Bx PMICs. +config RTC_DRV_DA9055 + tristate "Dialog Semiconductor DA9055 RTC" + depends on MFD_DA9055 + help + If you say yes here you will get support for the + RTC of the Dialog DA9055 PMIC. + + This driver can also be built as a module. If so, the module + will be called rtc-da9055 + config RTC_DRV_EFI tristate "EFI RTC" depends on IA64
drivers/rtc/Makefile +1 -0
@@ -29,6 +29,7 @@ obj-$(CONFIG_RTC_DRV_BQ4802) += rtc-bq4802.o obj-$(CONFIG_RTC_DRV_CMOS) += rtc-cmos.o obj-$(CONFIG_RTC_DRV_COH901331) += rtc-coh901331.o obj-$(CONFIG_RTC_DRV_DA9052) += rtc-da9052.o +obj-$(CONFIG_RTC_DRV_DA9055) += rtc-da9055.o obj-$(CONFIG_RTC_DRV_DAVINCI) += rtc-davinci.o obj-$(CONFIG_RTC_DRV_DM355EVM) += rtc-dm355evm.o obj-$(CONFIG_RTC_DRV_VRTC) += rtc-mrst.o
drivers/rtc/rtc-da9055.c new file +413 -0
413 lines not on this page · raw message
Signature · 1 line
1.7.0.4
Re: [Patch v3 3/7] rtc: DA9055 RTC driver
Quoted 35 lines · On Thu, 13 Dec 2012 18:34:20 +0530 Ashish Jangam…
(This rtc patch is resend for merge)
---
This is the RTC patch for the DA9055 PMIC. This patch has got dependency on the DA9055 MFD core.This patch is functionally tested on Samsung SMDKV6410.
...
drivers/rtc/rtc-da9055.c new file +26
-0+static int __init da9055_rtc_device_init (struct da9055 *da9055, +struct da9055_pdata *pdata) +{ +int ret; + +/* Enable RTC and the internal Crystal */ + ret =da9055_reg_update (da9055,DA9055_REG_CONTROL_B , +DA9055_RTC_EN ,DA9055_RTC_EN ); +if (ret <0 ) +return ret; + ret =da9055_reg_update (da9055,DA9055_REG_EN_32K , +DA9055_CRYSTAL_EN ,DA9055_CRYSTAL_EN ); +if (ret <0 ) +return ret; + +/* Enable RTC in Power Down mode */ + ret =da9055_reg_update (da9055,DA9055_REG_CONTROL_B , +DA9055_RTC_MODE_PD ,DA9055_RTC_MODE_PD ); +if (ret <0 ) +return ret; + +/* Enable RTC in Reset mode */ +if (pdata && pdata->reset_enable) { + ret = da9055_reg_update(da9055, DA9055_REG_CONTROL_B, + DA9055_RTC_MODE_SD, + DA9055_RTC_MODE_SD <<
drivers/rtc/rtc-da9055.c line 265
Quoted 10 lines
drivers/rtc/rtc-da9055.c new file +10
-0+ DA9055_RTC_MODE_SD_SHIFT); +if (ret <0 ) +return ret; + } + +/* Disable the RTC TICK ALM */ + ret =da9055_reg_update (da9055,DA9055_REG_ALARM_MO , +DA9055_RTC_TICK_WAKE_MASK ,0 ); +if (ret <0 ) +return ret;
If one of these failure paths is taken, the hardware will be left in a part-initialised state. Is this wise?
Quoted 2 lines
(quoted diff) +2
-0+ return 0; +}