Messages in this thread | | | Date | Thu, 7 Feb 2013 13:44:01 -0800 | From | Andrew Morton <> | Subject | Re: [PATCH v2] rtc: max8997: Add driver for max8997 rtc. |
| |
On Thu, 07 Feb 2013 20:01:27 +0900 Jonghwa Lee <jonghwa3.lee@samsung.com> wrote:
> This patch adds rtc driver for Maxim 8997 multifunction chip. > Max8997 has rtc module in it. and it can be used for timekeeping > clock and system alarm. It provide various operational mode those are > BCD/binary, 24/12hour, am/pm. Driver sets binary/24/ for default. > Maxim 8997 also supports SMPL(Sudden Momentary Power Loss), WTSR > (Watchdog Timeout and Software Reset). > > Signed-off-by: Jonghwa Lee <jonghwa3.lee@samsung.com> > --- > v2 > - Add module parameter to enable/disable WTSR, SMPL fuction. > - Add unregister_rtc_device() for accidental exit in probing. > > ... > > +/* Module parameter for WTSR function control */ > +static int wtsr_en = 1; > +module_param(wtsr_en, int, 0444); > +MODULE_PARM_DESC(wtsr_en, "Wachdog Timeout & Sofware Reset (default=on)"); > +/* Module parameter for SMPL function control */ > +static int smpl_en = 1; > +module_param(smpl_en, int, 0444); > +MODULE_PARM_DESC(smpl_en, "Sudden Momentary Power Loss (default=on)");
"default=on" is interesting. For what reasons would a user ever want to set this "off"? Can we just remove the options and leave these things enabled all the time for everyone?
| |