On 07/19/2016 06:51 AM, Neil Armstrong wrote:
> diff --git a/drivers/rtc/rtc-pm8xxx.c b/drivers/rtc/rtc-pm8xxx.c
> index 795fcbd..976f5f6 100644
> --- a/drivers/rtc/rtc-pm8xxx.c
> +++ b/drivers/rtc/rtc-pm8xxx.c
> @@ -393,6 +393,16 @@ static int pm8xxx_rtc_enable(struct pm8xxx_rtc *rtc_dd)
> return 0;
> }
>
> +static const struct pm8xxx_rtc_regs pm8018_regs = {
> + .ctrl = 0x11d,
> + .write = 0x11f,
> + .read = 0x123,
> + .alarm_rw = 0x127,
> + .alarm_ctrl = 0x11d,
> + .alarm_ctrl2 = 0x11e,
> + .alarm_en = BIT(1),
> +};
This is the same as pm8921_regs, so why do we need to duplicate it?
Can't we point the new compatible at the same data?
--
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project
--
You received this message because you are subscribed to "rtc-linux".
Membership options at http://groups.google.com/group/rtc-linux .
Please read http://groups.google.com/group/rtc-linux/web/checklist
before submitting a driver.
---
You received this message because you are subscribed to the Google Groups
"rtc-linux" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
