Re: [PATCH v2 03/14] mfd/rtc: sec/s5m: Rename SEC* symbols to S5M

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, 13 Feb 2014, Krzysztof Kozlowski wrote:

> This patch prepares for adding support for S2MPS14 RTC device to the
> rtc-s5m driver:
> 1. Renames SEC* symbols to S5M.
> 2. Adds S5M prefix to some of defines which are different between S5M876X
> and S2MPS14.
> 
> This is only a rename-like patch, new code is not added.
> 
> Signed-off-by: Krzysztof Kozlowski <k.kozlowski@xxxxxxxxxxx>
> Cc: Alessandro Zummo <a.zummo@xxxxxxxxxxxx>
> Cc: rtc-linux@xxxxxxxxxxxxxxxx

I Acked the MFD parts of this already.

> ---
>  drivers/mfd/sec-core.c          |    2 +-
>  drivers/rtc/rtc-s5m.c           |   64 ++++++++++++++++-----------------
>  include/linux/mfd/samsung/rtc.h |   76 +++++++++++++++++++--------------------
>  3 files changed, 71 insertions(+), 71 deletions(-)
> 
> diff --git a/drivers/mfd/sec-core.c b/drivers/mfd/sec-core.c
> index 0efa69e123ee..8504de82b7e0 100644
> --- a/drivers/mfd/sec-core.c
> +++ b/drivers/mfd/sec-core.c
> @@ -142,7 +142,7 @@ static const struct regmap_config sec_rtc_regmap_config = {
>  	.reg_bits = 8,
>  	.val_bits = 8,
>  
> -	.max_register = SEC_RTC_REG_MAX,
> +	.max_register = S5M_RTC_REG_MAX,
>  };
>  
>  #ifdef CONFIG_OF
> diff --git a/drivers/rtc/rtc-s5m.c b/drivers/rtc/rtc-s5m.c
> index 476af93543f6..d26e2480f8b3 100644
> --- a/drivers/rtc/rtc-s5m.c
> +++ b/drivers/rtc/rtc-s5m.c
> @@ -30,10 +30,10 @@
>  
>  /*
>   * Maximum number of retries for checking changes in UDR field
> - * of SEC_RTC_UDR_CON register (to limit possible endless loop).
> + * of S5M_RTC_UDR_CON register (to limit possible endless loop).
>   *
>   * After writing to RTC registers (setting time or alarm) read the UDR field
> - * in SEC_RTC_UDR_CON register. UDR is auto-cleared when data have
> + * in S5M_RTC_UDR_CON register. UDR is auto-cleared when data have
>   * been transferred.
>   */
>  #define UDR_READ_RETRY_CNT	5
> @@ -104,8 +104,8 @@ static inline int s5m8767_wait_for_udr_update(struct s5m_rtc_info *info)
>  	unsigned int data;
>  
>  	do {
> -		ret = regmap_read(info->regmap, SEC_RTC_UDR_CON, &data);
> -	} while (--retry && (data & RTC_UDR_MASK) && !ret);
> +		ret = regmap_read(info->regmap, S5M_RTC_UDR_CON, &data);
> +	} while (--retry && (data & S5M_RTC_UDR_MASK) && !ret);
>  
>  	if (!retry)
>  		dev_err(info->dev, "waiting for UDR update, reached max number of retries\n");
> @@ -118,16 +118,16 @@ static inline int s5m8767_rtc_set_time_reg(struct s5m_rtc_info *info)
>  	int ret;
>  	unsigned int data;
>  
> -	ret = regmap_read(info->regmap, SEC_RTC_UDR_CON, &data);
> +	ret = regmap_read(info->regmap, S5M_RTC_UDR_CON, &data);
>  	if (ret < 0) {
>  		dev_err(info->dev, "failed to read update reg(%d)\n", ret);
>  		return ret;
>  	}
>  
> -	data |= RTC_TIME_EN_MASK;
> -	data |= RTC_UDR_MASK;
> +	data |= S5M_RTC_TIME_EN_MASK;
> +	data |= S5M_RTC_UDR_MASK;
>  
> -	ret = regmap_write(info->regmap, SEC_RTC_UDR_CON, data);
> +	ret = regmap_write(info->regmap, S5M_RTC_UDR_CON, data);
>  	if (ret < 0) {
>  		dev_err(info->dev, "failed to write update reg(%d)\n", ret);
>  		return ret;
> @@ -143,17 +143,17 @@ static inline int s5m8767_rtc_set_alarm_reg(struct s5m_rtc_info *info)
>  	int ret;
>  	unsigned int data;
>  
> -	ret = regmap_read(info->regmap, SEC_RTC_UDR_CON, &data);
> +	ret = regmap_read(info->regmap, S5M_RTC_UDR_CON, &data);
>  	if (ret < 0) {
>  		dev_err(info->dev, "%s: fail to read update reg(%d)\n",
>  			__func__, ret);
>  		return ret;
>  	}
>  
> -	data &= ~RTC_TIME_EN_MASK;
> -	data |= RTC_UDR_MASK;
> +	data &= ~S5M_RTC_TIME_EN_MASK;
> +	data |= S5M_RTC_UDR_MASK;
>  
> -	ret = regmap_write(info->regmap, SEC_RTC_UDR_CON, data);
> +	ret = regmap_write(info->regmap, S5M_RTC_UDR_CON, data);
>  	if (ret < 0) {
>  		dev_err(info->dev, "%s: fail to write update reg(%d)\n",
>  			__func__, ret);
> @@ -203,7 +203,7 @@ static int s5m_rtc_read_time(struct device *dev, struct rtc_time *tm)
>  	u8 data[8];
>  	int ret;
>  
> -	ret = regmap_bulk_read(info->regmap, SEC_RTC_SEC, data, 8);
> +	ret = regmap_bulk_read(info->regmap, S5M_RTC_SEC, data, 8);
>  	if (ret < 0)
>  		return ret;
>  
> @@ -251,7 +251,7 @@ static int s5m_rtc_set_time(struct device *dev, struct rtc_time *tm)
>  		1900 + tm->tm_year, 1 + tm->tm_mon, tm->tm_mday,
>  		tm->tm_hour, tm->tm_min, tm->tm_sec, tm->tm_wday);
>  
> -	ret = regmap_raw_write(info->regmap, SEC_RTC_SEC, data, 8);
> +	ret = regmap_raw_write(info->regmap, S5M_RTC_SEC, data, 8);
>  	if (ret < 0)
>  		return ret;
>  
> @@ -267,20 +267,20 @@ static int s5m_rtc_read_alarm(struct device *dev, struct rtc_wkalrm *alrm)
>  	unsigned int val;
>  	int ret, i;
>  
> -	ret = regmap_bulk_read(info->regmap, SEC_ALARM0_SEC, data, 8);
> +	ret = regmap_bulk_read(info->regmap, S5M_ALARM0_SEC, data, 8);
>  	if (ret < 0)
>  		return ret;
>  
>  	switch (info->device_type) {
>  	case S5M8763X:
>  		s5m8763_data_to_tm(data, &alrm->time);
> -		ret = regmap_read(info->regmap, SEC_ALARM0_CONF, &val);
> +		ret = regmap_read(info->regmap, S5M_ALARM0_CONF, &val);
>  		if (ret < 0)
>  			return ret;
>  
>  		alrm->enabled = !!val;
>  
> -		ret = regmap_read(info->regmap, SEC_RTC_STATUS, &val);
> +		ret = regmap_read(info->regmap, S5M_RTC_STATUS, &val);
>  		if (ret < 0)
>  			return ret;
>  
> @@ -303,7 +303,7 @@ static int s5m_rtc_read_alarm(struct device *dev, struct rtc_wkalrm *alrm)
>  		}
>  
>  		alrm->pending = 0;
> -		ret = regmap_read(info->regmap, SEC_RTC_STATUS, &val);
> +		ret = regmap_read(info->regmap, S5M_RTC_STATUS, &val);
>  		if (ret < 0)
>  			return ret;
>  		break;
> @@ -312,7 +312,7 @@ static int s5m_rtc_read_alarm(struct device *dev, struct rtc_wkalrm *alrm)
>  		return -EINVAL;
>  	}
>  
> -	if (val & ALARM0_STATUS)
> +	if (val & S5M_ALARM0_STATUS)
>  		alrm->pending = 1;
>  	else
>  		alrm->pending = 0;
> @@ -326,7 +326,7 @@ static int s5m_rtc_stop_alarm(struct s5m_rtc_info *info)
>  	int ret, i;
>  	struct rtc_time tm;
>  
> -	ret = regmap_bulk_read(info->regmap, SEC_ALARM0_SEC, data, 8);
> +	ret = regmap_bulk_read(info->regmap, S5M_ALARM0_SEC, data, 8);
>  	if (ret < 0)
>  		return ret;
>  
> @@ -337,14 +337,14 @@ static int s5m_rtc_stop_alarm(struct s5m_rtc_info *info)
>  
>  	switch (info->device_type) {
>  	case S5M8763X:
> -		ret = regmap_write(info->regmap, SEC_ALARM0_CONF, 0);
> +		ret = regmap_write(info->regmap, S5M_ALARM0_CONF, 0);
>  		break;
>  
>  	case S5M8767X:
>  		for (i = 0; i < 7; i++)
>  			data[i] &= ~ALARM_ENABLE_MASK;
>  
> -		ret = regmap_raw_write(info->regmap, SEC_ALARM0_SEC, data, 8);
> +		ret = regmap_raw_write(info->regmap, S5M_ALARM0_SEC, data, 8);
>  		if (ret < 0)
>  			return ret;
>  
> @@ -366,7 +366,7 @@ static int s5m_rtc_start_alarm(struct s5m_rtc_info *info)
>  	u8 alarm0_conf;
>  	struct rtc_time tm;
>  
> -	ret = regmap_bulk_read(info->regmap, SEC_ALARM0_SEC, data, 8);
> +	ret = regmap_bulk_read(info->regmap, S5M_ALARM0_SEC, data, 8);
>  	if (ret < 0)
>  		return ret;
>  
> @@ -378,7 +378,7 @@ static int s5m_rtc_start_alarm(struct s5m_rtc_info *info)
>  	switch (info->device_type) {
>  	case S5M8763X:
>  		alarm0_conf = 0x77;
> -		ret = regmap_write(info->regmap, SEC_ALARM0_CONF, alarm0_conf);
> +		ret = regmap_write(info->regmap, S5M_ALARM0_CONF, alarm0_conf);
>  		break;
>  
>  	case S5M8767X:
> @@ -393,7 +393,7 @@ static int s5m_rtc_start_alarm(struct s5m_rtc_info *info)
>  		if (data[RTC_YEAR1] & 0x7f)
>  			data[RTC_YEAR1] |= ALARM_ENABLE_MASK;
>  
> -		ret = regmap_raw_write(info->regmap, SEC_ALARM0_SEC, data, 8);
> +		ret = regmap_raw_write(info->regmap, S5M_ALARM0_SEC, data, 8);
>  		if (ret < 0)
>  			return ret;
>  		ret = s5m8767_rtc_set_alarm_reg(info);
> @@ -435,7 +435,7 @@ static int s5m_rtc_set_alarm(struct device *dev, struct rtc_wkalrm *alrm)
>  	if (ret < 0)
>  		return ret;
>  
> -	ret = regmap_raw_write(info->regmap, SEC_ALARM0_SEC, data, 8);
> +	ret = regmap_raw_write(info->regmap, S5M_ALARM0_SEC, data, 8);
>  	if (ret < 0)
>  		return ret;
>  
> @@ -480,7 +480,7 @@ static const struct rtc_class_ops s5m_rtc_ops = {
>  static void s5m_rtc_enable_wtsr(struct s5m_rtc_info *info, bool enable)
>  {
>  	int ret;
> -	ret = regmap_update_bits(info->regmap, SEC_WTSR_SMPL_CNTL,
> +	ret = regmap_update_bits(info->regmap, S5M_WTSR_SMPL_CNTL,
>  				 WTSR_ENABLE_MASK,
>  				 enable ? WTSR_ENABLE_MASK : 0);
>  	if (ret < 0)
> @@ -491,7 +491,7 @@ static void s5m_rtc_enable_wtsr(struct s5m_rtc_info *info, bool enable)
>  static void s5m_rtc_enable_smpl(struct s5m_rtc_info *info, bool enable)
>  {
>  	int ret;
> -	ret = regmap_update_bits(info->regmap, SEC_WTSR_SMPL_CNTL,
> +	ret = regmap_update_bits(info->regmap, S5M_WTSR_SMPL_CNTL,
>  				 SMPL_ENABLE_MASK,
>  				 enable ? SMPL_ENABLE_MASK : 0);
>  	if (ret < 0)
> @@ -506,7 +506,7 @@ static int s5m8767_rtc_init_reg(struct s5m_rtc_info *info)
>  	int ret;
>  	struct rtc_time tm;
>  
> -	ret = regmap_read(info->regmap, SEC_RTC_UDR_CON, &tp_read);
> +	ret = regmap_read(info->regmap, S5M_RTC_UDR_CON, &tp_read);
>  	if (ret < 0) {
>  		dev_err(info->dev, "%s: fail to read control reg(%d)\n",
>  			__func__, ret);
> @@ -518,7 +518,7 @@ static int s5m8767_rtc_init_reg(struct s5m_rtc_info *info)
>  	data[1] = (0 << BCD_EN_SHIFT) | (1 << MODEL24_SHIFT);
>  
>  	info->rtc_24hr_mode = 1;
> -	ret = regmap_raw_write(info->regmap, SEC_ALARM0_CONF, data, 2);
> +	ret = regmap_raw_write(info->regmap, S5M_ALARM0_CONF, data, 2);
>  	if (ret < 0) {
>  		dev_err(info->dev, "%s: fail to write controlm reg(%d)\n",
>  			__func__, ret);
> @@ -540,7 +540,7 @@ static int s5m8767_rtc_init_reg(struct s5m_rtc_info *info)
>  		ret = s5m_rtc_set_time(info->dev, &tm);
>  	}
>  
> -	ret = regmap_update_bits(info->regmap, SEC_RTC_UDR_CON,
> +	ret = regmap_update_bits(info->regmap, S5M_RTC_UDR_CON,
>  				 RTC_TCON_MASK, tp_read | RTC_TCON_MASK);
>  	if (ret < 0)
>  		dev_err(info->dev, "%s: fail to update TCON reg(%d)\n",
> @@ -623,7 +623,7 @@ static void s5m_rtc_shutdown(struct platform_device *pdev)
>  	if (info->wtsr_smpl) {
>  		for (i = 0; i < 3; i++) {
>  			s5m_rtc_enable_wtsr(info, false);
> -			regmap_read(info->regmap, SEC_WTSR_SMPL_CNTL, &val);
> +			regmap_read(info->regmap, S5M_WTSR_SMPL_CNTL, &val);
>  			pr_debug("%s: WTSR_SMPL reg(0x%02x)\n", __func__, val);
>  			if (val & WTSR_ENABLE_MASK)
>  				pr_emerg("%s: fail to disable WTSR\n",
> diff --git a/include/linux/mfd/samsung/rtc.h b/include/linux/mfd/samsung/rtc.h
> index 4627f59ebd84..bdf0573891d0 100644
> --- a/include/linux/mfd/samsung/rtc.h
> +++ b/include/linux/mfd/samsung/rtc.h
> @@ -13,38 +13,38 @@
>  #ifndef __LINUX_MFD_SEC_RTC_H
>  #define __LINUX_MFD_SEC_RTC_H
>  
> -enum sec_rtc_reg {
> -	SEC_RTC_SEC,
> -	SEC_RTC_MIN,
> -	SEC_RTC_HOUR,
> -	SEC_RTC_WEEKDAY,
> -	SEC_RTC_DATE,
> -	SEC_RTC_MONTH,
> -	SEC_RTC_YEAR1,
> -	SEC_RTC_YEAR2,
> -	SEC_ALARM0_SEC,
> -	SEC_ALARM0_MIN,
> -	SEC_ALARM0_HOUR,
> -	SEC_ALARM0_WEEKDAY,
> -	SEC_ALARM0_DATE,
> -	SEC_ALARM0_MONTH,
> -	SEC_ALARM0_YEAR1,
> -	SEC_ALARM0_YEAR2,
> -	SEC_ALARM1_SEC,
> -	SEC_ALARM1_MIN,
> -	SEC_ALARM1_HOUR,
> -	SEC_ALARM1_WEEKDAY,
> -	SEC_ALARM1_DATE,
> -	SEC_ALARM1_MONTH,
> -	SEC_ALARM1_YEAR1,
> -	SEC_ALARM1_YEAR2,
> -	SEC_ALARM0_CONF,
> -	SEC_ALARM1_CONF,
> -	SEC_RTC_STATUS,
> -	SEC_WTSR_SMPL_CNTL,
> -	SEC_RTC_UDR_CON,
> +enum s5m_rtc_reg {
> +	S5M_RTC_SEC,
> +	S5M_RTC_MIN,
> +	S5M_RTC_HOUR,
> +	S5M_RTC_WEEKDAY,
> +	S5M_RTC_DATE,
> +	S5M_RTC_MONTH,
> +	S5M_RTC_YEAR1,
> +	S5M_RTC_YEAR2,
> +	S5M_ALARM0_SEC,
> +	S5M_ALARM0_MIN,
> +	S5M_ALARM0_HOUR,
> +	S5M_ALARM0_WEEKDAY,
> +	S5M_ALARM0_DATE,
> +	S5M_ALARM0_MONTH,
> +	S5M_ALARM0_YEAR1,
> +	S5M_ALARM0_YEAR2,
> +	S5M_ALARM1_SEC,
> +	S5M_ALARM1_MIN,
> +	S5M_ALARM1_HOUR,
> +	S5M_ALARM1_WEEKDAY,
> +	S5M_ALARM1_DATE,
> +	S5M_ALARM1_MONTH,
> +	S5M_ALARM1_YEAR1,
> +	S5M_ALARM1_YEAR2,
> +	S5M_ALARM0_CONF,
> +	S5M_ALARM1_CONF,
> +	S5M_RTC_STATUS,
> +	S5M_WTSR_SMPL_CNTL,
> +	S5M_RTC_UDR_CON,
>  
> -	SEC_RTC_REG_MAX,
> +	S5M_RTC_REG_MAX,
>  };
>  
>  #define RTC_I2C_ADDR		(0x0C >> 1)
> @@ -52,9 +52,9 @@ enum sec_rtc_reg {
>  #define HOUR_12			(1 << 7)
>  #define HOUR_AMPM		(1 << 6)
>  #define HOUR_PM			(1 << 5)
> -#define ALARM0_STATUS		(1 << 1)
> -#define ALARM1_STATUS		(1 << 2)
> -#define UPDATE_AD		(1 << 0)
> +#define S5M_ALARM0_STATUS	(1 << 1)
> +#define S5M_ALARM1_STATUS	(1 << 2)
> +#define S5M_UPDATE_AD		(1 << 0)
>  
>  /* RTC Control Register */
>  #define BCD_EN_SHIFT		0
> @@ -62,12 +62,12 @@ enum sec_rtc_reg {
>  #define MODEL24_SHIFT		1
>  #define MODEL24_MASK		(1 << MODEL24_SHIFT)
>  /* RTC Update Register1 */
> -#define RTC_UDR_SHIFT		0
> -#define RTC_UDR_MASK		(1 << RTC_UDR_SHIFT)
> +#define S5M_RTC_UDR_SHIFT	0
> +#define S5M_RTC_UDR_MASK	(1 << S5M_RTC_UDR_SHIFT)
>  #define RTC_TCON_SHIFT		1
>  #define RTC_TCON_MASK		(1 << RTC_TCON_SHIFT)
> -#define RTC_TIME_EN_SHIFT	3
> -#define RTC_TIME_EN_MASK	(1 << RTC_TIME_EN_SHIFT)
> +#define S5M_RTC_TIME_EN_SHIFT	3
> +#define S5M_RTC_TIME_EN_MASK	(1 << S5M_RTC_TIME_EN_SHIFT)
>  
>  /* RTC Hour register */
>  #define HOUR_PM_SHIFT		6

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux SoC Development]     [Linux Rockchip Development]     [Linux USB Development]     [Video for Linux]     [Linux Audio Users]     [Linux SCSI]     [Yosemite News]

  Powered by Linux